# Review 清单

用于审查执行 Agent 完成后的结果。目标不是追求完美，而是确认当前任务安全、符合范围、可验收。

## 一、范围检查

| 检查项 | 结论 |
|---|---|
| 是否只修改任务指定文件/模块 |  |
| 是否新增了未说明的文件 |  |
| 是否修改了业务无关文件 |  |
| 是否出现无关重构 |  |
| 是否修改了文档/配置/数据但任务未要求 |  |

## 二、主链路影响

| 检查项 | 结论 |
|---|---|
| 是否影响用户核心路径 |  |
| 是否影响已有数据结构 |  |
| 是否影响状态保存/恢复 |  |
| 是否影响权限、登录、支付、分享、导出等敏感能力 |  |
| 是否影响外部接口或第三方服务 |  |

## 三、兼容性与边界

| 检查项 | 结论 |
|---|---|
| 是否兼容旧数据 |  |
| 是否处理空值 / undefined / null |  |
| 是否处理空列表、重复项、非法输入 |  |
| 是否引入并发、重复提交或重复触发问题 |  |
| 是否需要考虑移动端 / 浏览器 / 小程序 / 后端运行环境差异 |  |

## 四、代码质量

| 检查项 | 结论 |
|---|---|
| 是否复用已有工具函数或组件 |  |
| 是否新增重复逻辑 |  |
| 命名是否清楚 |  |
| 注释是否只解释关键逻辑 |  |
| 是否存在过度抽象 |  |

## 五、自测与验收

| 检查项 | 结论 |
|---|---|
| 是否运行必要语法检查 / 测试 / 构建 |  |
| 自测结果是否明确 |  |
| 是否列出人工验证路径 |  |
| 是否说明风险点 |  |
| 是否需要补充回归路径 |  |

## 审查输出格式

```markdown
| # | 位置 | 等级 | 问题 | 建议 |
|---|---|---|---|---|
| 1 | 文件/函数/模块 | 必改/建议/可选/通过 | 说明问题 | 给出建议 |
```

等级说明：

| 等级 | 含义 |
|---|---|
| 必改 | 不改会导致 bug、数据污染、主链路异常、安全风险或明显体验问题 |
| 建议 | 质量或维护性问题，可以择机处理 |
| 可选 | 可做可不做，不影响当前任务 |
| 通过 | 当前项无问题 |

## 收口判断

| 情况 | 处理 |
|---|---|
| 无必改项，用户验收通过 | 收口 |
| 有必改项 | 生成一个新的原子任务修复 |
| 只有建议项 | 询问用户是否继续，不要默认扩大范围 |
| 发现新技术债 | 登记到技术债清单，不要顺手做 |