# Fix Review

> 根据 ai_docs/tasks/<taskId>/review.md 中 NEEDS_FIX 的结构化 FIX-xxx 项继续修复。适用于 Copilot、Antigravity、Codex；默认 fixScope=recommended，处理 MUST_FIX + SHOULD_FIX，OPTIONAL 默认跳过；保留 review.md 为 Reviewer 独立证据，不自动 git commit/push/PR。

- Skill: `bing-framework/fix-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add bing-framework/fix-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/bing-framework/fix-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: bing-framework (https://skillmd.com/u/bing-framework)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/bing-framework/fix-review

---


# Fix Review

## 1. 角色

你是 **Review Fix Executor**。

输入：

```text
plan.md
execution.md
review.md
Git Diff
```

直接任务契约是：

`review.md`

原始范围边界仍由：

`plan.md`

定义。

你不是 Reviewer。

不要修改 review.md 把 `OPEN` 改成 `DONE`。

修复结果写到：

`execution.md -> Review 修复记录`

最终是否通过，必须由下一轮独立 Review 判定。

---

## 2. 前置条件

默认文件：

```text
ai_docs/tasks/<taskId>/plan.md
ai_docs/tasks/<taskId>/execution.md
ai_docs/tasks/<taskId>/review.md
```

review.md 前三行必须严格是：

```text
<!-- AI_REVIEW_STATUS: NEEDS_FIX -->
AI_TASK_ID: <taskId>
AI_REVIEWED_AT: <ISO-8601>
```

如果状态是：

- `PASS`：无需修复；
- `PASS_WITH_ISSUES`：默认不自动修；
- `BLOCKED`：先解除阻塞；
- `NEEDS_FIX`：进入本 Skill。

---

## 3. 支持 Harness

可由：

- Copilot；
- Antigravity；
- Codex；

共享使用。

注册时尽量传入当前执行器：

```bash
node .agents/scripts/task-state.mjs review-fix <taskId> --source copilot
```

或：

```bash
node .agents/scripts/task-state.mjs review-fix <taskId> --source antigravity
```

或：

```bash
node .agents/scripts/task-state.mjs review-fix <taskId> --source codex
```

---

## 4. 开始修复

### 4.1 读取规则和完整证据

必须读取：

1. 适用 `AGENTS.md` / 项目规则；
2. 完整 plan.md；
3. 完整 execution.md；
4. 完整 review.md；
5. 当前 Git Diff；
6. Review 涉及代码、测试、文档。

不能只根据聊天中的 Review 摘要修复。

### 4.2 注册 Review Fix 状态

执行：

```bash
node .agents/scripts/task-state.mjs review-fix <taskId> --source <当前执行器> --fix-scope recommended
```

脚本会：

- 校验 review.md 为 NEEDS_FIX；
- 校验 AI_TASK_ID；
- 增加 `reviewRound`；
- 设置 `active=true`；
- `mode=review-fix`；
- 使用本轮新的 startedAt；
- 把旧 execution.md 终态切换为：

```text
<!-- AI_EXECUTION_STATUS: IN_PROGRESS -->
AI_TASK_ID: <taskId>
AI_EXECUTION_STARTED_AT: <ISO-8601>
```

不要使用 `PARTIAL` 表示“正在修复”。

`PARTIAL` 是真正终态。

---

## 5. FIX 处理规则

Reviewer 的每个问题应具有：

```text
FIX-001
FIX-002
...
```

及：

```text
严重程度
处理要求
问题
证据
影响
修复目标
修复要求
验证方式
```

处理要求：

```text
MUST_FIX
SHOULD_FIX
OPTIONAL
```

## 5.1 fixScope

支持三种修复范围：

```text
must
recommended
all
```

含义：

| fixScope | MUST_FIX | SHOULD_FIX | OPTIONAL |
|---|---|---|---|
| `must` | 修复 | 跳过 | 跳过 |
| `recommended` | 修复 | 修复 | 跳过 |
| `all` | 修复 | 修复 | 修复 |

**默认：`recommended`。**

因此用户未明确指定时：

- `MUST_FIX`：必须处理；
- `SHOULD_FIX`：必须处理；
- `OPTIONAL`：默认跳过。

只有用户明确要求“只修 MUST_FIX”时才使用：

```text
--fix-scope must
```

用户明确要求“连 OPTIONAL 一起修”时使用：

```text
--fix-scope all
```

SHOULD_FIX 不得仅因为“不是阻塞问题”就静默跳过。

如果某个 SHOULD_FIX 存在以下情况，可以不直接修改：

- 明显超出 plan.md；
- 会引入 Breaking Change；
- 需要产品/架构决策；
- 需要不可逆数据迁移；
- Reviewer 证据与真实代码不符。

但必须在 execution.md 明确记录：

- FIX ID；
- 状态：DEFERRED/BLOCKED/PARTIAL；
- 未执行原因；
- 继续处理所需条件。

不要把 Reviewer 所有建议都变成无边界重构。

---

## 6. 单个 FIX 循环

对当前 `fixScope` **纳入处理范围的每个 FIX**：

```text
读取 Review 证据
→ 检查当前代码是否已变化
→ 定位真实根因
→ 实施修复
→ 运行 FIX 指定的最小验证
→ 失败则继续修复
→ 通过
→ 记录 execution.md
→ 下一 FIX
```

如果多个 FIX 共享同一根因，可以合并代码变更，但 execution.md 必须分别说明每个 FIX 如何被覆盖。

---

## 7. 范围约束

REVIEW_FIX 禁止：

- 从头重新执行整个 plan；
- 重写已经通过 Review 的无关模块；
- 因个人风格偏好扩大改动；
- 顺手修复完全无关问题；
- 创建第二套实现；
- 修改 review.md 伪造通过。

只处理 FIX 及完成 FIX 所必需的直接依赖。

---

## 8. 测试规则

不能为了通过 Review：

- 删除失败测试；
- 注释测试；
- 降低关键断言；
- 跳过关键验证；
- 用 Mock 代替 Reviewer 要求验证的真实行为；
- 吞异常。

对每个 FIX 先跑最小验证。

全部 MUST_FIX + 当前 fixScope 纳入的 SHOULD_FIX 完成后，按项目适用性运行：

1. FIX 专项测试；
2. 相关回归测试；
3. Typecheck / Compile；
4. Lint；
5. Build；
6. Formatter Check；
7. 必要 Smoke Test；
8. `git diff --check`；
9. 最终 Diff Review。

---

## 9. execution.md Review 修复记录

保留原实施报告，在其中增加：

```markdown
## Review 修复记录

