Skip to content

allow user to configure default permission mode#309966

Merged
justschen merged 4 commits intomainfrom
justin/gogoat
Apr 14, 2026
Merged

allow user to configure default permission mode#309966
justschen merged 4 commits intomainfrom
justin/gogoat

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

fix #309562

Copilot AI review requested due to automatic review settings April 14, 2026 20:37
@justschen justschen enabled auto-merge (squash) April 14, 2026 20:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new chat setting to let users choose the default tool-permission mode for new chat sessions, while respecting enterprise policy that can disable auto-approval.

Changes:

  • Introduces chat.permissions.default configuration and surfaces it in the configuration registry.
  • Initializes and resets the chat permission picker based on the new setting for new/empty sessions.
  • Adds validation + policy clamping to ensure invalid values or policy-disabled modes fall back to “Default Approvals”.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/common/constants.ts Adds the new config key and a runtime type-guard for validating permission-level strings.
src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts Reads the new setting to initialize/reset permission level and clamps it when policy disables auto-approval.
src/vs/workbench/contrib/chat/browser/chat.contribution.ts Registers the new user-facing setting with enum options, labels, and descriptions.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 6

Comment thread src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts Outdated
Comment thread src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts
Comment thread src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts Outdated
Comment thread src/vs/workbench/contrib/chat/browser/widget/input/chatInputPart.ts
Comment thread src/vs/workbench/contrib/chat/browser/chat.contribution.ts Outdated
Comment thread src/vs/workbench/contrib/chat/common/constants.ts Outdated
@github-actions
Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: 72ccd935 Current: cf15db72

Changed (6)

chat/aiCustomizations/aiCustomizationListWidget/InstructionsTabWithItems/Dark
Before After
before after
chat/aiCustomizations/aiCustomizationListWidget/InstructionsTabWithItems/Light
Before After
before after
chat/aiCustomizations/aiCustomizationManagementEditor/HooksTab/Dark
Before After
before after
chat/aiCustomizations/aiCustomizationManagementEditor/HooksTab/Light
Before After
before after
chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Dark
Before After
before after
editor/inlineCompletions/other/JumpToHint/Dark
Before After
before after

@justschen justschen merged commit e7b1efb into main Apr 14, 2026
26 checks passed
@justschen justschen deleted the justin/gogoat branch April 14, 2026 21:25
@vs-code-engineering vs-code-engineering Bot added this to the 1.117.0 milestone Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Persist Autopilot mode across sessions

3 participants