// PRを専門家としてレビューし、Approve/Request Changesを判定する。第二引数で外部AI（codex / gemini）への委譲も可能
[HINT] Download the complete skill directory including SKILL.md and all related files
| name | review |
| description | PRを専門家としてレビューし、Approve/Request Changesを判定する。第二引数で外部AI（codex / gemini）への委譲も可能 |
| argument-hint | [PR番号] [AIエージェント(codex|gemini)] |
| disable-model-invocation | true |
| allowed-tools | ["Bash","Read","Glob","Grep"] |
直前PR、または引数で指定されたPRを専門家としてレビュー。
[PR番号]: レビュー対象のPR番号（省略時は直前のPR）[AIエージェント]: レビュー実行者（任意）
codex: Codex CLI に委譲gemini: Gemini CLI に委譲言語慣用性（Idiomatic）・可読性・コード品質・保守性・セキュリティ・テストカバレッジ
レビュー結果は GitHub の PR レビュー機能 を使って必ず PR 上に書き込む。 個別指摘は コード行に紐付くインラインコメント が原則。総評（review body）にだけ書くのは避ける。
| 指摘の種類 | 投稿先 |
|---|---|
| 特定ファイル・特定行への指摘 | インラインコメント (comments[].path + line) |
| 複数ファイルにまたがる設計指摘 | 代表箇所にインラインコメント + review body に補足 |
| 設計レベル・PR全体の所見 | review body（総評） |
| ファイル単位の指摘（行を絞れない） | そのファイルの代表行にインラインコメント |
gh api の Reviews API を使い、総評 + 複数のインラインコメント + 判定（event）を 1 回で送信 する。
PR=<PR番号>
OWNER_REPO=$(gh repo view --json nameWithOwner -q .nameWithOwner)
SHA=$(gh pr view "$PR" --json headRefOid -q .headRefOid)
# 1. インラインコメントを JSON 配列で組み立て
# （path / line / side / body の 4 つが必須。複数行レンジは start_line を併用）
#
# ▼ 推奨: jq -n でシェル変数を安全に流し込む（特殊文字混入時の JSON 破損を防ぐ）
SUMMARY=$'## 総評\n\n... 全体所見をここに ...'
jq -n \
--arg sha "$SHA" \
--arg event "REQUEST_CHANGES" \
--arg body "$SUMMARY" \
'{
commit_id: $sha,
event: $event,
body: $body,
comments: [
{path: "src/foo.py", line: 42, side: "RIGHT",
body: "[major / 可読性] この関数は 70 行ある。〇〇 と △△ に分割を推奨。"},
{path: "src/bar.py", start_line: 10, line: 25, side: "RIGHT",
body: "[minor / 性能] このループは内包表記化できる。"}
]
}' > /tmp/review-payload.json
# 2. Reviews API に POST
gh api -X POST "repos/$OWNER_REPO/pulls/$PR/reviews" --input /tmp/review-payload.json
💡 JSON 組み立てに heredoc (
<<JSON) は使わない: 変数展開は必要だが、$SHA等に特殊文字が混入した場合 JSON が壊れる（あるいはクオート未エスケープで JSON injection になる）。jq -n --arg経由なら値が自動で JSON エスケープされるため安全。クオート付き heredoc (<<'JSON') は逆に$SHAが展開されず使えない。
event の値:
APPROVE — 指摘なしREQUEST_CHANGES — 修正必須の指摘ありCOMMENT — 任意の指摘のみ（マージブロックしない）各 comments[].body の先頭に [重要度 / カテゴリ] を付けて視認性を上げる:
[critical / セキュリティ] SQL がエスケープなしで連結されている。プレースホルダ必須。
[major / 可読性] 70 行関数。〇〇 と △△ に分割を推奨。
[minor / 言語慣用性] Python なら内包表記で 1 行化可能。
[nit / スタイル] スペースが揃っていない。
重要度の目安:
critical — セキュリティ・データ破損・本番障害につながるmajor — 保守性 / 性能 / 仕様逸脱の重要問題minor — 改善推奨だがブロッカーではないnit — 好み・スタイル同じ箇所への二重指摘を避けるため、投稿前に既存コメントを確認する:
# 既存のレビューコメント一覧
gh api "repos/$OWNER_REPO/pulls/$PR/comments" --paginate \
| jq -r '.[] | "\(.path):\(.line) \(.body | split("\n")[0])"'
すでに同種の指摘があれば、その指摘は省くか、reply（既存コメントへの返信）にする。
# review body 単体（インラインなし）で投稿したい場合
gh pr review "$PR" --request-changes --body "..."
gh pr review "$PR" --approve --body "..."
# 会話タブへの普通のコメント（行に紐付かない）
gh pr comment "$PR" --body "..."
# 1 件だけインラインコメントを追加（既存 review に含めない）
gh api -X POST "repos/$OWNER_REPO/pulls/$PR/comments" \
-F commit_id="$SHA" \
-F path="src/foo.py" \
-F line=42 -F side=RIGHT \
-F body="..."
第二引数が指定された場合、上記「観点」「具体的なチェックポイント」「レビュー結果の投稿」の内容を レビュー指示プロンプト として組み立て、指定された CLI に渡す。
gh pr view <PR番号> --json title,body,baseRefName,headRefName,url,headRefOid で PR メタ情報を取得gh pr diff <PR番号> で差分を取得（または変更ファイル一覧 + 必要箇所を gh pr view <PR番号> --json files 経由で抽出）外部AIは Reviews API ペイロードを組み立てた後、自分自身で gh api を呼んで PR に投稿する。
（旧版では生成した JSON をメインに返してメインが投稿していたが、メイン context 消費と往復回数が無駄なので削除）
メインに返すのは「投稿が成功したか」「最終 verdict (event)」「review URL」「件数」の小さな結果サマリのみ。
## 出力形式と投稿手順（必須）
レビュー結果は以下の手順で **あなた自身が PR に投稿** してください。
メイン側に返すのは投稿結果サマリだけです。
### 1. ペイロード組み立て
以下の JSON を `/tmp/<agent>-review-pr<番号>-payload.json` に書き出す
（codex なら `apply_patch`、gemini なら `write_file` を使用）:
\`\`\`json
{
"commit_id": "<headRefOid>",
"event": "REQUEST_CHANGES" | "APPROVE" | "COMMENT",
"body": "## 総評\n\n...(設計レベル・PR全体所見のみ)...",
"comments": [
{
"path": "src/foo.py",
"line": 42,
"side": "RIGHT",
"body": "[major / 可読性] ..."
}
]
}
\`\`\`
ルール:
- 個別指摘は必ず `comments[]` のインラインコメントにすること（行を絞れない場合はファイル代表行）
- `body` (総評) には設計・横断的な所見のみ書く。個別指摘の繰り返しは禁止
- 各 `comments[].body` の先頭に `[重要度 / カテゴリ]` を付ける（critical/major/minor/nit）
- `path` は **PR差分に登場するファイルのみ**（事前に `gh pr diff <PR> --name-only` で取得した一覧から選ぶ）
- `line` は **差分に含まれる行**（追加行・コンテキスト行）に限る。`side=RIGHT` がデフォルト
- `commit_id` は `gh pr view <PR> --json headRefOid -q .headRefOid` の値を使う
### 2. 投稿
\`\`\`bash
OWNER_REPO=$(gh repo view --json nameWithOwner -q .nameWithOwner)
gh api -X POST "repos/$OWNER_REPO/pulls/<PR>/reviews" \
--input /tmp/<agent>-review-pr<番号>-payload.json \
> /tmp/<agent>-review-pr<番号>-response.json
\`\`\`
### 3. 結果サマリの書き出し（メインが読む）
`/tmp/<agent>-review-pr<番号>-result.json` に投稿結果を書き出す:
\`\`\`json
{
"status": "posted" | "failed",
"event": "REQUEST_CHANGES" | "APPROVE" | "COMMENT",
"review_url": "https://github.com/.../pull/<PR>#pullrequestreview-...",
"comments_count": 5,
"by_severity": {"critical": 0, "major": 2, "minor": 2, "nit": 1},
"payload_path": "/tmp/<agent>-review-pr<番号>-payload.json",
"error": null
}
\`\`\`
投稿失敗時は `status: "failed"`、`error` にエラーメッセージ、`payload_path` で payload は残す
（メイン側のフォールバック投稿で使う）。
### 4. 重要度の運用ガイド（auto-fix 判定に直結）
| 重要度 | 定義 | 後段の扱い |
|---|---|---|
| critical | セキュリティ・データ破損・本番障害につながる | **必ず自動修正** |
| major | 保守性・性能・仕様逸脱の重要問題 | **必ず自動修正** |
| minor | 改善推奨だがブロッカーではない | **自動修正対象**（明らかな改善のみ。判断要なら nit に格下げ） |
| nit | 好み・スタイル | **修正しない、最後にユーザ判断にまとめる** |
過剰な nit 量産は避ける。critical/major で対応すべき真の問題に集中すること。
codex 指定時呼び出し手順の詳細は /ndf:codex skill（plugins/ndf/skills/codex/SKILL.md）に従う。要点:
/tmp/codex-review-pr<番号>-prompt.md に書き出し/tmp/codex-output-review-pr<番号>.md として プロンプト内で apply_patch 書き出しを必須化codex exec --dangerously-bypass-approvals-and-sandbox --config reasoning.effort=medium -C "$PWD" < prompt > stdout 2> err & でバックグラウンド起動grep -q '^tokens used$' err で完了検知⚠️
--dangerously-bypass-approvals-and-sandboxのセキュリティ注意: このフラグは codex の bwrap サンドボックスを完全に無効化し、 任意のシェル実行・任意のファイル編集を無確認で許可する。必ず Docker / devcontainer / VM / CI ランナー等の外部隔離環境内 でのみ使用すること。 ホスト直接実行や本番リポジトリでは使わない。詳細な背景・代替策（unprivileged_userns_clone有効化など）は/ndf:codexskill の 「サンドボックス制約」節を参照。
gemini 指定時呼び出し手順の詳細は /ndf:gemini skill（plugins/ndf/skills/gemini/SKILL.md）に従う。要点:
/tmp/gemini-review-pr<番号>-prompt.md に書き出し--yolo 必須（gh api -X POST がシェル実行のため、plan / auto_edit だとブロックされる）gh api で投稿するだけ」 を強く明示することで --yolo のリスクを抑えるgemini --yolo --output-format text -p "$(cat prompt.md)" > stdout 2> err & でバックグラウンド起動kill -0 $PID ポーリングで完了検知（Codex と異なり sentinel 不要 / プロセス exit を見る）/tmp/gemini-review-pr<番号>-result.json で回収⚠️
--yoloの制約は依然有効:/ndf:geminiskill のセキュリティ警告通り、必ず外部隔離環境内でのみ実行する。プロンプトで「リポジトリ編集禁止」を明示することは必須だが、それは sandbox の代替にはならない。
メインエージェントの責務は 結果サマリ読み込みと検証のみ:
AGENT=codex # or gemini
RESULT=/tmp/$AGENT-review-pr$PR-result.json
if [ ! -s "$RESULT" ]; then
echo "❌ $AGENT: 結果サマリ未生成。完了検知 or プロンプト指示に問題あり" >&2
exit 1
fi
STATUS=$(jq -r '.status' "$RESULT")
EVENT=$(jq -r '.event // empty' "$RESULT")
if [ "$STATUS" = "failed" ]; then
echo "⚠️ $AGENT: 投稿失敗。payload からメインがフォールバック投稿します" >&2
PAYLOAD=$(jq -r '.payload_path' "$RESULT")
OWNER_REPO=$(gh repo view --json nameWithOwner -q .nameWithOwner)
SHA=$(gh pr view "$PR" --json headRefOid -q .headRefOid)
jq --arg sha "$SHA" '.commit_id = $sha' "$PAYLOAD" > /tmp/review-fallback.json
gh api -X POST "repos/$OWNER_REPO/pulls/$PR/reviews" --input /tmp/review-fallback.json
fi
echo "$AGENT: event=$EVENT url=$(jq -r .review_url $RESULT)"
Claude 自身による追加判定は行わず、外部AIの判定（event）と指摘内容をそのまま採用する。
レビュー結果は PR 上に投稿済み であることが前提。ユーザーへの報告は以下に絞る:
詳細な指摘内容は PR 上のインラインコメントに残っているため、ユーザー宛報告では繰り返さない。