chore(PP-3449): update dependencies to latest - #185
Merged
Conversation
…xtures
Bump root, test-nextjs, test-nextjs-cjs, and test-commonjs devDependencies/
dependencies to latest, including major bumps (http-proxy-middleware v4,
ejs v6, @types/node v26, Next.js/React/Tailwind patch releases).
Intentionally NOT bumped:
- undici stays on latest 7.x (not v8): the root/test-nextjs override forces
a single undici resolution repo-wide, and jsdom pins ^7.25.0 and reaches
into undici's internal file layout (require('undici/lib/handler/
wrap-handler.js')), which v8 restructured. Bumping breaks jsdom at runtime.
- eslint stays on latest 9.x (not v10) in test-nextjs/test-nextjs-cjs:
eslint-config-next@16.2.10 only supports eslint ^3-^9 as a peer; v10
installs only via forced ERESOLVE overrides.
- jquery/@types/jquery stay on latest 3.x (not v4) in test-commonjs: the
fixture mirrors a real Portal Page template, and the MI backend that
serves /js/jquery/jquery-latest.min.js in production still serves v3.
Verified: npm run audit:all (0 vulnerabilities across all 4 packages),
tsc --noEmit, eslint, 185 unit + 39 integration tests, and production
builds all green for root, test-nextjs, test-nextjs-cjs, and test-commonjs.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
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.
Summary
Updates dependencies to latest across the root package and all three test fixtures (
test-nextjs,test-nextjs-cjs,test-commonjs), including several major-version bumps, with a handful of deliberate exceptions where the ecosystem/runtime isn't ready yet.Key changes
http-proxy-middleware3→4,ejs5→6,@types/node25→26, plus routine minor/patch bumps (next,vite,vitest,rollup,sass,prettier,axios,sharp,eslint,semantic-release,@playwright/test, etc.)test-nextjs: samehttp-proxy-middleware/ejs/@types/nodemajors, plusnext/react/react-dom/tailwindcss/sass/rolluppatch bumpstest-nextjs-cjs:@types/node25→26,next/react/react-dom/tailwindcss/sharppatch bumpstest-commonjs:vitepatch bump,@types/luxonpatch bumpIntentionally NOT bumped (verified, not overlooked)
undicistays on latest7.x(not8.x) — the root/test-nextjsoverridesforce a singleundiciresolution repo-wide, andjsdompins^7.25.0and reaches into undici's internal file layout (require('undici/lib/handler/wrap-handler.js')), which v8 restructured. Bumping brokejsdomat runtime (Cannot find module 'undici/lib/handler/wrap-handler.js') — reproduced and confirmed before reverting.eslintstays on latest9.x(not10.x) intest-nextjs/test-nextjs-cjs—eslint-config-next@16.2.10's peer dependency only allowseslint@^3...^9; installing10.xonly succeeds via forcedERESOLVEoverrides, which is unsafe for a config package whose plugins may not actually support the new major.jquery/@types/jquerystay on latest3.x(not4.x) intest-commonjs— this fixture mirrors a real Portal Page template, and the MI backend that serves/js/jquery/jquery-latest.min.jsin production still serves jQuery 3, so the fixture needs to match runtime reality rather than the npm registry's latest.Pre-existing issues found during verification (unrelated to this bump, confirmed via git stash on original deps)
next lintno longer exists as a CLI command in Next.js 16 (removed upstream) — not exercised by CI, not caused by this PR.test-nextjs/test-nextjs-cjslocalnext buildcan fail type-checking due to a stray empty lockfile atC:\Users\<user>\repos\package-lock.jsonconfusing Turbopack's workspace-root inference on this machine, and separately due to stale untrackeddist/dev/types/validator.tscolliding with fresh.next/types/validator.ts. Both are local-environment artifacts (gitignored dirs / sibling-repo lockfile), not present in a clean CI checkout.test-nextjs-cjs'stailwind.config.tsfailstsctype resolution under itsmoduleResolution: "node"setting against Tailwind v4's.d.mts-only export map — reproduced identically on the original, unbumpedtailwindcss@4.2.2, so this is pre-existing and out of scope here.Testing
npm run audit:all— 0 vulnerabilities across root + all 3 fixturesnpx tsc --noEmit(root) — cleannpm run lint(root) — cleannpm run test:unit— 185 passednpm run test:integration— 39 passednpm run build(root + all 3 fixtures) — all succeedMerge Request:
origin/pp-3449-deps-update→origin/develop