Skip to content

fix(ci): Copilot build uses public registry#319982

Merged
rzhao271 merged 2 commits into
mainfrom
rzhao271/wild-firefly
Jun 4, 2026
Merged

fix(ci): Copilot build uses public registry#319982
rzhao271 merged 2 commits into
mainfrom
rzhao271/wild-firefly

Conversation

@rzhao271
Copy link
Copy Markdown
Collaborator

@rzhao271 rzhao271 commented Jun 4, 2026

Verification build: https://dev.azure.com/monacotools/Monaco/_build/results?buildId=444974&view=results

Do not merge until the verification build succeeds.

@rzhao271 rzhao271 requested review from Copilot and dileepyavan June 4, 2026 18:42
@rzhao271 rzhao271 added this to the 1.124.0 milestone Jun 4, 2026
@rzhao271 rzhao271 self-assigned this Jun 4, 2026
@vs-code-engineering
Copy link
Copy Markdown
Contributor

vs-code-engineering Bot commented Jun 4, 2026

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@lszomoru

Matched files:

  • build/azure-pipelines/copilot/setup-steps.yml

@father2mykids

This comment was marked as spam.

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

Updates the Copilot extension Azure Pipelines setup to explicitly configure the NPM registry (and authentication) during CI, aligning Copilot’s build environment with the registry configuration used by other VS Code product pipelines and ensuring cache keys reflect registry-dependent lockfile content.

Changes:

  • Adds a “Setup NPM Registry” step to rewrite package-lock.json registry URLs via build/setup-npm-registry.ts.
  • Adds NPM registry + authentication setup (npm config set registry + npmAuthenticate@0) when the Copilot build cache is not restored.
  • Bumps the Copilot build cache salt to force a fresh cache.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
extensions/copilot/build/.cachesalt Cache-busts the Copilot build cache by updating the salt timestamp.
build/azure-pipelines/copilot/setup-steps.yml Configures NPM registry and authentication in the Copilot pipeline setup steps.

Comment thread build/azure-pipelines/copilot/setup-steps.yml Outdated
Comment thread build/azure-pipelines/copilot/setup-steps.yml
Comment thread extensions/copilot/build/.cachesalt
@rzhao271
Copy link
Copy Markdown
Collaborator Author

rzhao271 commented Jun 4, 2026

@rzhao271 rzhao271 merged commit 38c912e into main Jun 4, 2026
40 checks passed
@rzhao271 rzhao271 deleted the rzhao271/wild-firefly branch June 4, 2026 20:49
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.

5 participants