chore(deps): resolve npm security advisories (consolidated) - #100
Conversation
|
Warning Review limit reached
Next review available in: 12 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
✨ 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 |
Consolidated fix for the Dependabot npm_and_yarn security PRs — one lockfile update instead of merging each PR individually (which serially conflicts on package-lock.json). - Bump @wordpress/scripts 30 -> 32.5.1 (dev): clears the shell-quote critical plus a batch of build-tool highs (ws, serialize-javascript, @babel/core, ...) - Bump react-router-dom -> ^7.18.1: patches the shipped react-router advisory - overrides: @babel/runtime ^7.26.10 (fixes the copy pulled via @emotion/styled), serialize-javascript ^7.0.7 - withRouter: type `navigation` as ReturnType<typeof useNavigation>; react-router 7.18 narrowed useNavigation()'s public return type (dropped matches/historyAction) Result: 49 -> 28 advisories. Remaining are dev/test-only tooling that never ships in dist/ (lighthouse -> @sentry/node -> @opentelemetry/*, webpack-dev-server, markdownlint) plus three prod moderates with no upstream fix yet (@wordpress/components, @wedevs/plugin-ui, uuid). Verified: npm run typecheck + npm run build both pass.
6541980 to
bb48c85
Compare
What
Single consolidated lockfile update that resolves the open Dependabot
npm_and_yarnsecurity PRs, instead of merging ~19 of them one-by-one (each rebasespackage-lock.json→ serial conflicts).Advisories: 49 → 28. Critical cleared; high 16 → 5.
Changes
@wordpress/scripts30 → 32.5.1 (dev). Clears theshell-quotecritical plus a batch of build-tool highs (ws,serialize-javascript,@babel/core,minimatch, …).react-router-dom→^7.18.1(prod). Patches the shippedreact-routeradvisory (Dependabot chore(deps): bump react-router and react-router-dom #98).overrides:@babel/runtime ^7.26.10(fixes the copy pulled in via@emotion/styledthat actually ships indist/),serialize-javascript ^7.0.7.src/routing/index.tsx: typenavigationasReturnType<typeof useNavigation>— react-router 7.18 narroweduseNavigation()'s public return type (droppedmatches/historyAction).What ships in
dist/Production-only audit: 8 → 3 moderate, all with no upstream fix yet —
@wordpress/components,@wedevs/plugin-ui(github dep, fix belongs in that repo),uuid(fix is a breaking major that would break CJS consumers).The residual 28
Almost entirely dev/test tooling that never reaches users:
lighthouse → @sentry/node → @opentelemetry/*(pulled by@wordpress/scripts' e2e utils),webpack-dev-server,markdownlint. No safe fix exists (npm only offers a bogus@wordpress/scriptsdowngrade), and forcing overrides there risks breaking the build for tooling with zero shipped exposure.Verification
npm run typecheck✅npm run build✅ (webpack 5.105.2, only pre-existing asset-size perf warnings)Note
Once merged to
develop, Dependabot re-evaluates and auto-closes the supersedednpm_and_yarnPRs whose advisories are now resolved in the lockfile.