コード品質レビュー（統合版）

統合スコープ

•アーキテクチャ - クリーンアーキテクチャ、DDD、依存関係
•コード臭 - 長すぎる関数、重複、マジックナンバー、複雑度
•パフォーマンス - N+1問題、メモリリーク、非効率アルゴリズム
•型安全性 - any/as使用、null安全性、型ガード

使用タイミング

•コードレビュー時 / リファクタリング時 / /review コマンド実行時

レビュー観点

🔴 Critical（修正必須）

カテゴリ	観点	検出パターン	対策
アーキ	依存逆転	Domain→Infrastructure参照	Interface分離
アーキ	ロジック配置	Controller内ビジネスロジック	UseCase/Domainに移動
コード臭	長い関数	100行超	責務分離で分割
コード臭	マジックナンバー	`if (age > 18)`	定数化
パフォ	N+1問題	ループ内クエリ	バッチ取得
パフォ	メモリリーク	cleanup未実装	useEffect cleanup
型安全	any使用	`data: any`	unknown + 型ガード
型安全	無検証as	`data as User`	型ガード関数

🟡 Warning（要改善）

カテゴリ	観点	検出パターン	対策
アーキ	Fat Service	500行超/複数責務	責務分離
コード臭	深いネスト	3階層以上	早期リターン
パフォ	非効率	O(n²)アルゴリズム	Map使用でO(n)
型安全	冗長注釈	推論可能な型指定	型推論活用

コード例が必要な場合: Context7で「clean architecture」「TypeScript type guard」を検索

チェックリスト

アーキテクチャ: Domain外部依存なし / ロジックがDomain/UseCaseに / 循環依存なし

コード臭: 関数50行以内 / マジックナンバーなし / 重複3回未満

パフォーマンス: N+1なし / cleanup実装 / O(n²)以上なし

型安全: any未使用 / as使用時型ガード / strictNullChecks有効

出力形式

code

## コード品質レビュー結果

### アーキテクチャ
🔴 Critical: `ファイル:行` - 問題 - 修正案

### コード臭 / パフォーマンス / 型安全性
🔴 Critical: `ファイル:行` - 問題 - 修正案

📊 Summary: Critical X件 / Warning Y件

外部リソース

•Context7: DDD、クリーンアーキテクチャ、SOLID原則、TypeScript公式
•Serena memory: プロジェクト固有のレイヤー構成・コーディング規約