@max1874/feishu 0.2.1 → 0.2.3

package/package.json

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

package/src/docx.ts

@@ -65,9 +65,11 @@ const BLOCK_TYPE_NAMES: Record<number, string> = {
 };
 
 // Block types that cannot be created via documentBlockChildren.create API
-const UNSUPPORTED_CREATE_TYPES = new Set([
-  31, // Table - must use different API or workaround
-  32, // TableCell - child of Table
+// 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
 ]);
 
 /** Clean blocks for insertion (remove unsupported types and read-only fields) */
@@ -83,7 +85,7 @@ function cleanBlocksForInsert(blocks: any[]): { cleaned: any[]; skipped: string[
       return true;
     })
     .map((block) => {
-      // Remove any read-only fields that might slip through
+      // Remove read-only fields from table blocks that might cause API errors
       if (block.block_type === 31 && block.table?.merge_info) {
         const { merge_info, ...tableRest } = block.table;
         return { ...block, table: tableRest };
@@ -121,12 +123,34 @@ async function insertBlocks(
     return { children: [], skipped };
   }
 
-  const res = await client.docx.documentBlockChildren.create({
-    path: { document_id: docToken, block_id: blockId },
-    data: { children: cleaned },
-  });
-  if (res.code !== 0) throw new Error(res.msg);
-  return { children: res.data?.children ?? [], skipped };
+  // Log table blocks for debugging
+  const tableBlocks = cleaned.filter((b) => b.block_type === 31);
+  if (tableBlocks.length > 0) {
+    console.log(`[feishu_doc] Attempting to insert ${tableBlocks.length} table block(s)`);
+  }
+
+  try {
+    const res = await client.docx.documentBlockChildren.create({
+      path: { document_id: docToken, block_id: blockId },
+      data: { children: cleaned },
+    });
+    if (res.code !== 0) {
+      throw new Error(`API error ${res.code}: ${res.msg}`);
+    }
+    return { children: res.data?.children ?? [], skipped };
+  } catch (err: any) {
+    // Enhanced error logging
+    const errMsg = err?.response?.data?.msg || err?.message || String(err);
+    const errCode = err?.response?.data?.code || err?.code;
+    console.error(`[feishu_doc] insertBlocks failed: code=${errCode}, msg=${errMsg}`);
+
+    // If table blocks exist, report them as problematic
+    if (tableBlocks.length > 0) {
+      console.error(`[feishu_doc] Table block structure sample:`, JSON.stringify(tableBlocks[0], null, 2).slice(0, 500));
+      throw new Error(`Table insertion failed (code=${errCode}): ${errMsg}. Tables may not be supported via this API.`);
+    }
+    throw err;
+  }
 }
 
 /** Delete all child blocks from a parent */
@@ -570,6 +594,32 @@ async function listAppScopes(client: Lark.Client) {
   };
 }
 
+/** Debug: convert markdown and show resulting block structure */
+async function debugConvertMarkdown(client: Lark.Client, markdown: string) {
+  const { blocks } = await convertMarkdown(client, markdown);
+
+  const blockTypeCounts: Record<string, number> = {};
+  for (const block of blocks) {
+    const typeName = BLOCK_TYPE_NAMES[block.block_type] || `type_${block.block_type}`;
+    blockTypeCounts[typeName] = (blockTypeCounts[typeName] || 0) + 1;
+  }
+
+  // Find table blocks and show their structure
+  const tableBlocks = blocks.filter((b) => b.block_type === 31);
+  let tableSample: any = null;
+  if (tableBlocks.length > 0) {
+    tableSample = tableBlocks[0];
+  }
+
+  return {
+    total_blocks: blocks.length,
+    block_types: blockTypeCounts,
+    has_tables: tableBlocks.length > 0,
+    table_count: tableBlocks.length,
+    table_sample: tableSample ? JSON.stringify(tableSample, null, 2) : null,
+  };
+}
+
 // ============ Schemas ============
 
 const DocTokenSchema = Type.Object({
@@ -626,6 +676,10 @@ const SetPermissionSchema = Type.Object({
   }),
 });
 
+const DebugConvertSchema = Type.Object({
+  content: Type.String({ description: "Markdown content to convert (for debugging block structure)" }),
+});
+
 // Wiki Schemas
 const WikiTokenSchema = Type.Object({
   wiki_token: Type.String({ description: "Wiki token (extract from URL /wiki/XXX)" }),
@@ -961,5 +1015,26 @@ export function registerFeishuDocTools(api: OpenClawPluginApi) {
     { name: "feishu_app_scopes" },
   );
 
-  api.logger.info?.(`feishu_doc: Registered 14 document/wiki tools`);
+  // Tool 15: feishu_debug_convert (for diagnosing markdown conversion issues)
+  api.registerTool(
+    {
+      name: "feishu_debug_convert",
+      label: "Feishu Debug Convert",
+      description:
+        "Debug tool: convert markdown to blocks and show the resulting structure. Use to diagnose why certain markdown (like tables) may fail to write.",
+      parameters: DebugConvertSchema,
+      async execute(_toolCallId, params) {
+        const { content } = params as { content: string };
+        try {
+          const result = await debugConvertMarkdown(getClient(), content);
+          return json(result);
+        } catch (err) {
+          return json({ error: err instanceof Error ? err.message : String(err) });
+        }
+      },
+    },
+    { name: "feishu_debug_convert" },
+  );
+
+  api.logger.info?.(`feishu_doc: Registered 15 document/wiki tools`);
 }