@m1heng-clawd/feishu 0.1.11 → 0.1.13

package/src/drive-tools/common.ts

@@ -0,0 +1,18 @@
+import { createFeishuClient } from "../client.js";
+import {
+  errorResult,
+  json,
+  runFeishuApiCall,
+  type FeishuApiResponse,
+} from "../tools-common/feishu-api.js";
+
+export type DriveClient = ReturnType<typeof createFeishuClient>;
+
+export { json, errorResult };
+
+export async function runDriveApiCall<T extends FeishuApiResponse>(
+  context: string,
+  fn: () => Promise<T>,
+): Promise<T> {
+  return runFeishuApiCall(context, fn);
+}

package/src/drive-tools/index.ts

@@ -0,0 +1,2 @@
+export { registerFeishuDriveTools } from "./register.js";
+export { FeishuDriveSchema, type FeishuDriveParams } from "./schemas.js";

package/src/drive-tools/register.ts

@@ -0,0 +1,71 @@
+import type { TSchema } from "@sinclair/typebox";
+import type { OpenClawPluginApi } from "openclaw/plugin-sdk";
+import type { ResolvedFeishuAccount } from "../types.js";
+import { hasFeishuToolEnabledForAnyAccount, withFeishuToolClient } from "../tools-common/tool-exec.js";
+import { runDriveAction } from "./actions.js";
+import { errorResult, json, type DriveClient } from "./common.js";
+import { FeishuDriveSchema, type FeishuDriveParams } from "./schemas.js";
+
+type DriveToolSpec<P> = {
+  name: string;
+  label: string;
+  description: string;
+  parameters: TSchema;
+  run: (args: { client: DriveClient; account: ResolvedFeishuAccount }, params: P) => Promise<unknown>;
+};
+
+function registerDriveTool<P>(api: OpenClawPluginApi, spec: DriveToolSpec<P>) {
+  api.registerTool(
+    {
+      name: spec.name,
+      label: spec.label,
+      description: spec.description,
+      parameters: spec.parameters,
+      async execute(_toolCallId, params) {
+        try {
+          return await withFeishuToolClient({
+            api,
+            toolName: spec.name,
+            requiredTool: "drive",
+            run: async ({ client, account }) =>
+              json(await spec.run({ client: client as DriveClient, account }, params as P)),
+          });
+        } catch (err) {
+          return errorResult(err);
+        }
+      },
+    },
+    { name: spec.name },
+  );
+}
+
+export function registerFeishuDriveTools(api: OpenClawPluginApi) {
+  if (!api.config) {
+    api.logger.debug?.("feishu_drive: No config available, skipping drive tools");
+    return;
+  }
+
+  if (!hasFeishuToolEnabledForAnyAccount(api.config)) {
+    api.logger.debug?.("feishu_drive: No Feishu accounts configured, skipping drive tools");
+    return;
+  }
+
+  if (!hasFeishuToolEnabledForAnyAccount(api.config, "drive")) {
+    api.logger.debug?.("feishu_drive: drive tool disabled in config");
+    return;
+  }
+
+  registerDriveTool<FeishuDriveParams>(api, {
+    name: "feishu_drive",
+    label: "Feishu Drive",
+    description:
+      "Feishu cloud storage operations. Actions: list, info, create_folder, move, delete, import_document. Use 'import_document' to create documents from Markdown with better structure preservation than block-by-block writing.",
+    parameters: FeishuDriveSchema,
+    run: async ({ client, account }, params) => {
+      const mediaMaxBytes = (account.config?.mediaMaxMb ?? 30) * 1024 * 1024;
+      return runDriveAction(client, params, mediaMaxBytes);
+    },
+  });
+
+  api.logger.debug?.("feishu_drive: Registered feishu_drive tool");
+}

package/src/{drive-schema.ts → drive-tools/schemas.ts}

