| name | review4-execute |
| description | [Defender] Read a finalized review document, implement 已共识·实施 items after explicit user confirmation, and skip 已共识·存档 items. Refuse to proceed if any 需补充回复 or 待裁决 items exist. Part of a 5-stage adversarial ping-pong review: raise(R) → reply(D) → confirm(R) → execute(D) → accept(R). Requires the user to explicitly provide the document path. |
review4-execute — 达成共识后实施
消费 review 文档，只实施 已共识·实施 的条目，跳过 已共识·存档，拒绝任何未闭环条目。本阶段是纯粹的实施阶段，不做任何仲裁——分歧和澄清均已在 review3-confirm 闭环。
角色
Defender Agent — 本阶段由 Defender 执行，是五阶段对抗性 ping-pong 流程的第四棒：
raise(R) → reply(D) → confirm(R) → execute(D) → accept(R)
对抗假设：Reviewer 将在 review5-accept 逐条精确对照实施结果。改动要清晰、可追溯，严格在共识范围内——不扩大（不借机重构无关代码）、不缩小（不遗漏约定的改点）。预设 Reviewer 会发现偏差并退回，因此实施前先在上下文中逐条列出每个 Rn 的具体改点，再动手。
参数要求
必须由用户显式提供文档路径。若未提供，提示用户：
请提供 review 文档路径，例如：review4-execute .temp/review-250416-1430-persona.md
步骤
- 门禁 — 重新读取文档并核验阶段状态：
- 必须先调用 Read 工具重新读取文档，禁止依赖上下文中已有的旧版本——文档可能已被其他 Agent/会话更新
- 逐项检查「阶段状态」checklist
- 阶段 3 未勾选 → 禁止继续，提示用户先完成
review3-confirm
- 禁止凭跨会话记忆判断前置条件——以本次 Read 返回的文件内容为唯一依据
- 实施完成后调
set-stage 标记阶段 4，其内部自动校验阶段 3（门禁不通过直接 exit 1）
- 调用脚本读取文档到上下文（若步骤 0 已在门禁中完成读取，可跳过）：
python .claude/skills/review1-raise/review_record.py read <filename>
- 解析所有
Rn 的 共识状态，生成实施清单：
已共识·实施：列出 Confirm/Reply 中约定的具体改点，纳入实施已共识·否决：Defender 不采纳且已被接受，无需代码改动，跳过已共识·延后：已归档至 backlog，无需代码改动，跳过（实施时提示"已归档至 backlog: B-..."）已共识·存档（旧状态，兼容）：同 已共识·否决，跳过需补充回复：拒绝实施，提示用户先完成 review2-reply → review3-confirm待裁决（异常）：拒绝实施，提示用户先完成 review3-confirm
- 若存在拒绝条目，停止并告知用户；若全部可处理，与用户确认实施范围（逐条或批量均可）
- 门禁：仅当用户明确授权后才动手
- 实施优先级：除非条目间存在依赖关系（如某条修改必须先完成另一条），否则优先实施重要程度和复杂程度高的修改。简单修改可以放在后面处理，但不得遗漏
- 修改代码，严重问题的修复遵循 TDD 顺序：
- 对
已共识·实施 的严重问题，若 Confirm/Reply 中约定了回归测试（未明确否决即视为约定），按以下顺序实施：
- 先写失败的测试：根据约定的测试场景编写测试用例，确认测试因 bug 存在而失败
- 再修复代码：修改业务代码使测试通过
- 确认测试通过：运行该测试确认修复有效
- 若 Defender 在 Reply 中已说明不适合添加测试且 Reviewer 在 Confirm 中已确认，跳过测试编写，直接修复代码
- 非严重问题及未涉及测试约定的条目，直接修改代码即可
- 跑项目配套的测试命令验证（如构建验证、单元测试等，依项目实际情况而定），确保不引入回归
- 向用户汇报完成情况和测试结果
- 标记阶段 4 完成：
python .claude/skills/review1-raise/review_record.py set-stage <filename> 4
约束
- 未获明确授权前禁止改代码
- 实施范围不得超过用户确认的子集
- 实施完成后禁止提交代码，必须等待 review5-accept 验收通过后再 commit——提前提交会导致改动脱离
git diff HEAD 的追踪范围，review5-accept 将无法正确验收
- 修改后必须跑测试
