用途:review-code 是 agentic-workflow 评审包中的内部步骤,由 review-change 和 product-audit 在对话中组合调用,不作为菜单入口。它在任意 agent 上以内联方式按文件执行,只产出审查发现,从不编辑或重构代码。
主要能力:针对调用方传入的 diff,或路径/glob 进行审查,默认对比当前改动与默认分支,并在返回表格顶部声明范围。检查清单要求逐项评估,缺一不可,不适用的项必须说明原因:改动路径无逻辑错误(含边界值追踪);所有错误路径被处理,无吞异常、空 catch、静默忽略返回码;无重复逻辑(引用已有 helper);未引入死代码(未用函数、参数、导入、不可达分支);diff 中无遗留 TODO/FIXME/HACK;命名与文件约定符合项目文档(先读文档并引用被违反的约定);不超出 SPEC 要求的新抽象(单一实现的接口/基类算发现);无 SPEC 未说明理由的新依赖;可简化的改动块需给出前后对比;每个改动符号/API 需做引用搜索,未更新调用方算发现;SPEC 开发场景所述的边界情况需在代码中处理,而非只在测试中。
使用方式:调用方传入 diff 或路径/glob,agent 按该文件内联执行。返回固定格式:REVIEW CODE — scope、发现表格(#、Finding、Sev、Evidence、Suggested fix)、Checklist 统计、Summary、Decision: PASS 或 FAIL。
限制与判定:仅当合格用户的结果会改变,或项目明确声明的规则被违反时才记录一行,并附违规规则与证据;注释/标点错字、纯格式漂移、无引用规则的风格偏好、超出 SPEC 场景的假设性健壮性不算发现。格式良好的改动应返回空表且 Decision: PASS,不得凑行。存在未解决的 critical 或 major 发现则 FAIL,否则 PASS;minor 不阻塞,交调用方分流。完成条件:每项检查均以证据或明确 n/a 及原因评估,返回上述固定格式块,且不改动任何代码。