React Review Audit

Abierto

#4 abierto el 25/5/2026 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 4/5
Tiempo estimado: 3-5 días
Aptitud para principiantes: 48/100

Tipo de issue: Error
Claridad: Bien especificado
Estado de actividad: Tranquilo
Stack tecnológico: nextjs, react, typescript
Área: accessibility, frontend, performance

Línea de trabajo

Start with the diagnostics in apps/demo/src/app.tsx, apps/website/app/page.tsx, apps/website/app/claude-spinner.tsx, and apps/website/package.json, using the listed line numbers and rule guidance. Review each warning and error, then rerun the React Review audit; done means the reported reduced-motion error and all listed warnings are resolved.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

❌ 1 error, ⚠️ 22 warnings

Copy as prompt

Fix the following React Review diagnostics in my codebase.

## Errors (1)

1. [error] require-reduced-motion — apps/website/package.json:0
   Project uses a motion library but has no prefers-reduced-motion handling — required for accessibility (WCAG 2.3.3)

## Warnings (22)

2. [warning] exhaustive-deps — apps/demo/src/app.tsx:72
   React Hook `useEffect` is missing dependency `inputValue.trim` — list it in the dependency array, or call the hook unconditionally.

3. [warning] no-array-index-key — apps/demo/src/app.tsx:96
   Array index in `key` doesn't uniquely identify the element — re-renders may use stale state.

4. [warning] no-array-index-as-key — apps/demo/src/app.tsx:96
   Array index "index" used as key — causes bugs when list is reordered or filtered

5. [warning] control-has-associated-label — apps/demo/src/app.tsx:107
   A control must be associated with a text label — add visible text, `aria-label`, or `aria-labelledby`.

6. [warning] button-has-type — apps/demo/src/app.tsx:118
   `<button>` elements must have an explicit `type` attribute.

7. [warning] nextjs-missing-metadata — apps/website/app/page.tsx:6
   Page without metadata or generateMetadata export — hurts SEO

8. [warning] use-lazy-motion — apps/website/app/page.tsx:9
   Import "m" with LazyMotion instead of "motion" — saves ~30kb in bundle size

9. [warning] exhaustive-deps — apps/website/app/page.tsx:261
   React Hook `useEffect` is missing dependency `config` — list it in the dependency array, or call the hook unconditionally.

10. [warning] click-events-have-key-events — apps/website/app/page.tsx:579
   Visible non-interactive elements with click handlers must have a corresponding keyboard listener (`onKeyUp`, `onKeyDown`, or `onKeyPress`).

11. [warning] no-static-element-interactions — apps/website/app/page.tsx:579
   Static HTML elements with event handlers require a role — add `role="…"` or use a semantic HTML element instead.

12. [warning] button-has-type — apps/website/app/page.tsx:596
   `<button>` elements must have an explicit `type` attribute.

13. [warning] no-array-index-key — apps/website/app/page.tsx:758
   Array index in `key` doesn't uniquely identify the element — re-renders may use stale state.

14. [warning] no-array-index-as-key — apps/website/app/page.tsx:758
   Array index "index" used as key — causes bugs when list is reordered or filtered

15. [warning] click-events-have-key-events — apps/website/app/page.tsx:759
   Visible non-interactive elements with click handlers must have a corresponding keyboard listener (`onKeyUp`, `onKeyDown`, or `onKeyPress`).

16. [warning] no-static-element-interactions — apps/website/app/page.tsx:759
   Static HTML elements with event handlers require a role — add `role="…"` or use a semantic HTML element instead.

17. [warning] no-array-index-as-key — apps/website/app/claude-spinner.tsx:38
   Array index "index" used as key — causes bugs when list is reordered or filtered

18. [warning] unused-dependency — apps/website/package.json:0
   Unused dependency: `class-variance-authority`

19. [warning] unused-dependency — apps/website/package.json:0
   Unused dependency: `lucide-react`

20. [warning] unused-dependency — apps/website/package.json:0
   Unused dependency: `pasito`

21. [warning] unused-dependency — apps/website/package.json:0
   Unused dependency: `torph`

22. [warning] unused-dependency — apps/website/package.json:0
   Unused dependency: `web-haptics`

23. [warning] unused-dependency — apps/website/package.json:0
   Unused dependency: `zustand`

❌ Errors (1)

Project uses a motion library but has no prefers-reduced-motion handling — required for accessibility (WCAG 2.3.3) _{· 1 in 1 file}

_{require-reduced-motion}

Add useReducedMotion() from your animation library, or a @media (prefers-reduced-motion: reduce) CSS query

File	Lines
`apps/website/package.json`	0

⚠️ Warnings (22)

Unused dependency: `class-variance-authority` _{· 6 in 1 file}

_{unused-dependency}

Remove the dependency from package.json if it is genuinely unused.

File	Lines
`apps/website/package.json`	0, 0, 0, 0, 0, 0

Array index "index" used as key — causes bugs when list is reordered or filtered _{· 3 in 3 files}

_{no-array-index-as-key}

Use a stable unique identifier: key={item.id} or key={item.slug} — index keys break on reorder/filter

