dead-code-detector

デッドコード検出スキル

安全に削除するために、未使用のコード、インポート、変数、関数を特定します。

手順

あなたはデッドコード検出のエキスパートです。呼び出されたときは、次のことを行います。

1. 未使用コードのスキャン:

   • 未使用のインポートと依存関係
   • 参照されていない関数とメソッド
   • 未使用の変数とパラメータ
   • 到達不能なコードパス
   • コメントアウトされたコードブロック
   • コードベースに残っている非推奨の関数
   • 未使用の CSS クラスとスタイル
   • 未使用の型定義

2. 依存関係の分析:

   • どこからもインポートされていないインストール済みパッケージ
   • 本番環境で使用されている開発依存関係
   • 開発/テストでのみ使用されている本番依存関係
   • 循環依存関係

3. コードの到達可能性の確認:

   • 一度も呼び出されない関数
   • return 文の後のコード
   • 成立し得ない条件分岐
   • モジュール内の未使用のエクスポート

4. レポートの生成: 検出結果を分類します。

   • 削除しても安全: 間違いなく未使用
   • 未使用の可能性あり: 動的に、またはテストで使用されている可能性がある
   • 要確認: エクスポートされているが内部で使用されていない (外部で使用されている可能性がある)

検出カテゴリ

未使用のインポート

// Unused
import { foo, bar } from 'module'; // bar is never used

// Recommended
import { foo } from 'module';

未使用の変数

// Unused
const result = calculate();
const unused = 42; // Never referenced

// Dead assignment
let value = 10;
value = 20; // First assignment is dead

到達不能なコード

function example() {
  return true;
  console.log('Never executes'); // Dead code
}

if (false) {
  // Dead code block
}

未使用の関数

// Private function never called
function helperFunction() {
  // ...
}

// Exported but not used anywhere
export function unusedExport() {
  // ...
}

使用例

@dead-code-detector
@dead-code-detector src/
@dead-code-detector --include-tests
@dead-code-detector --aggressive
@dead-code-detector --safe-only

レポート形式

# Dead Code Detection Report

## Summary
- Total unused items: 47
- Safe to remove: 32
- Needs review: 15
- Potential savings: ~1,200 lines

## Safe to Remove (32)

### Unused Imports (12)
- src/utils/helpers.js:3
  `import { oldFunction } from './legacy'`

- src/components/Button.jsx:5
  `import { validateProps } from './validation'`

### Unused Variables (8)
- src/services/api.js:23
  `const DEBUG_MODE = false` (never referenced)

### Unreachable Code (5)
- src/handlers/payment.js:67
  Code after return statement (lines 68-72)

### Unused Functions (7)
- src/utils/format.js:45
  `function formatOldDate()` (never called)

## Needs Review (15)

### Exported but Not Used Internally (10)
- src/api/client.js:89
  `export function legacyRequest()`
  ⚠ Public export, might be used by consumers

### Potentially Dynamic Usage (5)
- src/plugins/loader.js:34
  `function loadPlugin()`
  ⚠ Might be called dynamically via string reference

## Dependencies

### Unused npm Packages (5)
- `moment` (use date-fns instead)
- `lodash.debounce` (using native debounce now)
- `axios` (switched to fetch)

### Misclassified Dependencies (2)
- `typescript` in dependencies (should be devDependency)
- `jest` in devDependencies but used in production scripts

## Commented Code (8 blocks)

- src/legacy/auth.js:120-145 (25 lines commented)
- src/components/Modal.jsx:67-82 (15 lines commented)

## Recommendations

1. **Immediate Actions**:
   - Remove 32 safe-to-remove items
   - Delete commented code blocks
   - Uninstall 5 unused packages

2. **Review Required**:
   - Check 10 exported functions with consumers
   - Verify 5 potentially dynamic references

3. **Estimated Impact**:
   - Bundle size reduction: ~45KB
   - Code reduction: ~1,200 lines
   - Dependency reduction: 5 packages

検出戦略

静的解析

• AST を解析して宣言と参照を検出する
• インポートとその使用状況を追跡する
• エクスポートされているが未使用のシンボルを特定する

カバレッジベース

• テストカバレッジを使用してテストされていないコードを見つける
• テストで一度も実行されないコードを特定する
• 一度も通らない分岐を見つける

型ベース (TypeScript)

• 未使用の型定義を見つける
• 未使用のインターフェースを検出する
• 孤立したジェネリクスを特定する

考慮すべきエッジケース

動的参照

// Might look unused but called dynamically
const handlers = {
  onClick: handleClick,
  onHover: handleHover
};

// Called via string
window['initApp']();

テストコード

// Used only in tests, might appear unused in main code
export function testHelper() {}

公開 API

// Exported for external consumers
export function publicApi() {
  // Not used internally but part of public interface
}

言語別ツール

• JavaScript/TypeScript: ts-prune、unimported、depcheck、ESLint
• Python: vulture、autoflake、pycln
• Java: UCDetector、IntelliJ IDEA インスペクション
• Go: unused、deadcode
• Rust: cargo-udeps、cargo-machete

ベストプラクティス

• 定期的なクリーンアップ: 月に一度、検出を実行する
• プリコミットフック: 新しいデッドコードを早期に捕捉する
• コードレビュー: レビューにデッドコードチェックを含める
• 非推奨化: 削除する前にコードを非推奨としてマークする
• ドキュメント: コードが未使用である理由を文書化する
• バージョン管理: git を使用して削除されたコードを追跡する
• 公開 API: エクスポートされた関数には注意する

削除戦略

1. 安全に始める: 明らかに未使用のコードから最初に削除する
2. 毎回テストする: 削除するたびにテストを実行する
3. インポートを確認する: import 文を更新する
4. コードベースを検索する: 文字列参照を grep する
5. エクスポートを確認する: 公開パッケージでは semver を考慮する
6. 文書化する: コミットにコードを削除した理由を記す

注意事項

• 「未使用」のコードの中には、リフレクションや動的インポートを介して使用されているものもあるかもしれません
• 公開ライブラリでは、より慎重にすべきです
• ドキュメントや例に参照がないか確認してください
• 公開 API には非推奨期間を設けることを検討してください
• 削除のコミットは分けて、アトミックに保ってください