Code Review Skill
目的
コードベース全体を体系的に監査し、品質、セキュリティ、テスト、SDD準拠性の観点から改善点を特定します。
入力
src/- ソースコード全体docs/functional-design.md- 機能設計書（SDD）CLAUDE.md- 開発ガイドライン- プロジェクト設定ファイル（eslint.config.js, tsconfig.json, package.json等）
出力
docs/code-review-YYYY-MM-DD.md - コードレビューレポート（日付付き）
実行手順
1. 準備フェーズ
- プロジェクト構造を理解（README.md, CLAUDE.md確認）
- 既存の実装パターンを把握（lessonNotes, studentTraits参照）
- SDDの機能一覧と要件を読み込む
2. スコープ決定
- レビュー対象を決定（全体 or 特定feature）
- ユーザーに確認を求める
3. コードベーススキャン
以下の4つの観点で体系的にレビュー:
3.1 コーディング規約とベストプラクティス
- guide.mdの「1. コーディング規約チェックリスト」を参照
- TypeScript型安全性、React 19パターン、命名規則を確認
- ESLint違反、コンソールログ残存をチェック
3.2 テストカバレッジと品質
- guide.mdの「2. テストカバレッジチェックリスト」を参照
- カバレッジ80%以上を確認（npm run test:coverage）
- MSWモック実装パターンを検証
- エッジケースとエラーケースのテストを確認
3.3 セキュリティとバリデーション
- guide.mdの「3. セキュリティチェックリスト」を参照
- OWASP Top 10に基づく脆弱性チェック
- Zodバリデーション実装を確認
- XSS/CSRF対策、認証・認可の実装を検証
3.4 SDD準拠性
- guide.mdの「4. SDD準拠性チェックリスト」を参照
- 実装が受入基準を満たしているか確認
- APIエンドポイントが仕様通りか検証
- バリデーションルールがSDD定義と整合しているか確認
4. レポート生成
- template.mdを参照してレポートを作成
- 発見した問題を重要度別に分類（Critical/High/Medium/Low）
- 具体的な改善推奨事項を記載
- コード例とファイルパスを含める
5. 出力と確認
docs/code-review-YYYY-MM-DD.mdに保存- サマリーをユーザーに提示
- 優先度の高い問題から対応を推奨
参照ファイル
- template.md - レビューレポートのテンプレート構造
- guide.md - 詳細なレビューガイドライン（チェックポイント、判断基準、実装例）
効率的なスキャン戦略
大規模コードベースでは以下の戦略を採用:
- 優先度付けスキャン:
- Critical機能から優先的にレビュー（auth, lessonNotes, studentTraits） - 共通ライブラリ（lib/errors, lib/api）を重点的に確認
- パターンマッチング:
- 既知の実装パターン（lessonNotes）と比較 - 逸脱パターンを特定
- 自動化活用:
- Glob/Grepで効率的にファイル検索 - ESLintレポート、テストカバレッジレポートを活用
- 段階的レビュー:
- 1回のレビューで全てをカバーせず、feature単位で段階実施可能
使い方
基本的な使い方
/code-review特定featureのレビュー
/code-review lessonNotesレビュー後の対応
- 生成されたレポート（docs/code-review-YYYY-MM-DD.md）を確認
- Critical/High優先度の問題から対応
- 必要に応じて段階的に改善
注意事項
- このスキルは読み取り専用で、コード修正は行いません
- レポートは改善提案であり、全て対応必須ではありません
- プロジェクトの文脈や制約を考慮して判断してください
- 定期的（週次/月次）に実行し、継続的な品質改善に活用してください