review-pr-changes

PR の変更をレビューする

プロジェクトの品質基準に照らして、PR の変更を体系的にレビューします。

クイックリファレンス

# Get PR files changed
gh pr diff <pr> --name-only

# View specific file diff
gh pr diff <pr> -- path/to/file

# Get PR review status
gh pr view <pr> --json reviews

# Check file statistics
gh pr diff <pr> | diffstat

# Get PR body/description
gh pr view <pr> --json body

レビューチェックリスト

コード品質：

• コードが読みやすく、適切に構成されている
• 関数／クラスの目的が明確である
• コードが重複していない（DRY 原則）
• 適切なエラー処理が行われている
• 不必要な複雑さがない

テスト：

• 新しい機能に対するテストがある
• テストが成功している（CI がグリーンである）
• エッジケースがカバーされている
• 変更に対して十分なカバレッジがある

ドキュメント：

• 公開 API に docstring がある
• 型ヒントがある
• 一目でわからないコードにコメントがある

セキュリティと Git：

• ハードコードされたシークレット／トークンがない
• 入力検証が行われている
• PR が課題にリンクされている（説明内）
• 意図しないファイルが含まれていない

レビューワークフロー

1. コンテキストを確認する：PR の説明とリンク先の課題を確認します
2. 変更をざっと確認する：ファイル一覧と統計を確認します
3. コードを読む：実際の変更を注意深く確認します
4. チェックリストを使う：各カテゴリを順に確認します
5. ローカルでテストする：必要に応じて変更を取得してテストします
6. コメントを作成する：問題をコードコメントとして指摘します
7. 結論を示す：承認、変更を要求、またはコメントを選択します

出力形式

以下のセクションのみを使って、レビュー結果を報告します。

1. 概要 — 全体的な評価を1～2文で記述
2. 見つかった問題 — 修正が必要な問題（なければ省略）
3. 判定 — 承認／変更を要求／コメント

エラー処理

問題	解決策
PR にアクセスできない	gh auth status を確認します
コードを理解できない	コメントで明確化を求めます
ローカルでのテストが必要	worktree-create skill を使ってテストします
問題が複数ある	重大な問題を先に、任意の問題を後に優先します
スタイルについて意見が一致しない	CLAUDE.md の基準を参照します

参考資料

• プロジェクトの基準については CLAUDE.md を参照してください
• マージ準備の確認については verify-pr-ready を参照してください
• 一括レビューのワークフローについては gh-batch-merge-by-labels を参照してください