Repository navigation
fix: resolve npm audit vulnerabilities - #73
Conversation
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. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe web package manifest updates ChangesWeb dependency updates
Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other ✨ 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. A rabbit checks the package row Comment |
|
Preview deployed!
This is a Cloudflare Workers preview version of this PR's build. |
Summary
The dependency tree reported 20 vulnerable packages, including a critical Next.js advisory. Upgrade Next.js from 16.2.10 to 16.3.5 and fflate from 0.4.8 to 0.4.9, and refresh affected dependencies in the lockfile. The updated tree reports zero vulnerabilities.
Validation
npm audit: 0 vulnerabilitiesnpm ci --ignore-scripts: passed; subsequent audit also reports 0 vulnerabilitiesnpm test: 168 tests passed across web and routernpm --workspace router run typecheck: passednpm run build: passed TypeScript checks and generated 444 static pagesgit diff --check: passedExisting peer-dependency warnings predate this change. No deployment or preview was run.
Note
Medium Risk
Next.js patch upgrade affects the entire web runtime and build; lockfile churn touches image processing (sharp) and Cloudflare dev tooling, so regressions are possible though the change is dependency-only with validation noted in the PR.
Overview
Addresses npm audit findings (including a critical Next.js advisory) by bumping direct dependencies and regenerating the lockfile.
Direct
webchanges:next16.2.10 → 16.3.5 andfflate0.4.8 → 0.4.9. No application source changes.The lockfile also pulls in related upgrades used by the web build and Cloudflare tooling—e.g. sharp 0.35.x (stricter Node
>=20.9.0), @next/swc binaries, tailwindcss 4.3.3, wrangler/miniflare/workerd for local Workers dev, and vitest/rolldown/vite dev transitive updates.@cloudflare/workers-typesis no longer hoisted at the root; the router workspace keeps its own pinned types entry in the lockfile.Reviewed by Cursor Bugbot for commit d73f1bc. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by cubic
Resolves all 20 npm audit vulnerabilities in the dependency tree, including a critical Next.js advisory. Upgrades
nextfrom 16.2.10 to 16.3.5 andfflatefrom 0.4.8 to 0.4.9, and refreshes the lockfile.Validation
npm auditreports 0 vulnerabilities afternpm ci.npm testpasses 168 tests across web and router.git diff --checkall pass.Written for commit d73f1bc. Summary will update on new commits.