chore: resolve open dependabot security alerts - #425
Conversation
|
Important Review skippedNo new commits to review since the last review. ⚙️ Run configurationConfiguration used: Organization UI 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:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe package configuration replaces one ChangesDependency override update
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change updates vulnerable dependency resolutions in the package manifests and lockfile, with build, lint, and test checks reported as passing; no actionable merge-blocking risk remains beyond normal review and checks. Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
|
@coderabbitai review |
✅ Action performedReview finished.
|
|
@coderabbitai full review |
|
|
Summary
Resolves the open Dependabot alerts that have a lockfile-level fix available. Two alerts are deferred (unresolvable without a larger migration) and one has no patched version at all — see notes below.
Dependabot Alerts Resolved
js-yamljs-yaml@>=4.0.0 <4.3.1to^4.3.1(nx/eslint/cosmiconfig pin 4.3.0)js-yamljs-yaml@>=3.0.0 <3.15.1to^3.15.1(splitio 10.x/istanbul/markdown-toc pin 3.15.0)Deferred / Unresolvable
react-router-dom(moderate) — no patched version exists for the 6.x line; the advisory's fix requires the v7 major.react-router(moderate) — patched version is7.18.0, but the app is pinned toreact-router-dom@^6.30.3, which requiresreact-router-domv7. That in turn requires bumping React 17 → 18 (a breaking peer-dependency change) plus a router API migration. Deferred to a dedicated migration rather than folding into a security-only PR.nx(medium) — patched version22.7.2is 4+ majors ahead of the pinnednx/@nx/*toolchain (17.3.2/18.3.5) and would require a fullnx migrateacross the monorepo. The advisory affects thenx graphdev server's permissive CORS policy, which this repo's scripts never invoke, so practical risk is low.Verified locally:
build,lint, andtestall pass withnpm install --legacy-peer-deps --ignore-scripts(matching CI's install flags).