code-review:双轴代码审查


文档摘要

code-review:双轴代码审查 code-review 是模型触发的代码审查技能:用两个互相独立的轴审查自某个固定点以来的改动——「规范」轴(是否符合仓库编码规范 + Fowler 坏味道基线)和「规格」轴(是否忠实实现了来源 issue/PRD)。两轴并行跑,互不污染。 双轴并行审代码:一轴看规范坏味道,一轴看是否对得起原始需求。 它解决什么问题 单轴审查容易偏:只盯风格就漏了需求偏差,只盯需求又放过坏味道。而且审一堆改动容易顾此失彼。

code-review:双轴代码审查

code-review 是模型触发的代码审查技能:用两个互相独立的轴审查自某个固定点以来的改动——「规范」轴(是否符合仓库编码规范 + Fowler 坏味道基线)和「规格」轴(是否忠实实现了来源 issue/PRD)。两轴并行跑,互不污染。

双轴并行审代码:一轴看规范坏味道,一轴看是否对得起原始需求。

它解决什么问题

单轴审查容易偏:只盯风格就漏了需求偏差,只盯需求又放过坏味道。而且审一堆改动容易顾此失彼。

何时使用

  • 一次改动准备提交前需要把关
  • 想同时检查「代码质量」和「需求符合度」
  • 作为 implement 收尾的强制关卡

核心工作流

关键原则与反模式

  • 规范轴:对照仓库编码规范,加 Fowler 坏味道基线
  • 规格轴:核对是否忠实实现来源 issue / PRD
  • 两轴作为并行子代理运行,互不影响、互不污染
  • 是 implement 提交前的强制关卡

与其他技能的配合

code-review 是主链路的守门员,implement 在提交前必须过它。它的「规格轴」直接回溯 to-spec / to-tickets 的原始意图。

相关技能:

  • implement:按规格构建
  • to-spec:把对话变成规格
  • to-tickets:拆成追踪子弹工单
  • tdd:测试驱动开发

常见问题

为什么分成两个轴?

规范(代码质量)和规格(需求符合度)关注点不同,分开并行跑能避免互相干扰,结论更干净。

它审的是整个仓库吗?

审的是「自某个固定点以来的改动(diff)」,聚焦本次提交,而不是全量代码。

返回章节:工程技能完全指南


发布者: 作者: 灏天文库 转发
评论区 (0)
U