deps: upgrade npm to 12.0.2 - #64882
Conversation
|
Review requested:
|
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
The outcome is that npm 12 wont be backported. That has been discussed and decided by the release wg |
This comment was marked as resolved.
This comment was marked as resolved.
Since the decision is owned by the release team, and the final consensus of the release team is to ship it as semver major, I think it is pretty much decided. |
Commit Queue failed- Loading data for nodejs/node/pull/64882 ✔ Done loading data for nodejs/node/pull/64882 ----------------------------------- PR info ------------------------------------ Title deps: upgrade npm to 12.0.2 (#64882) ⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile! Branch npm-cli-bot:npm-v12.0.2-main -> nodejs:main Labels semver-major, npm, author ready, needs-ci Commits 1 - deps: upgrade npm to 12.0.2 Committers 1 - npm CLI robot <npm-cli+bot@github.com> PR-URL: https://github.com/nodejs/node/pull/64882 Reviewed-By: Jordan Harband <ljharb@gmail.com> Reviewed-By: Aviv Keller <me@aviv.sh> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Mike McCready <66998419+MikeMcC399@users.noreply.github.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/64882 Reviewed-By: Jordan Harband <ljharb@gmail.com> Reviewed-By: Aviv Keller <me@aviv.sh> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Mike McCready <66998419+MikeMcC399@users.noreply.github.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> -------------------------------------------------------------------------------- ℹ This PR was created on Fri, 31 Jul 2026 20:17:08 GMT ✔ Approvals: 6 ✔ - Jordan Harband (@ljharb): https://github.com/nodejs/node/pull/64882#pullrequestreview-4832185636 ✔ - Aviv Keller (@avivkeller): https://github.com/nodejs/node/pull/64882#pullrequestreview-4832322700 ✔ - Colin Ihrig (@cjihrig): https://github.com/nodejs/node/pull/64882#pullrequestreview-4834841084 ✔ - Mike McCready (@MikeMcC399): https://github.com/nodejs/node/pull/64882#pullrequestreview-4834975352 ✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64882#pullrequestreview-4835142203 ✔ - Trivikram Kamat (@trivikr): https://github.com/nodejs/node/pull/64882#pullrequestreview-4836771561 ✘ semver-major requires at least 2 TSC approvals ✔ Last GitHub CI successful ℹ Last Full PR CI on 2026-08-01T14:25:21Z: https://ci.nodejs.org/job/node-test-pull-request/75367/ - Querying data for job/node-test-pull-request/75367/ ✔ Build data downloaded ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncuhttps://github.com/nodejs/node/actions/runs/30766015471 |
This comment was marked as resolved.
This comment was marked as resolved.
|
In nodejs/Release#1161 there is a statement from the npm team (@JamieMagee):
That sounds like it was supposed to be an acceptance test, so I ran it in my fork https://github.com/MikeMcC399/npm-cli against Node.js Comparing npm 11.19.0 with npm 12.0.2, there were 2 regressions:
which both look like they can be ignored. These are the test failures:
As a result of testing, I submitted 2 PRs to improve the test process: npm/cli#9836 to update the actions used by node-integration workflow to supported versions (Edit: now landed) and |
|
it is absolutely critical that npm forever continue to be able to install non-lowercased package names, both for legacy packages like JSONStream and for custom registries, so if that's revealing a real regression then it's a problem. |
I don't see any evidence that the mixed case of the package name is a problem. The package installs correctly from the npm@12.0.2 CLI. It only fails in
I didn't investigate that further, as One of npm@12's breaking changes is to disable git by default. allow-git defaults to Edit: I did actually investigate further and the git dependency Hopefully somebody from the citgm team can take a look at nodejs/citgm#1114 with a view to removing the obsolete |
This PR contains changes from:
npm@12.0.0npm@12.0.1npm@12.0.212.0.0
12.0.0 (2026-07-08)
npm sbom --sbom-format=cyclonedxnow reports thenamefield from each package'spackage.jsoninstead of the on-disk directory name. Thename,bom-ref, andpurlof the root component and of aliased dependencies may change.man npm-installwill no longer work, butnpm help installis unaffected.npm pkgoutput is no longer forced to json. This means you can get single values without having to worry about wrapping of the values. It also outputs non-json content more similarly tonpm view.npm shrinkwrapis removed, theshrinkwrapconfig alias is removed, andnpm-shrinkwrap.jsonis no longer loaded or honored at the project root or from inside dependency tarballs. Rename project-rootnpm-shrinkwrap.jsontopackage-lock.json; usebundleDependenciesif you need to ship a locked dependency tree.npm packandnpm publishhave changed. They are now always consistent, and in the same format.star,starsandunstarcommands have been removednpm addusercommand has been removed. Create and manage user accounts on the npm website, and usenpm loginto authenticate on the command line.npm inithas been changed from "ISC" to an empty string. If not set, the license field will be omitted from new packages.npmnow supports node^22.22.2 || ^24.15.0 || >=26.0.0Chores
b77b532#9735 remove pre-release mode from npm 12 and workspaces (#9735) (@reggi, @Copilot)Dependencies
@npmcli/arborist@10.0.0@npmcli/config@11.0.0libnpmaccess@11.0.0libnpmdiff@9.0.0libnpmexec@11.0.0libnpmfund@8.0.0libnpmorg@9.0.0libnpmpack@10.0.0libnpmpublish@12.0.0libnpmsearch@10.0.0libnpmteam@9.0.0libnpmversion@9.0.012.0.1
12.0.1 (2026-07-10)
Bug Fixes
ecb02a8#9745 view: avoid wrapping array results (#9745) (@reggi, @martinrrm, @Copilot)47fc8b1#9740 correct bundled sigstore from dev dependency conflict (#9740) (@james-pre)Dependencies
@npmcli/arborist@10.0.1libnpmdiff@9.0.1libnpmexec@11.0.1libnpmfund@8.0.1libnpmpack@10.0.112.0.2
12.0.2 (2026-07-27)
Bug Fixes
834408e#9760 pack: honor min-release-age-exclude (#9760) (@martinrrm, @Copilot)ef6cfea#9786 owner: use scoped registry for user lookup (#9786) (@martinrrm, @Copilot)Documentation
9e17dc0#9805 clarify npm 12 unknown-config breaking change in changelogs (#9805) (@Hashim1999164)Chores
7b1f6c1#9747 parse pack --json object output in node integration (#9747) (@reggi, @Copilot)Dependencies
@npmcli/arborist@10.0.2@npmcli/config@11.0.1libnpmdiff@9.0.2libnpmexec@11.0.2libnpmfund@8.0.2libnpmpack@10.0.2