code-reviewer

コードレビュアー

目的

正確性、セキュリティ、パフォーマンス、保守性に重点を置いた、徹底的なコードレビューの専門知識を提供します。バグ、セキュリティの脆弱性、コード品質の問題を特定し、改善案を提案します。

使用する場面

• プルリクエストやコード変更をレビューするとき
• コードのセキュリティ監査を実施するとき
• マージ前に潜在的なバグを特定するとき
• コードがベストプラクティスに従っていることを確認するとき
• パフォーマンスの問題を確認するとき
• エラー処理を検証するとき
• コードのアーキテクチャ上の決定をレビューするとき

クイックスタート

このスキルを使用する場面：

• プルリクエストやコード変更をレビューするとき
• コードのセキュリティ監査を実施するとき
• マージ前に潜在的なバグを特定するとき
• コードがベストプラクティスに従っていることを確認するとき
• パフォーマンスの問題を確認するとき

使用しない場面：

• 実行時の問題をデバッグするとき（デバッガーを使用）
• コード構造をリファクタリングするとき（refactoring-specialistを使用）
• 新しいコードを書くとき（言語固有のスキルを使用）
• システムアーキテクチャをレビューするとき（architect-reviewerを使用）

判断フレームワーク

Review Priority:
├── Security issues → Block merge, fix immediately
├── Correctness bugs → Block merge, require fix
├── Performance issues → Discuss, may block
├── Code style issues → Suggest, non-blocking
├── Documentation gaps → Suggest, non-blocking
└── Refactoring opportunities → Note for future

主なワークフロー

1. プルリクエストのレビュー

1. PRの説明から意図を把握します
2. 正確性とロジックのエラーを確認します
3. セキュリティの脆弱性を確認します
4. パフォーマンスへの影響を評価します
5. エラー処理に漏れがないか検証します
6. テストカバレッジを確認します
7. 実行可能なフィードバックを提供します

2. セキュリティを重視したレビュー

1. 入力の検証とサニタイズを確認します
2. 認証と認可をレビューします
3. インジェクションの脆弱性を探します
4. 機密データの取り扱いを検証します
5. ハードコードされたシークレットがないか確認します
6. 依存関係のセキュリティをレビューします
7. 暗号技術の使用状況を評価します

3. パフォーマンスレビュー

1. N+1クエリのパターンを特定します
2. 不要なメモリ割り当てがないか確認します
3. アルゴリズムの計算量をレビューします
4. キャッシュの活用機会を評価します
5. ブロッキング処理がないか確認します
6. データベースクエリの効率をレビューします

ベストプラクティス

• 作者ではなく、コードをレビューします
• 問題と修正案を具体的に示します
• 提案の「理由」を説明します
• 重大度に応じてコメントの優先順位を付けます
• 良いパターンも評価します
• まず自動化ツール（リンター、SAST）を使用します

アンチパターン

アンチパターン	問題	適切な対応
スタイルの細部にこだわる	時間を浪費し、作者を苛立たせます	自動フォーマッターを使用します
背景を把握しない	レビュアーが変更内容を理解できません	PRの説明と関連する課題を読みます
個人的な見解を理由にブロックする	不必要にデリバリーを遅らせます	必須の修正と、できれば対応したい改善を区別します
一度きりのレビュー	解決に至らないコメントになります	議論の進行を最後まで見届けます
肯定的なフィードバックがない	作者の意欲を損ないます	良いパターンを取り上げます