Align pipeline npm registry with .npmrc feed#475
Merged
Conversation
edvilme
force-pushed
the
fix/align-npm-registry-feed
branch
from
c5661bf to
7b19c42
Compare
Update AZURE_ARTIFACTS_FEED to use the azure-public/python-environments feed to match the .npmrc configuration, fixing E401 auth errors during vsce package in CI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
edvilme
force-pushed
the
fix/align-npm-registry-feed
branch
from
7b19c42 to
f84523b
Compare
Keep .npmrc pointing to the public python-environments feed for local dev. Only the pipeline YAML files should use msft_consumption. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Align .npmrc with pipeline YAML files to use the same msft_consumption feed for @VScode scoped packages. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The msft_consumption feed requires auth in CI pipelines, causing E401 during npm ci. Keep .npmrc on the public python-environments feed for both local dev and CI compatibility. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Switch @VScode scoped registry to msft_consumption feed and explicitly disable auth for it, preventing E401 in CI where npmAuthenticate sets always-auth=true for a different feed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Explicitly disable auth for the msft_consumption scoped registry to prevent CI's npmAuthenticate from sending an invalid token to this public feed, fixing E401 during vsce package. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Revert AZURE_ARTIFACTS_FEED to Pylance_PublicPackages (same org as pipeline, auth works). Keep .npmrc on public python-environments feed but add per-registry always-auth=false to prevent CI's npmAuthenticate from sending the wrong auth token during vsce package. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Update .npmrc to use msft_consumption feed for @VScode scoped packages and remove trigger/PR sections from azure-pipeline.validation.yml. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace @VScode:registry in .npmrc with a direct tarball URL in package.json for @vscode/python-environments from the msft_consumption feed. This avoids scoped registry conflicts with other @VScode packages in CI. Also update validation pipeline triggers to wildcard branches with drafts enabled. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use MicroBuild.1ES.Unofficial.yml for PR/validation builds and add sourceAnalysisPool, matching the isort repo configuration. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
edvilme
commented
Mar 17, 2026
|
|
||
| extends: | ||
| template: azure-pipelines/MicroBuild.1ES.Official.yml@MicroBuildTemplate | ||
| template: azure-pipelines/MicroBuild.1ES.Unofficial.yml@MicroBuildTemplate |
Contributor
Author
There was a problem hiding this comment.
Switched from MicroBuild.1ES.Official.yml to MicroBuild.1ES.Unofficial.yml and added sourceAnalysisPool to match the isort repo. The Official template was failing with a 1ES PT pool/os mismatch error. Unofficial is the correct template for PR/validation builds.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add per-registry
always-auth=falsefor thepython-environmentsfeed in.npmrc.In CI, the pipeline template sets
always-auth=trueglobally and writes an auth token for thePylance_PublicPackagesfeed. Duringvsce package, npm tries to reach the@vscode:registry(python-environmentsfeed) and sends the wrong auth token, causing E401. Thealways-auth=falseoverride for that specific registry URL prevents this.Pipeline
AZURE_ARTIFACTS_FEEDremainsPylance_PublicPackages(unchanged from main).