magic-agent 1.0.3 → 1.0.49

package/internal/agent/dsh_test.go

@@ -65,8 +65,15 @@ func TestDshCapabilities(t *testing.T) {
 	if got := WorkspaceSupportOf("dsh"); got != "cwd" {
 		t.Errorf("WorkspaceSupportOf(dsh) = %q want cwd（调用目录即 workspace 根）", got)
 	}
+	// 附件：SDK 通道（默认）把图片作为原生 image block 发出；显式切 headless 后退化为
+	// 提示词路径（能力字段随实际通道走，见 AttachmentSupportOf）。
+	t.Setenv(DshProfileEnv, "")
+	if got := AttachmentSupportOf("dsh"); got != "sdk:content-blocks" {
+		t.Errorf("AttachmentSupportOf(dsh) = %q want sdk:content-blocks（SDK 通道原生收图）", got)
+	}
+	t.Setenv(DshProfileEnv, dshDefaultProfile)
 	if got := AttachmentSupportOf("dsh"); got != "prompt" {
-		t.Errorf("AttachmentSupportOf(dsh) = %q want prompt", got)
+		t.Errorf("headless 下 AttachmentSupportOf(dsh) = %q want prompt（无图片通道）", got)
 	}
 	if got := PermissionSupportOf("dsh"); got != "none" {
 		t.Errorf("PermissionSupportOf(dsh) = %q want none（预设无法表达四档）", got)
@@ -197,29 +204,173 @@ func TestDshEmptyOutput(t *testing.T) {
 	}
 }
 
-// 会话续接：headless 每次都是全新会话 → --session / -c 都明确报错（不静默开新会话）。
-func TestDshSessionUnsupported(t *testing.T) {
+// 会话续接 = **真续接**：走 headless + --session-id，由 dsh 自己把上下文接回来。
+// SDK 通道没有 resume 方法，所以续接轮**必须绕开它**（见 dsh_resume.go）。
+func TestDshResumeUsesHeadlessSessionID(t *testing.T) {
 	dir := t.TempDir()
-	w, log := argsCaptureCLI(t, dir, "dsh", `echo '不该被执行'`)
+	w, log := argsCaptureCLI(t, dir, "dsh", `echo 'ok'`)
+	// 默认 Profile 空 = 平时优先 SDK 通道，但续接轮不该走它。
 	e := &DshEngine{BinPath: w.bin}
 
 	if _, err := e.Complete(context.Background(), Request{
+		Messages:  []Message{{Role: "user", Content: "接着问"}},
+		SessionID: "sess-1",
+	}); err != nil {
+		t.Fatalf("续接不该报错: %v", err)
+	}
+	args := readFileString(t, log)
+	for _, want := range []string{"--profile headless", "--session-id sess-1", "接着问"} {
+		if !strings.Contains(args, want) {
+			t.Errorf("续接轮参数里应有 %q：%q", want, args)
+		}
+	}
+	if strings.Contains(args, "--profile sdk") {
+		t.Errorf("续接轮不该走 SDK 通道（它没有 resume）：%q", args)
+	}
+}
+
+// 续接轮不告警「--model 不生效」：那种轮的模型由**会话**决定。
+func TestDshResumeSkipsModelWarning(t *testing.T) {
+	dir := t.TempDir()
+	w, _ := argsCaptureCLI(t, dir, "dsh", `echo 'ok'`)
+	old := stderr
+	var buf strings.Builder
+	stderr = &buf
+	defer func() { stderr = old }()
+
+	if _, err := (&DshEngine{BinPath: w.bin}).Complete(context.Background(), Request{
 		Messages:  []Message{{Role: "user", Content: "hi"}},
 		SessionID: "sess-1",
-	}); err == nil {
-		t.Error("--session 应报错")
-	} else if !strings.Contains(err.Error(), "不支持会话续接") {
-		t.Errorf("错误文案应说明不支持续接: %v", err)
+		Model:     "modelverse/deepseek-v4.1-flash",
+	}); err != nil {
+		t.Fatalf("Complete: %v", err)
 	}
-	if _, err := e.Complete(context.Background(), Request{
-		Messages: []Message{{Role: "user", Content: "hi"}},
-		Continue: true,
-	}); err == nil {
-		t.Error("-c/--continue 应报错")
+	if strings.Contains(buf.String(), "不生效") {
+		t.Errorf("续接轮不该告警 -m 不生效：%q", buf.String())
+	}
+}
+
+// dsh 太老（headless 不认 --session-id）→ **明确报错**，而不是把 flag 拼进任务文本。
+func TestDshResumeUnsupportedHintsUpgrade(t *testing.T) {
+	dir := t.TempDir()
+	bin := filepath.Join(dir, "dsh")
+	script := "#!/bin/sh\n" +
+		"case \"$*\" in *--help*) echo 'Usage: dsh --profile headless [--json] [task...]'; exit 0;; esac\n" +
+		"echo 'ok'\n"
+	if err := os.WriteFile(bin, []byte(script), 0o755); err != nil {
+		t.Fatal(err)
 	}
-	// 两种情况下都不该真的启动 CLI。
-	if _, err := os.Stat(log); err == nil {
-		t.Error("续接被拒时不应启动 dsh 进程")
+	_, err := (&DshEngine{BinPath: bin}).Complete(context.Background(), Request{
+		Messages:  []Message{{Role: "user", Content: "hi"}},
+		SessionID: "s1",
+	})
+	if err == nil || !strings.Contains(err.Error(), "不认 --session-id") {
+		t.Fatalf("应明确报「不支持 --session-id」, got %v", err)
+	}
+}
+
+// --json 事件流 → 与 SDK 通道同构的思考 / 工具 / 正文 / 收尾事件。
+func TestDshStreamHeadlessJSONEvents(t *testing.T) {
+	dir := t.TempDir()
+	lines := []string{
+		`{"type":"session","sessionId":"sess-1","cwd":"/tmp"}`,
+		`{"type":"status","phase":"turn_start","turn":1}`,
+		`{"type":"thinking","text":"想了想"}`,
+		`{"type":"tool_call","callId":"c1","tool":"Bash","input":{"command":"ls"}}`,
+		`{"type":"tool_result","callId":"c1","status":"completed","result":"a.txt"}`,
+		`{"type":"text","text":"答案是 42"}`,
+		`{"type":"status","phase":"turn_end","turn":1,"reason":{"kind":"completed"}}`,
+		`{"type":"final","text":"答案是 42"}`,
+	}
+	body := "case \"$*\" in *--help*) echo 'Usage: dsh --profile headless [--json] [--session-id <id>] [task...]'; exit 0;; esac\n"
+	for _, l := range lines {
+		body += "printf '%s\\n' '" + l + "'\n"
+	}
+	w, log := argsCaptureCLI(t, dir, "dsh", body)
+
+	var got []StreamEvent
+	var sid string
+	res, err := (&DshEngine{BinPath: w.bin}).Stream(context.Background(), Request{
+		Messages:    []Message{{Role: "user", Content: "问"}},
+		SessionID:   "sess-1",
+		OnSessionID: func(id string) { sid = id },
+	}, func(ev StreamEvent) { got = append(got, ev) })
+	if err != nil {
+		t.Fatalf("Stream: %v", err)
+	}
+	if args := readFileString(t, log); !strings.Contains(args, "--json") {
+		t.Errorf("headless 流式应带 --json：%q", args)
+	}
+	if res.Response.Text != "答案是 42" {
+		t.Errorf("Text = %q want 答案是 42", res.Response.Text)
+	}
+	if res.Response.SessionID != "sess-1" {
+		t.Errorf("SessionID = %q want sess-1（续接轮原样回报）", res.Response.SessionID)
+	}
+	if sid != "sess-1" {
+		t.Errorf("OnSessionID = %q want sess-1", sid)
+	}
+	if res.Thinking != "想了想" {
+		t.Errorf("Thinking = %q want 想了想", res.Thinking)
+	}
+	if len(res.Tools) != 1 || res.Tools[0].Name != "Bash" || res.Tools[0].Result != "a.txt" {
+		t.Errorf("Tools = %+v", res.Tools)
+	}
+	var kinds []StreamEventKind
+	for _, ev := range got {
+		kinds = append(kinds, ev.Kind)
+	}
+	want := []StreamEventKind{KindThinking, KindToolUse, KindToolResult, KindText, KindTurnEnd}
+	if fmt.Sprint(kinds) != fmt.Sprint(want) {
+		t.Errorf("事件序列 = %v want %v", kinds, want)
+	}
+}
+
+// 参数形态：启动器 flag 在前，--session-id / --json 在 app 参数区（headless 自己的 flag）。
+func TestDshHeadlessArgs(t *testing.T) {
+	cases := []struct {
+		name      string
+		sessionID string
+		json      bool
+		want      []string
+	}{
+		{"纯 headless", "", false, []string{"--profile", "headless", "任务"}},
+		{"续接", "s1", false, []string{"--profile", "headless", "--session-id", "s1", "任务"}},
+		{"事件流", "", true, []string{"--profile", "headless", "--json", "任务"}},
+		{"续接 + 事件流", "s1", true, []string{"--profile", "headless", "--session-id", "s1", "--json", "任务"}},
+	}
+	for _, c := range cases {
+		got := dshHeadlessArgs("headless", c.sessionID, "任务", c.json)
+		if fmt.Sprint(got) != fmt.Sprint(c.want) {
+			t.Errorf("%s: %v want %v", c.name, got, c.want)
+		}
+	}
+}
+
+// -c/--continue：按会话登记表找**同一 workspace** 下最近一条 dsh 会话（其它引擎不算）。
+func TestDshLatestSessionIDPrefersWorkspace(t *testing.T) {
+	sess := t.TempDir()
+	writeRun := func(name, engine, id, ws, at string) {
+		t.Helper()
+		rec := `{"run_id":"` + name + `","session_id":"` + id + `","engine":"` + engine +
+			`","workspace":"` + ws + `","updated_at":"` + at + `"}`
+		if err := os.WriteFile(filepath.Join(sess, name+".json"), []byte(rec), 0o600); err != nil {
+			t.Fatal(err)
+		}
+	}
+	writeRun("run-a", "dsh", "sid-a", "/proj/a", "2026-10-09T12:00:00+08:00")
+	writeRun("run-b", "dsh", "sid-b", "/proj/b", "2026-10-09T13:00:00+08:00")    // 更晚，别的 workspace
+	writeRun("run-c", "claude", "sid-c", "/proj/a", "2026-10-09T14:00:00+08:00") // 更晚，但不是 dsh
+	t.Setenv(dshSessionsEnv, sess)
+
+	if got := dshLatestSessionID("/proj/a"); got != "sid-a" {
+		t.Errorf("dshLatestSessionID(/proj/a) = %q want sid-a（同 workspace 的最近一条）", got)
+	}
+	if got := dshLatestSessionID(""); got != "sid-b" {
+		t.Errorf("dshLatestSessionID(\"\") = %q want sid-b（不限 workspace → 最新 dsh）", got)
+	}
+	if got := dshLatestSessionID("/proj/none"); got != "sid-b" {
+		t.Errorf("dshLatestSessionID(/proj/none) = %q want sid-b（无同 workspace 记录时兜底最新）", got)
 	}
 }
 
@@ -551,6 +702,195 @@ func TestDshConfiguredModel(t *testing.T) {
 	}
 }
 
+// ── dsh 0.1.7+：provider 配置搬到了 profile 的 cordis.patch.yml ──────────
+//
+// 背景（2026-10-09 实测）：dsh 0.1.7 重构 settings 后，首启会把 $DSH_HOME/settings.yaml
+// 改名为 settings.yaml.imported，providers 改由各 profile 的 cordis.patch.yml 承载。
+// magic-agent 若只认 settings.yaml，--engines 的 dsh 模型清单就会整段消失。
+
+// dshPatchFixture profile patch 形态的示例配置（与 dsh 0.1.7 实际写出的形状一致）。
+const dshPatchFixture = `# Your patch layer for this dsh profile, applied after every bundle layer:
+# a top-level YAML array of loader patch entries.
+- id: llm-pi-ai
+  config:
+    providers:
+      modelverse:
+        displayName: Modelverse
+        api: openai-completions
+        baseURL: https://api.modelverse.cn/v1
+        apiKeyEnv: MODELVERSE_API_KEY
+        models:
+          - id: deepseek-v4.1-flash
+            name: DeepSeek V4.1 Flash
+          - id: glm-5.3
+            name: GLM-5.3
+- id: agent-default-model
+  config:
+    provider: modelverse
+    model: deepseek-v4.1-flash
+`
+
+// writeDshPatch 把 profile patch 写进 $DSH_HOME/profiles/<profile>/cordis.patch.yml，
+// 返回 $DSH_HOME（调用方自行 t.Setenv）。
+func writeDshPatch(t *testing.T, profile, body string) string {
+	t.Helper()
+	home := t.TempDir()
+	dir := filepath.Join(home, "profiles", profile)
+	if err := os.MkdirAll(dir, 0o755); err != nil {
+		t.Fatal(err)
+	}
+	if err := os.WriteFile(filepath.Join(dir, "cordis.patch.yml"), []byte(body), 0o644); err != nil {
+		t.Fatal(err)
+	}
+	return home
+}
+
+// 归一化：patch 条目形态 → settings.yaml 顶层键形态，复用同一套解析器。
+func TestDshPatchAsLegacy(t *testing.T) {
+	cases := []struct {
+		name     string
+		patch    string
+		models   []string
+		provider string
+		model    string
+	}{
+		{
+			name:     "标准两个条目",
+			patch:    dshPatchFixture,
+			models:   []string{"modelverse/deepseek-v4.1-flash", "modelverse/glm-5.3"},
+			provider: "modelverse",
+			model:    "deepseek-v4.1-flash",
+		},
+		{
+			name:   "空 patch（[]）",
+			patch:  "# 注释\n[]\n",
+			models: nil,
+		},
+		{
+			name:   "没有 config 的条目跳过",
+			patch:  "- id: hmr\n  disabled: true\n- id: llm-pi-ai\n  config:\n    providers:\n      gw:\n        models:\n          - id: a\n",
+			models: []string{"gw/a"},
+		},
+		{
+			// 回归点：models 里缩进更深的 "- id:" 不能被当成新条目边界，
+			// 否则第一个模型之后的内容会被整段丢掉。
+			name:   "条目内嵌套 - id 不作为边界",
+			patch:  "- id: llm-pi-ai\n  config:\n    providers:\n      gw:\n        models:\n          - id: a\n          - id: b\n",
+			models: []string{"gw/a", "gw/b"},
+		},
+		{
+			// config 之后、缩进回到同层的键（disabled）不属于 config 内容。
+			name:   "config 之后的同层键不搬",
+			patch:  "- id: llm-pi-ai\n  config:\n    providers:\n      gw:\n        models:\n          - id: a\n  disabled: true\n",
+			models: []string{"gw/a"},
+		},
+		{
+			name:     "无 providers 时回落 agent-default-model",
+			patch:    "- id: agent-default-model\n  config:\n    provider: gw\n    model: only\n",
+			models:   []string{"gw/only"},
+			provider: "gw",
+			model:    "only",
+		},
+	}
+	for _, c := range cases {
+		legacy := dshPatchAsLegacy(c.patch)
+		if got := parseDshModelRoutes(legacy); !equalStrings(got, c.models) {
+			t.Errorf("%s: models = %v want %v（归一化文本：\n%s）", c.name, got, c.models, legacy)
+		}
+		if p, m := parseDshDefaultModelRoute(legacy); p != c.provider || m != c.model {
+			t.Errorf("%s: route = (%q,%q) want (%q,%q)", c.name, p, m, c.provider, c.model)
+		}
+	}
+}
+
+// dsh 0.1.7+：从 profile 的 cordis.patch.yml 读出模型清单。
+func TestDshListModelsFromProfilePatch(t *testing.T) {
+	t.Setenv("MAGIC_AGENT_DSH_PROFILE", "")
+	home := writeDshPatch(t, "sdk", dshPatchFixture)
+	t.Setenv("DSH_HOME", home)
+
+	got, err := (&DshEngine{}).ListModels(context.Background())
+	if err != nil {
+		t.Fatalf("ListModels: %v", err)
+	}
+	want := []string{"modelverse/deepseek-v4.1-flash", "modelverse/glm-5.3"}
+	if !equalStrings(got, want) {
+		t.Errorf("ListModels() = %v\n want %v", got, want)
+	}
+}
+
+// 两条通道（sdk + headless 回退）的 patch 取并集：magic-agent 两条都会用到。
+func TestDshListModelsUnionAcrossProfiles(t *testing.T) {
+	t.Setenv("MAGIC_AGENT_DSH_PROFILE", "")
+	home := writeDshPatch(t, "sdk",
+		"- id: llm-pi-ai\n  config:\n    providers:\n      gw1:\n        models:\n          - id: a\n")
+	headlessDir := filepath.Join(home, "profiles", "headless")
+	if err := os.MkdirAll(headlessDir, 0o755); err != nil {
+		t.Fatal(err)
+	}
+	if err := os.WriteFile(filepath.Join(headlessDir, "cordis.patch.yml"),
+		[]byte("- id: llm-pi-ai\n  config:\n    providers:\n      gw2:\n        models:\n          - id: b\n"), 0o644); err != nil {
+		t.Fatal(err)
+	}
+	t.Setenv("DSH_HOME", home)
+
+	got, err := (&DshEngine{}).ListModels(context.Background())
+	if err != nil {
+		t.Fatalf("ListModels: %v", err)
+	}
+	want := []string{"gw1/a", "gw2/b"}
+	if !equalStrings(got, want) {
+		t.Errorf("ListModels() = %v\n want %v", got, want)
+	}
+}
+
+// MAGIC_AGENT_DSH_PROFILE 指定的 profile 也要被读取（不只看默认的 sdk）。
+func TestDshListModelsHonorsProfileEnv(t *testing.T) {
+	prof := "rescue"
+	home := writeDshPatch(t, prof,
+		"- id: llm-pi-ai\n  config:\n    providers:\n      gw:\n        models:\n          - id: a\n")
+	t.Setenv("DSH_HOME", home)
+	t.Setenv(DshProfileEnv, prof)
+
+	got, err := (&DshEngine{}).ListModels(context.Background())
+	if err != nil {
+		t.Fatalf("ListModels: %v", err)
+	}
+	if want := []string{"gw/a"}; !equalStrings(got, want) {
+		t.Errorf("ListModels() = %v want %v", got, want)
+	}
+}
+
+// settings.yaml.imported 是 dsh 迁移留下的**残留**，dsh 已不再读它 ——
+// 采信它会让用户选到运行时拒绝的 route，所以清单里绝不能出现它的模型。
+func TestDshListModelsIgnoresImportedResidue(t *testing.T) {
+	home := t.TempDir()
+	t.Setenv("DSH_HOME", home)
+	legacy := "agent-default-model:\n  provider: modelverse\n  model: deepseek-v4.1-flash\n" +
+		"llm-pi-ai:\n  providers:\n    modelverse:\n      models:\n        - id: deepseek-v4.1-flash\n"
+	if err := os.WriteFile(filepath.Join(home, "settings.yaml.imported"), []byte(legacy), 0o600); err != nil {
+		t.Fatal(err)
+	}
+	_, err := (&DshEngine{}).ListModels(context.Background())
+	if !errors.Is(err, ErrNoModelSource) {
+		t.Fatalf("want ErrNoModelSource（不该采信 .imported）, got %v", err)
+	}
+}
+
+// DshConfiguredModel / DshConfiguredRoute：同样认 profile patch 形态。
+func TestDshConfiguredDefaultFromProfilePatch(t *testing.T) {
+	t.Setenv("MAGIC_AGENT_DSH_PROFILE", "")
+	home := writeDshPatch(t, "sdk", dshPatchFixture)
+	t.Setenv("DSH_HOME", home)
+
+	if got := DshConfiguredModel(); got != "deepseek-v4.1-flash" {
+		t.Errorf("DshConfiguredModel() = %q want deepseek-v4.1-flash", got)
+	}
+	if p, m := DshConfiguredRoute(); p != "modelverse" || m != "deepseek-v4.1-flash" {
+		t.Errorf("DshConfiguredRoute() = (%q,%q) want (modelverse,deepseek-v4.1-flash)", p, m)
+	}
+}
+
 // Response.Model：配置里读不到时回落 "config-default"（标签，不是模型名）。
 func TestDshResponseModelFallback(t *testing.T) {
 	t.Setenv("DSH_HOME", t.TempDir())
@@ -580,10 +920,10 @@ func TestDshHome(t *testing.T) {
 	}
 }
 
-// 默认超时要盖住「dsh 自带工具循环」的长任务（不短于 5 分钟）。
+// 默认超时要盖住「dsh 自带工具循环」的长任务（不短于 1 小时）。
 func TestDefaultDshTimeoutCoversAgentTasks(t *testing.T) {
-	if DefaultDshTimeout < 5*time.Minute {
-		t.Errorf("DefaultDshTimeout = %v，agent 任务常常分钟级，应留足余量", DefaultDshTimeout)
+	if DefaultDshTimeout < time.Hour {
+		t.Errorf("DefaultDshTimeout = %v，agent 任务常常分钟级，应留足余量（≥ 1 小时）", DefaultDshTimeout)
 	}
 	// Runner 按引擎类型取默认超时。
 	r := &Runner{Engine: &DshEngine{}}
@@ -594,14 +934,17 @@ func TestDefaultDshTimeoutCoversAgentTasks(t *testing.T) {
 
 // ── 流式（--stream）────────────────────────────────────────────
 //
-// dsh 的增量在 stderr（"dsh: reasoning:" 起头），正文一次性走 stdout。
-// 实测依据：2026-09-21，dsh 0.1.5-rc.2（见 dsh.go 的 Stream 注释）。
+// 这两个用例覆盖的是**老版 dsh**（headless 没有 --json）：增量在 stderr
+// （"dsh: reasoning:" 起头），正文一次性走 stdout。实测依据：2026-09-21，dsh 0.1.5-rc.2。
+// 假 CLI 因此要在 `--help` 里如实报出「没有 --json」—— 能力探测据此才选老版解析路径
+// （新版的 --json 事件流由 TestDshStreamHeadlessJSONEvents 覆盖）。
 
-// dshStreamStub 造一个假 dsh：推理分批写 stderr，正文最后写 stdout。
+// dshStreamStub 造一个假 dsh：推理分批写 stderr，正文最后写 stdout；--help 报老版形态。
 func dshStreamStub(t *testing.T, dir string) (bin string) {
 	t.Helper()
 	bin = filepath.Join(dir, "dsh")
 	script := `#!/bin/sh
+case "$*" in *--help*) echo 'Usage: dsh --profile headless [task...]'; exit 0;; esac
 echo "dsh: reasoning:" >&2
 echo "第一步：读题" >&2
 sleep 0.05
@@ -684,6 +1027,7 @@ func TestDshStreamIgnoresNonReasoningStderr(t *testing.T) {
 	dir := t.TempDir()
 	bin := filepath.Join(dir, "dsh")
 	script := `#!/bin/sh
+case "$*" in *--help*) echo 'Usage: dsh --profile headless [task...]'; exit 0;; esac
 echo "openviking: 插件已注册" >&2
 echo "dsh: reasoning:" >&2
 echo "真正的推理" >&2
@@ -709,19 +1053,41 @@ echo "正文"
 	}
 }
 
-// 流式路径同样拒续接（与 Complete 同一契约），且不启动进程。
-func TestDshStreamRejectsSession(t *testing.T) {
+// 流式路径同样走回放式续接（与 Complete 同一契约）。
+// 流式续接轮：同样走 headless + --session-id（不是 SDK，也不把历史塞进 prompt）。
+func TestDshStreamResumesViaSessionID(t *testing.T) {
 	dir := t.TempDir()
-	w, log := argsCaptureCLI(t, dir, "dsh", `echo '不该被执行'`)
-	_, err := (&DshEngine{BinPath: w.bin}).Stream(context.Background(), Request{
-		Messages:  []Message{{Role: "user", Content: "hi"}},
+	w, log := argsCaptureCLI(t, dir, "dsh", `echo 'ok'`)
+
+	var got []StreamEvent
+	if _, err := (&DshEngine{BinPath: w.bin}).Stream(context.Background(), Request{
+		Messages:  []Message{{Role: "user", Content: "接着说"}},
 		SessionID: "s1",
-	}, nil)
-	if err == nil || !strings.Contains(err.Error(), "不支持会话续接") {
-		t.Fatalf("流式路径也应拒续接, got %v", err)
+	}, func(ev StreamEvent) { got = append(got, ev) }); err != nil {
+		t.Fatalf("流式续接不该报错: %v", err)
 	}
-	if _, serr := os.Stat(log); serr == nil {
-		t.Error("拒续接时不应启动 dsh 进程")
+	args := readFileString(t, log)
+	for _, want := range []string{"--profile headless", "--session-id s1", "接着说"} {
+		if !strings.Contains(args, want) {
+			t.Errorf("流式续接参数里应有 %q：%q", want, args)
+		}
+	}
+	if strings.Contains(args, "--profile sdk") {
+		t.Errorf("流式续接不该走 SDK 通道：%q", args)
+	}
+	// 收尾事件仍要发（一次调用 = 一轮），且必须带会话锚点 ——
+	// 客户端只从 turn_end / turn_failed 学会话 id（少了它，下一步追问会发成新会话）。
+	var sawEnd bool
+	for _, ev := range got {
+		if ev.Kind == KindTurnEnd {
+			sawEnd = true
+			if ev.SessionID != "s1" {
+				t.Errorf("turn_end 应带会话锚点: got %q want %q", ev.SessionID, "s1")
+			}
+		}
+	}
+	if !sawEnd {
+		t.Errorf("该补一条 turn_end，实得 %v", got)
 	}
 }
 
@@ -741,19 +1107,26 @@ func TestDshStreamNonZeroExit(t *testing.T) {
 	}
 }
 
-// 参数矩阵：MaxTokens / Temperature / Tools / Timeout 在 dsh 上不产生任何 flag。
+// 参数矩阵：MaxTokens / Temperature / Tools / Timeout 在 dsh 上不产生任何 flag
+// （Request 只在 headless 上有 prompt + 可选的 --session-id / --json）。
 func TestDshIgnoredParamsProduceNoFlags(t *testing.T) {
 	t.Setenv(DshProfileEnv, "") // 隔离外部 profile 覆盖
 	e := &DshEngine{}
 	temp := 0.7
-	args := e.buildArgs(Request{
+	req := Request{
 		MaxTokens:   4096,
 		Temperature: &temp,
 		Tools:       ToolsOn,
 		Timeout:     3 * time.Minute,
-	}, "【用户】\nhi")
+	}
+	// 显式读一遍这些字段，证明它们确实没进参数（旧实现里是 `_ = req.X` 那几行）。
+	_ = req.MaxTokens
+	_ = req.Temperature
+	_ = req.Tools
+	_ = req.Timeout
+	args := dshHeadlessArgs(e.profile(), "", "【用户】\nhi", false)
 	want := []string{"--profile", "headless", "【用户】\nhi"}
 	if fmt.Sprint(args) != fmt.Sprint(want) {
-		t.Errorf("buildArgs = %v want %v", args, want)
+		t.Errorf("headless args = %v want %v", args, want)
 	}
 }

package/internal/agent/engine.go

@@ -40,9 +40,12 @@ type Message struct {
 
 // ToolsMode 工具开关模式。
 //
-//	ToolsOff      禁用全部工具（默认）：纯 chat 一次成型，输出可解析。
+//	ToolsOff      禁用全部工具：纯 chat 一次成型，输出可解析。
 //	ToolsOn       引擎默认工具集 + 权限旁路：真 agent 模式，模型可执行
 //	              工具调用（读文件、跑命令等），代价是输出可能是工具痕迹。
+//	              ⚠️ 2026-10-01：默认档从 Off 改成 On —— 用户报障「claude 只会
+//	                闲聊、不读文件、不能用工具」就是因为这个默认。magic-agent
+//	                装在本机就是要当 agent 跑，不是 chat 套壳。
 //	ToolsAllowlist 白名单：仅允许列出的工具 + 权限旁路。
 //
 // 判定方法一律导出：Engine 实现与本包外的调用方（如 CLI 层测试）
@@ -93,7 +96,7 @@ type Request struct {
 	//（claude/codebuddy用 --append-system-prompt，trae 拼进 prompt 头）。
 	SystemPrompt string
 
-	// Tools 工具开关模式（默认 ToolsOff）。
+	// Tools 工具开关模式（默认 ToolsOn —— 2026-10-01 实测用户期望）。
 	Tools ToolsMode
 
 	// Permission 四档权限档位（默认 DefaultPermissionTier = full，保持既有行为）。
@@ -141,6 +144,16 @@ type Request struct {
 	// 忽略（不上报错误）。
 	Temperature *float64
 
+	// Reasoning 推理/思考档位。空串 = 引擎默认；off/none/disabled = 不传参数。
+	// 取值 minimal | low | medium | high | xhigh | max。
+	//
+	// 各引擎映射（只对原生支持的引擎生效，其余静默忽略）：
+	//	claude       --effort <level>（无 minimal，降级为 low）
+	//	codebuddy    --effort <level>（codebuddy-ai 同族同映射）
+	//	codex        -c model_reasoning_effort=<level>（exec 与 exec resume 均可用）
+	//	trae/llm/openclaw/dsh/arkclaw 无对应参数。
+	Reasoning string
+
 	// JSONSchema 是可选的结构化输出约束。
 	//
 	// 各引擎行为：
@@ -426,18 +439,25 @@ func agentEnginesFrom(agents []config.AgentConfig, existing []Engine) []Engine {
 	return out
 }
 
-// toolsOrDefault nil ToolsMode 视为 ToolsOff。
+// toolsOrDefault nil ToolsMode 视为 ToolsOn。
+//
+// ⚠️ 2026-10-01 反转：用户报障「claude 只聊天、不读文件、不能用工具」，
+//
+//	根因就是这里把 nil 兜底成 Off。magic-agent 的定位是本机 agent，不是 chat
+//	套壳，nil（=调用方不显式声明工具）按"未指定 = 想要工具"处理才符合直觉。
+//	想禁用工具的调用方传 `agent.ToolsOff` 即可（API 与 CLI 都已暴露）。
+//
 // 返回接口（而非具体实现）：判定一律走 ToolsMode 的导出方法。
 func toolsOrDefault(t ToolsMode) ToolsMode {
 	if t == nil {
-		return ToolsOff
+		return ToolsOn
 	}
 	return t
 }
 
-// toolsIsOff 是否「禁用工具」模式（nil 视为 off）。
+// toolsIsOff 是否「禁用工具」模式（nil 视为 on，详见 toolsOrDefault 注释）。
 // 各引擎用它决定是否注入 noToolSuffix —— 该后缀明文禁止工具调用，
-// 只能在 off 模式出现，否则会和 on / 白名单模式互相打架。
+// 只能在显式 off 模式出现，否则会和 on / 白名单模式互相打架。
 func toolsIsOff(req Request) bool {
 	return toolsOrDefault(req.Tools).IsOff()
 }