@memtensor/memos-local-openclaw-plugin 1.0.6-beta.3 → 1.0.6-beta.5

package/openclaw.plugin.json

@@ -3,7 +3,7 @@
   "name": "MemOS Local Memory",
   "description": "Full-write local conversation memory with hybrid search (RRF + MMR + recency), task summarization, skill evolution, and team sharing (Hub-Client). Provides memory_search, memory_get, task_summary, skill_search, task_share, network_skill_pull, network_team_info, memory_viewer for layered retrieval and team collaboration.",
   "kind": "memory",
-  "version": "0.1.12",
+  "version": "1.0.6-beta.5",
   "skills": [
     "skill/memos-memory-guide"
   ],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@memtensor/memos-local-openclaw-plugin",
-  "version": "1.0.6-beta.3",
+  "version": "1.0.6-beta.5",
   "description": "MemOS Local memory plugin for OpenClaw — full-write, hybrid-recall, progressive retrieval",
   "type": "module",
   "main": "index.ts",
@@ -11,6 +11,7 @@
     "dist",
     "skill",
     "prebuilds",
+    "scripts/native-binding.cjs",
     "scripts/postinstall.cjs",
     "openclaw.plugin.json",
     "telemetry.credentials.json",

package/scripts/native-binding.cjs

@@ -0,0 +1,32 @@
+"use strict";
+
+function errorMessage(error) {
+  if (error && typeof error.message === "string") return error.message;
+  return String(error || "Unknown native binding error");
+}
+
+function defaultLoadBinding(bindingPath) {
+  process.dlopen({ exports: {} }, bindingPath);
+}
+
+function validateNativeBinding(bindingPath, loadBinding = defaultLoadBinding) {
+  if (!bindingPath) {
+    return { ok: false, reason: "missing", message: "Native binding path not found" };
+  }
+
+  try {
+    loadBinding(bindingPath);
+    return { ok: true, reason: "ok", message: "" };
+  } catch (error) {
+    const message = errorMessage(error);
+    if (/NODE_MODULE_VERSION/.test(message)) {
+      return { ok: false, reason: "node-module-version", message };
+    }
+    return { ok: false, reason: "load-error", message };
+  }
+}
+
+module.exports = {
+  defaultLoadBinding,
+  validateNativeBinding,
+};

package/scripts/postinstall.cjs

@@ -4,6 +4,7 @@
 const { spawnSync } = require("child_process");
 const path = require("path");
 const fs = require("fs");
+const { validateNativeBinding } = require("./native-binding.cjs");
 
 const RESET = "\x1b[0m";
 const GREEN = "\x1b[32m";
