npm - claude-mem-lite - Versions diffs - 3.39.0 → 3.39.2 - Mend

claude-mem-lite 3.39.0 → 3.39.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (9) hide show

package/.claude-plugin/marketplace.json +1 -1
package/.claude-plugin/plugin.json +1 -1
package/commands/bug.md +6 -3
package/commands/lesson.md +6 -3
package/hook-optimize.mjs +4 -0
package/lib/activity.mjs +38 -16
package/lib/cite-back-hint.mjs +5 -2
package/lib/save-observation.mjs +4 -1
package/package.json +1 -1

package/.claude-plugin/marketplace.json CHANGED Viewed

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.39.0",
+      "version": "3.39.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 CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.39.0",
+  "version": "3.39.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/commands/bug.md CHANGED Viewed

@@ -43,13 +43,16 @@ Map severity to importance:
 Build the body as `<description>\n\nRepro:\n<repro-steps>` (or just
 `<description>` if `--repro` is absent).
-Run via Bash — the body is the positional content; the description goes in
-`--lesson` so it lands in the high-weight `lesson_learned` field:
+Run via Bash — the full body is the positional content; a distilled description
+goes in `--lesson` so it lands in the high-weight `lesson_learned` field.
+`--lesson` is capped at 500 chars (longer values are rejected), so keep the full
+body in the positional content and trim `--lesson` to the core description if
+longer:
     node ${CLAUDE_PLUGIN_ROOT}/cli.mjs save "<body>" \
       --type bugfix \
       --title "<first 60 chars of description>" \
-      --lesson "<description>" \
+      --lesson "<description, ≤500 chars>" \
       [--files f1,f2,...] \
       --importance <1|2|3>

package/commands/lesson.md CHANGED Viewed

@@ -36,13 +36,16 @@ Don't use for trivial fixes (typos, renames), for rules that belong in memdir
 ## Execution
-Run via Bash — pass the lesson text as the positional content and repeat it in
-`--lesson` so it lands in the high-weight `lesson_learned` field:
+Run via Bash — pass the full lesson text as the positional content, and a
+distilled version in `--lesson` so it lands in the high-weight `lesson_learned`
+field. `--lesson` is capped at 500 chars (longer values are rejected), so for a
+long lesson keep the full text in the positional content and trim `--lesson` to
+the core insight:
     node ${CLAUDE_PLUGIN_ROOT}/cli.mjs save "<full text>" \
       --type discovery \
       --title "<first 60 chars of text>" \
-      --lesson "<full text>" \
+      --lesson "<core insight, ≤500 chars>" \
       [--files f1,f2,...] \
       --importance <1|2|3>

package/hook-optimize.mjs CHANGED Viewed

@@ -171,6 +171,10 @@ Give 3-6 aliases: words a user might search for the SAME concept but that are NO
         const safe = scrubRecord('observations', { text: appendedText, search_aliases: searchAliases });
         db.prepare(`UPDATE observations SET search_aliases = ?, text = ? WHERE id = ?`)
           .run(safe.search_aliases, safe.text, cand.id);
+        // Refresh the TF-IDF vector from the just-updated FTS text so the new
+        // aliases reach the vector arm too — the narrow/wide branch rebuilds, this
+        // one must as well. No-ops when the vector arm is off / vocab unbuilt.
+        rebuildVector(db, cand.id, [safe.text]);
         processed++;
         continue;
       }

package/lib/activity.mjs CHANGED Viewed

@@ -7,6 +7,7 @@
 import { sanitizeFtsQuery } from '../utils.mjs';
 import { scrubRecord } from './scrub-record.mjs';
 import { saveObservation } from './save-observation.mjs';
+import { notLowSignalTitleClause } from '../scoring-sql.mjs';
 // Observation types (mirrors the observations.type enum) — events carry a wider
 // set, so promotion maps the extras (lesson/bug/observation) onto valid obs types.
@@ -130,30 +131,40 @@ export function recentEvents(db, { project, type = null, limit = 20 } = {}) {
  * importance>=minImportance) into searchable observations. mem_search / passive
  * injection never read the events table, so explicit /bug /lesson history and
  * high-value auto-captured events were unfindable. Each promoted event is marked
- * (superseded_at_epoch + superseded_by_id → the new obs id) so re-runs are
- * idempotent. The source event row is kept (activity log intact), just retired.
+ * with superseded_at_epoch ONLY (superseded_by_id is a self-FK → events(id), so an
+ * observation id there fails the FK on the real DB — the marker alone gives
+ * idempotency) so re-runs skip it. The source event row is kept (activity log
+ * intact), just retired.
  *
