self-reviewerlisted
Install: claude install-skill kai-kou/gem-hunter
# PR 作成前セルフレビュースキル(汎用)
PR 作成前に問題を自分で検出・修正してレビュー往復を減らす。`pre-pr-create-check.sh` フックが
`tools/self_review_check.py` を呼ぶため、Error が残ると PR 作成自体がブロックされる。
> **🔴 レビューは Claude 自身のセルフレビューで完結する**: 外部 AI レビュアー(Copilot / Gemini)への
> 依頼は **行わない**。組み込み `/code-review` は disable-model-invocation で自律起動不可のため、
> **自前 `code-review` スキル(`.claude/skills/code-review/`・同名 project スキルが bundled を置換・
> 自律起動可)が Layer 1 の標準実行手段**(#275 → #280)。本スキルは PR 作成前の事前チェック
> (Step 0-3)を担い、Step 4 で自前 `code-review` スキルを呼び出す。
> 構成の SSOT は `docs/rules/ai-reviewer-strategy.md`。
## トリガー条件
- PR 作成直前の最終��ェック時
- 「セルフレビューして」「PR 前にチェックして」「レビューコスト下げたい」等の依頼時
## 実行フロー
### Step 0: 機械チェック(必須・最初に実行)
```bash
python3 tools/self_review_check.py
# .md を新規作成・修正した場合は CJK 半角スペースを必ず自動整形してから PR を作る
# (目視では大規模ドキュメントで必ず見落とし、AI レビュ��ーに同種指摘される・再発防止)
python3 tools/check_cjk_markdown.py --fix --changed
```
- Error(マージコンフリクト痕跡・巨大ファイル等)が残る限り PR 作成は禁止(フックでもブロック)
- Warning は修正するか、PR 説明文「設計意図・既知の警告」に理由を記載する
- **CJK 半角スペース Warning が出たら必ず `check_cjk_markdown.py --fix` で整形してから PR**
(変更 .md がある PR では `--fix --changed` の実行を必須化する)
- プロジェクト固有のチェック項目は `docs/rules/self-review-checklist.md` に追記し、
`self_review_check.py` に検査関数を足して機械化する(同一 PR でシートとランナーを更新)
### Step 1: 差分スコープの確認
```bash
git diff main...HEAD --name-only
```
- **タスク外のファイルが混入していないか**(「ついで」の変更は別 PR / 別 Issue に分離)
- 自動生成ファイル・state ファイルが意図せず含まれていないか
- 削除・リネームが意図どおりか
- 対象 Issue の Done Criteria(Issue 本文または最初のコミットメッセージのいずれかに記載)を diff が満たすか照合する。どちらにも未記載の場合はスキップし、その旨を 1 行記録する
### Step 2: 変更内容の