0455307418
Phase A cleanup - Safe deletions with zero risk: ## Deleted Files (6 files, 373 lines total) 1. Cranelift/JIT Backend (321 lines): - src/runner/modes/cranelift.rs (45 lines) - src/runner/modes/aot.rs (55 lines) - src/runner/jit_direct.rs (152 lines) - src/tests/core13_smoke_jit.rs (42 lines) - src/tests/core13_smoke_jit_map.rs (27 lines) 2. Legacy MIR Builder (52 lines): - src/mir/builder/exprs_legacy.rs - Functionality inlined into exprs.rs (control flow constructs) ## Module Reference Cleanup - src/backend/mod.rs: Removed cranelift feature gate exports - src/runner/mod.rs: Removed jit_direct module reference - src/runner/modes/mod.rs: Removed aot module reference - src/mir/builder.rs: Removed exprs_legacy module ## Impact Analysis - Build: Success (cargo build --release) - Tests: All passing - Risk Level: None (feature already archived, code unused) - Related: Phase 15 JIT archival (archive/jit-cranelift/) ## BID Copilot Status - Already removed in previous cleanup - Not part of this commit Total Reduction: 373 lines (~0.4% of codebase) Next: Phase B - Dead code investigation Related: #phase-21.0-cleanup Part of: Legacy Code Cleanup Initiative
4.8 KiB
4.8 KiB
モジュール構造改善 - クイックリファレンス
最終更新: 2025-11-06
📋 3つのドキュメント
| ドキュメント | 用途 | 対象読者 |
|---|---|---|
| MODULE_STRUCTURE_ANALYSIS.md | 全体分析・戦略 | チームリーダー・アーキテクト |
| PHASE1_IMPLEMENTATION_GUIDE.md | Phase 1実装手順 | 開発者 |
| このファイル | クイックリファレンス | 全員 |
🎯 緊急対応（Phase 1）- 1-2週間
対象ファイル
| ファイル | 現在行数 | 目標 | 優先度 |
|---|---|---|---|
handlers/calls.rs |
49,750 | 6ファイルに分割 | 🔥最高 |
runner/modes/common.rs |
14,000 | 1,000行以下 | 🔥最高 |
runtime/plugin_loader_v2/ |
5階層 | 4階層に削減 | 🔥最高 |
クイック実装手順
1. calls.rs分割
# 1. ディレクトリ作成
mkdir -p src/backend/mir_interpreter/handlers/calls
# 2. 責務別に分割
# - resolution.rs (呼び出し解決)
# - dispatch.rs (ディスパッチ)
# - arguments.rs (引数処理)
# - returns.rs (戻り値処理)
# - errors.rs (エラー)
# - shared.rs (共有型)
# 3. テスト
cargo build --release && cargo test
2. common.rs分割
# 1. 新構造作成
mkdir -p src/runner/{execution,pipeline,resolution}
# 2. 責務別に移動
# execution/ - VM/LLVM/PyVM実行
# pipeline/ - MIRコンパイル・前処理
# resolution/ - using/namespace解決
# 3. modesを薄いラッパーに
# modes/*.rs → execution/*.rs を呼ぶだけ
# 4. テスト
./tools/smokes/v2/run.sh --profile quick
3. plugin_loader_v2階層整理
# 1. 新構造作成
mkdir -p src/runtime/plugins/{core,loader,bridge}
# 2. ファイル移動
# enabled/ → core/
# enabled/loader/ → loader/
# enabled/*_bridge.rs → bridge/
# 3. runtime/mod.rs更新
# pub mod plugins;
# 4. テスト
NYASH_SKIP_TOML_ENV=1 ./tools/smoke_plugins.sh
📊 統計サマリー
現状
- 総ファイル数: 500+
- 最大ファイル: calls.rs (49,750行)
- 平均ファイル: 500-1000行
- 最深階層: 5階層
目標（Phase 1完了後）
- 1,000行超ファイル: 20+ → 10以下
- 最大ファイル: 10,000行以下
- 平均ファイル: 300-500行
- 最深階層: 4階層以下
🚨 重要な注意点
やってはいけないこと
- ❌ 一度に全ての変更を行う（段階的に！）
- ❌ テストをスキップする（毎回確認！）
- ❌ 後方互換性を無視する（deprecation使用）
- ❌ ドキュメント更新を忘れる
必ずやること
- ✅ Gitブランチを作成してから作業
- ✅ 各ステップでコンパイル確認
- ✅ 全テストが通ることを確認
- ✅ 進捗をチームに共有
🔧 便利なコマンド
ファイル分析
# ファイル行数ランキング
find src -name "*.rs" -exec wc -l {} \; | sort -rn | head -20
# 特定モジュールの行数
wc -l src/backend/mir_interpreter/handlers/*.rs | sort -rn
# 関数一覧抽出
rg "^pub fn|^fn " src/path/to/file.rs
依存関係分析
# インポート分析
rg "^use crate::" src/module/ | sort | uniq -c | sort -rn
# 特定モジュールへの依存を検索
rg "use crate::runtime::plugin_loader_v2" src/
テスト実行
# フルビルド
cargo build --release
# 特定モジュールのテスト
cargo test --package nyash_rust --lib backend::mir_interpreter
# スモークテスト（VM）
./tools/smokes/v2/run.sh --profile quick
# スモークテスト（LLVM統合）
./tools/smokes/v2/run.sh --profile integration
# プラグインテスト
NYASH_SKIP_TOML_ENV=1 ./tools/smoke_plugins.sh
📚 追加リソース
Rustのベストプラクティス
プロジェクト内ドキュメント
/docs/development/architecture/- アーキテクチャ設計/docs/development/roadmap/- ロードマップ/CURRENT_TASK.md- 現在のタスク
🎯 次のアクション
今すぐやること
- MODULE_STRUCTURE_ANALYSIS.mdを読む
- PHASE1_IMPLEMENTATION_GUIDE.mdを読む
- Gitブランチを作成
- calls.rs分析を開始
1週間後
- calls.rs分割完了
- common.rs分析・分割開始
2週間後
- Phase 1全項目完了
- PR作成・レビュー
- Phase 2計画開始
📞 サポート
質問・相談は以下へ：
- Issue: GitHub Issueで質問
- PR: レビューリクエスト
- CURRENT_TASK.md: 進捗報告
頑張って！段階的に、確実に進めよう！ 🚀