chore(deps): bump nuxt from 3.21.2 to 3.21.6 in the nuxt-ecosystem group across 1 directory#458
Conversation
|
🚅 Deployed to the euler-lite-pr-458 environment in euler-lite
|
32c9402 to
6a302df
Compare
LeonardEulerXYZ
left a comment
There was a problem hiding this comment.
Reviewed PR #458 at head 6a302df6b240280d1a6e9ea1cbb576690e02abaa.
Verdict: Comment — no blocking issues found.
Scope:
- Dependency bump only:
nuxt3.21.2->3.21.5plus lockfile updates. - Reviewed
package.jsonandpackage-lock.jsondiff, including new/removed transitive packages and install-script surface. - Supply-chain sweep notes: changed packages resolve from
https://registry.npmjs.org/; no changed package addedhasInstallScript; newly added transitives areproper-lockfile, itssignal-exit, andretryvia the Nuxt ecosystem update.
Validation performed locally in /home/leonard/euler-lite-worktrees/pr-458:
- Managed worktree bootstrap with Node 24 / npm 11, Nuxt prepare completed.
npm run test:runpassed: 74 files passed, 1 skipped; 843 tests passed, 1 skipped.npm run buildpassed with existing-style Sentry/source-map, Rollup pure annotation, dynamic-import, and large-chunk warnings; no build failure.- Production-built local server smoke on
http://127.0.0.1:3458:/returned 200./api/euler-chainsreturned 200 with 17 entries./api/token-list?chainId=1returned 200./api/rpc/1returnedeth_chainId = 0x1.- Browser route smoke after “Connect later”:
/,/explore,/lend, and/borrowrendered populated content. Console noise was limited to local missingAPPKIT_PROJECT_ID/ Reown remote-config warnings expected for this minimal local env.
No inline comments: I did not find a concrete changed-line issue worth anchoring. This is a framework dependency bump, so the useful proof is the lockfile/supply-chain sweep plus build/test/runtime smoke rather than source-line nits.
6a302df to
94b48d5
Compare
LeonardEulerXYZ
left a comment
There was a problem hiding this comment.
Reviewed PR #458 at head 94b48d5.
Verdict: COMMENT — no blocking issues found.
Scope checked:
package.json/package-lock.jsondiff for the Nuxt 3.21.2 -> 3.21.6 bump.- Lockfile package/version churn: Nuxt/Nitro/Vite-builder related transitive updates, including oxc 0.117.0 -> 0.131.0 family and related Nuxt runtime packages. No source-code or app config changes in this PR.
- npm audit comparison: base reported the Nuxt island cache-poisoning advisory on Nuxt <=3.21.5; this PR removes that Nuxt advisory. Remaining moderate audit items are pre-existing / not introduced here (
serialize-javascript,unheadoverride,wsvia viem/euler SDK).
Validation performed locally in /home/leonard/euler-lite-worktrees/pr-458:
npm ci --ignore-scriptsvia the managed dependency seed, thennpx nuxt preparenpm run test:run— 803 passed, 1 skippednpm run typecheck— passednpm run build— passed with existing warning classes (Sentry token/sourcemaps, Rollup pure annotations, dynamic/static import chunking, large chunk warning)- GitHub check observed:
euler-lite - dev-buildpassed
Browser smoke: not run — this is a package-only Nuxt patch bump with no app source or route changes; build/typecheck/test coverage is the appropriate bounded check here.
94b48d5 to
6da0908
Compare
LeonardEulerXYZ
left a comment
There was a problem hiding this comment.
Reviewed PR #458 at head 6da0908.
Verdict: Comment — no blocking findings found.
Scope reviewed:
- package-only Dependabot diff: nuxt 3.21.2 -> 3.21.6 in package.json plus package-lock refresh
- lockfile supply-chain shape: registry.npmjs.org tarballs only among touched packages; no touched package lifecycle/install scripts detected
- Nuxt/Nitro/Vite/OXC/Vue transitive changes implied by the lockfile
Validation performed:
- Fresh managed Lite worktree: /home/leonard/euler-lite-worktrees/pr-458
- npm ci --ignore-scripts via lockfile-hashed dependency seed, then npx nuxt prepare
- npm run test:run: 72 files passed, 813 tests passed, 1 skipped
- npm run build: passed with existing-style Sentry/source-map, Rollup annotation/chunk-size warnings
- Local built-server smoke on http://127.0.0.1:3458:
- GET / -> HTTP 200
- GET /api/euler-chains -> HTTP 200, 17 chains parsed
- POST /api/rpc/1 eth_chainId -> HTTP 200, result 0x1
- GET /api/token-list?chainId=1 -> HTTP 200, 6304 tokens parsed
- npm audit --omit=dev compared against base: moderate advisories remain in the tree, but the PR removes the Nuxt island shared-cache poisoning advisory present on 3.21.2; the remaining reported issues are existing/transitive rather than a blocker introduced by this bump.
Caveat:
- Browser-level Playwright smoke was not run because no Chromium/Chrome executable is currently installed in this VM. The production build and local HTTP/API smoke passed.
No inline comments; I did not find a changed-line issue worth anchoring.
6da0908 to
628b43a
Compare
LeonardEulerXYZ
left a comment
There was a problem hiding this comment.
Reviewed PR #458 at head 628b43a.
Verdict: COMMENT — no PR-specific code/supply-chain blocker found in the changed files, but validation is bounded because install/build could not be completed in this environment and the PR deploy check is currently failing.
Scope reviewed:
package.json: direct Nuxt bump3.21.2 -> 3.21.6only.package-lock.json: Nuxt ecosystem transitive update, including@nuxt/nitro-server,@nuxt/vite-builder,@nuxt/schema,nitropack,oxc-*,vite-plugin-checker, and related lockfile integrity/resolved metadata.
Validation performed:
- Confirmed checked-out PR head:
628b43a957554d35010835699c7c4d0ce3df801c. - Diff inspected against
origin/development; changed files are onlypackage.jsonandpackage-lock.json. - Parsed both JSON files successfully.
- Ran
npm ls nuxt @nuxt/vite-builder @nuxt/nitro-server nitropack oxc-parser oxc-transform --package-lock-only --depth=1; lock resolves Nuxt packages to3.21.6and expected transitive versions. - Compared base vs PR lockfile package graph: 13 added package entries, 5 removed, 73 version changes, all in the Nuxt/build-tool dependency surface.
- Ran package-lock audit with
npm audit --package-lock-only --omit=devon base and PR locks. Both report 11 moderate vulnerabilities; the PR removes the Nuxt<=3.21.5advisory ranges for the redirect/island issues, while existing moderate findings remain throughunhead/serialize-javascript/viemsurfaces. - Supply-chain sweep of the diff found no new app code, scripts, GitHub Actions, install-script changes, credential material, or non-registry dependency sources introduced by the PR.
Validation caveats:
npm ci --ignore-scriptsfailed on both the PR lock and the current base lock because@eulerxyz/euler-v2-sdk@0.2.13-betaresolves to a registry tarball that returned 404. Because of that, I did not runnpm run test:run,npm run build, or browser smoke from a clean install.- GitHub/Railway currently shows
euler-lite - dev-buildfailed for this PR. I did not treat that as a PR-specific Nuxt regression without logs, but it should be resolved or shown to be known baseline noise before merge.
No inline comments from me; the diff is mechanical and I did not find a changed-line issue worth anchoring.
628b43a to
e7948b5
Compare
Bumps the nuxt-ecosystem group with 1 update in the / directory: [nuxt](https://github.com/nuxt/nuxt/tree/HEAD/packages/nuxt). Updates `nuxt` from 3.21.2 to 3.21.6 - [Release notes](https://github.com/nuxt/nuxt/releases) - [Commits](https://github.com/nuxt/nuxt/commits/v3.21.6/packages/nuxt) --- updated-dependencies: - dependency-name: nuxt dependency-version: 3.21.5 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: nuxt-ecosystem ... Signed-off-by: dependabot[bot] <support@github.com>
e7948b5 to
7a41df6
Compare
Rebasing might not happen immediately, so don't worry if this takes some time.
Note: if you make any changes to this PR yourself, they will take precedence over the rebase.
Bumps the nuxt-ecosystem group with 1 update in the / directory: nuxt.
Updates
nuxtfrom 3.21.2 to 3.21.6Release notes
Sourced from nuxt's releases.
... (truncated)
Commits
1a8fff3v3.21.6d152a5efix(nuxt): run middleware for page islands (#35092)d6caa8efix(nuxt): render component-less parent routes during client-side nav (#35036)63e5437chore(deps): update all non-major dependencies (3.x) (#35076)21c110afix(nitro): validate island request hash matches props (#35077)17b27b0fix(nuxt): encode html-significant characters in external redirect body (#35052)c67675cfix(nuxt): markuseLoadingIndicatorproperties as readonly (#35062)702c02bfix(nuxt): preservesetPageLayoutprops on same-path navigation (#35055)aacb18dfix(nuxt): handle errors fetching app manifest (#35050)db4b5fffix(nuxt): prefer our own builder/server deps (#35029)