feishu-user-plugin 1.1.2 → 1.1.3

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "feishu-user-plugin",
-  "version": "1.1.2",
+  "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,15 @@ 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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "feishu-user-plugin",
-  "version": "1.1.2",
+  "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.2"
+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.2' },
+  { name: 'feishu-user-plugin', version: '1.1.3' },
   { capabilities: { tools: {} } }
 );
 
@@ -663,10 +671,10 @@ async function handleTool(name, args) {
     case 'read_p2p_messages': {
       const official = getOfficialClient();
       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_')) {
-        let uc = null;
-        try { uc = await getUserClient(); } catch (_) {}
         if (uc) {
           const results = await uc.search(chatId);
           const user = results.find(r => r.type === 'user');
@@ -684,8 +692,6 @@ async function handleTool(name, args) {
           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.`);
         }
       }
-      let uc = null;
-      try { uc = await getUserClient(); } catch (_) {}
       return json(await official.readMessagesAsUser(chatId, {
         pageSize: args.page_size, startTime: args.start_time, endTime: args.end_time,
         sortType: args.sort_type,
@@ -798,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.2 — ${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)) {
@@ -144,8 +144,11 @@ function _persistToClaudeJson(updates) {
           return;
         }
       }
-    } catch {}
+    } 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;
@@ -448,7 +449,7 @@ class LarkOfficialClient {
     // Fallback: resolve remaining unknowns via cookie-based user identity client
     if (userClient) {
       for (const id of unknownIds) {
-        if (!this._userNameCache.has(id) || !this._userNameCache.get(id)) {
+        if (!this._userNameCache.has(id)) {
           try {
             const name = await userClient.getUserName(id);
             if (name) this._userNameCache.set(id, name);

package/src/test-comprehensive.js

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