From 0fe007f71b2e91f8a6c4b54a3b08b03e7a48f26e Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 1 May 2026 14:32:25 +0100 Subject: [PATCH] refactor: trim extension test hooks --- extensions/feishu/src/dedup-runtime-api.ts | 2 -- extensions/imessage/src/monitor/monitor-provider.ts | 5 ----- extensions/mattermost/src/mattermost/runtime-api.ts | 6 +----- extensions/mattermost/src/setup.accounts.runtime.ts | 6 +----- extensions/msteams/src/monitor-handler/access.ts | 2 -- extensions/nextcloud-talk/src/secret-input.ts | 2 -- extensions/qa-lab/src/docker-runtime.ts | 5 ----- extensions/qa-lab/src/live-transports/discord/cli.ts | 5 ----- .../shared/credential-lease.runtime.ts | 12 ------------ .../qa-lab/src/live-transports/telegram/cli.ts | 5 ----- extensions/qa-lab/src/node-exec.ts | 4 ---- .../qa-lab/src/qa-credentials-admin.runtime.ts | 10 ---------- extensions/qa-matrix/src/docker-runtime.ts | 5 ----- extensions/slack/src/monitor/conversation.runtime.ts | 1 - .../telegram/src/bot-message-dispatch.runtime.ts | 1 - 15 files changed, 2 insertions(+), 69 deletions(-) diff --git a/extensions/feishu/src/dedup-runtime-api.ts b/extensions/feishu/src/dedup-runtime-api.ts index ca9b5cec8d4..e252fbeb4f9 100644 --- a/extensions/feishu/src/dedup-runtime-api.ts +++ b/extensions/feishu/src/dedup-runtime-api.ts @@ -1,3 +1 @@ -export { createDedupeCache } from "openclaw/plugin-sdk/core"; export { createPersistentDedupe } from "openclaw/plugin-sdk/persistent-dedupe"; -export { readJsonFileWithFallback } from "openclaw/plugin-sdk/json-store"; diff --git a/extensions/imessage/src/monitor/monitor-provider.ts b/extensions/imessage/src/monitor/monitor-provider.ts index c705a0a1bdb..0b0353786cb 100644 --- a/extensions/imessage/src/monitor/monitor-provider.ts +++ b/extensions/imessage/src/monitor/monitor-provider.ts @@ -648,9 +648,4 @@ export async function monitorIMessageProvider(opts: MonitorIMessageOpts = {}): P } } -export const __testing = { - resolveIMessageRuntimeGroupPolicy: resolveOpenProviderRuntimeGroupPolicy, - resolveDefaultGroupPolicy, -}; - export const resolveIMessageRuntimeGroupPolicy = resolveOpenProviderRuntimeGroupPolicy; diff --git a/extensions/mattermost/src/mattermost/runtime-api.ts b/extensions/mattermost/src/mattermost/runtime-api.ts index b3c71361175..282f895882b 100644 --- a/extensions/mattermost/src/mattermost/runtime-api.ts +++ b/extensions/mattermost/src/mattermost/runtime-api.ts @@ -33,15 +33,11 @@ export { warnMissingProviderGroupPolicyFallbackOnce, } from "openclaw/plugin-sdk/runtime-group-policy"; export { evaluateSenderGroupAccessForPolicy } from "openclaw/plugin-sdk/group-access"; -export { - getAgentScopedMediaLocalRoots, - resolveChannelMediaMaxBytes, -} from "openclaw/plugin-sdk/media-runtime"; +export { resolveChannelMediaMaxBytes } from "openclaw/plugin-sdk/media-runtime"; export { loadOutboundMediaFromUrl } from "openclaw/plugin-sdk/outbound-media"; export { DEFAULT_GROUP_HISTORY_LIMIT, buildPendingHistoryContextFromMap, - clearHistoryEntriesIfEnabled, recordPendingHistoryEntryIfEnabled, } from "openclaw/plugin-sdk/reply-history"; export { registerPluginHttpRoute } from "openclaw/plugin-sdk/webhook-targets"; diff --git a/extensions/mattermost/src/setup.accounts.runtime.ts b/extensions/mattermost/src/setup.accounts.runtime.ts index f1bec12ecdf..982c75a6ab9 100644 --- a/extensions/mattermost/src/setup.accounts.runtime.ts +++ b/extensions/mattermost/src/setup.accounts.runtime.ts @@ -1,5 +1 @@ -export { - listMattermostAccountIds, - resolveMattermostAccount, - type ResolvedMattermostAccount, -} from "./mattermost/accounts.js"; +export { resolveMattermostAccount, type ResolvedMattermostAccount } from "./mattermost/accounts.js"; diff --git a/extensions/msteams/src/monitor-handler/access.ts b/extensions/msteams/src/monitor-handler/access.ts index 70591fff800..8ce96699774 100644 --- a/extensions/msteams/src/monitor-handler/access.ts +++ b/extensions/msteams/src/monitor-handler/access.ts @@ -16,8 +16,6 @@ import { resolveMSTeamsAllowlistMatch, resolveMSTeamsRouteConfig } from "../poli import { getMSTeamsRuntime } from "../runtime.js"; import type { MSTeamsTurnContext } from "../sdk-types.js"; -export type MSTeamsResolvedSenderAccess = Awaited>; - export async function resolveMSTeamsSenderAccess(params: { cfg: OpenClawConfig; activity: MSTeamsTurnContext["activity"]; diff --git a/extensions/nextcloud-talk/src/secret-input.ts b/extensions/nextcloud-talk/src/secret-input.ts index f1b2aae5c92..10e757b4145 100644 --- a/extensions/nextcloud-talk/src/secret-input.ts +++ b/extensions/nextcloud-talk/src/secret-input.ts @@ -1,6 +1,4 @@ export { buildSecretInputSchema, - hasConfiguredSecretInput, normalizeResolvedSecretInputString, - normalizeSecretInputString, } from "openclaw/plugin-sdk/secret-input"; diff --git a/extensions/qa-lab/src/docker-runtime.ts b/extensions/qa-lab/src/docker-runtime.ts index f8b24032a8b..8f9b51b6caf 100644 --- a/extensions/qa-lab/src/docker-runtime.ts +++ b/extensions/qa-lab/src/docker-runtime.ts @@ -276,8 +276,3 @@ export async function resolveComposeServiceUrl( } return (await isHealthy(`${baseUrl}healthz`, fetchImpl)) ? baseUrl : null; } - -export const __testing = { - fetchHealthUrl, - normalizeDockerServiceStatus, -}; diff --git a/extensions/qa-lab/src/live-transports/discord/cli.ts b/extensions/qa-lab/src/live-transports/discord/cli.ts index a2e66cc7f02..e59987a48e8 100644 --- a/extensions/qa-lab/src/live-transports/discord/cli.ts +++ b/extensions/qa-lab/src/live-transports/discord/cli.ts @@ -1,4 +1,3 @@ -import type { Command } from "commander"; import { createLazyCliRuntimeLoader, createLiveTransportQaCliRegistration, @@ -31,7 +30,3 @@ export const discordQaCliRegistration: LiveTransportQaCliRegistration = sutAccountHelp: "Temporary Discord account id inside the QA gateway config", run: runQaDiscord, }); - -export function registerDiscordQaCli(qa: Command) { - discordQaCliRegistration.register(qa); -} diff --git a/extensions/qa-lab/src/live-transports/shared/credential-lease.runtime.ts b/extensions/qa-lab/src/live-transports/shared/credential-lease.runtime.ts index fefe0eccc99..eaaff019579 100644 --- a/extensions/qa-lab/src/live-transports/shared/credential-lease.runtime.ts +++ b/extensions/qa-lab/src/live-transports/shared/credential-lease.runtime.ts @@ -518,15 +518,3 @@ export function startQaCredentialLeaseHeartbeat( }, }; } - -export const __testing = { - DEFAULT_ACQUIRE_TIMEOUT_MS, - DEFAULT_ENDPOINT_PREFIX, - DEFAULT_HEARTBEAT_INTERVAL_MS, - DEFAULT_LEASE_TTL_MS, - computeAcquireBackoffMs, - normalizeQaCredentialRole, - normalizeQaCredentialSource, - parsePositiveIntegerEnv, - resolveConvexCredentialBrokerConfig, -}; diff --git a/extensions/qa-lab/src/live-transports/telegram/cli.ts b/extensions/qa-lab/src/live-transports/telegram/cli.ts index 2a40142d578..b0f2c0de177 100644 --- a/extensions/qa-lab/src/live-transports/telegram/cli.ts +++ b/extensions/qa-lab/src/live-transports/telegram/cli.ts @@ -1,4 +1,3 @@ -import type { Command } from "commander"; import { createLazyCliRuntimeLoader, createLiveTransportQaCliRegistration, @@ -31,7 +30,3 @@ export const telegramQaCliRegistration: LiveTransportQaCliRegistration = sutAccountHelp: "Temporary Telegram account id inside the QA gateway config", run: runQaTelegram, }); - -export function registerTelegramQaCli(qa: Command) { - telegramQaCliRegistration.register(qa); -} diff --git a/extensions/qa-lab/src/node-exec.ts b/extensions/qa-lab/src/node-exec.ts index 0e887d0af53..d2cd592221b 100644 --- a/extensions/qa-lab/src/node-exec.ts +++ b/extensions/qa-lab/src/node-exec.ts @@ -63,7 +63,3 @@ export async function resolveQaNodeExecPath(params?: { } return resolved; } - -export const __testing = { - isNodeExecPath, -}; diff --git a/extensions/qa-lab/src/qa-credentials-admin.runtime.ts b/extensions/qa-lab/src/qa-credentials-admin.runtime.ts index 2f845954c1c..1fe2bec5491 100644 --- a/extensions/qa-lab/src/qa-credentials-admin.runtime.ts +++ b/extensions/qa-lab/src/qa-credentials-admin.runtime.ts @@ -496,13 +496,3 @@ export async function listQaCredentialSets(options: ListQaCredentialSetsOptions) }, }); } - -export const __testing = { - DEFAULT_ENDPOINT_PREFIX, - DEFAULT_HTTP_TIMEOUT_MS, - normalizeConvexSiteUrl, - normalizeEndpointPrefix, - normalizeStatus, - parsePositiveIntegerEnv, - resolveAdminConfig, -}; diff --git a/extensions/qa-matrix/src/docker-runtime.ts b/extensions/qa-matrix/src/docker-runtime.ts index e64b69a105b..5e75c3f1863 100644 --- a/extensions/qa-matrix/src/docker-runtime.ts +++ b/extensions/qa-matrix/src/docker-runtime.ts @@ -273,8 +273,3 @@ export async function resolveComposeServiceUrl( } return (await isHealthy(`${baseUrl}healthz`, fetchImpl)) ? baseUrl : null; } - -export const __testing = { - fetchHealthUrl, - normalizeDockerServiceStatus, -}; diff --git a/extensions/slack/src/monitor/conversation.runtime.ts b/extensions/slack/src/monitor/conversation.runtime.ts index 226a70f2dd9..323679a51e8 100644 --- a/extensions/slack/src/monitor/conversation.runtime.ts +++ b/extensions/slack/src/monitor/conversation.runtime.ts @@ -1,7 +1,6 @@ export { buildPluginBindingResolvedText, parsePluginBindingApprovalCustomId, - readChannelAllowFromStore, recordInboundSession, resolveConversationLabel, resolvePluginConversationBindingApproval, diff --git a/extensions/telegram/src/bot-message-dispatch.runtime.ts b/extensions/telegram/src/bot-message-dispatch.runtime.ts index c8470c4590f..c0e4c0223a5 100644 --- a/extensions/telegram/src/bot-message-dispatch.runtime.ts +++ b/extensions/telegram/src/bot-message-dispatch.runtime.ts @@ -1,7 +1,6 @@ export { loadSessionStore, resolveSessionStoreEntry, - resolveStorePath, } from "openclaw/plugin-sdk/session-store-runtime"; export { resolveMarkdownTableMode } from "openclaw/plugin-sdk/markdown-table-runtime"; export { getAgentScopedMediaLocalRoots } from "openclaw/plugin-sdk/media-runtime";