Dev Phase Reviewer

Phase Reviewer

每个开发阶段完成后的强制审查关卡。根据阶段类型加载对应的审查清单，执行自动检查 + 人工审查，生成 Review 报告。

🔴 核心规则

1. 不可自审 — Reviewer 必须是与作者不同的角色
2. 严格审查 — Reviewer 必须独立、严格审查，不迁就作者。发现问题必须如实记录
3. 改了就要 Re-Review — 任何修复后必须由 Reviewer 重新验证，验证内容包括：
   • 修复是否正确解决了原问题
   • 修复是否引入了新问题（结构错误、内容矛盾等）
   • 修复与其他章节/文档的一致性
4. Review 报告持久化 — 所有 Review 结果保存到 docs/reviews/phase-{NN}-{phase_key}.md

Reviewer 分配表

Review Protocol

每个阶段完成后，按以下流程执行：

阶段执行完成
    ↓
[1] 确定 Reviewer — 查 Reviewer 分配表，不可与作者同角色
    ↓
[2] 确定 Review 类型 — 根据阶段映射表（见下方）
    ↓
[3] 自动验证 — 运行自动检查命令
    ↓
[4] 产出物审查 — 检查产出物的内容质量（不仅仅是"文件存在"）
    ↓
[5] 阶段特定 Review — 加载 references/ 中对应的审查清单并逐项检查
    ↓
[6] 生成 Review 报告 — 保存到 docs/reviews/phase-{NN}-{phase_key}.md
    ↓
[7] 问题处理:
    - 🔴 CRITICAL/HIGH → 作者修复
        ↓
    [7.1] Re-Review（强制）:
        - 验证原问题是否修复
        - 检查修复是否引入新问题
        - 检查修复与其他章节/文档一致性
        - Re-Review 结果追加到同一 Review 文件
        - 若发现新问题 → 重复 [7]
    - 🟡 MEDIUM → 记录处理计划（明确在哪个阶段解决），用户确认后可继续
    - 🟢 全部通过 → 标记阶段完成
    ↓
[8] 用户确认 — 展示结果，用户确认后进入下一阶段

阶段 → Review 类型映射

自动验证命令

在人工审查之前，先运行自动检查：

Review 报告格式

Review 报告保存路径: docs/reviews/phase-{NN}-{phase_key}.md

命名规则: phase-{序号:02d}-{phase_key}.md，例如 phase-04-architecture.md

# Phase Review: {phase_name}

**Review 类型**: {文档类/代码类/测试类/部署类}
**执行时间**: {timestamp}
**产出物**: `{output_path}`
**作者**: {author_name} ({author_role})
**审查人**: {reviewer_name} ({reviewer_role})

---

## 自动检查

- ✅/❌ [检查项]: [结果]

## 审查清单

### 通用检查 — 完整性

- ✅ [检查项]: 通过
- ⚠️ [检查项]: 建议优化 — [描述]
- ❌ [检查项]: 必须修复 — [描述]

### {phase_key} 特定检查

- ...

## 发现的问题

| #   | 严重度   | 描述 | 位置      | 建议修复 |
| --- | -------- | ---- | --------- | -------- |
| 1   | CRITICAL | ...  | file:line | ...      |
| 2   | HIGH     | ...  | file:line | ...      |
| 3   | MEDIUM   | ...  | file:line | ...      |

## 结论 (初审)

- 🟢 通过 — 可以进入下一阶段
- 🟡 条件通过 — 有 MEDIUM 问题，建议修复但可继续
- 🔴 阻止 — 有 CRITICAL/HIGH 问题，必须修复后重新 Review

## 统计 (初审)

- 检查项总数: X
- 通过: X | 警告: X | 失败: X

---

## 🔄 Re-Review ({timestamp})

### HIGH 问题修复验证

| #   | 问题 | 修复状态 | 验证     |
| --- | ---- | -------- | -------- |
| 1   | ...  | ✅/❌    | 验证描述 |

### 修复引入的新问题

| #   | 严重度 | 描述     | 建议修复 |
| --- | ------ | -------- | -------- |
| -   | -      | 无 / ... | ...      |

### MEDIUM 问题处理计划

| #   | 问题 | 处理方式              |
| --- | ---- | --------------------- |
| 1   | ...  | → 在 {phase} 阶段解决 |

## Re-Review 结论

- 🟢 通过 / 🔴 仍有问题需继续修复

## 最终统计

- 检查项总数: X
- 通过: X | 待后续: X | 已修复: X

Instructions

1. Read the current phase from .dev-state.yaml
2. Determine the Reviewer using the Reviewer Assignment Table — Reviewer MUST differ from author
3. Determine the review type using the mapping table
4. Run automatic validations first
5. Load the corresponding checklist from references/
6. Execute strict review — each checklist item against the phase outputs, being honest and rigorous
7. Generate the review report and save to docs/reviews/phase-{NN}-{phase_key}.md
8. If HIGH/CRITICAL found: a. Author fixes the issues b. Re-Review is mandatory — verify fixes AND check for newly introduced issues c. Append Re-Review results to the same review file d. If Re-Review finds new issues, repeat from 8a
9. Update .dev-state.yaml with review field pointing to the review file
10. Present results to user for confirmation