review-gatelisted

# Review Gate 実装後に 6 観点でレビューを行い、critical finding を Completion Gate に伝達する。 ## Iron Law `NO MERGE WITHOUT TWO-STAGE REVIEW` severity=critical の finding がある場合、fix なしに Completion Gate を通過させない。 ## Common Rationalizations | こう思ったら | 現実 | | -------------------------------------- | ---------------------------------------------------------------------------------- | | 「テストが通ったからレビュー不要」 | テスト通過はロジック正確性の一部に過ぎない。セキュリティ・仕様準拠は別途確認が必要 | | 「小さな変更だから critical は出ない」 | 規模に関わらず 6 観点でチェックせよ。1 行の変更でも脆弱性は混入する | | 「外部レビューを受けたから大丈夫」 | review type の EvidenceItem として記録せよ。記録なき承認は存在しない | ## 手順 ### ステップ 1: `/pg-check` を実行して finding を収集する ```bash # 差分レビューを実行して severity 付き finding を取得する /pg-check <対象ブランチ or PR番号> ``` ### ステップ 2: 6 観点で finding を分類・severity を付与する `/pg-check` の Findings を以下の 6 観点に分類する: | # | 観点 | チェック内容 | | --- | ------------------ | -------------------------------------------- | | 1 | **仕様準拠** | 受入基準・設計書との一致 | | 2 | **コード品質** | 可読性・命名・構造の明確さ | | 3 | **セキュリティ** | 入力バリデーション・認証・認可・機密情報 | | 4 | **パフォーマンス** | N+1 クエリ・ループ内 I/O・不要データ取得 | | 5 | **テスト不足** | カバレッジ・エッジケース・重要パスの未テスト | | 6 | **破壊的変更** | 後方互換性・API 変更・スキーマ変更 | > **観点フレームの正本との対応**: 本表の 6 観点は本 Skill の運用チェックリストであり、 > `review-principles.md` §2 の 5 観点（可読性・拡張性・パ