| name | verify-quality-code-quality |
| description | 代码质量审查。当需要对已通过 spec compliance 的代码进行质量评估,或提到"代码质量""重构""质量审查" |
Code Quality — 代码质量审查
入口/出口
- 入口: 已通过 Spec Compliance 审查的代码(功能完整)
- 出口: 代码质量审查报告(通过/不通过 + 五轴评分 + 改进建议)
- 指向: 通过 → ship 阶段;不通过 → 退回 build 修复质量问题
- 前置加载: CANON.md, verify-team-code-review-standards
- 输出路径:
docs/features/YYYYMMDD-<name>/04-review.md(质量审查部分) → ship-workflow-ship(通过时)或 build-workflow-execute(不通过时退回修复质量问题)
何时不使用
- 代码尚未通过 Spec Compliance 审查(功能不完整时不进入质量审查)
- 非软件产物(document/article/deck/visual 使用对应的 content-review 或 visual-review)
Iron Law
代码质量审查只在 Spec Compliance 通过后进行。
功能不完整的代码不进入质量审查阶段。
质量审查关注"如何实现",不关注"实现了什么"。
五轴(Correctness/Readability/Architecture/Security/Performance)必须全部覆盖。
核心职责
评估已经功能完整的代码的实现质量。关心:逻辑正确性、可读性、架构合理性、安全措施、性能。不关心:spec 需求覆盖(Spec Compliance 职责)、功能完整性(Spec Compliance 职责)、scope creep(Spec Compliance 职责)。
审查流程
使用 verify-team-code-review-standards 技能执行五轴审查。
Step 1: 确认前置条件
确认代码已通过 Spec Compliance。接受以下证据:当前 /review 中刚完成的结果、04-review.md 中的 Stage 1 区块、或外部分派 auditor 的 PASS。不要要求 04-review.md 已存在(它通常是两阶段合并后的最终产物)。
无证据或不通过时,立即停止并告知用户先完成 Spec Compliance。
Step 2: 加载审查标准
加载 verify-team-code-review-standards,获取五轴标准:Correctness / Readability / Architecture / Security / Performance。
Step 3: 执行五轴审查
按 verify-team-code-review-standards 逐项检查。详细检查项见 rubric.md。要求:
- 每个轴必须有结论和证据引用
- Blocking / Important / Suggestion 必须分级
以下任何一个出现,立即标记为 Blocking: