Skip to content

Run VSIX packaging validation for extension dependency PRs - #3575

Draft
KarishmaGhiya with Copilot wants to merge 2 commits into
mainfrom
copilot/run-vsix-packaging-on-prs
Draft

Run VSIX packaging validation for extension dependency PRs#3575
KarishmaGhiya with Copilot wants to merge 2 commits into
mainfrom
copilot/run-vsix-packaging-on-prs

Conversation

Copilot AI commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

VSIX dependency updates previously bypassed packaging in PR validation, allowing invalid extension dependency graphs to reach release packaging. This adds a credential-free packaging check for changed VS Code extension manifests.

  • PR VSIX pack stage

    • Adds a PR-only VSIXPack stage after build completion.
    • Detects changes to servers/*/vscode/package.json and package-lock.json, including deletions.
  • Credential-free packaging

    • Downloads existing unsigned build artifacts.
    • Consolidates artifacts without signing preparation.
    • Runs Pack-Vsix.ps1, including npm ci --omit=optional and vsce package.
    • Does not authenticate to npm feeds, sign, publish, or upload VSIX artifacts.
  • Artifact preparation

    • Adds -PrepareForVsix to Compress-ForSigning.ps1 so PR packaging preserves macOS binaries and supports multi-server build artifacts.
  • Documentation

    • Notes the conditional VSIX packaging validation in CONTRIBUTING.md.
condition: and(succeeded(), eq(variables['Build.Reason'], 'PullRequest'))

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: KarishmaGhiya <8968058+KarishmaGhiya@users.noreply.github.com>
Copilot AI changed the title [WIP] Run VSIX packaging on dependency-update PRs to catch failures Run VSIX packaging validation for extension dependency PRs Sep 4, 2026
Copilot AI requested a review from KarishmaGhiya September 4, 2026 19:07
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.

Run VSIX packaging on dependency-update PRs to catch quarantine failures before merge

3 participants