File	Lines
`apps/demo/src/app.tsx`	96
`apps/website/app/page.tsx`	758
`apps/website/app/claude-spinner.tsx`	38

React Hook `useEffect` is missing dependency `inputValue.trim` — list it in the dependency array, or call the hook unconditionally. _{· 2 in 2 files}

_{exhaustive-deps}

List every value the hook callback captures in its dependency array.

File	Lines
`apps/demo/src/app.tsx`	72
`apps/website/app/page.tsx`	261

Array index in `key` doesn't uniquely identify the element — re-renders may use stale state. _{· 2 in 2 files}

_{no-array-index-key}

Use a stable, data-derived key instead of the array index.

File	Lines
`apps/demo/src/app.tsx`	96
`apps/website/app/page.tsx`	758

`<button>` elements must have an explicit `type` attribute. _{· 2 in 2 files}

_{button-has-type}

Set type="button" (or "submit" / "reset") explicitly on every <button>.

File	Lines
`apps/demo/src/app.tsx`	118
`apps/website/app/page.tsx`	596

Visible non-interactive elements with click handlers must have a corresponding keyboard listener (`onKeyUp`, `onKeyDown`, or `onKeyPress`). _{· 2 in 1 file}

_{click-events-have-key-events}

Pair onClick with onKeyUp / onKeyDown / onKeyPress for keyboard users.

File	Lines
`apps/website/app/page.tsx`	579, 759

Static HTML elements with event handlers require a role — add `role="…"` or use a semantic HTML element instead. _{· 2 in 1 file}

_{no-static-element-interactions}

Static HTML elements with event handlers require a role, or use a semantic HTML element instead.

File	Lines
`apps/website/app/page.tsx`	579, 759

A control must be associated with a text label — add visible text, `aria-label`, or `aria-labelledby`. _{· 1 in 1 file}

_{control-has-associated-label}

Every interactive control must have an accessible label.

File	Lines
`apps/demo/src/app.tsx`	107

Page without metadata or generateMetadata export — hurts SEO _{· 1 in 1 file}

_{nextjs-missing-metadata}

Add export const metadata = { title: '...', description: '...' } or export async function generateMetadata()

File	Lines
`apps/website/app/page.tsx`	6

Import "m" with LazyMotion instead of "motion" — saves ~30kb in bundle size _{· 1 in 1 file}

_{use-lazy-motion}

Use import { LazyMotion, m } from "framer-motion" with domAnimation features — saves ~30kb

File	Lines
`apps/website/app/page.tsx`	9

^{Reviewed by reactreview for commit b47aab7435d488f12b3fe6c415cc92244cf5f6fb. Configure here.}

Lenguaje dominante: TypeScript
Estrellas: 352
Forks: 23
Métricas de merge de PR: Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

Lee el issue completo y luego la guía de contribución del proyecto.
Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
Haz un fork del repositorio y trabaja en una rama.
Abre un pull request que haga referencia al número del issue.

Más de millionco/debug-agent

React Review: errors & warnings on default branch
Abierto
- ¿Algo no está bien?
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100

millionco/debug-agent#2 · Abierto el 13/5/2026

Todos los issues de millionco/debug-agent

Issues similares

Replace discontinued crypto-js in the theme
Posiblemente ocupada @aelmanaa la tomó hoy. Abierto
- ¿Algo no está bien?
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100

PaloAltoNetworks/docusaurus-openapi-docs#1689 · Abierto el 31/10/2026

Los mantenedores suelen responder en 2 días
v1.x: bump jayson to ^5.0.0 to drop uuid@8 for consumers
Abierto
- ¿Algo no está bien?
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100

solana-foundation/solana-web3.js#4015 · 1 comentario · Abierto el 31/10/2026

Los mantenedores suelen responder en 1 día
formatSearchQuery drops the :present modifier, so `email:present=true` is sent as `email=true`
Abierto
- ¿Algo no está bien?
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100

medplum/medplum#11231 · Abierto el 31/10/2026

Los mantenedores suelen responder en 1 día
plugin-api-notes: record why courier_later does not use OpenCode's schedules
Abierto
- ¿Algo no está bien?
documentation

Dificultad 1/5 1-3 horas Aptitud para principiantes 90/100

ivopogace/opencode-courier#422 · Abierto el 31/10/2026

Los mantenedores suelen responder en 1 día
Error status bar item stays visible after disabling a language server
Abierto
- ¿Algo no está bien?
bug

Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100

chapel-lang/chapel-vscode#158 · Abierto el 31/10/2026

Los mantenedores suelen responder en 1 día

Más issues de TypeScript

React Review Audit

¿Algo no está bien?

Evaluación

Línea de trabajo

Descripción

❌ Errors (1)

⚠️ Warnings (22)

Preparar el entorno

Primeros pasos

Más de millionco/debug-agent

¿Algo no está bien?

Issues similares

¿Algo no está bien?

¿Algo no está bien?

¿Algo no está bien?

¿Algo no está bien?

¿Algo no está bien?

Recibe los nuevos issues en tu correo