test(qa-matrix): preserve reply preview semantics

steipete · steipete · commit 38655d5462df · 2026-07-09T12:59:21.000+01:00
diff --git a/extensions/qa-matrix/src/runners/contract/scenario-runtime-shared.test.ts b/extensions/qa-matrix/src/runners/contract/scenario-runtime-shared.test.ts
@@ -1,6 +1,9 @@
 // Qa Matrix tests cover scenario runtime shared plugin behavior.
 import { afterEach, describe, expect, it, vi } from "vitest";
-import { resolveMatrixQaNoReplyWindowMs } from "./scenario-runtime-shared.js";
+import {
+  buildMatrixReplyArtifact,
+  resolveMatrixQaNoReplyWindowMs,
+} from "./scenario-runtime-shared.js";
 
 describe("matrix scenario runtime shared", () => {
   afterEach(() => {
@@ -19,4 +22,18 @@ describe("matrix scenario runtime shared", () => {
       expect(resolveMatrixQaNoReplyWindowMs(30_000)).toBe(8_000);
     }
   });
+
+  it("keeps reply previews UTF-16 safe without changing empty-body artifacts", () => {
+    const event = {
+      kind: "message" as const,
+      roomId: "!room:matrix-qa.test",
+      eventId: "$event",
+      sender: "@sut:matrix-qa.test",
+      type: "m.room.message",
+    };
+    const prefix = "a".repeat(199);
+
+    expect(buildMatrixReplyArtifact({ ...event, body: `${prefix}😀tail` }).bodyPreview).toBe(prefix);
+    expect(buildMatrixReplyArtifact({ ...event, body: " " }).bodyPreview).toBe("");
+  });
 });
diff --git a/extensions/qa-matrix/src/runners/contract/scenario-runtime-shared.ts b/extensions/qa-matrix/src/runners/contract/scenario-runtime-shared.ts
@@ -190,7 +190,7 @@ export function buildMatrixReplyArtifact(
 ): MatrixQaReplyArtifact {
   const replyBody = event.body?.trim();
   return {
-    bodyPreview: replyBody ? truncateUtf16Safe(replyBody, 200) : undefined,
+    bodyPreview: replyBody === undefined ? undefined : truncateUtf16Safe(replyBody, 200),
     eventId: event.eventId,
     mentions: event.mentions,
     relatesTo: event.relatesTo,
