跳转到内容
关系图谱

code-review(Matt)

451 字 2 分钟

属性 8
repo
mattpocock_skills
skill
code-review
source
mattpocock_skills/skills/engineering/code-review/SKILL.md
invoke
model
phase
review
cost
mid
status
stable
local
outdated
一句话

对 HEAD 到你指定固定点的 diff 做双轴评审:Standards(是否遵守本仓库规范)和 Spec(是否忠实实现 issue/spec),两个子代理并行出报告,分开报不混名次。

何时用 ​

  • 适合:评审分支、PR、未提交改动、任何「review since X」;implement 和 implement-spec 收尾时自动调它。
  • 不适合:要在 diff 里猎空指针、竞态、off-by-one 这类 bug;整个库走样不是一个 diff 的事,用 matt-improve-codebase-architecture。

怎么用 ​

text
把这个分支相对 main 的改动评审一遍
  1. 固定评审点(SHA、分支、tag、HEAD~5);先 git rev-parse 确认 ref 有效且 diff 非空,没给就问。
  2. 按顺序找 spec 来源:commit message 的 issue 引用、用户给的路径、docs/、specs/、.scratch/ 下匹配文件;找不到就问。
  3. 收齐规范来源:CODING_STANDARDS.md/CONTRIBUTING.md 必收;仓库成文标准永远压过 Fowler 异味基线。
  4. 两个子代理并行:Standards 报违规和异味,Spec 对照 spec 报缺失、越界、实现错,各 400 词内。
  5. 聚合时每轴给 finding 总数和最严重一条,不跨轴选总冠军。

产出 ​

  • 双轴评审报告;没有 spec 时 Spec 轴标注 "no spec available"。

要点与坑 ​

  • 异味基线是 12 种 Fowler 味(Mysterious Name、Duplicated Code、Feature Envy、Data Clumps、Shotgun Surgery 等),永远是 judgement call;只有成文标准可能是硬违规;工具已强制的不报。
  • 两轴分开报的意义:全合规范但做错事,或做对事但破规范,混合结论会让过的一轴盖住挂的一轴。
本机版本差异

仓库版明确要求两个子代理在前台同时发出再聚合报告;规范来源表述更宽("every file that documents how code should be written")。

关联 ​