用途
本技能用于审查已有的代码变更,而非从零编写新功能。主要任务包括:审查 PR、合并请求或 diff;发现正确性、安全、可靠性或可维护性问题;评估测试缺口与回归风险;在合并前评审 AI 生成或智能体创建的变更;建议保持行为不变的小型重构。
不适用场景:全新架构设计、深度 AppSec 设计或正式威胁建模、需要实测的性能调查、从零编写新功能、选择或启动审查智能体团队。
主要能力
- 按关注点提供审查清单与严重性分级标准。
- 按技术栈分流:前端、后端/API、移动端、基础设施、区块链、数据/ML。
- 按平台分流:GitHub、GitLab、Bitbucket 的原生审查与自动化控制。
- 提供 AI 生成代码的专项检查清单(如幻觉导入、过时 API、安全缺口、缺失错误处理、冗余抽象、复制粘贴漂移、测试表演、无障碍遗漏、设计系统违规、自信但错误的注释)。
- 提供严重性分级(P0 BLOCKER、P1 REQUIRED、P2 SUGGESTED、P3 OPTIONAL)与校准规则。
- 提供橡皮图章式审查的识别信号、测试代码审查要点、常见陷阱与反模式。
- 提供默认审查输出格式与评论意图标注约定(Conventional Comments)。
使用方式
工作流程:确认审查范围(diff、PR、合并请求、生成代码或指定文件集);将纯架构或正式安全设计问题转交相邻技能;追踪变更的控制流与数据契约,按顺序应用审查模式:正确性、安全、可靠性、性能、可维护性;仅在代码确实需要时引入技术栈叠加层;给出带证据的具体发现,并通过导航来源核实平台相关声明。
审查模式顺序:正确性与边界情况;安全与数据处理(追踪可控输入到 diff 中的每个汇聚点);可靠性与可运维性;性能与成本(仅限 diff 中可见的热路径回归);可维护性与测试覆盖。
默认输出:意图与风险的简短摘要;按 P0/P1/P2/P3 分组的发现;REQUIRED 与 OPTIONAL 标注;具体修复建议(最小 diff、测试用例或配置修复);仅在需求确实不清时提出后续问题。评论意图使用 Conventional Comments 约定标注(suggestion:、issue:、question:、nitpick:、praise:)。
适用场景
适用于审查 PR、合并请求或 diff,发现正确性、安全、可靠性、可维护性问题,评估测试缺口与回归风险,评审 AI 生成或智能体创建的变更,以及建议保持行为不变的小型重构。
限制与规则
- 人工审查为权威,AI 发现在被确认前仅为建议。
- AI 生成代码与智能体 PR 应视为更高上下文审查任务,而非更低工作量任务。
- 优先使用平台原生控制(仓库或路径级审查指令、排除文件/生成文件过滤、必需检查、审批规则、合并队列),而非第三方机器人。
- 不得凭记忆接受基准、定价或功能对比声明,须先核实当前工具状态。
- 不得将特定 AI 模型版本/ID 作为面向审查者的事实引用。
- 报告阻塞或高严重性发现前须提供最小证明(可达执行路径、失败示例或测试、具体契约不匹配)。
- 不确定的发现应作为问题提出,按其若为真时的严重性标注,不得为回避摩擦而降至 P3,也不得凭猜测断言 P0。
- 无实测的性能发现上限为 P2,除非 diff 在热路径引入明显无界操作。
- 不应阻塞合并的情形:仅 P3 发现、无缺陷的写法差异、低风险低变更频率代码的测试缺口(作者明确承认且已跟踪)、diff 影响范围外的既有问题、实为产品/需求分歧的问题。
- 应阻塞合并的情形:P0/P1 发现、真正有风险的新行为缺失测试、合并后修复昂贵或不安全的事项(数据迁移、公共 API 形态、安全边界)。
参考资料与模板
核心参考包括:operational-playbook.md、automation-tools.md、review-checklist-comprehensive.md、security-focused-review-guide.md、large-pr-review-strategies.md、code-review-metrics.md、psychological-safety-guide.md、dotnet-efcore-crypto-rules.md、adversarial-review-protocol.md、complexity-only-review-pass.md、deterministic-vs-llm-routing.md。
模板包括:pull-request-description-template.md、review-comment-guidelines.md、api-review.md、frontend-review.md、mobile-review.md、infrastructure-review.md、crypto-review.md。
来源:data/sources.json。
新鲜度协议
当用户询问最佳或最新代码审查工具、当前 PR 自动化实践、GitHub Copilot 与 CodeRabbit、Qodo 或平台原生方案的对比,或某工具是否仍可用时,须先核实当前状态。