Repository navigation
chore(deps): clear npm audit highs — react-router v8 + transitive bumps - #203
Merged
Merged
Conversation
Two advisory waves landed 2026-07-30 and turned the npm-audit CI gate red on every branch: - brace-expansion (high, GHSA-mh99-v99m-4gvg) and tar (moderate) — fixed by a plain lockfile bump. - react-router: the new RSC CSRF advisory (GHSA-qwww-vcr4-c8h2) covers 7.12.0–8.2.0 while the older XSS/RCE batch covers <=7.17.0, so no 7.x release is clean. Only react-router@8.3.0 clears both. Migrated the client from react-router-dom@7 to react-router@8.3.0: in v7+ react-router-dom is a re-export shim and v8 retires it, so this is a pure import-path swap (13 files) plus the dependency rename. All router APIs used (BrowserRouter, Routes, Route, Navigate, Memory/useNavigate/useParams/ useLocation/useSearchParams, Link) are unchanged. npm audit: 0 vulnerabilities. Typecheck all workspaces + full client suite (27 files, 186 tests) green locally.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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
1 participant
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.
Why
Two advisory waves published today turned the
npm audit (high)CI gate red on every open PR (#198–#200 all fail identically on main's lockfile):react-router@8.3.0is the only clear release.What
npm audit fixfor brace-expansion/tar (root lockfile only)react-router-dom@^7→react-router@^8.3.0. The dom package is a v7 re-export shim retired in v8, so this is an import-path swap across 13 files + dependency rename; every API used is unchanged. Per-workspace lockfiles untouched (CI cache keys).Verification
npm audit --audit-level=high: 0 vulnerabilitiesnpm run typecheck: all four workspaces cleannpm test -w client: 27 files, 186 tests greenUnblocks the merge gate for #198, #199, #200.