review

# Review Skills コードレビューと品質チェックを担当するスキル群です。 ## 含まれる小スキル | スキル | 用途 | |--------|------| | review-changes | 変更内容のレビュー | | review-quality | コード品質チェック | | review-security | セキュリティレビュー | | review-performance | パフォーマンスレビュー | | review-accessibility | アクセシビリティチェック | ## ルーティング ユーザーの意図に応じて適切な小スキルを選択: - 一般的なレビュー: review-changes/doc.md - 品質重視: review-quality/doc.md - セキュリティ重視: review-security/doc.md - パフォーマンス重視: review-performance/doc.md - アクセシビリティ重視: review-accessibility/doc.md ## 実行手順 1. **品質判定ゲート**（Step 0） 2. ユーザーのリクエストを分類 3. **（Claude-mem 有効時）過去のレビュー指摘を検索** 4. 並列実行の判定（下記参照） 5. 適切な小スキルの doc.md を読む、または並列サブエージェント起動 6. 結果を統合してレビュー完了 ### Step 0: 品質判定ゲート（レビュー重点領域の特定） レビュー開始前に変更内容を分析し、重点領域を特定: ``` 変更ファイル分析 ↓ ┌─────────────────────────────────────────┐ │ 品質判定ゲート │ ├─────────────────────────────────────────┤ │ 判定項目: │ │ ├── カバレッジ不足？（テストなし） │ │ ├── セキュリティ注意？（auth/api/） │ │ ├── a11y 注意？（UI コンポーネント） │ │ └── パフォーマンス注意？（DB/ループ） │ └─────────────────────────────────────────┘ ↓ 重点レビュー領域を決定 ``` #### カバレッジ判定 | 状況 | 指摘内容 | |------|---------| | 新規ファイルにテストなし | 「テストが不足しています」 | | 変更ファイルのテストが古い | 「テストの更新を検討してください」 | | カバレッジ < 60% | 「カバレッジ向上を推奨」 | #### セキュリティ重点レビュー | パス | 追加チェック項目 | |------|-----------------| | auth/, api/ | OWASP Top 10 チェックリスト | | 入力処理 | サニタイズ、バリデーション | | DB クエリ | パラメータ化確認 | #### a11y 重点レビュー | パス | チェック項目 | |------|------------| | src/components/ | alt, aria, キーボード操作 | | src/pages/ | 見出し構...