claude-mem-lite 3.9.0 → 3.9.1

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.9.0",
+      "version": "3.9.1",
       "source": "./",
       "description": "Persistent long-term memory for Claude Code via MCP — captures coding decisions, bugfixes, and context across sessions. Hybrid FTS5 + TF-IDF search with episode batching. Single SQLite DB, no external services. A lighter, lower-cost alternative to claude-mem (episode batching + a smaller model; cost savings are an internal estimate, not a measured benchmark)."
     }

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.9.0",
+  "version": "3.9.1",
   "description": "Persistent long-term memory for Claude Code via MCP — captures coding decisions, bugfixes, and context across sessions. Hybrid FTS5 + TF-IDF search with episode batching. Single SQLite DB, no external services. A lighter, lower-cost alternative to claude-mem (episode batching + a smaller model; cost savings are an internal estimate, not a measured benchmark).",
   "author": {
     "name": "sdsrss"

package/hook.mjs

@@ -553,7 +553,11 @@ async function handleStop() {
           // Union closed by extractAllInjected — one integration point so the
           // contract test in tests/citation-tracker-userprompt.test.mjs covers it.
           try {
-            const injected = extractAllInjected(transcriptPath);
+            // mainOnly: the injected denominator must use the same thread
+            // filter as citedMain (the numerator, below) — an obs injected only
+            // inside a subagent (sidechain) would otherwise enter the denominator
+            // but never the numerator and streak-demote despite being used there.
+            const injected = extractAllInjected(transcriptPath, { mainOnly: true });
             // P5 ①: cite-back signals — observations whose warned file the agent
             // edited this session. Union into injected so they're resolved (they
             // were injected via pre-tool-recall) and, below, into cited so the

package/lib/citation-tracker.mjs

@@ -182,8 +182,15 @@ function normalizeHookCommand(command) {
  *
  * @param {string|null|undefined} transcriptPath
  * @param {(ctx: {command: string, text: string}) => void} fn
+ * @param {object} [opts]
+ * @param {boolean} [opts.mainOnly=false] If true, skip attachments on sidechain
+ *   (subagent) transcript records. Mirrors extractCitationsFromTranscript's
+ *   mainOnly so the citation-decay injected DENOMINATOR uses the same thread
+ *   filter as the cited NUMERATOR — an obs injected only inside a subagent must
+ *   not enter the denominator, else it streak-demotes despite being used there.
  */
-function eachHookAttachment(transcriptPath, fn) {
+function eachHookAttachment(transcriptPath, fn, opts = {}) {
+  const { mainOnly = false } = opts;
   if (!transcriptPath || !existsSync(transcriptPath)) return;
   let raw;
   try { raw = readFileSync(transcriptPath, 'utf8'); } catch { return; }
@@ -192,6 +199,7 @@ function eachHookAttachment(transcriptPath, fn) {
     let entry;
     try { entry = JSON.parse(line); } catch { continue; }
     if (entry.type !== 'attachment') continue;
+    if (mainOnly && entry.isSidechain === true) continue;
     const att = entry.attachment;
     if (!att || att.type !== 'hook_success') continue;
     const stdout = att.stdout || '';
@@ -217,14 +225,14 @@ function eachHookAttachment(transcriptPath, fn) {
  * @param {string|null|undefined} transcriptPath
  * @returns {Set<number>} unique injected IDs (empty set on missing path/file)
  */
-export function extractInjectedFromPreToolUse(transcriptPath) {
+export function extractInjectedFromPreToolUse(transcriptPath, opts = {}) {
   const ids = new Set();
   eachHookAttachment(transcriptPath, ({ command, text }) => {
     if (!command.includes('pre-tool-recall')) return;
     INJECTED_RE.lastIndex = 0;
     let m;
     while ((m = INJECTED_RE.exec(text))) addObsId(ids, m[1]);
-  });
+  }, opts);
   return ids;
 }
 
@@ -251,7 +259,7 @@ const UPS_COMMAND_SUFFIX = 'hook.mjs user-prompt';
  * @param {string|null|undefined} transcriptPath
  * @returns {Set<number>}
  */
-export function extractInjectedFromUserPromptSubmit(transcriptPath) {
+export function extractInjectedFromUserPromptSubmit(transcriptPath, opts = {}) {
   const ids = new Set();
   eachHookAttachment(transcriptPath, ({ command, text }) => {
     if (!command.includes(UPS_COMMAND_SUFFIX)) return;
@@ -265,7 +273,7 @@ export function extractInjectedFromUserPromptSubmit(transcriptPath) {
       if (matches.length === 0) continue;
       addObsId(ids, matches[matches.length - 1][1]);
     }
-  });
+  }, opts);
   return ids;
 }
 
@@ -280,7 +288,7 @@ export function extractInjectedFromUserPromptSubmit(transcriptPath) {
  * @param {string|null|undefined} transcriptPath
  * @returns {Set<number>}
  */
-export function extractInjectedFromErrorRecall(transcriptPath) {
+export function extractInjectedFromErrorRecall(transcriptPath, opts = {}) {
   const ids = new Set();
   eachHookAttachment(transcriptPath, ({ command, text }) => {
     if (!command.includes('post-tool-use')) return;
@@ -290,7 +298,7 @@ export function extractInjectedFromErrorRecall(transcriptPath) {
     INJECTED_RE.lastIndex = 0;
     let m;
     while ((m = INJECTED_RE.exec(text))) addObsId(ids, m[1]);
-  });
+  }, opts);
   return ids;
 }
 
@@ -311,7 +319,7 @@ const FYI_LINE_ID_RE = /^#(\d{1,7})\s/;
  * @param {string|null|undefined} transcriptPath
  * @returns {Set<number>}
  */
-export function extractInjectedFromFyi(transcriptPath) {
+export function extractInjectedFromFyi(transcriptPath, opts = {}) {
   const ids = new Set();
   eachHookAttachment(transcriptPath, ({ command, text }) => {
     if (!command.includes('user-prompt-search')) return;
@@ -320,7 +328,7 @@ export function extractInjectedFromFyi(transcriptPath) {
       const m = FYI_LINE_ID_RE.exec(fyiLine);
       if (m) addObsId(ids, m[1]);
     }
-  });
+  }, opts);
   return ids;
 }
 
@@ -330,14 +338,18 @@ export function extractInjectedFromFyi(transcriptPath) {
  * user-prompt-search FYI block. Single integration point the Stop handler calls.
  *
  * @param {string|null|undefined} transcriptPath
+ * @param {object} [opts]
+ * @param {boolean} [opts.mainOnly=false] Skip sidechain-injected IDs. The
+ *   citation-decay caller passes true so the injected denominator matches the
+ *   mainOnly cited numerator; the P4 access-bump caller omits it (broader).
  * @returns {Set<number>}
  */
-export function extractAllInjected(transcriptPath) {
+export function extractAllInjected(transcriptPath, opts = {}) {
   return new Set([
-    ...extractInjectedFromPreToolUse(transcriptPath),
-    ...extractInjectedFromUserPromptSubmit(transcriptPath),
-    ...extractInjectedFromErrorRecall(transcriptPath),
-    ...extractInjectedFromFyi(transcriptPath),
+    ...extractInjectedFromPreToolUse(transcriptPath, opts),
+    ...extractInjectedFromUserPromptSubmit(transcriptPath, opts),
+    ...extractInjectedFromErrorRecall(transcriptPath, opts),
+    ...extractInjectedFromFyi(transcriptPath, opts),
   ]);
 }
 

package/mem-cli.mjs

@@ -1405,8 +1405,11 @@ function cmdUpdate(db, args) {
          `Prompts and sessions are append-only.`);
     return;
   }
+  // Strict parseIdToken gate (aligned with cmdDelete): a bare parseInt fallback
+  // truncated "3.9" → 3 and silently UPDATE'd the WRONG row #3 (no preview, no
+  // --confirm). Require an exact obs-id token; non-matching input → usage error.
   const parsed = raw ? parseIdToken(raw) : null;
-  const id = parsed && parsed.source === null ? parsed.id : parseInt(raw, 10);
+  const id = parsed && parsed.source === null ? parsed.id : NaN;
   if (!id || isNaN(id)) {
     fail('[mem] Usage: claude-mem-lite update <id> [--title T] [--type T] [--importance N] [--lesson T] [--narrative T] [--concepts T]');
     return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.9.0",
+  "version": "3.9.1",
   "description": "Persistent long-term memory for Claude Code via MCP — captures coding decisions, bugfixes, and context across sessions. Hybrid FTS5 + TF-IDF search with episode batching. Single SQLite DB, no external services. A lighter, lower-cost alternative to claude-mem (episode batching + a smaller model; cost savings are an internal estimate, not a measured benchmark).",
   "type": "module",
   "packageManager": "npm@10.9.2",