refactor: dedupe extension lowercase readers

extensions/discord/src/approval-native.ts

@@ -1,5 +1,6 @@
 import type { DiscordExecApprovalConfig, OpenClawConfig } from "openclaw/plugin-sdk/config-runtime";
 import type { ExecApprovalRequest, PluginApprovalRequest } from "openclaw/plugin-sdk/infra-runtime";
+import { normalizeLowercaseStringOrEmpty } from "openclaw/plugin-sdk/text-runtime";
 import { listDiscordAccountIds, resolveDiscordAccount } from "./accounts.js";
 import {
   createChannelApproverDmTargetResolver,
@@ -103,7 +104,7 @@ function createDiscordOriginTargetResolver(configOverride?: DiscordExecApprovalC
       }),
     resolveTurnSourceTarget: (request) => {
       const sessionKind = extractDiscordSessionKind(request.request.sessionKey?.trim() || null);
-      const turnSourceChannel = request.request.turnSourceChannel?.trim().toLowerCase() || "";
+      const turnSourceChannel = normalizeLowercaseStringOrEmpty(request.request.turnSourceChannel);
       const rawTurnSourceTo = request.request.turnSourceTo?.trim() || "";
       const turnSourceTo = normalizeDiscordOriginChannelId(rawTurnSourceTo);
       const hasExplicitOriginTarget = /^(?:channel|group):/i.test(rawTurnSourceTo);

extensions/feishu/src/directory.ts

@@ -1,3 +1,4 @@
+import { normalizeLowercaseStringOrEmpty } from "openclaw/plugin-sdk/text-runtime";
 import type { ClawdbotConfig } from "../runtime-api.js";
 import { resolveFeishuAccount } from "./accounts.js";
 import { createFeishuClient } from "./client.js";
@@ -37,9 +38,9 @@ export async function listFeishuDirectoryPeersLive(params: {
       throw new Error(response.msg || `code ${response.code}`);
     }
 
+    const q = normalizeLowercaseStringOrEmpty(params.query);
     for (const user of response.data?.items ?? []) {
       if (user.open_id) {
-        const q = params.query?.trim().toLowerCase() || "";
         const name = user.name || "";
         if (!q || user.open_id.toLowerCase().includes(q) || name.toLowerCase().includes(q)) {
           peers.push({
@@ -90,9 +91,9 @@ export async function listFeishuDirectoryGroupsLive(params: {
       throw new Error(response.msg || `code ${response.code}`);
     }
 
+    const q = normalizeLowercaseStringOrEmpty(params.query);
     for (const chat of response.data?.items ?? []) {
       if (chat.chat_id) {
-        const q = params.query?.trim().toLowerCase() || "";
         const name = chat.name || "";
         if (!q || chat.chat_id.toLowerCase().includes(q) || name.toLowerCase().includes(q)) {
           groups.push({

extensions/matrix/src/approval-native.ts

@@ -9,6 +9,7 @@ import {
 } from "openclaw/plugin-sdk/approval-native-runtime";
 import type { ExecApprovalRequest, PluginApprovalRequest } from "openclaw/plugin-sdk/infra-runtime";
 import {
+  normalizeLowercaseStringOrEmpty,
   normalizeOptionalString,
   normalizeOptionalStringifiedId,
 } from "openclaw/plugin-sdk/text-runtime";
@@ -55,7 +56,7 @@ function resolveMatrixNativeTarget(raw: string): string | null {
 }
 
 function resolveTurnSourceMatrixOriginTarget(request: ApprovalRequest): MatrixOriginTarget | null {
-  const turnSourceChannel = request.request.turnSourceChannel?.trim().toLowerCase() || "";
+  const turnSourceChannel = normalizeLowercaseStringOrEmpty(request.request.turnSourceChannel);
   const turnSourceTo = request.request.turnSourceTo?.trim() || "";
   const target = resolveMatrixNativeTarget(turnSourceTo);
   if (turnSourceChannel !== "matrix" || !target) {

extensions/matrix/src/exec-approvals.ts

@@ -11,6 +11,7 @@ import type { OpenClawConfig } from "openclaw/plugin-sdk/config-runtime";
 import type { ExecApprovalRequest, PluginApprovalRequest } from "openclaw/plugin-sdk/infra-runtime";
 import type { ReplyPayload } from "openclaw/plugin-sdk/reply-runtime";
 import { normalizeAccountId } from "openclaw/plugin-sdk/routing";
+import { normalizeLowercaseStringOrEmpty } from "openclaw/plugin-sdk/text-runtime";
 import { listMatrixAccountIds, resolveMatrixAccount } from "./matrix/accounts.js";
 import { normalizeMatrixUserId } from "./matrix/monitor/allowlist.js";
 
@@ -82,7 +83,9 @@ function matchesMatrixRequestAccount(params: {
   accountId?: string | null;
   request: ApprovalRequest;
 }): boolean {
-  const turnSourceChannel = params.request.request.turnSourceChannel?.trim().toLowerCase() || "";
+  const turnSourceChannel = normalizeLowercaseStringOrEmpty(
+    params.request.request.turnSourceChannel,
+  );
   const boundAccountId = resolveApprovalRequestChannelAccountId({
     cfg: params.cfg,
     request: params.request,

extensions/mattermost/src/mattermost/directory.ts

@@ -1,4 +1,5 @@
 import { isPrivateNetworkOptInEnabled } from "openclaw/plugin-sdk/ssrf-runtime";
+import { normalizeLowercaseStringOrEmpty } from "openclaw/plugin-sdk/text-runtime";
 import { listMattermostAccountIds, resolveMattermostAccount } from "./accounts.js";
 import {
   createMattermostClient,
@@ -69,7 +70,7 @@ export async function listMattermostDirectoryGroups(
   if (!clients.length) {
     return [];
   }
-  const q = params.query?.trim().toLowerCase() || "";
+  const q = normalizeLowercaseStringOrEmpty(params.query);
   const seenIds = new Set<string>();
   const entries: ChannelDirectoryEntry[] = [];
 
@@ -140,7 +141,7 @@ export async function listMattermostDirectoryPeers(
     }
     // Uses first team — multi-team setups may need iteration in the future
     const teamId = teams[0].id;
-    const q = params.query?.trim().toLowerCase() || "";
+    const q = normalizeLowercaseStringOrEmpty(params.query);
 
     let users: MattermostUser[];
     if (q) {

extensions/slack/src/approval-native.ts

@@ -7,7 +7,10 @@ import {
   createChannelNativeOriginTargetResolver,
 } from "openclaw/plugin-sdk/approval-native-runtime";
 import type { ExecApprovalRequest, PluginApprovalRequest } from "openclaw/plugin-sdk/infra-runtime";
-import { normalizeOptionalString } from "openclaw/plugin-sdk/text-runtime";
+import {
+  normalizeLowercaseStringOrEmpty,
+  normalizeOptionalString,
+} from "openclaw/plugin-sdk/text-runtime";
 import { listSlackAccountIds } from "./accounts.js";
 import { isSlackApprovalAuthorizedSender } from "./approval-auth.js";
 import {
@@ -46,7 +49,7 @@ function normalizeSlackThreadMatchKey(threadId?: string): string {
 }
 
 function resolveTurnSourceSlackOriginTarget(request: ApprovalRequest): SlackOriginTarget | null {
-  const turnSourceChannel = request.request.turnSourceChannel?.trim().toLowerCase() || "";
+  const turnSourceChannel = normalizeLowercaseStringOrEmpty(request.request.turnSourceChannel);
   const turnSourceTo = request.request.turnSourceTo?.trim() || "";
   if (turnSourceChannel !== "slack" || !turnSourceTo) {
     return null;

extensions/telegram/src/approval-native.ts

@@ -8,7 +8,10 @@ import {
 } from "openclaw/plugin-sdk/approval-native-runtime";
 import type { ChannelApprovalCapability } from "openclaw/plugin-sdk/channel-contract";
 import type { ExecApprovalRequest, PluginApprovalRequest } from "openclaw/plugin-sdk/infra-runtime";
-import { normalizeOptionalString } from "openclaw/plugin-sdk/text-runtime";
+import {
+  normalizeLowercaseStringOrEmpty,
+  normalizeOptionalString,
+} from "openclaw/plugin-sdk/text-runtime";
 import { listTelegramAccountIds } from "./accounts.js";
 import {
   getTelegramExecApprovalApprovers,
@@ -27,7 +30,7 @@ type TelegramOriginTarget = { to: string; threadId?: number };
 function resolveTurnSourceTelegramOriginTarget(
   request: ApprovalRequest,
 ): TelegramOriginTarget | null {
-  const turnSourceChannel = request.request.turnSourceChannel?.trim().toLowerCase() || "";
+  const turnSourceChannel = normalizeLowercaseStringOrEmpty(request.request.turnSourceChannel);
   const rawTurnSourceTo = request.request.turnSourceTo?.trim() || "";
   const parsedTurnSourceTarget = rawTurnSourceTo ? parseTelegramTarget(rawTurnSourceTo) : null;
   const turnSourceTo = normalizeTelegramChatId(parsedTurnSourceTarget?.chatId ?? rawTurnSourceTo);

extensions/telegram/src/exec-approvals.ts

@@ -11,6 +11,7 @@ import type { TelegramExecApprovalConfig } from "openclaw/plugin-sdk/config-runt
 import type { ExecApprovalRequest, PluginApprovalRequest } from "openclaw/plugin-sdk/infra-runtime";
 import type { ReplyPayload } from "openclaw/plugin-sdk/reply-runtime";
 import { normalizeAccountId } from "openclaw/plugin-sdk/routing";
+import { normalizeLowercaseStringOrEmpty } from "openclaw/plugin-sdk/text-runtime";
 import { listTelegramAccountIds, resolveTelegramAccount } from "./accounts.js";
 import { resolveTelegramInlineButtonsConfigScope } from "./inline-buttons.js";
 import { normalizeTelegramChatId, resolveTelegramTargetChatType } from "./targets.js";
@@ -107,7 +108,9 @@ function matchesTelegramRequestAccount(params: {
   accountId?: string | null;
   request: ExecApprovalRequest | PluginApprovalRequest;
 }): boolean {
-  const turnSourceChannel = params.request.request.turnSourceChannel?.trim().toLowerCase() || "";
+  const turnSourceChannel = normalizeLowercaseStringOrEmpty(
+    params.request.request.turnSourceChannel,
+  );
   const boundAccountId = resolveApprovalRequestChannelAccountId({
     cfg: params.cfg,
     request: params.request,

packages/plugin-sdk/src/text-runtime.ts

@@ -1 +1,6 @@
-export { normalizeOptionalString, readStringValue } from "../../../src/shared/string-coerce.js";
+export {
+  normalizeLowercaseStringOrEmpty,
+  normalizeOptionalLowercaseString,
+  normalizeOptionalString,
+  readStringValue,
+} from "../../../src/shared/string-coerce.js";

src/plugin-sdk/text-runtime.ts

@@ -29,7 +29,9 @@ export * from "../utils/reaction-level.js";
 export * from "../utils/with-timeout.js";
 export {
   hasNonEmptyString,
+  normalizeLowercaseStringOrEmpty,
   normalizeNullableString,
+  normalizeOptionalLowercaseString,
   normalizeOptionalString,
   readStringValue,
 } from "../shared/string-coerce.js";