Code Review by Matt Pocock

使用 Matt Pocock 的 Code Review,对照指定 Git 基准审查分支。分别检查仓库规范与原始需求,并列查看带有代码位置的问题,避免将符合规范误当作需求已经完整实现。

安装与快速上手

选择你使用的 Agent,查看对应的安装或使用方式。在 Floatboat 中,可复制请求让 Agent 读取原始说明;其他选项会显示该 Skill 的安装命令或发布者提供的原生步骤。

选择 Agent

在 Floatboat 中开始

复制请求,粘贴到 Floatboat 对话

请阅读 https://github.com/mattpocock/skills/blob/main/skills/engineering/code-review/SKILL.md 中的 Code Review 原始说明,介绍它的用途、所需工具和安装方式,再协助我使用。

由 Agent 读取原始说明,再确认当前环境的使用方式。

下载 Floatboat ↗

试试这样提问

使用 Matt Pocock 的 Code Review 对照 main 审查分支,需求文件是 docs/specs/import.md,分别报告规范与需求发现。

关于 Code Review

Code Review 是 Matt Pocock 工程集合中的双轴审查 Skill。它先解析比较参考,确认基于合并基点的差异非空,再查找原始需求与仓库编码规范。两个独立子代理分别检查规范和需求覆盖,最终报告保留两方面发现。规范轴还使用有明确标识的代码异味基线,但项目文档中的约定具有优先权。没有需求说明时,会明确跳过需求轴,而不是自行编造。使用需要可访问的 Git 历史、预期比较点及集合的需求跟踪配置;它提供审查流程,不会自动批准修改,也不保证完成问题修复。

代码审查的使用场景

探索同时需要检查实现规范和需求符合度的审查任务。可以检查特性分支、核对完成的工单,或与发布基线比较;将规范违背与缺失、部分完成或超出要求的行为分别列出,并保留对应证据。

审查特性分支

与约定基线比较改动,同时检查代码约定与预期行为。

核对工单实现

找出缺失、部分完成或与需求不同的实现。

复核发布差异

使用固定发布参考,使审查范围可以复现。

如何审查分支变更

提供可解析的 Git 比较点和原始工单或需求说明。让 Agent 收集规范、确定差异范围,再独立运行两个审查轴;分别核对发现后,决定需要修改哪些内容,以及补充哪些验证。

  1. 01

    固定比较点

    提供有效提交、分支或标签,并确认合并基点差异覆盖目标工作。

  2. 02

    提供规范与需求

    指出仓库约定与原始需求;缺少需求说明时明确记录。

  3. 03

    分别核对两组结果

    独立阅读规范和需求发现,不把某一轴通过当作另一轴也合格。

探索更多 Agent Skills

查看其他 Skill 指南,并在使用前核对原始来源。

Code Review 常见问题

流程使用哪种 Git 比较?
它使用所选参考与 HEAD 的三点比较,基于二者合并基点审查。应先确认该范围符合目标。
参考无效或差异为空怎么办?
原文在分发审查前检查这些条件。应修正比较范围,而不是把空审查视为成功评估。
必须提供需求说明吗?
它会尝试查找原始工单或说明。没有时应明确报告并跳过需求轴,而不是猜测要求。
哪些编码规范优先?
仓库文档规范优先于通用代码异味基线。异味属于判断线索,不自动等于硬性违规。
没有子代理的宿主能原样运行吗?
原始流程使用并行子代理来分离审查上下文。顺序审查属于适配方式,与原流程的执行结构不同。
为什么不把两个轴的结果合并?
符合规范的代码可能漏需求,行为正确的代码也可能违背项目约定。分开报告可以保留不同问题。
如何配置需求跟踪方式?
集合中的 setup-matt-pocock-skills 配置 Skill 会记录仓库需求跟踪方式。工程使用前应先配置,以便审查找到正确来源。