feat(react-doctor): adopt user lint config, ship as ESLint plugin, remove browser surface #398
Annotations
22 warnings
|
test (24)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/setup-node@v4, pnpm/action-setup@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/giant-component.tsx#L4
Variable 'x2' is declared but never used. Unused variables should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/giant-component.tsx#L3
Variable 'x1' is declared but never used. Unused variables should start with a '_'.
|
|
react-hooks(exhaustive-deps):
packages/react-doctor/tests/fixtures/basic-react/src/namespace-hooks.tsx#L70
React Hook useCallback has a complex expression in the dependency array.
|
|
react-hooks(exhaustive-deps):
packages/react-doctor/tests/fixtures/basic-react/src/namespace-hooks.tsx#L69
React Hook useEffect has a complex expression in the dependency array.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/namespace-hooks.tsx#L59
Variable 'setValue' is declared but never used. Unused variables should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/performance-issues.tsx#L39
Variable 'setIsLoading' is declared but never used. Unused variables should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/tanstack-start-app/src/routes/edge-cases.tsx#L4
Parameter 'schema' is declared but never used. Unused parameters should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/tanstack-start-app/src/routes/edge-cases.tsx#L3
Parameter 'options' is declared but never used. Unused parameters should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/architecture-issues.tsx#L1
Identifier 'useState' is imported but never used.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/tanstack-start-app/src/routes/route-issues.tsx#L5
Variable 'redirect' is declared but never used. Unused variables should start with a '_'.
|
|
test (22)
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/setup-node@v4, pnpm/action-setup@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/giant-component.tsx#L4
Variable 'x2' is declared but never used. Unused variables should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/giant-component.tsx#L3
Variable 'x1' is declared but never used. Unused variables should start with a '_'.
|
|
react-hooks(exhaustive-deps):
packages/react-doctor/tests/fixtures/basic-react/src/namespace-hooks.tsx#L70
React Hook useCallback has a complex expression in the dependency array.
|
|
react-hooks(exhaustive-deps):
packages/react-doctor/tests/fixtures/basic-react/src/namespace-hooks.tsx#L69
React Hook useEffect has a complex expression in the dependency array.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/namespace-hooks.tsx#L59
Variable 'setValue' is declared but never used. Unused variables should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/tanstack-start-app/src/routes/route-issues.tsx#L5
Variable 'redirect' is declared but never used. Unused variables should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/tanstack-start-app/src/routes/edge-cases.tsx#L4
Parameter 'schema' is declared but never used. Unused parameters should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/tanstack-start-app/src/routes/edge-cases.tsx#L3
Parameter 'options' is declared but never used. Unused parameters should start with a '_'.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/architecture-issues.tsx#L1
Identifier 'useState' is imported but never used.
|
|
eslint(no-unused-vars):
packages/react-doctor/tests/fixtures/basic-react/src/performance-issues.tsx#L39
Variable 'setIsLoading' is declared but never used. Unused variables should start with a '_'.
|