mcpField
R
review-pr-in-worktree

review-pr-in-worktree

github.com/mtngtools测试与质量040 次安装3 周前更新
代码仓库

关于 review-pr-in-worktree

review-pr-in-worktree 是一项由人工明确点名启动的 PR 审查技能,供代码审查者在独立 worktree 中检查变更、测试和实现计划的符合情况。它负责确定唯一 PR 并管理检出与清理,报告问题但不修复。

主要功能

  1. 01要求人工明确启动
  2. 02根据现有上下文确定仓库和 PR
  3. 03在独立 worktree 中审查
  4. 04检查提交、测试及计划偏差
  5. 05报告结论而不修复代码

适用场景

  1. 01隔离检出待审查的 PR
  2. 02核对 PR 是否实现其声明计划
  3. 03审查结束后清理 worktree

安装

复制下面的命令,在终端中运行。

$npx skills add https://github.com/mtngtools/agents --skill review-pr-in-worktree

安装后在 Claude Code 中输入/review-pr-in-worktree确认已加载。

附带文件worktree-review.md

详细介绍

review-pr-in-worktree

仅限人工操作。 只有在人类按名称要求时才启动此流程。如果你是从另一项技能跳转到这里,请停止,并在执行任何步骤前获得明确确认。

将 PR 检出到专属工作树中,并在那里进行审查:它提交了什么、测试是否通过、是否符合其声称要实现的计划,以及偏离了计划的地方。只报告,不要修复。

此技能负责确定要审查哪个 PR;其余流程是共享的。 将 PR 检出到工作树中、逐轮审查,然后拆除检出环境的流程是工作树审查,/review-pr-in-lane 也会遵循该流程。审查本身——要阅读什么、如何运行检查、依据什么标准评判代码,以及以 yes 或 no 结尾的固定报告——由 /review-a-pr-and-report 负责,该流程会在每一轮调用它。

流程

1. 确定仓库和 PR——使用现有信息,先询问,不要搜索

按以下顺序,从当前已有的信息中确定两者:

  1. 如果给出了参数(PR 编号、PR URL、owner/name),使用该参数。
  2. 当前工作目录的 git 远程仓库和分支——gh 会在克隆仓库中推断仓库,gh pr status 会指出当前分支对应的 PR。
  3. 明确无误的现有上下文——本会话打开了该 PR,或者人类之前指定了某个 PR,且之后没有任何信息表明并非如此。

如果以上信息都无法确定目标,或者其中两项互相矛盾,立即询问——只问一个问题,并将它作为你要做的第一件事。绝不能因为某个 PR 最新、它是唯一未关闭的 PR,或其标题看起来像人类提到的内容,就擅自选择它。审查错误的 PR 会浪费整个会话,而且看起来像权威结论。

**完成标准:**无需搜索,即可确定唯一的一个仓库和一个 PR 编号。

2. 在工作树中审查

使用该仓库和 PR,按照 worktree-review.md 从第一节开始执行。该流程会检查 PR、以分离状态检出、逐轮审查并打印结论区块,在每轮之间暂停等待你操作,完成后清理,并确认 PR 的后续状态。

**完成标准:**该流程的最后一节已报告 PR 的状态。

在整个流程中的位置

/create-pr-for-branch 创建 PR,/review-pr-in-worktree 负责评判 PR——作者需要多少轮就进行多少轮——然后由 /squash-merge-and-clean-up 合并。每一步都需要人类单独启动,因为它们分别由不同的人负责:作者、审查者和维护者。合并通常会在完全不同的会话中进行,因此流程会在最后询问跟踪器 PR 的后续状态,而不是假定自己的报告就是最终结论。

在本流程中,/review-a-pr-and-report 负责实际审查,将其单独拆分出来,是因为无论检出环境来自何处,评判标准都相同。工作树审查流程则确保安全地检出,并在之后移除检出环境;它与 /review-pr-in-lane 共享此部分,二者唯一的区别是如何找到 PR:后者会等待某个 lane 创建 PR。

评论

支持 Markdown