[Analysis] Phase 0 完成後重構準備度評估 — 路線 A IIFE coverage 阻斷分析

[cheerc]

背景

Phase 0（#129 Phase 0）全部完成：10 issues closed, 5 PRs merged (#139-#143), 1147 tests (+206), 16 DI modules (99.45% coverage)。

本 issue 評估：Phase 0 完成後，啟動 #129 Phase 1（路線 A IIFE 重構）是否仍有 test coverage 阻斷。

Lead 初步分析結論

零 coverage 阻斷項 — #129 Phase 0 checklist 8+1 項全部完成。#107 殘餘項目都是 P1-P3（非 P0）。

安全網現況

指標	值
Tests	1087 Vitest + 60 PHPUnit = 1147
Statements	70.5%
Branches	76.7%
Functions	84.9%
Lines	69.8%
DI modules	16 (tests/lib/, 99.45% coverage)
Wiring contracts	17 backend funcs + 16 lib modules

關注點（不阻斷但需討論）

1. 程式碼.js 仍 0% v8 coverage（instrument 限制，DI extraction 已覆蓋 logic）
2. DOM wiring 部分覆蓋（domWiringContracts + domWiringIntegration 已覆蓋 binding contracts）
3. E2E/deploy smoke test 無自動化（[Test-Coverage] Phase 2 — 覆蓋缺口補齊 + 重構前置強化 #107 標 P2）
4. Coverage threshold 可上調至 65/72/80/65（防重構退步）

待辯論

全 team 深度 review + 正反兩輪辯論：

• 正方（可以啟動重構）：Phase 0 安全網是否足夠？
• 反方（不宜啟動）：有哪些隱藏風險？DI extraction ≠ production 行為的 gap 有多大？IIFE 封裝會破壞哪些隱式契約？
• 最終結論由 Lead 收報告彙整。

Related

• refs [Test-Coverage] Phase 2 — 覆蓋缺口補齊 + 重構前置強化 #107 — Phase 2 覆蓋缺口（Phase 0 項全完成）
• refs [Refactor] GAS 重構準備度決策 — 路線 A + Phase 0 行為測試前置 #129 — GAS 重構決策（Phase 0 checklist source）
• refs [Test-Coverage] 達成 80%+ critical path coverage — 重構前置條件 #61 — Phase 1 覆蓋率（已 closed，502 tests 基線）

🤖 opened by: lead (cb-team-lead)

[cheerc]

最終結論（兩輪辯論 + reviewer 審定後）

Verdict: ✅ Phase 1 可啟動，零 coverage 阻斷

辯論過程：正方（impl）vs 反方（impl2）兩輪辯論 + reviewer 獨立審計 + reviewer 最終審定。

核心爭點：反方提出 5 個風險（DI extraction 先天限制、隱式契約破壞、wiring noise、3 具體風險情境、需補 5-8 天）。

解決：正方第二輪以源碼舉證證明反方的全部論點建立在錯誤前提——Phase 1 只拆分 JavaScript.html 的內部方法，不碰其他 7 個 bare-script 檔案。因此：

• escapeHtml/AppConfig/AppElements/ServerApi 的全域可見性不受影響
• this 綁定不受影響（App 是物件字面量 + [Refactor] GAS 重構準備度決策 — 路線 A + Phase 0 行為測試前置 #129 目標模式用 App.xxx 取代 this.xxx）
• 載入順序不受影響（Index.html include 順序不變）
• 5-8 天前置工作基於錯誤 scope 假設，不需要

Reviewer 獨立驗證：逐一檢查 8 個 .html 檔案的 head，確認只有 JavaScript.html 是 IIFE，其他 7 個都是 bare script。VERIFIED Lead 結論。

附帶收穫：反方的 scope/this/load-order 風險在 Phase 2（路線 B bundler）時會全面爆發 → 作為 Phase 2 前置 flag 記錄。

建議

1. 啟動 [Refactor] GAS 重構準備度決策 — 路線 A + Phase 0 行為測試前置 #129 Phase 1（JavaScript.html 7 職責域分離）
2. Coverage threshold 上調至 65/72/80/65
3. 第一個 IIFE PR 選最小職責域作 canary
4. 每個 PR 附 wiring test 更新對照

🤖 concluded by: lead (cb-team-lead)

[cheerc]

Phase 0 完成後分析已完成，Phase 1 已全部 merge + 驗證通過。