From 1fca7c39289a1a3bb204580859f2dfcfde61fa60 Mon Sep 17 00:00:00 2001
From: Clawdbot <bot@clawd.bot>
Date: Mon, 16 Feb 2026 13:19:30 +0100
Subject: [PATCH] fix(discord): strip user:/discord:/pk: prefixes in command
 allowFrom
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Discord's formatAllowFrom now strips these prefixes before matching,
aligning with normalizeDiscordAllowList behavior used in DM admission.

Before: commands.allowFrom: ["user:123"] → no match (senderCandidates: ["123", "discord:123"])
After: commands.allowFrom: ["user:123"] → "123" → matches sender "123"

Fixes #17937
---
 src/channels/dock.ts | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/channels/dock.ts b/src/channels/dock.ts
index f8e8641327..a51eee56ea 100644
--- a/src/channels/dock.ts
+++ b/src/channels/dock.ts
@@ -219,7 +219,17 @@ const DOCKS: Record<ChatChannelId, ChannelDock> = {
           String(entry),
         );
       },
-      formatAllowFrom: ({ allowFrom }) => formatLower(allowFrom),
+      formatAllowFrom: ({ allowFrom }) =>
+        allowFrom
+          .map((entry) => String(entry).trim())
+          .filter(Boolean)
+          .map((entry) =>
+            entry
+              .replace(/^discord:/i, "")
+              .replace(/^user:/i, "")
+              .replace(/^pk:/i, "")
+              .toLowerCase(),
+          ),
     },
     groups: {
       resolveRequireMention: resolveDiscordGroupRequireMention,