docs(phase21.7): Phase 21.7++ Phase 4完了 - ドキュメント整備
## Phase 4 実装内容 ### Phase 4.1: README 更新 - docs/development/roadmap/phases/phase-21.7-normalization/README.md - "Phase 21.7++ 実装完了" セクション追加(60行) - SSOT ルール・実装箇所・デバッグ環境変数を文書化 - 全4フェーズの完了状況をコミットハッシュ付きで記録 - 技術的成果の総括(Silent Failure根絶、arity バグ根治等) ### Phase 4.2: トラブルシューティングガイド作成 - docs/development/troubleshooting/using-resolution.md(新規作成、200+行) - 4つのエラーパターン別対処法 - デバッグフローチャート - FAQ セクション - Phase 21.7++ 実装前後の比較表 ### チェックリスト更新 - docs/development/current/main/phase-21.7-naming-ssot-checklist.md - Phase 0-4 完了状況サマリー追加 - 累計工数: 10時間(進捗率: 50-67%) ## 技術的成果サマリー ✅ **Phase 0**: Silent Failure 根絶(時間→分) ✅ **Phase 1**: StaticMethodId SSOT 確立 ✅ **Phase 2**: arity バグ根治(Hotfix 卒業) ✅ **Phase 3**: 素手 split 根絶(Builder 統一) ✅ **Phase 4**: ドキュメント整備(再発防止) **Phase 21.7++ 全フェーズ完了!** 🎊 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
@ -40,20 +40,63 @@ Rollout Plan
|
||||
Rollback
|
||||
- Disable HAKO_MIR_BUILDER_METHODIZE. Revert to Global("Box.method") resolution path (current 21.6 behavior).
|
||||
|
||||
Current notes (Phase 25.x bring-up)
|
||||
- NamingBox / static 名 SSOT
|
||||
- `src/mir/naming.rs` に NamingBox を実装済み。`Main.main` / `main._nop` などの揺れを `"Main.main/0"` 形式に正規化する経路は Rust VM/LLVM/JSON bridge から既に利用中。
|
||||
- VM 側は `normalize_static_global_name` を通して static box 名を一元化するよう更新済み。
|
||||
## Phase 21.7++ 実装完了 (2025-11-22)
|
||||
|
||||
- 既知のギャップ
|
||||
- static box 内のローカル呼び出し(例: `Main.main` → `me._nop()`)が Global 呼び出しのまま落ちるケースを確認済み。NamingBox で `main`→`Main` 正規化は済んでいるが、「Method 化+receiver 付与」が未完了。
|
||||
- MeCallPolicy / method_call_handlers 周りで、static box method に対しても一律で receiver(`me`)を先頭に追加してしまうパスがあり、arity 不一致や miss-call の温床になり得る。
|
||||
### Global 名の SSOT ルール
|
||||
|
||||
- このフェーズでやること(具体タスク)
|
||||
1. `src/mir/builder/method_call_handlers.rs` / MeCallPolicy で「static box method かどうか」を判定し、static のときは receiver を追加しない(args はそのまま Global 互換で流す)ガードを入れる。
|
||||
2. methodization ロジック(HAKO_MIR_BUILDER_METHODIZE=1 有効時)で、static box 呼びを `ensure_static_box_instance(Box)` 経由の Method 呼び出しに寄せる:
|
||||
- Global("Main._nop/0") → Method{ receiver = ensure_static_box_instance("Main"), box="Main", method="_nop", arity=0 }。
|
||||
3. 最小テストを追加:
|
||||
- `.hako` 側 minimal: `static box Main { static method _nop() { return 0 } }` を呼ぶケース。
|
||||
- Rust 側: `mir_stage1_static_main_nop_resolves_and_execs`(仮)で、MIR verify + VM 実行が methodization ON/OFF の両方で安定することを確認。
|
||||
4. docs: 本ファイルに「static box 正規化の完了条件」と「NamingBox / ensure_static_box_instance / MeCallPolicy の責務分担」を 1 ページにまとめ、Phase 21.7 の完了ラインを明文化する。
|
||||
#### 原則
|
||||
- Global 関数名は **`Box.method/N`** が SSOT
|
||||
- VM/LLVM で `Box.method` を受け取ったら、arity は `args.len()` から補完
|
||||
- すべての名前解決は `NamingBox::StaticMethodId` 経由
|
||||
|
||||
#### 実装箇所
|
||||
|
||||
**NamingBox**: `src/mir/naming.rs`
|
||||
- `StaticMethodId::parse()`: 名前のパース("Box.method/N" or "Box.method")
|
||||
- `StaticMethodId::format()`: 正規化された名前生成
|
||||
- `StaticMethodId::with_arity()`: arity 補完
|
||||
- 13 テストケースで検証済み(src/tests/namingbox_static_method_id.rs)
|
||||
|
||||
**VM**: `src/backend/mir_interpreter/handlers/calls/global.rs`
|
||||
- `StaticMethodId` で名前解決
|
||||
- arity 無し → `args.len()` で補完
|
||||
- "Did you mean?" エラーメッセージ実装
|
||||
|
||||
**UnifiedCallEmitter**: `src/mir/builder/calls/unified_emitter.rs`
|
||||
- Methodization で `StaticMethodId` 使用
|
||||
- TypeRegistry と連携して static box method 判定
|
||||
- 素手 split 根絶
|
||||
|
||||
**Rewrite Known**: `src/mir/builder/rewrite/known.rs`
|
||||
- split_once → StaticMethodId::parse() に統一
|
||||
|
||||
#### デバッグ環境変数
|
||||
- `NYASH_DEBUG_FUNCTION_LOOKUP=1`: VM 関数ルックアップ詳細(box/method/arity 表示)
|
||||
- `NYASH_DEBUG_USING=1`: using 解決詳細
|
||||
- `NYASH_METHODIZE_TRACE=1`: Global→Method 変換ログ
|
||||
|
||||
#### 実装フェーズ(全完了)
|
||||
- ✅ **Phase 0: 観測ライン** (commit 63012932) - Silent Failure 根絶
|
||||
- ✅ **Phase 1: 基盤整備** (commit 96c1345e) - StaticMethodId SSOT 確立
|
||||
- ✅ **Phase 2: VM 統一** (commit 1b413da5) - arity バグ根治
|
||||
- ✅ **Phase 3: 全体統一** (commit c8ad1dae) - Builder 側統一、素手 split 根絶
|
||||
|
||||
#### 技術的成果
|
||||
- Silent Failure 根絶(デバッグ時間: 時間→分)
|
||||
- arity バグ根治(Hotfix 卒業)
|
||||
- 素手 split 根絶(全箇所を SSOT 経由に統一)
|
||||
- 型安全化(構造化表現で誤用防止)
|
||||
- テスト完全通過(349 passed, 退行なし)
|
||||
|
||||
---
|
||||
|
||||
## 旧ノート(Phase 25.x bring-up 時点)
|
||||
|
||||
### NamingBox / static 名 SSOT
|
||||
- `src/mir/naming.rs` に NamingBox を実装済み。`Main.main` / `main._nop` などの揺れを `"Main.main/0"` 形式に正規化する経路は Rust VM/LLVM/JSON bridge から既に利用中。
|
||||
- VM 側は `normalize_static_global_name` を通して static box 名を一元化するよう更新済み。
|
||||
- **Phase 21.7++ で完全 SSOT 化完了**(2025-11-22)
|
||||
|
||||
### 既知のギャップ(解決済み)
|
||||
- ~~static box 内のローカル呼び出し(例: `Main.main` → `me._nop()`)が Global 呼び出しのまま落ちるケースを確認済み。~~ → Phase 2 で解決
|
||||
- ~~MeCallPolicy / method_call_handlers 周りで、static box method に対しても一律で receiver(`me`)を先頭に追加してしまうパスがあり、arity 不一致や miss-call の温床になり得る。~~ → Phase 3 で解決
|
||||
|
||||
Reference in New Issue
Block a user