diff --git a/extensions/matrix/src/matrix/monitor/inbound-body.test.ts b/extensions/matrix/src/matrix/monitor/inbound-body.test.ts deleted file mode 100644 index 8b5c63c89a9..00000000000 --- a/extensions/matrix/src/matrix/monitor/inbound-body.test.ts +++ /dev/null @@ -1,73 +0,0 @@ -import { describe, expect, it } from "vitest"; -import { - resolveMatrixBodyForAgent, - resolveMatrixInboundSenderLabel, - resolveMatrixSenderUsername, -} from "./inbound-body.js"; - -describe("resolveMatrixSenderUsername", () => { - it("extracts localpart without leading @", () => { - expect(resolveMatrixSenderUsername("@bu:matrix.example.org")).toBe("bu"); - }); -}); - -describe("resolveMatrixInboundSenderLabel", () => { - it("uses provided senderUsername when present", () => { - expect( - resolveMatrixInboundSenderLabel({ - senderName: "Bu", - senderId: "@bu:matrix.example.org", - senderUsername: "BU_CUSTOM", - }), - ).toBe("Bu (BU_CUSTOM)"); - }); - - it("includes sender username when it differs from display name", () => { - expect( - resolveMatrixInboundSenderLabel({ - senderName: "Bu", - senderId: "@bu:matrix.example.org", - }), - ).toBe("Bu (bu)"); - }); - - it("falls back to sender username when display name is blank", () => { - expect( - resolveMatrixInboundSenderLabel({ - senderName: " ", - senderId: "@zhang:matrix.example.org", - }), - ).toBe("zhang"); - }); - - it("falls back to sender id when username cannot be parsed", () => { - expect( - resolveMatrixInboundSenderLabel({ - senderName: "", - senderId: "matrix-user-without-colon", - }), - ).toBe("matrix-user-without-colon"); - }); -}); - -describe("resolveMatrixBodyForAgent", () => { - it("keeps direct message body unchanged", () => { - expect( - resolveMatrixBodyForAgent({ - isDirectMessage: true, - bodyText: "show me my commits", - senderLabel: "Bu (bu)", - }), - ).toBe("show me my commits"); - }); - - it("prefixes non-direct message body with sender label", () => { - expect( - resolveMatrixBodyForAgent({ - isDirectMessage: false, - bodyText: "show me my commits", - senderLabel: "Bu (bu)", - }), - ).toBe("Bu (bu): show me my commits"); - }); -}); diff --git a/extensions/matrix/src/matrix/monitor/inbound-body.ts b/extensions/matrix/src/matrix/monitor/inbound-body.ts deleted file mode 100644 index 48ad8d31e79..00000000000 --- a/extensions/matrix/src/matrix/monitor/inbound-body.ts +++ /dev/null @@ -1,28 +0,0 @@ -export function resolveMatrixSenderUsername(senderId: string): string | undefined { - const username = senderId.split(":")[0]?.replace(/^@/, "").trim(); - return username ? username : undefined; -} - -export function resolveMatrixInboundSenderLabel(params: { - senderName: string; - senderId: string; - senderUsername?: string; -}): string { - const senderName = params.senderName.trim(); - const senderUsername = params.senderUsername ?? resolveMatrixSenderUsername(params.senderId); - if (senderName && senderUsername && senderName !== senderUsername) { - return `${senderName} (${senderUsername})`; - } - return senderName || senderUsername || params.senderId; -} - -export function resolveMatrixBodyForAgent(params: { - isDirectMessage: boolean; - bodyText: string; - senderLabel: string; -}): string { - if (params.isDirectMessage) { - return params.bodyText; - } - return `${params.senderLabel}: ${params.bodyText}`; -}