---
name: review-pr
description: 审查 GitHub PR 的准确 head/base diff、关联 issue、CI 和 review threads，交付前刷新。创建 PR 使用 submit-pr；本地审查使用 review。
---

# GitHub PR 审查

本 Skill 编排 GitHub PR 的身份、问题背景、本地准备、CI、线程和最终刷新。
语义与正确性审查使用配套 `review`；审查本地工作树、提交、范围、文件或模块时直接使用
`review`，不启动 PR 编排。

`<review-pr-skill-dir>` 指实际加载的本 Skill 目录。优先从当前 Skill 清单定位 `review`，
未提供位置时才检查 [同级安装位置](../review/SKILL.md)。开始 Git 准备前确认依赖可读；
缺失时可收集获准的只读证据，但不得声称完整代码审查已完成。

## 模式与授权

- **默认本地审查**：明确请求审查 PR 时，包含必要的 remote 添加、fetch、安全分支创建或
  fast-forward、upstream 设置和 checkout。
- **隔离 worktree**：只在用户明确要求 worktree、并行或并发 review 时新建；本人 PR 分支
  已在其他 worktree 打开时可复用其干净 checkout，后续命令使用回执路径。
- **latest-base 集成审查**：只在用户明确要求更新最新 base、解决冲突或审查集成结果时使用。
- **不改变 Git 状态**：遵守用户的只读或不切分支限制，改用可访问的准确远程 diff、源码和评论；
  证据不足时报告限制。

上述本地准备授权不包含产品修复、push、发布评论、approve、删除评论或关闭 PR。
线程 resolve 需要单独的远程操作授权和当前远程证据。仅方案或解释不执行 Git 准备。

## 核心安全边界

- 默认本地模式从 `git status --short --branch` 开始；当前 checkout 有未提交变更时，
  在切换分支前停止。显式 worktree 模式不得改变原 checkout，因此可从脏状态继续。
- 多阶段 review 必须把首次准备回执中的 `checkout.branch` 作为后续阶段的显式输入；使用
  helper 的 `--reuse-branch` 复验该分支、HEAD、upstream 和 worktree 占用，失败时停止，
  不静默改选另一个分支。回执记录实际 helper 路径、入口 SHA 和准备依赖组合指纹，
  按本地准备协议复验版本漂移。
- 不覆盖、删除、重命名、rebase、reset、强制更新、stash 或丢弃本地分支、worktree 或变更。
- 普通审查必须对应 PR 元数据的准确 `headRefOid` 和真实 base/merge-base diff；
  不用 detached HEAD、已 fetch ref 或无关 `origin` 绕过身份检查。
- 本人 PR 优先复用已核实身份的同名分支及等价 upstream。相同或落后的分支安全准备到远程
  head；领先时保留额外提交，按冻结 Git 对象审查，不新建 review 分支。回执区分 checkout
  与 review SHA；源码上下文和验证归属遵循 [证据协议](references/evidence-workflow.md)。
  本人分支分叉或 upstream 不匹配时停止；其他作者仍按本地准备协议处理 collision fallback。
- `mergeable: CONFLICTING`、`mergeStateStatus: DIRTY` 或 base 领先不构成 latest-base 授权。
- 除非用户明确要求，审查、准备、冲突处理和线程维护都不 push。审查后保留准备好的
  分支或 worktree，不自动删除。

## 审查流程

1. 读取 [证据收集与刷新](references/evidence-workflow.md)，收集并冻结 PR 身份、head/base、
   [问题背景](references/problem-review.md)、checks 和完整 threads/replies。
2. 允许 Git 准备时读取 [本地准备与 latest-base](references/local-preparation.md)；只读模式直接使用准确远程证据。
3. 将 PR 目标、关键验收条件、冻结 base/head 和完整 raw diff 交给 `review`，检查需求满足程度、
   实现方式与代码正确性。
4. 评估每个开放 thread，包括 outdated、bot 和所有回复。已有评论只放在对应评论条目，
   不重复列为独立 finding。需要 resolve 且已获授权时才读取
   [线程维护](references/thread-resolution.md)。
5. 如任务包含冲突处理、验证或本地提交，先完成这些获准操作；结论前按证据协议立即刷新 PR、
   选定问题证据、checks 和完整 threads/replies，处理所有新活动。
6. 生成最终回复前重新读取 [PR 审查报告](references/reporting.md)；此前已读过或上下文摘要中
   提及过，均不替代此次读取。按该协议组织报告与必要的提交回执，并对实际回复草稿完成
   [交付检查](references/reporting.md#交付检查) 后再输出。

大 PR 或需要复用快照文件时才读取
[快照传输协议](references/snapshot-protocol.md)；复杂复审报告可再读取 [完整示例](references/report-example.md)。

## 完成与停止条件

只有准确 remote head/base 与 merge-base、完整 diff、目标/问题证据、CI 状态、全部开放 threads
及回复均已审查，最终刷新没有未检查活动，且最终报告通过交付检查时，才算完成。最终刷新发现 head/base、
需求或线程变化时，根据影响更新 checkout、范围和判断，处理后再刷新一次。

引用有歧义、准备会覆盖本地状态、依赖或必需证据不可用、身份/内容持续漂移、冲突需要产品判断，
或最终刷新失败时，保留当前快照和本地状态，报告已审查 SHA 与未覆盖缺口；不无限重试。
受阻或未完成报告仍遵循报告协议，保留内容介绍及验证限制；证据不足的字段明确标注缺失，
不为满足格式编造内容，也不将报告格式完整视为审查已完成。