pr-code-review-and-quality

PR コードレビューと品質

依頼された変更について、正確性、読みやすさ、アーキテクチャ、セキュリティ、パフォーマンスの観点からレビューします。改善点は、リポジトリの規約と具体的なリスクに照らして評価します。完璧さや個人的な好みは、マージの必須条件ではありません。

PR の対象: ユーザーが指定した PR 番号、owner/repo#number、または URL を必須とします。以前の明確な言及も指定として扱います。現在のブランチから推測したり、オープン中の PR を選んだりしないでください。対象が指定されていない場合は、ユーザーに確認してください。

出力ルール: ユーザーがこのスキルの GitHub 投稿ワークフローを明示的に依頼した場合、その依頼を COMMENT レビュー 1 件の投稿許可として扱います。再度確認する必要はありません。投稿後、レビューの URL と 1 段落の概要を報告してください。スキルの自動選択や PR の一般的な調査依頼は投稿の許可にはなりません。明示的なローカルのみ、または読み取り専用の依頼には従ってください。確定した提出方法を、委任先にも伝えてください。

対象と権限

ユーザーの入力、確立された会話の文脈、または会話内で以前に明示された PR から、所有者、リポジトリ、PR 番号、head SHA を特定します。対象が不明な場合にのみ確認してください。

上記の出力ルールで確定した提出方法を使用してください。レビュー中に実装コードを変更しないでください。

根拠と網羅性

PR の意図、差分、関連する CI 結果を確認します。動作を評価するために必要に応じて、変更されたファイル、呼び出し元、テストを読みます。リスクに応じて、確認する順序と深さを選びます。大規模な変更は一貫性のあるグループに分けてレビューし、確認できなかった範囲があれば明示します。

5 つの評価軸とチーム標準には criteria.md を使用してください。セキュリティに関わる変更では security-checklist.md を、パフォーマンスが対象となる場合は performance-checklist.md を読み込んでください。

作成者が実施したチェックによって、変更後の動作が確認できているか評価します。根拠が不足している場合は、対象を絞った検証を追加してください。新たな変更、失敗、または未解決の懸念がない限り、成功したチェックを繰り返さないでください。

指摘事項

以下の重要度ラベルを一貫して使用してください。

接頭辞	意味
なし	必須の修正
Critical:	マージを妨げるセキュリティ脆弱性、データ損失、または機能の破損
Nit:	任意の軽微なスタイル上の好み
Optional: / Consider:	改善の提案
FYI:	作成者が把握していない関連情報

各指摘には、具体的な根拠、役立つファイル名と行番号の参照、影響の説明が必要です。称賛、コードの言い換え、推測に基づく懸念は記載しないでください。指摘事項がない場合は、問題がないことを簡潔に伝えてください。

提出

投稿の許可がある場合は、github-delivery.md を使用してください。イベント COMMENT でレビューを 1 件投稿し、APPROVE または REQUEST_CHANGES は決して使用しないでください。インラインコメントはレビュー対象の差分に紐付ける必要があります。それ以外の指摘は概要に記載してください。ユーザーおよびリポジトリの帰属ポリシーを守ってください。

依頼された範囲をレビューし、許可された提出先に指摘事項を届け、検証上の制限を明示した時点で完了です。GitHub に投稿する場合は、投稿されたレビューを確認し、その URL を報告してください。新しいコミットが追加された後は、以前の指摘事項を重複させずに差分を評価するため、re-review.md を使用してください。