fix: cover compound post-tool acknowledgements

mushuiyu886 · mushuiyu886 · commit 6483c4911ead · 2026-07-11T01:39:25.000+08:00
diff --git a/src/agents/embedded-agent-helpers/messaging-dedupe.ts b/src/agents/embedded-agent-helpers/messaging-dedupe.ts
@@ -7,9 +7,9 @@ const MIN_DUPLICATE_TEXT_LENGTH = 10;
 const MIN_REVERSE_SUBSTRING_DUPLICATE_RATIO = 0.5;
 const MAX_POST_TOOL_SEND_META_ACK_LENGTH = 80;
 const POST_TOOL_SEND_META_ACK_PATTERNS: readonly RegExp[] = [
-  /^(?:已发(?:送|出|完毕)?|回复已发(?:送|出)?|主回复已发|消息已发(?:送|出)?)(?:[\s,，。.!！:：-]*(?:[#＃][\p{L}\p{N}_-]+|\d+))?[\s。.!！]*$/iu,
-  /^(?:核心回答如下|总结如下|答案如下|不再追加(?:内容|回复)?)[\s。.!！:：]*$/u,
-  /^(?:sent(?:\s+(?:above|already|[#＃][\p{L}\p{N}_-]+))?|posted|done|ok(?:ay)?|roger|got\s+it|ack(?:nowledged)?|replied\s+above)[\s。.!！]*$/iu,
+  /^(?:已发(?:送|出|完毕)?|回复已发(?:送|出)?|主回复已发|消息已发(?:送|出)?)(?:[\s,，。.!！:：-]*(?:[#＃][\p{L}\p{N}_-]+|\d+|\((?:[#＃][\p{L}\p{N}_-]+|\d+)\)|不再追加(?:总结|内容|回复)?))?[\s。.!！]*$/iu,
+  /^(?:核心回答如下|总结如下|答案如下|不再追加(?:总结|内容|回复)?)[\s。.!！:：]*$/u,
+  /^(?:sent(?:\s+(?:above|already|[#＃][\p{L}\p{N}_-]+))?|posted|done|ok(?:ay)?|roger|got\s+it|ack(?:nowledged)?|replied\s+above)(?:[\s,，。.!！:：-]*(?:replied\s+(?:above|in\s+thread)|that's\s+the\s+fix|no\s+more\s+content))?[\s。.!！]*$/iu,
 ];
 
 /**
diff --git a/src/auto-reply/reply/followup-delivery.test.ts b/src/auto-reply/reply/followup-delivery.test.ts
@@ -345,6 +345,30 @@ describe("resolveFollowupDeliveryPayloads", () => {
     ).toStrictEqual([]);
   });
 
+  it("drops compound meta acknowledgements without dropping substantive short replies", () => {
+    expect(
+      resolveFollowupDeliveryPayloads({
+        cfg: baseConfig,
+        payloads: [
+          { text: "已发, 不再追加总结" },
+          { text: "Sent. Replied in thread." },
+          { text: "OK, that's the fix." },
+          { text: "OK, here is the actual answer." },
+        ],
+        messageProvider: "telegram",
+        originatingTo: "telegram:123",
+        sentTargets: [
+          {
+            tool: "message",
+            provider: "telegram",
+            to: "telegram:123",
+            text: "完整主回复内容已经通过 message tool 发出",
+          },
+        ],
+      }),
+    ).toEqual([{ text: "OK, here is the actual answer." }]);
+  });
+
   it("dedupes duplicate replies when a messaging tool already sent to the same provider and target", () => {
     expect(
       resolveFollowupDeliveryPayloads({
