Peter Code Review

---

name: peter-code-review

description: 提交前质量闸门。快速完成本地测试、静态检查和风险审查，判断“是否可提交”。

version: 1.5.0

---

# Peter Code Review

30 秒简介

用于“提交前最后一关”。

它会基于当前改动，执行最小但有效的验证闭环：

1. 识别改动范围

2. 运行基础检查（lint/type/test）

3. 对 UI、配置、依赖变更做附加检查

4. 输出可提交结论与修复清单

适用场景

使用前提

执行步骤

1) 识别改动范围

git status -s
git diff --stat
git diff --name-only

若 `git status -s` 为空，执行：

git show --name-only --stat --pretty="" HEAD

并在报告中标注“基于 HEAD 审查，非工作区审查”。

2) 运行基础质量检查（按技术栈选择）

- 必跑：`npm run lint`、`npx tsc --noEmit`

- 优先执行非端到端测试（如 `npm run test:unit`、`npm run test:integration`）

- 若仓库测试入口仅为端到端测试，则跳过测试并记录“未执行原因 + 影响范围”

如果命令不存在或环境缺失，必须明确记录“未执行原因 + 影响范围”。

3) 条件触发附加检查

- `package.json`、`pnpm-lock.yaml`、`yarn.lock`

- `next.config.*`、`vite.config.*`、`webpack*.js`

- `prisma/`、`src/app/api/`、CI/构建脚本目录

1. `npm run workflow:check`（若命令存在）

2. `npm run build`

3. `npm run gate:db`（若命令存在）

- 标记为 `High` 风险，禁止标记为“通过”

- 总结论允许为 `可提交（高风险）`

- 必须附带“PR 阶段需强制复核（建议立刻执行 `peter-ci-gate`）”

4) UI 改动验证策略

检测 UI 改动：

git diff --name-only \
  | grep -E '\.(tsx|jsx|css|scss)$|^src/components/|^src/app/' \
  | grep -Ev '^src/app/api/'

- 关键页面可打开且无明显报错

- 关键交互可触发且无阻塞

- 关键文案/布局无明显回归

- 未完成手工 UI 验证时，不得把 UI 项标记为“通过”；应标记为“未执行（有风险）”。

- 总结论可给出“可提交（含 UI 未验证风险）”或“需修复后提交”，由改动风险决定。

5) 安全与实现质量审查

至少覆盖以下检查点：

6) 需求完成度核对

对照用户需求逐条标注：

并给出对应文件位置。

输出格式（固定）

1. `## 代码审查报告`

2. `### 提交标准检查清单`（lint / typecheck / test / workflow-check(如执行) / gate:db(如执行) / build(如执行) / 安全 / 需求 / UI(通过/未执行)）

3. `### Git 改动摘要`（文件数、关键文件）

4. `### 执行结果`（每条命令是否通过，失败摘录）

5. `### 问题列表`（Critical / High / Medium / Low，含文件:行号、原因、修复建议）

6. `### 结论`（`可提交` / `可提交（高风险）` / `需修复后提交`）

护栏