fix(cron): preserve model override for text payloads

vincentkoc · vincentkoc · commit a75513ec6b80 · 2026-04-28T07:33:05.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,7 @@ Docs: https://docs.openclaw.ai
 
 ### Fixes
 
+- Cron: preserve model overrides when text cron payloads are promoted to isolated agent turns through nested payload and legacy top-level inputs. Carries forward #64060; refs #28905. Thanks @liaoandi.
 - Control UI/WebChat: keep large attachment payloads out of Lit state and optimistic chat messages, using object URL previews plus send-time payload serialization so PDF/image uploads no longer trigger `RangeError: Maximum call stack size exceeded`. Fixes #73360; refs #54378 and #63432. Thanks @hejunhui-73, @Ansub, and @christianhernandez3-afk.
 - Agents/models: keep per-agent primary models strict when `fallbacks` is omitted, so probe-only custom providers are not tried as hidden fallback candidates unless the agent explicitly opts in. Fixes #73332. Thanks @haumanto.
 - Cron/Telegram: preserve explicit `:topic:` delivery targets over stale session-derived thread IDs when isolated cron announces to Telegram forum topics. Carries forward #59069; refs #49704 and #43808. Thanks @roytong9.
diff --git a/src/cron/normalize.test.ts b/src/cron/normalize.test.ts
@@ -616,6 +616,7 @@ describe("normalizeCronJobCreate", () => {
     expect(payload.message).toBe("Run daily sync");
     expect(payload.model).toBe("anthropic/claude-sonnet-4-6");
     expect(payload).not.toHaveProperty("text");
+    expect(validateCronAddParams(normalized)).toBe(true);
   });
 
   it("promotes top-level text to agentTurn on create when model is present", () => {
@@ -631,6 +632,7 @@ describe("normalizeCronJobCreate", () => {
     expect(payload.message).toBe("Run daily sync");
     expect(payload.model).toBe("anthropic/claude-sonnet-4-6");
     expect(payload).not.toHaveProperty("text");
+    expect(validateCronAddParams(normalized)).toBe(true);
   });
 
   it("resolves current sessionTarget to a persistent session when context is available", () => {
diff --git a/src/cron/normalize.ts b/src/cron/normalize.ts
@@ -311,20 +311,26 @@ function coerceDelivery(delivery: UnknownRecord) {
 function inferTopLevelPayload(next: UnknownRecord) {
   const message = normalizeOptionalString(next.message) ?? "";
   if (message) {
-    return { kind: "agentTurn", message } satisfies UnknownRecord;
+    const payload = { kind: "agentTurn", message } satisfies UnknownRecord;
+    copyTopLevelAgentTurnFields(next, payload);
+    return payload;
   }
 
   const text = normalizeOptionalString(next.text) ?? "";
   if (text && hasAgentTurnPayloadHint(next)) {
     // text + agentTurn-only fields (e.g. model) -> promote to agentTurn, text -> message.
-    return { kind: "agentTurn", message: text } satisfies UnknownRecord;
+    const payload = { kind: "agentTurn", message: text } satisfies UnknownRecord;
+    copyTopLevelAgentTurnFields(next, payload);
+    return payload;
   }
   if (text) {
     return { kind: "systemEvent", text } satisfies UnknownRecord;
   }
 
   if (hasAgentTurnPayloadHint(next)) {
-    return { kind: "agentTurn" } satisfies UnknownRecord;
+    const payload = { kind: "agentTurn" } satisfies UnknownRecord;
+    copyTopLevelAgentTurnFields(next, payload);
+    return payload;
   }
 
   return null;
