diff --git a/internal/core/channels_clear_credentialless_test.go b/internal/core/channels_clear_credentialless_test.go index e85043f..72d24c9 100644 --- a/internal/core/channels_clear_credentialless_test.go +++ b/internal/core/channels_clear_credentialless_test.go @@ -193,3 +193,43 @@ func accountIDs(t *testing.T, h *accountCleanupHarness, platform string) []strin } return out } + +// ★ 字符串 body(真实 HTTP 路径)+ 不传 platform 时,dry_run 必须被尊重。 +// +// 反例(实测踩到,我自己写的):照其它 handler 写 +// +// `if args.platform == nil then args = req end` +// +// —— 而"不传 platform"在本函数里是**合法用法**(清全部平台),于是 args 被换成 +// HandlerRequest,body 里解出来的 dry_run 就丢了 ⇒ 传 dry_run=false 却被当成 +// 未传,静默只预演。**破坏性操作里最糟的形态**:操作者以为执行了、实际没执行。 +// +// 夹具必须用**字符串 body**(我第一版传 map,走不到这条路径,所以没咬住)。 +func TestAccountClearCredentiallessRespectsDryRunViaStringBody(t *testing.T) { + h := newAccountCleanupHarness(t) + + // 不带 platform + dry_run=false,且 body 是**字符串**(HTTP 真实形态) + res, err := h.ps.InvokeHandler("channels", "account_clear_credentialless", + &lua.HandlerRequest{Method: "POST", Body: `{"dry_run":false}`}) + if err != nil { + t.Fatal(err) + } + if res.Status != 200 { + t.Fatalf("status=%d body=%v", res.Status, res.Body) + } + j, _ := res.JSON.(map[string]interface{}) + if j["dry_run"] == true { + t.Fatalf("字符串 body 里传了 dry_run=false 却被当成未传(回退吃掉 body): %v", j) + } + if j["deleted"] == nil { + t.Fatalf("dry_run=false 时必须真的执行(应有 deleted 字段): %v", j) + } + // 全平台的 5 个空壳都被删(codearts 3 + trae 2) + if got := len(accountIDs(t, h, "codearts")); got != 1 { + t.Fatalf("codearts 应剩 1(有凭据那个),实得 %d", got) + } + if got := len(accountIDs(t, h, "trae")); got != 0 { + t.Fatalf("trae 的 2 个空壳应被删,实得 %d", got) + } + t.Logf("✓ 字符串 body 且无 platform 时 dry_run=false 生效: deleted=%v", j["deleted"]) +} diff --git a/internal/lua/plugins/channels.lua b/internal/lua/plugins/channels.lua index 6a538de..629011d 100644 --- a/internal/lua/plugins/channels.lua +++ b/internal/lua/plugins/channels.lua @@ -6440,7 +6440,15 @@ end function plugin.account_clear_credentialless(req) local args = bodyOf(req) - if args.platform == nil then args = req end + -- ⚠️ 不要照其它 handler 写 `if args.platform == nil then args = req end`: + -- 那个回退是为「直接把参数当 req 传」的旧调用准备的,但它会**丢掉 body** + -- —— 本函数不传 platform 是**合法用法**(清全部平台),此时 args.platform + -- 自然为 nil,回退会把 args 换成 HandlerRequest ⇒ 刚解出来的 dry_run 就没了, + -- 于是「传 dry_run=false 却被当成未传」而静默只预演(破坏性操作里最糟的形态: + -- 操作者以为执行了、实际没执行,或者反过来)。 + -- 实测踩到:不带 platform 时 deleted 恒为 nil。 + -- 兜底只认「req 本身就是参数表」这种形态。 + if type(req) == "table" and req.body == nil and req.query == nil then args = req end local dry = args.dry_run if dry == nil then dry = true end