Updated NextJS and other dependencies#8
Conversation
|
Worried about impact? Review this PR in Change Stack to explore blast radius before you approve or request changes. Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis PR updates the Docker base image from Bun 1.2 to 1.3, upgrades major framework and tooling dependencies (Next.js 16.2.7, React 19.2.7, TypeScript 6.0.3, ESLint 10.4.1), migrates ESLint configuration to flat config format with direct imports, removes static asset caching headers, and reformats TypeScript configuration files. ChangesDependency Upgrades and Configuration Alignment
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
There was a problem hiding this comment.
Pull request overview
This PR upgrades the app’s Next.js/React toolchain (including TypeScript/ESLint) and adjusts configuration to align with the new ecosystem defaults (linting, dev server, Docker runtime, and caching headers).
Changes:
- Upgrade Next.js to 16.2.7 and refresh a broad set of runtime + dev dependencies (React, TypeScript, ESLint, Tailwind, etc.).
- Switch linting to run
eslintdirectly and move toeslint-config-nextflat config, with targeted rule overrides. - Update build/runtime configuration: Bun base image bump, remove Turbopack flag for dev, and simplify
next.config.tsheaders.
Reviewed changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.json | Adjust TS compiler options (notably jsx) and includes for updated Next/TS behavior. |
| package.json | Update dependency versions; change scripts for dev and linting. |
| next.config.ts | Remove custom static asset cache headers; keep HTML caching headers and rewrites. |
| eslint.config.mjs | Replace FlatCompat usage with Next’s flat config imports and add rule overrides. |
| Dockerfile | Bump Bun base image version used for builds and runtime. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| { | ||
| ignores: ['eslint.config.mjs'], | ||
| }, |
This pull request upgrades the project to Next.js 16 and updates several dependencies, while also making changes to ESLint configuration and build tooling. The updates aim to keep the codebase current, improve compatibility, and simplify configuration.
Dependency and Tooling Upgrades:
nextto version 16.2.7, along with updates toreact,react-dom, and many related dependencies for improved compatibility and new features.@radix-ui/*,drizzle-orm,framer-motion,ioredis,lucide-react,nanoid,postgres,posthog-js,posthog-node,tailwind-merge,zod,zustand,@types/*,drizzle-kit,eslint,eslint-config-next,prettier,tailwindcss,tsx, andtypescriptto their latest versions.ESLint and Linting Configuration:
eslint.config.mjsto use the neweslint-config-nextflat config, directly importingcore-web-vitalsandtypescriptconfigs, and set React version to 19.2. Also, temporarily disabled newreact-hooksrules introduced in the upgrade to avoid false positives until a dedicated refactor.lintscript inpackage.jsonto useeslintdirectly instead ofnext lint.Build and Runtime Configuration:
oven/bun:1.3-alpinefor the latest Bun runtime.next.config.ts, relying on Next.js 16’s improved built-in static asset caching.devscript to use the default Next.js dev server, removing the--turbopackflag.Summary by CodeRabbit