Add back confirmation for reading active pr tool (#7504)

alexr00 · web-flow · commit fea9a137708d · 2025-08-04T09:54:28.000-07:00
diff --git a/src/lm/tools/activePullRequestTool.ts b/src/lm/tools/activePullRequestTool.ts
@@ -36,6 +36,7 @@ export class ActivePullRequestTool implements vscode.LanguageModelTool<FetchIssu
 		return {
 			pastTenseMessage: pullRequest ? vscode.l10n.t('Read pull request "{0}"', pullRequest.title) : vscode.l10n.t('No active pull request'),
 			invocationMessage: pullRequest ? vscode.l10n.t('Reading pull request "{0}"', pullRequest.title) : vscode.l10n.t('Reading active pull request'),
+			confirmationMessages: { title: vscode.l10n.t('Active Pull Request'), message: pullRequest ? vscode.l10n.t('Allow reading the details of "{0}"?', pullRequest.title) : vscode.l10n.t('Allow reading the details of the active pull request?') },
 		};
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,7 @@ export class ActivePullRequestTool implements vscode.LanguageModelTool<FetchIssu`
`36`	`36`	`return {`
`37`	`37`	`pastTenseMessage: pullRequest ? vscode.l10n.t('Read pull request "{0}"', pullRequest.title) : vscode.l10n.t('No active pull request'),`
`38`	`38`	`invocationMessage: pullRequest ? vscode.l10n.t('Reading pull request "{0}"', pullRequest.title) : vscode.l10n.t('Reading active pull request'),`
	`39`	`+ confirmationMessages: { title: vscode.l10n.t('Active Pull Request'), message: pullRequest ? vscode.l10n.t('Allow reading the details of "{0}"?', pullRequest.title) : vscode.l10n.t('Allow reading the details of the active pull request?') },`
`39`	`40`	`};`
`40`	`41`	`}`
`41`	`42`