Repository navigation
Conversation
Bumps [next](https://github.com/vercel/next.js) from 16.3.3 to 16.3.6. - [Release notes](https://github.com/vercel/next.js/releases) - [Commits](vercel/next.js@v16.3.3...v16.3.6) --- updated-dependencies: - dependency-name: next dependency-version: 16.3.6 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. 🧰 Additional context used📚 Code guidelines (2)No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe mobile package updates its declared Next.js dependency from version 16.3.3 to 16.3.6. ChangesMobile dependency update
Priority: ⬆️ High Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to No issue requiring a change before merge was identified for the mobile dependency update. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 4 | ❓ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
Full details: Out of Scope Changes checkExplanation The reviewed change to ✨ 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 |
Website previewPreview URL: https://pr-7319-agenta-website-preview.mahmoud-637.workers.dev Built from |
Railway Preview Environment
Dependent tests finished. Post /preview for manual review. Environment absence verified; retained storage requires separate provider accounting. Post |
Summary
Bump
nextfrom16.3.3to16.3.6inweb/mobile/package.jsonand refreshweb/pnpm-lock.yaml.This applies the exact Dependabot patch from #7298 to
main. That PR was merged only intorelease/v0.121.8.Security alerts fixed
GHSA-vcvr-r3jv-pc5j(critical): Next.js remote code execution innext/ogImageResponse; manifestweb/mobile/package.jsonGHSA-vcvr-r3jv-pc5j(critical): the same vulnerable Next.js resolution inweb/pnpm-lock.yamlnext16.3.3→16.3.6Hetzner verification
Tested from an isolated worktree based on
main(9c9f97b7ca) on the Hetzner server.pnpm install --frozen-lockfile: passpnpm lint-fix: pass (28 tasks; no file changes beyond the dependency patch)pnpm --filter @agenta/mobile types:check: passpnpm build-mobile: pass with Next.js16.3.6docker build -f mobile/docker/Dockerfile.gh ...: pass; imageb320243ddb6996373b61a6a58712825717ca9cd120f45f70fc1d35953ee07611pnpm --filter @agenta/mobile test: 365 pass, 1 failThe one failed test is verified as pre-existing on an unchanged clean
mainworktree. In both worktrees,tests/unit/projectPermission.test.tsexpectscheckPermissionsto receive one argument, but the implementation also passes{queryParams: {project_id: "project-1"}}. The exact test and assertion diff are identical. This dependency patch does not change that code.