fix(agents): report accurate omitted char count in capDirectTextContent marker

wangmiao0668000666 · wangmiao0668000666 · commit 070b8b8eec60 · 2026-06-18T20:48:36.000+08:00
diff --git a/src/agents/subagent-announce-delivery.test.ts b/src/agents/subagent-announce-delivery.test.ts
@@ -4644,7 +4644,8 @@ describe("capDirectTextContent", () => {
     const capped = testing.capDirectTextContent(longText);
 
     expect(capped.length).toBeLessThanOrEqual(4000);
-    expect(capped).toContain("... [truncated 1000 chars] ...");
+    // 5000 chars - 2600 head - 1000 tail = 1400 omitted (not 1000)
+    expect(capped).toContain("... [truncated 1400 chars] ...");
     expect(capped.startsWith("x".repeat(2600))).toBe(true);
     expect(capped.endsWith("x".repeat(1000))).toBe(true);
   });
@@ -4659,7 +4660,8 @@ describe("capDirectTextContent", () => {
     const text = "z".repeat(3000);
     const capped = testing.capDirectTextContent(text, 2000);
     expect(capped.length).toBeLessThanOrEqual(2000);
-    expect(capped).toContain("... [truncated 1000 chars] ...");
+    // 3000 chars - 1300 head (0.65*2000) - 500 tail (0.25*2000) = 1200 omitted
+    expect(capped).toContain("... [truncated 1200 chars] ...");
   });
 
   it("handles edge case of exactly maxChars", () => {
@@ -4668,6 +4670,16 @@ describe("capDirectTextContent", () => {
     expect(capped).toBe(text);
     expect(capped.length).toBe(4000);
   });
+
+  it("reports accurate omitted count for head+tail cap (regression for misleading marker)", () => {
+    // 5000 chars, cap 4000 → head 2600 + tail 1000 = 3600 kept, 1400 omitted.
+    const text = "z".repeat(5000);
+    const capped = testing.capDirectTextContent(text);
+    const match = capped.match(/\[truncated (\d+) chars\]/);
+    expect(match).not.toBeNull();
+    const reported = Number(match?.[1]);
+    expect(reported).toBe(1400);
+  });
 });
 
 /**
@@ -4860,7 +4872,7 @@ describe("active wake failure fallback", () => {
 
     expect(sendMessage).toHaveBeenCalledWith(
       expect.objectContaining({
-        content: expect.stringContaining("... [truncated 2000 chars] ..."),
+        content: expect.stringContaining("... [truncated 2400 chars] ..."),
       }),
     );
   });
diff --git a/src/agents/subagent-announce-delivery.ts b/src/agents/subagent-announce-delivery.ts
@@ -922,7 +922,13 @@ function capDirectTextContent(text: string, maxChars = 4000): string {
 
   const headChars = Math.floor(maxChars * 0.65); // 2600
   const tailChars = Math.floor(maxChars * 0.25); // 1000
-  const marker = `\n\n... [truncated ${text.length - maxChars} chars] ...\n\n`;
+  // The omitted count is `text.length - headChars - tailChars`, NOT
+  // `text.length - maxChars`. The marker used to over-state how many
+  // characters were dropped relative to the original `maxChars` cap
+  // (e.g. for 5000 chars with the default 4000 cap, head=2600 and
+  // tail=1000 so we keep 3600 and drop 1400, not the reported 1000).
+  const omitted = text.length - headChars - tailChars;
+  const marker = `\n\n... [truncated ${omitted} chars] ...\n\n`;
 
   return text.slice(0, headChars) + marker + text.slice(-tailChars);
 }
