用途:以资深工程师审阅同事 PR 的方式,对某分支相对其基线的 diff(含未提交改动)进行定性评审,判断变更是否解决正确的问题、设计是否合理、是否可维护,并给出可执行的反馈。它不充当 CI 工具:不执行测试、linter、类型检查或构建,那属于 quality-check 的范畴;本技能的 verdict 只覆盖定性层面,不说明测试是否通过或每条验收标准是否被覆盖。
主要能力:按三个维度评估 diff(而非整个仓库):一、语义分析(意图),识别声明意图与代码实际行为的偏差,如解决的不是被要求的问题、验收标准未覆盖、非预期副作用、逻辑与命名矛盾;二、架构、设计与产品质量,对照 ISO/IEC 25010 中被触及的质量特性(可维护性、可靠性、安全性、性能效率、兼容性),每个发现都标注对应特性;三、资深风格反馈,说明原因、提出具体改进,按影响排序,不过度纠缠细枝末节。
严重度与判定:CRITICAL、MAJOR 会阻塞(除非修正或理由被接受),MINOR、SUGGESTION 不阻塞。判定为 APPROVED(无未解决的阻塞项)、REJECTED(存在未修正且未说明理由的阻塞项)、INCOMPLETE(无待处理阻塞项但有维度无法评估);优先级 REJECTED > INCOMPLETE > APPROVED。阻塞发现可按配置暂停询问用户选择修正或说明理由,或直接应用建议修正后重启评审。
用法:默认评审分支相对基线的完整 diff;可用 base <rama> 指定基线,working-tree 只评审未提交改动,scope <ruta…> 限定范围。默认包含未提交内容,因为要评审的是即将集成的代码。报告写入 docs/audits/code-review.md,具有幂等性:若自上次生成后相关代码或其基线无变化,不重新评审,直接返回既有判定与摘要。最小输入是一个可识别的 diff 与变更意图。
限制:意图来源按顺序取插件工件(US-XXX/WI-XXX/FT-XXX)、其他规范文档、外部跟踪系统工单,或从分支、提交与 PR 描述推断;若无法确定且用户未提供,不得从代码本身环形推断,该维度记为 NOT_ASSESSED 并给出 INCOMPLETE。技能只分析、推理并提出建议;仅在用户明确授权或配置 verification.codeReview.confirmFix 为 never 时应用修正,无明确指示不执行 commit/push/merge。失败重试上限由 escalation.maxAttempts 决定,超限则升级而非绕过测试或放宽断言。具体配置键与完整执行步骤见资料中列出的参考文件。