@yaoyuanchao/dingtalk 1.4.4 → 1.4.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yaoyuanchao/dingtalk",
-  "version": "1.4.4",
+  "version": "1.4.5",
   "type": "module",
   "description": "DingTalk channel plugin for Clawdbot with Stream Mode support",
   "license": "MIT",

package/src/monitor.ts

@@ -164,6 +164,7 @@ async function extractMessageContent(
         if (chatRecordStr && typeof chatRecordStr === 'string') {
           const records = JSON.parse(chatRecordStr) as Array<{
             senderId?: string;
+            senderStaffId?: string;  // Non-encrypted userId (available when app is published)
             senderNick?: string;
             msgType?: string;
             content?: string;
@@ -171,16 +172,21 @@ async function extractMessageContent(
           }>;
 
           if (Array.isArray(records) && records.length > 0) {
-            // Debug: log first record structure to understand available fields
-            log?.info?.("[dingtalk] chatRecord first record structure: " + JSON.stringify(records[0]));
+            // Debug: log first record structure with all keys
+            const firstRecord = records[0];
+            log?.info?.("[dingtalk] chatRecord first record keys: " + Object.keys(firstRecord).join(', '));
+            log?.info?.("[dingtalk] chatRecord first record: " + JSON.stringify(firstRecord));
 
-            // Collect unique senderIds for batch lookup
+            // Collect unique userIds for batch lookup
+            // Prefer senderStaffId (non-encrypted) over senderId
             const senderIds = [...new Set(
               records
-                .map(r => r.senderId)
-                .filter((id): id is string => !!id && !id.startsWith('$:')) // Skip encrypted IDs
+                .map(r => r.senderStaffId || (r.senderId && !r.senderId.startsWith('$:') ? r.senderId : null))
+                .filter((id): id is string => !!id)
             )].slice(0, 10); // Limit to 10 users
 
+            log?.info?.("[dingtalk] chatRecord senderIds for lookup: " + JSON.stringify(senderIds));
+
             // Try to resolve sender names via API
             let senderNameMap = new Map<string, string>();
             if (senderIds.length > 0 && account.clientId && account.clientSecret) {
@@ -193,10 +199,18 @@ async function extractMessageContent(
             }
 
             const formattedRecords = records.map((record, idx) => {
-              // Try: senderNick from record > API resolved name > senderId > '未知'
+              // Try: senderNick > API resolved name (via staffId or senderId) > fallback
               let sender = record.senderNick;
-              if (!sender && record.senderId) {
-                sender = senderNameMap.get(record.senderId) || (record.senderId.startsWith('$:') ? '成员' : record.senderId);
+              if (!sender) {
+                // Try to get name from API lookup
+                const lookupId = record.senderStaffId || record.senderId;
+                if (lookupId) {
+                  sender = senderNameMap.get(lookupId);
+                }
+                // Fallback for encrypted IDs
+                if (!sender && record.senderId?.startsWith('$:')) {
+                  sender = '成员';
+                }
               }
               sender = sender || '未知';