@max1874/feishu 0.3.0 → 0.3.2

package/package.json

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

package/src/docx.ts

@@ -1,6 +1,7 @@
 import { Type } from "@sinclair/typebox";
 import type { OpenClawPluginApi } from "openclaw/plugin-sdk";
 import { createFeishuClient } from "./client.js";
+import { resolveFeishuCredentials, mergeFeishuAccountConfig } from "./accounts.js";
 import { getConversationContext } from "./runtime.js";
 import type { FeishuConfig } from "./types.js";
 import type * as Lark from "@larksuiteoapi/node-sdk";
@@ -887,16 +888,50 @@ const WikiSpaceIdSchema = Type.Object({
   ),
 });
 
+/**
+ * Resolve a FeishuConfig that has credentials, checking:
+ * 1. Top-level appId/appSecret (legacy / pre-v2026.3.2 layout)
+ * 2. The defaultAccount entry under accounts
+ * 3. The first account that has valid credentials
+ */
+function resolveEffectiveFeishuConfig(cfg: FeishuConfig): FeishuConfig | null {
+  // 1. Top-level credentials
+  if (resolveFeishuCredentials(cfg)) return cfg;
+
+  // 2. defaultAccount (e.g. "main")
+  const defaultId = (cfg as any).defaultAccount as string | undefined;
+  if (defaultId && cfg.accounts?.[defaultId]) {
+    const merged = mergeFeishuAccountConfig(cfg, defaultId);
+    if (resolveFeishuCredentials(merged)) return merged;
+  }
+
+  // 3. First account with credentials
+  for (const id of Object.keys(cfg.accounts ?? {})) {
+    const merged = mergeFeishuAccountConfig(cfg, id);
+    if (resolveFeishuCredentials(merged)) return merged;
+  }
+
+  return null;
+}
+
 // ============ Tool Registration ============
 
 export function registerFeishuDocTools(api: OpenClawPluginApi) {
   const feishuCfg = api.config?.channels?.feishu as FeishuConfig | undefined;
-  if (!feishuCfg?.appId || !feishuCfg?.appSecret) {
-    api.logger.debug?.("feishu_doc: Feishu credentials not configured, skipping doc tools");
+  if (!feishuCfg) {
+    api.logger.debug?.("feishu_doc: Feishu not configured, skipping doc tools");
+    return;
+  }
+
+  // Resolve effective config: try top-level credentials first, then fall back
+  // to defaultAccount or the first account that has credentials configured.
+  const resolvedCfg = resolveEffectiveFeishuConfig(feishuCfg);
+  if (!resolvedCfg) {
+    api.logger.debug?.("feishu_doc: Feishu credentials not configured (appId, appSecret required), skipping doc tools");
     return;
   }
 
-  const getClient = () => createFeishuClient(feishuCfg);
+  const getClient = () => createFeishuClient(resolvedCfg);
 
   // Tool 1: feishu_doc_read
   api.registerTool(

package/src/monitor.ts

@@ -19,6 +19,32 @@ const wsClients = new Map<string, Lark.WSClient>();
 const httpServers = new Map<string, http.Server>();
 const botOpenIds = new Map<string, string>();
 
+// --- Message dedup ---
+// Feishu may deliver the same event multiple times (webhook retries, websocket reconnects).
+// Track seen message_ids for a short window to skip duplicates.
+const DEDUP_TTL_MS = 10 * 60 * 1000; // 10 minutes
+const seenMessageIds = new Map<string, number>();
+
+function isDuplicateMessage(messageId: string | undefined): boolean {
+  if (!messageId) return false;
+  const now = Date.now();
+
+  // Lazy cleanup: prune expired entries when map grows
+  if (seenMessageIds.size > 200) {
+    for (const [id, ts] of seenMessageIds) {
+      if (now - ts > DEDUP_TTL_MS) seenMessageIds.delete(id);
+    }
+  }
+
+  const seenAt = seenMessageIds.get(messageId);
+  if (seenAt !== undefined) {
+    if (now - seenAt <= DEDUP_TTL_MS) return true;
+    // Expired — treat as new message
+  }
+  seenMessageIds.set(messageId, now);
+  return false;
+}
+
 async function fetchBotOpenId(cfg: FeishuConfig): Promise<string | undefined> {
   try {
     const result = await probeFeishu(cfg);
@@ -88,6 +114,10 @@ async function monitorWebSocket(params: {
     "im.message.receive_v1": async (data) => {
       try {
         const event = data as unknown as FeishuMessageEvent;
+        if (isDuplicateMessage(event.message?.message_id)) {
+          log(`feishu[${accountId}]: skipping duplicate message ${event.message.message_id}`);
+          return;
+        }
         await handleFeishuMessage({
           cfg,
           event,
@@ -199,6 +229,10 @@ async function monitorWebhook(params: {
     "im.message.receive_v1": async (data) => {
       try {
         const event = data as unknown as FeishuMessageEvent;
+        if (isDuplicateMessage(event.message?.message_id)) {
+          log(`feishu[${accountId}]: skipping duplicate message ${event.message.message_id}`);
+          return;
+        }
         await handleFeishuMessage({
           cfg,
           event,