Skip to content

Remove API version concept#321391

Open
alexr00 wants to merge 2 commits into
mainfrom
alexr00/underground-sparrow
Open

Remove API version concept#321391
alexr00 wants to merge 2 commits into
mainfrom
alexr00/underground-sparrow

Conversation

@alexr00

@alexr00 alexr00 commented Jun 15, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings June 15, 2026 10:51
@alexr00 alexr00 self-assigned this Jun 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 removes the “API proposal version” mechanism from VS Code’s proposed-API pipeline, simplifying proposal metadata and eliminating version-based validation logic in extension scanning/installation.

Changes:

  • Removes // version: N markers from proposed vscode.proposed.*.d.ts files and strips version metadata from the generated extensionsApiProposals map.
  • Deletes proposal-version compatibility checks in extension validation/scanning/installation flows and removes the related product configuration plumbing.
  • Updates tooling/docs: adds a linter warning for enabledApiProposals entries containing @<version>, updates Copilot contributing guidance, and removes the CI workflow/instructions dedicated to proposal version checks.
Show a summary per file
File Description
src/vscode-dts/vscode.proposed.mcpServerDefinitions.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.languageModelToolSupportsModel.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.languageModelThinkingPart.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.findFiles2.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.defaultChatParticipant.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.chatSessionsProvider.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.chatProvider.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.chatPromptFiles.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.chatParticipantPrivate.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.chatParticipantAdditions.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.chatHooks.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.chatDebug.d.ts Removes proposal version marker comment.
src/vscode-dts/vscode.proposed.aiTextSearchProvider.d.ts Removes proposal version marker comment.
src/vs/workbench/services/extensionManagement/electron-browser/remoteExtensionManagementService.ts Drops install-time API-proposal-version incompatibility error path.
src/vs/workbench/services/extensionManagement/browser/webExtensionsScannerService.ts Removes conditional API-version validation and always normalizes proposal names (strips @...).
src/vs/workbench/contrib/extensions/browser/extensionsWorkbenchService.ts Removes API-proposal-version-related branch from “incompatible extensions” notification logic.
src/vs/server/node/webClientServer.ts Removes server-side plumbing that injected extensionsEnabledWithApiProposalVersion via CLI args.
src/vs/platform/extensions/test/common/extensionValidator.test.ts Removes unit tests for the deleted areApiProposalsCompatible helper.
src/vs/platform/extensions/common/extensionValidator.ts Removes areApiProposalsCompatible and the validate-manifest API-version enforcement.
src/vs/platform/extensions/common/extensionsApiProposals.ts Removes version entries from the generated proposals registry.
src/vs/platform/extensions/common/extensions.ts Removes parseApiProposals helper; keeps name-only normalization.
src/vs/platform/extensionManagement/common/extensionsScannerService.ts Removes product-gated API-version validation and always normalizes proposal names.
src/vs/platform/extensionManagement/common/extensionGalleryService.ts Removes marketplace filtering based on API proposal versions.
src/vs/platform/extensionManagement/common/abstractExtensionManagementService.ts Drops install-time API-proposal-version incompatibility error path.
src/vs/base/common/product.ts Removes extensionsEnabledWithApiProposalVersion from product configuration schema.
extensions/extension-editing/src/extensionLinter.ts Adds a warning diagnostic for enabledApiProposals values containing @<version>.
extensions/copilot/CONTRIBUTING.md Updates guidance to rely on engines.vscode (date) rather than proposal versions.
build/lib/compilation.ts Updates the proposal-names generator to stop extracting/printing proposal versions and updates the generated type shape.
.github/workflows/api-proposal-version-check.yml Deletes the workflow that enforced/acknowledged proposal version bumps.
.github/instructions/api-version.instructions.md Deletes contributor instructions about proposal version bumping.

Copilot's findings

  • Files reviewed: 17/30 changed files
  • Comments generated: 1

Comment thread src/vs/platform/extensions/common/extensionsApiProposals.ts
@alexr00 alexr00 marked this pull request as ready for review June 15, 2026 13:41
@alexr00 alexr00 enabled auto-merge (squash) June 15, 2026 13:42
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