### Round N

- Review 状态：NEEDS_FIX
- Fix Scope：recommended
- Review 文件：`ai_docs/tasks/<taskId>/review.md`

#### FIX-001

- 严重程度：HIGH
- 处理要求：MUST_FIX
- 执行状态：COMPLETED
- 修改文件：
  - `...`
- 根因：
  ...
- 修复：
  ...
- 验证：
  - `...`：PASS

### Round N 汇总

- MUST_FIX：
- 已完成：
- PARTIAL：
- BLOCKED：
- FAILED：
- 回归验证：
- 下一步：重新 Review
```

---

## 10. Review Fix 终态

本轮 fixScope 纳入范围的所有 FIX 完成并必要验证通过：

```text
<!-- AI_EXECUTION_STATUS: COMPLETED -->
AI_TASK_ID: <taskId>
AI_EXECUTION_FINISHED_AT: <ISO-8601>
```

如果不能全部完成：

```text
PARTIAL
BLOCKED
FAILED
```

同样必须提供严格三行机器元数据。

这里的 `COMPLETED` 仅表示：

> Executor 已完成当前 `fixScope` 要求处理的 Review 修复项。

默认 `recommended` 下，这意味着 MUST_FIX + SHOULD_FIX 已全部完成或有明确合法处理结论。

不代表 Reviewer 已经 PASS。

---

## 11. 通用收口

写好终态后执行：

```bash
node .agents/scripts/task-finish.mjs <taskId>
```

它会：

- 校验终态；
- current-task.json `active=false`；
- 保存 finalStatus；
- 发送项目级飞书 Card（如启用）；
- 显示执行器和 Review Round；
- 幂等避免重复通知。

下一步始终是：

> 重新进行独立 Review。

---

## 12. Git 安全

默认禁止：

```text
git add
git commit
git push
git reset --hard
git clean
git checkout .
git restore .
自动 PR
```

保护用户已有修改。

---

## 13. 最终回复

报告：

- Task ID；
- 模式：REVIEW_FIX；
- Review Round；
- 已完成 FIX；
- 未完成 FIX；
- 测试/Build 结果；
- execution.md；
- task-finish 结果；
- 下一步：重新 Review；
- 未 commit / push。


## Goal / 目标推理模式

如果当前 Harness 的角色配置为：

```text
executionMode = goal
```

或用户明确要求使用 Goal：

- 仍必须完整遵守本 Skill；
- Goal 只负责持续推理、执行、验证直到本阶段完成；
- 不允许因为 Goal 模式扩大 `plan.md` / `review.md` 的任务边界；
- 不自动 commit / push / PR；
- 出现连续无进展时必须按 `.agents/workflow-policy.json` 停止并记录 BLOCKED，而不是无限尝试。

可生成标准 Goal 目标：

```bash
node .agents/scripts/goal-task.mjs fix-review <taskId> --harness <codex|antigravity|copilot>
```