jim800121chen c54f16fca0 Initial commit: visionA monorepo with local-tool subproject

local-tool/: visionA-local desktop app
- M1: Wails shell + Go server + Next.js UI + Mock mode (macOS dmg ready)
- M2: i18n (zh-TW/en) + Settings 4-tab refactor
- M3: Embedded Python 3.12 runtime (python-build-standalone) + KneronPLUS wheels
- M4: Windows Inno Setup script (build on Windows runner)
- M5: Linux AppImage script + udev rule (build on Linux runner)
- M6: ffmpeg (GPL, pending legal review) + yt-dlp bundled
- Lifecycle: watchServer health check, fatal native dialog,
            Wails IPC raise endpoint, stale process cleanup

.autoflow/: full PRD / Design Spec / Architecture / Testing docs
            (4 rounds tri-party discussion + cross review)
.github/workflows/: macOS / Windows / Linux build CI

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

2026-04-11 22:10:38 +08:00

3.9 KiB

Raw Blame History

Code Review 報告 — M1-5（Go binary + smoke test）

審查摘要

審查對象：dist/visiona-local-server 及 Makefile 的 server / build-server target
產出 Agent：Backend Agent
審查結果：✅ 通過
問題統計：Critical: 0 / Major: 0 / Minor: 1 / Suggestion: 1

驗收項目清單

#	驗收項目	結果	證據
1	Binary 存在且可執行	✅	`file dist/visiona-local-server` → `Mach-O 64-bit executable x86_64`，31 MB
2	`--help` 輸出包含 `--mock --dev --port --data-dir --python-mode`	✅	`-mock`, `-dev`, `-port`, `-data-dir`, `-python-mode` 皆存在
3	`--help` 無 `--relay-url --tray --gui`	✅	已完全移除，flag 清單乾淨
4	`--host` 強制 127.0.0.1	✅	flag 預設值為 `127.0.0.1`，help 字串 `(forced to 127.0.0.1 for local-only use)`
5	Smoke：`./visiona-local-server --mock --port 13721` 可啟動	✅	log 顯示 `Server listening on 127.0.0.1:13721`
6	`GET /api/system/health` 回 200	✅	`{"status":"ok"}`
7	`GET /api/system/info` 回 200	✅	`{"data":{"goVersion":"go1.26.0","platform":"darwin/amd64","uptime":1.51,"version":"dev"},"success":true}`
8	Bind 在 127.0.0.1 而非 0.0.0.0	✅	`lsof -nP -iTCP:13721` → `TCP 127.0.0.1:13721 (LISTEN)`
9	SIGTERM 可優雅關閉	✅	`Received signal terminated, shutting down gracefully...`
10	Makefile `server` / `build-server` target 可執行	✅	兩個 target 已定義，`server` 為 `build-server` 的 alias，內容為 `cd server && go build -o ../dist/visiona-local-server .`
11	Embedded frontend 已掛載	✅	log：`Serving embedded frontend static files`，註冊 `/_next/*filepath`、`/favicon.ico`
12	外部依賴自檢	✅	ffmpeg 8.0.1、yt-dlp 2026.02.04、python3 3.14.3 全部 OK

問題清單

Minor（建議修復）

#	檔案	問題描述	建議
1	`dist/data/models.json`	啟動時 warning：`could not load models from .../dist/data/models.json: no such file or directory`	M1-8 打包時應把 seed 的 `models.json` 放進 `dist/data/`，或讓 backend 在檔案不存在時靜默跳過（目前就是走這條，只是 log 成 warning 略吵）

Suggestion（非必要）

#	建議
1	預設 `GIN_MODE=debug` 在正式 binary 跑會印很多 route 註冊 log；M1-9 installer build 時透過 ldflags 或環境變數設為 `release` 更乾淨

備註：Port 衝突與歷史 daemon

執行 smoke test 時發現系統上有一隻 legacy edge-ai-server (PID 44243) 已經佔用 127.0.0.1:3721（帶 --relay-url / --relay-token / --tray 參數，顯然是 M0 時代的背景 daemon）。這不影響 M1-5 交付，但：

第一次 smoke test 若跑在 3721，curl 回應其實來自 legacy daemon，不是新 binary。我改用 port 13721 重測，確認新 binary 行為正確。
建議 M1-9（Wails installer）或交付文件裡加一段「升級注意事項」：提醒使用者先 pkill edge-ai-server 或手動移除舊 launchd plist，否則新舊 binary 搶 port。

優點

--host flag 明確標示 forced to 127.0.0.1 for local-only use，意圖清楚，local-only 原則落實到 CLI 層
啟動時主動做外部依賴自檢（ffmpeg / yt-dlp / python3），對使用者除錯很友善
Graceful shutdown 正確實作（SIGTERM → 關閉 → 進程退出，無殘留）
路由註冊完整：system、models、devices、camera、media、ws 全部對齊 TDD 規格
Makefile target 命名直覺，server 作為 build-server alias 符合慣用做法

總結意見

M1-5 完整通過。 Binary 可執行、CLI flags 符合規格、強制 bind 127.0.0.1、health/info endpoint 正常、graceful shutdown 正確、Makefile target 可用。僅一個 Minor（models.json warning）與一個 Suggestion（GIN debug mode），皆不阻斷 M1 驗收。

可進入 M1-9（Wails installer shell）。

3.9 KiB Raw Blame History Unescape Escape