Skip to content

chore: omit registry URLs from npm lockfiles - #49

Merged
Jordan Selig (seligj95) merged 1 commit into
Azure-Samples:mainfrom
JamieMagee:fix/omit-lockfile-registry-resolved
Aug 25, 2026
Merged

chore: omit registry URLs from npm lockfiles#49
Jordan Selig (seligj95) merged 1 commit into
Azure-Samples:mainfrom
JamieMagee:fix/omit-lockfile-registry-resolved

Conversation

@JamieMagee

Copy link
Copy Markdown
Contributor

Purpose

  • Add omit-lockfile-registry-resolved=true to the Application Insights sample's .npmrc.
  • Regenerate the lockfile without changing its dependency graph, versions, integrity values, lockfileVersion, or other metadata.
  • Remove registry resolved fields so Azure Artifacts feed URLs are not persisted.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other: lockfile metadata hardening

How to Test

  • Get the code
git clone https://github.com/JamieMagee/app-service-labs.git
cd app-service-labs
git checkout fix/omit-lockfile-registry-resolved
cd samples/monitor-app-insights-node
  • Test the code
npx --yes npm@10.9.8 config get omit-lockfile-registry-resolved
npx --yes npm@10.9.8 install --package-lock-only --ignore-scripts
npx --yes npm@10.9.8 ci --ignore-scripts

What to Check

  • The config command returns true.
  • A second lockfile regeneration makes no changes.
  • ms-feed-2, ms-feed-12, ms-feed-17, and ms-feed-25 are absent from the lockfile.
  • npm ci succeeds. This sample does not define a test script.

Other Information

The lockfile keeps the same dependency graph, versions, integrity values, and lockfileVersion.

Set npm's omit-lockfile-registry-resolved option for the Application Insights sample and regenerate the lockfile without changing dependencies.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@seligj95
Jordan Selig (seligj95) merged commit 80bac1e into Azure-Samples:main Aug 25, 2026
2 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