Pr Triage

PRトリアージ

PRの変更内容を分析し、後続のコードレビューフェーズに必要な情報を構造化して出力するスキルです。 軽量モデル（Haiku）での実行を想定しており、CI環境でのコスト最適化に寄与します。

手順

1. gh pr view <PR番号> --json title,body,headRefName,baseRefName,changedFiles でPR情報を取得
2. gh pr diff <PR番号> --name-only で変更ファイル一覧を取得
3. 差分の取得（changedFilesが15以上の場合、手順3はスキップしてファイル名とPR説明のみで分析する）：
   • changedFilesが15未満の場合のみ gh pr diff <PR番号> でコード差分を取得
   • 表層チェックが必要なファイル（.ts, .js, .tsx, .jsx）が多い場合は個別に gh pr diff <PR番号> -- <file> で取得
4. .pr-review-state.json が存在するか確認する（前回レビュー状態）
5. 以下の分析を行う：
   • 変更ファイルのカテゴリ分類（added/modified/deleted）
   • 使用言語・フレームワークの検出
   • 変更カテゴリの判定（認可変更、DB変更、RLS変更、API変更、テスト変更、設定変更、スキル変更）
   • 必要なリファレンスファイルの判定
   • Minor/Suggestionレベルの表層的問題の検出（インクリメンタルモード時は変更ファイルのみ）
   • 差分の要約（200文字以内）
   • レビュー時に注目すべきポイント
6. 分析結果を .pr-triage.json ファイルに出力する

インクリメンタルモード（.pr-review-state.json が存在する場合）

前回のレビュー状態を活用してチェック範囲を最適化する：

1. .pr-review-state.json の last_reviewed_commit を取得する
2. git diff <last_reviewed_commit>..HEAD --name-only で前回レビュー以降に変更されたファイルを特定する
3. 変更のないファイルの surface_issues は前回の状態からそのまま引き継ぐ（"carried_over": true を付与）。再チェックしない。
4. 変更があったファイルのみ表層チェックを実行する
5. 前回の surface_issues と review_comments のうち、該当箇所が修正されたものを resolved_issues に含める
6. 出力の .pr-triage.json に "incremental": true、"base_commit"、"changed_since_last_review"、"unchanged_since_last_review"、"resolved_issues" フィールドを追加する

resolved_issues のフォーマット：

{  "comment_id": 12345678,  "file": "<ファイルパス>",  "line": 42,  "issue": "<元の問題の説明>",  "resolution": "fixed"}

.pr-review-state.json が存在しない場合は、通常のフルトリアージを実行する。 .pr-review-state.json が不正な形式（JSONパースエラー等）の場合は、警告を出力してフルトリアージを実行する。破損した状態ファイルに基づいてインクリメンタルモードを実行してはならない。

必要なリファレンスの判定基準

以下の条件に該当する場合のみ、対応するリファレンスを required_references に含める：

• TypeScript/JavaScript ファイルの変更あり → typescript-best-practices.md
• 認証・認可に関わるコード変更あり（auth, permission, role, session, token等のキーワード） → authorization-review-general.md
• PostgreSQL RLS に関わる変更あり（RLS, row level security, policy等のキーワード） → authorization-review-postgres-rls.md
• SKILL.md ファイルの変更あり → skill-review.md
• CI環境でのGitHub投稿が必要 → github-pr-review-actions.md（常に含める）

表層チェック項目（Minor/Suggestion）

差分を確認し、以下の問題を検出する（該当するもののみ）：

• any 型の使用（TypeScript）
• var キーワードの使用（TypeScript/JavaScript）
• 空のインターフェイス定義
• マジックナンバーの使用
• 命名規則違反（camelCase/PascalCase/UPPER_CASE）
• テストの未追加（新規ファイルがあるのにテストファイルがない）

出力フォーマット

.pr-triage.json に以下のJSON構造で出力すること：

{  "pr_number": 123,  "incremental": false,  "base_commit": "<前回レビュー時のコミットSHA（インクリメンタル時のみ）>",  "summary": "<変更の概要（1-2文）>",  "files": {    "added": ["<追加ファイルパス>"],    "modified": ["<変更ファイルパス>"],    "deleted": ["<削除ファイルパス>"]  },  "changed_since_last_review": ["<前回から変更があったファイル（インクリメンタル時のみ）>"],  "unchanged_since_last_review": ["<前回から変更がないファイル（インクリメンタル時のみ）>"],  "languages": ["<検出された言語>"],  "frameworks": ["<検出されたフレームワーク>"],  "change_categories": {    "has_auth_changes": false,    "has_db_changes": false,    "has_rls_changes": false,    "has_api_changes": false,    "has_test_changes": false,    "has_config_changes": true,    "has_skill_changes": true  },  "required_references": ["<必要なリファレンスファイル名>"],  "surface_issues": [    {      "severity": "Minor|Suggestion",      "file": "<ファイルパス>",      "line": 15,      "issue": "<問題の説明>",      "suggestion": "<改善案>",      "carried_over": true    }  ],  "resolved_issues": [    {      "comment_id": 12345678,      "file": "<ファイルパス>",      "line": 42,      "issue": "<元の問題の説明>",      "resolution": "fixed"    }  ],  "diff_summary": "<差分の要約（200文字以内）>",  "estimated_complexity": "low|medium|high",  "focus_areas": ["<レビュー時の注目ポイント>"]}

重要:

• テキスト出力は最小限にし、.pr-triage.json の出力に集中してください。