メニュー
fraktor-rsのモジュール・型設計を「Less is more」「YAGNI」の観点でレビューし、過剰設計を検出する。参照実装（pekko/protoactor-go）との比較、定量分析、公開範囲の最適化を行う。トリガー：「型の設計をレビューして」「過剰設計チェック」「YAGNIチェック」「ファイル統合」「公開範囲を見直して」「type review」「overengineering check」等の型設計レビューリクエスト時に使用。
Codex または Claude でインストール この Prompt をコピーして Codex、Claude、または他のアシスタントに貼り付けると、Skill ページを確認してインストールできます。
SOC 職業分類に基づく
| name | reviewing-fraktor-types |
| description | fraktor-rsのモジュール・型設計を「Less is more」「YAGNI」の観点でレビューし、過剰設計を検出する。参照実装（pekko/protoactor-go）との比較、定量分析、公開範囲の最適化を行う。トリガー：「型の設計をレビューして」「過剰設計チェック」「YAGNIチェック」「ファイル統合」「公開範囲を見直して」「type review」「overengineering check」等の型設計レビューリクエスト時に使用。 |
モジュール・型の設計を「Less is more」「YAGNI」の観点でレビューし、過剰設計を検出・改善提案する。
レビュー対象を確認する：
対象ディレクトリに対して以下を計測する：
# ファイル数・行数（テストファイル除外）
find <target> -name "*.rs" ! -name "tests.rs" ! -name "*_test.rs" ! -name "*_tests.rs" | wc -l
find <target> -name "*.rs" ! -name "tests.rs" ! -name "*_test.rs" ! -name "*_tests.rs" -exec wc -l {} + | sort -n
# 30行以下のファイル数（基準値テーブルの「30行以下ファイル率」に対応）
find <target> -name "*.rs" ! -name "tests.rs" ! -name "*_test.rs" ! -name "*_tests.rs" -exec wc -l {} + | awk '$1 <= 30 {count++} END {print count}'
# 公開型の数
grep -r "^pub struct\|^pub trait\|^pub enum" <target> --include="*.rs" | wc -l
基準値（Phase 1-4 の実績から導出）:
| 指標 | 正常 | 要注意 | 過剰 |
|---|---|---|---|
| 30行以下ファイル率 | < 20% | 20-35% | > 35% |
| 平均行数/ファイル | > 80行 | 50-80行 | < 50行 |
| pub 型 vs pub(crate) 型比率 | 外部利用に応じた比率 | - | pub が大半 |
対象と同等の機能を持つ参照実装の型数を確認する：
references/pekko/): Scala の対応クラス数を調査references/protoactor-go/): Go の対応型数を調査fraktor-rs の型数 / 参照実装の型数 = 倍率
倍率 1-2x: 正常（Rust の所有権・ライフタイム要件で型が増えるのは自然）
倍率 2-3x: 要注意（no_std/std 分離や Shared 型を差し引いても多い可能性）
倍率 3x+: 過剰（設計見直し推奨。Phase 1-4 で message_adapter が 6.5x → 統合で改善した実績あり）
以下のパターンを探す：
pub だが crate 外から参照されていない型検出した問題に対して具体的な改善案を提示する：
| 問題 | 改善案 |
|---|---|
| 薄い型の過剰分離 | 親型と同居（ただし同居先200行以下） |
| 型の重複 | 統合して単一型に |
| 不要な公開 | pub(crate) に変更 |
| 過剰な抽象化 | 具体型に簡素化 |
mcp__serena__find_referencing_symbols で各公開型の参照元を調査し、以下に分類：
リクエスト: 「tick_driver の型を見直したい」
定量分析結果:
参照実装比較:
検出パターン:
TickDriverId(u64): 20行、独立ファイル → newtype は維持（ドメインプリミティブ）HardwareKind: 再エクスポートのみ、直接参照なし → 削除候補TickMetricsMode: 22行、TickDriverConfig でのみ使用 → pub(crate) 化候補改善提案:
リクエスト: 「ClusterMembership の設計をレビューして」
参照実装比較:
判定: 3倍超 → 設計の簡素化を推奨
claudedocs/actor-module-overengineering-analysis.md: Phase 1-4 の分析基準と実績。基準値テーブルの根拠や過去の改善事例を確認する際に参照references/pekko/: Scala 参照実装。ステップ3 で対応クラス数を調査する際に参照references/protoactor-go/: Go 参照実装。ステップ3 で対応型数を調査する際に参照