@@ -377,35 +378,31 @@ function findSqliteBinding() {
 
 function sqliteBindingsExist() {
   const found = findSqliteBinding();
-  if (found) {
-    log(`Native binding found: ${DIM}${found}${RESET}`);
-    return true;
+  if (!found) return false;
+  log(`Native binding found: ${DIM}${found}${RESET}`);
+  const status = validateNativeBinding(found);
+  if (status.ok) return true;
+  if (status.reason === "node-module-version") {
+    warn("Native binding exists but was compiled for a different Node.js version.");
+  } else {
+    warn("Native binding exists but failed to load.");
   }
+  warn(`${DIM}${status.message}${RESET}`);
   return false;
 }
 
 if (sqliteBindingsExist()) {
   ok("better-sqlite3 is ready.");
 } else {
-  warn("better-sqlite3 native bindings not found in plugin dir.");
+  warn("better-sqlite3 native bindings are missing or not loadable.");
   log(`Searched in: ${DIM}${sqliteModulePath}/build/${RESET}`);
   log("Running: npm rebuild better-sqlite3 (may take 30-60s)...");
-}
-
-const startMs = Date.now();
-
-const result = spawnSync(npmCmd, ["rebuild", "better-sqlite3"], {
-  cwd: pluginDir,
-  stdio: "pipe",
-  shell: false,
-  timeout: 180_000,
-});
 
   const startMs = Date.now();
-  const result = spawnSync("npm", ["rebuild", "better-sqlite3"], {
+  const result = spawnSync(npmCmd, ["rebuild", "better-sqlite3"], {
     cwd: pluginDir,
     stdio: "pipe",
-    shell: true,
+    shell: false,
     timeout: 180_000,
   });
   const elapsed = ((Date.now() - startMs) / 1000).toFixed(1);

package/src/viewer/server.ts

@@ -3356,6 +3356,32 @@ export class ViewerServer {
         const shortName = pluginName?.replace(/^@[\w-]+\//, "") ?? "memos-local-openclaw-plugin";
         const extDir = path.join(os.homedir(), ".openclaw", "extensions", shortName);
         const tmpDir = path.join(os.tmpdir(), `openclaw-update-${Date.now()}`);
+        const backupDir = path.join(path.dirname(extDir), `${shortName}.backup-${Date.now()}`);
+        let backupReady = false;
+
+        const cleanupTmpDir = () => {
+          try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
+        };
+        const rollbackInstall = () => {
+          try { fs.rmSync(extDir, { recursive: true, force: true }); } catch {}
+          if (!backupReady) return;
+          try {
+            fs.renameSync(backupDir, extDir);
+            backupReady = false;
+            this.log.info(`update-install: restored previous version from ${backupDir}`);
+          } catch (restoreErr: any) {
+            this.log.warn(`update-install: failed to restore previous version: ${restoreErr?.message ?? restoreErr}`);
+          }
+        };
+        const discardBackup = () => {
+          if (!backupReady) return;
+          try {
+            fs.rmSync(backupDir, { recursive: true, force: true });
+            backupReady = false;
+          } catch (cleanupErr: any) {
+            this.log.warn(`update-install: failed to remove backup dir ${backupDir}: ${cleanupErr?.message ?? cleanupErr}`);
+          }
+        };
 
         // Download via npm pack, extract, and replace extension dir.
         // Does NOT touch openclaw.json → no config watcher SIGUSR1.
@@ -3365,7 +3391,7 @@ export class ViewerServer {
           if (packErr) {
             this.log.warn(`update-install: npm pack failed: ${packErr.message}`);
             this.jsonResponse(res, { ok: false, error: `Download failed: ${packErr.message}` });
-            try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
+            cleanupTmpDir();
             return;
           }
           const tgzFile = packOut.trim().split("\n").pop()!;
@@ -3378,7 +3404,7 @@ export class ViewerServer {
             if (tarErr) {
               this.log.warn(`update-install: tar extract failed: ${tarErr.message}`);
               this.jsonResponse(res, { ok: false, error: `Extract failed: ${tarErr.message}` });
-              try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
+              cleanupTmpDir();
               return;
             }
 
@@ -3386,23 +3412,36 @@ export class ViewerServer {
             const srcDir = path.join(extractDir, "package");
             if (!fs.existsSync(srcDir)) {
               this.jsonResponse(res, { ok: false, error: "Extracted package has no 'package' dir" });
-              try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
+              cleanupTmpDir();
               return;
             }
 
             // Replace extension directory
             this.log.info(`update-install: replacing ${extDir}...`);
-            try { fs.rmSync(extDir, { recursive: true, force: true }); } catch {}
-            fs.mkdirSync(path.dirname(extDir), { recursive: true });
-            fs.renameSync(srcDir, extDir);
+            try {
+              fs.mkdirSync(path.dirname(extDir), { recursive: true });
+              try { fs.rmSync(backupDir, { recursive: true, force: true }); } catch {}
+              if (fs.existsSync(extDir)) {
+                fs.renameSync(extDir, backupDir);
+                backupReady = true;
+              }
+              fs.renameSync(srcDir, extDir);
+            } catch (replaceErr: any) {
+              this.log.warn(`update-install: replace failed: ${replaceErr?.message ?? replaceErr}`);
+              cleanupTmpDir();
+              rollbackInstall();
+              this.jsonResponse(res, { ok: false, error: `Replace failed: ${replaceErr?.message ?? replaceErr}` });
+              return;
+            }
 
             // Install dependencies
             this.log.info(`update-install: installing dependencies...`);
             const npmCmd = process.platform === "win32" ? "npm.cmd" : "npm";
             execFile(npmCmd, ["install", "--omit=dev", "--ignore-scripts"], { cwd: extDir, timeout: 120_000 }, (npmErr, npmOut, npmStderr) => {
               if (npmErr) {
-                try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
                 this.log.warn(`update-install: npm install failed: ${npmErr.message}`);
+                cleanupTmpDir();
+                rollbackInstall();
                 this.jsonResponse(res, { ok: false, error: `Dependency install failed: ${npmStderr || npmErr.message}` });
                 return;
               }
@@ -3416,12 +3455,15 @@ export class ViewerServer {
 
                 this.log.info(`update-install: running postinstall...`);
                 execFile(process.execPath, ["scripts/postinstall.cjs"], { cwd: extDir, timeout: 180_000 }, (postErr, postOut, postStderr) => {
-                  try { fs.rmSync(tmpDir, { recursive: true, force: true }); } catch {}
+                  cleanupTmpDir();
 
                   if (postErr) {
                     this.log.warn(`update-install: postinstall failed: ${postErr.message}`);
                     const postStderrStr = String(postStderr || "").trim();
                     if (postStderrStr) this.log.warn(`update-install: postinstall stderr: ${postStderrStr.slice(0, 500)}`);
+                    rollbackInstall();
+                    this.jsonResponse(res, { ok: false, error: `Postinstall failed: ${postStderrStr || postErr.message}` });
+                    return;
                   }
 
                   // Read new version
@@ -3431,6 +3473,7 @@ export class ViewerServer {
                     newVersion = newPkg.version ?? newVersion;
                   } catch {}
 
+                  discardBackup();
                   this.log.info(`update-install: success! Updated to ${newVersion}`);
                   this.jsonResponse(res, { ok: true, version: newVersion });