fix(session-memory): narrow transcript filter to delivery-mirror only, preserve gateway-injected

SunnyShu0925 · claude · SunnyShu0925 · commit e1a74c63236e · 2026-06-23T20:33:57.000+08:00
ClawSweeper review flagged that using isTranscriptOnlyOpenClawAssistantMessage
also filters standalone gateway-injected rows, which are visible assistant replies
and should be preserved. Switch to isOpenClawDeliveryMirrorAssistantMessage which
only filters delivery-mirror duplicates.

- Change import from isTranscriptOnlyOpenClawAssistantMessage to isOpenClawDeliveryMirrorAssistantMessage
- Update regression test to verify gateway-injected standalone rows are preserved
- All 195 hooks tests pass

Co-Authored-By: claude-sonnet-4-6 &lt;noreply@anthropic.com&gt;
diff --git a/src/hooks/bundled/session-memory/handler.test.ts b/src/hooks/bundled/session-memory/handler.test.ts
@@ -813,7 +813,7 @@ describe("session-memory hook", () => {
     expect(memoryContent).toContain("assistant: Only message 2");
   });
 
-  it("skips delivery-mirror and gateway-injected assistant rows (fixes #92563)", async () => {
+  it("filters delivery-mirror duplicates but preserves standalone gateway-injected assistant rows (fixes #92563)", async () => {
     const sessionContent = [
       JSON.stringify({ type: "message", message: { role: "user", content: "What is 2+2?" } }),
       JSON.stringify({
@@ -840,14 +840,15 @@ describe("session-memory hook", () => {
           role: "assistant",
           provider: "openclaw",
           model: "gateway-injected",
-          content: [{ type: "text", text: "should be filtered" }],
+          content: [{ type: "text", text: "standalone gateway reply" }],
         },
       }),
     ].join("\n");
 
     const memoryContent = await readSessionTranscript({ sessionContent });
     const assistantLines = memoryContent!.split("\n").filter((l) => l.startsWith("assistant:"));
-    expect(assistantLines).toEqual(["assistant: 2+2 = 4"]);
-    expect(memoryContent).not.toContain("should be filtered");
+    // delivery-mirror duplicate is filtered, gateway-injected standalone is preserved
+    expect(assistantLines).toEqual(["assistant: 2+2 = 4", "assistant: standalone gateway reply"]);
+    expect(memoryContent).toContain("standalone gateway reply");
   });
 });
diff --git a/src/hooks/bundled/session-memory/transcript.ts b/src/hooks/bundled/session-memory/transcript.ts
@@ -2,7 +2,7 @@
 import fs from "node:fs/promises";
 import path from "node:path";
 import { hasInterSessionUserProvenance } from "../../../sessions/input-provenance.js";
-import { isTranscriptOnlyOpenClawAssistantMessage } from "../../../shared/transcript-only-openclaw-assistant.js";
+import { isOpenClawDeliveryMirrorAssistantMessage } from "../../../shared/transcript-only-openclaw-assistant.js";
 
 function extractTextMessageContent(content: unknown): string | undefined {
   if (typeof content === "string") {
@@ -46,7 +46,7 @@ export async function getRecentSessionContent(
             if (role === "user" && hasInterSessionUserProvenance(msg)) {
               continue;
             }
-            if (isTranscriptOnlyOpenClawAssistantMessage(msg)) {
+            if (isOpenClawDeliveryMirrorAssistantMessage(msg)) {
               continue;
             }
             const text = extractTextMessageContent(msg.content);
