mcpField
C
code-reviewer

code-reviewer

github.com/nodnarbnitramテスト・品質16293 回インストール27 週間前に更新
リポジトリ

code-reviewer について

開発者向けのコードレビュー用スキルです。品質、セキュリティ、エラー処理、性能、テストを確認し、重大度別にフィードバックを整理します。

主な機能

  1. 01コード品質チェックリスト
  2. 02セキュリティと入力の確認
  3. 03エラーと性能のレビュー
  4. 04テストカバレッジの確認
  5. 05重大度別のフィードバック

活用シーン

  1. 01複数のソースファイルのレビュー
  2. 02セキュリティや検証漏れの発見
  3. 03重大度に応じた修正の優先付け

インストール

コマンドをコピーしてターミナルで実行します。

$npx skills add https://github.com/nodnarbnitram/claude-code-extensions --skill code-reviewer

インストール後、Claude Code で/code-reviewerを入力して読み込まれたか確認します。

詳細

コードレビュー

品質、セキュリティ、保守性に重点を置いて、包括的なコードレビューを実施します。

手順

  1. Read ツールを使って対象ファイルを読みます
  2. Grep を使ってパターンや関連コードを検索します
  3. Glob を使って関連ファイルを見つけます
  4. レビューチェックリストに照らしてコードを分析します
  5. 重要度別に整理したフィードバックを提示します

レビューチェックリスト

コード品質
  • コードがシンプルで読みやすい
  • 関数と変数に適切な名前が付けられている
  • コードが重複していない（DRY 原則）
  • 複雑なロジックに適切なコメントが付けられている
  • コードスタイルに一貫性がある
セキュリティ
  • シークレットや API キーが公開されていない
  • 入力値が検証されている
  • SQL インジェクションが防止されている
  • Web コードで XSS が防止されている
  • 認証／認可チェックが適切に行われている
エラー処理
  • エラーが捕捉され、適切に処理されている
  • エラーメッセージが分かりやすい
  • エラーが黙って無視されない
  • デバッグに必要なログが適切に記録される
パフォーマンス
  • 明らかなパフォーマンスのボトルネックがない
  • アルゴリズムとデータ構造が効率的である
  • 必要に応じて適切なキャッシュが使われている
  • データベースクエリが最適化されている
テスト
  • テストカバレッジが十分である
  • エッジケースがテストされている
  • テストが読みやすく、保守しやすい

出力形式

重要度別にフィードバックを整理します。

Critical（必ず修正）

セキュリティの脆弱性、データ損失、クラッシュにつながる可能性のある問題です。

Warning（修正を推奨）

バグ、パフォーマンスの低下、保守性の問題につながる可能性のある問題です。

Suggestion（検討）

読みやすさ、一貫性、ベストプラクティスを改善する提案です。

フィードバック例

### Critical
- **SQL Injection vulnerability** in `user_service.py:45`
  - User input passed directly to query without sanitization
  - Fix: Use parameterized queries

### Warning
- **Missing error handling** in `api_client.py:23`
  - Network errors will crash the application
  - Fix: Add try/catch with appropriate error response

### Suggestion
- Consider extracting the validation logic in `validators.py:78-95` into a separate function for reusability

コメント

Markdown 対応