claude-mem-lite 3.70.1 → 3.70.2

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.70.1",
+      "version": "3.70.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.70.1",
+  "version": "3.70.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/binding-probe.mjs

@@ -51,6 +51,29 @@ export function isNativeBindingError(err) {
   return NATIVE_BINDING_PATTERNS.some((re) => re.test(msg));
 }
 
+/**
+ * Render a binding error as ONE line without losing the diagnosis.
+ *
+ * Every surface used to do `String(err).split('\n')[0]`, which is exactly wrong for
+ * the ABI-mismatch family this subsystem exists to detect. Node's message puts the
+ * filename on line 0 and the `NODE_MODULE_VERSION 127 … requires 137` on lines 2-3,
+ * so first-line truncation printed a bare path and dropped the only part that says
+ * what is wrong. (The comment in probeBindingInFreshProcess below called that string
+ * "the highest-value line doctor prints"; for a stale binding it carried no diagnosis
+ * at all.) Collapsing whitespace keeps both the path and the numbers while staying
+ * safe for a JSON envelope, a JSONL log record and a one-line hook receipt.
+ *
+ * @param {unknown} err Error, string, or anything thrown.
+ * @param {number} [max] Hard cap; a probe must not be able to flood a receipt.
+ * @returns {string}
+ */
+export function flattenBindingError(err, max = 240) {
+  const raw = err instanceof Error ? err.message : err;
+  const s = String(raw ?? '').replace(/\s+/g, ' ').trim();
+  if (!s) return 'unknown';
+  return s.length > max ? `${s.slice(0, max - 1)}…` : s;
+}
+
 /**
  * Probe better-sqlite3's native binding by importing it from `installDir`'s
  * node_modules and opening an in-memory DB. Returns {ok, error?}.

package/lib/install-shape.mjs

@@ -34,7 +34,7 @@ import { existsSync, readdirSync, realpathSync } from 'node:fs';
 import { join } from 'node:path';
 import { homedir } from 'node:os';
 
-import { probeBindingInFreshProcess, NATIVE_BINDING_REBUILD_CMD } from './binding-probe.mjs';
+import { probeBindingInFreshProcess, NATIVE_BINDING_REBUILD_CMD, flattenBindingError } from './binding-probe.mjs';
 
 // Module-private: nothing outside needs these, and a new unused export is a
 // review signal against the knip baseline recorded in CLAUDE.md.
@@ -216,7 +216,10 @@ export function probeRuntimeRoots(roots, deps = {}) {
     // not installed exits 0 and heals nothing, so an unowned tree that also cannot
     // resolve from an ancestor needs an install; a present-but-unloadable tree needs
     // the rebuild.
-    const error = String(r.error || 'unknown').split('\n')[0];
+    // NOT `.split('\n')[0]`: Node puts the filename on line 0 and the
+    // `NODE_MODULE_VERSION 127 … requires 137` on lines 2-3, so first-line truncation
+    // showed a bare path for the one fault family this check exists for.
+    const error = flattenBindingError(r.error);
     return ownDeps
       ? { label, root, ok: false, error, repair: `cd ${root} && ${NATIVE_BINDING_REBUILD_CMD}` }
       : {

package/lib/native-binding-hint.mjs

@@ -24,7 +24,7 @@
 import { join } from 'node:path';
 import { readFileSync, writeFileSync, mkdirSync, renameSync, unlinkSync } from 'node:fs';
 import { fileURLToPath } from 'node:url';
-import { isNativeBindingError } from './binding-probe.mjs';
+import { isNativeBindingError, flattenBindingError } from './binding-probe.mjs';
 
 export const NATIVE_BINDING_HINT_COOLDOWN_MS = 6 * 60 * 60 * 1000; // 6h
 const MARKER_NAME = 'native-binding-hint-last';
@@ -136,7 +136,7 @@ export function recordNativeBindingBreakage(runtimeDir, { reason = '', event = '
     const tmp = `${marker}.tmp-${process.pid}`;
     // First line only: the ABI error is multi-line and the marker is read by the
     // launcher (pure node:, no parser beyond JSON.parse) and by `doctor`.
-    writeFileSync(tmp, JSON.stringify({ reason: String(reason).split('\n')[0], event, ts: now }));
+    writeFileSync(tmp, JSON.stringify({ reason: flattenBindingError(reason), event, ts: now }));
     renameSync(tmp, marker);
   } catch { /* best-effort */ }
 }

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.70.1",
+  "version": "3.70.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "claude-mem-lite",
-      "version": "3.70.1",
+      "version": "3.70.2",
       "dependencies": {
         "@modelcontextprotocol/sdk": "^1.26.0",
         "better-sqlite3": "^12.6.2",

package/package.json

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

package/scripts/binding-probe-cli.mjs

@@ -42,8 +42,16 @@ function bareProbe(root) {
   const r = spawnSync(process.execPath, ['-e', script], { stdio: 'pipe', timeout: 8000 });
   if (!r.error && r.status === 0) return true;
   // Say WHY. The inline predecessor printed the cause here; dropping it left the
-  // user with setup.sh's generic "binding unusable" and nothing to act on.
-  const why = String(r.stdout || '').trim().split('\n')[0]
+  // user with setup.sh's generic "binding unusable" and nothing to act on. Flattened
+  // rather than first-lined: Node's ABI message puts the filename on line 0 and the
+  // NODE_MODULE_VERSION pair on lines 2-3, so `.split('\n')[0]` said WHERE but never
+  // WHY — for the exact fault this probe exists to find.
+  //
+  // Flattening is inlined, NOT lib/binding-probe.mjs::flattenBindingError, because
+  // this function is the fallback for a tree where lib/ failed to import — `helpers`
+  // is still null on every path that reaches here. Duplicated deliberately; keep the
+  // two in step.
+  const why = String(r.stdout || '').replace(/\s+/g, ' ').trim().slice(0, 240)
     || (r.error && r.error.message)
     || `probe exited ${r.status ?? `on signal ${r.signal}`}`;
   process.stderr.write(`[claude-mem-lite] binding probe: ${why}\n`);
@@ -82,9 +90,9 @@ try {
 }
 const release = helpers.acquireLock(lockPath);
 if (!release) {
-  const firstLine = String(first.error).split('\n')[0];
   process.stderr.write(
-    `[claude-mem-lite] binding probe: ${firstLine} (another install/repair in flight — deferring heal)\n`,
+    `[claude-mem-lite] binding probe: ${helpers.flattenBindingError(first.error)} `
+    + '(another install/repair in flight — deferring heal)\n',
   );
   process.exit(1);
 }