Staging #82
Staging #82
Conversation
There was a problem hiding this comment.
Pull request overview
This PR upgrades the Next.js framework from version 15.1.6 to 16.0.7, along with associated dependency updates and configuration adjustments to support the new version.
Key changes:
- Next.js upgraded from 15.1.6 to 16.0.7, with related updates to next-auth (5.0.0-beta.30) and nodemailer (7.0.11)
- TypeScript and ESLint configurations updated to align with Next.js 16 requirements
- Dockerfile base image changed from Alpine to Slim variant
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Upgrades Next.js, next-auth, and nodemailer versions; modifies lint command to use direct ESLint invocation |
| tsconfig.json | Updates TypeScript configuration with formatting changes, new dev types path, and jsx compiler option modification |
| eslint.config.mjs | Adds ignore patterns and disables @typescript-eslint/no-require-imports rule |
| src/lib/server/stripe.ts | Updates Stripe API version from 2025-04-30 to 2025-08-27 |
| src/components/ui/button.tsx | Reformats interface extends clause across multiple lines |
| src/components/ui/badge.tsx | Reformats interface extends clause across multiple lines |
| Dockerfile | Changes base image from node:23-alpine3.20 to node:23-slim |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "next": "^16.0.7", | ||
| "next-auth": "^5.0.0-beta.30", |
There was a problem hiding this comment.
Version mismatch detected: next is upgraded to ^16.0.7 while eslint-config-next in devDependencies remains at 15.1.6 (line 74). These versions should be kept in sync to ensure ESLint rules are compatible with Next.js 16.
Update eslint-config-next to match the Next.js version:
"eslint-config-next": "^16.0.7"| "check:build": "next build", | ||
| "check": "npm run check:format && npm run check:types && npm run check:lint && npm run test && npm run check:build", | ||
| "check:lint": "next lint", | ||
| "check:lint": "eslint --cache --cache-location .next/cache/eslint/", |
There was a problem hiding this comment.
The lint command changed from next lint to eslint --cache --cache-location .next/cache/eslint/ but is missing file/directory arguments. This will cause ESLint to attempt linting the entire project root, including configuration files and potentially unwanted directories.
Specify explicit paths to lint, for example:
"check:lint": "eslint --cache --cache-location .next/cache/eslint/ ./src"| "check:lint": "eslint --cache --cache-location .next/cache/eslint/", | |
| "check:lint": "eslint --cache --cache-location .next/cache/eslint/ ./src", |
Upgrade to next JS 16+