SuperAPI AI 工具集
合作ENAI 行业资讯收藏
Skill 广场code-review

code-review

编程与开发

查看项目

更新于 10-23 17:20

技能介绍

用途

Code Review 用于对一段代码变更进行结构化审查。审查范围是 HEAD 与用户提供的固定点（commit SHA、分支名、tag、main、HEAD~5 等）之间的 diff，采用三点比较（git diff <fixed-point>...HEAD），即与合并基比较。

主要能力：三轴独立审查

  1. 正确性（Correctness）：这次变更会不会破坏什么？关注缺陷、故障模式（副作用失败、重复或竞态时的行为）、安全、数据、破坏的契约、缺失的测试。该轴不接收任何 PR 审查评论，确保是未被其塑造的视角。输出为带具体失败用例的 BLOCKING / MAJOR / MINOR 行。

  2. 标准（Standards）：代码是否符合本仓库的编写方式？按优先级查找标准源：根目录 AGENTS.md（及其指向的文档）、CONTRIBUTING.md / CODING_STANDARDS.md、以及 diff 触及目录中的 CLAUDE.md、.cursor/rules 等。工具已强制执行的领域（格式化、类型检查、机械升级工具）被跳过。此外始终携带一组 Fowler 代码坏味道基线（如神秘命名、重复代码、特性依恋、数据泥团、基本类型偏执、重复 switch、散弹式修改、发散式变化、投机性泛化、消息链、中间人、被拒绝的遗赠），每条坏味道都是带标签的启发式判断，仓库文档标准始终优先，可压制基线中的标记。

  3. 规格（Spec）：代码是否忠实实现了来源工单 / PRD / 规格？按顺序查找规格：提交消息或分支名中的工单引用（读 .scratch/ 下对应 markdown 文件）、用户传入的路径、docs/、specs/ 或 .scratch/ 下匹配分支名或特性的 PRD/spec 文件。找不到则询问用户；用户说没有，则跳过该轴并报告“无可用规格”。报告内容包括规格要求但缺失或部分实现的需求、diff 中未被要求的行为（范围蔓延）、看起来实现了但实现有误的需求，每条引用规格行。

使用方式

  1. 固定固定点：捕获 diff 命令和提交列表；若调用方传入路径范围，追加到 diff 与 log 命令；先确认固定点可解析且 diff 非空，否则在进入三轴审查前失败。
  2. 识别规格源与标准源（如上）。
  3. 运行三轴：支持并行子代理的宿主（如 Claude Code Agent 工具、Codex spawn_agent + wait_agent、Grok spawn_subagent 等）并行启动三个前台子代理，等待全部结果后再写任何内容，不要后台运行；不支持子代理时在本会话内联按正确性、标准、规格顺序执行。小 diff（git diff --shortstat 显示最多 3 个文件、约 150 行变更）走内联。复审收窄：调用方传入上一轮发现和 head 时，只审查 <previous head>..HEAD 的修复提交，且只运行有可操作发现的轴，加上正确性（当修复引入新机制如 job、锁、认领、重试时）。未运行的轴报告为未复审。
  4. 聚合：在 ## Correctness、## Standards、## Spec 标题下逐字或轻度清理地呈现报告，不合并或重排发现。独立调用时通过宿主的发现列表报告（如 Claude Code 一次 ReportFindings 调用包含三轴，类别设为轴名，轴内最严重在前，判断性坏味道标为 PLAUSIBLE），然后只打印摘要行；否则打印三个带标题的节。结尾一行摘要：每轴发现总数及每轴最严重问题（如有），不跨轴选单一赢家，每轴用 ✅ 无发现 / ⚠️ 有发现标记。

适用场景

适用于提交前或 PR 前对变更做多维审查，尤其适合需要同时检查正确性、仓库规范与需求忠实度的场景。三轴分离可防止一轴掩盖另一轴：符合所有标准但实现错误（标准过、规格败）、忠实需求但破坏约定（规格过、标准败）、规范且忠实但失败时丢失工作或竞态（正确性败）等情况都能被分别暴露。

限制

  • 必须提供固定点，否则会询问；固定点不可解析或 diff 为空则直接失败。
  • 规格缺失时无法进行规格轴审查。
  • 标准轴不覆盖格式化、类型检查、机械升级工具已强制执行的领域。
  • 子代理路径要求宿主支持并行子代理且不能后台运行；否则只能内联串行。
  • 未提供具体安装命令或收费信息。