@max1874/feishu 0.2.2 → 0.2.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@max1874/feishu",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "type": "module",
   "description": "OpenClaw Feishu/Lark channel plugin",
   "license": "MIT",

package/src/docx.ts

@@ -65,11 +65,9 @@ const BLOCK_TYPE_NAMES: Record<number, string> = {
 };
 
 // Block types that cannot be created via documentBlockChildren.create API
-// Note: Table (31) and TableCell (32) were previously excluded, but the convert API
-// returns them with proper structure, so we attempt to create them directly.
-// If creation fails, the error will be reported to the user.
-const UNSUPPORTED_CREATE_TYPES = new Set<number>([
-  // Empty for now - let's try creating tables directly
+const UNSUPPORTED_CREATE_TYPES = new Set([
+  31, // Table - must use different API or workaround
+  32, // TableCell - child of Table
 ]);
 
 /** Clean blocks for insertion (remove unsupported types and read-only fields) */
@@ -85,7 +83,7 @@ function cleanBlocksForInsert(blocks: any[]): { cleaned: any[]; skipped: string[
       return true;
     })
     .map((block) => {
-      // Remove read-only fields from table blocks that might cause API errors
+      // Remove any read-only fields that might slip through
       if (block.block_type === 31 && block.table?.merge_info) {
         const { merge_info, ...tableRest } = block.table;
         return { ...block, table: tableRest };
@@ -97,10 +95,77 @@ function cleanBlocksForInsert(blocks: any[]): { cleaned: any[]; skipped: string[
 
 // ============ Core Functions ============
 
+/**
+ * Convert Markdown tables to code blocks.
+ * Feishu API doesn't support creating Table blocks directly, so we preserve
+ * table formatting by wrapping them in code blocks.
+ */
+function convertTablesToCodeBlocks(markdown: string): string {
+  const lines = markdown.split("\n");
+  const result: string[] = [];
+  let tableLines: string[] = [];
+  let inTable = false;
+
+  const isTableLine = (line: string): boolean => {
+    const trimmed = line.trim();
+    // Table line starts with | or is a separator line like |---|---|
+    return trimmed.startsWith("|") && trimmed.endsWith("|");
+  };
+
+  const isSeparatorLine = (line: string): boolean => {
+    const trimmed = line.trim();
+    // Separator line contains only |, -, :, and spaces
+    return /^\|[\s\-:|]+\|$/.test(trimmed);
+  };
+
+  const flushTable = () => {
+    if (tableLines.length >= 2) {
+      // Check if we have a valid table (at least header + separator)
+      const hasSeparator = tableLines.some(isSeparatorLine);
+      if (hasSeparator) {
+        // Wrap table in code block to preserve formatting
+        result.push("```");
+        result.push(...tableLines);
+        result.push("```");
+      } else {
+        // Not a valid table, keep as-is
+        result.push(...tableLines);
+      }
+    } else {
+      // Single line with |, keep as-is
+      result.push(...tableLines);
+    }
+    tableLines = [];
+  };
+
+  for (const line of lines) {
+    if (isTableLine(line)) {
+      inTable = true;
+      tableLines.push(line);
+    } else {
+      if (inTable) {
+        flushTable();
+        inTable = false;
+      }
+      result.push(line);
+    }
+  }
+
+  // Flush remaining table if any
+  if (inTable) {
+    flushTable();
+  }
+
+  return result.join("\n");
+}
+
 /** Convert markdown to Feishu blocks using the Convert API */
 async function convertMarkdown(client: Lark.Client, markdown: string) {
+  // Pre-process: convert tables to code blocks since Feishu API doesn't support Table blocks
+  const processedMarkdown = convertTablesToCodeBlocks(markdown);
+
   const res = await client.docx.document.convert({
-    data: { content_type: "markdown", content: markdown },
+    data: { content_type: "markdown", content: processedMarkdown },
   });
   if (res.code !== 0) throw new Error(res.msg);
   return {