vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-06 07:20:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(hooks): normalize thread ownership slack id casing

Browse Source
This commit is contained in:
Vincent Koc
2026-04-22 12:32:08 -07:00
parent 5567f4cb01
commit f9f836eba4
2 changed files with 8 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
extensions/thread-ownership/index.test.ts
View File

@@ -132,7 +132,7 @@ describe("thread-ownership plugin", () => {
{
content: "hello",
replyToId: "1234.5678",
to: "channel:C123",
to: "channel:c123",
},
{ channelId: "slack", conversationId: "" },
);
@@ -148,7 +148,7 @@ describe("thread-ownership plugin", () => {
});
it("canonicalizes configured ab-test channel allowlists before matching", async () => {
api.pluginConfig = { abTestChannels: ["channel:C123"] };
api.pluginConfig = { abTestChannels: ["channel:c123"] };
register.register(api as unknown as OpenClawPluginApi);
vi.mocked(globalThis.fetch).mockResolvedValue(
new Response(JSON.stringify({ owner: "test-agent" }), { status: 200 }),
@@ -158,7 +158,7 @@ describe("thread-ownership plugin", () => {
{
content: "hello",
replyToId: "1234.5678",
to: "channel:C123",
to: "channel:c123",
},
{ channelId: "slack", conversationId: "" },
);
@@ -227,7 +227,7 @@ describe("thread-ownership plugin", () => {
{
content: "Hey @TestBot help me",
threadId: "9999.0002",
metadata: { channelId: "channel:C456" },
metadata: { channelId: "channel:c456" },
},
{ channelId: "slack", conversationId: "C456" },
);
@@ -250,7 +250,7 @@ describe("thread-ownership plugin", () => {
{
content: "Hey @TestBot help me",
threadId: "9999.0003",
metadata: { channelId: "channel:C456" },
metadata: { channelId: "channel:c456" },
},
{ channelId: "slack", conversationId: "" },
);
@@ -259,7 +259,7 @@ describe("thread-ownership plugin", () => {
{
content: "Sure!",
replyToId: "9999.0003",
to: "C456",
to: "c456",
},
{ channelId: "slack", conversationId: "C456" },
);

3
extensions/thread-ownership/index.ts
View File

@@ -31,7 +31,8 @@ function resolveSlackConversationId(value: unknown): string {
}
const trimmed = raw.trim();
const match = /^(?:slack:)?channel:(.+)$/i.exec(trimmed);
return match?.[1]?.trim() || trimmed;
const resolved = match?.[1]?.trim() || trimmed;
return /^[CDGUW][A-Z0-9]+$/i.test(resolved) ? resolved.toUpperCase() : resolved;
}
function cleanExpiredMentions(): void {