@yaoyuanchao/dingtalk 1.7.7 → 1.7.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yaoyuanchao/dingtalk",
-  "version": "1.7.7",
+  "version": "1.7.8",
   "type": "module",
   "description": "DingTalk channel plugin for ClawdBot/OpenClaw with Stream Mode support",
   "license": "MIT",

package/src/api.ts

@@ -116,11 +116,16 @@ export async function getDingTalkAccessToken(clientId: string, clientSecret: str
 export async function sendViaSessionWebhook(
   sessionWebhook: string,
   text: string,
+  atUserIds?: string[],
 ): Promise<{ ok: boolean; errcode?: number; errmsg?: string; processQueryKey?: string }> {
-  const res = await jsonPost(sessionWebhook, {
+  const body: any = {
     msgtype: "text",
     text: { content: text },
-  });
+  };
+  if (atUserIds?.length) {
+    body.at = { atUserIds, isAtAll: false };
+  }
+  const res = await jsonPost(sessionWebhook, body);
   const ok = res?.errcode === 0 || !res?.errcode;
   if (!ok) {
     console.warn(`[dingtalk] SessionWebhook text error: errcode=${res?.errcode}, errmsg=${res?.errmsg}`);
@@ -133,11 +138,16 @@ export async function sendMarkdownViaSessionWebhook(
   sessionWebhook: string,
   title: string,
   text: string,
+  atUserIds?: string[],
 ): Promise<{ ok: boolean; errcode?: number; errmsg?: string; processQueryKey?: string }> {
-  const res = await jsonPost(sessionWebhook, {
+  const body: any = {
     msgtype: "markdown",
     markdown: { title, text },
-  });
+  };
+  if (atUserIds?.length) {
+    body.at = { atUserIds, isAtAll: false };
+  }
+  const res = await jsonPost(sessionWebhook, body);
   const ok = res?.errcode === 0 || !res?.errcode;
   if (!ok) {
     console.warn(`[dingtalk] SessionWebhook markdown error: errcode=${res?.errcode}, errmsg=${res?.errmsg}`);

package/src/monitor.ts

@@ -369,8 +369,8 @@ export async function startDingTalkMonitor(ctx: DingTalkMonitorContext): Promise
   });
 
   // Reconnection configuration
-  const HEARTBEAT_CHECK_MS = 15_000;     // Check connectivity every 15s
-  const HEARTBEAT_TIMEOUT_MS = 90_000;   // 90s no activity = force reconnect
+  const HEARTBEAT_CHECK_MS = 30_000;     // Check connectivity every 30s
+  const HEARTBEAT_TIMEOUT_MS = 5 * 60 * 1000; // 5 min no activity = force reconnect (safety net only)
   const RECONNECT_BASE_MS = 1_000;       // 1s initial backoff
   const RECONNECT_CAP_MS = 30_000;       // 30s max backoff
   let reconnectAttempt = 0;
@@ -421,6 +421,7 @@ export async function startDingTalkMonitor(ctx: DingTalkMonitorContext): Promise
   });
 
   client.registerAllEventListener((msg: any) => {
+    touchActivity(); // SDK events (including ping/pong) count as activity
     return { status: "SUCCESS", message: "OK" };
   });
 
@@ -437,7 +438,7 @@ export async function startDingTalkMonitor(ctx: DingTalkMonitorContext): Promise
       log?.info?.("[dingtalk:" + account.accountId + "] Stream connected");
       setStatus?.({ running: true, lastStartAt: connectTime });
 
-      // Start heartbeat monitor: if no activity for 90s, force disconnect to trigger reconnect.
+      // Start heartbeat monitor: if no activity for 5 min, force disconnect to trigger reconnect.
       // The SDK's keepAlive ping/pong (8s interval) handles socket-level liveness and sets
       // client.connected=false on missed pongs, which our poll loop below detects.
       // This heartbeat is a secondary safety net for higher-level silent failures where
@@ -2025,6 +2026,10 @@ async function deliverReply(target: any, text: string, log?: any): Promise<void>
     }
   }
 
+  // Auto @mention sender in group chats (only on first chunk to avoid spam)
+  const atUserIds = (!target.isDm && target.senderId) ? [target.senderId] : undefined;
+  let atApplied = false; // Only @ on the first chunk
+
   for (const chunk of chunks) {
     let webhookSuccess = false;
     const maxRetries = 2;
@@ -2036,12 +2041,13 @@ async function deliverReply(target: any, text: string, log?: any): Promise<void>
           await throttleSend();
           log?.info?.("[dingtalk] Using sessionWebhook (attempt " + attempt + "/" + maxRetries + "), format=" + messageFormat);
           log?.info?.("[dingtalk] Sending text (" + chunk.length + " chars): " + chunk.substring(0, 200));
+          const currentAt = (!atApplied && atUserIds) ? atUserIds : undefined;
           let sendResult: { ok: boolean; errcode?: number; errmsg?: string; processQueryKey?: string };
           if (isMarkdown) {
             const markdownTitle = buildMarkdownPreviewTitle(chunk, "Jax");
-            sendResult = await sendMarkdownViaSessionWebhook(target.sessionWebhook, markdownTitle, chunk);
+            sendResult = await sendMarkdownViaSessionWebhook(target.sessionWebhook, markdownTitle, chunk, currentAt);
           } else {
-            sendResult = await sendViaSessionWebhook(target.sessionWebhook, chunk);
+            sendResult = await sendViaSessionWebhook(target.sessionWebhook, chunk, currentAt);
           }
           if (!sendResult.ok) {
             throw new Error(`SessionWebhook rejected: errcode=${sendResult.errcode}, errmsg=${sendResult.errmsg}`);
@@ -2055,6 +2061,7 @@ async function deliverReply(target: any, text: string, log?: any): Promise<void>
             // so interactiveCard quotes can resolve via repliedMsg.createdAt
             cacheOutboundMessageByTime(chunk);
           }
+          if (currentAt) atApplied = true;
           webhookSuccess = true;
           break;
         } catch (err) {