@max1874/feishu 0.2.8 → 0.2.10

package/package.json

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

package/src/docx.ts

@@ -157,16 +157,20 @@ function extractTableFromBlocks(blocks: any[]): { tables: TableData[]; otherBloc
   return { tables, otherBlocks };
 }
 
-/** Clean blocks for insertion (remove read-only fields) */
+/** Clean blocks for insertion (remove read-only fields and unsupported nested structures) */
 function cleanBlocksForInsert(blocks: any[]): { cleaned: any[]; skipped: string[] } {
   const skipped: string[] = [];
   const cleaned = blocks.map((block) => {
-    // Remove any read-only fields that might slip through
+    const { children, parent_id, block_id, ...rest } = block;
+
+    // Remove children from non-table blocks (nested lists not supported in create API)
+    // Table children are handled separately via table.cells
     if (block.block_type === TABLE_BLOCK_TYPE && block.table?.merge_info) {
       const { merge_info, ...tableRest } = block.table;
-      return { ...block, table: tableRest };
+      return { ...rest, table: tableRest };
     }
-    return block;
+
+    return rest;
   });
   return { cleaned, skipped };
 }
@@ -185,7 +189,7 @@ async function convertMarkdown(client: Lark.Client, markdown: string) {
   };
 }
 
-/** Insert blocks as children of a parent block */
+/** Insert blocks as children of a parent block (with batching for >50 blocks) */
 async function insertBlocks(
   client: Lark.Client,
   docToken: string,
@@ -199,12 +203,32 @@ async function insertBlocks(
     return { children: [], skipped };
   }
 
-  const res = await client.docx.documentBlockChildren.create({
+  // Feishu API limits to 50 blocks per request
+  const BATCH_SIZE = 50;
+  const allChildren: any[] = [];
+
+  // Get current children count to determine insert index
+  let insertIndex = 0;
+  const existing = await client.docx.documentBlockChildren.get({
     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 };
+  if (existing.code === 0) {
+    insertIndex = existing.data?.items?.length ?? 0;
+  }
+
+  for (let i = 0; i < cleaned.length; i += BATCH_SIZE) {
+    const batch = cleaned.slice(i, i + BATCH_SIZE);
+    const res = await client.docx.documentBlockChildren.create({
+      path: { document_id: docToken, block_id: blockId },
+      data: { children: batch, index: insertIndex },
+    });
+    if (res.code !== 0) throw new Error(res.msg);
+    const inserted = res.data?.children ?? [];
+    allChildren.push(...inserted);
+    insertIndex += inserted.length;
+  }
+
+  return { children: allChildren, skipped };
 }
 
 /** Delete all child blocks from a parent */