Leo
4d3a6a09a6
leo 08-06 裁決：「不要兩支，寫成一支檔案」。
## 為什麼
v0.18.7-8 的「單一 exe」其實是**一支包著另一支**：collector.exe 被 go:embed
進 Arcrun.exe，執行時攤到 ~/.arcrun-rag/bin/ 再跑。
那正是防毒軟體眼中的 dropper 特徵 —— 封測者實撞
`Trojan:Win32/Sabsik.FL.A!ml`，檔案當場被隔離、自動刪除。
⚠️ 誠實界定：`!ml` 結尾＝**機器學習判定**，Sabsik 是最常見的通用誤判家族，
主因是「未簽章＋下載次數少」，**不是**特別指向 dropper 行為。
所以本次改動**不保證**解除誤判——真正的解是上架 MS Store（微軟簽章）。
但「執行時把第二支 PE 寫到磁碟再執行」本來就該拿掉，這是對的方向且順手變小。
## 怎麼做
- `collector/` 39 個檔 `package main` → `package collector`，`main()` → 匯出的 `Run(args) int`
- 新增 `collector/cmd/collector/`（薄殼 CLI，讓單獨跑 collector 這條路仍可用）
- App 直接 import 該套件；`main()` 第一件事就判 `--collector`，是的話走 `collector.Run` 不碰 GUI
- `supervisor` 加 `ArgPrefix`，App 把 `BinPath` 指向 `os.Executable()` 自己
- 刪掉 `bundled_collector_{windows,other}.go`（embed + 攤檔那套）
- 三支打包腳本不再編/複製第二支；版本注入同時打到兩個 package
- build-win.sh 的機械閘改成**直接問它**：`--collector --version` 回得出版本才放行
（舊閘是比大小，只能證明「有 embed」，證明不了「分派是對的」）
## 驗（真機實跑）
· `.app/Contents/MacOS/` 只有 **一個** 執行檔（原本兩個）
· 跑起來兩個行程是**同一個 exe**：
…/MacOS/arcrun-app
…/MacOS/arcrun-app --collector direct --config …
· `~/.arcrun-rag/bin` **不存在**（沒有任何東西被攤出來）
· 端到端：丟檔進看守資料夾 → collector.log `"status":"ingested","http_status":200`
· `lsappinfo` 仍是 `type="UIElement"`、`Version="0.18.9"`
· collector 39 檔測試全過；app 測試過；go vet 全綠；mac + windows 交叉編譯皆過
· 單檔 26MB → 22MB（不再夾帶第二份完整程式）
67 lines
2.3 KiB
Go
67 lines
2.3 KiB
Go
// extract_test.go — task 3 claude 萃取路（用 stub 執行檔驗呼叫契約，不需真 claude）。
|
||
package collector
|
||
|
||
import (
|
||
"fmt"
|
||
"os"
|
||
"path/filepath"
|
||
"runtime"
|
||
"testing"
|
||
)
|
||
|
||
// writeStubClaude 產生一支假 claude：驗證收到的參數與 cwd，並寫一張卡進 cards/。
|
||
func writeStubClaude(t *testing.T, dir string, script string) string {
|
||
t.Helper()
|
||
if runtime.GOOS == "windows" {
|
||
t.Skip("stub 腳本測試僅跑 unix")
|
||
}
|
||
p := filepath.Join(dir, "claude")
|
||
if err := os.WriteFile(p, []byte("#!/bin/sh\n"+script), 0o755); err != nil {
|
||
t.Fatal(err)
|
||
}
|
||
return p
|
||
}
|
||
|
||
// 正常路：stub 收到 -p "/rag-extract-file <檔>"、cwd＝監看根，寫卡後被 diff 偵測到。
|
||
func TestExtractWithClaudeHappyPath(t *testing.T) {
|
||
root := t.TempDir()
|
||
if err := os.MkdirAll(filepath.Join(root, "system-dev", "wiki", "cards"), 0o755); err != nil {
|
||
t.Fatal(err)
|
||
}
|
||
stub := writeStubClaude(t, t.TempDir(), fmt.Sprintf(`
|
||
[ "$1" = "-p" ] || { echo "第一參數應為 -p，got $1" >&2; exit 9; }
|
||
case "$2" in "/rag-extract-file 我的筆記.md") ;; *) echo "prompt 錯：$2" >&2; exit 9;; esac
|
||
[ "$(pwd)" = "%s" ] || { echo "cwd 錯：$(pwd)" >&2; exit 9; }
|
||
mkdir -p system-dev/wiki/cards
|
||
printf '# 我的筆記\n\n## 摘要\nstub 卡\n' > "system-dev/wiki/cards/我的筆記.md"
|
||
`, root))
|
||
cards, err := ExtractWithClaude(stub, root, "我的筆記.md")
|
||
if err != nil {
|
||
t.Fatal(err)
|
||
}
|
||
if len(cards) != 1 || cards[0] != "system-dev/wiki/cards/我的筆記.md" {
|
||
t.Fatalf("cards=%v", cards)
|
||
}
|
||
}
|
||
|
||
// claude 跑完但沒寫卡＝誠實報錯（不假綠）。
|
||
func TestExtractWithClaudeNoCard(t *testing.T) {
|
||
root := t.TempDir()
|
||
stub := writeStubClaude(t, t.TempDir(), "exit 0\n")
|
||
if _, err := ExtractWithClaude(stub, root, "x.md"); err == nil {
|
||
t.Fatal("沒寫卡應報錯")
|
||
}
|
||
}
|
||
|
||
// 找不到執行檔＝引導改用 gemma 路的錯誤訊息。
|
||
// t92：覆蓋 fallback 清單為空，確保即使本機有 claude 安裝，測試仍能驗到「找不到」路徑。
|
||
func TestExtractWithClaudeMissingBin(t *testing.T) {
|
||
orig := claudeFallbackPaths
|
||
claudeFallbackPaths = nil // 停用 fallback，讓找不到 /no/such/claude-bin 就直接報錯
|
||
defer func() { claudeFallbackPaths = orig }()
|
||
|
||
if _, err := ExtractWithClaude("/no/such/claude-bin", t.TempDir(), "x.md"); err == nil {
|
||
t.Fatal("缺執行檔應報錯")
|
||
}
|
||
}
|