sessions: move session grouping from management service to providers (#307198)

* Move session grouping from management service to providers

Sessions providers now own chat-to-session grouping and return ISession[]
directly. The management service becomes a thin pass-through that tracks
the active session and active chat.

Key changes:
- ISessionsProvider returns ISession[] from getSessions() with capabilities
- CopilotChatSessionsProvider owns SessionsGroupModel and chat grouping
- IChat slimmed to resource, title, status, and reactive properties
- ISession gains mainChat; IActiveSession adds activeChat (management layer)
- Unified sendAndCreateChat replaces separate sendRequest/sendAndCreateChat
- Provider capabilities (multipleChatsPerSession) gate multi-chat UI
- ISessionWorkspace.repositories renamed to folders
- ISessionRepository renamed to ISessionFolder

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix compilation

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: sandy081

src/vs/sessions/browser/parts/sessionCompositeBar.ts

@@ -73,9 +73,9 @@ export class SessionCompositeBar extends Disposable {
 			}
 
 			const chats = activeSession.chats.read(reader);
-			const activeChatId = activeSession.activeChat.map(c => c.chatId).read(reader);
-			const mainChatId = activeSession.mainChat.chatId;
-			this._rebuildTabs(chats, activeChatId, mainChatId);
+			const activeChatUri = activeSession.activeChat.read(reader)?.resource.toString() ?? '';
+			const mainChatUri = activeSession.mainChat.resource.toString();
+			this._rebuildTabs(chats, activeChatUri, mainChatUri);
 		}));
 
 
@@ -89,7 +89,7 @@ export class SessionCompositeBar extends Disposable {
 		reset(this._tabsContainer);
 
 		for (const chat of chats) {
-			this._createTab(chat, chat.chatId === mainChatId);
+			this._createTab(chat, chat.resource.toString() === mainChatId);
 		}
 
 		this._updateActiveTab(activeChatId);
@@ -130,11 +130,19 @@ export class SessionCompositeBar extends Disposable {
 				prompt: localize('renameChat.prompt', "Rename Chat"),
 			});
 			if (newTitle) {
-				await this._sessionsManagementService.renameChat(chat, newTitle);
+				const session = this._sessionsManagementService.activeSession.get();
+				if (session) {
+					await this._sessionsManagementService.renameChat(session, chat.resource, newTitle);
+				}
 			}
 		}));
 
