code-review-checklist について
Pull Requestの変更内容に合わせ、優先度付きのコードレビュー項目を作成するスキルです。変更の種類や言語、プロジェクトの慣習を踏まえ、安全性や保守性などの確認を支援します。
主な機能
- 01変更ファイルと種類の分析
- 02言語別チェックの適用
- 03プロジェクト慣習の考慮
- 04実行可能な項目の優先順位付け
- 05安全性・性能・テスト・保守性の確認
活用シーン
- 01Pull Requestのレビュー前に項目を準備
- 02新メンバーへのレビュー基準共有
- 03不慣れなコード領域のレビュー
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/インストール後、Claude Code で/code-review-checklistを入力して読み込まれたか確認します。
同梱ファイル.claude-plugin/plugin.json
詳細
コードレビュー用チェックリストジェネレーター
各 PR に固有の重要事項にレビュー担当者が集中できるよう、状況に即した詳細なコードレビュー用チェックリストを生成します。
使用するタイミング
- コードレビューを始める前に、確認すべき点を把握したいとき
- 新しいチームメンバーにレビュー基準を説明するとき
- チーム全体で一貫したレビュー品質を保ちたいとき
- コードベースの詳しくない部分をレビューするとき
アプローチ
- 差分を分析する：変更されたファイルと変更内容を把握します
- パターンを特定する：変更の種類（機能追加、バグ修正、リファクタリングなど）を検出します
- 言語固有のチェック：プログラミング言語に関連するチェックを適用します
- プロジェクトのコンテキスト：コードベース内の既存パターンや規約を考慮します
- チェックリストを生成する：優先順位を付けた、実行可能なレビュー項目を作成します
チェックリストのカテゴリ
セキュリティ
- 入力が検証されている
- ハードコードされたシークレットや認証情報がない
- 適切な認証／認可チェックがある
- SQL インジェクションが防止されている
- Web コードで XSS が防止されている
パフォーマンス
- N+1 クエリのパターンがない
- 適切なキャッシュが検討されている
- 不要なループや反復処理がない
- 効率的なデータ構造が使われている
保守性
- コードが読みやすく、自己説明的である
- 関数のサイズが適切である
- コードの重複がない
- 命名規則に一貫性がある
テスト
- ユニットテストが新機能をカバーしている
- エッジケースがテストされている
- テストがカバレッジ目的だけでなく、意味のあるものになっている
ベストプラクティス
- セキュリティ上の問題を最優先します
- スタイルに関する細かな指摘より、ロジックの誤りに重点を置きます
- レビュー担当者の時間を考慮し、重要な項目を強調します
- プロジェクトの成熟度に合わせてチェックリストを調整します
コメント
Markdown 対応