fix(deps): add missing oxc-parser subtree so npm ci works on Node 22.22 - #271
fix(deps): add missing oxc-parser subtree so npm ci works on Node 22.22#271escooterclinic wants to merge 1 commit into
Conversation
The committed lockfile omits the oxc-parser@0.144.0 subtree required by the six nested copies of unctx, which declare `oxc-parser >=0.140.0` as a peer. The only oxc-parser recorded is 0.128.0 (satisfying @nuxtjs/i18n's ^0.128.0). npm 11 resolves that without complaint. npm 10.9.8 — the npm bundled with the `node:22.22-alpine` image the Dockerfile pins, and with the `node-version: 22.22` the CI workflows pin — refuses it with EUSAGE, so `npm ci` fails and the Docker image build has been red on main since 63a2968. Regenerated with npm 10.9.8 (`npm install --package-lock-only`). The change is purely additive: 126 packages added, 0 versions changed, 0 removed, and `npm audit` still reports 0 vulnerabilities. `npm ci` now succeeds under both npm 10.9.8 and npm 11.11.1. Signed-off-by: Patryk Radek <patryk@escooterclinic.co.uk>
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Verified in the actual image the Dockerfile pins, rather than only against a local npm — same checkout, same host, only the commit changing: So the failure is the image's bundled npm 10.9.8 rejecting the committed lockfile, and this branch is enough to get I have no way to run your Actions workflows from a fork without approval, so the two Docker jobs on this PR are unrun — the above is the closest equivalent I could get to them. |
Summary
npm cifails onmain, and the Docker image build has been red since63a2968.The committed lockfile omits the
oxc-parser@0.144.0subtree. Six nested copies ofunctx(undernuxt,@nuxt/vite-builder,@nuxt/nitro-server,@nuxt/devtools,@nuxt/devtools-kit,@dxup/nuxt) declareoxc-parser >=0.140.0as a peer; the onlyoxc-parserthe lockfile records is0.128.0, which is there to satisfy@nuxtjs/i18n's^0.128.0.npm 11 resolves that without complaint. npm 10.9.8 refuses it — and npm 10.9.8 is the
npm bundled with the Node this repo pins in four places:
node:22.22-alpinein bothDockerfile stages, and
node-version: 22.22inpr-validation.ymlande2e-tests.yml.That is the exact failure in Publish Docker Image
(run 31464238203) and
Docker Setup Integration Test
(run 31464238289) on
main. Both were green at0122023e(2026-08-09) and have failed since.Why it wasn't caught
pr-validation.ymlande2e-tests.ymltrigger onpull_requestonly, so nothing runsnpm cion a push tomain. The lockfile arrived via #269, a Dependabot PR whose ownBuild, typecheck, and test, Playwright E2E and Simulate new-user setup checks
were all failing (11s, 23s, 21s) when
dependabot-automerge.ymlmerged it.That gap is not addressed here — this PR only repairs the lockfile.
The change
Regenerated with npm 10.9.8:
npm install --package-lock-only. Purely additive.oxc-parser/oxc-transformpeer subtrees)package.jsonnpm auditType of change
Validation
Same tree, same machine, only the npm changing:
npm@10.9.8 cinpm@11.11.1 cinpm auditThe npm 11 path is unchanged, so this cannot regress anyone already building on npm 11.
DCO
Signed-off-by) viagit commit -s