Skip to content

Omit registry URLs from npm lockfiles - #1530

Merged
Jackson Weber (JacksonWeber) merged 1 commit into
microsoft:mainfrom
JamieMagee:chore/omit-lockfile-registry-resolved
Aug 25, 2026
Merged

Omit registry URLs from npm lockfiles#1530
Jackson Weber (JacksonWeber) merged 1 commit into
microsoft:mainfrom
JamieMagee:chore/omit-lockfile-registry-resolved

Conversation

@JamieMagee

Copy link
Copy Markdown
Member

Summary

The root and performance test lockfiles recorded registry URLs in resolved fields, including Azure Artifacts feed URLs. This change adds omit-lockfile-registry-resolved=true to both project-level .npmrc files and regenerates both lockfiles with npm 11.19.0.

The regeneration removes 951 registry resolved fields. Dependency versions, integrity hashes, lockfileVersion 3, and os, cpu, and libc metadata are unchanged.

Validation

  • npm config get omit-lockfile-registry-resolved returns true in both projects.
  • Azure Artifacts feed URLs are absent from both lockfiles.
  • Repeated npm install --package-lock-only --ignore-scripts --no-audit --no-fund runs produce identical SHA-256 hashes.
  • npm ci --no-audit --no-fund succeeds in both projects with Node.js 22.23.2 and npm 11.19.0.
  • The root project and performance test harness both build successfully.
  • The agent and shared utility unit test subset passes all 106 tests.
  • The full npm test command still has console timing and TelemetryClient assertion failures. The unchanged base commit, 4dc1c066747690daf7b27d280c7da60cb26bb609, fails in the same test areas under the same toolchain.

Add project-level npm settings for the root and performance test projects, then regenerate both lockfiles without changing dependency versions or integrity data.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@JacksonWeber
Jackson Weber (JacksonWeber) merged commit dadabd7 into microsoft:main Aug 25, 2026
13 checks passed
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.

2 participants