feishu-user-plugin 1.1.1 → 1.1.3

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "feishu-user-plugin",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "description": "All-in-one Feishu plugin for Claude Code — send messages as yourself, read chats, manage docs/tables/wiki. 33 tools + 9 skills, 3 auth layers.",
   "author": {
     "name": "EthanQC"

package/CHANGELOG.md

@@ -4,6 +4,23 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/), and this project adheres to [Semantic Versioning](https://semver.org/).
 
+## [1.1.3] - 2026-03-11
+
+### Fixed
+- **Case-insensitive chat name matching**: All name resolution strategies (bot group list, im.chat.search, search_contacts) now use case-insensitive matching. "ai技术解决" now correctly matches "AI技术解决（内部）".
+- **expires_in NaN bug**: UAT token refresh and OAuth now validate `expires_in` field, defaulting to 7200s if missing/invalid, preventing NaN corruption in config.
+- **_populateSenderNames inefficiency**: Fixed redundant condition in cookie-based name fallback.
+- **OAuth silent persistence failure**: Now logs warnings when token persistence to `~/.claude.json` fails, instead of silently swallowing errors.
+- **Null safety**: Added null check in `resolveToOcId` for undefined chat_id.
+
+## [1.1.2] - 2026-03-11
+
+### Fixed
+- **Double OAuth on first install**: `oauth.js` now writes tokens to both `.env` and `~/.claude.json` MCP config directly, so MCP restart picks them up immediately without needing a second OAuth run.
+- **readMessagesAsUser fails with start_time but no end_time**: Auto-sets `end_time` to current timestamp when `start_time` is provided but `end_time` is not, preventing "end_time earlier than start_time" error.
+- **read_p2p_messages rejects chat names**: Now resolves user/group names automatically via search_contacts.
+- **External group messages show sender IDs instead of names**: `_populateSenderNames` now falls back to cookie-based user identity lookup for external tenant users.
+
 ## [1.1.1] - 2026-03-11
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "feishu-user-plugin",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "description": "All-in-one Feishu plugin for Claude Code — send messages as yourself, read chats, manage docs/tables/wiki. 33 tools + 9 skills, 3 auth layers.",
   "main": "src/index.js",
   "bin": {

package/skills/feishu-user-plugin/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: feishu-user-plugin
-version: "1.1.1"
+version: "1.1.3"
 description: "All-in-one Feishu plugin — send messages as yourself, read group/P2P chats, manage docs/tables/wiki. Replaces and extends the official Feishu MCP."
 allowed-tools: send_to_user, send_to_group, send_as_user, send_image_as_user, send_file_as_user, send_post_as_user, send_sticker_as_user, send_audio_as_user, search_contacts, create_p2p_chat, get_chat_info, get_user_info, get_login_status, read_p2p_messages, list_user_chats, list_chats, read_messages, reply_message, forward_message, search_docs, read_doc, create_doc, list_bitable_tables, list_bitable_fields, search_bitable_records, create_bitable_record, update_bitable_record, list_wiki_spaces, search_wiki, list_wiki_nodes, list_files, create_folder, find_user
 user_invocable: true

package/src/index.js

@@ -33,20 +33,28 @@ class ChatIdMapper {
     }
   }
 
+  // Case-insensitive name matching helper
+  static _nameMatch(haystack, needle, exact = false) {
+    if (!haystack || !needle) return false;
+    const h = haystack.toLowerCase(), n = needle.toLowerCase();
+    return exact ? h === n : h.includes(n);
+  }
+
   async findByName(name, official) {
     await this._refresh(official);
-    // Exact match first
+    // Exact match first (case-insensitive)
     for (const [ocId, chatName] of this.nameCache) {
-      if (chatName === name) return ocId;
+      if (ChatIdMapper._nameMatch(chatName, name, true)) return ocId;
     }
-    // Partial match
+    // Partial match (case-insensitive)
     for (const [ocId, chatName] of this.nameCache) {
-      if (chatName && chatName.includes(name)) return ocId;
+      if (ChatIdMapper._nameMatch(chatName, name)) return ocId;
     }
     return null;
   }
 
   async resolveToOcId(chatIdOrName, official) {
+    if (!chatIdOrName) return null;
     if (chatIdOrName.startsWith('oc_')) return chatIdOrName;
     // Also accept raw numeric IDs (from search_contacts)
     if (/^\d+$/.test(chatIdOrName)) return chatIdOrName;
@@ -58,11 +66,11 @@ class ChatIdMapper {
       const results = await official.chatSearch(chatIdOrName);
       for (const chat of results) {
         this.nameCache.set(chat.chat_id, chat.name || '');
-        if (chat.name === chatIdOrName) return chat.chat_id;
+        if (ChatIdMapper._nameMatch(chat.name, chatIdOrName, true)) return chat.chat_id;
       }
-      // Partial match on search results
+      // Partial match on search results (case-insensitive)
       for (const chat of results) {
-        if (chat.name && chat.name.includes(chatIdOrName)) return chat.chat_id;
+        if (ChatIdMapper._nameMatch(chat.name, chatIdOrName)) return chat.chat_id;
       }
     } catch (e) {
       console.error('[feishu-user-plugin] chatSearch fallback failed:', e.message);
@@ -77,13 +85,13 @@ class ChatIdMapper {
     try {
       const results = await userClient.search(chatName);
       const groups = results.filter(r => r.type === 'group');
-      // Exact match first
+      // Exact match first (case-insensitive)
       for (const g of groups) {
-        if (g.title === chatName) return String(g.id);
+        if (ChatIdMapper._nameMatch(g.title, chatName, true)) return String(g.id);
       }
-      // Partial match
+      // Partial match (case-insensitive)
       for (const g of groups) {
-        if (g.title && g.title.includes(chatName)) return String(g.id);
+        if (ChatIdMapper._nameMatch(g.title, chatName)) return String(g.id);
       }
     } catch (e) {
       console.error('[feishu-user-plugin] search_contacts fallback failed:', e.message);
@@ -531,7 +539,7 @@ const TOOLS = [
 // --- Server ---
 
 const server = new Server(
-  { name: 'feishu-user-plugin', version: '1.1.1' },
+  { name: 'feishu-user-plugin', version: '1.1.3' },
   { capabilities: { tools: {} } }
 );
 
@@ -662,10 +670,32 @@ async function handleTool(name, args) {
 
     case 'read_p2p_messages': {
       const official = getOfficialClient();
-      return json(await official.readMessagesAsUser(args.chat_id, {
+      let chatId = args.chat_id;
+      let uc = null;
+      try { uc = await getUserClient(); } catch (_) {}
+      // If chat_id is not numeric or oc_, try to resolve as user name → P2P chat
+      if (!/^\d+$/.test(chatId) && !chatId.startsWith('oc_')) {
+        if (uc) {
+          const results = await uc.search(chatId);
+          const user = results.find(r => r.type === 'user');
+          if (user) {
+            const pChatId = await uc.createChat(String(user.id));
+            if (pChatId) chatId = String(pChatId);
+            else return text(`Found user "${user.title}" but failed to create P2P chat.`);
+          } else {
+            // Maybe it's a group name
+            const group = results.find(r => r.type === 'group');
+            if (group) chatId = String(group.id);
+            else return text(`Cannot resolve "${args.chat_id}" to a chat. Use search_contacts to find the ID first.`);
+          }
+        } else {
+          return text(`"${args.chat_id}" is not a valid chat ID. Provide a numeric ID or oc_xxx format. Use search_contacts + create_p2p_chat to get the ID.`);
+        }
+      }
+      return json(await official.readMessagesAsUser(chatId, {
         pageSize: args.page_size, startTime: args.start_time, endTime: args.end_time,
         sortType: args.sort_type,
-      }));
+      }, uc));
     }
     case 'list_user_chats':
       return json(await getOfficialClient().listChatsAsUser({ pageSize: args.page_size, pageToken: args.page_token }));
@@ -677,18 +707,21 @@ async function handleTool(name, args) {
     case 'read_messages': {
       const official = getOfficialClient();
       const msgOpts = { pageSize: args.page_size, startTime: args.start_time, endTime: args.end_time, sortType: args.sort_type };
+      // Get userClient for name resolution fallback (best-effort)
+      let uc = null;
+      try { uc = await getUserClient(); } catch (_) {}
       const resolvedChatId = await chatIdMapper.resolveToOcId(args.chat_id, official);
 
       // Try bot API first if we resolved an oc_ ID
       if (resolvedChatId) {
         try {
-          return json(await official.readMessages(resolvedChatId, msgOpts));
+          return json(await official.readMessages(resolvedChatId, msgOpts, uc));
         } catch (botErr) {
           // Bot API failed (e.g. bot not in group, no permission) — fall through to UAT
           console.error(`[feishu-user-plugin] read_messages bot API failed for ${resolvedChatId}: ${botErr.message}`);
           if (official.hasUAT) {
             try {
-              return json(await official.readMessagesAsUser(resolvedChatId, msgOpts));
+              return json(await official.readMessagesAsUser(resolvedChatId, msgOpts, uc));
             } catch (uatErr) {
               console.error(`[feishu-user-plugin] read_messages UAT fallback also failed for ${resolvedChatId}: ${uatErr.message}`);
             }
@@ -699,11 +732,10 @@ async function handleTool(name, args) {
 
       // Bot couldn't resolve the chat name — try search_contacts + UAT for external groups
       if (official.hasUAT) {
-        let contactClient = null;
-        try { contactClient = await getUserClient(); } catch (_) {}
-        const contactChatId = await chatIdMapper.resolveViaContacts(args.chat_id, contactClient);
+        if (!uc) try { uc = await getUserClient(); } catch (_) {}
+        const contactChatId = await chatIdMapper.resolveViaContacts(args.chat_id, uc);
         if (contactChatId) {
-          return json(await official.readMessagesAsUser(contactChatId, msgOpts));
+          return json(await official.readMessagesAsUser(contactChatId, msgOpts, uc));
         }
       }
 
@@ -772,7 +804,7 @@ async function main() {
   const hasCookie = !!process.env.LARK_COOKIE;
   const hasApp = !!(process.env.LARK_APP_ID && process.env.LARK_APP_SECRET);
   const hasUAT = !!process.env.LARK_USER_ACCESS_TOKEN;
-  console.error(`[feishu-user-plugin] MCP Server v1.1.1 — ${TOOLS.length} tools`);
+  console.error(`[feishu-user-plugin] MCP Server v1.1.3 — ${TOOLS.length} tools`);
   console.error(`[feishu-user-plugin] Auth: Cookie=${hasCookie ? 'YES' : 'NO'} App=${hasApp ? 'YES' : 'NO'} UAT=${hasUAT ? 'YES' : 'NO'}`);
   if (!hasCookie) console.error('[feishu-user-plugin] WARNING: LARK_COOKIE not set — user identity tools (send_to_user, etc.) will fail');
   if (!hasApp) console.error('[feishu-user-plugin] WARNING: LARK_APP_ID/SECRET not set — official API tools (read_messages, docs, etc.) will fail');

package/src/oauth.js

@@ -108,7 +108,7 @@ function saveToken(tokenData) {
     LARK_USER_ACCESS_TOKEN: tokenData.access_token,
     LARK_USER_REFRESH_TOKEN: tokenData.refresh_token || '',
     LARK_UAT_SCOPE: tokenData.scope || '',
-    LARK_UAT_EXPIRES: String(Math.floor(Date.now() / 1000 + tokenData.expires_in)),
+    LARK_UAT_EXPIRES: String(Math.floor(Date.now() / 1000 + (typeof tokenData.expires_in === 'number' && tokenData.expires_in > 0 ? tokenData.expires_in : 7200))),
   };
 
   for (const [key, val] of Object.entries(updates)) {
@@ -121,6 +121,34 @@ function saveToken(tokenData) {
   }
 
   fs.writeFileSync(envPath, envContent.trim() + '\n');
+
+  // Also persist to ~/.claude.json MCP config so MCP restart picks up tokens immediately
+  _persistToClaudeJson(updates);
+}
+
+function _persistToClaudeJson(updates) {
+  const claudeJsonPaths = [
+    path.join(process.env.HOME || '', '.claude.json'),
+    path.join(process.env.HOME || '', '.claude', '.claude.json'),
+  ];
+  for (const cjPath of claudeJsonPaths) {
+    try {
+      const raw = fs.readFileSync(cjPath, 'utf8');
+      const config = JSON.parse(raw);
+      const servers = config.mcpServers || {};
+      for (const name of ['feishu-user-plugin', 'feishu']) {
+        if (servers[name]?.env) {
+          Object.assign(servers[name].env, updates);
+          fs.writeFileSync(cjPath, JSON.stringify(config, null, 2) + '\n');
+          console.log(`[feishu-user-plugin] OAuth tokens persisted to ${cjPath}`);
+          return;
+        }
+      }
+    } catch (e) {
+      console.error(`[feishu-user-plugin] Failed to persist tokens to ${cjPath}: ${e.message}`);
+    }
+  }
+  console.error('[feishu-user-plugin] WARNING: Could not persist tokens to ~/.claude.json. Tokens saved to .env only — copy them to your MCP config manually.');
 }
 
 const server = http.createServer(async (req, res) => {

package/src/official.js

@@ -58,7 +58,8 @@ class LarkOfficialClient {
 
     this._uat = tokenData.access_token;
     this._uatRefresh = tokenData.refresh_token || this._uatRefresh;
-    this._uatExpires = Math.floor(Date.now() / 1000) + tokenData.expires_in;
+    const expiresIn = typeof tokenData.expires_in === 'number' && tokenData.expires_in > 0 ? tokenData.expires_in : 7200;
+    this._uatExpires = Math.floor(Date.now() / 1000) + expiresIn;
     this._persistUAT();
     console.error('[feishu-user-plugin] UAT refreshed successfully');
     return this._uat;
@@ -156,7 +157,11 @@ class LarkOfficialClient {
     return { items: data.data.items || [], pageToken: data.data.page_token, hasMore: data.data.has_more };
   }
 
-  async readMessagesAsUser(chatId, { pageSize = 20, startTime, endTime, pageToken, sortType = 'ByCreateTimeDesc' } = {}) {
+  async readMessagesAsUser(chatId, { pageSize = 20, startTime, endTime, pageToken, sortType = 'ByCreateTimeDesc' } = {}, userClient) {
+    // Feishu API requires end_time >= start_time; auto-set end_time to now if missing
+    if (startTime && !endTime) {
+      endTime = String(Math.floor(Date.now() / 1000));
+    }
     const params = new URLSearchParams({
       container_id_type: 'chat', container_id: chatId, page_size: String(pageSize),
       sort_type: sortType,
@@ -172,7 +177,7 @@ class LarkOfficialClient {
     });
     if (data.code !== 0) throw new Error(`readMessagesAsUser failed (${data.code}): ${data.msg}`);
     const items = (data.data.items || []).map(m => this._formatMessage(m));
-    await this._populateSenderNames(items);
+    await this._populateSenderNames(items, userClient);
     return { items, hasMore: data.data.has_more, pageToken: data.data.page_token };
   }
 
@@ -186,14 +191,14 @@ class LarkOfficialClient {
     return { items: res.data.items || [], pageToken: res.data.page_token, hasMore: res.data.has_more };
   }
 
-  async readMessages(chatId, { pageSize = 20, startTime, endTime, pageToken, sortType = 'ByCreateTimeDesc' } = {}) {
+  async readMessages(chatId, { pageSize = 20, startTime, endTime, pageToken, sortType = 'ByCreateTimeDesc' } = {}, userClient) {
     const params = { container_id_type: 'chat', container_id: chatId, page_size: pageSize, sort_type: sortType };
     if (startTime) params.start_time = startTime;
     if (endTime) params.end_time = endTime;
     if (pageToken) params.page_token = pageToken;
     const res = await this._safeSDKCall(() => this.client.im.message.list({ params }), 'readMessages');
     const items = (res.data.items || []).map(m => this._formatMessage(m));
-    await this._populateSenderNames(items);
+    await this._populateSenderNames(items, userClient);
     return { items, hasMore: res.data.has_more, pageToken: res.data.page_token };
   }
 
@@ -429,7 +434,7 @@ class LarkOfficialClient {
     return null;
   }
 
-  async _populateSenderNames(items) {
+  async _populateSenderNames(items, userClient) {
     // Collect unique sender IDs that aren't cached
     const unknownIds = new Set();
     for (const item of items) {
@@ -437,10 +442,21 @@ class LarkOfficialClient {
         unknownIds.add(item.senderId);
       }
     }
-    // Batch resolve (sequential, with caching to avoid duplicate calls)
+    // Batch resolve via official contact API
     for (const id of unknownIds) {
       await this.getUserById(id);
     }
+    // Fallback: resolve remaining unknowns via cookie-based user identity client
+    if (userClient) {
+      for (const id of unknownIds) {
+        if (!this._userNameCache.has(id)) {
+          try {
+            const name = await userClient.getUserName(id);
+            if (name) this._userNameCache.set(id, name);
+          } catch {}
+        }
+      }
+    }
     // Populate senderName field
     for (const item of items) {
       if (item.senderId) {

package/src/test-comprehensive.js

@@ -274,7 +274,7 @@ async function testUAT() {
 }
 
 async function main() {
-  console.log('=== feishu-user-plugin v1.1.1 — Comprehensive Test ===\n');
+  console.log('=== feishu-user-plugin v1.1.3 — Comprehensive Test ===\n');
 
   await testUserIdentity();
   console.log('');