From 58b77e787d107bb026f546c8d585f939b057310b Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Thu, 18 Jun 2026 15:05:35 +0800 Subject: [PATCH] refactor(extensions): remove unused helper exports --- .../src/memory/embedding.test-mocks.ts | 5 ---- .../src/memory/manager-sync-control.ts | 27 +------------------ .../msteams/src/bot-framework-service-url.ts | 12 --------- extensions/qqbot/src/engine/utils/platform.ts | 12 --------- 4 files changed, 1 insertion(+), 55 deletions(-) diff --git a/extensions/memory-core/src/memory/embedding.test-mocks.ts b/extensions/memory-core/src/memory/embedding.test-mocks.ts index c91e65d37c47..14a7088effc8 100644 --- a/extensions/memory-core/src/memory/embedding.test-mocks.ts +++ b/extensions/memory-core/src/memory/embedding.test-mocks.ts @@ -4,7 +4,6 @@ import "./test-runtime-mocks.js"; // Avoid exporting vitest mock types (TS2742 under pnpm + d.ts emit). type EmbedBatchMock = Mock<(texts: string[]) => Promise>; -type EmbedQueryMock = Mock<() => Promise>; const hoisted = vi.hoisted(() => ({ embedBatch: vi.fn(async (texts: string[]) => texts.map(() => [0, 1, 0])), @@ -15,10 +14,6 @@ export function getEmbedBatchMock(): EmbedBatchMock { return hoisted.embedBatch; } -export function getEmbedQueryMock(): EmbedQueryMock { - return hoisted.embedQuery; -} - export function resetEmbeddingMocks(): void { hoisted.embedBatch.mockReset(); hoisted.embedQuery.mockReset(); diff --git a/extensions/memory-core/src/memory/manager-sync-control.ts b/extensions/memory-core/src/memory/manager-sync-control.ts index 6190216400f0..f566917150de 100644 --- a/extensions/memory-core/src/memory/manager-sync-control.ts +++ b/extensions/memory-core/src/memory/manager-sync-control.ts @@ -1,10 +1,7 @@ // Memory Core plugin module implements manager sync control behavior. import type { DatabaseSync } from "node:sqlite"; import { formatErrorMessage } from "openclaw/plugin-sdk/error-runtime"; -import { - createSubsystemLogger, - type OpenClawConfig, -} from "openclaw/plugin-sdk/memory-core-host-engine-foundation"; +import { createSubsystemLogger } from "openclaw/plugin-sdk/memory-core-host-engine-foundation"; import type { MemorySyncProgressUpdate } from "openclaw/plugin-sdk/memory-core-host-engine-storage"; const log = createSubsystemLogger("memory"); @@ -166,25 +163,3 @@ export function enqueueMemoryTargetedSessionSync( } return state.getQueuedSessionSync() ?? Promise.resolve(); } - -export function createMemorySyncControlConfigForTests( - workspaceDir: string, - indexPath: string, -): OpenClawConfig { - return { - agents: { - defaults: { - workspace: workspaceDir, - memorySearch: { - provider: "openai", - model: "mock-embed", - store: { path: indexPath, vector: { enabled: false } }, - cache: { enabled: false }, - query: { minScore: 0, hybrid: { enabled: false } }, - sync: { watch: false, onSessionStart: false, onSearch: false }, - }, - }, - list: [{ id: "main", default: true }], - }, - } as OpenClawConfig; -} diff --git a/extensions/msteams/src/bot-framework-service-url.ts b/extensions/msteams/src/bot-framework-service-url.ts index 7cbec5f3fca4..6bfdb8ae0065 100644 --- a/extensions/msteams/src/bot-framework-service-url.ts +++ b/extensions/msteams/src/bot-framework-service-url.ts @@ -1,9 +1,7 @@ // Msteams plugin module implements bot framework service url behavior. import { - buildHostnameAllowlistPolicyFromSuffixAllowlist, isHttpsUrlAllowedByHostnameSuffixAllowlist, normalizeHostnameSuffixAllowlist, - type SsrFPolicy, } from "openclaw/plugin-sdk/ssrf-policy"; const DEFAULT_BOT_FRAMEWORK_SERVICE_URL_HOST_ALLOWLIST = [ @@ -22,16 +20,6 @@ export const BOT_FRAMEWORK_SERVICE_URL_HOST_ALLOWLIST = normalizeHostnameSuffixA DEFAULT_BOT_FRAMEWORK_SERVICE_URL_HOST_ALLOWLIST, ); -const serviceUrlSsrfPolicy = buildHostnameAllowlistPolicyFromSuffixAllowlist( - BOT_FRAMEWORK_SERVICE_URL_HOST_ALLOWLIST, -); - -if (!serviceUrlSsrfPolicy) { - throw new Error("Microsoft Teams Bot Framework serviceUrl allowlist is empty"); -} - -export const BOT_FRAMEWORK_SERVICE_URL_SSRF_POLICY: SsrFPolicy = serviceUrlSsrfPolicy; - export function describeBotFrameworkServiceUrlHost(serviceUrl: string): string { try { const parsed = new URL(serviceUrl.trim()); diff --git a/extensions/qqbot/src/engine/utils/platform.ts b/extensions/qqbot/src/engine/utils/platform.ts index fa1517fb9248..be946cc9b07a 100644 --- a/extensions/qqbot/src/engine/utils/platform.ts +++ b/extensions/qqbot/src/engine/utils/platform.ts @@ -132,18 +132,6 @@ function getOpenClawMediaDir(): string { return path.join(resolveOpenClawHome(), ".openclaw", "media"); } -// ---- Basic platform information ---- - -type PlatformType = "darwin" | "linux" | "win32" | "other"; - -export function getPlatform(): PlatformType { - const p = process.platform; - if (p === "darwin" || p === "linux" || p === "win32") { - return p; - } - return "other"; -} - export function isWindows(): boolean { return process.platform === "win32"; }