Skip to content

handle edge cases while updating built in extensions#308991

Merged
sandy081 merged 3 commits intomainfrom
sandy081/rising-muskox
Apr 10, 2026
Merged

handle edge cases while updating built in extensions#308991
sandy081 merged 3 commits intomainfrom
sandy081/rising-muskox

Conversation

@sandy081
Copy link
Copy Markdown
Member

No description provided.

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

Note

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

Adds safeguards to prevent unsupported updates/downgrades of built-in (system) extensions during install/update flows, using product quality and built-in extension metadata.

Changes:

  • Injects IProductService into InstallExtensionInProfileTask to access product quality.
  • Scans system extensions during installation to detect built-in extensions and block disallowed updates.
  • Prevents downgrading below the built-in extension’s version.
Show a summary per file
File Description
src/vs/platform/extensionManagement/node/extensionManagementService.ts Adds product-aware validation that blocks certain built-in extension updates/downgrades.

Copilot's findings

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

…rvice.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
chrmarti
chrmarti previously approved these changes Apr 10, 2026
deepak1556
deepak1556 previously approved these changes Apr 10, 2026
@sandy081 sandy081 dismissed stale reviews from deepak1556 and chrmarti via 519f156 April 10, 2026 16:23
@sandy081 sandy081 merged commit 88e21a0 into main Apr 10, 2026
26 checks passed
@sandy081 sandy081 deleted the sandy081/rising-muskox branch April 10, 2026 17:02
@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.

6 participants