Repository navigation
refactor: Bump npm from 11.20.0 to 11.21.1 - #3157
Conversation
|
🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review. Tip
Note Please respond to review comments from AI agents just like you would to comments from a human reviewer. Let the reviewer resolve their own comments, unless they have reviewed and accepted your commit, or agreed with your explanation for why the feedback was incorrect. Caution Pull requests must be written using an AI agent with human supervision. Pull requests written entirely by a human will likely be rejected, because of lower code quality, higher review effort and the higher risk of introducing bugs. Please note that AI review comments on this pull request alone do not satisfy this requirement. Our CI and AI review are safeguards, not development tools. If many issues are flagged, rethink your development approach. Invest more effort in planning and design rather than using review cycles to fix low-quality code. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## alpha #3157 +/- ##
=======================================
Coverage 99.98% 99.98%
=======================================
Files 64 64
Lines 6339 6339
Branches 1522 1526 +4
=======================================
Hits 6338 6338
Misses 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Security update for packages bundled inside the npm CLI.
npmis a transitive development dependency (lock-file-only change). This PR upgrades only the copy nested under@semantic-release/npm@13.1.5(node_modules/@semantic-release/npm/node_modules/npm, range^11.6.2) from 11.20.0 to 11.21.1, the newest 11.x release; npmlatestis 12.x, which is outside the range. npm is only used by therelease-automated.ymlworkflow through@semantic-release/npm; it is not part of the SDK build, tests or bundles, and PR CI does not exercise it.npm 11.21.1 bundles patched versions of
brace-expansion(5.0.12),ip-address(10.7.2),postcss-selector-parser(7.1.6) andundici(6.29.0).Security
Resolved by this PR:
brace-expansion>= 4.0.0, < 5.0.12ip-address>= 10.2.0, <= 10.5.0undici< 6.28.1Fixed in the nested npm 11 copy only. These alerts stay open because the second npm copy (see below) still bundles a vulnerable version:
ip-address<= 10.7.0ip-address<= 10.5.0postcss-selector-parser< 7.1.6The second copy,
node_modules/npm@10.9.9, is required by@semantic-release/npm@11.0.3(range^10.5.0), which comes from the nestedsemantic-release@22.0.12of@saithodev/semantic-release-backmerge@4.0.1. 10.9.9 is already the newest 10.x release, so its bundledip-address10.1.0 andpostcss-selector-parser7.1.1 cannot be updated through a regular dependency update. This PR leaves that copy unchanged.Changes
npm dist-tagsupports OIDC authentication (feat(dist-tag): support OIDC authentication npm/cli#10056);npm stagedocumentation update (docs: update npm stage docs to reflect that it can create new packages npm/cli#10050); bundled@npmcli/config10.14.0allowScriptskeys underinstall-strategy=linked(fix: write registry allowScripts keys under install-strategy=linked npm/cli#10059); bundled dependencies updated tobrace-expansion5.0.12,ip-address10.7.2,postcss-selector-parser7.1.6,undici6.29.0 andhttp-cache-semantics4.3.0 (deps: resolve production audit vulnerabilities npm/cli#10089); bundled workspace packages@npmcli/arborist9.9.3,libnpmdiff8.1.14,libnpmexec10.3.4,libnpmfund7.0.28 andlibnpmpack9.1.15Breaking Changes
None
Code Changes Required
None — the upgrade is a drop-in replacement.
Verified against this repository's release workflow:
@semantic-release/npm13.1.5 runsnpm version,npm pack,npm whoami,npm publishandnpm dist-tag add. None of them changed in 11.21.0 or 11.21.1 for token authentication.npm dist-tag(and refactors fornpm publish) only runs when the CI environment provides an OIDC token, and it never throws. Thereleasejob inrelease-automated.ymldoes not grantid-token: writeand authenticates withNPM_TOKEN; the most recent release run that published (8.6.2-alpha.1) logs "Retrieval of GitHub Actions OIDC token failed" followed by "Wrote NPM_TOKEN to .npmrc". Publishing and dist-tag handling are therefore unchanged. Provenance is not configured.enginesfield of npm (^20.17.0 || >=22.9.0) is identical in 11.20.0 and 11.21.1. The release job uses Nodelatest, and the CI matrix (20.19.0, 22.13.0, 24.1.0) is also within the range.The lock file diff is limited to the nested
npmentry (version, resolved URL, integrity and dependency ranges) and its bundled entries, in both thepackagesand the legacydependenciessections of the lockfile. All 143 bundled entries match the contents of the publishednpm-11.21.1.tgz, whose integrity matches the registry.Summary by CodeRabbit