Skip to content

guard installing built in extensions in cli#308967

Merged
sandy081 merged 1 commit intomainfrom
sandy081/witty-dragonfly
Apr 10, 2026
Merged

guard installing built in extensions in cli#308967
sandy081 merged 1 commit intomainfrom
sandy081/witty-dragonfly

Conversation

@sandy081
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 10, 2026 08:15
@sandy081 sandy081 enabled auto-merge (squash) April 10, 2026 08:15
@sandy081 sandy081 self-assigned this Apr 10, 2026
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

This PR adds a CLI-side guard to prevent installing/updating certain built-in extensions when the current product quality disallows those updates (as indicated by builtInExtensionsEnabledWithAutoUpdates combined with forceAutoUpdate).

Changes:

  • Block gallery-based installs/updates when the target is already installed as a built-in extension that is not permitted to update in the current product quality.
  • Apply the same restriction to VSIX installs by validating against the already-installed extension list.
  • Introduce a shared helper to compute/log the “blocked” message.
Show a summary per file
File Description
src/vs/platform/extensionManagement/common/extensionManagementCLI.ts Adds validations to stop CLI installs/updates of restricted built-in extensions (gallery + VSIX paths).

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 2

@sandy081 sandy081 merged commit a6e4912 into main Apr 10, 2026
30 checks passed
@sandy081 sandy081 deleted the sandy081/witty-dragonfly branch April 10, 2026 09:33
@vs-code-engineering vs-code-engineering bot added this to the 1.116.0 milestone Apr 10, 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.

3 participants