fix(discord): preserve underscores in compacted progress

charles-openclaw · charles-openclaw · commit e8170be59e97 · 2026-06-01T03:59:38.000Z
diff --git a/src/channels/streaming.ts b/src/channels/streaming.ts
@@ -860,16 +860,22 @@ function removeUnbalancedInlineBackticks(value: string): string {
   return value.trimStart().startsWith("`") ? value.replaceAll("`", "'") : value.replaceAll("`", "");
 }
 
-function removeUnbalancedLeadingItalics(value: string): string {
-  const underscoreCount = Array.from(value).filter((char) => char === "_").length;
-  if (underscoreCount % 2 === 0 || !value.trimStart().startsWith("_")) {
+function removeUnbalancedLeadingItalicMarker(value: string): string {
+  const leadingItalicMatch = value.match(/^(\s*)_(?!_)(.*)$/su);
+  if (!leadingItalicMatch) {
     return value;
   }
-  return value.replaceAll("_", "");
+
+  const [, leadingWhitespace, detail] = leadingItalicMatch;
+  const hasClosingItalicMarker = /_(?=\s|$|[.,;:!?…])/u.test(detail);
+  if (hasClosingItalicMarker) {
+    return value;
+  }
+  return `${leadingWhitespace}${detail}`;
 }
 
 function removeUnbalancedInlineMarkdown(value: string): string {
-  return removeUnbalancedLeadingItalics(removeUnbalancedInlineBackticks(value));
+  return removeUnbalancedLeadingItalicMarker(removeUnbalancedInlineBackticks(value));
 }
 
 function compactPlainProgressLine(line: string, maxChars: number): string {
diff --git a/src/plugin-sdk/channel-streaming.test.ts b/src/plugin-sdk/channel-streaming.test.ts
@@ -363,6 +363,38 @@ describe("channel-streaming", () => {
     ).toBe("Checking whether the Discord progress draft keeps markdown…");
   });
 
+  it("preserves underscores inside truncated commentary after dropping the italic marker", () => {
+    expect(
+      formatChannelProgressDraftText({
+        entry: { streaming: { progress: { label: false, maxLineChars: 52 } } },
+        lines: [
+          {
+            kind: "item",
+            text: "_Checking the test_name path before the Discord progress draft truncates._",
+            label: "Commentary",
+            prefix: false,
+          },
+        ],
+      }),
+    ).toBe("Checking the test_name path before the Discord…");
+  });
+
+  it("keeps compacted commentary when a leading italic span still has a closer", () => {
+    expect(
+      formatChannelProgressDraftText({
+        entry: { streaming: { progress: { label: false, maxLineChars: 52 } } },
+        lines: [
+          {
+            kind: "item",
+            text: "_Reviewed_ the Discord progress draft before continuing with follow-up verification.",
+            label: "Commentary",
+            prefix: false,
+          },
+        ],
+      }),
+    ).toBe("_Reviewed_ the Discord progress draft before…");
+  });
+
   it("honors configured progress draft line length and cuts prose on word boundaries", () => {
     expect(
       formatChannelProgressDraftText({
