perf(memory): skip FTS-only plugin discovery

steipete · steipete · commit fbf574ad9db2 · 2026-07-06T11:02:05.000-04:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -21,6 +21,7 @@ Docs: https://docs.openclaw.ai
 
 ### Fixes
 
+- **FTS-only memory startup:** skip plugin capability discovery when `memorySearch.provider` is explicitly `none`, avoiding an unnecessary cold-start scan.
 - **Control UI agent model labels:** show each selected agent's effective model in the Default picker option instead of the global model. (#100719, #77690, #77440) Thanks @hyspacex.
 - **Control UI inbound image previews:** render canonical inbound media references through the authenticated ticket route after chat-history reloads. (#100725, #90172, #89591) Thanks @sweetcornna.
 - **Small-context compaction:** cap the effective reserve against the known model context window so small local models do not enter compaction from the first token. (#100621) Thanks @vincentkoc.
diff --git a/src/agents/memory-search.test.ts b/src/agents/memory-search.test.ts
@@ -16,7 +16,12 @@ import { MAX_TIMER_TIMEOUT_MS } from "../shared/number-coercion.js";
 import { resolveOpenClawAgentSqlitePath } from "../state/openclaw-agent-db.paths.js";
 import { resolveMemorySearchConfig, resolveMemorySearchSyncConfig } from "./memory-search.js";
 
-const asConfig = (cfg: OpenClawConfig): OpenClawConfig => cfg;
+const asConfig = (cfg: OpenClawConfig): OpenClawConfig => ({
+  ...cfg,
+  // Provider registries are supplied explicitly below; plugin loading belongs
+  // to its integration tests and would turn these pure config cases into cold scans.
+  plugins: cfg.plugins ?? { enabled: false },
+});
 let registeredEmbeddingProvidersSnapshot: RegisteredEmbeddingProvider[];
 
 function registerBaseMemoryEmbeddingProviders(options?: { includeGemini?: boolean }): void {
@@ -254,9 +259,37 @@ describe("memory search config", () => {
   });
 
   it("resolves explicit provider-none", () => {
-    const resolved = resolveMemorySearchConfig(configWithDefaultProvider("none"), "main");
+    const resolved = resolveMemorySearchConfig(
+      asConfig({
+        plugins: { enabled: true },
+        agents: {
+          defaults: { memorySearch: { provider: "none", fallback: "deepinfra" } },
+        },
+      }),
+      "main",
+    );
+
+    expect(resolved?.provider).toBe("none");
+  });
+
+  it("skips multimodal provider discovery for provider-none", () => {
+    const resolved = resolveMemorySearchConfig(
+      asConfig({
+        plugins: { enabled: true },
+        agents: {
+          defaults: {
+            memorySearch: {
+              provider: "none",
+              multimodal: { enabled: true, modalities: ["image"] },
+            },
+          },
+        },
+      }),
+      "main",
+    );
 
     expect(resolved?.provider).toBe("none");
+    expect(resolved?.multimodal.modalities).toEqual(["image"]);
   });
 
   it("resolves custom provider ids through their configured api owner", () => {
diff --git a/src/agents/memory-search.ts b/src/agents/memory-search.ts
@@ -185,6 +185,11 @@ function getConfiguredMemoryEmbeddingProvider(
   providerId: string,
   cfg: OpenClawConfig,
 ): ConfiguredMemoryEmbeddingProvider | undefined {
+  // `none` is the built-in FTS-only sentinel, never a plugin capability.
+  // Avoid cold plugin discovery when semantic memory is intentionally disabled.
+  if (normalizeProviderId(providerId) === "none") {
+    return undefined;
+  }
   const directAdapter = getMemoryEmbeddingProvider(providerId);
   if (directAdapter) {
     return directAdapter;
@@ -225,7 +230,7 @@ function mergeConfig(
   const overrideRemote = overrides?.remote;
   const fallback = overrides?.fallback ?? defaults?.fallback ?? "none";
   const fallbackAdapter =
-    fallback && fallback !== "none"
+    normalizeProviderId(provider) !== "none" && fallback && fallback !== "none"
       ? getConfiguredMemoryEmbeddingProvider(fallback, cfg)
       : undefined;
   const hasRemoteConfig = Boolean(
@@ -476,11 +481,15 @@ export function resolveMemorySearchConfig(
   if (!resolved.enabled) {
     return null;
   }
+  const isFtsOnly = normalizeProviderId(resolved.provider) === "none";
   const multimodalActive = isMemoryMultimodalEnabled(resolved.multimodal);
-  const multimodalProvider = getConfiguredMemoryEmbeddingProvider(resolved.provider, cfg);
+  const multimodalProvider = isFtsOnly
+    ? undefined
+    : getConfiguredMemoryEmbeddingProvider(resolved.provider, cfg);
   // Custom provider ids can map to a memory adapter through models.providers.<id>.api.
   // Keep multimodal validation on that config-aware adapter, not the raw id.
   if (
+    !isFtsOnly &&
     multimodalActive &&
     ((multimodalProvider &&
       !(multimodalProvider.supportsMultimodalEmbeddings?.({ model: resolved.model }) ?? false)) ||
