Switch to use native sessions view

Removes the webview based session view in favor of the native session view

Author: mjbvz

package.json

@@ -57,7 +57,6 @@
 		"onFileSystem:githubpr",
 		"onFileSystem:githubcommit",
 		"onFileSystem:review",
-		"onWebviewPanel:pr.codingAgentSessionLogView",
 		"onWebviewPanel:IssueOverview",
 		"onWebviewPanel:PullRequestOverview"
 	],
@@ -3463,10 +3462,6 @@
 					"command": "pr.checkoutOnVscodeDevFromDescription",
 					"group": "checkout@1",
 					"when": "webviewId == PullRequestOverview && github:checkoutMenu"
-				},
-				{
-					"command": "pr.openSessionLogFromDescription",
-					"when": "webviewId == PullRequestOverview && github:codingAgentMenu"
 				}
 			],
 			"chat/chatSessions": [
@@ -4246,7 +4241,6 @@
 		"mocha": "^9.0.1",
 		"mocha-junit-reporter": "1.23.0",
 		"mocha-multi-reporters": "1.1.7",
-		"monaco-editor-webpack-plugin": "^7.1.0",
 		"os-browserify": "^0.3.0",
 		"p-all": "^1.0.0",
 		"path-browserify": "1.0.1",
@@ -4287,10 +4281,8 @@
 		"lru-cache": "6.0.0",
 		"markdown-it": "^14.1.0",
 		"marked": "^4.0.10",
-		"monaco-editor": "^0.52.2",
 		"react": "^16.12.0",
 		"react-dom": "^16.12.0",
-		"shiki": "^3.7.0",
 		"ssh-config": "4.1.1",
 		"stream-http": "^3.2.0",
 		"temporal-polyfill": "^0.3.0",
@@ -4304,4 +4296,4 @@
 		"string_decoder": "^1.3.0"
 	},
 	"license": "MIT"
-}
+}

src/@types/vscode.proposed.chatParticipantAdditions.d.ts

@@ -600,6 +600,11 @@ declare module 'vscode' {
 		 * TODO Needed for now to drive the variableName-type reference, but probably both of these should go away in the future.
 		 */
 		readonly name: string;
+
+		/**
+		 * The list of tools were referenced in the value of the reference
+		 */
+		readonly toolReferences?: readonly ChatLanguageModelToolReference[];
 	}
 
 	export interface ChatResultFeedback {
@@ -642,5 +647,6 @@ declare module 'vscode' {
 
 	export interface ChatRequest {
 		modeInstructions?: string;
+		modeInstructionsToolReferences?: readonly ChatLanguageModelToolReference[];
 	}
 }

src/commands.ts

@@ -33,13 +33,12 @@ import { PullRequestOverviewPanel } from './github/pullRequestOverview';
 import { chooseItem } from './github/quickPicks';
 import { RepositoriesManager } from './github/repositoriesManager';
 import { getIssuesUrl, getPullsUrl, isInCodespaces, ISSUE_OR_URL_EXPRESSION, parseIssueExpressionOutput, vscodeDevPrLink } from './github/utils';
-import { CodingAgentContext, OverviewContext } from './github/views';
+import { OverviewContext } from './github/views';
 import { isNotificationTreeItem, NotificationTreeItem } from './notifications/notificationItem';
 import { PullRequestsTreeDataProvider } from './view/prsTreeDataProvider';
 import { ReviewCommentController } from './view/reviewCommentController';
 import { ReviewManager } from './view/reviewManager';
 import { ReviewsManager } from './view/reviewsManager';
-import { SessionLogViewManager } from './view/sessionLogView';
 import { CategoryTreeNode } from './view/treeNodes/categoryNode';
 import { CommitNode } from './view/treeNodes/commitNode';
 import {
@@ -863,18 +862,6 @@ export function registerCommands(
 		return vscode.env.openExternal(vscode.Uri.parse(vscodeDevPrLink(resolved.pr)));
 	}));
 
