@tobeyoureyes/feishu 1.1.0 → 1.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tobeyoureyes/feishu",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "OpenClaw Feishu (Lark) channel plugin",
   "type": "module",
   "files": [
@@ -8,17 +8,11 @@
     "src",
     "openclaw.plugin.json"
   ],
-  "devDependencies": {
-    "@larksuiteoapi/node-sdk": "^1.58.0",
-    "openclaw": "workspace:*"
-  },
-  "peerDependencies": {
+  "dependencies": {
     "@larksuiteoapi/node-sdk": "^1.58.0"
   },
-  "peerDependenciesMeta": {
-    "@larksuiteoapi/node-sdk": {
-      "optional": true
-    }
+  "devDependencies": {
+    "openclaw": "workspace:*"
   },
   "openclaw": {
     "extensions": [

package/src/channel.ts

@@ -37,6 +37,11 @@ import { createGroupHistoryManager, type HistoryEntry } from "./history.js";
 import { buildFeishuMessageContext } from "./context.js";
 import { dispatchFeishuMessage, createDefaultReplySender, createDefaultMediaSender } from "./dispatch.js";
 
+// Node.js imports for media handling
+import { writeFile, mkdir } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+
 // ============ Helper Functions ============
 
 /**
@@ -68,6 +73,66 @@ async function fetchReplyContext(
   return inbound;
 }
 
+/**
+ * Save media data to a temporary file
+ */
+async function saveTempMedia(
+  data: ArrayBuffer,
+  key: string,
+  fileName?: string,
+  defaultExt: string = "bin",
+): Promise<string> {
+  const dir = join(tmpdir(), "openclaw-feishu-media");
+  await mkdir(dir, { recursive: true });
+  const ext = fileName?.split(".").pop() ?? defaultExt;
+  const path = join(dir, `${key}.${ext}`);
+  await writeFile(path, Buffer.from(data));
+  return path;
+}
+
+/**
+ * Download media from Feishu message (image/file/audio/video)
+ */
+async function downloadMessageMedia(
+  account: ResolvedFeishuAccount,
+  message: FeishuInboundMessage,
+  log?: { warn?: (msg: string) => void },
+): Promise<FeishuInboundMessage> {
+  // Image message
+  if (message.imageKey) {
+    try {
+      const result = await api.downloadImage(account, message.imageKey);
+      if (result.ok && result.data) {
+        const tempPath = await saveTempMedia(result.data, message.imageKey, undefined, "png");
+        return { ...message, mediaPath: tempPath, mediaType: "image/png" };
+      }
+      log?.warn?.(`failed to download image ${message.imageKey}: ${result.error ?? "unknown error"}`);
+    } catch (err) {
+      log?.warn?.(`failed to download image ${message.imageKey}: ${String(err)}`);
+    }
+  }
+
+  // File/audio/video message
+  if (message.fileKey) {
+    try {
+      const result = await api.downloadFile(account, message.fileKey);
+      if (result.ok && result.data) {
+        const ext = message.messageType === "audio" ? "ogg" : message.messageType === "media" ? "mp4" : undefined;
+        const tempPath = await saveTempMedia(result.data, message.fileKey, message.fileName, ext);
+        const mimeType = message.messageType === "audio" ? "audio/ogg"
+          : message.messageType === "media" ? "video/mp4"
+          : "application/octet-stream";
+        return { ...message, mediaPath: tempPath, mediaType: mimeType };
+      }
+      log?.warn?.(`failed to download file ${message.fileKey}: ${result.error ?? "unknown error"}`);
+    } catch (err) {
+      log?.warn?.(`failed to download file ${message.fileKey}: ${String(err)}`);
+    }
+  }
+
+  return message;
+}
+
 // Channel metadata
 const meta: ChannelMeta = {
   id: "feishu",
@@ -748,6 +813,9 @@ export const feishuPlugin: ChannelPlugin<ResolvedFeishuAccount> = {
         // Fetch reply context if this is a reply
         const messageWithContext = await fetchReplyContext(account, inbound, ctx.log);
 
+        // Download media (image/file/audio/video) if present
+        const messageWithMedia = await downloadMessageMedia(account, messageWithContext, ctx.log);
+
         // Prepare group history
         const isGroup = inbound.chatType === "group";
         const historyEntry: HistoryEntry | undefined = isGroup
@@ -765,7 +833,7 @@ export const feishuPlugin: ChannelPlugin<ResolvedFeishuAccount> = {
         try {
           // Build message context using local module
           const context = await buildFeishuMessageContext({
-            message: messageWithContext,
+            message: messageWithMedia,
             account,
             cfg: ctx.cfg,
             botOpenId,

package/src/webhook.ts

@@ -341,6 +341,10 @@ function handleBotRemoved(event: FeishuBotRemovedEvent): FeishuWebhookResult {
 
 /**
  * Extract plain text from post content
+ * Supports multiple formats:
+ * - { zh_cn: { title, content } } - with language tag
+ * - { title, content } - direct format (common in received messages)
+ * - { post: { zh_cn: { ... } } } - nested post field
  */
 function extractTextFromPost(content: unknown): string {
   const texts: string[] = [];
@@ -351,30 +355,58 @@ function extractTextFromPost(content: unknown): string {
         continue;
       }
 
-      const el = element as { tag?: string; text?: string; content?: unknown[][] };
+      const el = element as { tag?: string; text?: string; user_name?: string };
 
-      if (el.tag === "text" && el.text) {
-        texts.push(el.text);
-      } else if (el.tag === "a" && el.text) {
+      if ((el.tag === "text" || el.tag === "a") && el.text) {
         texts.push(el.text);
+      } else if (el.tag === "at" && el.user_name) {
+        texts.push(`@${el.user_name}`);
       }
     }
   }
 
-  if (typeof content === "object" && content !== null) {
-    const post = content as {
-      zh_cn?: { content?: unknown[][] };
-      en_us?: { content?: unknown[][] };
-    };
+  if (typeof content !== "object" || content === null) {
+    return "";
+  }
+
+  const obj = content as Record<string, unknown>;
+
+  let postContent: unknown[][] | undefined;
+  let title: string | undefined;
+
+  // Format 1: { zh_cn/en_us: { title, content } }
+  const langPost = (obj.zh_cn || obj.en_us) as { title?: string; content?: unknown[][] } | undefined;
+  if (langPost?.content) {
+    postContent = langPost.content;
+    title = langPost.title;
+  }
+
+  // Format 2: { title, content } - direct format
+  if (!postContent && Array.isArray(obj.content)) {
+    postContent = obj.content as unknown[][];
+    title = obj.title as string | undefined;
+  }
 
-    // Try zh_cn first, then en_us
-    const postContent = post.zh_cn?.content || post.en_us?.content;
+  // Format 3: { post: { zh_cn: { ... } } }
+  if (!postContent && obj.post) {
+    const nested = obj.post as Record<string, unknown>;
+    const nestedLang = (nested.zh_cn || nested.en_us) as { title?: string; content?: unknown[][] } | undefined;
+    if (nestedLang?.content) {
+      postContent = nestedLang.content;
+      title = nestedLang.title;
+    }
+  }
+
+  // Extract title
+  if (title) {
+    texts.push(title);
+  }
 
-    if (Array.isArray(postContent)) {
-      for (const line of postContent) {
-        if (Array.isArray(line)) {
-          extractFromElements(line);
-        }
+  // Extract content
+  if (Array.isArray(postContent)) {
+    for (const line of postContent) {
+      if (Array.isArray(line)) {
+        extractFromElements(line);
       }
     }
   }