Files
arcrun-collector/upstream_error_visible_test.go
Leo c7675a10a0 不再把開發用 template 塞進使用者資料夾;上游錯誤不再被退避訊息吞掉
## leo 三條裁決,逐條落實
① 「**別人的錯誤一律要顯示給用戶看,不然就會變成我的錯誤,導致客服**」
   → manifest entry 新增 `LastError`(存**原文**),退避訊息改成
     「上次失敗(第 N 次),58m 後重試|**原因:**<上游原文>」。
     成功後清空,不留舊錯誤嚇人。
     測試 `TestUpstreamErrorSurvivesBackoff` 從上游錯誤字串出發,
     驗它活到給使用者看的那句話;反向驗證拿掉即紅。

② 「**拿來開發一般人用不到的根本別安裝**」
   → daemon 不再代裝 system-dev template(`CLAUDE.md`/`scripts/`/`system-dev/`,37 檔)。
     兩層傷害:把人家資料夾弄亂 + 那些檔被當知識吃進去
     ⇒ 知識庫長出 `kb`/`t195-watch` 這種不是使用者內容的庫(leo 實撞)。
     `template-install` 子命令保留,開發者情境不受影響。

③ 「**它也不能只看隱藏檔內,因為 template 在我所有的 repo 裡不是隱藏的**」
   → 排除規則用**路徑身分**(`TemplateOwns()`,來源是內嵌 templatefs 的實際路徑
     + `system-dev/`・`scripts/` 目錄前綴),**不是**「有沒有以點開頭」。
     測試刻意把 template 檔放成**不隱藏**(就像 leo 的 repo),驗它仍被排除;反向驗證即紅。
     這些檔也不計進「有 N 個檔案沒有被整理」——那欄是給使用者看他自己的檔案的。

## 順帶修 leo 截圖上的重複
「有 2 個檔案沒有被整理」底下 `scripts/sdd-active-check.sh` 出現兩次
——多帳號時同一個資料夾被掃多輪、每輪都 append。已去重。

## 殘項(誠實)
App 端失敗卡還沒接上這條線 ⇒ **畫面尚未真的顯示原因**。未打包、未送達。
2026-08-06 21:33:37 +08:00

45 lines
1.8 KiB
Go

package collector
import (
"strings"
"testing"
)
// TestUpstreamErrorSurvivesBackoff 釘住 leo 2026-08-06 立的原則:
//
// 「**別人的錯誤一律要顯示給用戶看,不然就會變成我的錯誤,導致客服**」
//
// 事故現場:Cloudflare 回「當日免費額度用完」、某 PDF「沒有可抽取的文字」,
// 但檔案一進退避,畫面就只剩「上次失敗(第 4 次),58m 後重試」
// ⇒ 上游的錯被我們吞掉,使用者只看得到「Arcrun 在失敗」。
//
// 這支測試從**上游錯誤字串**出發,驗它一路活到給使用者看的那句話裡。
func TestUpstreamErrorSurvivesBackoff(t *testing.T) {
const upstream = "雲端萃取失敗(HTTP 502):Workers AI 執行失敗:4006: you have used up your daily free allocation of 10,000 neurons"
m := &Manifest{Entries: map[string]*ManifestEntry{"報告.pdf": {}}}
if !m.MarkFailed("報告.pdf", 1000, upstream) {
t.Fatal("MarkFailed 應該成功")
}
// ① 原因要存得住(退避跨輪次,不能只活在當次記憶體裡)
if got := m.Entries["報告.pdf"].LastError; got != upstream {
t.Fatalf("上游原因沒存下來,實得 %q", got)
}
// ② 使用者看到的那句話裡要有它
reason := retrySkipReason(m, "報告.pdf", 1010)
if !strings.Contains(reason, "neurons") {
t.Fatalf("退避訊息吞掉了上游的錯 ⇒ 使用者會以為是我們壞掉。實得:%s", reason)
}
if !strings.Contains(reason, "後重試") {
t.Errorf("重試排程也要講(兩件事都要說),實得:%s", reason)
}
// ③ 成功之後要清乾淨,不能一直掛著舊錯誤嚇人
m.MarkIngestedBy("報告.pdf", "hash", 2000, "workers-ai")
if m.Entries["報告.pdf"].LastError != "" {
t.Error("成功後舊的失敗原因應該清掉")
}
}