Smart Review

---
description: "現在の状況を分析し最適なロール・アプローチを自動提案する。「レビューして」「コード品質をチェック」「最適なアプローチは？」「どう改善すべき？」などで起動。"
allowed-tools:
  - Read
  - Grep
  - Glob
---

# 状況分析し最適なロール・アプローチを自動提案

現在の状況を分析し、最適なロール・アプローチを自動提案するコマンド。

## 使い方

```bash
/smart-review                    # 現在のディレクトリを分析
/smart-review <ファイル/ディレクトリ> # 特定対象を分析
```

## 自動判定ロジック

## ファイル拡張子による判定

- `package.json`, `*.tsx`, `*.jsx`, `*.css`, `*.scss` → **frontend**
- `Dockerfile`, `docker-compose.yml`, `*.yaml` → **architect**
- `*.test.js`, `*.spec.ts`, `test/`, `__tests__/` → **qa**
- `*.rs`, `Cargo.toml`, `performance/` → **performance**

## セキュリティ関連ファイル検出

- `auth.js`, `security.yml`, `.env`, `config/auth/` → **security**
- `login.tsx`, `signup.js`, `jwt.js` → **security + frontend**
- `api/auth/`, `middleware/auth/` → **security + architect**

## 複合判定パターン

- `mobile/` + `*.swift`, `*.kt`, `react-native/` → **mobile**
- `webpack.config.js`, `vite.config.js`, `large-dataset/` → **performance**
- `components/` + `responsive.css` → **frontend + mobile**
- `api/` + `auth/` → **security + architect**

## エラー・問題分析

- スタックトレース、`error.log`, `crash.log` → **analyzer**
- `memory leak`, `high CPU`, `slow query` → **performance + analyzer**
- `SQL injection`, `XSS`, `CSRF` → **security + analyzer**

## 提案パターン

## 単一ロール提案

```bash
$ /smart-review src/auth/login.js
→ 「認証ファイルを検出しました」
→ 「security ロールでの分析を推奨します」
→ 「実行しますか？ [y]es / [n]o / [m]ore options」
```

## 複数ロール提案

```bash
$ /smart-review src/mobile/components/
→ 「📱🎨 モバイル + フロントエンド要素を検出」
→ 「推奨アプローチ:」
→ 「[1] mobile ロール単体」
→ 「[2] frontend ロール単体」
→ 「[3] multi-role mobile,frontend」
→ 「[4] role-debate mobile,frontend」
```

## 問題分析時の提案

```bash
$ /smart-review error.log
→ 「⚠️ エラーログを検出しました」
→ 「analyzer ロールで根本原因分析を開始します」
→ 「[自動実行] /role analyzer」

$ /smart-review slow-api.log
→ 「🐌 パフォーマンス問題を検出」
→ 「推奨: [1]/role performance [2]/role-debate performance,analyzer」
```

## 複雑な設計決定時の提案

```bash
$ /smart-review architecture-design.md
→ 「🏗️🔒⚡ アーキテクチャ + セキュリティ + パフォーマンス要素検出」
→ 「複雑な設計決定のため、議論形式を推奨します」
→ 「[推奨] /role-debate architect,security,performance」
→ 「[代替] /multi-role architect,security,performance」
```

## 提案ロジックの詳細

## 優先度判定

1. **Security** - 認証・認可・暗号化関連は最優先
2. **Critical Errors** - システム停止・データ損失は緊急
3. **Architecture** - 大規模変更・技術選定は慎重検討
4. **Performance** - ユーザー体験に直結
5. **Frontend/Mobile** - UI/UX 改善
6. **QA** - 品質保証・テスト関連

## 議論推奨条件

- 3 つ以上のロールが関連する場合
- セキュリティ vs パフォーマンスのトレードオフがある場合
- アーキテクチャの大幅変更が含まれる場合
- モバイル + Web の両方に影響がある場合

## 基本例

```bash
# 現在のディレクトリを分析
/smart-review
「最適なロールとアプローチを提案して」

# 特定ファイルを分析
/smart-review src/auth/login.js
「このファイルに最適なレビュー方法を提案して」

# エラーログを分析
/smart-review error.log
「このエラーの解決に最適なアプローチを提案して」
```

## 実裁例

## プロジェクト全体の分析

```bash
$ /smart-review
→ 「📊 プロジェクト分析中...」
→ 「React + TypeScript プロジェクトを検出」
→ 「認証機能 + API + モバイル対応を確認」
→ 「」
→ 「💡 推奨ワークフロー:」
→ 「1. security で認証系チェック」
→ 「2. frontend で UI/UX 評価」
→ 「3. mobile でモバイル最適化確認」
→ 「4. architect で全体設計レビュー」
→ 「」
→ 「自動実行しますか？ [y]es / [s]elect role / [c]ustom」
```

## 特定問題の分析

```bash
$ /smart-review "JWT の有効期限をどう設定すべきか"
→ 「🤔 技術的な設計判断を検出」
→ 「複数の専門観点が必要な問題です」
→ 「」
→ 「推奨アプローチ:」
→ 「/role-debate security,performance,frontend」
→ 「理由: セキュリティ・パフォーマンス・UX のバランスが重要」
```

## Claude との連携

```bash
# ファイル内容と組み合わせた分析
cat src/auth/middleware.js
/smart-review
「このファイルの内容を含めてセキュリティ観点で分析して」

# エラーと組み合わせた分析
npm run build 2>&1 | tee build-error.log
/smart-review build-error.log
「ビルドエラーの解決方法を提案して」

# 設計相談
/smart-review
「React Native と Progressive Web App のどちらを選ぶべきか議論して」
```

## 注意事項

- 提案は参考情報です。最終的な判断はユーザーが行ってください
- 複雑な問題ほど議論形式 (role-debate) を推奨します
- 単純な問題は single role で十分な場合が多いです
- セキュリティ関連は必ず専門ロールでの確認を推奨します