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。
Datos de origen
- Repositorio
- bing-framework/Bing.Offices
- Última actividad en el origen
- 7 de septiembre de 2026 a las 07:52
- Idioma detectado de SKILL.md
- chino
- Estrellas
- 16
- Forks
- 9
Opciones de instalación
De forma predeterminada está seleccionado el prompt que primero revisa el origen. Puedes cambiar a un comando directo o descargar una copia local.
Revisa los archivos de origen
Lee SKILL.md y los archivos complementarios que muestra SkillsMP antes de decidir si quieres instalarlo.
Mostrando SKILL.md
SKILL.md
Instrucciones de origen · Vista previa de solo lectura- name
- fix-review
- description
- 根据 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。
# 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>
```
Ver en GitHub