Repository navigation
Omit registry URLs from the npm lockfile - #781
Merged
Nikola Metulev (nmetulev) merged 1 commit intoAug 25, 2026
Merged
Nikola Metulev (nmetulev) merged 1 commit into
Nikola Metulev (nmetulev) merged 1 commit into
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Removes registry-specific URLs from the npm lockfile to improve portability.
Changes:
- Enables omission of registry-resolved URLs.
- Regenerates the lockfile without changing dependencies or integrity data.
Show a summary per file
| File | Description |
|---|---|
src/winapp-npm/.npmrc |
Enables registry URL omission. |
src/winapp-npm/package-lock.json |
Removes 94 resolved fields. |
Review details
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Files not reviewed (1)
- src/winapp-npm/package-lock.json: Generated file
- Files reviewed: 1/2 changed files
- Comments generated: 0
- Review effort level: Balanced
Chiara Mooney (chiaramooney)
requested a review
from Nikola Metulev (nmetulev)
August 25, 2026 18:17
Nikola Metulev (nmetulev)
approved these changes
Aug 25, 2026
This was referenced Sep 24, 2026
This was referenced Oct 2, 2026
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
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.
Description
Add
omit-lockfile-registry-resolved=truetosrc/winapp-npm/.npmrcand regenerate the lockfile with npm 11.19.0. This removes 94 registryresolvedfields, including the Azure Artifacts proxy URL.Dependency versions, integrity hashes,
lockfileVersion, and platform metadata are unchanged.Usage Example
Not applicable. This change only affects npm lockfile metadata.
Related Issue
None.
Type of Change
Checklist
Screenshots / Demo
Not applicable.
Additional Notes
Validation completed in
src/winapp-npm:npx --yes npm@11.19.0 config get omit-lockfile-registry-resolvedreturnedtrue.npx --yes npm@11.19.0 install --package-lock-only --ignore-scripts --no-audit --no-fund --forcemade no changes.npx --yes npm@11.19.0 ci --ignore-scripts --no-audit --no-fund --forcepassed.--forceis required on Linux because the package declaresos: ["win32"].npx --yes npm@11.19.0 run compilepassed.npx --yes npm@11.19.0 testran 230 tests: 227 passed and 3 failed on Linux because they assert Windows path behavior.The full
scripts/build-cli.ps1build was not run because this environment does not havepwshor Windows.npm config reference: https://docs.npmjs.com/cli/v8/using-npm/config#omit-lockfile-registry-resolved
AI Description
This section is auto-generated by AI when the PR is opened or updated. To opt out, delete this entire section including the marker comments.