クリーンコードレビュー

『クリーンコード』の原則に基づき、7つの高収益なチェック項目に焦点を当てています。

レビューワークフロー

Review Progress:
- [ ] 1. Scan codebase: identify files to review
- [ ] 2. Check each dimension (naming, functions, DRY, YAGNI, magic numbers, clarity, conventions)
- [ ] 3. Rate severity (高/中/低) for each issue
- [ ] 4. Generate report sorted by severity

核となる原則：機能の保持

すべての提案は実装方法の最適化のみを対象とし、コードの機能、出力、または動作の変更は一切推奨しません。

チェック項目

1. 命名の問題【意味のある命名】

チェックの兆候：

• data1、temp、result、info、objなどの無意味な命名
• 同じ概念に複数の命名（get/fetch/retrieveの混用）

// ❌ 
const d = new Date();
const data1 = fetchUser();

// ✅ 
const currentDate = new Date();
const userProfile = fetchUser();

2. 関数の問題【関数は短く + SRP】

チェックの兆候：

• 関数が100行を超える
• パラメータが3つを超える
• 関数が複数のことを行っている

// ❌ 7つのパラメータ
function processOrder(user, items, address, payment, discount, coupon, notes)

// ✅ パラメータオブジェクトを使用
interface OrderParams { user: User; items: Item[]; shipping: Address; payment: Payment }
function processOrder(params: OrderParams)

3. 重複の問題【DRY】

チェックの兆候：

• 類似したif-else構造
• 類似したデータ変換/エラー処理ロジック
• コピー＆ペーストの痕跡

4. 過剰な設計【YAGNI】

チェックの兆候：

• 一度もtrueにならないif (config.legacyMode)分岐
• 実装が1つしかないインターフェース
• 無駄なtry-catchまたはif-else

// ❌ YAGNI違反：一度も使用されない互換性コード
if (config.legacyMode) {
  // 100行の互換性コード
}

5. マジックナンバー【ハードコーディングの回避】

チェックの兆候：

• 説明のないむき出しの数字
• ハードコーディングされた文字列

// ❌ 
if (retryCount > 3) // 3は何ですか？
setTimeout(fn, 86400000) // これはどのくらいの時間ですか？

// ✅ 
const MAX_RETRY_COUNT = 3;
const ONE_DAY_MS = 24 * 60 * 60 * 1000;

6. 構造の明確さ【可読性優先】

チェックの兆候：

• ネストされた三項演算子
• 過度に密な1行コード
• 深すぎる条件ネスト（3層以上）

// ❌ ネストされた三項演算子
const status = a ? (b ? 'x' : 'y') : (c ? 'z' : 'w');

// ✅ switchまたはif/elseを使用
function getStatus(a, b, c) {
  if (a) return b ? 'x' : 'y';
  return c ? 'z' : 'w';
}

7. プロジェクトの規約【一貫性】

チェックの兆候：

• importの順序が混乱している（外部ライブラリ vs 内部モジュール）
• 関数宣言のスタイルが不一致
• 命名規約が不統一（camelCase vs snake_caseの混用）

// ❌ スタイルが不一致
import { api } from './api'
import axios from 'axios'  // 外部ライブラリは前にあるべき
const handle_click = () => { ... }  // 命名スタイルが混用

// ✅ スタイルを統一
import axios from 'axios'
import { api } from './api'
function handleClick(): void { ... }

[!TIP] プロジェクトの規約は、CLAUDE.md、AGENTS.md、またはプロジェクトで合意されたコーディング標準を参照してください。

深刻度レベル

出力形式

### [問題タイプ]: [要約]

- **原則**: [Clean Code 原則]
- **位置**: `ファイル:行番号`
- **レベル**: 高/中/低
- **問題**: [具体的な説明]
- **提案**: [修正の方向性]

参考文献

詳細な例: references/detailed-examples.mdを参照してください。

• 各次元の完全なケース（命名、関数、DRY、YAGNI、マジックナンバー）

言語パターン: references/language-patterns.mdを参照してください。

• TypeScript/JavaScriptの一般的な問題
• Pythonの一般的な問題
• Goの一般的な問題

マルチエージェント並列処理

以下の次元で複数のエージェントに分割して並列処理を行います。

1. チェック次元別 - 7つの次元それぞれに1つのエージェント
2. モジュール/ディレクトリ別 - 異なるモジュールそれぞれに1つのエージェント
3. 言語別 - TypeScript、Python、Goそれぞれに1つのエージェント
4. ファイルタイプ別 - コンポーネント、hooks、ユーティリティ関数、型定義

例：/clean-code-reviewer --scope=components または --dimension=naming

集計時には重複排除と深刻度評価の統一が必要です。