-
Notifications
You must be signed in to change notification settings - Fork 35.7k
Closed
Closed
Copy link
Labels
chat-terminalThe run in terminal tool in chatThe run in terminal tool in chatfeature-requestRequest for new features or functionalityRequest for new features or functionalityinsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code Insiderson-release-notesIssue/pull request mentioned in release notesIssue/pull request mentioned in release noteson-testplan
Milestone
Description
Forking off from #252496
Blocked by #252650
We should have a dropdown on the Continue button here:
What this contains needs some thinking.
Some exploration from #252496 (comment)
I'm also thinking how this is similar to workspace trust, where you may want to trust that particular folder, or it's parent:
I wonder if we would help user configure quickly. Say you run
npm run build, the "Continue" button dropdown could present options to easily configure it for the future:
- Auto-approve commands starting with
npm- Auto-approve commands starting with
npm run- Auto-approve commands matching
npm run buildMaybe that's overkill, we could just show the first and the last one instead.
Similar feature for extension/mcp tools:
Lines 88 to 104 in 01fbd91
| const buttons: IChatConfirmationButton[] = [ | |
| { | |
| label: continueLabel, | |
| data: ConfirmationOutcome.Allow, | |
| tooltip: continueTooltip, | |
| moreActions: !allowAutoConfirm ? undefined : [ | |
| { label: localize('allowSession', 'Allow in this Session'), data: ConfirmationOutcome.AllowSession, tooltip: localize('allowSesssionTooltip', 'Allow this tool to run in this session without confirmation.') }, | |
| { label: localize('allowWorkspace', 'Allow in this Workspace'), data: ConfirmationOutcome.AllowWorkspace, tooltip: localize('allowWorkspaceTooltip', 'Allow this tool to run in this workspace without confirmation.') }, | |
| { label: localize('allowGlobally', 'Always Allow'), data: ConfirmationOutcome.AllowGlobally, tooltip: localize('allowGloballTooltip', 'Always allow this tool to run without confirmation.') }, | |
| ], | |
| }, | |
| { | |
| label: localize('cancel', "Cancel"), | |
| data: ConfirmationOutcome.Disallow, | |
| isSecondary: true, | |
| tooltip: cancelTooltip | |
| }]; |
davidfstr and Ali-ELMASCopilot
Metadata
Metadata
Labels
chat-terminalThe run in terminal tool in chatThe run in terminal tool in chatfeature-requestRequest for new features or functionalityRequest for new features or functionalityinsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code Insiderson-release-notesIssue/pull request mentioned in release notesIssue/pull request mentioned in release noteson-testplan


