Skip to content

feat(frontend): migrate to eslint 10#98

Merged
vnykmshr merged 1 commit into
mainfrom
feat/eslint-10-migration
Mar 14, 2026
Merged

feat(frontend): migrate to eslint 10#98
vnykmshr merged 1 commit into
mainfrom
feat/eslint-10-migration

Conversation

@vnykmshr
Copy link
Copy Markdown
Collaborator

Summary

  • Bump eslint 9 → 10 and @eslint/js 9 → 10
  • Add npm overrides for eslint-plugin-react-hooks peer dep (eslint 10 support merged upstream in [eslint-plugin-react-hooks] Add ESLint v10 support facebook/react#35720 but not yet published to npm)
  • Fix 6 no-useless-assignment errors from eslint 10's expanded recommended ruleset
  • Fix 2 high-severity npm audit vulnerabilities (flatted DoS, undici multiple)

Closes #80. Supersedes #97 (Dependabot eslint bump that failed CI due to peer dep conflict).

Test plan

  • npx eslint . — 0 errors, 0 warnings
  • npx tsc -b --noEmit — clean
  • npx vitest run — 47 files, 541 tests passing
  • npm run build — successful production build
  • CI passes (lint, typecheck, tests, build, security scan)

Bump eslint 9 → 10 and @eslint/js 9 → 10. Add npm overrides for
eslint-plugin-react-hooks peer dep (eslint 10 support merged upstream
but not yet published to npm).

Fix 6 new no-useless-assignment errors from eslint 10's expanded
recommended ruleset by removing redundant initializers.

Also fixes 2 high-severity npm audit vulnerabilities (flatted, undici).
@vnykmshr vnykmshr merged commit 2198317 into main Mar 14, 2026
4 checks passed
@vnykmshr vnykmshr deleted the feat/eslint-10-migration branch March 14, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant