[process] impl 開發流程加入 build .app + 檢查 log 步驟

[cheerc]

WHY / 症狀

目前的 impl workflow 是：

code → test (pytest) → lint (ruff) → typecheck (mypy) → build (npm) → PR

缺少 build .app → 實際執行 → 檢查 runtime log 有無異常 的步驟。

這導致 frozen-build-only 的 bug 在 PR 階段無法被發現——例如 #174（check_update() 在 .app 中因 importlib.metadata.version() 找不到 package metadata 而失效），在 uv run 開發模式下完全正常，只在 PyInstaller 打包的 .app 中浮現。

需求

討論並決定：impl 在送 PR 前是否應該：

1. Build .app：./deploy.sh build
2. 啟動 .app：open dist/simple-edge-tts.app
3. 檢查 log：讀取 ~/Library/Logs/simple-edge-tts/ 中最新的 log 檔，確認無 ERROR / exception / failed 等異常
4. 關閉 app 後才送 PR

待討論

• 誰該做？Impl 送 PR 前？Reviewer 審查時？Lead post-merge？
  • 建議：Impl 送 PR 前必做（跟跑 test/lint 一樣，是 pre-PR gate）
  • Reviewer：若 diff 觸及 src/**（Python backend / frozen 路徑相關），也該 build 驗證
  • Lead：post-merge 已跑 deploy.sh build（LEAD.md §11.1），但那是最後一道防線，不該是第一道
• 頻率：每次 PR 都做？還是只有涉及 src/** / frozen 路徑的 PR？
  • 建議：涉及 src/** 的 PR 必做；純 frontend / docs / config 可跳過
• 自動化：能否在 CI 中跑？還是只能手動？
  • macOS .app 在 CI 可跑（open + sleep + log check），但需要 macOS runner
  • 短期手動、長期 CI 化

Related

• refs [bug] check_update() broken in frozen (PyInstaller) build — importlib.metadata.version fails #174 — frozen build bug（本流程缺口的第一個 victim）
• refs feat: auto-update UI — Settings + Toast actions + design doc (#170, #171, #172) #173 — auto-update UI PR（合併後才發現此 bug）
• LEAD.md §11.1 — Post-Merge App Build（lead 已在做，但太晚）

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

[cheerc]

✅ Fixed in #186 (379c846)

根因：frozen-build-only bug 在 PR 階段無法被現有 pipeline 攔截——uv run 正常但 .app 崩潰。IMPL.md §6 的 Risk-Flags 觸發條件太窄（#174 的 importlib.metadata 不在任何 Risk-Flag 範圍內卻有 frozen 行為分歧）。

修法：三管齊下——

1. deploy.sh verify — build .app + launch + grep log (pre-PR script gate)
2. tests/test_frozen_paths.py — frozen-path mock tests (17 tests, 含 importlib 3-tier fallback + 7-module import smoke)
3. SOP 更新 — IMPL.md + REVIEWER.md (agend-customization#216)

Test evidence：gh pr checks 186 → Lint & Test pass; ./deploy.sh verify → log clean; uv run pytest tests/test_frozen_paths.py -v → 17 passed

Lessons：四份獨立研究報告一致認定「impl pre-PR 為正確防禦層級」。reviewer2 的 frozen-path mock tests 建議將本 PR 的 C/P 從 "3 min build per PR" 提升到 "零成本 pytest 覆蓋 frozen path"——這是本次最有價值的 insight。