Repository navigation
chore: bump major dependencies and migrate to ESLint flat config - #6
Conversation
gokhantaskan
commented
Jan 14, 2026
gokhantaskan
commented
- Upgrade React 18 to React 19
- Upgrade MUI 5 to MUI 7
- Upgrade Vite 5 to Vite 7
- Upgrade TypeScript 4.x to TypeScript 5.x
- Upgrade ESLint 8 to ESLint 9 with flat config
- Upgrade Vitest 2 to Vitest 4
- Migrate .eslintrc.cjs to eslint.config.ts
- Migrate .prettierrc.json to prettier.config.ts
- Update pnpm 9 to pnpm 10
- Upgrade React 18 to React 19 - Upgrade MUI 5 to MUI 7 - Upgrade Vite 5 to Vite 7 - Upgrade TypeScript 4.x to TypeScript 5.x - Upgrade ESLint 8 to ESLint 9 with flat config - Upgrade Vitest 2 to Vitest 4 - Migrate .eslintrc.cjs to eslint.config.ts - Migrate .prettierrc.json to prettier.config.ts - Update pnpm 9 to pnpm 10
✅ Deploy Preview for extraordinary-meerkat-398bc9 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
This pull request aims to upgrade major dependencies including React 18→19, MUI 5→7, Vite 5→7, TypeScript 4.x→5.x, ESLint 8→9, and Vitest 2→4, along with migrating to ESLint flat config and modernizing configuration files. The changes include widespread code reformatting due to updated Prettier settings and removal of deprecated configuration files.
Changes:
- Major dependency version bumps across the entire stack (React, MUI, Vite, TypeScript, ESLint, Vitest, and many supporting libraries)
- Migration from ESLint legacy config (.eslintrc.cjs) to flat config (eslint.config.ts)
- Migration from Prettier JSON config to TypeScript config file
- Code reformatting and minor code improvements (type safety, syntax modernization)
Reviewed changes
Copilot reviewed 28 out of 31 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updates all major dependencies to newer versions |
| eslint.config.ts | New ESLint 9 flat config implementation |
| prettier.config.ts | New TypeScript-based Prettier config |
| .eslintrc.cjs | Removed legacy ESLint config |
| .prettierrc.json | Removed JSON Prettier config |
| .eslintignore | Removed (replaced by gitignore integration) |
| tsconfig.node.json | Updated moduleResolution to "Bundler" |
| pnpm-workspace.yaml | Added build dependency configuration |
| vitest.config.ts | Simplified nested defineConfig call |
| vite.config.ts | Code reformatting |
| src/**/*.tsx | Code reformatting and minor improvements (type safety, imports) |
| README.md | Comprehensive documentation rewrite with new structure |
| index.html | Updated DOCTYPE to lowercase |
| .gitignore | Added *.tsbuildinfo to ignore list |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| { | ||
| files: ["**/*.{js,mjs,cjs,ts,mts,cts,jsx,tsx}"], | ||
| plugins: { js }, | ||
| extends: ["js/recommended"], |
There was a problem hiding this comment.
The ESLint flat config is using an invalid "extends" property within a config object. In ESLint 9's flat config format, there is no "extends" property. Instead, you should directly spread the recommended config or reference it properly. The line should be removed, and the js.configs.recommended should be spread at the array level instead.
| { | |
| files: ["**/*.{js,mjs,cjs,ts,mts,cts,jsx,tsx}"], | |
| plugins: { js }, | |
| extends: ["js/recommended"], | |
| ...js.configs.recommended, | |
| { |
| position="bottom-center" | ||
| toastOptions={{ | ||
| duration: 500000000, | ||
| duration: 1_000_000, |
There was a problem hiding this comment.
The toast duration has been changed from 500000000 milliseconds (approximately 5787 days) to 1000000 milliseconds (16.67 minutes). While the previous value was likely a placeholder, this new value may still be too long for typical toast notifications. Consider using a more reasonable default like 5000 (5 seconds) for error/success messages, or document why such a long duration is needed.
| duration: 1_000_000, | |
| duration: 5_000, |
- Update GitHub Actions to v4, Node.js to 20, and pnpm to 10. - Migrate tests to use @testing-library/user-event for more realistic interactions. - Fix toast duration and add timing tolerance for flaky tests. - Fix copilot review issues
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 31 out of 34 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
