docs: 全庫程式碼審查 + 分 Tier 改善規劃 (2026-07)

[dofliu]

做什麼 / What

新增 docs/CODE_REVIEW_2026-07.md — 一份全庫(~65k 行 Python + React 前端)程式碼審查
與分 Tier 的後續改善規劃。純文件,無程式碼改動。接續 docs/CODE_REVIEW.md(v3.1+v3.2)。

方法:5 條並行子系統審查(server/API · pipeline/render · AI/內容生成 · 安全/設定 ·
品質/架構/測試)+ 主審對每條最嚴重發現逐一 Read 原始碼驗證。

先報喜:上一份 CODE_REVIEW.md 列的 4 個 P0 已全部修掉(logging lock、SystemExit
catch、上傳 200MB 上限、utc_now);安全四大常見致命傷(命令注入 / path traversal /
secrets / CORS)都有系統性防護且落地,無遠端未授權嚴重漏洞。

發現依「修了對產品最有感」分 4 個 Tier:

Tier	主題	代表項
T0 產品核心正確性	直擊「絕不發布錯誤數字」	clean_json_escapes 靜默破壞 \theta/\times/\frac;exam review gate 可被 require_review=false 繞過;解題寫死舊模型
T1 穩定性	會壞掉 / 卡死	dubber filtergraph 索引錯位(缺音檔即崩);外部呼叫全線無 timeout(23 subprocess 僅 2 個);無並行上限;state.json 非原子寫
T2 安全縱深	受自架定位緩解	url 來源 SSRF;base compose 綁 0.0.0.0+預設無 token;editor.py j.error 未跳脫
T3 架構	償技術債	core/ 是頂層腳本的空殼再匯出層(依賴方向反);Gemini client 13+ 檔重複;ffmpeg 14+ 檔手刻;app.jsx 3508 行單檔

文末附分 4 個 Sprint 的執行建議(先產品止血 → 統一抽象順帶解 timeout/model → 安全縱深 →
架構償債)。

為什麼 / Why

使用者要求對現有專案做程式碼檢查並提出後續改善規劃。產出一份可長期參照、可分批認領的
稽核文件,沿用 repo 既有 docs/ 慣例。本 PR 只提出規劃,不含修復實作——待逐項認領。

怎麼測 / How tested

• 純文件 PR,未動 server/runner/schemas → pytest 不適用(無程式碼改動)
• 未動前端 → npm run build 不適用
• 手動驗證:標 ✓ 的 10 項發現皆由主審親自 Read 原始碼複核屬實(file:line 對應本 commit)

自我檢查 / Checklist

• 沒有繞過 AI 產出的人工 review 關卡(純文件;反而指出 review gate 的 T0-2 缺口待修)
• offline-first 改動(純文件,不花 API 額度 / 不動架構)
• 沒有寫死字型路徑 / 另立路徑常數
• 沒有 commit 機密 / 本機檔
• 無 schema 型別改動

🤖 Generated with Claude Code

https://claude.ai/code/session_017YfSHitPvSuuzVdVkhspWR

---

Generated by Claude Code