fix(cron): apply responsePrefix to direct delivery payloads

ethanperezz · claude · ethanperezz · commit 0aed3a78100c · 2026-03-04T23:49:33.000-05:00
Cron direct delivery (delivery.mode: "announce" with direct path) was sending raw response text without applying the configured responsePrefix. Interactive replies correctly resolve the prefix via resolveEffectiveMessagesConfig, but the cron deliverViaDirect path skipped this step entirely. Resolve responsePrefix for the delivery channel/account and prepend it to payload text, matching the behavior of route-reply and heartbeat delivery paths. Fixes #29600 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
diff --git a/src/cron/isolated-agent/delivery-dispatch.ts b/src/cron/isolated-agent/delivery-dispatch.ts
@@ -1,3 +1,4 @@
+import { resolveEffectiveMessagesConfig } from "../../agents/identity.js";
 import { runSubagentAnnounceFlow } from "../../agents/subagent-announce.js";
 import { countActiveDescendantRuns } from "../../agents/subagent-registry.js";
 import { SILENT_REPLY_TOKEN } from "../../auto-reply/tokens.js";
@@ -193,13 +194,28 @@ export async function dispatchCronDelivery(
     delivery: SuccessfulDeliveryTarget,
   ): Promise<RunCronAgentTurnResult | null> => {
     const identity = resolveAgentOutboundIdentity(params.cfgWithAgentDefaults, params.agentId);
+    // Apply responsePrefix to direct delivery payloads (same logic as interactive replies).
+    const { responsePrefix } = resolveEffectiveMessagesConfig(
+      params.cfgWithAgentDefaults,
+      params.agentId,
+      { channel: delivery.channel, accountId: delivery.accountId },
+    );
+    const applyPrefix = (text: string | undefined): string | undefined => {
+      if (!responsePrefix || !text || text.startsWith(responsePrefix)) {
+        return text;
+      }
+      return `${responsePrefix} ${text}`;
+    };
     try {
-      const payloadsForDelivery =
+      const rawPayloads =
         deliveryPayloads.length > 0
           ? deliveryPayloads
           : synthesizedText
             ? [{ text: synthesizedText }]
             : [];
+      const payloadsForDelivery = responsePrefix
+        ? rawPayloads.map((p) => (p.text ? { ...p, text: applyPrefix(p.text) } : p))
+        : rawPayloads;
       if (payloadsForDelivery.length === 0) {
         return null;
       }
