|
|
2e90c2cb5a
|
refactor(webui): openai_compat 判据迁到 test/，包内重建共享辅助
openai_compat_test.go → webui/test/ （4 条判据）
webui/ 178 → 174 条
webui/test/ 24 → 28 条
合计 202 条不变
## 迁移碰到的连带问题
`openai_compat_test.go` 里除了自己的判据，还定义了三个**被包内其他文件
依赖**的共享件：
newOpenAITestServer(t) auth_hardening_test.go 在用
testAuthAPIKey auth_hardening / gzip / legacy_proxy(×2) 在用
seedWebUIConfig 多处
文件一移走，包内立刻 build failed：
vet: auth_hardening_test.go:373:15: undefined: newOpenAITestServer
vet: auth_hardening_test.go:377:30: undefined: testAuthAPIKey
这说明「迁一个测试文件」从来不是孤立的 —— 它可能是包内共享辅助的**宿主**。
## 处理方式：两边各留一份，不跨包共享
包内重定义 `testAuthAPIKey` 与 `newOpenAITestServer`（放 handler_test.go），
test/ 侧保留自己那份（`seedWebUIConfig` 统一收进 helpers_test.go）。
**为什么不抽到共享包**：包内版本要用包内的 `testSDK`/`seedWebUIConfig`，
test/ 版本要用那边的 helper。抽公共包等于让 test/ 反向依赖包内的未导出
辅助 —— 那正是这次迁移要摆脱的东西。两份各写 20 行，换来两侧互不牵连，
比一个"共享测试工具包"更符合这次的目的。
## 顺带修掉一处重复定义
`seedWebUIConfig` 在 secret_meta_test.go 与 helpers_test.go 各出现过一次
（上一批我加的那份没和已有的对齐）。收拢到 helpers_test.go 一份，
字段与包内版本保持一致（api_key/username/password/session_ttl_hours）。
## 验证
go build ./... PASS
go vet ./... PASS
gofmt -l 干净
go test ./... -count=1 67 包全过，0 FAIL
webui/ + webui/test/ 174 + 28 = 202 条，与迁移前一致
|
2026-11-04 21:52:52 +08:00 |
|
|
|
b3ae99e32f
|
refactor(webui): secret meta 判据迁到 test/ —— 走导出路由而非未导出 handler
## 迁移
secret_meta_test.go → webui/test/
## 这一条迁移暴露的问题比前几次多
原测试用两种白盒写法：
h := &Handler{settings: ...} // 直接构造，设私有字段
h.handleSettings(w, req) // 直接调未导出 handler
h.handleSettingsReveal(...) // 同上
改成黑箱后依次撞到四件事，逐个解决：
### ① 路由是 `/api/v1/settings`，不是 `/api/settings`
原测试写的路径 `/api/settings` 从来没被路由表匹配过 —— 它直接调 handler，
所以路径是摆设。真实注册在 handler.go:551：
mux.HandleFunc("/api/v1/settings", h.requireAPI(h.handleSettings))
⇒ 走真实路由后必须用对路径，否则 404。
### ② `Handler()` 会套三层中间件，依赖完整装配
`Handler()` = proxyDispatch + gzipMW + logged + mux。其中 `logged` 读
`h.settings` 的 webui 配置、`requestLog` 读 history store 等 ——
这些在最小装配下是 nil，直接 panic。
这不是"中间件写坏了"，是本判据不该测它们。**用导出的 `RegisterRoutes`**：
mux := http.NewServeMux()
h.RegisterRoutes(mux) // 只注册业务路由，不套中间件
`RegisterRoutes` 是导出接口，拿到的是真实路由表 —— 仍是黑箱，
只是不带外层。判据测的是 settings 的 meta 与 reveal，不该被请求日志和
反代的依赖拖下水。
### ③ `requireAPI` 需要 api_key，没配就 503
{"error":"webui api_key not configured"}
补 `seedWebUIConfig`（与包内同名辅助一致），并在请求上带
`X-API-Key`。这是生产上非浏览器客户端的正常路径
（`validAPIKey` 优先于 cookie session）。
### ④ `SecretPlaceholder` 需导出访问
改成 `webui.SecretPlaceholder`。
## 判据有效性（变异验证）
secret 标记的传递链是
`ConfigDef.Secret → mapDefs → sdk.ConfigDef.Secret → HTTP meta`。
在 `internal/sdk/settings_impl.go` 的 mapDefs 里删掉 `Secret: d.Secret`：
--- FAIL: TestSecretFlagReachesSettingsMeta
Secret:true 未传递到 meta（前端会明文回显）:
map[default: display_name:标志式 key:by_flag type:string]
恢复后全绿。这条判据守的是真实缺陷形态：SDK 的 ConfigDef 一直有 Secret
字段，但内核侧三处转换都漏过它，导致敏感项被渲染成普通文本框明文回显。
## 进度
webui/ 178 条（迁走 2 条）
webui/test/ 24 条
`webui/` 剩 19 个测试文件。剩下的大头是 `proxy_test.go`（45 处未导出引用）
与 `handler_test.go`（90 处），它们直接调 `handleXxx` 并依赖
`newTestHandler`/`testSDK` 这套装配辅助 —— 迁移需要先把那套辅助搬到
test/ 共享，再逐个改调用点。
## 验证
go build ./... PASS
go vet ./... PASS
gofmt -l 干净
go test ./... -count=1 67 包全过，0 FAIL
|
2026-11-04 21:34:17 +08:00 |
|
|
|
b8ec1e84d2
|
refactor(webui): 再迁 3 个判据到 test/ —— 去重辅助函数
## 迁移
dashboard_js_scope_test.go → webui/test/
embed_read_test.go → webui/test/
mascot_asset_test.go → webui/test/
这三个原先都直接读未导出的 `dashboardFS`（`go:embed` 的变量）。
上一批已经加了导出入口 `FrontendAsset(name)`，所以只需换调用即可，
不涉及行为改动。
## 迁移暴露的三处重复（一并去重）
迁过去之后编译报了三组重复定义：
stripJSComments helpers_test.go 与 dashboard_js_scope_test.go 各一份
dashboardJSSource helpers_test.go 与 dashboard_js_scope_test.go 各一份
mustReadEmbedded embed_read_test.go 里定义，kernel_buttons_render_test.go 在用
前两个是**迁移造成的镜像**：原本包内的辅助函数在迁到 test/ 时被复制了一份。
逐字比较确认两份 `stripJSComments` 实现完全相同，删掉包内那份、保留
`test/helpers_test.go` 里的共享版本。
第三个不同：`mustReadEmbedded` 只有包内的 `kernel_buttons_render_test.go` 在用。
把它改成直接调导出的 `FrontendAsset`，不再需要这个辅助函数 ——
包内测试从此也不再依赖"哪个变量装着 embed 资源"这种实现细节。
## 顺带清掉一批未使用导入
批量迁移+脚本改包名之后，多个文件留着已经不用的 `webui` 导入
（有的只在注释里提到 `webui.FrontendAsset`）。用 `go vet` 的输出逐个清理，
最后 `go vet ./internal/plugins/webui/...` 干净。
## 判据未失效（变异验证）
把 dashboard.js 里 ack 超时 `}, 600000);` 改成 `}, 30000);`：
--- FAIL: TestChatAckTimeoutIsGenerous
chat_ack_timeout_test.go:59: ★ 受理超时只有 30000 ms —— 太小。
恢复后全绿。
## 进度
webui/ 180 条（原 186，迁走 6 条）
webui/test/ 22 条
合计 202 条，与迁移前一致（+ 之前新增的契约判据）
`webui/` 剩下 20 个测试文件，全部依赖 `handleXxx` / `newTestHandler` /
`testSDK` 等未导出 handler 与装配辅助，需要先有导出入口才能迁。
其中 `proxy_test.go`（45 处）和 `handler_test.go`（90 处）是最大两块，
要迁必须先给 webui 导出"起一个带路由的 handler"的测试入口。
## 验证
go build ./... PASS
go vet ./internal/plugins/webui/... PASS
gofmt -l 干净
go test ./... -count=1 65 包全过，0 FAIL
webui 连跑 3 次 0 失败
|
2026-11-04 20:23:34 +08:00 |
|
|
|
87f874e06f
|
refactor(webui): 7 个黑箱判据迁到 test/ + 导出前端资源入口
## 迁移了什么
chat_ack_timeout_test.go → test/
chat_attachment_img_test.go → test/
cmd_history_render_test.go → test/
config_namespace_test.go → test/
dashboard_tabs_deps_test.go → test/
plugin_status_render_test.go → test/
starmap_perf_test.go → test/
包名 `webui` → `test`。`webui/` 目录少 7 个测试文件。
## 为什么是这 7 个（分类依据）
先按「对未导出符号的依赖数」把 29 个测试文件测了一遍，这 7 个是
**0 依赖**（既不用 handleXxx，也不用 testSDK/newTestHandler 等测试辅助）。
其余 22 个按强度递增留在包内，后续逐步处理。
## 新增两个导出入口
### webui.FrontendAsset(name)
返回打包进二进制的前端资源原文（dashboard.js / dashboard.css /
mascot.webp / logo.svg …）。
判据要检查「实际会被发出去的那份文本」：dashboard.js 在严格模式下
会不会抛 ReferenceError（用 node 真跑）、mascot.webp 的魔数是否合法、
HTML 属性里有没有残留转义引号。
这些无法通过 HTTP 端点可靠断言（要登录态、且拿到的是拼接后整页，
定位不到单个 asset）。而留在包内测就得读 `dashboardFS` 这个**未导出变量**
—— 那正是"测试耦合实现"。导出后，判据变成「输入 asset 名 → 输出字节」，
与资源存在哪个变量、怎么 embed 无关。
### webui.DashboardPage()
返回组装完成的整页 HTML（外壳 + 内联 CSS/JS）—— 浏览器实际收到的那份。
与 FrontendAsset 的分工：那个取单个源资产，这个取成品。
## 顺带修掉迁移暴露的两个隐患
① 三个判据原本 `os.ReadFile("dashboard.js")` 读**磁盘相对路径**。
在包内恰好能跑（工作目录就是包目录），但那读的是**工作区文件**，
不是**二进制里实际嵌入的那份** —— 两者在生产上可能不同
（比如打包后 dashboard.js 被替换）。统一改走 FrontendAsset。
② starmap_perf_test 的相对路径 `../../../shared/starmap/layout.js`
与 `static/starmap-layout-worker.js` 在移入子目录后失效，已修正为
`../../../../shared/...` 与 `../static/...`。
## 判据仍有效（变异验证）
把 dashboard.js 里 ack 超时 `}, 600000);` 改成 `}, 30000);`：
--- FAIL: TestChatAckTimeoutIsGenerous
★ 受理超时只有 30000 ms —— 太小。
超时太短会让前端 abort，而 agent 仍在正常干活
恢复后全绿。迁移不是搬完就废。
## 验证
go build ./... PASS
go vet ./... PASS
gofmt -l 干净
go test ./... -count=1 63 包全过，0 FAIL
（webui/test 是新包，故 62 → 63）
干净检出（worktree） build + webui/... 全绿
## 后续
webui 剩余 22 个文件（依赖 handleXxx / testSDK 等），按强度递增处理
agent/core 739 处白盒引用，先分类再动
CI 补 coverage + staticcheck
|
2026-11-04 18:17:31 +08:00 |
|
