clawmem 0.10.4 → 0.10.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clawmem",
-  "version": "0.10.4",
+  "version": "0.10.5",
   "description": "On-device memory layer for AI agents. Claude Code, OpenClaw, and Hermes. Hooks + MCP server + hybrid RAG search.",
   "type": "module",
   "bin": {

package/src/openclaw/index.ts

@@ -37,8 +37,8 @@
  *   4. REST API service (`clawmem serve`) lifecycle — unchanged.
  *
  * §14.3 critical correctness contract: `agent_end` is fire-and-forget at
- * `attempt.ts:3379-3402`. Precompact-extract MUST run inside
- * `handleBeforePromptBuild` (which IS awaited at `attempt.ts:2610`), gated
+ * `attempt.ts:3870-3892`. Precompact-extract MUST run inside
+ * `handleBeforePromptBuild` (which IS awaited at `attempt.ts:2973`), gated
  * by the proximity heuristic in `compaction-threshold.ts`. See `engine.ts`
  * top-of-file comment for the full rationale.
  */
@@ -161,7 +161,7 @@ const clawmemPlugin = {
     // ----- Plugin Hook: before_prompt_build (AWAITED — load-bearing path) -----
     // Both context-surfacing retrieval injection and pre-emptive precompact
     // extraction live here. handleBeforePromptBuild is async and the OpenClaw
-    // attempt path awaits the result at attempt.ts:2610 before building the
+    // attempt path awaits the result at attempt.ts:2973 before building the
     // effective prompt. precompact-extract therefore runs strictly before
     // the LLM call that could trigger compaction on this turn.
     api.on(
@@ -175,7 +175,7 @@ const clawmemPlugin = {
     // ----- Plugin Hook: agent_end (FIRE-AND-FORGET in core) -----
     // Decision-extractor, handoff-generator, and feedback-loop run here.
     // These writes are eventually-consistent (saveMemory dedupes), so the
-    // fire-and-forget context at attempt.ts:3379-3402 is acceptable.
+    // fire-and-forget context at attempt.ts:3870-3892 is acceptable.
     // OpenClaw v2026.4.26+ also enforces a 30s default void-hook timeout
     // (DEFAULT_VOID_HOOK_TIMEOUT_MS_BY_HOOK in src/plugins/hooks.ts) — a
     // timed-out handler is logged but our underlying postrun work is not

package/src/store.ts

@@ -298,13 +298,20 @@ if (process.platform === "darwin") {
 }
 
 function initializeDatabase(db: Database): void {
+  // Set busy_timeout FIRST so subsequent PRAGMAs (journal_mode in particular,
+  // which acquires a write lock when switching or initializing WAL state) wait
+  // instead of returning SQLITE_BUSY when concurrent Stop hooks
+  // (decision-extractor, handoff-generator, feedback-loop) — and the parallel
+  // before_reset hook fan-out in src/openclaw/engine.ts — open the DB
+  // simultaneously. busy_timeout is a connection-level setting that only
+  // governs *subsequent* statements (default busy handler is NULL → SQLITE_BUSY
+  // returns immediately), so it must precede the contending PRAGMAs. 15s is
+  // well within the 30s Stop hook timeout. createStore() resets to operational
+  // value (5000ms or opts.busyTimeout) after DDL completes. Issue #13.
+  db.exec("PRAGMA busy_timeout = 15000");
   sqliteVec.load(db);
   db.exec("PRAGMA journal_mode = WAL");
   db.exec("PRAGMA foreign_keys = ON");
-  // Set generous busy_timeout during DDL — concurrent Stop hooks (decision-extractor,
-  // handoff-generator, feedback-loop) all run initializeDatabase simultaneously.
-  // 15s is well within the 30s Stop hook timeout. Reset to normal after DDL completes.
-  db.exec("PRAGMA busy_timeout = 15000");
 
   // Drop legacy tables that are now managed in YAML
   db.exec(`DROP TABLE IF EXISTS path_contexts`);
@@ -1123,13 +1130,20 @@ export function createStore(dbPath?: string, opts?: { readonly?: boolean; busyTi
   if (!opts?.readonly) {
     initializeDatabase(db);
   } else {
-    // Readonly: load sqlite-vec extension and set WAL mode pragma only
+    // Readonly: set busy_timeout FIRST so the journal_mode PRAGMA below
+    // doesn't race when concurrent processes open the DB. PRAGMA
+    // journal_mode=WAL can contend when switching or initializing WAL
+    // state, even on readonly handles. Public-API hardening — no
+    // production caller in this repo currently passes readonly:true,
+    // but the ordering invariant should hold regardless. Issue #13.
+    db.exec(`PRAGMA busy_timeout = ${opts?.busyTimeout ?? 5000}`);
     sqliteVec.load(db);
     db.exec("PRAGMA journal_mode = WAL");
     db.exec("PRAGMA query_only = ON");
   }
-  // Reset busy_timeout to operational value after DDL init (which uses 15s).
-  // Default 5000ms for normal operations — callers can override via opts.
+  // For the writable branch: initializeDatabase() set 15000 during DDL —
+  // reset to operational value here. For readonly: already set inside the
+  // branch above; this assignment is a no-op rewrite to the same value.
   db.exec(`PRAGMA busy_timeout = ${opts?.busyTimeout ?? 5000}`);
 
   return {