peter-code-review

# Peter Code Review

## 30 秒简介
用于“提交前最后一关”。

它会基于当前改动，执行最小但有效的验证闭环：
1. 识别改动范围
2. 运行基础检查（lint/type/test）
3. 对 UI、配置、依赖变更做附加检查
4. 输出可提交结论与修复清单

## 适用场景
- 用户提到“提交前检查”“测试一下再 commit”“pre-commit review”
- 准备开 PR 前，希望先清理明显问题

## 使用前提
- 在仓库根目录运行。
- 默认审查“当前工作区改动”；如果工作区为空，则回退审查 `HEAD` 最近一次提交，并在报告中明确标注。

## 执行步骤
### 1) 识别改动范围
```bash
git status -s
git diff --stat
git diff --name-only
```

若 `git status -s` 为空，执行：
```bash
git show --name-only --stat --pretty="" HEAD
```
并在报告中标注“基于 HEAD 审查，非工作区审查”。

### 2) 运行基础质量检查（按技术栈选择）
- Node/TS：
  - 必跑：`npm run lint`、`npx tsc --noEmit`
  - 优先执行非端到端测试（如 `npm run test:unit`、`npm run test:integration`）
  - 若仓库测试入口仅为端到端测试，则跳过测试并记录“未执行原因 + 影响范围”
- Python：`ruff check .`、`pytest`
- Go：`go test ./...`
- Rust：`cargo test`

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

### 3) 条件触发附加检查
- 构建配置/依赖变更时执行 build。
- 推荐触发模式：
  - `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`（若命令存在）
- 若 `workflow:check` / `gate:db` 命令不存在或执行环境缺失，必须记录“未执行原因 + 影响范围”，且不得标记为通过。
- 若 `gate:db` 输出包含 `GATE_DB_UNREACHABLE`（或等价 DB 不可达证据，如 `P1001`）：
  - 标记为 `High` 风险，禁止标记为“通过”
  - 总结论允许为 `可提交（高风险）`
  - 必须附带“PR 阶段需强制复核（建议立刻执行 `peter-ci-gate`）”

### 4) UI 改动验证策略
检测 UI 改动：
```bash
git diff --name-only \
  | grep -E '\.(tsx|jsx|css|scss)$|^src/components/|^src/app/' \
  | grep -Ev '^src/app/api/'
```

- 若存在 UI 改动，优先执行“可运行的最小验证...