claude-mem-lite 3.38.0 → 3.38.2

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.38.0",
+      "version": "3.38.2",
       "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.38.0",
+  "version": "3.38.2",
   "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/lib/citation-tracker.mjs

@@ -377,8 +377,51 @@ export function extractAllInjected(transcriptPath, opts = {}) {
  * @param {string} transcriptPath
  * @returns {{injected: number, cited: number, recalled: number, ratio: number}}
  */
+// pre-agent-inject.js (CLAUDE_MEM_SUBAGENT_INJECT) APPENDS formatSubagentContext to a
+// dispatched subagent's task prompt via PreToolUse updatedInput — a PROMPT-embedded
+// injection, NOT a hook attachment. extractAllInjected (the attachment path) misses it,
+// so the sidechain instrument read a false 0 ("subagents memory-blind") even while the
+// dispatch surface was live. The block is `[Project memory — surfaced by your operator's
+// claude-mem-lite ...]` followed by a `  #NN — <lesson>.` tag line.
+const SUBAGENT_INJECT_MARKER = /surfaced by your operator's claude-mem-lite/;
+// Row-anchored to the `#NN — ` tag so a #NN quoted inside the lesson body does NOT enter
+// the injected set — same discipline as INJECTED_ROW_RE for the attachment surfaces.
+const SUBAGENT_INJECT_ID_RE = /^\s{0,4}#(\d{1,7})\s+—/;
+
+/**
+ * Extract observation ids injected into a subagent's PROMPT by pre-agent-inject.js
+ * (formatSubagentContext). Only the `#NN — ` tag line counts; a body cross-reference is
+ * ignored. Returns numeric ids.
+ * @param {string|null|undefined} transcriptPath
+ * @returns {Set<number>}
+ */
+export function extractInjectedFromSubagentPrompt(transcriptPath) {
+  const ids = new Set();
+  let raw;
+  try { raw = readFileSync(transcriptPath, 'utf8'); } catch { return ids; }
+  for (const line of raw.split('\n')) {
+    if (!line) continue;
+    let entry;
+    try { entry = JSON.parse(line); } catch { continue; }
+    const c = entry.message?.content;
+    const text = typeof c === 'string'
+      ? c
+      : Array.isArray(c) ? c.map((x) => (typeof x === 'string' ? x : x?.text || '')).join('\n') : '';
+    if (!text || !SUBAGENT_INJECT_MARKER.test(text)) continue;
+    for (const bl of text.split('\n')) {
+      const m = SUBAGENT_INJECT_ID_RE.exec(bl);
+      if (m) addObsId(ids, m[1]);
+    }
+  }
+  return ids;
+}
+
 export function computeThreadCiteRecall(transcriptPath) {
   const injected = extractAllInjected(transcriptPath);
+  // Subagent files carry NO hook-attachment injection; pre-agent-inject.js injects into
+  // the PROMPT (updatedInput). Fold that in so sidechain recall isn't a false 0. On a
+  // main transcript this marker is absent → no-op.
+  for (const id of extractInjectedFromSubagentPrompt(transcriptPath)) injected.add(id);
   const cited = extractCitationsFromTranscript(transcriptPath);
   let recalled = 0;
   for (const id of injected) if (cited.has(id)) recalled++;

package/mem-cli.mjs

@@ -2349,10 +2349,10 @@ function _reportSidechainCiteRecall({ days, json }) {
   out(`  main        ${pct(mainRate).padStart(6)}   recalled ${main.recalled} / injected ${main.injected}   (${main.files} transcript(s))`);
   out(`  sidechain   ${pct(sideRate).padStart(6)}   recalled ${sidechain.recalled} / injected ${sidechain.injected}   (${sidechain.files} subagent file(s), ${sidechain.withInjections} with injections)`);
   if (sidechain.files > 0 && sidechain.injected === 0) {
-    out('  → subagent transcripts exist but received ZERO memory injections: claude-mem-lite');
-    out('    hooks do NOT fire inside subagents (no PreToolUse/PostToolUse recall, no');
-    out('    SessionStart block, no mem_* tools). Subagents are memory-blind — giving them');
-    out('    memory needs a NEW surface (inject at Agent/Task dispatch), not deepening.');
+    out('  → subagent transcripts exist but none carried a detectable memory injection in');
+    out('    this window. claude-mem-lite hooks do NOT fire inside subagents; the dispatch-');
+    out('    time surface (pre-agent-inject.js, CLAUDE_MEM_SUBAGENT_INJECT) prompt-injects');
+    out('    a lesson when enabled — a 0 here means it was off or selected none for these.');
   } else if (sidechain.files === 0) {
     out('  → no subagent transcripts in window.');
   }
@@ -2443,16 +2443,34 @@ function cmdCitationStats(db, args) {
   // as the per-project cite rate above; funnel.prior/delta_pt show the direction.
   const funnel = computeCitationFunnelTrend(db, { days });
 
+  // Survivorship-honesty: the per-project rate (cited_count/decay_seen_count over
+  // SURVIVING in-window obs) is doubly biased — GC drops uncited obs from the
+  // denominator and the window excludes older obs — so it overstates adoption (a
+  // project can read 91% while its true lifetime inject→cite is ~6%). citation_log
+  // rows persist across GC, so their per-project sum is the honest historical rate.
+  // Attach both to each row so the text + JSON surface the biased and honest numbers.
+  const funnelByProject = new Map(
+    db.prepare(`SELECT project, COALESCE(SUM(injected_n), 0) inj, COALESCE(SUM(cited_n), 0) cit
+                  FROM citation_log GROUP BY project`).all().map(r => [r.project, r])
+  );
+  for (const r of perProject) {
+    const f = funnelByProject.get(r.project);
+    r.funnel_injected = f ? f.inj : 0;
+    r.funnel_cited = f ? f.cit : 0;
+  }
+
   if (json) {
     out(JSON.stringify({ window_days: days, per_project: perProject, decay_queue: decayQueue, promoted, demoted, data_pollution_note: dataPollutionNote, funnel }, null, 2));
     return;
   }
 
   if (dataPollutionNote) out(`Note: ${dataPollutionNote}\n`);
-  out(`Cite rate by project (last ${days}d, cited / decay-resolutions):`);
+  out(`Cite rate by project (last ${days}d):`);
+  out(`  funnel = lifetime injected→cited (citation_log, GC-durable = honest) · surviving = cited/decay over in-window non-GC'd obs (survivorship-biased, reads high):`);
   for (const r of perProject) {
-    const rate = r.resolved > 0 ? (r.cited * 100 / r.resolved).toFixed(1) + '%' : '—';
-    out(`  ${r.project.padEnd(34)} ${String(rate).padStart(6)}   cited:${r.cited}/${r.resolved}   at_risk:${r.at_risk}`);
+    const funnelRate = r.funnel_injected > 0 ? (r.funnel_cited * 100 / r.funnel_injected).toFixed(1) + '%' : '—';
+    const survRate = r.resolved > 0 ? (r.cited * 100 / r.resolved).toFixed(1) + '%' : '—';
+    out(`  ${r.project.padEnd(30)} funnel ${String(funnelRate).padStart(6)} (${r.funnel_cited}/${r.funnel_injected}) · surviving ${String(survRate).padStart(6)} (${r.cited}/${r.resolved}) · at_risk:${r.at_risk}`);
   }
   out('');
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.38.0",
+  "version": "3.38.2",
   "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",