@yaoyuanchao/dingtalk 1.7.2 → 1.7.3

package/package.json

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

package/src/monitor.ts

@@ -1656,6 +1656,7 @@ async function dispatchWithFullPipeline(params: {
           log, setStatus, onFirstReply } = params;
   
   let firstReplyFired = false;
+  let typingSafetyTimeout: ReturnType<typeof setTimeout> | null = null;
 
   // 1. Resolve agent route via own bindings matching (like official plugin).
   // OpenClaw's resolveAgentRoute doesn't handle accountId correctly for multi-account.
@@ -1751,6 +1752,7 @@ async function dispatchWithFullPipeline(params: {
       // Recall typing indicator on first delivery
       if (!firstReplyFired && onFirstReply) {
         firstReplyFired = true;
+        if (typingSafetyTimeout) { clearTimeout(typingSafetyTimeout); typingSafetyTimeout = null; }
         await onFirstReply().catch((err) => {
           log?.info?.("[dingtalk] onFirstReply error: " + err);
         });
@@ -1778,12 +1780,20 @@ async function dispatchWithFullPipeline(params: {
     await rt.channel.reply.dispatchReplyFromConfig({ ctx, cfg, dispatcher, replyOptions });
   } finally {
     markDispatchIdle();
-    // Recall typing indicator if no reply was sent (queued or error).
-    // Note: when a message is queued (session has active run), dispatch returns
-    // quickly with no delivery — this is normal, not an error. The queued message
-    // will be processed after the current run completes.
+    // Don't recall typing immediately — dispatchReplyFromConfig resolves when
+    // dispatch is *initiated*, not when the agent finishes. The agent may still
+    // be doing tool calls for minutes before producing the first text reply.
+    // The deliver callback above handles the normal recall on first delivery.
+    // Set a safety timeout to recall if no delivery ever arrives (edge case).
     if (!firstReplyFired && onFirstReply) {
-      await onFirstReply().catch(() => {});
+      const TYPING_SAFETY_TIMEOUT_MS = 3 * 60 * 1000; // 3 minutes
+      typingSafetyTimeout = setTimeout(async () => {
+        if (!firstReplyFired && onFirstReply) {
+          firstReplyFired = true;
+          log?.info?.('[dingtalk] Typing safety timeout — recalling after no delivery');
+          await onFirstReply().catch(() => {});
+        }
+      }, TYPING_SAFETY_TIMEOUT_MS);
     }
   }