MS Teams: harden messageBack card action forwarding

BradGroux · BradGroux · commit 55a5ad889a41 · 2026-05-08T12:11:50.000-05:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -187,6 +187,7 @@ Docs: https://docs.openclaw.ai
 
 ### Fixes
 
+- MS Teams: forward messageBack card action payloads to the agent instead of dropping empty-text card submit activities. (#64503) Thanks @ndholakia.
 - Cron/agents: recognize same-target `edit`↔`write` recovery in `isSameToolMutationAction`, so a successful `write` to a path clears an earlier failed `edit` on the same path. Stops cron from reporting fatal failures when an agent self-heals across `edit` and `write`, while preserving same-tool fingerprint matching, blocking different-target writes, and excluding tools (including `apply_patch`) whose real call args do not produce a stable `path` fingerprint segment. Fixes #79024. Thanks @RenzoMXD.
 - Gateway/Tailscale: add opt-in `gateway.tailscale.preserveFunnel` so when `tailscale.mode = "serve"` and an externally configured Tailscale Funnel route already covers the gateway port, OpenClaw skips re-applying `tailscale serve` on startup and skips the `resetOnExit` teardown for that run, keeping operator-managed Funnel exposure alive across gateway restarts. Fixes #57241. Thanks @RenzoMXD.
 - Agents/compaction: keep the recent tail after manual `/compact` when Pi returns an empty or no-op compaction summary, preventing blank checkpoints from replacing the live context.
diff --git a/extensions/msteams/src/monitor-handler/message-handler.ts b/extensions/msteams/src/monitor-handler/message-handler.ts
@@ -204,11 +204,7 @@ export function createMSTeamsMessageHandler(deps: MSTeamsMessageHandlerDeps) {
       maxInlineBytes: mediaMaxBytes,
       maxInlineTotalBytes: mediaMaxBytes,
     });
-    // If text and attachments are both empty but activity.value is present,
-    // this is a messageBack card action (Action.Submit). Serialize the value
-    // payload so it reaches the agent instead of being dropped. (#60952)
-    // Accept any truthy value — object, string, or primitive — since
-    // messageBack payloads are not guaranteed to be objects.
+    // Forward messageBack card action payloads instead of dropping empty text.
     let rawBody = text || attachmentPlaceholder;
     if (!rawBody && activity.value != null) {
       try {
@@ -221,7 +217,7 @@ export function createMSTeamsMessageHandler(deps: MSTeamsMessageHandlerDeps) {
           });
         }
       } catch {
-        log.warn("failed to serialize messageBack value payload");
+        log.warn?.("failed to serialize messageBack value payload");
       }
     }
     const quoteInfo = extractMSTeamsQuoteInfo(attachments);
