git-pr-reviewer について
開発者向けのプルリクエストレビュー用スキルです。差分からロジック、セキュリティ、性能、コード品質、エラー処理、テスト範囲を確認し、重要度別に結果をまとめます。
主な機能
- 01変更ファイルの確認
- 02ロジックの不具合確認
- 03セキュリティリスクの確認
- 04性能とコード品質の評価
- 05テスト範囲の不足確認
活用シーン
- 01mainとの差分レビュー
- 02PRのセキュリティ問題の発見
- 03変更コードのテスト確認
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ onewave-ai/ claude-skills --skill git-pr-reviewerインストール後、Claude Code で/git-pr-reviewerを入力して読み込まれたか確認します。
詳細
Git PR Reviewer
手順
プルリクエストをレビューするときは、次の手順に従います。
- 差分を取得する:
git diff main...HEADまたはgit diff <base-branch>...HEADを実行します - 変更されたファイルを分析する:変更、追加、削除されたすべてのファイルを特定します
- 各ファイルをレビューする:
- ロジックエラーとバグ
- セキュリティ脆弱性(SQL インジェクション、XSS、ハードコードされたシークレット)
- パフォーマンスの問題(N+1 クエリ、不要な再レンダリング、メモリリーク)
- コードスタイルと一貫性
- エラーハンドリングの不足
- テストカバレッジの不足
レビューチェックリスト
セキュリティ
- 認証情報や API キーがハードコードされていない
- ユーザーデータの入力検証が行われている
- 適切な認証/認可チェックが行われている
- SQL インジェクションの脆弱性がない
- XSS 対策が実装されている
コード品質
- 関数が小さく、役割が明確である
- コードの重複がない
- 変数名/関数名が明確である
- 適切なエラーハンドリングが行われている
- 未使用のインポートやデッドコードがない
パフォーマンス
- 不要なデータベースクエリがない
- 効率的なアルゴリズムが使用されている
- 必要に応じて適切なキャッシュが使用されている
- メモリリークがない
テスト
- 新しいコードにテストがある
- エッジケースがカバーされている
- テストが単にカバレッジを得るためではなく、有意義なものである
出力形式
## PR Review Summary
### Overview
[Brief summary of changes]
### Issues Found
#### Critical
- [Issue description + file:line]
#### Warnings
- [Issue description + file:line]
#### Suggestions
- [Improvement ideas]
### Approval Status
[APPROVE / REQUEST CHANGES / NEEDS DISCUSSION]
例
# Review current branch against main
git diff main...HEAD --stat
git diff main...HEAD
コメント
Markdown 対応