claude-mem-lite 2.33.1 → 2.33.3

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "2.33.1",
+      "version": "2.33.3",
       "source": "./",
       "description": "Lightweight persistent memory system for Claude Code — FTS5 search, episode batching, error-triggered recall"
     }

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.33.1",
+  "version": "2.33.3",
   "description": "Lightweight persistent memory system for Claude Code — FTS5 search, episode batching, error-triggered recall",
   "author": {
     "name": "sdsrss"

package/hook-handoff.mjs

@@ -16,13 +16,21 @@ import * as taskReaderModule from './lib/task-reader.mjs';
 /**
  * Build and save a handoff snapshot to session_handoffs table.
  * Called synchronously during handleStop (/exit) or handleSessionStart (/clear).
+ *
+ * Dual id: `sessionId` is the mem-internal id that user_prompts / observations
+ * were written with (handleUserPrompt uses getSessionId()) — it drives all
+ * DB lookups. `scopeSessionId` is the CC UUID from hook stdin used to scope
+ * the stored row so parallel CC sessions don't clobber each other. When
+ * `scopeSessionId` is null/undefined, `sessionId` is used for both (legacy).
+ *
  * @param {Database} db Opened main database
- * @param {string} sessionId Session being handed off
+ * @param {string} sessionId Mem-internal session id (query key)
  * @param {string} project Project identifier
  * @param {'clear'|'exit'} type Handoff type
  * @param {object|null} episodeSnapshot Episode buffer captured before flushing
+ * @param {string|null} [scopeSessionId=null] CC UUID for session_handoffs.session_id column
  */
-export function buildAndSaveHandoff(db, sessionId, project, type, episodeSnapshot) {
+export function buildAndSaveHandoff(db, sessionId, project, type, episodeSnapshot, scopeSessionId = null) {
   // 1. Working objective — from user prompts
   const prompts = db.prepare(`
     SELECT prompt_text FROM user_prompts
@@ -122,6 +130,9 @@ export function buildAndSaveHandoff(db, sessionId, project, type, episodeSnapsho
 
   // UPSERT keyed on (project, type, session_id) — parallel sessions coexist.
   // Same session re-writing its own handoff (e.g. repeated /clear) updates in place.
+  // `scopeSessionId` (CC UUID) tags the row for parallel scoping; falls back to
+  // the mem-internal `sessionId` when the caller didn't supply one (tests + legacy).
+  const storedSessionId = scopeSessionId || sessionId;
   db.prepare(`
     INSERT INTO session_handoffs (project, type, session_id, working_on, completed, unfinished, key_files, key_decisions, match_keywords, created_at_epoch, git_sha_at_handoff)
     VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
@@ -135,7 +146,7 @@ export function buildAndSaveHandoff(db, sessionId, project, type, episodeSnapsho
       created_at_epoch = excluded.created_at_epoch,
       git_sha_at_handoff = excluded.git_sha_at_handoff
   `).run(
-    project, type, sessionId,
+    project, type, storedSessionId,
     truncate(workingOn, 1000),
     completed.map(c => `[${c.type}] ${c.title}`).join('\n'),
     unfinished.length > 3000 ? unfinished.slice(0, 2999) + '…' : unfinished,

package/hook.mjs

@@ -3,6 +3,20 @@
 // Selective encoding, episodic batching, error-triggered recall
 // Hooks (fast <100ms): post-tool-use, session-start, stop
 // Background workers (slow): llm-episode, llm-summary
+//
+// ─── Session-id invariant (do not violate — see bf121aa / v2.33.2) ──────────
+// Two session identifiers coexist in this codebase:
+//   • mem-internal id: `hook-<project>-<hash>`, produced by getSessionId().
+//     handleUserPrompt writes it into user_prompts / sdk_sessions.content_session_id
+//     / observations.memory_session_id. Treat as the ONLY valid WHERE / JOIN key
+//     for those three tables.
+//   • CC UUID: `hookData.session_id` from stdin. Use ONLY for
+//     session_handoffs.session_id (parallel-session scoping, per bf121aa).
+// Mixing them silently breaks everything — UPDATE matches 0 rows, SELECT returns
+// empty, buildAndSaveHandoff early-returns, no throw. Precedent: v2.33.1 shipped
+// with the two mixed since 2026-04-12; 48 stale 'active' sessions + 0 handoffs
+// for projects--mem went unnoticed for 4 days. Keep the split or document why
+// you're changing it.
 
 import { randomUUID } from 'crypto';
 import { join } from 'path';
@@ -95,7 +109,11 @@ if (!event) process.exit(0);
 
 // ─── Episode Flush ──────────────────────────────────────────────────────────
 
-function flushEpisode(episode) {
+// hookEventName defaults to 'PostToolUse' since that's the most common caller.
+// Stop / SessionStart callers MUST pass their own event name — CC rejects
+// hook output whose hookEventName doesn't match the triggering event
+// (regression introduced in v2.33.1's structured receipt, fixed in v2.33.3).
+function flushEpisode(episode, hookEventName = 'PostToolUse') {
   if (!episode || episode.entries.length === 0) return;
 
   // Collect Read file paths tracked by post-tool-use.sh
@@ -160,7 +178,7 @@ function flushEpisode(episode) {
       process.stdout.write(JSON.stringify({
         suppressOutput: true,
         hookSpecificOutput: {
-          hookEventName: 'PostToolUse',
+          hookEventName,
           additionalContext: lines.join('\n'),
         },
       }));
@@ -326,9 +344,12 @@ async function handleStop() {
     }
   } catch { /* stdin unavailable — fall back to local session id */ }
 
-  // Capture session info BEFORE cleanup. Prefer CC session id (parallel-safe);
-  // fall back to file-based id for backward compat.
-  const sessionId = ccSessionId || getSessionId();
+  // Capture session info BEFORE cleanup. All DB lookups use the mem-internal id
+  // (that's what handleUserPrompt wrote into user_prompts / sdk_sessions / observations
+  // via getSessionId()). `ccSessionId` is used only to tag session_handoffs rows
+  // for parallel-session scoping — it must not be used as a query key, otherwise
+  // queries miss and UPDATE sdk_sessions becomes a no-op (v2.33.2 regression fix).
+  const sessionId = getSessionId();
   const project = inferProject();
 
   // Snapshot episode BEFORE flush for handoff extraction
@@ -339,7 +360,7 @@ async function handleStop() {
     try {
       const episode = readEpisode();
       if (episode) {
-        flushEpisode(episode);
+        flushEpisode(episode, 'Stop');
       }
     } finally {
       releaseLock();
@@ -381,8 +402,11 @@ async function handleStop() {
         UPDATE sdk_sessions SET status = 'completed', completed_at = ?, completed_at_epoch = ?
         WHERE content_session_id = ? AND status = 'active'
       `).run(new Date().toISOString(), Date.now(), sessionId);
-      // Save handoff snapshot for cross-session continuity
-      try { buildAndSaveHandoff(db, sessionId, project, 'exit', episodeSnapshot); }
+      // Save handoff snapshot for cross-session continuity.
+      // sessionId = mem-internal (query key); ccSessionId = CC UUID (scope key for
+      // parallel-safe row identity). Without the split, CC UUID-based queries miss
+      // user_prompts and the handoff row is silently skipped (see hook-handoff.mjs).
+      try { buildAndSaveHandoff(db, sessionId, project, 'exit', episodeSnapshot, ccSessionId || sessionId); }
       catch (e) { debugCatch(e, 'handleStop-handoff'); }
 
       // Fast summary baseline — ensures summary exists even if background LLM fails
@@ -489,7 +513,7 @@ async function handleSessionStart() {
     try {
       const prevEpisode = readEpisode();
       if (prevEpisode && prevEpisode.entries && prevEpisode.entries.length > 0) {
-        flushEpisode(prevEpisode);
+        flushEpisode(prevEpisode, 'SessionStart');
       }
     } finally {
       releaseLock();
@@ -664,12 +688,12 @@ async function handleSessionStart() {
 
     if (prevSessionId) {
       // Save handoff for cross-session continuity (/clear or /compact).
-      // Prefer CC session id (stable across /clear within same CC session, and
-      // unique across parallel sessions for the same project) so UserPromptSubmit
-      // can scope by hookData.session_id. Fall back to the mem plugin's file-based
-      // id for legacy/test paths.
-      const handoffSessionId = ccSessionId || prevSessionId;
-      try { buildAndSaveHandoff(db, handoffSessionId, prevProject || project, 'clear', episodeSnapshot); }
+      // prevSessionId is the mem-internal id — use it to look up the finished session's
+      // user_prompts / observations. ccSessionId (same CC session across /clear) scopes
+      // the stored row so UserPromptSubmit can read its own handoff back.
+      // Legacy/test paths (no stdin) fall back to prevSessionId for both.
+      const handoffScopeId = ccSessionId || prevSessionId;
+      try { buildAndSaveHandoff(db, prevSessionId, prevProject || project, 'clear', episodeSnapshot, handoffScopeId); }
       catch (e) { debugCatch(e, 'session-start-handoff'); }
 
       // Read the just-saved handoff for downstream consumers (fast summary remaining, working state).
@@ -677,7 +701,7 @@ async function handleSessionStart() {
       try {
         prevClearHandoff = db.prepare(
           'SELECT working_on, unfinished, key_files FROM session_handoffs WHERE project = ? AND type = ? AND session_id = ?'
-        ).get(prevProject || project, 'clear', handoffSessionId);
+        ).get(prevProject || project, 'clear', handoffScopeId);
       } catch {}
 
       // Generate session summary for previous session (background Haiku — richer version)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.33.1",
+  "version": "2.33.3",
   "description": "Lightweight persistent memory system for Claude Code",
   "type": "module",
   "engines": {