@@ -52,7 +52,8 @@ export const FeishuDriveSchema = Type.Union([
       description: "Document title",
     }),
     content: Type.String({
-      description: "Markdown content to import. Supports full Markdown syntax including tables, lists, code blocks, etc.",
+      description:
+        "Markdown content to import. Supports full Markdown syntax including tables, lists, code blocks, etc.",
     }),
     folder_token: Type.Optional(
       Type.String({

package/src/media.ts

@@ -354,7 +354,8 @@ export async function sendFileFeishu(params: {
   cfg: ClawdbotConfig;
   to: string;
   fileKey: string;
-  msgType?: "file" | "media";
+  /** Use "audio" for audio, "media" for video, "file" for documents */
+  msgType?: "file" | "audio" | "media";
   replyToMessageId?: string;
   accountId?: string;
 }): Promise<SendMediaResult> {
@@ -469,10 +470,30 @@ export async function sendMediaFeishu(params: {
     buffer = mediaBuffer;
     name = fileName ?? "file";
   } else if (mediaUrl) {
-    const loaded = await getFeishuRuntime().media.loadWebMedia(mediaUrl, {
-      maxBytes: mediaMaxBytes,
-      optimizeImages: false,
-    });
+    const mediaLocalRoots = (account.config?.mediaLocalRoots ?? [])
+      .map((root) => root.trim())
+      .filter((root) => root.length > 0);
+    const loaded = await (async () => {
+      try {
+        return await getFeishuRuntime().media.loadWebMedia(mediaUrl, {
+          maxBytes: mediaMaxBytes,
+          optimizeImages: false,
+        });
+      } catch (err) {
+        const shouldRetryWithCustomRoots =
+          mediaLocalRoots.length > 0 &&
+          err instanceof Error &&
+          err.message.includes("Local media path is not under an allowed directory");
+        if (!shouldRetryWithCustomRoots) {
+          throw err;
+        }
+        return await getFeishuRuntime().media.loadWebMedia(mediaUrl, {
+          maxBytes: mediaMaxBytes,
+          optimizeImages: false,
+          localRoots: mediaLocalRoots,
+        });
+      }
+    })();
     buffer = loaded.buffer;
     name = fileName ?? loaded.fileName ?? "file";
   } else {
@@ -495,12 +516,13 @@ export async function sendMediaFeishu(params: {
       fileType,
       accountId,
     });
-    const isMedia = fileType === "mp4" || fileType === "opus";
+    // Feishu requires msg_type "audio" for audio, "media" for video, "file" for documents
+    const msgType = fileType === "opus" ? "audio" : fileType === "mp4" ? "media" : "file";
     return sendFileFeishu({
       cfg,
       to,
       fileKey,
-      msgType: isMedia ? "media" : "file",
+      msgType,
       replyToMessageId,
       accountId,
     });

package/src/perm-tools/actions.ts

@@ -0,0 +1,111 @@
+import { runPermApiCall, type PermClient } from "./common.js";
+import type { FeishuPermParams } from "./schemas.js";
+
+type ListTokenType =
+  | "doc"
+  | "sheet"
+  | "file"
+  | "wiki"
+  | "bitable"
+  | "docx"
+  | "mindnote"
+  | "minutes"
+  | "slides";
+type CreateTokenType =
+  | "doc"
+  | "sheet"
+  | "file"
+  | "wiki"
+  | "bitable"
+  | "docx"
+  | "folder"
+  | "mindnote"
+  | "minutes"
+  | "slides";
+type MemberType =
+  | "email"
+  | "openid"
+  | "unionid"
+  | "openchat"
+  | "opendepartmentid"
+  | "userid"
+  | "groupid"
+  | "wikispaceid";
+type PermType = "view" | "edit" | "full_access";
+
+async function listMembers(client: PermClient, token: string, type: string) {
+  const res = await runPermApiCall("drive.permissionMember.list", () =>
+    client.drive.permissionMember.list({
+      path: { token },
+      params: { type: type as ListTokenType },
+    }),
+  );
+
+  return {
+    members:
+      res.data?.items?.map((m) => ({
+        member_type: m.member_type,
+        member_id: m.member_id,
+        perm: m.perm,
+        name: m.name,
+      })) ?? [],
+  };
+}
+
+async function addMember(
+  client: PermClient,
+  token: string,
+  type: string,
+  memberType: string,
+  memberId: string,
+  perm: string,
+) {
+  const res = await runPermApiCall("drive.permissionMember.create", () =>
+    client.drive.permissionMember.create({
+      path: { token },
+      params: { type: type as CreateTokenType, need_notification: false },
+      data: {
+        member_type: memberType as MemberType,
+        member_id: memberId,
+        perm: perm as PermType,
+      },
+    }),
+  );
+
+  return {
+    success: true,
+    member: res.data?.member,
+  };
+}
+
+async function removeMember(
+  client: PermClient,
+  token: string,
+  type: string,
+  memberType: string,
+  memberId: string,
+) {
+  await runPermApiCall("drive.permissionMember.delete", () =>
+    client.drive.permissionMember.delete({
+      path: { token, member_id: memberId },
+      params: { type: type as CreateTokenType, member_type: memberType as MemberType },
+    }),
+  );
+
+  return {
+    success: true,
+  };
+}
+
+export async function runPermAction(client: PermClient, params: FeishuPermParams) {
+  switch (params.action) {
+    case "list":
+      return listMembers(client, params.token, params.type);
+    case "add":
+      return addMember(client, params.token, params.type, params.member_type, params.member_id, params.perm);
+    case "remove":
+      return removeMember(client, params.token, params.type, params.member_type, params.member_id);
+    default:
+      return { error: `Unknown action: ${(params as any).action}` };
+  }
+}

package/src/perm-tools/common.ts

@@ -0,0 +1,18 @@
+import { createFeishuClient } from "../client.js";
+import {
+  errorResult,
+  json,
+  runFeishuApiCall,
+  type FeishuApiResponse,
+} from "../tools-common/feishu-api.js";
+
+export type PermClient = ReturnType<typeof createFeishuClient>;
+
+export { json, errorResult };
+
+export async function runPermApiCall<T extends FeishuApiResponse>(
+  context: string,
+  fn: () => Promise<T>,
+): Promise<T> {
+  return runFeishuApiCall(context, fn);
+}

package/src/perm-tools/index.ts

@@ -0,0 +1,2 @@
+export { registerFeishuPermTools } from "./register.js";
+export { FeishuPermSchema, type FeishuPermParams } from "./schemas.js";

package/src/perm-tools/register.ts

@@ -0,0 +1,65 @@
+import type { TSchema } from "@sinclair/typebox";
+import type { OpenClawPluginApi } from "openclaw/plugin-sdk";
+import { hasFeishuToolEnabledForAnyAccount, withFeishuToolClient } from "../tools-common/tool-exec.js";
+import { runPermAction } from "./actions.js";
+import { errorResult, json, type PermClient } from "./common.js";
+import { FeishuPermSchema, type FeishuPermParams } from "./schemas.js";
+
+type PermToolSpec<P> = {
+  name: string;
+  label: string;
+  description: string;
+  parameters: TSchema;
+  run: (client: PermClient, params: P) => Promise<unknown>;
+};
+
+function registerPermTool<P>(api: OpenClawPluginApi, spec: PermToolSpec<P>) {
+  api.registerTool(
+    {
+      name: spec.name,
+      label: spec.label,
+      description: spec.description,
+      parameters: spec.parameters,
+      async execute(_toolCallId, params) {
+        try {
+          return await withFeishuToolClient({
+            api,
+            toolName: spec.name,
+            requiredTool: "perm",
+            run: async ({ client }) => json(await spec.run(client as PermClient, params as P)),
+          });
+        } catch (err) {
+          return errorResult(err);
+        }
+      },
+    },
+    { name: spec.name },
+  );
+}
+
+export function registerFeishuPermTools(api: OpenClawPluginApi) {
+  if (!api.config) {
+    api.logger.debug?.("feishu_perm: No config available, skipping perm tools");
+    return;
+  }
+
+  if (!hasFeishuToolEnabledForAnyAccount(api.config)) {
+    api.logger.debug?.("feishu_perm: No Feishu accounts configured, skipping perm tools");
+    return;
+  }
+
+  if (!hasFeishuToolEnabledForAnyAccount(api.config, "perm")) {
+    api.logger.debug?.("feishu_perm: perm tool disabled in config (default: false)");
+    return;
+  }
+
+  registerPermTool<FeishuPermParams>(api, {
+    name: "feishu_perm",
+    label: "Feishu Perm",
+    description: "Feishu permission management. Actions: list, add, remove",
+    parameters: FeishuPermSchema,
+    run: (client, params) => runPermAction(client, params),
+  });
+
+  api.logger.debug?.("feishu_perm: Registered feishu_perm tool");
+}

package/src/policy.ts

@@ -1,6 +1,8 @@
 import type { ChannelGroupContext, GroupToolPolicyConfig } from "openclaw/plugin-sdk";
 import type { FeishuConfig, FeishuGroupConfig } from "./types.js";
 
+export type FeishuGroupCommandMentionBypass = "never" | "single_bot" | "always";
+
 export type FeishuAllowlistMatch = {
   allowed: boolean;
   matchKey?: string;
@@ -90,3 +92,14 @@ export function resolveFeishuReplyPolicy(params: {
 
   return { requireMention };
 }
+
+export function resolveFeishuGroupCommandMentionBypass(params: {
+  globalConfig?: FeishuConfig;
+  groupConfig?: FeishuGroupConfig;
+}): FeishuGroupCommandMentionBypass {
+  return (
+    params.groupConfig?.groupCommandMentionBypass ??
+    params.globalConfig?.groupCommandMentionBypass ??
+    "single_bot"
+  );
+}

package/src/reply-dispatcher.ts

@@ -9,6 +9,7 @@ import {
 import { resolveFeishuAccount } from "./accounts.js";
 import { createFeishuClient } from "./client.js";
 import { buildMentionedCardContent, type MentionTarget } from "./mention.js";
+import { normalizeFeishuMarkdownLinks } from "./text/markdown-links.js";
 import { getFeishuRuntime } from "./runtime.js";
 import { sendMarkdownCardFeishu, sendMessageFeishu } from "./send.js";
 import { FeishuStreamingSession } from "./streaming-card.js";
@@ -102,7 +103,7 @@ export function createFeishuReplyDispatcher(params: CreateFeishuReplyDispatcherP
         params.runtime.log?.(`feishu[${account.accountId}] ${message}`),
       );
       try {
-        await streaming.start(chatId, resolveReceiveIdType(chatId));
+        await streaming.start(chatId, resolveReceiveIdType(chatId), replyToMessageId);
       } catch (error) {
         params.runtime.error?.(`feishu: streaming start failed: ${String(error)}`);
         streaming = null;
@@ -120,7 +121,7 @@ export function createFeishuReplyDispatcher(params: CreateFeishuReplyDispatcherP
       if (mentionTargets?.length) {
         text = buildMentionedCardContent(mentionTargets, text);
       }
-      await streaming.close(text);
+      await streaming.close(normalizeFeishuMarkdownLinks(text));
     }
     streaming = null;
     streamingStartPromise = null;
@@ -217,11 +218,12 @@ export function createFeishuReplyDispatcher(params: CreateFeishuReplyDispatcherP
       onModelSelected: prefixContext.onModelSelected,
       onPartialReply: streamingEnabled
         ? (payload: ReplyPayload) => {
-            if (!payload.text || payload.text === lastPartial) {
+            const partialText = normalizeFeishuMarkdownLinks(payload.text ?? "");
+            if (!partialText || partialText === lastPartial) {
               return;
             }
-            lastPartial = payload.text;
-            streamText = payload.text;
+            lastPartial = partialText;
+            streamText = partialText;
             partialUpdateQueue = partialUpdateQueue.then(async () => {
               if (streamingStartPromise) {
                 await streamingStartPromise;

package/src/send.ts

@@ -3,6 +3,7 @@ import type { FeishuSendResult, ResolvedFeishuAccount } from "./types.js";
 import type { MentionTarget } from "./mention.js";
 import { buildMentionedMessage, buildMentionedCardContent } from "./mention.js";
 import { createFeishuClient } from "./client.js";
+import { normalizeFeishuMarkdownLinks } from "./text/markdown-links.js";
 import { resolveReceiveIdType, normalizeFeishuTarget } from "./targets.js";
 import { getFeishuRuntime } from "./runtime.js";
 import { resolveFeishuAccount } from "./accounts.js";
@@ -71,6 +72,31 @@ export async function getMessageFeishu(params: {
       const parsed = JSON.parse(content);
       if (item.msg_type === "text" && parsed.text) {
         content = parsed.text;
+      } else if (parsed.content || parsed.elements) {
+        // Extract plain text from rich text (post) or interactive (card) format.
+        // Both use nested arrays: Array<Array<{tag, text?, href?, ...}>>
+        const blocks = parsed.content ?? parsed.elements ?? [];
+        const lines: string[] = [];
+        for (const paragraph of blocks) {
+          if (!Array.isArray(paragraph)) continue;
+          const line = paragraph
+            .map((node: { tag?: string; text?: string; href?: string }) => {
+              if (node.tag === "text") return node.text ?? "";
+              if (node.tag === "a") return node.text ?? node.href ?? "";
+              if (node.tag === "at") return "";
+              if (node.tag === "img") return "[图片]";
+              return node.text ?? "";
+            })
+            .join("");
+          if (line.trim()) lines.push(line);
+        }
+        const extracted = (parsed.title ? parsed.title + "\n" : "") + lines.join("\n");
+        // Filter out Feishu's degraded card placeholder text
+        if (extracted.trim() && !extracted.includes("请升级至最新版本客户端")) {
+          content = extracted;
+        } else if (extracted.includes("请升级至最新版本客户端")) {
+          content = "[卡片消息]";
+        }
       }
     } catch {
       // Keep raw content if parsing fails
@@ -147,7 +173,9 @@ export async function sendMessageFeishu(params: SendFeishuMessageParams): Promis
   if (mentions && mentions.length > 0) {
     rawText = buildMentionedMessage(mentions, rawText);
   }
-  const messageText = getFeishuRuntime().channel.text.convertMarkdownTables(rawText, tableMode);
+  const messageText = normalizeFeishuMarkdownLinks(
+    getFeishuRuntime().channel.text.convertMarkdownTables(rawText, tableMode),
+  );
 
   const { content, msgType } = buildFeishuPostMessagePayload({ messageText });
 
@@ -317,6 +345,7 @@ export async function sendMarkdownCardFeishu(params: {
   if (mentions && mentions.length > 0) {
     cardText = buildMentionedCardContent(mentions, text);
   }
+  cardText = normalizeFeishuMarkdownLinks(cardText);
   const card = buildMarkdownCard(cardText);
   return sendCardFeishu({ cfg, to, card, replyToMessageId, accountId });
 }
@@ -342,7 +371,9 @@ export async function editMessageFeishu(params: {
     cfg,
     channel: "feishu",
   });
-  const messageText = getFeishuRuntime().channel.text.convertMarkdownTables(text ?? "", tableMode);
+  const messageText = normalizeFeishuMarkdownLinks(
+    getFeishuRuntime().channel.text.convertMarkdownTables(text ?? "", tableMode),
+  );
 
   const { content, msgType } = buildFeishuPostMessagePayload({ messageText });
 

package/src/streaming-card.ts

@@ -72,6 +72,7 @@ export class FeishuStreamingSession {
   async start(
     receiveId: string,
     receiveIdType: "open_id" | "user_id" | "union_id" | "email" | "chat_id" = "chat_id",
+    replyToMessageId?: string,
   ): Promise<void> {
     if (this.state) {
       return;
@@ -108,14 +109,16 @@ export class FeishuStreamingSession {
     }
     const cardId = createData.data.card_id;
 
-    const sendRes = await this.client.im.message.create({
-      params: { receive_id_type: receiveIdType },
-      data: {
-        receive_id: receiveId,
-        msg_type: "interactive",
-        content: JSON.stringify({ type: "card", data: { card_id: cardId } }),
-      },
-    });
+    const cardContent = JSON.stringify({ type: "card", data: { card_id: cardId } });
+    const sendRes = replyToMessageId
+      ? await this.client.im.message.reply({
+          path: { message_id: replyToMessageId },
+          data: { msg_type: "interactive", content: cardContent },
+        })
+      : await this.client.im.message.create({
+          params: { receive_id_type: receiveIdType },
+          data: { receive_id: receiveId, msg_type: "interactive", content: cardContent },
+        });
     if (sendRes.code !== 0 || !sendRes.data?.message_id) {
       throw new Error(`Send card failed: ${sendRes.msg}`);
     }