-		const deleteAction = this._tabDisposables.add(new Action('sessionCompositeBar.deleteChat', localize('deleteChat', "Delete"), undefined, !isMainChat, () => this._sessionsManagementService.deleteChat(chat)));
+		const deleteAction = this._tabDisposables.add(new Action('sessionCompositeBar.deleteChat', localize('deleteChat', "Delete"), undefined, !isMainChat, async () => {
+			const session = this._sessionsManagementService.activeSession.get();
+			if (session) {
+				await this._sessionsManagementService.deleteChat(session, chat.resource);
+			}
+		}));
 
 		this._tabDisposables.add(addDisposableListener(tab, EventType.CONTEXT_MENU, (e: MouseEvent) => {
 			e.preventDefault();
@@ -153,12 +161,15 @@ export class SessionCompositeBar extends Disposable {
 	}
 
 	private _onTabClicked(chat: IChat): void {
-		this._sessionsManagementService.openChat(chat.resource);
+		const session = this._sessionsManagementService.activeSession.get();
+		if (session) {
+			this._sessionsManagementService.openChat(session, chat.resource);
+		}
 	}
 
 	private _updateActiveTab(activeChatId: string): void {
 		for (const tab of this._tabs) {
-			const isActive = tab.chat.chatId === activeChatId;
+			const isActive = tab.chat.resource.toString() === activeChatId;
 			tab.element.classList.toggle('active', isActive);
 			tab.element.setAttribute('aria-selected', String(isActive));
 		}

src/vs/sessions/contrib/chat/browser/extensionToolbarPickers.ts

@@ -14,14 +14,14 @@ import { IConfigurationService } from '../../../../platform/configuration/common
 import { IDialogService } from '../../../../platform/dialogs/common/dialogs.js';
 import { ISessionsManagementService } from '../../sessions/browser/sessionsManagementService.js';
 import { ISessionsProvidersService } from '../../sessions/browser/sessionsProvidersService.js';
-import { CopilotCLISession } from '../../copilotChatSessions/browser/copilotChatSessionsProvider.js';
 import { renderIcon } from '../../../../base/browser/ui/iconLabel/iconLabels.js';
 import { ThemeIcon } from '../../../../base/common/themables.js';
 import { ChatConfiguration, ChatPermissionLevel } from '../../../../workbench/contrib/chat/common/constants.js';
 import Severity from '../../../../base/common/severity.js';
 import { MarkdownString } from '../../../../base/common/htmlContent.js';
 import { IOpenerService } from '../../../../platform/opener/common/opener.js';
 import { URI } from '../../../../base/common/uri.js';
+import { CopilotChatSessionsProvider } from '../../copilotChatSessions/browser/copilotChatSessionsProvider.js';
 
 // Track whether warnings have been shown this VS Code session
 const shownWarnings = new Set<ChatPermissionLevel>();
@@ -68,11 +68,10 @@ export class NewChatPermissionPicker extends Disposable {
 		// Write permission level to the active session data when it changes
 		this._register(this.onDidChangeLevel(level => {
 			const session = this.sessionsManagementService.activeSession.get();
-			const providerSession = session ? this.sessionsProvidersService.getUntitledSession(session.providerId) : undefined;
-			if (!(providerSession instanceof CopilotCLISession)) {
-				throw new Error('NewChatPermissionPicker requires a CopilotCLISession');
+			if (!session) {
+				return;
 			}
-			providerSession.setPermissionLevel(level);
+			this.sessionsProvidersService.getProvider<CopilotChatSessionsProvider>(session.providerId)?.getSession(session.sessionId)?.setPermissionLevel(level);
 		}));
 	}
 

src/vs/sessions/contrib/chat/browser/newChatPermissionPicker.ts

@@ -514,7 +514,7 @@ class NewChatWidget extends Disposable implements IHistoryNavigationWidget {
 			return;
 		}
 		const hasText = !!this._editor?.getModel()?.getValue().trim();
-		const session = this.sessionsManagementService.activeSession.get()?.activeChat.get();
+		const session = this.sessionsManagementService.activeSession.get();
 		const hasActiveSession = !!session;
 		const isLoading = session?.loading.get() ?? false;
 		this._sendButton.enabled = !this._sending && hasText && hasActiveSession && !isLoading;
@@ -559,11 +559,11 @@ class NewChatWidget extends Disposable implements IHistoryNavigationWidget {
 		this._updateInputLoadingState();
 
 		try {
-			const chat = this.sessionsManagementService.activeSession.get()?.activeChat.get();
-			if (!chat) {
+			const session = this.sessionsManagementService.activeSession.get();
+			if (!session) {
 				return;
 			}
-			await this.sessionsManagementService.sendRequest(chat, { query, attachedContext });
+			await this.sessionsManagementService.sendAndCreateChat(session, { query, attachedContext });
 			this._contextAttachments.clear();
 			this._editor.getModel()?.setValue('');
 		} catch (e) {

src/vs/sessions/contrib/chat/browser/newChatViewPane.ts

@@ -261,7 +261,7 @@ export class RunScriptContribution extends Disposable implements IWorkbenchContr
 				}
 
 				async run(): Promise<void> {
-					await that._sessionManagementService.sendAndCreateChat({ query: '/generate-run-commands' }, session);
+					await that._sessionManagementService.sendAndCreateChat(session, { query: '/generate-run-commands' });
 				}
 			}));
 		}));
@@ -698,7 +698,7 @@ class RunScriptActionViewItem extends BaseActionViewItem {
 			class: undefined,
 			category: addCategory,
 			run: async () => {
-				await this._sessionsManagementService.sendAndCreateChat({ query: '/generate-run-commands' }, session);
+				await this._sessionsManagementService.sendAndCreateChat(session, { query: '/generate-run-commands' });
 			},
 		});
 

src/vs/sessions/contrib/chat/browser/runScriptAction.ts

@@ -77,8 +77,8 @@ export class SessionTypePicker extends Disposable {
 			return;
 		}
 
-		const chat = this.sessionsManagementService.activeSession.get()?.activeChat.get();
-		if (!chat) {
+		const session = this.sessionsManagementService.activeSession.get();
+		if (!session) {
 			return;
 		}
 
@@ -93,7 +93,7 @@ export class SessionTypePicker extends Disposable {
 		const delegate: IActionListDelegate<ISessionType> = {
 			onSelect: (type) => {
 				this.actionWidgetService.hide();
-				this.sessionsManagementService.setSessionType(chat, type);
+				this.sessionsManagementService.setSessionType(session, type);
 			},
 			onHide: () => { triggerElement.focus(); },
 		};

src/vs/sessions/contrib/chat/browser/sessionTypePicker.ts

@@ -51,17 +51,17 @@ function createMockProvider(id: string, opts?: {
 		onDidChangeSessions: Event.None,
 		getSessions: () => [],
 		createNewSession: () => { throw new Error('Not implemented'); },
-		createNewSessionFrom: () => { throw new Error('Not implemented'); },
 		setSessionType: () => { throw new Error('Not implemented'); },
 		getSessionTypes: () => [],
-		renameSession: async () => { },
+		renameChat: async () => { },
 		setModel: () => { },
 		archiveSession: async () => { },
 		unarchiveSession: async () => { },
 		deleteSession: async () => { },
+		deleteChat: async () => { },
 		setRead: () => { },
-		getUntitledSession: () => undefined,
-		sendRequest: async () => { throw new Error('Not implemented'); },
+		sendAndCreateChat: async () => { throw new Error('Not implemented'); },
+		capabilities: { multipleChatsPerSession: false },
 	};
 }