fix(agent-runner): deliver assistant text as fallback when model skips message tool

billehunt · billehunt · commit 4c5aaf767f8b · 2026-05-19T21:28:28.000Z
When sourceReplyDeliveryMode is message_tool_only (default for Telegram
group chats), model responses were silently dropped if the model generated
text without calling the message tool. Mark those payloads for fallback
delivery so the user always sees the response.
diff --git a/src/auto-reply/reply/agent-runner.ts b/src/auto-reply/reply/agent-runner.ts
@@ -1903,6 +1903,31 @@ export async function runReplyAgent(params: {
       });
     }
 
+    // When sourceReplyDeliveryMode is "message_tool_only" the dispatch layer
+    // silently drops payloads that lack deliverDespiteSourceReplySuppression.
+    // If the model produced visible text but never called the message tool (and
+    // no block-streaming or other side-effect delivery occurred), mark the
+    // payloads for fallback delivery so the user still sees the response.
+    if (
+      opts?.sourceReplyDeliveryMode === "message_tool_only" &&
+      guardedReplyPayloads.length > 0 &&
+      !hasSuccessfulSideEffectDelivery({
+        blockReplyPipeline,
+        directlySentBlockKeys,
+        messagingToolSentTexts: runResult.messagingToolSentTexts,
+        messagingToolSentMediaUrls: runResult.messagingToolSentMediaUrls,
+        messagingToolSentTargets: runResult.messagingToolSentTargets,
+        successfulCronAdds: runResult.successfulCronAdds,
+        didSendDeterministicApprovalPrompt: runResult.didSendDeterministicApprovalPrompt,
+      })
+    ) {
+      for (let i = 0; i < guardedReplyPayloads.length; i++) {
+        guardedReplyPayloads[i] = markReplyPayloadForSourceSuppressionDelivery(
+          guardedReplyPayloads[i],
+        );
+      }
+    }
+
     // Prepend verbose operational notices. Model fallback notices are prepared
     // earlier so they pass through normal reply threading and stream-dedupe.
     let finalPayloads = guardedReplyPayloads;
