mirror of
https://github.com/openclaw/openclaw.git
synced 2026-03-12 07:20:45 +00:00
Matrix: remove orphaned inbound body helpers
This commit is contained in:
@@ -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");
|
||||
});
|
||||
});
|
||||
@@ -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}`;
|
||||
}
|
||||
Reference in New Issue
Block a user