fix(slack): truncate on code-point boundaries to avoid splitting surrogate pairs (#96382)

ly-wang19 · claude · web-flow · commit 1069c60e1e25 · 2026-06-24T19:30:29.000+08:00
truncateSlackText sliced by UTF-16 code unit ('trimmed.slice(0, max - 1)'), so an
emoji or other astral character straddling the limit was cut in half, leaving a
lone high surrogate before the ellipsis — e.g. truncateSlackText('abc😀def', 5)
returned 'abc\uD83D…' instead of 'abc…'. That invalid half-character is sent in
live Slack payloads (message text and Block Kit section/button/header/option
labels, which truncate at limits as small as 75).

Use the repo's canonical sliceUtf16Safe (already re-exported from
plugin-sdk/text-utility-runtime, the module slack code imports from) so a
straddling pair is dropped whole. Behavior is byte-identical for all-BMP input.

Co-authored-by: ly-wang19 &lt;ly-wang19@users.noreply.github.com&gt;
Co-authored-by: Claude Opus 4.8 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/extensions/slack/src/truncate.test.ts b/extensions/slack/src/truncate.test.ts
@@ -0,0 +1,27 @@
+// Slack tests cover truncate plugin behavior.
+import { describe, expect, it } from "vitest";
+import { truncateSlackText } from "./truncate.js";
+
+describe("truncateSlackText", () => {
+  it("drops a surrogate-pair emoji whole when it straddles the limit", () => {
+    // "abc😀def": 😀 (U+1F600) sits at the cut point. Slicing by UTF-16 code unit
+    // would keep only its high surrogate — a lone \uD83D — before the ellipsis,
+    // which serializes to an invalid character in the Slack payload.
+    const out = truncateSlackText("abc😀def", 5);
+    expect(out).toBe("abc…");
+    // No dangling high surrogate (a high surrogate not followed by a low one).
+    expect(/[\uD800-\uDBFF](?![\uDC00-\uDFFF])/.test(out)).toBe(false);
+  });
+
+  it("truncates plain BMP text unchanged", () => {
+    expect(truncateSlackText("hello world", 5)).toBe("hell…");
+  });
+
+  it("keeps an emoji that fits before the cut", () => {
+    expect(truncateSlackText("😀abcdef", 5)).toBe("😀ab…");
+  });
+
+  it("returns the trimmed input unchanged when it fits", () => {
+    expect(truncateSlackText("ab😀cd", 10)).toBe("ab😀cd");
+  });
+});
diff --git a/extensions/slack/src/truncate.ts b/extensions/slack/src/truncate.ts
@@ -1,11 +1,16 @@
 // Slack plugin module implements truncate behavior.
+import { sliceUtf16Safe } from "openclaw/plugin-sdk/text-utility-runtime";
+
 export function truncateSlackText(value: string, max: number): string {
   const trimmed = value.trim();
   if (trimmed.length <= max) {
     return trimmed;
   }
+  // Slice on a code-point boundary so a surrogate pair (emoji / astral char)
+  // straddling the limit is dropped whole, instead of leaving a lone surrogate
+  // half that serializes to an invalid `\uD83D` in the Slack payload.
   if (max <= 1) {
-    return trimmed.slice(0, max);
+    return sliceUtf16Safe(trimmed, 0, max);
   }
-  return `${trimmed.slice(0, max - 1)}…`;
+  return `${sliceUtf16Safe(trimmed, 0, max - 1)}…`;
 }

Original file line number	Diff line number	Diff line change
`@@ -1,11 +1,16 @@`
`1`	`1`	`// Slack plugin module implements truncate behavior.`
	`2`	`+import { sliceUtf16Safe } from "openclaw/plugin-sdk/text-utility-runtime";`
	`3`	`+`
`2`	`4`	`export function truncateSlackText(value: string, max: number): string {`
`3`	`5`	`const trimmed = value.trim();`
`4`	`6`	`if (trimmed.length <= max) {`
`5`	`7`	`return trimmed;`
`6`	`8`	`}`
	`9`	`+ // Slice on a code-point boundary so a surrogate pair (emoji / astral char)`
	`10`	`+ // straddling the limit is dropped whole, instead of leaving a lone surrogate`
	`11`	+ // half that serializes to an invalid `\uD83D` in the Slack payload.
`7`	`12`	`if (max <= 1) {`
`8`		`- return trimmed.slice(0, max);`
	`13`	`+ return sliceUtf16Safe(trimmed, 0, max);`
`9`	`14`	`}`
`10`		- return `${trimmed.slice(0, max - 1)}…`;
	`15`	+ return `${sliceUtf16Safe(trimmed, 0, max - 1)}…`;
`11`	`16`	`}`