Apply a few updates:#123
Conversation
seratch
commented
Dec 19, 2025
- migrate to next 16
- minor code and setting file improvements
- AGENTS.md
- GitHub Actions job to run lint and build
- migrate to next 16 - minor code and setting file improvements - AGENTS.md - GitHub Actions job to run lint and build
There was a problem hiding this comment.
Pull request overview
This PR upgrades the project to Next.js 16 and introduces CI automation. The changes streamline ESLint configuration, remove unnecessary code comments, add repository documentation for AI agents, and establish automated lint and build checks via GitHub Actions.
- Upgraded Next.js from v15 to v16 along with corresponding eslint-config-next
- Simplified ESLint configuration and removed obsolete eslint-disable comments
- Added AGENTS.md documentation and GitHub Actions CI workflow
Reviewed changes
Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated Next.js and eslint-config-next to v16, changed lint script to use eslint directly |
| eslint.config.mjs | Simplified ESLint config by importing next config directly instead of using FlatCompat |
| src/lib/copyText.ts | Removed unused error variables in catch blocks |
| AGENTS.md | Added repository guidelines for AI agents covering structure, development, and conventions |
| .github/workflows/ci.yml | Added CI workflow to run lint and build on push/PR to main branch |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
For this i18n route work, a short fallback behavior note (missing locale) would reduce review ambiguity. |
|
Large upgrade diff is broad. Since this now includes AGENTS.md and lint/build changes, could you confirm this repo has no additional AGENTS conventions already, and avoid duplicate instructions if any existed. |