fix(feishu): support text-tag fallback card parsing

vincentkoc · vincentkoc · commit bf6f95446e93 · 2026-04-28T18:25:53.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -35,6 +35,7 @@ Docs: https://docs.openclaw.ai
 - Channels/Telegram: persist native command metadata on target sessions so topic, helper, and ACP-bound slash commands keep their session metadata attached to the routed conversation. (#57548) Thanks @GaosCode.
 - Channels/native commands: keep validated native slash command replies visible in group chats while preserving explicit owner allowlists for command authorization. (#73672) Thanks @obviyus.
 - Auto-reply/session: carry the tail of user/assistant turns into the freshly-rotated transcript on silent in-reply session resets (compaction failure, role-ordering conflict) so direct-chat continuity survives the rebind. Fixes #70853. (#70898) Thanks @neeravmakwana.
+- Feishu: extract text-tag and nested-array text from replied interactive cards, and fall back to direct post-format card content when element arrays are empty. Fixes #60380; supersedes #60383; carries forward #73203. Thanks @taozengabc and @lskun.
 
 ## 2026.4.27
 
diff --git a/extensions/feishu/src/send.test.ts b/extensions/feishu/src/send.test.ts
@@ -180,8 +180,14 @@ describe("getMessageFeishu", () => {
             body: {
               content: JSON.stringify({
                 elements: [
-                  [{ tag: "text", text: "Service Name" }, { tag: "text", text: "eip-admin-api" }],
-                  [{ tag: "text", text: "Alert" }, { tag: "text", text: "Health check failed 3 times" }],
+                  [
+                    { tag: "text", text: "Service Name" },
+                    { tag: "text", text: "eip-admin-api" },
+                  ],
+                  [
+                    { tag: "text", text: "Alert" },
+                    { tag: "text", text: "Health check failed 3 times" },
+                  ],
                 ],
               }),
             },
@@ -331,6 +337,42 @@ describe("getMessageFeishu", () => {
     );
   });
 
+  it("falls back to direct post-format content when interactive card elements are empty", async () => {
+    mockClientGet.mockResolvedValueOnce({
+      code: 0,
+      data: {
+        items: [
+          {
+            message_id: "om_direct_post_card",
+            chat_id: "oc_direct_post_card",
+            msg_type: "interactive",
+            body: {
+              content: JSON.stringify({
+                elements: [],
+                title: "Direct card summary",
+                content: [[{ tag: "text", text: "fallback body" }]],
+              }),
+            },
+          },
+        ],
+      },
+    });
+
+    const result = await getMessageFeishu({
+      cfg: {} as ClawdbotConfig,
+      messageId: "om_direct_post_card",
+    });
+
+    expect(result).toEqual(
+      expect.objectContaining({
+        messageId: "om_direct_post_card",
+        chatId: "oc_direct_post_card",
+        contentType: "interactive",
+        content: "Direct card summary\n\nfallback body",
+      }),
+    );
+  });
+
   it("extracts text content from post messages", async () => {
     mockClientGet.mockResolvedValueOnce({
       code: 0,
diff --git a/extensions/feishu/src/send.ts b/extensions/feishu/src/send.ts
@@ -252,23 +252,25 @@ function extractInteractiveElementsText(
   variables: Map<string, string>,
 ): string {
   const texts: string[] = [];
-  for (const element of elements) {
-    // Some cards use nested arrays (e.g. [[row1_col1, row1_col2], [row2_col1, ...]])
-    // for table-like layouts. Flatten one level and extract text from each cell.
+
+  function collectText(element: unknown): void {
     if (Array.isArray(element)) {
       for (const cell of element) {
-        const text = extractInteractiveElementText(cell, variables);
-        if (text !== undefined) {
-          texts.push(text);
-        }
+        collectText(cell);
       }
-      continue;
+      return;
     }
+
     const text = extractInteractiveElementText(element, variables);
     if (text !== undefined) {
       texts.push(text);
     }
   }
+
+  for (const element of elements) {
+    collectText(element);
+  }
+
   return texts.join("\n").trim();
 }
 
