@max1874/feishu 0.2.4 → 0.2.6

package/package.json

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

package/src/docx.ts

@@ -161,11 +161,8 @@ function convertTablesToCodeBlocks(markdown: string): string {
 
 /** 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: processedMarkdown },
+    data: { content_type: "markdown", content: markdown },
   });
   if (res.code !== 0) throw new Error(res.msg);
   return {
@@ -188,12 +185,19 @@ 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 };
+  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(res.msg);
+    return { children: res.data?.children ?? [], skipped };
+  } catch (err) {
+    // Log block types for debugging
+    const blockTypes = cleaned.map((b) => BLOCK_TYPE_NAMES[b.block_type] || `type_${b.block_type}`);
+    const errMsg = err instanceof Error ? err.message : String(err);
+    throw new Error(`insertBlocks failed: ${errMsg}. Block types: [${blockTypes.join(", ")}]. Count: ${cleaned.length}`);
+  }
 }
 
 /** Delete all child blocks from a parent */
@@ -483,43 +487,65 @@ async function createDoc(
 }
 
 async function writeDoc(client: Lark.Client, docToken: string, markdown: string) {
-  // 1. Clear existing content
-  const deleted = await clearDocumentContent(client, docToken);
+  let step = "init";
+  try {
+    // 1. Clear existing content
+    step = "clear_content";
+    const deleted = await clearDocumentContent(client, docToken);
+
+    // 2. Pre-process markdown (convert tables to code blocks)
+    step = "preprocess_markdown";
+    const processedMarkdown = convertTablesToCodeBlocks(markdown);
+
+    // 3. Convert markdown to blocks
+    step = "convert_markdown";
+    const res = await client.docx.document.convert({
+      data: { content_type: "markdown", content: processedMarkdown },
+    });
+    if (res.code !== 0) throw new Error(`convert failed: ${res.msg}`);
+    const blocks = res.data?.blocks ?? [];
 
-  // 2. Convert markdown to blocks
-  const { blocks } = await convertMarkdown(client, markdown);
-  if (blocks.length === 0) {
-    return { success: true, blocks_deleted: deleted, blocks_added: 0, images_processed: 0 };
-  }
+    if (blocks.length === 0) {
+      return { success: true, blocks_deleted: deleted, blocks_added: 0, images_processed: 0 };
+    }
 
-  // 3. Insert new blocks (unsupported types like Table are filtered)
-  const { children: inserted, skipped } = await insertBlocks(client, docToken, blocks);
+    // 4. Insert new blocks (unsupported types like Table are filtered)
+    step = "insert_blocks";
+    const { children: inserted, skipped } = await insertBlocks(client, docToken, blocks);
 
-  // 4. Process images
-  const imagesProcessed = await processImages(client, docToken, markdown, inserted);
+    // 5. Process images
+    step = "process_images";
+    const imagesProcessed = await processImages(client, docToken, markdown, inserted);
 
-  return {
-    success: true,
-    blocks_deleted: deleted,
-    blocks_added: inserted.length,
-    images_processed: imagesProcessed,
-    ...(skipped.length > 0 && {
-      warning: `Skipped unsupported block types: ${skipped.join(", ")}. Tables are not supported via this API.`,
-    }),
-  };
+    return {
+      success: true,
+      blocks_deleted: deleted,
+      blocks_added: inserted.length,
+      images_processed: imagesProcessed,
+      ...(skipped.length > 0 && {
+        warning: `Skipped unsupported block types: ${skipped.join(", ")}. Tables are not supported via this API.`,
+      }),
+    };
+  } catch (err) {
+    const errMsg = err instanceof Error ? err.message : String(err);
+    throw new Error(`writeDoc failed at step '${step}': ${errMsg}`);
+  }
 }
 
 async function appendDoc(client: Lark.Client, docToken: string, markdown: string) {
-  // 1. Convert markdown to blocks
-  const { blocks } = await convertMarkdown(client, markdown);
+  // 1. Pre-process markdown (convert tables to code blocks)
+  const processedMarkdown = convertTablesToCodeBlocks(markdown);
+
+  // 2. Convert markdown to blocks
+  const { blocks } = await convertMarkdown(client, processedMarkdown);
   if (blocks.length === 0) {
     throw new Error("Content is empty");
   }
 
-  // 2. Insert blocks (unsupported types like Table are filtered)
+  // 3. Insert blocks (unsupported types like Table are filtered)
   const { children: inserted, skipped } = await insertBlocks(client, docToken, blocks);
 
-  // 3. Process images
+  // 4. Process images
   const imagesProcessed = await processImages(client, docToken, markdown, inserted);
 
   return {