self-review - PR提出前のセルフレビュー

自分の作業ブランチのdiff全量を包括チェックし、消し忘れ・変更漏れ・意図しない変更を検出する。 コード品質のレビュー（バグ・設計）ではなく、変更の完全性と純度の検証に特化する。

既存設定との関係

• Phase 4品質確認（context/workflow-rules.md）: lint/format/typecheck/testを補完する。外部レビューの前に行うと指摘のノイズが減る
• pr-reviewスキル: 他者PRのレビュー用。本スキルは自分のブランチのPR提出前チェック用

ワークフロー

1. ベースの確定とdiff全量取得

# BASE_BRANCHはPJ CLAUDE.md参照（未定義時: develop→main→master）
git log --oneline <BASE_BRANCH>..HEAD
git diff <BASE_BRANCH>...HEAD --stat
git diff <BASE_BRANCH>...HEAD
git status   # 未コミット変更の確認

diffは省略せず全ファイル確認する（一部だけでは消し忘れを拾えない）。

2. 消し忘れ・残存の検出

diffで削除・リネームされた識別子を抽出し、リポジトリ全体で残存検索する:

• 対象識別子: メソッド名・クラス名・定数・env key・docker service名・キュー名・設定キー・ルーティングパス等
• 検索範囲: コードに加えてconfig / docker-compose / .env* / CI定義（.github/workflows等）/ docs / テストfixture
• コマンド: git grep -n "<identifier>" または rg -n "<identifier>"

# 例: 削除した識別子ごとに実行
git grep -n "<削除した識別子>"

• 残存あり → 消し忘れとして報告（削除対象か、意図的に残すのかを判定）
• 逆方向も確認: 削除した機能を参照していた呼び出し元が全て更新されているか

3. 意図しない変更の検出

• format副作用: 自分のタスクと無関係なファイルの整形のみの変更（git diff --stat で変更行数が多い割に意図が不明なファイル）
• 混入: .idea/ .vscode/ .env* *.log lockファイルの意図しない変更、一時ファイル
• デバッグ残骸: console.log / print / debugger / コメントアウトされた試行コード / TODOの置き忘れ
• 無関係コミット: git log に他タスクのコミットが混入していないか（rebase漏れ）

4. タスク要件との突合

双方向で確認する:

1. 要件 → diff: 今回のタスク要件（ユーザー指示・計画書・チケット）の各項目が、diffのどの変更で満たされているか対応付ける。満たされていない要件があれば「漏れ」
2. diff → 要件: diffの各変更が要件のどれかに紐づくか。紐づかない変更は「過剰（スコープ外変更）」として報告
3. 足しすぎ: 要件に紐づく変更でも、頼まれていない機能・オプション・「後で使う」仕組み、既存の helper や標準ライブラリで済む新しい実装、数行のための新しい依存、実装が1つしかない抽象は「過剰（足しすぎ）」として報告する（探す順序は AGENTS.md「新しく作る前に」）

5. レポート

# セルフレビュー: <ブランチ名>

## 対象
- ブランチ: <branch> → <base>（コミットN件、ファイルM件）

## 消し忘れ・残存
- [なし / 検出内容（ファイル:行）]

## 意図しない変更
- [なし / 検出内容と推奨対応（戻す・別コミット化等）]

## 要件との突合
| 要件 | 対応する変更 | 判定 |
|------|------------|------|
| ... | ... | OK / 漏れ / 過剰 |

## 推奨アクション
1. [修正が必要な項目]

注意事項

• 検出した問題の修正は報告後にユーザー判断を仰ぐ（勝手にファイルを戻さない。特に git checkout -- はformat副作用と断定できるファイルのみ・判断不能時はユーザー確認）
• 「漏れなく」は要件に対する漏れの確認であり、無関係な要素の網羅列挙ではない（スコープ膨張禁止）