- Notifications
You must be signed in to change notification settings - Fork33.8k
Open
Assignees
Labels
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 in01fbd91
| constbuttons: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 | |
| }]; |