fix(reply): filter short trailing acks after message-tool sends

zw-xysk · claude · zw-xysk · commit eeed81696d26 · 2026-06-25T19:48:23.000+08:00
After an agent delivers its main reply via the message tool, models often append a short meta-ack (已发, OK, Done, Sent, etc.) that becomes a second visible message. Add isMessagingToolAck to detect these by pattern. Fixes #96681 Co-Authored-By: Claude (via Anthropic API) <noreply@anthropic.com>
diff --git a/src/agents/embedded-agent-helpers.ts b/src/agents/embedded-agent-helpers.ts
@@ -62,6 +62,7 @@ export {
 } from "./embedded-agent-helpers/openai.js";
 export { sanitizeSessionMessagesImages } from "./embedded-agent-helpers/images.js";
 export {
+  isMessagingToolAck,
   isMessagingToolDuplicate,
   isMessagingToolDuplicateNormalized,
   normalizeTextForComparison,
diff --git a/src/agents/embedded-agent-helpers/messaging-dedupe.ts b/src/agents/embedded-agent-helpers/messaging-dedupe.ts
@@ -6,6 +6,17 @@ import { normalizeLowercaseStringOrEmpty } from "@openclaw/normalization-core/st
 const MIN_DUPLICATE_TEXT_LENGTH = 10;
 const MIN_REVERSE_SUBSTRING_DUPLICATE_RATIO = 0.5;
 
+/** Common ack patterns that carry no information beyond a preceding message-tool send. */
+const ACK_PATTERNS = [
+  // English — trailing after message-tool send
+  /\b(sent|done|ok|okay|roger|got it|thanks|thank you)\b\s*\.?\s*#?\d*$/iu,
+  // Chinese
+  /(已发|已发送|已回复|已提交|已通知|不再追加|总结如下|核心回答如下|主回复已发|回复已发)/u,
+];
+
+/** Maximum length for a message to be considered a meta-ack. */
+const MAX_ACK_LENGTH = 200;
+
 /**
  * Normalize text for duplicate comparison.
  * - Trims whitespace
@@ -45,6 +56,18 @@ export function isMessagingToolDuplicateNormalized(
   });
 }
 
+/** Return true when the text is a short meta-ack after a message-tool send. */
+export function isMessagingToolAck(text: string, sentTexts: string[]): boolean {
+  if (sentTexts.length === 0) {
+    return false;
+  }
+  const trimmed = (text ?? "").trim();
+  if (!trimmed || trimmed.length > MAX_ACK_LENGTH) {
+    return false;
+  }
+  return ACK_PATTERNS.some((pattern) => pattern.test(trimmed));
+}
+
 /** Return true when raw message text duplicates a prior sent message. */
 export function isMessagingToolDuplicate(text: string, sentTexts: string[]): boolean {
   if (sentTexts.length === 0) {
diff --git a/src/auto-reply/reply/reply-payloads-dedupe.ts b/src/auto-reply/reply/reply-payloads-dedupe.ts
@@ -3,7 +3,10 @@ import {
   normalizeLowercaseStringOrEmpty,
   normalizeOptionalString,
 } from "@openclaw/normalization-core/string-coerce";
-import { isMessagingToolDuplicate } from "../../agents/embedded-agent-helpers.js";
+import {
+  isMessagingToolAck,
+  isMessagingToolDuplicate,
+} from "../../agents/embedded-agent-helpers.js";
 import type { MessagingToolSend } from "../../agents/embedded-agent-messaging.types.js";
 import { getChannelPlugin } from "../../channels/plugins/index.js";
 import { getLoadedChannelPluginForRead } from "../../channels/plugins/registry-loaded-read.js";
@@ -31,7 +34,8 @@ export function filterMessagingToolDuplicates(params: {
     if (payload.mediaUrl || payload.mediaUrls?.length) {
       return true;
     }
-    return !isMessagingToolDuplicate(payload.text ?? "", sentTexts);
+    const text = payload.text ?? "";
+    return !isMessagingToolDuplicate(text, sentTexts) && !isMessagingToolAck(text, sentTexts);
   });
 }
 
