Merge pull request microsoft#257881 from microsoft/osortega/support-icon-data-uri

Support data URI for chat sessions

Author: osortega

src/vs/workbench/api/browser/mainThreadChatSessions.ts

@@ -257,6 +257,11 @@ export class MainThreadChatSessions extends Disposable implements MainThreadChat
 			return iconPath; // ThemeIcon doesn't need conversion
 		}
 
+		// handle single URI
+		if (typeof iconPath === 'object' && 'scheme' in iconPath) {
+			return URI.revive(iconPath);
+		}
+
 		// Handle light/dark theme icons
 		if (typeof iconPath === 'object' && ('light' in iconPath && 'dark' in iconPath)) {
 			return {

src/vs/workbench/contrib/chat/browser/chatSessions.ts

@@ -611,10 +611,16 @@ class SessionsRenderer extends Disposable implements ITreeRenderer<IChatSessionI
 		// Handle different icon types
 		let iconResource: URI | undefined;
 		let iconTheme: ThemeIcon | undefined;
+		let iconUri: URI | undefined;
 
 		if (session.iconPath) {
 			if (session.iconPath instanceof URI) {
-				iconResource = session.iconPath;
+				// Check if it's a data URI - if so, use it as icon option instead of resource
+				if (session.iconPath.scheme === 'data') {
+					iconUri = session.iconPath;
+				} else {
+					iconResource = session.iconPath;
+				}
 			} else if (ThemeIcon.isThemeIcon(session.iconPath)) {
 				iconTheme = session.iconPath;
 			} else {
@@ -634,7 +640,7 @@ class SessionsRenderer extends Disposable implements ITreeRenderer<IChatSessionI
 			resource: iconResource
 		}, {
 			fileKind: undefined,
-			icon: iconTheme
+			icon: iconTheme || iconUri
 		});
 	}
 

src/vs/workbench/contrib/chat/browser/media/chatSessions.css

@@ -15,3 +15,9 @@
 .composite.viewlet[id="workbench.view.chat.sessions"] .pane-header.expanded.not-collapsible {
 	background-color: var(--vscode-sideBarSectionHeader-background) !important;
 }
+
+/* Add padding to chat session item icons */
+.chat-sessions-tree .chat-session-item .monaco-icon-label-iconpath {
+	margin-right: 4px;
+	margin-left: 4px;
+}