- * @returns {{eligible:number, promoted:number, deduped:number}}
+ * @returns {{eligible:number, promoted:number, deduped:number, skipped:number}}
  */
 export function promoteInsightEvents(db, { project = null, minImportance = 2, execute = false, limit = 5000 } = {}) {
   const projClause = project ? 'AND project = ?' : '';
+  // Exclude low-signal titles (Modified X / Error while … / Worked on X / raw tool
+  // logs): those are the activity-log noise the events split was meant to contain,
+  // so "lesson-bearing" must not sweep them into search. Same canonical clause the
+  // re-enrich candidate query uses.
   const sql = `
     SELECT id, project, event_type, title, body, file_paths, importance, created_at_epoch
     FROM events
     WHERE body IS NOT NULL AND TRIM(body) != '' AND importance >= ?
-      AND superseded_at_epoch IS NULL ${projClause}
+      AND superseded_at_epoch IS NULL
+      AND ${notLowSignalTitleClause('')}
+      ${projClause}
     ORDER BY created_at_epoch DESC LIMIT ?
   `;
   const rows = project ? db.prepare(sql).all(minImportance, project, limit) : db.prepare(sql).all(minImportance, limit);
-  if (!execute) return { eligible: rows.length, promoted: 0, deduped: 0 };
+  if (!execute) return { eligible: rows.length, promoted: 0, deduped: 0, skipped: 0 };
-  const mark = db.prepare('UPDATE events SET superseded_at_epoch = ?, superseded_by_id = ? WHERE id = ?');
-  let promoted = 0;
-  let deduped = 0;
-  for (const ev of rows) {
-    const type = OBS_TYPES.has(ev.event_type) ? ev.event_type : (EVENT_TO_OBS_TYPE[ev.event_type] || 'discovery');
-    let files = [];
-    try { const p = JSON.parse(ev.file_paths || '[]'); if (Array.isArray(p)) files = p; } catch { /* keep [] */ }
+  // Mark the source event promoted. Only `superseded_at_epoch` — NOT
+  // superseded_by_id: that column is a self-FK (REFERENCES events(id)), so storing
+  // an observation id there fails a FOREIGN KEY constraint on the real DB (the
+  // :memory: test had FK enforcement off, which masked it). The marker alone gives
+  // idempotency (superseded_at_epoch IS NULL selects unpromoted rows).
+  const mark = db.prepare('UPDATE events SET superseded_at_epoch = ? WHERE id = ?');
+  // Each event's (observation insert + mark) is one transaction so a failure can
+  // never leave an orphan observation with an unmarked source event.
+  const promoteOne = db.transaction((ev, type, files) => {
     const r = saveObservation(db, {
       content: ev.body,
       title: ev.title || undefined,
@@ -166,9 +177,20 @@ export function promoteInsightEvents(db, { project = null, minImportance = 2, ex
       lesson_learned: ev.body.slice(0, 500),
       now: new Date(ev.created_at_epoch),
     });
-    const linkId = r.kind === 'saved' ? r.id : r.existingId;
-    mark.run(Date.now(), linkId, ev.id);
-    if (r.kind === 'saved') promoted++; else deduped++;
+    mark.run(Date.now(), ev.id);
+    return r;
+  });
+  let promoted = 0;
+  let deduped = 0;
+  let skipped = 0;
+  for (const ev of rows) {
+    const type = OBS_TYPES.has(ev.event_type) ? ev.event_type : (EVENT_TO_OBS_TYPE[ev.event_type] || 'discovery');
+    let files = [];
+    try { const p = JSON.parse(ev.file_paths || '[]'); if (Array.isArray(p)) files = p; } catch { /* keep [] */ }
+    try {
+      const r = promoteOne(ev, type, files);
+      if (r.kind === 'saved') promoted++; else deduped++;
+    } catch { skipped++; } // a single bad event must not abort the whole backfill
   }
-  return { eligible: rows.length, promoted, deduped };
+  return { eligible: rows.length, promoted, deduped, skipped };
 }

package/lib/cite-back-hint.mjs CHANGED Viewed

@@ -128,8 +128,11 @@ const ACTIVITY_SAVE_LESSON_RE = /activity\s+save\s+--type\s+(lesson|bugfix)\b/;
 // P2(a): /bug and /lesson were redirected from `activity save` (events) to
 // `cli.mjs save … --lesson` (searchable observations). Recognize that shape too,
 // or the unsaved-bugfix nudge over-fires after an explicit save. Anchored on the
-// mem CLI + the --lesson flag both skills always pass.
-const OBS_INSIGHT_SAVE_RE = /(?:cli\.mjs|claude-mem-lite)['"]?\s+save\b[^\n]*--lesson(?:-learned)?\b/;
+// mem CLI + the --lesson flag both skills always pass. Uses `[\s\S]*?` (not
+// `[^\n]*`) because the skill templates document a multi-line, backslash-
+// continued command — after JSON.parse the command carries real newlines, so the
+// gap between `save` and `--lesson` must be allowed to span them (lazy → nearest).
+const OBS_INSIGHT_SAVE_RE = /(?:cli\.mjs|claude-mem-lite)['"]?\s+save\b[\s\S]*?--lesson(?:-learned)?\b/;
 const MEM_SAVE_TOOL_NAMES = new Set([
   'mem_save',
   'mcp__claude_mem_lite__mem_save',

package/lib/save-observation.mjs CHANGED Viewed

@@ -113,7 +113,10 @@ export function saveObservation(db, params) {
     });
     insertObservationFiles(db, savedId, files);
-    insertObservationVector(db, savedId, safeTitle + ' ' + safeContent);
+    // Vector text mirrors the FTS-indexed content (title + content + lesson) so a
+    // lesson-only term isn't invisible to the vector arm (finding #8). Reuses the
+    // same indexText the FTS `text` field is built from.
+    insertObservationVector(db, savedId, indexText);
     return savedId;
   });

package/package.json CHANGED Viewed

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