memory-lancedb-pro 1.0.24 → 1.0.26

package/index.ts

@@ -17,6 +17,7 @@ import { createScopeManager } from "./src/scopes.js";
 import { createMigrator } from "./src/migrate.js";
 import { registerAllMemoryTools } from "./src/tools.js";
 import { shouldSkipRetrieval } from "./src/adaptive-retrieval.js";
+import { AccessTracker } from "./src/access-tracker.js";
 import { createMemoryCLI } from "./cli.js";
 
 // ============================================================================
@@ -56,6 +57,8 @@ interface PluginConfig {
     lengthNormAnchor?: number;
     hardMinScore?: number;
     timeDecayHalfLifeDays?: number;
+    reinforcementFactor?: number;
+    maxHalfLifeMultiplier?: number;
   };
   scopes?: {
     default?: string;
@@ -134,12 +137,13 @@ const CAPTURE_EXCLUDE_PATTERNS = [
   /(删除|刪除|清理|清除).{0,12}(记忆|記憶|memory)/i,
 ];
 
-
 export function shouldCapture(text: string): boolean {
   const s = text.trim();
 
   // CJK characters carry more meaning per character, use lower minimum threshold
-  const hasCJK = /[\u4e00-\u9fff\u3040-\u309f\u30a0-\u30ff\uac00-\ud7af]/.test(s);
+  const hasCJK = /[\u4e00-\u9fff\u3040-\u309f\u30a0-\u30ff\uac00-\ud7af]/.test(
+    s,
+  );
   const minLen = hasCJK ? 4 : 10;
   if (s.length < minLen || s.length > 500) {
     return false;
@@ -167,18 +171,36 @@ export function shouldCapture(text: string): boolean {
   return MEMORY_TRIGGERS.some((r) => r.test(s));
 }
 
-export function detectCategory(text: string): "preference" | "fact" | "decision" | "entity" | "other" {
+export function detectCategory(
+  text: string,
+): "preference" | "fact" | "decision" | "entity" | "other" {
   const lower = text.toLowerCase();
-  if (/prefer|radši|like|love|hate|want|偏好|喜歡|喜欢|討厭|讨厌|不喜歡|不喜欢|愛用|爱用|習慣|习惯/i.test(lower)) {
+  if (
+    /prefer|radši|like|love|hate|want|偏好|喜歡|喜欢|討厭|讨厌|不喜歡|不喜欢|愛用|爱用|習慣|习惯/i.test(
+      lower,
+    )
+  ) {
     return "preference";
   }
-  if (/rozhodli|decided|we decided|will use|we will use|we'?ll use|switch(ed)? to|migrate(d)? to|going forward|from now on|budeme|決定|决定|選擇了|选择了|改用|換成|换成|以後用|以后用|規則|流程|SOP/i.test(lower)) {
+  if (
+    /rozhodli|decided|we decided|will use|we will use|we'?ll use|switch(ed)? to|migrate(d)? to|going forward|from now on|budeme|決定|决定|選擇了|选择了|改用|換成|换成|以後用|以后用|規則|流程|SOP/i.test(
+      lower,
+    )
+  ) {
     return "decision";
   }
-  if (/\+\d{10,}|@[\w.-]+\.\w+|is called|jmenuje se|我的\S+是|叫我|稱呼|称呼/i.test(lower)) {
+  if (
+    /\+\d{10,}|@[\w.-]+\.\w+|is called|jmenuje se|我的\S+是|叫我|稱呼|称呼/i.test(
+      lower,
+    )
+  ) {
     return "entity";
   }
-  if (/\b(is|are|has|have|je|má|jsou)\b|總是|总是|從不|从不|一直|每次都|老是/i.test(lower)) {
+  if (
+    /\b(is|are|has|have|je|má|jsou)\b|總是|总是|從不|从不|一直|每次都|老是/i.test(
+      lower,
+    )
+  ) {
     return "fact";
   }
   return "other";
@@ -199,7 +221,10 @@ function sanitizeForContext(text: string): string {
 // Session Content Reading (for session-memory hook)
 // ============================================================================
 
-async function readSessionMessages(filePath: string, messageCount: number): Promise<string | null> {
+async function readSessionMessages(
+  filePath: string,
+  messageCount: number,
+): Promise<string | null> {
   try {
     const lines = (await readFile(filePath, "utf-8")).trim().split("\n");
     const messages: string[] = [];
@@ -214,12 +239,16 @@ async function readSessionMessages(filePath: string, messageCount: number): Prom
             const text = Array.isArray(msg.content)
               ? msg.content.find((c: any) => c.type === "text")?.text
               : msg.content;
-            if (text && !text.startsWith("/") && !text.includes("<relevant-memories>")) {
+            if (
+              text &&
+              !text.startsWith("/") &&
+              !text.includes("<relevant-memories>")
+            ) {
               messages.push(`${role}: ${text}`);
             }
           }
         }
-      } catch { }
+      } catch {}
     }
 
     if (messages.length === 0) return null;
@@ -229,7 +258,10 @@ async function readSessionMessages(filePath: string, messageCount: number): Prom
   }
 }
 
-async function readSessionContentWithResetFallback(sessionFilePath: string, messageCount = 15): Promise<string | null> {
+async function readSessionContentWithResetFallback(
+  sessionFilePath: string,
+  messageCount = 15,
+): Promise<string | null> {
   const primary = await readSessionMessages(sessionFilePath, messageCount);
   if (primary) return primary;
 
@@ -238,13 +270,18 @@ async function readSessionContentWithResetFallback(sessionFilePath: string, mess
     const dir = dirname(sessionFilePath);
     const resetPrefix = `${basename(sessionFilePath)}.reset.`;
     const files = await readdir(dir);
-    const resetCandidates = files.filter(name => name.startsWith(resetPrefix)).sort();
+    const resetCandidates = files
+      .filter((name) => name.startsWith(resetPrefix))
+      .sort();
 
     if (resetCandidates.length > 0) {
-      const latestResetPath = join(dir, resetCandidates[resetCandidates.length - 1]);
+      const latestResetPath = join(
+        dir,
+        resetCandidates[resetCandidates.length - 1],
+      );
       return await readSessionMessages(latestResetPath, messageCount);
     }
-  } catch { }
+  } catch {}
 
   return primary;
 }
@@ -254,14 +291,21 @@ function stripResetSuffix(fileName: string): string {
   return resetIndex === -1 ? fileName : fileName.slice(0, resetIndex);
 }
 
-async function findPreviousSessionFile(sessionsDir: string, currentSessionFile?: string, sessionId?: string): Promise<string | undefined> {
+async function findPreviousSessionFile(
+  sessionsDir: string,
+  currentSessionFile?: string,
+  sessionId?: string,
+): Promise<string | undefined> {
   try {
     const files = await readdir(sessionsDir);
     const fileSet = new Set(files);
 
     // Try recovering the non-reset base file
-    const baseFromReset = currentSessionFile ? stripResetSuffix(basename(currentSessionFile)) : undefined;
-    if (baseFromReset && fileSet.has(baseFromReset)) return join(sessionsDir, baseFromReset);
+    const baseFromReset = currentSessionFile
+      ? stripResetSuffix(basename(currentSessionFile))
+      : undefined;
+    if (baseFromReset && fileSet.has(baseFromReset))
+      return join(sessionsDir, baseFromReset);
 
     // Try canonical session ID file
     const trimmedId = sessionId?.trim();
@@ -271,19 +315,26 @@ async function findPreviousSessionFile(sessionsDir: string, currentSessionFile?:
 
       // Try topic variants
       const topicVariants = files
-        .filter(name => name.startsWith(`${trimmedId}-topic-`) && name.endsWith(".jsonl") && !name.includes(".reset."))
-        .sort().reverse();
+        .filter(
+          (name) =>
+            name.startsWith(`${trimmedId}-topic-`) &&
+            name.endsWith(".jsonl") &&
+            !name.includes(".reset."),
+        )
+        .sort()
+        .reverse();
       if (topicVariants.length > 0) return join(sessionsDir, topicVariants[0]);
     }
 
     // Fallback to most recent non-reset JSONL
     if (currentSessionFile) {
       const nonReset = files
-        .filter(name => name.endsWith(".jsonl") && !name.includes(".reset."))
-        .sort().reverse();
+        .filter((name) => name.endsWith(".jsonl") && !name.includes(".reset."))
+        .sort()
+        .reverse();
       if (nonReset.length > 0) return join(sessionsDir, nonReset[0]);
     }
-  } catch { }
+  } catch {}
 }
 
 // ============================================================================
@@ -293,7 +344,9 @@ async function findPreviousSessionFile(sessionsDir: string, currentSessionFile?:
 function getPluginVersion(): string {
   try {
     const pkgUrl = new URL("./package.json", import.meta.url);
-    const pkg = JSON.parse(readFileSync(pkgUrl, "utf8")) as { version?: string };
+    const pkg = JSON.parse(readFileSync(pkgUrl, "utf8")) as {
+      version?: string;
+    };
     return pkg.version || "unknown";
   } catch {
     return "unknown";
@@ -307,7 +360,8 @@ function getPluginVersion(): string {
 const memoryLanceDBProPlugin = {
   id: "memory-lancedb-pro",
   name: "Memory (LanceDB Pro)",
-  description: "Enhanced LanceDB-backed long-term memory with hybrid retrieval, multi-scope isolation, and management CLI",
+  description:
+    "Enhanced LanceDB-backed long-term memory with hybrid retrieval, multi-scope isolation, and management CLI",
   kind: "memory" as const,
 
   register(api: OpenClawPluginApi) {
@@ -323,13 +377,13 @@ const memoryLanceDBProPlugin = {
     } catch (err) {
       api.logger.warn(
         `memory-lancedb-pro: storage path issue — ${String(err)}\n` +
-        `  The plugin will still attempt to start, but writes may fail.`
+          `  The plugin will still attempt to start, but writes may fail.`,
       );
     }
 
     const vectorDim = getVectorDimensions(
       config.embedding.model || "text-embedding-3-small",
-      config.embedding.dimensions
+      config.embedding.dimensions,
     );
 
     // Initialize core components
@@ -348,13 +402,22 @@ const memoryLanceDBProPlugin = {
       ...DEFAULT_RETRIEVAL_CONFIG,
       ...config.retrieval,
     });
+
+    // Access reinforcement tracker (debounced write-back)
+    const accessTracker = new AccessTracker({
+      store,
+      logger: api.logger,
+      debounceMs: 5000,
+    });
+    retriever.setAccessTracker(accessTracker);
+
     const scopeManager = createScopeManager(config.scopes);
     const migrator = createMigrator(store);
 
     const pluginVersion = getPluginVersion();
 
     api.logger.info(
-      `memory-lancedb-pro@${pluginVersion}: plugin registered (db: ${resolvedDbPath}, model: ${config.embedding.model || "text-embedding-3-small"})`
+      `memory-lancedb-pro@${pluginVersion}: plugin registered (db: ${resolvedDbPath}, model: ${config.embedding.model || "text-embedding-3-small"})`,
     );
 
     // ========================================================================
@@ -372,7 +435,7 @@ const memoryLanceDBProPlugin = {
       },
       {
         enableManagementTools: config.enableManagementTools,
-      }
+      },
     );
 
     // ========================================================================
@@ -387,7 +450,7 @@ const memoryLanceDBProPlugin = {
         migrator,
         embedder,
       }),
-      { commands: ["memory-pro"] }
+      { commands: ["memory-pro"] },
     );
 
     // ========================================================================
@@ -398,7 +461,10 @@ const memoryLanceDBProPlugin = {
     // Default is OFF to prevent the model from accidentally echoing injected context.
     if (config.autoRecall === true) {
       api.on("before_agent_start", async (event, ctx) => {
-        if (!event.prompt || shouldSkipRetrieval(event.prompt, config.autoRecallMinLength)) {
+        if (
+          !event.prompt ||
+          shouldSkipRetrieval(event.prompt, config.autoRecallMinLength)
+        ) {
           return;
         }
 
@@ -411,6 +477,7 @@ const memoryLanceDBProPlugin = {
             query: event.prompt,
             limit: 3,
             scopeFilter: accessibleScopes,
+            source: "auto-recall",
           });
 
           if (results.length === 0) {
@@ -418,11 +485,14 @@ const memoryLanceDBProPlugin = {
           }
 
           const memoryContext = results
-            .map((r) => `- [${r.entry.category}:${r.entry.scope}] ${sanitizeForContext(r.entry.text)} (${(r.score * 100).toFixed(0)}%${r.sources?.bm25 ? ', vector+BM25' : ''}${r.sources?.reranked ? '+reranked' : ''})`)
+            .map(
+              (r) =>
+                `- [${r.entry.category}:${r.entry.scope}] ${sanitizeForContext(r.entry.text)} (${(r.score * 100).toFixed(0)}%${r.sources?.bm25 ? ", vector+BM25" : ""}${r.sources?.reranked ? "+reranked" : ""})`,
+            )
             .join("\n");
 
           api.logger.info?.(
-            `memory-lancedb-pro: injecting ${results.length} memories into context for agent ${agentId}`
+            `memory-lancedb-pro: injecting ${results.length} memories into context for agent ${agentId}`,
           );
 
           return {
@@ -461,7 +531,10 @@ const memoryLanceDBProPlugin = {
 
             const role = msgObj.role;
             const captureAssistant = config.captureAssistant === true;
-            if (role !== "user" && !(captureAssistant && role === "assistant")) {
+            if (
+              role !== "user" &&
+              !(captureAssistant && role === "assistant")
+            ) {
               continue;
             }
 
@@ -501,7 +574,9 @@ const memoryLanceDBProPlugin = {
             const vector = await embedder.embedPassage(text);
 
             // Check for duplicates using raw vector similarity (bypasses importance/recency weighting)
-            const existing = await store.vectorSearch(vector, 1, 0.1, [defaultScope]);
+            const existing = await store.vectorSearch(vector, 1, 0.1, [
+              defaultScope,
+            ]);
 
             if (existing.length > 0 && existing[0].score > 0.95) {
               continue;
@@ -519,7 +594,7 @@ const memoryLanceDBProPlugin = {
 
           if (stored > 0) {
             api.logger.info(
-              `memory-lancedb-pro: auto-captured ${stored} memories for agent ${agentId} in scope ${defaultScope}`
+              `memory-lancedb-pro: auto-captured ${stored} memories for agent ${agentId} in scope ${defaultScope}`,
             );
           }
         } catch (err) {
@@ -544,9 +619,12 @@ const memoryLanceDBProPlugin = {
           api.logger.debug("session-memory: hook triggered for /new command");
 
           const context = (event.context || {}) as Record<string, unknown>;
-          const sessionEntry = (context.previousSessionEntry || context.sessionEntry || {}) as Record<string, unknown>;
+          const sessionEntry = (context.previousSessionEntry ||
+            context.sessionEntry ||
+            {}) as Record<string, unknown>;
           const currentSessionId = sessionEntry.sessionId as string | undefined;
-          let currentSessionFile = (sessionEntry.sessionFile as string) || undefined;
+          let currentSessionFile =
+            (sessionEntry.sessionFile as string) || undefined;
           const source = (context.commandSource as string) || "unknown";
 
           // Resolve session file (handle reset rotation)
@@ -558,10 +636,16 @@ const memoryLanceDBProPlugin = {
             if (workspaceDir) searchDirs.add(join(workspaceDir, "sessions"));
 
             for (const sessionsDir of searchDirs) {
-              const recovered = await findPreviousSessionFile(sessionsDir, currentSessionFile, currentSessionId);
+              const recovered = await findPreviousSessionFile(
+                sessionsDir,
+                currentSessionFile,
+                currentSessionId,
+              );
               if (recovered) {
                 currentSessionFile = recovered;
-                api.logger.debug(`session-memory: recovered session file: ${recovered}`);
+                api.logger.debug(
+                  `session-memory: recovered session file: ${recovered}`,
+                );
                 break;
               }
             }
@@ -573,9 +657,14 @@ const memoryLanceDBProPlugin = {
           }
 
           // Read session content
-          const sessionContent = await readSessionContentWithResetFallback(currentSessionFile, sessionMessageCount);
+          const sessionContent = await readSessionContentWithResetFallback(
+            currentSessionFile,
+            sessionMessageCount,
+          );
           if (!sessionContent) {
-            api.logger.debug("session-memory: no session content found, skipping");
+            api.logger.debug(
+              "session-memory: no session content found, skipping",
+            );
             return;
           }
 
@@ -610,7 +699,9 @@ const memoryLanceDBProPlugin = {
             }),
           });
 
-          api.logger.info(`session-memory: stored session summary for ${currentSessionId || "unknown"}`);
+          api.logger.info(
+            `session-memory: stored session summary for ${currentSessionId || "unknown"}`,
+          );
         } catch (err) {
           api.logger.warn(`session-memory: failed to save: ${String(err)}`);
         }
@@ -628,7 +719,9 @@ const memoryLanceDBProPlugin = {
 
     async function runBackup() {
       try {
-        const backupDir = api.resolvePath(join(resolvedDbPath, "..", "backups"));
+        const backupDir = api.resolvePath(
+          join(resolvedDbPath, "..", "backups"),
+        );
         await mkdir(backupDir, { recursive: true });
 
         const allMemories = await store.list(undefined, undefined, 10000, 0);
@@ -637,28 +730,34 @@ const memoryLanceDBProPlugin = {
         const dateStr = new Date().toISOString().split("T")[0];
         const backupFile = join(backupDir, `memory-backup-${dateStr}.jsonl`);
 
-        const lines = allMemories.map(m => JSON.stringify({
-          id: m.id,
-          text: m.text,
-          category: m.category,
-          scope: m.scope,
-          importance: m.importance,
-          timestamp: m.timestamp,
-          metadata: m.metadata,
-        }));
+        const lines = allMemories.map((m) =>
+          JSON.stringify({
+            id: m.id,
+            text: m.text,
+            category: m.category,
+            scope: m.scope,
+            importance: m.importance,
+            timestamp: m.timestamp,
+            metadata: m.metadata,
+          }),
+        );
 
         await writeFile(backupFile, lines.join("\n") + "\n");
 
         // Keep only last 7 backups
-        const files = (await readdir(backupDir)).filter(f => f.startsWith("memory-backup-") && f.endsWith(".jsonl")).sort();
+        const files = (await readdir(backupDir))
+          .filter((f) => f.startsWith("memory-backup-") && f.endsWith(".jsonl"))
+          .sort();
         if (files.length > 7) {
           const { unlink } = await import("node:fs/promises");
           for (const old of files.slice(0, files.length - 7)) {
-            await unlink(join(backupDir, old)).catch(() => { });
+            await unlink(join(backupDir, old)).catch(() => {});
           }
         }
 
-        api.logger.info(`memory-lancedb-pro: backup completed (${allMemories.length} entries → ${backupFile})`);
+        api.logger.info(
+          `memory-lancedb-pro: backup completed (${allMemories.length} entries → ${backupFile})`,
+        );
       } catch (err) {
         api.logger.warn(`memory-lancedb-pro: backup failed: ${String(err)}`);
       }
@@ -675,10 +774,17 @@ const memoryLanceDBProPlugin = {
         // If embedding/retrieval tests hang (bad network / slow provider), the gateway
         // may never bind its HTTP port, causing restart timeouts.
 
-        const withTimeout = async <T>(p: Promise<T>, ms: number, label: string): Promise<T> => {
+        const withTimeout = async <T>(
+          p: Promise<T>,
+          ms: number,
+          label: string,
+        ): Promise<T> => {
           let timeout: ReturnType<typeof setTimeout> | undefined;
           const timeoutPromise = new Promise<never>((_, reject) => {
-            timeout = setTimeout(() => reject(new Error(`${label} timed out after ${ms}ms`)), ms);
+            timeout = setTimeout(
+              () => reject(new Error(`${label} timed out after ${ms}ms`)),
+              ms,
+            );
           });
           try {
             return await Promise.race([p, timeoutPromise]);
@@ -690,25 +796,39 @@ const memoryLanceDBProPlugin = {
         const runStartupChecks = async () => {
           try {
             // Test components (bounded time)
-            const embedTest = await withTimeout(embedder.test(), 8_000, "embedder.test()");
-            const retrievalTest = await withTimeout(retriever.test(), 8_000, "retriever.test()");
+            const embedTest = await withTimeout(
+              embedder.test(),
+              8_000,
+              "embedder.test()",
+            );
+            const retrievalTest = await withTimeout(
+              retriever.test(),
+              8_000,
+              "retriever.test()",
+            );
 
             api.logger.info(
               `memory-lancedb-pro: initialized successfully ` +
-              `(embedding: ${embedTest.success ? "OK" : "FAIL"}, ` +
-              `retrieval: ${retrievalTest.success ? "OK" : "FAIL"}, ` +
-              `mode: ${retrievalTest.mode}, ` +
-              `FTS: ${retrievalTest.hasFtsSupport ? "enabled" : "disabled"})`
+                `(embedding: ${embedTest.success ? "OK" : "FAIL"}, ` +
+                `retrieval: ${retrievalTest.success ? "OK" : "FAIL"}, ` +
+                `mode: ${retrievalTest.mode}, ` +
+                `FTS: ${retrievalTest.hasFtsSupport ? "enabled" : "disabled"})`,
             );
 
             if (!embedTest.success) {
-              api.logger.warn(`memory-lancedb-pro: embedding test failed: ${embedTest.error}`);
+              api.logger.warn(
+                `memory-lancedb-pro: embedding test failed: ${embedTest.error}`,
+              );
             }
             if (!retrievalTest.success) {
-              api.logger.warn(`memory-lancedb-pro: retrieval test failed: ${retrievalTest.error}`);
+              api.logger.warn(
+                `memory-lancedb-pro: retrieval test failed: ${retrievalTest.error}`,
+              );
             }
           } catch (error) {
-            api.logger.warn(`memory-lancedb-pro: startup checks failed: ${String(error)}`);
+            api.logger.warn(
+              `memory-lancedb-pro: startup checks failed: ${String(error)}`,
+            );
           }
         };
 
@@ -719,7 +839,15 @@ const memoryLanceDBProPlugin = {
         setTimeout(() => void runBackup(), 60_000); // 1 min after start
         backupTimer = setInterval(() => void runBackup(), BACKUP_INTERVAL_MS);
       },
-      stop: () => {
+      stop: async () => {
+        // Flush pending access reinforcement data before shutdown
+        try {
+          await accessTracker.flush();
+        } catch (err) {
+          api.logger.warn("memory-lancedb-pro: flush failed on stop:", err);
+        }
+        accessTracker.destroy();
+
         if (backupTimer) {
           clearInterval(backupTimer);
           backupTimer = null;
@@ -728,7 +856,6 @@ const memoryLanceDBProPlugin = {
       },
     });
   },
-
 };
 
 function parsePluginConfig(value: unknown): PluginConfig {
@@ -743,28 +870,59 @@ function parsePluginConfig(value: unknown): PluginConfig {
   }
 
   // Accept single key (string) or array of keys for round-robin rotation
-  const apiKey: string | string[] = typeof embedding.apiKey === "string"
-    ? embedding.apiKey
-    : Array.isArray(embedding.apiKey) && embedding.apiKey.length > 0 && embedding.apiKey.every((k: unknown) => typeof k === "string")
-      ? (embedding.apiKey as string[])
-      : process.env.OPENAI_API_KEY || "";
+  let apiKey: string | string[];
+  if (typeof embedding.apiKey === "string") {
+    apiKey = embedding.apiKey;
+  } else if (Array.isArray(embedding.apiKey) && embedding.apiKey.length > 0) {
+    // Validate every element is a non-empty string
+    const invalid = embedding.apiKey.findIndex(
+      (k: unknown) => typeof k !== "string" || (k as string).trim().length === 0,
+    );
+    if (invalid !== -1) {
+      throw new Error(
+        `embedding.apiKey[${invalid}] is invalid: expected non-empty string`,
+      );
+    }
+    apiKey = embedding.apiKey as string[];
+  } else if (embedding.apiKey !== undefined) {
+    // apiKey is present but wrong type — throw, don't silently fall back
+    throw new Error("embedding.apiKey must be a string or non-empty array of strings");
+  } else {
+    apiKey = process.env.OPENAI_API_KEY || "";
+  }
 
   if (!apiKey || (Array.isArray(apiKey) && apiKey.length === 0)) {
     throw new Error("embedding.apiKey is required (set directly or via OPENAI_API_KEY env var)");
   }
+  }
 
   return {
     embedding: {
       provider: "openai-compatible",
       apiKey,
-      model: typeof embedding.model === "string" ? embedding.model : "text-embedding-3-small",
-      baseURL: typeof embedding.baseURL === "string" ? resolveEnvVars(embedding.baseURL) : undefined,
+      model:
+        typeof embedding.model === "string"
+          ? embedding.model
+          : "text-embedding-3-small",
+      baseURL:
+        typeof embedding.baseURL === "string"
+          ? resolveEnvVars(embedding.baseURL)
+          : undefined,
       // Accept number, numeric string, or env-var string (e.g. "${EMBED_DIM}").
       // Also accept legacy top-level `dimensions` for convenience.
       dimensions: parsePositiveInt(embedding.dimensions ?? cfg.dimensions),
-      taskQuery: typeof embedding.taskQuery === "string" ? embedding.taskQuery : undefined,
-      taskPassage: typeof embedding.taskPassage === "string" ? embedding.taskPassage : undefined,
-      normalized: typeof embedding.normalized === "boolean" ? embedding.normalized : undefined,
+      taskQuery:
+        typeof embedding.taskQuery === "string"
+          ? embedding.taskQuery
+          : undefined,
+      taskPassage:
+        typeof embedding.taskPassage === "string"
+          ? embedding.taskPassage
+          : undefined,
+      normalized:
+        typeof embedding.normalized === "boolean"
+          ? embedding.normalized
+          : undefined,
     },
     dbPath: typeof cfg.dbPath === "string" ? cfg.dbPath : undefined,
     autoCapture: cfg.autoCapture !== false,
@@ -772,17 +930,28 @@ function parsePluginConfig(value: unknown): PluginConfig {
     autoRecall: cfg.autoRecall === true,
     autoRecallMinLength: parsePositiveInt(cfg.autoRecallMinLength),
     captureAssistant: cfg.captureAssistant === true,
-    retrieval: typeof cfg.retrieval === "object" && cfg.retrieval !== null ? cfg.retrieval as any : undefined,
-    scopes: typeof cfg.scopes === "object" && cfg.scopes !== null ? cfg.scopes as any : undefined,
+    retrieval:
+      typeof cfg.retrieval === "object" && cfg.retrieval !== null
+        ? (cfg.retrieval as any)
+        : undefined,
+    scopes:
+      typeof cfg.scopes === "object" && cfg.scopes !== null
+        ? (cfg.scopes as any)
+        : undefined,
     enableManagementTools: cfg.enableManagementTools === true,
-    sessionMemory: typeof cfg.sessionMemory === "object" && cfg.sessionMemory !== null
-      ? {
-        enabled: (cfg.sessionMemory as Record<string, unknown>).enabled !== false,
-        messageCount: typeof (cfg.sessionMemory as Record<string, unknown>).messageCount === "number"
-          ? (cfg.sessionMemory as Record<string, unknown>).messageCount as number
-          : undefined,
-      }
-      : undefined,
+    sessionMemory:
+      typeof cfg.sessionMemory === "object" && cfg.sessionMemory !== null
+        ? {
+            enabled:
+              (cfg.sessionMemory as Record<string, unknown>).enabled !== false,
+            messageCount:
+              typeof (cfg.sessionMemory as Record<string, unknown>)
+                .messageCount === "number"
+                ? ((cfg.sessionMemory as Record<string, unknown>)
+                    .messageCount as number)
+                : undefined,
+          }
+        : undefined,
   };
 }