chore(release): release v2.35.0

package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-mem",
-  "version": "2.34.0",
+  "version": "2.35.0",
   "description": "OpenCode plugin that gives coding agents persistent memory using local Turso/libSQL vector search",
   "type": "module",
   "main": "dist/plugin.js",

tests/add-memory-phase-logging.test.ts

@@ -38,6 +38,7 @@ mock.module(${JSON.stringify(configUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: (message, data) => logs.push({ message, data }),
+  safeLog: (message, data) => logs.push({ message, data }),
   serializeError: (error) => ({
     error: error instanceof Error ? error.message : String(error),
     name: error instanceof Error ? error.name : undefined,

tests/ai-cleanup.test.ts

@@ -120,7 +120,7 @@ mock.module(${JSON.stringify(configUrl)}, () => ({
   },
 }));
 
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 
 mock.module(${JSON.stringify(opencodeProviderLoaderUrl)}, () => ({
   loadOpencodeProvider: async () => ({

tests/api-handlers-container-tag-traversal.test.ts

@@ -74,6 +74,7 @@ mock.module(${JSON.stringify(userPromptManagerUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/api-handlers-profile-strip.test.ts

@@ -58,6 +58,7 @@ const profileWithEmbeddings = ${JSON.stringify(profileWithEmbeddings)};
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/auto-capture-concurrency.test.ts

@@ -89,7 +89,7 @@ mock.module(${JSON.stringify(u("src/services/context.js"))}, () => ({ formatCont
 mock.module(${JSON.stringify(u("src/services/privacy.js"))}, () => ({
   stripPrivateContent: (v) => v, isFullyPrivate: () => false,
 }));
-mock.module(${JSON.stringify(u("src/services/logger.js"))}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(u("src/services/logger.js"))}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(u("src/services/language-detector.js"))}, () => ({
   detectLanguage: () => "en", getLanguageName: () => "English",
 }));

tests/auto-capture-config-toast.test.ts

@@ -60,6 +60,7 @@ mock.module(${JSON.stringify(configUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: (message) => logs.push(message),
+  safeLog: () => {},
 }));
 
 const { notifyAutoCaptureProviderStatus } = await import(${JSON.stringify(statusUrl)});

tests/auto-capture-provider-order.test.ts

@@ -112,6 +112,7 @@ mock.module(${JSON.stringify(promptManagerUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: (message, data) => logs.push({ message, data }),
+  safeLog: (message, data) => logs.push({ message, data }),
   serializeError: (error) => ({
     error: error instanceof Error ? error.message : String(error),
     name: error instanceof Error ? error.name : undefined,

tests/auto-capture.test.ts

@@ -150,7 +150,7 @@ mock.module(${JSON.stringify(promptManagerUrl)}, () => ({
   },
 }));
 
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(languageUrl)}, () => ({
   detectLanguage: () => "en",
   getLanguageName: () => "English",
@@ -279,7 +279,7 @@ mock.module(${JSON.stringify(promptManagerUrl)}, () => ({
   },
 }));
 
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(languageUrl)}, () => ({
   detectLanguage: () => "en",
   getLanguageName: () => "English",
@@ -430,7 +430,7 @@ mock.module(${JSON.stringify(promptManagerUrl)}, () => ({
   },
 }));
 
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(languageUrl)}, () => ({
   detectLanguage: () => "en",
   getLanguageName: () => "English",

tests/cleanup-ai-sessions.test.ts

@@ -46,6 +46,7 @@ mock.module(${JSON.stringify(configUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: String(error) }),
 }));
 

tests/client-scope-traversal.test.ts

@@ -73,6 +73,7 @@ mock.module(${JSON.stringify(configUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/compaction-agent-preservation.test.ts

@@ -198,7 +198,7 @@ mock.module(${JSON.stringify(webServerUrl)}, () => ({
   startWebServer: async () => null,
   WebServer: class {},
 }));
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(languageUrl)}, () => ({ getLanguageName: () => "English" }));
 
 const mockClient = {

tests/fixtures/profile-learning-lock-worker.mts

@@ -81,7 +81,7 @@ mock.module(configUrl, () => ({
   initConfig: () => {},
   isConfigured: () => true,
 }));
-mock.module(loggerUrl, () => ({ log: () => {} }));
+mock.module(loggerUrl, () => ({ log: () => {}, safeLog: () => {} }));
 
 // current-boot-invalid: the LOCAL machine's boot identity reads as garbage.
 // The implementation must treat the local boot id as "unknown" and never use

tests/fixtures/profile-learning-memory-writer.mjs

@@ -74,6 +74,7 @@ mock.module(configUrl, () => ({
 const loggerUrl = new URL("../../src/services/logger.js", import.meta.url).href;
 mock.module(loggerUrl, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/fixtures/profile-learning-service-worker.mjs

@@ -100,6 +100,7 @@ mock.module(tagsUrl, () => ({
 // off the lock rather than silently doing nothing.
 mock.module(loggerUrl, () => ({
   log: (label, data) => event("log", { label, data }),
+  safeLog: (label, data) => event("log", { label, data }),
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/fixtures/profile-learning-storage-error-worker.mjs

@@ -127,6 +127,7 @@ mock.module(tagsUrl, () => ({
 
 mock.module(loggerUrl, () => ({
   log: (label, data) => event("log", { label, data }),
+  safeLog: (label, data) => event("log", { label, data }),
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/fixtures/v2-provider-refresh.mjs

@@ -142,6 +142,8 @@ mock.module(moduleUrl("services/user-prompt/user-prompt-manager"), () => ({
 mock.module(moduleUrl("services/logger"), () => ({
   log: (...args) =>
     logs.push(args.map((a) => (typeof a === "string" ? a : JSON.stringify(a))).join(" ")),
+  safeLog: (...args) =>
+    logs.push(args.map((a) => (typeof a === "string" ? a : JSON.stringify(a))).join(" ")),
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/memory-portability-tool.test.ts

@@ -93,7 +93,7 @@ mock.module(${JSON.stringify(webServerUrl)}, () => ({
   startWebServer: async () => null,
   WebServer: class {},
 }));
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(languageUrl)}, () => ({ getLanguageName: () => "English" }));
 
 const { OpenCodeMemPlugin } = await import(${JSON.stringify(indexUrl)});

tests/profile-learning-idle.test.ts

@@ -88,7 +88,7 @@ mock.module(${JSON.stringify(privacyUrl)}, () => ({
   isFullyPrivate: () => false,
 }));
 mock.module(${JSON.stringify(promptManagerUrl)}, () => ({ userPromptManager: { savePrompt() {} } }));
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(languageUrl)}, () => ({ getLanguageName: () => "English" }));
 // The web server only starts once the Turso readiness gate passes, and that
 // gate is what decides whether an owner exists at all.

tests/tag-migration-opencode-provider.test.ts

@@ -116,6 +116,7 @@ mock.module(${JSON.stringify(userPromptManagerUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/tag-migration-skip-tagged.test.ts

@@ -110,6 +110,7 @@ mock.module(${JSON.stringify(userPromptManagerUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/tag-migration-soft-failure.test.ts

@@ -104,6 +104,7 @@ mock.module(${JSON.stringify(userPromptManagerUrl)}, () => ({
 
 mock.module(${JSON.stringify(loggerUrl)}, () => ({
   log: () => {},
+  safeLog: () => {},
   serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }),
 }));
 

tests/tool-scope.test.ts

@@ -96,7 +96,7 @@ mock.module(${JSON.stringify(webServerUrl)}, () => ({
   startWebServer: async () => null,
   WebServer: class {},
 }));
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 mock.module(${JSON.stringify(languageUrl)}, () => ({ getLanguageName: () => "English" }));
 
 const { OpenCodeMemPlugin } = await import(${JSON.stringify(indexUrl)});

tests/turso-wal-index-recovery.test.ts

@@ -162,27 +162,30 @@ describe("turso WAL index recovery", () => {
       try {
         const db = await manager.getConnection(dbPath);
         values = (await db.all<{ v: string }>("SELECT v FROM t ORDER BY id")).map((row) => row.v);
+
+        // Assert while the connection is still open: closeAll runs
+        // PRAGMA wal_checkpoint(TRUNCATE) and empties the -wal afterward.
+        expect(values).toEqual(["checkpointed", "wal-only"]);
+        expect(existsSync(walPath)).toBe(true);
+        expect(statSync(walPath).size).toBeGreaterThan(0);
+        // Same inode: the file was kept, not moved and recreated.
+        expect(statSync(walPath).ino).toBe(walInode);
+
+        const [quarantined] = quarantineDirs(baseDir);
+        const quarantinedFiles = readdirSync(
+          join(baseDir, "recovery", quarantined as string)
+        ).sort();
+        expect(quarantinedFiles).toContain("memories.db-tshm");
+        expect(quarantinedFiles.every((name) => /-(shm|tshm)$/.test(name))).toBe(true);
+        // Engine evidence, not an assumption: the old -tshm is in quarantine *and*
+        // a fresh, non-empty index exists beside the WAL again, rebuilt on reopen.
+        // The seeded WAL has no torn trailing frame; a torn WAL is the
+        // "retry fails, original error surfaces, WAL untouched" case.
+        expect(existsSync(`${dbPath}-tshm`)).toBe(true);
+        expect(statSync(`${dbPath}-tshm`).size).toBeGreaterThan(0);
       } finally {
         await manager.closeAll();
       }
-
-      // The WAL was not discarded, and its committed row survived the recovery.
-      expect(values).toEqual(["checkpointed", "wal-only"]);
-      expect(existsSync(walPath)).toBe(true);
-      expect(statSync(walPath).size).toBeGreaterThan(0);
-      // Same inode: the file was kept, not moved and recreated.
-      expect(statSync(walPath).ino).toBe(walInode);
-
-      const [quarantined] = quarantineDirs(baseDir);
-      const quarantinedFiles = readdirSync(join(baseDir, "recovery", quarantined as string)).sort();
-      expect(quarantinedFiles).toContain("memories.db-tshm");
-      expect(quarantinedFiles.every((name) => /-(shm|tshm)$/.test(name))).toBe(true);
-      // Engine evidence, not an assumption: the old -tshm is in quarantine *and*
-      // a fresh, non-empty index exists beside the WAL again, rebuilt on reopen.
-      // The seeded WAL has no torn trailing frame; a torn WAL is the
-      // "retry fails, original error surfaces, WAL untouched" case.
-      expect(existsSync(`${dbPath}-tshm`)).toBe(true);
-      expect(statSync(`${dbPath}-tshm`).size).toBeGreaterThan(0);
     }
   );
 

tests/user-profile-learning-error.test.ts

@@ -89,7 +89,7 @@ mock.module(${JSON.stringify(profileManagerUrl)}, () => ({
   },
 }));
 
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 
 mock.module(${JSON.stringify(opencodeProviderLoaderUrl)}, () => ({
   loadOpencodeProvider: async () => ({

tests/user-profile-learning-language.test.ts

@@ -105,7 +105,7 @@ mock.module(${JSON.stringify(profileManagerUrl)}, () => ({
   },
 }));
 
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 
 let capturedSystemPrompt = null;
 let capturedUserPrompt = null;

tests/user-profile-learning-prompt-mode.test.ts

@@ -113,7 +113,7 @@ mock.module(${JSON.stringify(profileManagerUrl)}, () => ({
   },
 }));
 
-mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
+mock.module(${JSON.stringify(loggerUrl)}, () => ({ log: () => {}, safeLog: () => {}, serializeError: (error) => ({ error: error instanceof Error ? error.message : String(error) }) }));
 
 let structuredSystemPrompt = null;
 let externalSystemPrompt = null;