review-prlisted
Install: claude install-skill fitlab-ai/agent-infra
# 审查 PR
根据目标 PR 的生命周期证据完整度与变更风险,选择 `verify`(轻量复核)、`audit`(证据审计)或 `reconstruct`(重建式审查),并把可执行的检视结论作为正式 PR Review 发布到目标 PR,同时保留可追溯的审查过程证据(`pr-review.md` / `pr-review-r{N}.md`)。
## 行为边界 / 关键规则
- ���格遵循方案中固化的证据分级主决策流:宿主解析 → 证据分类 → 新鲜度/对齐 → 风险分级 → 模式选择
- `pr-review*` 是独立 artifact family:不接入 analysis/plan/code 阶段依赖链,不修改 `current_step`
- PR 上只发布正式 PR Review(head SHA、结论、finding、receipt、Issue artifact 链接),不发第二份完整过程原文到 PR 普通评论
- 完整过程原文与 `task.md` 的可逆镜像只在关联 Issue 上同步;`restore-task` 保持 Issue-only 契约
- 不得为外部 PR 倒填或伪造已发生的 `analysis`/`plan`/`code` 生命周期历史
- head SHA 唯一记录于 `pr-review*` 与正式 PR Review;head 漂移必须进入新轮次,不得沿用旧结论
- 审查深度由证据完整度和变更风险决定,不由提交者身份决定
- 执行本技能后,你**必须**立即更新 task.md(经 `task-activity` typed intent)
- 绝不自动执行 `git add` 或 `git commit`
- 一次性检视路径(无任务/Issue)走 `.agents/workspace/reviews/{pr-number}/`,标记 `recoverable: false`,不承担可恢复承诺
## 常见违规借口与反驳
| 借口 | 反驳 |
|------|------|
| 「代码 diff 直接看就行了,不需要重建上下文」 | 仅看 diff 会遗漏需求边界、架构选择与迁移策略;`reconstruct` 必须先产出最低充分重建记录。 |
| 「PR 没有关联任务,直接写个报告发出去」 | 默认阻塞并要求先关联 Issue/task;只有显式「仅一次性检视」才走不可恢复降级路径。 |
| 「把完整重建过程也发到 PR 普通评论,方便贡献者看」 | 过程原文只同步到 Issue artifact 评论;PR 只保留正式 Review,避免重复远端副本与恢复源混淆。 |
| 「审查完了顺手提交一下」 | 本技能绝不执行 `git add`/`git commit`;提交是用户显式发起的独立步骤。 |
## 第 0 步:状态核对(执行前硬约束)
在加载本技能与相关规则后、做任何任务状态判断或用户可见结论之前,必须先执行状态核对:
```bash
agent-infra-internal task-snapshot {task-id} --format text
```
任务锚定路径在开始前记录当前任务目录与既有 `pr-review*` 轮次;一次性路径(无任务)记录 PR 编号与目标目录。状态核对输出粘贴到本轮产物 `pr-review-rN.md` 的 `## 状态核对` 段。
## 执行步骤
### 1. 解析入参与宿主解析
解析目标 PR 编号 `{pr-number}`