magic-agent 1.0.3 → 1.0.49

package/internal/agent/streamjson.go

@@ -15,6 +15,7 @@ package agent
 import (
 	"context"
 	"encoding/json"
+	"errors"
 	"fmt"
 	"io"
 	"strings"
@@ -33,15 +34,19 @@ type streamJSONResult struct {
 // runStreamJSONIn 跑一次 stream-json 调用：增量喂给 acc，收尾行填进 fin。
 // 返回是否见到了 result 行（未见 = 协议没跑完，调用方按错误处理）。
 //
+// stderrOut 非空时把 CLI 的 stderr 尾巴收进它 —— 调用方在「没见到 result 行」时
+// 要拿它拼错误（见 streamNoResultErr）。
+//
 // stdin 为 nil 时不喂 stdin（提示词走命令行位置参数）。每见到一次 result 行，
 // 除填 fin 外还会向调用方发一个 KindTurnEnd 事件 —— 常驻会话（Request.Append）
 // 靠它判断「这轮做完了，可以继续追加」。
 func runStreamJSONIn(
 	ctx context.Context, dir string, extraEnv []string, bin string, args []string,
 	stdin io.Reader, acc *streamAccumulator, fin *streamJSONResult,
+	stderrOut *streamStderrBuf,
 ) (bool, error) {
 	seen := false
-	err := runStreamCLIStdinReaderIn(ctx, dir, extraEnv, bin, args, stdin, func(line string) error {
+	err := runStreamCLIStdinReaderIn(ctx, dir, extraEnv, bin, args, stdin, stderrOut, func(line string) error {
 		isResult, perr := acc.handleNDJSONLine(line)
 		if perr != nil {
 			return perr
@@ -49,6 +54,20 @@ func runStreamJSONIn(
 		if isResult {
 			_ = json.Unmarshal([]byte(line), fin)
 			seen = true
+			/* ⚠️ **先判「正文是不是供应商的通告」再发轮次边界**（2026-10-10，见 limitnotice.go）：
+			   发 turn_end 等于告诉消费方「这一轮成功收尾」—— 先发出去，界面上就成了
+			   「正文 + 额度通告 + 已完成」的假成功（用户报障原话：「codebuddy ai 没有额度了
+			   没有输出一个独立的错误消息用于提示吗 现在混在回复里了」）。
+			   命中时改发 turn_failed（原因 = 通告原文，Text = 剥掉通告的正文），并把错误
+			   抛给调用方 —— 与「没吐 result 行」走**同一个收口**（CLI 据此写失败信封、
+			   非零退出；消费方不必自己判）。
+			   ⚠️ 抛错会让 lineHandler 中止读循环，那里会顺手 kill 掉引擎进程组
+			   （见 runStreamCLIStdinReaderIn），所以这里不能改成「先把事件转发出去、
+			   等收尾再抛」—— 那样 turn_failed 会晚到，消费方早就按 turn_end 收尾了。 */
+			if notice, body, hit := splitLimitNotice(fin.Result); hit {
+				acc.emitLimitNoticeFailed(fin.SessionID, notice, body)
+				return errors.New(notice)
+			}
 			// 轮次边界事件（Text = 该轮最终正文，SessionID = 该轮会话 id）。
 			// ⚠️ 它必须带 session_id：常驻会话的最终 result 信封要等会话结束才输出
 			//（默认空闲 5m），调用方要靠这一条把会话锚点绑上（见 StreamEvent.SessionID）。
@@ -141,7 +160,7 @@ func hasImageAttachment(atts []Attachment) bool {
 // setArgValue 把 args 里 <flag> 的下一个值改成 want（找不到就追加 <flag> want）。
 // claude/codebuddy 的 --output-format json → stream-json 就靠它。
 func setArgValue(args []string, flag, want string) []string {
-	for i := 0; i < len(args)-1; i++ {
+	for i := 0; len(args) > 0 && i < len(args)-1; i++ {
 		if args[i] == flag {
 			args[i+1] = want
 			return args
@@ -183,3 +202,18 @@ func streamJSONArgs(baseArgs []string, prompt string, streaming bool) []string {
 func errEmptyStreamContent(label string) error {
 	return fmt.Errorf("%s CLI returned empty result", label)
 }
+
+// streamNoResultErr 「协议没跑完」的错误：把 CLI 自己的 stderr 尾巴带上。
+//
+// 为什么必须带（2026-10-09 加）：这条错误意味着 CLI **没走完协议就退出了**
+// —— 自己崩了 / 被上游拒了 / 鉴权或额度失败，而真正的原因几乎只出现在它的
+// stderr 里。不带的话用户和排障者永远只看到一句
+// 「stream ended without result line」，无从下手（用户报障截图就是这个）。
+//
+// stderr 为空（典型：被魔改的 CLI 什么都不打）时退回原句，不拼空摘要。
+func streamNoResultErr(label string, stderr string) error {
+	if s := stderrSummary(stderr); s != "" {
+		return fmt.Errorf("%s CLI stream ended without result line；CLI 最后输出：%s", label, s)
+	}
+	return fmt.Errorf("%s CLI stream ended without result line", label)
+}

package/internal/agent/toolkind.go

@@ -31,7 +31,20 @@ const (
 	ToolKindInputRequest = "input_request"
 	// ToolKindPermission 请求授权（权限确认那一类，会渲染成审批卡）。
 	ToolKindPermission = "permission"
-	// ToolKindToolCall 其余**已知**工具（读文件 / 列目录 / 待办…）。
+	// ToolKindTodo 待办清单（Claude 系 TodoWrite / Codex 系 update_plan 一类）。
+	//
+	// ⚠️ 这一档**不在**本文件上面那份「与 anywhere 的 ToolContentKind 对齐」的词表里
+	//（command / file_change / web_search / mcp / agent_call / input_request /
+	// permission / tool_call / unknown），是**本仓库补的**：anywhere 把待办画成 timeline 的
+	// `todo-row`，并没有一个同名的 ToolContentKind。
+	// 补它的理由：待办清单跨引擎高频出现，而且是唯一「形态判据比名字更可靠」的一类 ——
+	// 清单永远装在参数里（`todos` / `plan`），各家工具名却各不相同。所以两侧分工固定：
+	// **这里按名字判定**，名字表没见过的引擎由前端按参数兜底（见 29-msg-variants 的 qaToolKindOf）。
+	// ⚠️ 加这一档要同时改三处，缺一就是「服务端给了、前端不认」：
+	//    ① 这里（含下面的名字表）；② toolkind_test.go 的契约表；
+	//    ③ 前端 29-msg-variants.js 的 qaToolKindMerge 映射表。
+	ToolKindTodo = "todo"
+	// ToolKindToolCall 其余**已知**工具（读文件 / 列目录…）。
 	//
 	// ⚠️ 认不出来也归这里，**不归 unknown** —— 与 anywhere 的 claude projector 一致
 	//（未知工具名 → 通用 ToolCallContent）：它是「有一次工具调用但形态未细分」，
@@ -64,6 +77,12 @@ var toolKindByName = map[string]string{
 	// 子代理
 	"task": ToolKindAgentCall, "agent": ToolKindAgentCall, "subagent": ToolKindAgentCall,
 	"delegate": ToolKindAgentCall,
+	// 待办清单（Claude 系 TodoWrite，Codex 系 update_plan，以及各家的等价名）。
+	// ⚠️ 本函数**只看名字**（见下方 args 参数说明）—— 参数里的 todos / plan 由前端
+	// qaToolKindOf 兜底，两侧词表必须一致，别只改一边。
+	"todowrite": ToolKindTodo, "todo_write": ToolKindTodo, "todo__write": ToolKindTodo,
+	"update_plan": ToolKindTodo, "plan_write": ToolKindTodo,
+	"write_todos": ToolKindTodo, "writetodos": ToolKindTodo,
 	// 提问 / 要输入
 	"askuserquestion": ToolKindInputRequest, "ask_user_question": ToolKindInputRequest,
 	"elicit": ToolKindInputRequest, "input_request": ToolKindInputRequest,

package/internal/agent/toolkind_test.go

@@ -41,11 +41,17 @@ func TestToolKindFor(t *testing.T) {
 		{"builtin.Bash", ToolKindCommand},
 		{"tools.Edit", ToolKindFileChange},
 		{"acp.Task", ToolKindAgentCall},
+		// 待办清单（TodoWrite / update_plan → 本仓库补的一档，见 toolkind.go 的 ToolKindTodo）
+		{"TodoWrite", ToolKindTodo},
+		{"todo_write", ToolKindTodo},
+		{"update_plan", ToolKindTodo},
+		{"write_todos", ToolKindTodo},
+		// 命名空间形状的待办工具：剥前缀后同样要查得出来（引擎给的可能是 openclaw.TodoWrite）
+		{"builtin.TodoWrite", ToolKindTodo},
 		// 已知但未细分的工具 → tool_call（对齐 anywhere 的通用 ToolCallContent）
 		{"Read", ToolKindToolCall},
 		{"Glob", ToolKindToolCall},
 		{"Grep", ToolKindToolCall},
-		{"TodoWrite", ToolKindToolCall},
 		{"exit_plan_mode", ToolKindToolCall},
 		// 从没见过的名字也归 tool_call，**不是** unknown
 		{"some_brand_new_tool", ToolKindToolCall},

package/internal/agent/tools_suffix_test.go

@@ -220,10 +220,14 @@ func hasArgToken(args, want string) bool {
 	return false
 }
 
-// nil ToolsMode 必须按 off 处理（默认关工具，向后兼容）。
+// nil ToolsMode 必须按 on 处理（2026-10-01 反转默认）—— magic-agent 的定位是
+// 本机 agent，调用方不显式声明工具 = 想要工具；想关显式传 ToolsOff。
 func TestToolsIsOffDefaultsToOff(t *testing.T) {
-	if !toolsIsOff(Request{}) {
-		t.Error("空 Tools 应视为 off")
+	/* 旧断言是「空 Tools 应视为 off」—— 那是早期 magic-agent 当 chat 套壳的
+	   口径。2026-10-01 用户报障「claude 只闲聊不读文件」就是这个默认兜底。
+	   本测试改名 + 反转断言，钉住新默认。 */
+	if toolsIsOff(Request{}) {
+		t.Error("空 Tools 在 2026-10-01 反转后视为 on（magic-agent 默认开工具）")
 	}
 	if toolsIsOff(Request{Tools: ToolsOn}) {
 		t.Error("ToolsOn 不应判定为 off")
@@ -231,6 +235,9 @@ func TestToolsIsOffDefaultsToOff(t *testing.T) {
 	if toolsIsOff(Request{Tools: ToolsAllowlist([]string{"Read"})}) {
 		t.Error("白名单不应判定为 off")
 	}
+	if !toolsIsOff(Request{Tools: ToolsOff}) {
+		t.Error("显式 ToolsOff 仍应判定为 off（兜底反转不影响显式传值）")
+	}
 }
 
 // noToolSuffix 必须显式禁止「伪造工具返回」。

package/internal/agent/trae.go

@@ -45,8 +45,8 @@ type TraeEngine struct {
 // Name 实现 Engine。
 func (e *TraeEngine) Name() string { return "trae" }
 
-// DefaultTraeTimeout 单次尝试默认超时。
-const DefaultTraeTimeout = 10 * time.Minute
+// DefaultTraeTimeout 单次尝试默认超时（2026-10-10 由 10 分钟统一上调到 1 小时）。
+const DefaultTraeTimeout = time.Hour
 
 // traeOffDisableTools off 模式下逐个 --disallowed-tool 禁用的实体工具。
 //
@@ -302,7 +302,9 @@ func (e *TraeEngine) Stream(ctx context.Context, req Request, onEvent func(Strea
 		return StreamResult{}, err
 	}
 	if !seenResult {
-		return StreamResult{}, fmt.Errorf("trae CLI stream ended without result line")
+		err := fmt.Errorf("trae CLI stream ended without result line")
+		acc.emitTurnFailed(acc.SessionID, err)
+		return StreamResult{}, err
 	}
 	if fin.IsError {
 		return StreamResult{}, fmt.Errorf("trae CLI error (subtype=%s): %s", fin.Subtype, truncateStr(fin.Result, 500))

package/internal/agent/workspace_test.go

@@ -164,3 +164,55 @@ func TestCodexOmitsCDWithoutWorkspace(t *testing.T) {
 		t.Errorf("未指定 workspace 时不该有 -C：%s", args)
 	}
 }
+
+// codexPwdStub 造一个假 codex：既把 pwd 与收到的参数写进 log，又回放 codex 形状的 NDJSON
+// （pwdStub 那份回的是 claude 的 result 信封，codex 的 parseStdout 认不出来）。
+func codexPwdStub(t *testing.T, dir string) (bin, log string) {
+	t.Helper()
+	log = filepath.Join(dir, "ws.log")
+	bin = filepath.Join(dir, "codex")
+	script := "#!/bin/sh\n" +
+		"{ pwd; printf 'args: %s\\n' \"$*\"; } > " + log + "\n" +
+		`printf '%s\n' '{"type":"thread.started","thread_id":"th-1"}'` + "\n" +
+		`printf '%s\n' '{"type":"item.completed","item":{"type":"agent_message","text":"ok"}}'` + "\n"
+	if err := os.WriteFile(bin, []byte(script), 0o755); err != nil {
+		t.Fatal(err)
+	}
+	return bin, log
+}
+
+// 续接轮（`exec resume`）**不能带 `-C`** —— 2026-10-18 线上事故。
+//
+// codex 0.162.0 的 resume 子命令参数表里没有 `-C`（也没有 `-m`）：传了 clap 直接
+// `error: unexpected argument '-C' found` + exit 2、stdout 全空，core 侧只能报
+// 「codex CLI returned empty result · 总耗时 0.0s」（用户手机上那条「继续」就是这么死的）。
+// 工作目录改由**子进程 cwd** 承担（本用例同时钉住这点），sandbox 根延用首次会话。
+func TestCodexResumeOmitsCDButKeepsCwd(t *testing.T) {
+	dir := t.TempDir()
+	ws := filepath.Join(dir, "proj")
+	if err := os.MkdirAll(ws, 0o755); err != nil {
+		t.Fatal(err)
+	}
+	bin, log := codexPwdStub(t, dir)
+	e := &CodexEngine{BinPath: bin}
+	if _, err := e.Complete(context.Background(), Request{
+		SessionID: "th-1",
+		Workspace: ws,
+		Messages:  []Message{{Role: "user", Content: "继续"}},
+	}); err != nil {
+		t.Fatalf("Complete: %v", err)
+	}
+	data, _ := readFile(log)
+	out := string(data)
+	if strings.Contains(out, "-C ") {
+		t.Errorf("续接轮不该有 -C（codex exec resume 不认它）：%s", out)
+	}
+	if !strings.Contains(out, "exec resume th-1") {
+		t.Errorf("应走 exec resume 子命令：%s", out)
+	}
+	// cwd 仍要落在 workspace —— 这是续接轮唯一的目录表达方式。
+	first := strings.SplitN(out, "\n", 2)[0]
+	if !samePath(t, strings.TrimSpace(first), ws) {
+		t.Errorf("续接轮子进程 cwd = %q，期望 %q", strings.TrimSpace(first), ws)
+	}
+}