fix(deps): bump vitest to 4.1.11 for @vitest/mocker path traversal - #132
Conversation
Dependabot alerts #19 and #20 flag a path traversal / arbitrary file read in @vitest/mocker's redirect mock, reachable through vitest 4.1.10. Bump the direct vitest dev dependency to ^4.1.11, which pulls the patched @vitest/mocker 4.1.11 (and the rest of the 4.1.11 @vitest/* set) into the lockfile. Frontend tests (141) and svelte-check pass on the new version. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PwGJ1dnwAeD8gRWs7TPPXv
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe frontend package configuration updates the ChangesVitest dependency update
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This updates the frontend test dependency to Vitest 4.1.11 to address reported mocker vulnerabilities. The lockfile is aligned and frontend tests and checks pass, with no remaining merge-blocking risk identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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. Vitest gets a tasty new bite Comment |
Closes Dependabot alerts #19 and #20 — Vitest: Path Traversal / Arbitrary File Read via @vitest/mocker Redirect Mock (moderate, development scope).
What changed
frontend/package.json:vitestdev dependency^4.1.10→^4.1.11.frontend/package-lock.json: the whole@vitest/*set (mocker,runner,snapshot,spy,utils,expect,pretty-format) andvitestmove 4.1.10 → 4.1.11, plus a transitivetinyrainbow3.1.0 → 3.1.1. No other dependencies move.The alerts fire on
@vitest/mocker, which is not a direct dependency here — it comes in throughvitest, so bumping the direct dev dependency is what pulls the patched 4.1.11 into the lockfile.Verification
npm run test— 19 test files, 141 tests passing on vitest 4.1.11.npm run check— svelte-check: 647 files, 0 errors, 0 warnings.Notes
npm auditstill reports two unrelated advisories on this tree that are outside this change's scope:@sveltejs/kitReDoS in content negotiation (moderate) andnanoidzero-size generator loop (high). Neither is covered by the alerts this PR addresses; happy to take them in a follow-up.🤖 Generated with Claude Code
https://claude.ai/code/session_01PwGJ1dnwAeD8gRWs7TPPXv
Generated by Claude Code