Chore: treeview icon dialog refactors

[chloebyun-wd]

Move treeview and dialog icon refactors into their own branch so the sources bubble branch stays focused on citation UI changes.

[gemini-code-assist]

Code Review

This pull request centralizes icon components by moving local SVG definitions into a dedicated icons directory and exporting them. New components like ChevronRightIcon, FileIcon, FolderIcon, GlobeIcon, and XIcon were added and integrated into FeedbackContentDialog, RichTreeView, TreeView, and WorkflowTreeView. Review feedback identifies opportunities to simplify the code by removing redundant JSX fragments and refactoring the Dynamic component usage to pass props directly instead of using anonymous wrapper functions, which is more idiomatic in Solid.js.

[gemini-code-assist]

The fragment <>...</> is redundant here as the ternary operator already returns a single JSX element in both branches. Removing it simplifies the code and adheres better to Solid.js idioms.

Suggested change

	return <>{context.isExpanded(props.itemId) ? <FolderOpenIcon width="18" height="18" /> : <FolderIcon width="18" height="18" />}</>;
	return context.isExpanded(props.itemId) ? <FolderOpenIcon width="18" height="18" /> : <FolderIcon width="18" height="18" />;

[gemini-code-assist]

Instead of creating anonymous wrapper components inside the component prop of Dynamic, you can pass the component directly and provide the props to Dynamic. This is more idiomatic in Solid.js and avoids unnecessary component re-definitions on every state change.

            <Dynamic
              component={context.isExpanded(props.itemId) ? ChevronDownIcon : ChevronRightIcon}
              width="18"
              height="18"
              style={{ transition: 'transform 0.2s' }}
            />

[gemini-code-assist]

Instead of creating anonymous wrapper components inside the component prop of Dynamic, you can pass the component directly and provide the props to Dynamic. This is more idiomatic in Solid.js and avoids unnecessary component re-definitions on every state change.

Suggested change

	<Dynamic
	component={
	context.isExpanded(props.itemId)
	? () => <ChevronDownIcon width="20" height="20" class="transition-transform duration-200" />
	: () => <ChevronRightIcon width="20" height="20" class="transition-transform duration-200" />
	}
	/>
	<Dynamic
	component={context.isExpanded(props.itemId) ? ChevronDownIcon : ChevronRightIcon}
	width="20"
	height="20"
	class="transition-transform duration-200"
	/>