React Review Audit
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 48/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 冷清
- 技术栈
- nextjs, react, typescript
调研方向
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.
由索引模型根据 Issue 内容生成。
描述
| ❌ 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}orkey={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
keyinstead 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
onClickwithonKeyUp/onKeyDown/onKeyPressfor 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: '...' }orexport 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"withdomAnimationfeatures — saves ~30kb
| File | Lines |
|---|---|
apps/website/app/page.tsx |
9 |
Reviewed by reactreview for commit b47aab7435d488f12b3fe6c415cc92244cf5f6fb. Configure here.
- 主要语言
- TypeScript
- 星标
- 351
- 派生
- 23
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
millionco/debug-agent 的其他 Issue
-
难度 4/5 3-5 天 新手友好度 45/100
millionco/debug-agent#2 ·
查看 millionco/debug-agent 的全部 Issue
相似的 Issue
-
难度 1/5 1-3 小时 新手友好度 82/100
source-cooperative/docs.source.coop#41 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 76/100
Nimblesite/SharpLsp#468 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 85/100
solidjs-community/solid-primitives#1082 ·
维护者通常 1 天内回复
-
enhancement
难度 2/5 1-3 小时 新手友好度 64/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复