3.2 KiB
3.2 KiB
审查 openspec apply 完成后的实现是否与变更文档一致，按以下流程执行。
约束
- 仅审查代码和文档，不修改源码
- 每批修改建议执行前用提问工具获得用户确认
- 涉及删除/重写操作前必须备份原文件
1. 收集
并行读取：
- 本次变更涉及的所有文档（proposal.md、design.md、tasks.md、specs/*.md）
- 实际变更的源码文件
- 测试文件和测试结果
- openspec/config.yaml
2. 分析
根据上下文判断 apply 后是否有手动改动，将实现与文档双向对照：
| 维度 | 检查点 |
|---|---|
| 目标覆盖 | 代码实现是否覆盖 proposal 中的所有目标；是否有遗漏的功能点 |
| 方案一致性 | 实现是否与 design 中的技术方案一致；是否有偏离设计的地方 |
| 规范遵循 | 代码是否遵循 specs 中的规范要求；是否有违反 SHALL 约束的地方 |
| 任务完成度 | tasks 中每项任务是否真正完成；是否有未完成但标记完成的任务 |
| 测试完整性 | 测试是否覆盖所有场景；是否有跳过、取消、降低难度的测试；测试是否真正验证了功能 |
| 代码质量 | 是否有明显的代码问题（重复、复杂度过高、命名不清等）；是否有可优化的地方 |
重点识别：
- 文档要求但未实现的功能 → 需补充代码
- 实现与文档描述不一致的地方 → 根据上下文判断：apply 后手动改动则更新文档，否则确认修正方向
- 实现了但文档未提及的功能 → 根据上下文判断：apply 后手动改动则补充文档，否则标记为未讨论的新增
- 标记完成但实际未完成的任务
- 掩盖错误的测试（skip、only、降低断言强度等）
输出审查结果：
- 问题总览表：问题类型 × 涉及文件数
- 逐项分析：每个问题文件，说明问题、影响和建议
- 未覆盖清单：哪些文档要求未在代码中实现（需补充代码）
- 不一致清单：哪些实现与文档描述不一致（需确认修正方向）
- 需补充文档清单：哪些代码改动未在文档中体现（需补充文档）
- 任务问题清单：哪些任务未真正完成或标记错误
- 测试问题清单：哪些测试存在问题或掩盖了错误
- 优化建议清单：哪些代码可以优化
若所有清单均为空，输出"审查通过，未发现问题"，跳至步骤 5。
3. 计划（用户确认）
针对发现的问题，分类提出修复方案：
需补充代码：文档要求但未实现的功能，建议补充实现。
需补充文档：代码已实现但文档未记录，且上下文确认为 apply 后手动改动，建议更新 proposal/design/tasks/specs。
需确认修正方向：实现与文档不一致，且上下文无法判断：
- 若为 apply 后手动改动 → 更新文档
- 否则用提问工具确认以文档还是代码为准
任务和测试问题：逐项说明未完成原因、测试掩盖错误的风险，提出修复方案。
用提问工具展示完整修复方案，获得用户确认后执行。
4. 执行
逐项执行修复方案。
5. 收尾
列出所有修改的文件和变更摘要。