fix(active-memory): use truncateUtf16Safe for log value truncation (#102551)

lsr911 · steipete · web-flow · commit e39e628a8416 · 2026-07-09T11:10:42.000+01:00
* fix(active-memory): use truncateUtf16Safe for log value truncation

* fix(active-memory): use truncateUtf16Safe for log value truncation

* test(active-memory): cover UTF-16-safe log limits

---------

Co-authored-by: Peter Steinberger &lt;steipete@gmail.com&gt;
diff --git a/extensions/active-memory/index.test.ts b/extensions/active-memory/index.test.ts
@@ -4812,13 +4812,14 @@ describe("active-memory plugin", () => {
     ).not.toEqual([]);
   });
 
-  it("caps active-memory log field lengths", async () => {
+  it("caps active-memory log field lengths without splitting surrogate pairs", async () => {
     api.pluginConfig = {
       agents: ["main"],
       logging: true,
     };
     plugin.register(api as unknown as OpenClawPluginApi);
-    const hugeSession = `agent:main:${"x".repeat(500)}`;
+    const sessionPrefix = `agent:main:${"x".repeat(288)}`;
+    const hugeSession = `${sessionPrefix}😀tail`;
 
     await hooks.before_prompt_build(
       { prompt: "what wings should i order? long log value", messages: [] },
@@ -4836,7 +4837,8 @@ describe("active-memory plugin", () => {
     const startLine = infoLines.find((line: string) => line.includes(" start timeoutMs="));
     const line = requireNonEmptyString(startLine, "active memory start log line missing");
     expect(line.length).toBeLessThan(500);
-    expect(line).toContain("...");
+    expect(line).toContain(`session=${sessionPrefix}...`);
+    expect(line).not.toMatch(/[\uD800-\uDFFF]/u);
   });
 
   it("uses a canonical agent session key when only sessionId is available", async () => {
diff --git a/extensions/active-memory/index.ts b/extensions/active-memory/index.ts
@@ -1434,7 +1434,7 @@ function toSingleLineLogValue(value: unknown): string {
     .replace(/\s+/g, " ")
     .trim();
   return singleLine.length > MAX_LOG_VALUE_CHARS
-    ? `${singleLine.slice(0, MAX_LOG_VALUE_CHARS)}...`
+    ? `${truncateUtf16Safe(singleLine, MAX_LOG_VALUE_CHARS)}...`
     : singleLine;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1434,7 +1434,7 @@ function toSingleLineLogValue(value: unknown): string {`
`1434`	`1434`	`.replace(/\s+/g, " ")`
`1435`	`1435`	`.trim();`
`1436`	`1436`	`return singleLine.length > MAX_LOG_VALUE_CHARS`
`1437`		- ? `${singleLine.slice(0, MAX_LOG_VALUE_CHARS)}...`
	`1437`	+ ? `${truncateUtf16Safe(singleLine, MAX_LOG_VALUE_CHARS)}...`
`1438`	`1438`	`: singleLine;`
`1439`	`1439`	`}`
`1440`	`1440`