プルリクエストのレビュー
- 差分を取得します。PR番号がある場合は
gh pr diff <番号>を使用し、それ以外の場合はgit diff origin/main...HEADを使用します。また、PRの説明が存在する場合はそれを読んでください(gh pr view <番号>)。 - 差分だけでは不明瞭な場合、変更されたファイルを全文読んでください。読んでいないコードはレビューしないでください。
- 以下の順序で確認します:
- 正確性: 誤った条件、オフバイワンエラー、未処理のエラー、競合状態、null/空の入力。
- テスト: 各動作変更はカバーされていますか?変更なしでテストは失敗しますか?
- セキュリティ: 信頼できない入力がシェル、SQL、ファイルパス、HTMLに渡されていないか;差分にシークレットが含まれていないか。
- 互換性: 変更された公開インターフェース、マイグレーション、設定または環境変数。
- 可読性: 次のメンテナンス担当者の速度を低下させる問題のみ。リンターが既にカバーしている純粋なスタイルの好みはスキップします。
- テストが存在する場合は
npm testを実行し、結果を正直に報告します。
出力形式
## 概要
PRが何をするのかと、あなたの総合的な判定(承認 / 変更依頼 / コメント)を1~2文で記述します。
## 必須修正
- `path/file.js:42` 何が問題か、なぜ重要か、修正案。
## 検討すべき事項
- `path/file.js:90` ...
## あると良いこと
- ...
空のセクションは省略してください。発見事項ごとに ファイル:行番号 を明示してください。問題をでっち上げないでください:変更が問題なければ、概要でその旨を伝えてください。ユーザーが依頼しない限り、GitHubにコメントを投稿しないでください。