# PR 合并前审查 (review)

> 对照基准分支审查 PR 的 diff，查找测试发现不了的结构性问题，并自动修复或确认后修复。

- 分类: [代码评审](https://www.myskills.tools/zh/categories/code-review/)
- 来源: 社区 · Garry Tan
- 代码仓库: https://github.com/garrytan/gstack/tree/main/review (commit ed10791 · v1.0.0)
- SKILL.md: https://github.com/garrytan/gstack/blob/main/review/SKILL.md
- 安装命令: `npx skills add garrytan/gstack --skill review`
- 许可证: MIT
- 兼容工具: Claude Code, Codex, Cursor, Copilot
- 权限: read-repo ✓, write-files ✓, shell ✓, network ✓
- 状态: 扫描通过
- 规则扫描无法覆盖所有风险，安装前请阅读此技能的 SKILL.md，确认权限后再使用。
- ⚠ 会不经询问直接应用部分修复，可能回复 PR 上的 Greptile 评论，并通过 gstack 脚本记录遥测。

## 核心规则

先修复而非只读：AUTO-FIX 类直接修改，ASK 类经用户批准后才改，且从不提交、推送或创建 PR。

## 执行阶段

1. **确定基准与范围** — 识别 git 托管平台和基准分支，确认有 diff，并对照 TODOS.md、提交信息和 PR 描述检查范围偏移。
2. **读取清单与 diff** — 读取审查清单和 Greptile 评论，获取完整 diff 并运行 diff 扫描。
3. **关键审查** — 先做 CRITICAL 再做 INFORMATIONAL 检查，在其他调用点横向排查同类问题，并加入专项审查、探索式 QA 和对抗式审查。
4. **先修复** — 把问题分为 AUTO-FIX 和 ASK，直接应用前者，后者一次性询问用户并按批准结果修复。
5. **复审与记录** — 修改后重新审查（最多 3 轮），保存审查结果，回复 Greptile 评论并输出最终报告。

## 权限说明（据 SKILL.md 整理）

| 能力 | 是否需要 | 说明 |
|---|---|---|
| 文件写入 | 需要 | AUTO-FIX 类问题直接修改代码，ASK 类经批准后修改；不提交也不推送。 |
| 命令执行 | 需要 | 运行 git、gh/glab、gstack 辅助脚本以及探索式 QA 检查。 |
| 网络外发 | 需要 | 可用 WebSearch 调研修复方案，通过 gh 读取 PR 信息并回复 Greptile 评论。 |

## SKILL.md 头部信息（YAML）

```yaml
---
name: review
preamble-tier: 4
version: 1.0.0
description: Review this PR's diff. (gstack)
allowed-tools:
  - Bash
  - Read
  - Edit
  - Write
  - Grep
  - Glob
  - Agent
  - AskUserQuestion
  - WebSearch
triggers:
  - review this pr
  - code review
  - check my diff
  - pre-landing review
---
```

首页: https://www.myskills.tools/zh/ · URL: https://www.myskills.tools/zh/skills/review/