Context
In #72, we upgraded eslint-config-next from 15.x to 16.x to fix CI. The upgrade brought in eslint-plugin-react-hooks v7 which introduces several new rules. These were temporarily disabled to keep the fix focused.
Additionally, the scripts/ directory was excluded from linting since next lint never covered it, but it should be linted going forward.
Tasks
Enable new react-hooks v7 rules
The following rules are currently disabled in eslint.config.mjs and should be enabled (fixing violations as needed):
Lint the scripts directory
🤖 Generated with Claude Code
Context
In #72, we upgraded
eslint-config-nextfrom 15.x to 16.x to fix CI. The upgrade brought ineslint-plugin-react-hooksv7 which introduces several new rules. These were temporarily disabled to keep the fix focused.Additionally, the
scripts/directory was excluded from linting sincenext lintnever covered it, but it should be linted going forward.Tasks
Enable new react-hooks v7 rules
The following rules are currently disabled in
eslint.config.mjsand should be enabled (fixing violations as needed):react-hooks/set-state-in-effect(23 warnings)react-hooks/refs(18 warnings)react-hooks/exhaustive-deps— already enabled as warning, review remaining 17 violationsreact-hooks/immutability(5 warnings)react-hooks/purity(4 warnings)react-hooks/static-components(3 warnings)react-hooks/incompatible-library(1 warning)react-hooks/globals(1 warning)react-hooks/preserve-manual-memoization(1 warning)Lint the scripts directory
scripts/from the ESLint ignores ineslint.config.mjs@typescript-eslint/no-explicit-anyerrors inscripts/build-registry.mts🤖 Generated with Claude Code