moon-web: Skill for Cursor

代码审核

对当前 Git 变更 进行审核：排查潜在 bug、评估代码质量，并严格按 React 规范与 Ant Design v6 检查风格与实现。发现问题必须给出修正方案或直接修正。

触发场景

• 用户说「帮我审核当前变更」
• 用户说「帮我审核代码」

审核范围：仅针对 当前未提交的变更（git diff / staged 文件）。先获取变更内容再开始审核。

审核维度

1. 正确性与 Bug

• 逻辑是否正确，边界与空值是否处理。
• 异步请求：是否有卸载后 setState、是否缺少 loading/error 处理。
• 类型：是否滥用 any、类型断言是否合理、与 API 类型是否一致。枚举一致性：若 API 已改为枚举类型（如 GlobalStatus、DatasourceType、SampleMode、ConditionMetric），则：（1）所有调用处、回调参数、setState/表单状态 均使用该枚举类型，不得遗留 string/number 或 modeRaw/conditionRaw 等原始值落库到状态；（2）提交请求参数 直接传枚举值，不得对枚举字段使用 String()/Number() 转换（例如 mode: editingLevelData.mode 而非 String(editingLevelData.mode)）；（3）表单收集枚举字段 时用 Select + 枚举 options，勿用 Input type="number" 或裸字符串再转 Number。
• 依赖数组与副作用：useEffect/useMemo/useCallback 依赖是否完整、是否造成多余渲染或闭包陈旧。

2. React 规范（必须严格遵守）

• 组件：使用 React.FC<Props> 或 function Component(props: Props)，Props 用 interface 明确定义。
• 类型：优先 import type，状态与 API 返回类型一致。
• Hooks：只在顶层调用，条件/循环中不调用；自定义 Hook 命名以 use 开头。
• 列表 key：使用稳定、唯一标识，禁止用 index 作为 key（列表会增删重排时）。
• 受控/非受控：表单与可编辑 UI 的受控方式一致，无受控/非受控混用导致的警告或 bug。
• 参考项目内 react-antd-page 中的结构与类型、请求与状态等约定。

3. 复用与冗余（硬性要求）

• 能复用的必须复用：若变更中有与 src/components/ 或同项目其他模块可复用的逻辑/组件，必须改为复用，不得重复实现。
• 禁止额外创造不必要的东西：不新增重复工具函数、不新增与现有组件功能重叠的组件；若现有 API 模块（src/api/）已有对应接口与类型，必须使用，不得在页面内重复定义请求或类型。

4. Ant Design v6 与 UI 风格（必须严格遵守）

• 仅使用 Ant Design v6 当前推荐的组件与 API，以 官方文档 为准。
• 禁止使用已弃用的组件、属性或变量（如文档标注废弃的 List 等，需改用推荐替代方案）。遇控制台弃用警告须按提示替换，例如：Divider 的 orientationMargin 已废弃，须用 styles.content 的逻辑外边距替代——titlePlacement='left' 时用 styles={{ content: { marginInlineStart: 0 } }}，titlePlacement='right' 时用 marginInlineEnd: 0（与 antd 源码中 innerStyle 一致），勿仅写 margin: 0。
• Table 的 rowKey 若为函数，不得使用第二个参数 index（v6 已废弃且行为不保证）；须用行的稳定字段（如 record.uid、record.id 或组合字段）生成唯一 key。
• 按需从 antd 引入组件，图标从 @ant-design/icons 按需引入。
• 表格列使用 ColumnsType<YourRowType>，类型与 API 的 Item/Row 一致。
• 若项目已配置 ConfigProvider/App，主题与国际化保持一致。