pr-review-looplisted
Install: claude install-skill inakaegg/agent-kit
# PR Review Loop
## 前提権限
- PRのread、comment取得、CI log取得はread-onlyとして行える。
- push、PR comment投稿、review再依頼、merge等のremote writeは、共通AGENTSと最新ユーザー指示の許可範囲に従う。
- mergeはこのSkillの完了から自動的に許可されない。
## 1. 初期状態
確認・記録する。
- repositoryとPR番号
- PR state、draft、base branch、head branch、latest head SHA
- mergeabilityとrequired checks
- project `AGENTS.md`、verification command、merge policy
- reviewer/botの種類とreview signal
- local working treeとbranch
state directory:
```text
_ai/pr-review/<PR番号>/
```
各iterationに、head SHA、指摘、仕分け結果、変更、checks、push/review signalをJSONまたはMarkdownで残す。
## 2. latest headのfeedbackだけを対象にする
- review、inline comment、top-level comment、CI runをすべて確認する。
- 可能ならhead SHA、reviewed commit、workflow runのhead SHAで最新性を判定する。
- timestampだけに依存しない。使う場合はiteration startとtimezoneを明示する。
- 古いcommitへのcommentは、現在codeにも該当するか再確認する。
- silence、rate limit、timeout、botのmarketing section、walkthrough summaryをapprovalまたは指摘として数えない。
- 特定bot名やworkflow名をhardcodeせず、repositoryからdiscoverする。
## 3. 指摘の仕分け
対応が必要なすべての指摘を確認し、次へ分類する。
- **MUST-FIX**:確認済みbug、security、data loss、requirement violation、breaking regression
- **VALID-NONBLOCKING**:正しいが現在PRを止めないquality improvement
- **FALSE-POSITIVE**:code/specを誤読、outdated、再現不能
- **FOLLOW-UP**:実害や着手条件があるがscope外
- **BLOCKED**:credential、environment、product decision不足
各指摘について:
- location
- reviewer
- actual code pathまたはreproduction
- classification
- action/reason
を残す。bot同士の矛盾を両方機械的に満たそうとせず、自分でcodeとspecを判定する。
## 4. 修正iteration
1. MUST-FIXを、同型箇所の横断検索後にまとめて修正
2. 必要なregression te