UI Integration tweaks (#7438)

* UI Integration tweaks

* lint

Author: joshspicer

package.json

@@ -524,6 +524,14 @@
 						"experimental"
 					]
 				},
+				"githubPullRequests.codingAgent.promptForConfirmation": {
+					"type": "boolean",
+					"default": true,
+					"markdownDescription": "%githubPullRequests.codingAgent.promptForConfirmation.description%",
+					"tags": [
+						"experimental"
+					]
+				},
 				"githubPullRequests.codingAgent.uiIntegration": {
 					"type": "boolean",
 					"default": false,

package.nls.json

@@ -23,6 +23,7 @@
 	"githubPullRequests.codingAgent.description": "Enables integration with the asynchronous Copilot coding agent. The '#copilotCodingAgent' tool will be available in agent mode when this setting is enabled.",
 	"githubPullRequests.codingAgent.uiIntegration.description": "Enables UI integration within VS Code to create new coding agent sessions.",
 	"githubPullRequests.codingAgent.autoCommitAndPush.description": "Allow automatic git operations (commit, push) to be performed when starting a coding agent session",
+	"githubPullRequests.codingAgent.promptForConfirmation.description": "Prompt for confirmation before initiating a coding agent session from the UI integration.",
 	"githubPullRequests.remotes.markdownDescription": "List of remotes, by name, to fetch pull requests from.",
 	"githubPullRequests.queries.markdownDescription": "Specifies what queries should be used in the GitHub Pull Requests tree. All queries are made against **the currently opened repos**. Each query object has a `label` that will be shown in the tree and a search `query` using [GitHub search syntax](https://help.github.com/en/articles/understanding-the-search-syntax). The following variables can be used: \n - `${user}` will resolve to the currently logged in user \n - `${owner}` will resolve to the owner of the current repository, ex. `microsoft` in `microsoft/vscode` \n - `${repository}` will resolve to the repository name, ex. `vscode` in `microsoft/vscode` \n - `${today-Nd}`, where `N` is the number of days ago, will resolve to a date, ex. `2025-01-04`. \n\n By default these queries define the categories \"Waiting For My Review\", \"Assigned To Me\" and \"Created By Me\". If you want to preserve these, make sure they are still in the array when you modify the setting.",
 	"githubPullRequests.queries.label.description": "The label to display for the query in the Pull Requests tree",

src/common/settingKeys.ts

@@ -89,4 +89,5 @@ export const COLOR_THEME = 'colorTheme';
 
 export const CODING_AGENT = `${PR_SETTINGS_NAMESPACE}.codingAgent`;
 export const CODING_AGENT_ENABLED = 'enabled';
-export const CODING_AGENT_AUTO_COMMIT_AND_PUSH = 'autoCommitAndPush';
+export const CODING_AGENT_AUTO_COMMIT_AND_PUSH = 'autoCommitAndPush';
+export const CODING_AGENT_PROMPT_FOR_CONFIRMATION = 'promptForConfirmation';

src/github/copilotRemoteAgent.ts

@@ -12,7 +12,7 @@ import { commands } from '../common/executeCommands';
 import { Disposable } from '../common/lifecycle';
 import Logger from '../common/logger';
 import { GitHubRemote } from '../common/remote';
-import { CODING_AGENT, CODING_AGENT_AUTO_COMMIT_AND_PUSH, CODING_AGENT_ENABLED } from '../common/settingKeys';
+import { CODING_AGENT, CODING_AGENT_AUTO_COMMIT_AND_PUSH, CODING_AGENT_ENABLED, CODING_AGENT_PROMPT_FOR_CONFIRMATION } from '../common/settingKeys';
 import { ITelemetry } from '../common/telemetry';
 import { CommentEvent, CopilotFinishedEvent, CopilotStartedEvent, EventType, ReviewEvent, TimelineEvent } from '../common/timelineEvent';
 import { DataUri, toOpenPullRequestWebviewUri } from '../common/uri';
@@ -59,6 +59,7 @@ const CONTINUE = vscode.l10n.t('Continue');
 // With Pending Changes
 const PUSH_CHANGES = vscode.l10n.t('Include changes');
 const CONTINUE_WITHOUT_PUSHING = vscode.l10n.t('Ignore changes');
+const CONTINUE_AND_DO_NOT_ASK_AGAIN = vscode.l10n.t('Continue and don\'t ask again');
 const COMMIT_YOUR_CHANGES = vscode.l10n.t('Commit your changes to continue coding agent session. Close integrated terminal to cancel.');
 
 const COPILOT = '@copilot';
@@ -151,6 +152,11 @@ export class CopilotRemoteAgentManager extends Disposable {
 			.getConfiguration(CODING_AGENT).get(CODING_AGENT_ENABLED, false);
 	}
 
+	promptForConfirmation(): boolean {
+		return vscode.workspace
+			.getConfiguration(CODING_AGENT).get(CODING_AGENT_PROMPT_FOR_CONFIRMATION, true);
+	}
+
 	async isAssignable(): Promise<boolean> {
 		const repoInfo = await this.repoInfo();
 		if (!repoInfo) {
@@ -320,7 +326,7 @@ export class CopilotRemoteAgentManager extends Disposable {
 
 		let autoPushAndCommit = false;
 		const message = vscode.l10n.t('Copilot coding agent will continue your work in \'{0}\'.', repoName);
-		const detail = vscode.l10n.t('Your current chat session will end, and its context will be used to continue your work in a new pull request.');
+		const detail = vscode.l10n.t('Your chat context will be used to continue work in a new pull request.');
 		if (source !== 'prompt' && hasChanges && this.autoCommitAndPushEnabled()) {
 			// Pending changes modal
 			const modalResult = await vscode.window.showInformationMessage(
@@ -352,7 +358,7 @@ export class CopilotRemoteAgentManager extends Disposable {
 			if (modalResult === PUSH_CHANGES) {
 				autoPushAndCommit = true;
 			}
-		} else {
+		} else if (this.promptForConfirmation()) {
 			// No pending changes modal
 			const modalResult = await vscode.window.showInformationMessage(
 				source !== 'prompt' ? message : vscode.l10n.t('Copilot coding agent will implement the specification outlined in this prompt file'),
@@ -361,12 +367,17 @@ export class CopilotRemoteAgentManager extends Disposable {
 					detail: source !== 'prompt' ? detail : undefined
 				},
 				CONTINUE,
+				CONTINUE_AND_DO_NOT_ASK_AGAIN,
 				LEARN_MORE,
 			);
 			if (!modalResult) {
 				return;
 			}
 
+			if (modalResult === CONTINUE_AND_DO_NOT_ASK_AGAIN) {
+				await vscode.workspace.getConfiguration(CODING_AGENT).update(CODING_AGENT_PROMPT_FOR_CONFIRMATION, false, vscode.ConfigurationTarget.Global);
+			}
+
 			if (modalResult === LEARN_MORE) {
 				learnMoreCb();
 				return;