claude-mem-lite 2.53.0 → 2.53.2

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "2.53.0",
+      "version": "2.53.2",
       "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.53.0",
+  "version": "2.53.2",
   "description": "Lightweight persistent memory system for Claude Code — FTS5 search, episode batching, error-triggered recall",
   "author": {
     "name": "sdsrss"

package/install.mjs

@@ -524,14 +524,19 @@ async function install() {
     try {
       const cacheBase = join(homedir(), '.claude', 'plugins', 'cache', MARKETPLACE_KEY, 'claude-mem-lite');
       if (existsSync(cacheBase)) {
-        const srcLaunch = join(PROJECT_DIR, 'scripts', 'launch.mjs');
+        const launchSyncFiles = ['launch.mjs', 'launch-preflight.mjs'];
         let clearedHooks = 0;
         for (const ver of readdirSync(cacheBase)) {
           const verDir = join(cacheBase, ver);
 
-          // Sync launch.mjs
+          // Sync launch.mjs + its preflight companion (issue #15)
           if (existsSync(join(verDir, 'scripts'))) {
-            try { copyFileSync(srcLaunch, join(verDir, 'scripts', 'launch.mjs')); } catch {}
+            for (const f of launchSyncFiles) {
+              const src = join(PROJECT_DIR, 'scripts', f);
+              if (existsSync(src)) {
+                try { copyFileSync(src, join(verDir, 'scripts', f)); } catch { /* keep going */ }
+              }
+            }
           }
 
           // Clear cached hooks.json (runtime reads here, not marketplace source)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.53.0",
+  "version": "2.53.2",
   "description": "Lightweight persistent memory system for Claude Code",
   "type": "module",
   "engines": {
@@ -98,6 +98,7 @@
     "commands/bug.md",
     "hooks/hooks.json",
     "scripts/launch.mjs",
+    "scripts/launch-preflight.mjs",
     "scripts/setup.sh",
     "scripts/post-tool-use.sh",
     "scripts/user-prompt-search.js",

package/scripts/launch-preflight.mjs

@@ -0,0 +1,85 @@
+// launch-preflight.mjs — Detect incomplete installs at MCP server launch.
+//
+// Why: issue #15 — published v2.53.0 npm tarball contains all files, but some
+// users end up with a partial install (server.mjs present, search-engine.mjs
+// missing) and the resulting ERR_MODULE_NOT_FOUND has no actionable message.
+// We can't fix every cause (mirror lag / interrupted download / npm cache
+// corruption / permission) on the user side, but we can detect the broken
+// state and either fall back to ~/.claude-mem-lite/ (which hook-update.mjs
+// keeps healthy) or print a clear repair command instead of a Node stack.
+//
+// Pure module — no I/O at import time, all deps injected. Tested in isolation;
+// scripts/launch.mjs wires it to the actual filesystem + stderr.
+
+import { existsSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
+
+// Match relative `.mjs` imports — covers all four ESM forms:
+//   from './x.mjs'                    (static named/default/namespace)
+//   import './x.mjs'                  (side-effect static)
+//   await import('./x.mjs')           (dynamic)
+//   import('./x.mjs') without await   (dynamic)
+// Skips 'node:*' / package imports / non-mjs paths.
+const FROM_RE = /\bfrom\s+['"](\.\/[^'"]+\.mjs)['"]/g;
+const IMPORT_RE = /\bimport\s*\(?\s*['"](\.\/[^'"]+\.mjs)['"]/g;
+
+export function detectMissingImports(installRoot) {
+  const serverPath = join(installRoot, 'server.mjs');
+  if (!existsSync(serverPath)) return ['server.mjs'];
+
+  let src;
+  try {
+    src = readFileSync(serverPath, 'utf8');
+  } catch {
+    return ['server.mjs'];
+  }
+
+  // Strip line + block comments so example strings in docblocks (e.g. the very
+  // patterns this regex enumerates) can't false-fire as "missing files".
+  src = src
+    .replace(/\/\*[\s\S]*?\*\//g, '')
+    .replace(/(^|\s)\/\/[^\n]*/g, '$1');
+
+  const missing = new Set();
+  for (const re of [FROM_RE, IMPORT_RE]) {
+    re.lastIndex = 0;
+    for (const m of src.matchAll(re)) {
+      const rel = m[1].replace(/^\.\//, '');
+      if (!existsSync(join(installRoot, rel))) missing.add(rel);
+    }
+  }
+  return [...missing];
+}
+
+export function resolveLaunchEntry({ primaryRoot, fallbackRoot, warn = () => {} }) {
+  const primaryMissing = detectMissingImports(primaryRoot);
+  if (primaryMissing.length === 0) {
+    return { path: join(primaryRoot, 'server.mjs'), source: 'primary' };
+  }
+
+  if (fallbackRoot && fallbackRoot !== primaryRoot) {
+    const fallbackMissing = detectMissingImports(fallbackRoot);
+    if (fallbackMissing.length === 0) {
+      warn(
+        `[claude-mem-lite] Primary install incomplete at ${primaryRoot} ` +
+        `(missing: ${primaryMissing.join(', ')}). Falling back to ${fallbackRoot}.`,
+      );
+      return {
+        path: join(fallbackRoot, 'server.mjs'),
+        source: 'fallback',
+        missingFromPrimary: primaryMissing,
+      };
+    }
+  }
+
+  const repairCmd = 'npm install -g claude-mem-lite@latest --force';
+  const err = new Error(
+    `[claude-mem-lite] Install incomplete at ${primaryRoot}\n` +
+    `[claude-mem-lite]   Missing: ${primaryMissing.join(', ')}\n` +
+    `[claude-mem-lite] Repair: ${repairCmd}\n` +
+    `[claude-mem-lite] Or via Claude Code: /plugin uninstall claude-mem-lite && /plugin install claude-mem-lite@sdsrss`,
+  );
+  err.code = 'INSTALL_INCOMPLETE';
+  err.missing = primaryMissing;
+  throw err;
+}

package/scripts/launch.mjs

@@ -12,12 +12,22 @@ const ROOT = process.env.CLAUDE_PLUGIN_ROOT || join(__dirname, '..');
 
 if (!existsSync(join(ROOT, 'node_modules', 'better-sqlite3'))) {
   process.stderr.write('[claude-mem-lite] Installing dependencies...\n');
-  execSync('npm install --omit=dev', {
-    cwd: ROOT,
-    stdio: ['ignore', 'pipe', 'inherit'], // stdout piped (discard), stderr inherit
-    timeout: 120_000,
-  });
-  process.stderr.write('[claude-mem-lite] Dependencies installed\n');
+  try {
+    execSync('npm install --omit=dev', {
+      cwd: ROOT,
+      stdio: ['ignore', 'pipe', 'inherit'], // stdout piped (discard), stderr inherit
+      timeout: 120_000,
+    });
+    process.stderr.write('[claude-mem-lite] Dependencies installed\n');
+  } catch (e) {
+    // Plugin-cache / multi-user / disk-full installs can fail here. Without this
+    // catch the user sees a Node stack trace; with it they get an actionable line.
+    const detail = e.message?.split('\n')[0] || e.code || 'unknown error';
+    process.stderr.write(`[claude-mem-lite] npm install failed in ${ROOT} — ${detail}\n`);
+    process.stderr.write(`[claude-mem-lite] Likely cause: read-only directory, disk full, or network blocked.\n`);
+    process.stderr.write(`[claude-mem-lite] Repair: cd "${ROOT}" && npm install --omit=dev\n`);
+    process.exit(1);
+  }
 }
 
 // Verify MCP SDK is importable (exports mapping intact).
@@ -45,12 +55,31 @@ try {
 // Dev mode: prefer ~/.claude-mem-lite/server.mjs (symlinked to source) over
 // CLAUDE_PLUGIN_ROOT (potentially stale plugin cache). This ensures the MCP
 // server always runs the latest code when installed with `install --dev`.
-const devServer = join(homedir(), '.claude-mem-lite', 'server.mjs');
+const dataDir = join(homedir(), '.claude-mem-lite');
+const devServer = join(dataDir, 'server.mjs');
 let useDevServer = false;
 try { useDevServer = existsSync(devServer) && lstatSync(devServer).isSymbolicLink(); } catch {}
 
 if (useDevServer) {
   await import(pathToFileURL(devServer).href);
 } else {
-  await import(new URL('../server.mjs', import.meta.url).href);
+  // Preflight: detect incomplete primary install (issue #15) — if relative
+  // imports referenced by server.mjs are missing on disk, fall back to the
+  // hook-update.mjs-maintained ~/.claude-mem-lite/ copy when healthy, or exit
+  // with a clear repair command instead of a Node ERR_MODULE_NOT_FOUND stack.
+  const { resolveLaunchEntry } = await import('./launch-preflight.mjs');
+  try {
+    const entry = resolveLaunchEntry({
+      primaryRoot: ROOT,
+      fallbackRoot: dataDir,
+      warn: (msg) => process.stderr.write(msg + '\n'),
+    });
+    await import(pathToFileURL(entry.path).href);
+  } catch (e) {
+    if (e.code === 'INSTALL_INCOMPLETE') {
+      process.stderr.write(e.message + '\n');
+      process.exit(1);
+    }
+    throw e;
+  }
 }