Repository navigation
Conversation
|
Sorry @IanHollow, you've used your own review budget of 250,000 diff characters for the last 7 days. You can request another review in 6 days and 1 hour by commenting |
Reviewer's GuideUpgrades the frontend from Next.js 14/React 18 to Next.js 16.3.8/React 19.3, migrating configuration and linting to the Next 16-compatible TypeScript and ESLint 9 flat-config setup while preserving standalone deployment and the runtime API proxy. Supporting type fixes, dependency lockfile updates, documentation, and contributor workflow changes are included. Flow diagram for the Next.js 16 frontend validation workflowflowchart LR
Install[npm ci] --> Typecheck[npm run typecheck]
Install --> Lint[npm run lint]
Install --> Build[npm run build]
Build --> Smoke[Standalone server smoke test]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Upgrade the frontend from Next.js 14 to 16.3.8. The App Router upgrade also moves React and React DOM to 19.3.0. Update NextAuth to its current v4 release and Lucide to a version that supports React 19.
The upgrade replaces the removed
next lintcommand with the ESLint CLI and a flat config. It moves the Next config to TypeScript, removes options that Next 16 no longer accepts, and lets the production build check TypeScript. The existing standalone output and runtime API proxy remain in place. Contributor guidance now states Next's Node.js minimum.Type of change
Checklist
CONTRIBUTING.mdand the Next.js 15 and 16 upgrade guides.Testing
npm cicompleted without peer dependency overrides.npm run typecheck,npm run lint,npm run i18n:check, andnpm run vocab:checkpassed.npm test -- --runpassed: 19 test files, 194 tests.npm run buildpassed with Turbopack and TypeScript validation./,/login, and/api/auth/providers.git diff --checkpassed.Docker was unavailable locally, so I tested the standalone output directly. The new ESLint preset reports 79 warnings in existing code. Five newly enforced rules are set to warning severity so this framework upgrade does not require unrelated UI changes; lint still fails on all other errors.
Additional notes
PRs #237 and #246 also change the frontend package files. Whichever merges second will need a rebase and another install/build check.
Summary by Sourcery
Upgrade the frontend toolchain to Next.js 16 and React 19 while modernizing its build, lint, and type-check configuration.
Enhancements:
Build:
Documentation:
Tests:
Chores: