refactor: share directory config helpers

steipete · steipete · commit 592dd35ce947 · 2026-03-13T16:51:59.000Z
diff --git a/src/channels/plugins/directory-config-helpers.ts b/src/channels/plugins/directory-config-helpers.ts
@@ -8,7 +8,7 @@ function resolveDirectoryLimit(limit?: number | null): number | undefined {
   return typeof limit === "number" && limit > 0 ? limit : undefined;
 }
 
-function applyDirectoryQueryAndLimit(
+export function applyDirectoryQueryAndLimit(
   ids: string[],
   params: { query?: string | null; limit?: number | null },
 ): string[] {
@@ -18,7 +18,7 @@ function applyDirectoryQueryAndLimit(
   return typeof limit === "number" ? filtered.slice(0, limit) : filtered;
 }
 
-function toDirectoryEntries(kind: "user" | "group", ids: string[]): ChannelDirectoryEntry[] {
+export function toDirectoryEntries(kind: "user" | "group", ids: string[]): ChannelDirectoryEntry[] {
   return ids.map((id) => ({ kind, id }) as const);
 }
 
diff --git a/src/channels/plugins/directory-config.ts b/src/channels/plugins/directory-config.ts
@@ -5,6 +5,7 @@ import { inspectSlackAccount } from "../../slack/account-inspect.js";
 import { inspectTelegramAccount } from "../../telegram/account-inspect.js";
 import { resolveWhatsAppAccount } from "../../web/accounts.js";
 import { isWhatsAppGroupJid, normalizeWhatsAppTarget } from "../../whatsapp/normalize.js";
+import { applyDirectoryQueryAndLimit, toDirectoryEntries } from "./directory-config-helpers.js";
 import { normalizeSlackMessagingTarget } from "./normalize/slack.js";
 import type { ChannelDirectoryEntry } from "./types.js";
 
@@ -54,25 +55,6 @@ function normalizeTrimmedSet(
     .filter((id): id is string => Boolean(id));
 }
 
-function resolveDirectoryQuery(query?: string | null): string {
-  return query?.trim().toLowerCase() || "";
-}
-
-function resolveDirectoryLimit(limit?: number | null): number | undefined {
-  return typeof limit === "number" && limit > 0 ? limit : undefined;
-}
-
-function applyDirectoryQueryAndLimit(ids: string[], params: DirectoryConfigParams): string[] {
-  const q = resolveDirectoryQuery(params.query);
-  const limit = resolveDirectoryLimit(params.limit);
-  const filtered = ids.filter((id) => (q ? id.toLowerCase().includes(q) : true));
-  return typeof limit === "number" ? filtered.slice(0, limit) : filtered;
-}
-
-function toDirectoryEntries(kind: "user" | "group", ids: string[]): ChannelDirectoryEntry[] {
-  return ids.map((id) => ({ kind, id }) as const);
-}
-
 export async function listSlackDirectoryPeersFromConfig(
   params: DirectoryConfigParams,
 ): Promise<ChannelDirectoryEntry[]> {

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ function resolveDirectoryLimit(limit?: number \| null): number \| undefined {`
`8`	`8`	`return typeof limit === "number" && limit > 0 ? limit : undefined;`
`9`	`9`	`}`
`10`	`10`
`11`		`-function applyDirectoryQueryAndLimit(`
	`11`	`+export function applyDirectoryQueryAndLimit(`
`12`	`12`	`ids: string[],`
`13`	`13`	`params: { query?: string \| null; limit?: number \| null },`
`14`	`14`	`): string[] {`
`@@ -18,7 +18,7 @@ function applyDirectoryQueryAndLimit(`
`18`	`18`	`return typeof limit === "number" ? filtered.slice(0, limit) : filtered;`
`19`	`19`	`}`
`20`	`20`
`21`		`-function toDirectoryEntries(kind: "user" \| "group", ids: string[]): ChannelDirectoryEntry[] {`
	`21`	`+export function toDirectoryEntries(kind: "user" \| "group", ids: string[]): ChannelDirectoryEntry[] {`
`22`	`22`	`return ids.map((id) => ({ kind, id }) as const);`
`23`	`23`	`}`
`24`	`24`