From 6da52aa60f8b19bbe75b3bafff678a3cfcaeae45 Mon Sep 17 00:00:00 2001 From: Snider Date: Sat, 21 Mar 2026 17:46:04 +0000 Subject: [PATCH] =?UTF-8?q?fix:=20Codex=20round=208=20=E2=80=94=20remote?= =?UTF-8?q?=20success=20propagation=20+=20stale=20question=20cleanup?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Medium: remote dispatch now propagates inner dispatchOut.Success instead of hardcoding true. Low: updateStatus clears stale question field when new state has no question, preventing leftover rejection messages. Co-Authored-By: Virgil --- pkg/agentic/remote.go | 1 + pkg/monitor/harvest.go | 2 ++ 2 files changed, 3 insertions(+) diff --git a/pkg/agentic/remote.go b/pkg/agentic/remote.go index 8126476..9234bc3 100644 --- a/pkg/agentic/remote.go +++ b/pkg/agentic/remote.go @@ -143,6 +143,7 @@ func (s *PrepSubsystem) dispatchRemote(ctx context.Context, _ *mcp.CallToolReque } else if len(rpcResp.Result.Content) > 0 { var dispatchOut DispatchOutput if json.Unmarshal([]byte(rpcResp.Result.Content[0].Text), &dispatchOut) == nil { + output.Success = dispatchOut.Success output.WorkspaceDir = dispatchOut.WorkspaceDir output.PID = dispatchOut.PID output.Agent = dispatchOut.Agent diff --git a/pkg/monitor/harvest.go b/pkg/monitor/harvest.go index df04e8b..dc01562 100644 --- a/pkg/monitor/harvest.go +++ b/pkg/monitor/harvest.go @@ -280,6 +280,8 @@ func updateStatus(wsDir, status, question string) { st["status"] = status if question != "" { st["question"] = question + } else { + delete(st, "question") // clear stale question from previous state } updated, _ := json.MarshalIndent(st, "", " ") coreio.Local.Write(filepath.Join(wsDir, "status.json"), string(updated))