-	context.subscriptions.push(vscode.commands.registerCommand('pr.openSessionLogFromDescription', async (context: CodingAgentContext | undefined) => {
-		if (!context) {
-			return vscode.window.showErrorMessage(vscode.l10n.t('No pull request context provided for checkout.'));
-		}
-		const resolved = await resolvePr({ ...context, number: context.pullNumber });
-		if (!resolved) {
-			return vscode.window.showErrorMessage(vscode.l10n.t('Unable to resolve pull request for checkout.'));
-		}
-		return SessionLogViewManager.instance?.openForPull(resolved.pr, context, context.openToTheSide);
-
-	}));
-
 	context.subscriptions.push(
 		vscode.commands.registerCommand('pr.exit', async (pr: PRNode | RepositoryChangesNode | PullRequestModel | undefined) => {
 			let pullRequestModel: PullRequestModel | undefined;

src/extension.ts

@@ -49,7 +49,6 @@ import { PRNotificationDecorationProvider } from './view/prNotificationDecoratio
 import { PullRequestsTreeDataProvider } from './view/prsTreeDataProvider';
 import { ReviewManager, ShowPullRequest } from './view/reviewManager';
 import { ReviewsManager } from './view/reviewsManager';
-import { SessionLogViewManager } from './view/sessionLogView';
 import { TreeDecorationProviders } from './view/treeDecorationProviders';
 import { WebviewViewCoordinator } from './view/webviewViewCoordinator';
 
@@ -244,9 +243,6 @@ async function init(
 
 	context.subscriptions.push(new GitLensIntegration());
 
-	const sessionLogViewManager = new SessionLogViewManager(credentialStore, context, reposManager, telemetry, copilotRemoteAgentManager);
-	context.subscriptions.push(sessionLogViewManager);
-
 	context.subscriptions.push(new OverviewRestorer(reposManager, telemetry, context.extensionUri, credentialStore));
 
 	await vscode.commands.executeCommand('setContext', 'github:initialized', true);

src/github/copilotRemoteAgent.ts

@@ -45,6 +45,27 @@ const body_suffix = vscode.l10n.t('Created from VS Code via the [GitHub Pull Req
 
 const PREFERRED_GITHUB_CODING_AGENT_REMOTE_WORKSPACE_KEY = 'PREFERRED_GITHUB_CODING_AGENT_REMOTE';
 
+
+export namespace SessionIdForPr {
+
+	const prefix = 'pull-session-by-index';
+
+	export function getId(prNumber: number, sessionIndex: number): string {
+		return `${prefix}-${prNumber}-${sessionIndex}`;
+	}
+
+	export function parse(id: string): { prNumber: number; sessionIndex: number } | undefined {
+		const match = id.match(new RegExp(`^${prefix}-(\\d+)-(\\d+)$`));
+		if (match) {
+			return {
+				prNumber: parseInt(match[1], 10),
+				sessionIndex: parseInt(match[2], 10)
+			};
+		}
+		return undefined;
+	}
+}
+
 export class CopilotRemoteAgentManager extends Disposable {
 	public static ID = 'CopilotRemoteAgentManager';
 
@@ -1024,10 +1045,21 @@ export class CopilotRemoteAgentManager extends Disposable {
 				return await this.newSessionFlowFromPrompt(id);
 			}
 
-			const pullRequestNumber = parseInt(id);
-			if (isNaN(pullRequestNumber)) {
-				Logger.error(`Invalid pull request number: ${id}`, CopilotRemoteAgentManager.ID);
-				return this.createEmptySession();
+			let pullRequestNumber: number | undefined;
+			let sessionIndex: number | undefined;
+
+			const indexedSessionId = SessionIdForPr.parse(id);
+			if (indexedSessionId) {
+				pullRequestNumber = indexedSessionId.prNumber;
+				sessionIndex = indexedSessionId.sessionIndex;
+			}
+
+			if (typeof pullRequestNumber === 'undefined') {
+				pullRequestNumber = parseInt(id);
+				if (isNaN(pullRequestNumber)) {
+					Logger.error(`Invalid pull request number: ${id}`, CopilotRemoteAgentManager.ID);
+					return this.createEmptySession();
+				}
 			}
 
 			const pullRequest = await this.findPullRequestById(pullRequestNumber, true);
@@ -1039,7 +1071,7 @@ export class CopilotRemoteAgentManager extends Disposable {
 			// Parallelize independent operations
 			const timelineEvents = pullRequest.getTimelineEvents();
 			const changeModels = this.getChangeModels(pullRequest);
-			const sessions = await capi.getAllSessions(pullRequest.id);
+			let sessions = await capi.getAllSessions(pullRequest.id);
 
 			if (!sessions || sessions.length === 0) {
 				Logger.warn(`No sessions found for pull request ${pullRequestNumber}`, CopilotRemoteAgentManager.ID);
@@ -1051,6 +1083,16 @@ export class CopilotRemoteAgentManager extends Disposable {
 				return this.createEmptySession();
 			}
 
+			if (typeof sessionIndex === 'number') {
+				const target = sessions.at(sessionIndex);
+				if (!target) {
+					Logger.error(`Session not found: ${sessionIndex}`, CopilotRemoteAgentManager.ID);
+					return this.createEmptySession();
+				}
+
+				sessions = [target];
+			}
+
 			// Create content builder with pre-fetched change models
 			const contentBuilder = new ChatSessionContentBuilder(CopilotRemoteAgentManager.ID, COPILOT, changeModels);
 

src/github/pullRequestOverview.ts

@@ -17,7 +17,6 @@ import { ITelemetry } from '../common/telemetry';
 import { EventType, ReviewEvent, SessionLinkInfo, TimelineEvent } from '../common/timelineEvent';
 import { asPromise, formatError } from '../common/utils';
 import { IRequestMessage, PULL_REQUEST_OVERVIEW_VIEW_TYPE } from '../common/webview';
-import { SessionLogViewManager } from '../view/sessionLogView';
 import { getCopilotApi } from './copilotApi';
 import { FolderRepositoryManager } from './folderRepositoryManager';
 import {
@@ -38,6 +37,7 @@ import { PullRequestView } from './pullRequestOverviewCommon';
 import { pickEmail, reviewersQuickPick } from './quickPicks';
 import { parseReviewers } from './utils';
 import { CancelCodingAgentReply, MergeArguments, MergeResult, PullRequest, ReviewType, SubmitReviewReply } from './views';
+import { SessionIdForPr } from './copilotRemoteAgent';
 
 export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestModel> {
 	public static override ID: string = 'PullRequestOverviewPanel';
@@ -508,7 +508,7 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 
 	private async openSessionLog(message: IRequestMessage<{ link: SessionLinkInfo }>): Promise<void> {
 		try {
-			await SessionLogViewManager.instance?.openForPull(this._item, message.args.link, message.args.link.openToTheSide ?? false);
+			return vscode.window.showChatSession(COPILOT_SWE_AGENT, SessionIdForPr.getId(this._item.number, message.args.link.sessionIndex), {});
 		} catch (e) {
 			Logger.error(`Open session log view failed: ${formatError(e)}`, PullRequestOverviewPanel.ID);
 		}