magic-agent 1.0.2 → 1.0.46

package/internal/agent/permission_test.go

@@ -103,7 +103,7 @@ func TestClaudePermissionModeMapping(t *testing.T) {
 	}
 }
 
-// TestPermissionSupportOf 能力表：只有 claude / codebuddy / codebuddy-ai 接线。
+// TestPermissionSupportOf 能力表：claude / codebuddy / codebuddy-ai 走 flag，dsh 走环境变量。
 func TestPermissionSupportOf(t *testing.T) {
 	for _, e := range []string{"claude", "codebuddy", "codebuddy-ai"} {
 		if !PermissionSupported(e) {
@@ -113,8 +113,15 @@ func TestPermissionSupportOf(t *testing.T) {
 			t.Errorf("PermissionSupportOf(%q) = %q", e, got)
 		}
 	}
+	// dsh 没有逐调用权限 flag，档位走子进程环境变量（shipped 的 sdk profile 直接读它）。
+	if !PermissionSupported("dsh") {
+		t.Error("dsh 应支持四档模型（经 DSH_PERMISSION_MODE）")
+	}
+	if got := PermissionSupportOf("dsh"); got != "env:DSH_PERMISSION_MODE" {
+		t.Errorf("PermissionSupportOf(\"dsh\") = %q, want env:DSH_PERMISSION_MODE", got)
+	}
 	// 未接线的引擎必须报 none —— CLI 层据此 exit 2，而不是静默忽略。
-	for _, e := range []string{"trae", "llm", "codex", "openclaw", "dsh", "arkclaw"} {
+	for _, e := range []string{"trae", "llm", "codex", "openclaw", "arkclaw"} {
 		if PermissionSupported(e) {
 			t.Errorf("%s 不应报告支持四档模型", e)
 		}
@@ -124,6 +131,40 @@ func TestPermissionSupportOf(t *testing.T) {
 	}
 }
 
+// TestDshPermissionMode 四档 → dsh SandboxMode 的映射，以及**空档位不注入**这条红线。
+//
+// 空档位那条是重点：DefaultPermissionTier 是 full，若 dshPermissionMode 在空串上兜默认值，
+// `magic-agent -e dsh`（未显式传档）就会静默把 dsh 的沙箱整个关掉 —— 这正是本仓库最忌讳的
+// 「静默的安全降级」，故用测试钉住。
+func TestDshPermissionMode(t *testing.T) {
+	cases := []struct {
+		tier PermissionTier
+		mode string
+	}{
+		{"", ""}, // 空档 = 不注入（用 dsh 自己的默认，保持既有行为）
+		{PermissionManual, "read-only"},
+		{PermissionAcceptEdits, "workspace-write"},
+		{PermissionAuto, "workspace-write"},
+		{PermissionFull, "danger-full-access"},
+	}
+	for _, c := range cases {
+		if got := dshPermissionMode(c.tier); got != c.mode {
+			t.Errorf("dshPermissionMode(%q) = %q, want %q", c.tier, got, c.mode)
+		}
+		env := dshPermissionEnv(c.tier)
+		if c.mode == "" {
+			if env != nil {
+				t.Errorf("dshPermissionEnv(%q) 应返回 nil（不注入），得到 %v", c.tier, env)
+			}
+			continue
+		}
+		want := []string{"DSH_PERMISSION_MODE=" + c.mode}
+		if len(env) != 1 || env[0] != want[0] {
+			t.Errorf("dshPermissionEnv(%q) = %v, want %v", c.tier, env, want)
+		}
+	}
+}
+
 // ── settings 载荷 ─────────────────────────────────────────────
 
 // decodeSettings 把 --settings 的内联 JSON 解成嵌套 map 便于断言。
@@ -154,6 +195,13 @@ func dig(m map[string]any, path ...string) any {
 // 关键点：autoAllowBashIfSandboxed 是第 1/2 档与第 3 档的分水岭。
 // 该键在 claude 侧的默认值是 true，所以第 1/2 档必须显式写 false ——
 // 否则沙箱内的 Bash 会被自动放行，第 2 档就退化成了「编辑与命令都不问」。
+//
+// ⚠️ 2026-10-01 反转：第 4 档（PermissionFull / bypassPermissions）下，
+// sandboxSettings **返回 nil** 而不是 `{"enabled": false}`。原因是 claude CLI
+// 在 tools=default + bypassPermissions + `--settings {"sandbox":{"enabled":false}}`
+// 组合下报「Invalid JSON provided to --settings」、exit 0 零输出（实测）。
+// bypassPermissions 与 sandbox 互斥：开了也是空跑、显式写 enabled=false 反而触发 bug。
+// 让外层（agentSettingsPayload）整段不下发即可。
 func TestSandboxSettingsPerTier(t *testing.T) {
 	cases := []struct {
 		tier       PermissionTier
@@ -164,19 +212,16 @@ func TestSandboxSettingsPerTier(t *testing.T) {
 		{PermissionManual, true, false, true},
 		{PermissionAcceptEdits, true, false, true},
 		{PermissionAuto, true, true, true},
-		{PermissionFull, false, false, false},
 	}
 	for _, c := range cases {
 		sb := sandboxSettings(c.tier, PermissionOptions{}, "claude")
+		if sb == nil {
+			t.Errorf("%s: sandboxSettings 不应返回 nil", c.tier)
+			continue
+		}
 		if got := sb["enabled"]; got != c.enabled {
 			t.Errorf("%s: sandbox.enabled = %v, want %v", c.tier, got, c.enabled)
 		}
-		if c.tier == PermissionFull {
-			if _, has := sb["autoAllowBashIfSandboxed"]; has {
-				t.Errorf("full 档不该带 autoAllowBashIfSandboxed")
-			}
-			continue
-		}
 		if got := sb["autoAllowBashIfSandboxed"]; got != c.autoAllow {
 			t.Errorf("%s: sandbox.autoAllowBashIfSandboxed = %v, want %v（这是第1/2档与第3档的分水岭）",
 				c.tier, got, c.autoAllow)
@@ -199,6 +244,11 @@ func TestSandboxSettingsPerTier(t *testing.T) {
 	if got := sb["enabled"]; got != true {
 		t.Errorf("codebuddy auto 档 sandbox.enabled = %v, want true", got)
 	}
+
+	// 第 4 档：返回 nil，整段 sandbox 不下发。
+	if got := sandboxSettings(PermissionFull, PermissionOptions{}, "claude"); got != nil {
+		t.Errorf("full 档应返回 nil（实测 2026-10-01：写 enabled=false 触发 claude CLI bug）")
+	}
 }
 
 // TestSandboxSettingsOptions 可选项（排除命令 / 网络白名单 / 放宽逃逸舱口）。
@@ -221,9 +271,9 @@ func TestSandboxSettingsOptions(t *testing.T) {
 		t.Errorf("allowUnsandboxedCommands = %v, want true（显式放宽时）", got)
 	}
 
-	// 第 4 档无沙箱，可选项一律不生效。
-	if sbFull := sandboxSettings(PermissionFull, opts, "claude"); len(sbFull) != 1 {
-		t.Errorf("full 档 sandbox 应只有 enabled 一项，得到 %v", sbFull)
+	// 第 4 档：返回 nil，所有可选项一律不生效。
+	if sbFull := sandboxSettings(PermissionFull, opts, "claude"); sbFull != nil {
+		t.Errorf("full 档 sandbox 应返回 nil（整段不下发），得到 %v", sbFull)
 	}
 }
 
@@ -314,17 +364,17 @@ func TestAgentSettingsPayload(t *testing.T) {
 		t.Error("permissions.ask 丢失")
 	}
 
-	// ④ 第 4 档：显式关闭沙箱（覆盖用户 settings 里可能开着的 sandbox）。
+	// ④ 第 4 档：**不下发 sandbox 整段**（实测 2026-10-01：claude CLI 在
+	//   tools=default + bypassPermissions + `--settings {"sandbox":{"enabled":false}}`
+	//   组合下报「Invalid JSON provided to --settings」、exit 0 零输出）。
+	//   bypassPermissions 与 sandbox 互斥：开了也是空跑、显式写 enabled=false
+	//   反而触发那个 bug。让外层 sandboxSettings 返 nil、外层整段不下发即可。
 	payload, ok = agentSettingsPayload(Request{Tools: ToolsOn, Permission: PermissionFull}, "claude")
-	if !ok {
-		t.Fatal("full 档应注入 --settings（显式关闭沙箱）")
+	if ok {
+		t.Error("full 档应**不**注入 --settings（沙箱整段不下发）")
 	}
-	s = decodeSettings(t, payload)
-	if got := dig(s, "sandbox", "enabled"); got != false {
-		t.Errorf("full 档 sandbox.enabled = %v, want false", got)
-	}
-	if _, has := s["autoMode"]; has {
-		t.Error("full 档不该有 autoMode")
+	if payload != "" {
+		t.Errorf("full 档 payload 应为空，实际 %q", payload)
 	}
 }
 
@@ -374,7 +424,8 @@ func TestClaudeBuildArgsPermissionTiers(t *testing.T) {
 		{PermissionFull, "bypassPermissions"},
 	}
 	for _, c := range cases {
-		// --tools on
+		// --tools on：2026-10-01 必须显式 --tools default（不然 claude CLI 在
+		// stream-json 模式下不发 result 行）；同时仍带 --permission-mode。
 		args := e.buildArgsBase(Request{Tools: ToolsOn, Permission: c.tier})
 		got, ok := argValue(args, "--permission-mode")
 		if !ok {
@@ -383,8 +434,8 @@ func TestClaudeBuildArgsPermissionTiers(t *testing.T) {
 		if got != c.wantMode {
 			t.Errorf("%s(--tools on): --permission-mode = %q, want %q", c.tier, got, c.wantMode)
 		}
-		if _, has := argValue(args, "--tools"); has {
-			t.Errorf("%s(--tools on): 不该传 --tools（on = 引擎默认全工具）", c.tier)
+		if got, _ := argValue(args, "--tools"); got != "default" {
+			t.Errorf("%s(--tools on): --tools = %q, want default（显式开全部内置）", c.tier, got)
 		}
 		// 改造后不该再出现 --dangerously-skip-permissions（档位统一走 --permission-mode）。
 		for _, a := range args {
@@ -392,9 +443,22 @@ func TestClaudeBuildArgsPermissionTiers(t *testing.T) {
 				t.Errorf("%s: 仍在传 --dangerously-skip-permissions，应统一走 --permission-mode", c.tier)
 			}
 		}
-		// 沙箱只能经 --settings 进（claude 没有 --sandbox flag）。
-		if _, has := argValue(args, "--settings"); !has {
-			t.Errorf("%s: 缺少 --settings（沙箱配置没有 CLI flag，只能从这里进）", c.tier)
+		// 第 4 档**不**带 --settings（实测 2026-10-01：claude CLI 在
+		// tools=default + bypassPermissions + `{"sandbox":{"enabled":false}}`
+		// 组合下报「Invalid JSON provided to --settings」、exit 0 零输出）。
+		// 其他档：沙箱只能经 --settings 进（claude 没有 --sandbox flag）。
+		hasSettings := false
+		if _, has := argValue(args, "--settings"); has {
+			hasSettings = true
+		}
+		if c.tier == PermissionFull {
+			if hasSettings {
+				t.Errorf("%s(--tools on): full 档不应带 --settings（实测 2026-10-01 触发 claude CLI bug）", c.tier)
+			}
+		} else {
+			if !hasSettings {
+				t.Errorf("%s(--tools on): 缺少 --settings（沙箱配置没有 CLI flag，只能从这里进）", c.tier)
+			}
 		}
 
 		// --tools 白名单
@@ -422,22 +486,22 @@ func TestClaudeBuildArgsPermissionTiers(t *testing.T) {
 
 // TestClaudeBuildArgsDefaultTierUnchanged 默认档的 argv 与改造前的语义一致。
 //
-// 改造前：--tools on → --dangerously-skip-permissions（无 --settings）。
-// 改造后：--tools on → --permission-mode bypassPermissions + --settings{"sandbox":{"enabled":false}}。
-// 两者在 claude 官方文档里是**等价**的（"Equivalent to --permission-mode bypassPermissions"），
-// 且显式关沙箱能覆盖用户 settings 里可能开着的 sandbox.enabled。
+// 2026-10-01 修订：默认档（Permission 零值 = PermissionFull / bypassPermissions）
+// **不**带 --settings。原因是 claude CLI 在 tools=default + bypassPermissions +
+// `{"sandbox":{"enabled":false}}` 组合下报「Invalid JSON provided to --settings」、
+// exit 0 零输出（实测）。bypassPermissions 与 sandbox 互斥，写 enabled=false 反而
+// 触发那个 bug。让 sandboxSettings 在 full 档返 nil、外层整段不下发即可。
 func TestClaudeBuildArgsDefaultTierUnchanged(t *testing.T) {
 	e := &ClaudeEngine{}
 	args := e.buildArgsBase(Request{Tools: ToolsOn}) // Permission 零值
 	if got, _ := argValue(args, "--permission-mode"); got != "bypassPermissions" {
 		t.Errorf("默认档 --permission-mode = %q, want bypassPermissions", got)
 	}
-	payload, ok := argValue(args, "--settings")
-	if !ok {
-		t.Fatal("默认档应带 --settings（显式关闭沙箱）")
+	if got, _ := argValue(args, "--tools"); got != "default" {
+		t.Errorf("默认档 --tools = %q, want default（2026-10-01 必须显式 --tools default）", got)
 	}
-	if got := dig(decodeSettings(t, payload), "sandbox", "enabled"); got != false {
-		t.Errorf("默认档 sandbox.enabled = %v, want false", got)
+	if _, has := argValue(args, "--settings"); has {
+		t.Errorf("默认档（full）不应带 --settings（实测触发 claude CLI bug）")
 	}
 }
 

package/internal/agent/repair_models_test.go

@@ -0,0 +1,194 @@
+package agent
+
+// repair_models_test.go - 事故还原（--repair-models 的落地逻辑，见 custom_models.go）。
+//
+// 场景来自 2026-09-30 Windows 事故：桌面端 models.json 是**数组**，被反向同步覆盖成
+// `{availableModels, models}` 对象后桌面端解析不了，模型清单整个为空。
+// 这里锁住三件事：
+//  1. 明确的坏形状 → 还原成数组，id 去掉 custom-local: 前缀，且先留 .broken.bak
+//  2. 已经是数组 → 一个字都不动
+//  3. 形状对不上（坏 JSON / 空对象 / models 里混着非 custom-local 条目）→ 一律不动
+
+import (
+	"bytes"
+	"encoding/json"
+	"os"
+	"path/filepath"
+	"strings"
+	"testing"
+)
+
+// clobberedFixture 造一份「被我们覆盖写坏」的 models.json：顶层对象 + 全 custom-local 条目。
+func clobberedFixture() []byte {
+	return []byte(`{
+  "availableModels": [
+    "custom-local:MiniMax-M3",
+    "custom-local:glm-5.3"
+  ],
+  "models": [
+    {
+      "id": "custom-local:MiniMax-M3",
+      "name": "MiniMax-M3",
+      "vendor": "MiniMax",
+      "url": "https://api.minimaxi.com/v1/chat/completions",
+      "apiKey": "sk-user-own",
+      "supportsImages": false
+    },
+    {
+      "id": "custom-local:glm-5.3",
+      "name": "glm-5.3",
+      "vendor": "Zhipu",
+      "url": "https://open.bigmodel.cn/api/paas/v4",
+      "apiKey": "sk-user-own-2"
+    }
+  ]
+}
+`)
+}
+
+func TestRepairModelsFile_RestoresArrayFormat(t *testing.T) {
+	appHome := t.TempDir()
+	path := filepath.Join(appHome, customModelsFileName)
+	if err := os.WriteFile(path, clobberedFixture(), 0o600); err != nil {
+		t.Fatal(err)
+	}
+
+	repaired, note, err := repairModelsFile(appHome)
+	if err != nil {
+		t.Fatalf("repair: %v", err)
+	}
+	if !repaired {
+		t.Fatalf("坏形状应被还原，note=%q", note)
+	}
+
+	b, err := os.ReadFile(path)
+	if err != nil {
+		t.Fatal(err)
+	}
+	var arr []map[string]any
+	if err := json.Unmarshal(b, &arr); err != nil {
+		t.Fatalf("还原后应是数组: %v（内容 %s）", err, b)
+	}
+	if len(arr) != 2 {
+		t.Fatalf("条数 = %d, want 2", len(arr))
+	}
+	for _, m := range arr {
+		id, _ := m["id"].(string)
+		if id == "" || strings.HasPrefix(id, customModelIDPrefix) {
+			t.Errorf("id 必须去掉 %s 前缀，实际 %q", customModelIDPrefix, id)
+		}
+	}
+	if arr[0]["id"] != "MiniMax-M3" {
+		t.Errorf("首条 id = %v, want MiniMax-M3", arr[0]["id"])
+	}
+	// 端点与密钥必须原样保留（丢了就只能手抄）
+	if arr[0]["url"] != "https://api.minimaxi.com/v1/chat/completions" || arr[0]["apiKey"] != "sk-user-own" {
+		t.Errorf("端点/密钥没保留: %v", arr[0])
+	}
+
+	// 现场必须留备份，且备份是坏的那份原文
+	bak, err := os.ReadFile(path + ".broken.bak")
+	if err != nil {
+		t.Fatalf("应留 .broken.bak: %v", err)
+	}
+	if !bytes.Equal(bak, clobberedFixture()) {
+		t.Error(".broken.bak 应为被写坏的原文")
+	}
+}
+
+func TestRepairModelsFile_LeavesGoodShapesAlone(t *testing.T) {
+	// 桌面端原生数组格式
+	native := []byte("[\n  {\n    \"id\": \"MiniMax-M3\",\n    \"url\": \"https://x/v1\"\n  }\n]\n")
+
+	cases := []struct {
+		name    string
+		content []byte
+	}{
+		{"已是数组（桌面端原生格式）", native},
+		{"坏 JSON", []byte("{oops")},
+		{"空 JSON 对象", []byte("{}")},
+		{"顶层是字符串", []byte(`"nope"`)},
+		{"models 里混着非 custom-local 条目", []byte(`{"models":[{"id":"hy3","url":"https://x"}]}`)},
+		{"models 是空数组", []byte(`{"models":[],"availableModels":[]}`)},
+	}
+
+	for _, tc := range cases {
+		t.Run(tc.name, func(t *testing.T) {
+			appHome := t.TempDir()
+			path := filepath.Join(appHome, customModelsFileName)
+			if err := os.WriteFile(path, tc.content, 0o600); err != nil {
+				t.Fatal(err)
+			}
+
+			repaired, _, err := repairModelsFile(appHome)
+			if err != nil {
+				t.Fatalf("形状对不上应静默不改，不该报错: %v", err)
+			}
+			if repaired {
+				t.Errorf("形状对不上不该动手")
+			}
+			after, rerr := os.ReadFile(path)
+			if rerr != nil {
+				t.Fatal(rerr)
+			}
+			if !bytes.Equal(after, tc.content) {
+				t.Errorf("原文件被改写了\nbefore=%s\nafter =%s", tc.content, after)
+			}
+			if _, serr := os.Stat(path + ".broken.bak"); !os.IsNotExist(serr) {
+				t.Errorf("没还原就不该留 .broken.bak")
+			}
+		})
+	}
+}
+
+func TestRepairModelsFile_MissingFileIsNoop(t *testing.T) {
+	appHome := t.TempDir()
+	repaired, note, err := repairModelsFile(appHome)
+	if err != nil {
+		t.Fatalf("没有文件不该报错: %v", err)
+	}
+	if repaired {
+		t.Error("没有文件时不谈还原")
+	}
+	if note == "" {
+		t.Error("应给出「不存在、无需修复」的说明，不静默")
+	}
+}
+
+// 幂等：还原过一次之后，再跑一次必须什么都不做（已经是数组）。
+func TestRepairModelsFile_Idempotent(t *testing.T) {
+	appHome := t.TempDir()
+	path := filepath.Join(appHome, customModelsFileName)
+	if err := os.WriteFile(path, clobberedFixture(), 0o600); err != nil {
+		t.Fatal(err)
+	}
+	if repaired, _, err := repairModelsFile(appHome); err != nil || !repaired {
+		t.Fatalf("首次应还原: repaired=%v err=%v", repaired, err)
+	}
+	first, err := os.ReadFile(path)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	repaired, _, err := repairModelsFile(appHome)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if repaired {
+		t.Error("第二次不该再动手")
+	}
+	second, err := os.ReadFile(path)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if !bytes.Equal(first, second) {
+		t.Error("第二次不该改动文件")
+	}
+}
+
+// 引擎名到桌面端数据目录的映射：不认识的名字必须报错，而不是猜一个目录去写。
+func TestRepairDesktopModels_UnknownEngine(t *testing.T) {
+	if _, _, err := RepairDesktopModels("no-such-engine"); err == nil {
+		t.Fatal("没有桌面端目录的引擎应报错")
+	}
+}

package/internal/agent/runcmd.go

@@ -73,7 +73,10 @@ func runCLIEnv(ctx context.Context, extra []string, bin string, args ...string)
 
 // runCLIEnvIn = runCLIIn + runCLIEnv（工作目录 + 追加环境变量）。
 func runCLIEnvIn(ctx context.Context, dir string, extra []string, bin string, args ...string) (string, string, error) {
-	cmd := exec.Command(bin, args...)
+	// launchWrap 与流式路径同源：.cmd/.bat 在 Windows 上不能直接 exec，
+	// 参数会被 cmd.exe 拆坏（实测含换行的提示词整段丢失）。unix 上原样透传。
+	spec := launchWrap(bin, args)
+	cmd := exec.Command(spec.bin, spec.args...)
 	if dir != "" {
 		cmd.Dir = dir
 	}
@@ -147,7 +150,8 @@ func waitDrained(ch <-chan struct{}, n int, d time.Duration) {
 	}
 }
 
-// envWithDefaults 继承当前环境（CLI 依赖 HOME/PATH 等基础变量）。
+// envWithDefaults 继承当前环境（CLI 依赖 HOME/PATH 等基础变量），
+// 父-side 专属项已在 environ() 阶段过滤（参见 envDenylist）。
 func envWithDefaults() []string {
 	return environ()
 }
@@ -158,8 +162,14 @@ func newStreamCmd(bin string, args []string) *exec.Cmd {
 }
 
 // newStreamCmdIn 同 newStreamCmd，可指定工作目录（workspace，见 runCLIIn 注释）。
+//
+// ⚠️ 这里的 launchWrap 是 Windows 存活的关键（2026-10-01 实测）：
+// npm 装的 claude / codebuddy 是 .cmd 批处理，直接 exec 会让
+// 含换行 / 引号 / & | ^ % 的参数被 cmd.exe 拆坏 → 静默退 0、零输出。
+// unix 上它原样返回，行为不变。见 launch.go。
 func newStreamCmdIn(dir string, bin string, args []string) *exec.Cmd {
-	cmd := exec.Command(bin, args...)
+	spec := launchWrap(bin, args)
+	cmd := exec.Command(spec.bin, spec.args...)
 	if dir != "" {
 		cmd.Dir = dir
 	}

package/internal/agent/runner.go

@@ -53,6 +53,8 @@ type Runner struct {
 }
 
 // defaultTimeout 按引擎取默认超时。
+//
+// 2026-10-10 起各引擎统一为 1 小时；未在上面列出的引擎同样回落到 1 小时。
 func (r *Runner) defaultTimeout() time.Duration {
 	switch r.Engine.(type) {
 	case *ClaudeEngine:
@@ -71,8 +73,12 @@ func (r *Runner) defaultTimeout() time.Duration {
 		return DefaultCBGatewayTimeout
 	case *DshEngine:
 		return DefaultDshTimeout
+	case *OpenClawEngine:
+		return DefaultOpenClawTimeout
+	case *CodexEngine:
+		return DefaultCodexTimeout
 	}
-	return 5 * time.Minute
+	return time.Hour
 }
 
 // isRetryable 判断错误是否值得重试。

package/internal/agent/runner_test.go

@@ -252,7 +252,7 @@ func TestRunnerDefaultTimeoutByEngine(t *testing.T) {
 		{&CodeBuddyEngine{}, DefaultCodeBuddyTimeout},
 		{&CodeBuddyAIEngine{}, DefaultCodeBuddyTimeout},
 		{&TraeEngine{}, DefaultTraeTimeout},
-		{&counterEngine{name: "x"}, 5 * time.Minute},
+		{&counterEngine{name: "x"}, time.Hour},
 	}
 	for _, c := range cases {
 		r := &Runner{Engine: c.e}