@wipcomputer/wip-ldm-os 0.2.9 → 0.2.10

package/SKILL.md

@@ -5,7 +5,7 @@ license: MIT
 interface: [cli, skill]
 metadata:
   display-name: "LDM OS"
-  version: "0.2.9"
+  version: "0.2.10"
   homepage: "https://github.com/wipcomputer/wip-ldm-os"
   author: "Parker Todd Brooks"
   category: infrastructure

package/bin/ldm.js

@@ -82,7 +82,6 @@ async function cmdInit() {
     join(LDM_ROOT, 'agents'),
     join(LDM_ROOT, 'memory'),
     join(LDM_ROOT, 'state'),
-    join(LDM_ROOT, 'secrets'),
     join(LDM_ROOT, 'shared', 'boot'),
   ];
 
@@ -300,10 +299,37 @@ async function cmdInstall() {
     return;
   }
 
-  // Resolve target: GitHub URL, org/repo shorthand, or local path
+  // Resolve target: npm package, GitHub URL, org/repo shorthand, or local path
   let repoPath;
 
-  if (target.startsWith('http') || target.startsWith('git@') || target.match(/^[\w-]+\/[\w.-]+$/)) {
+  // Check if target looks like an npm package (starts with @ or is a plain name without /)
+  if (target.startsWith('@') || (!target.includes('/') && !existsSync(resolve(target)))) {
+    // Try npm install to temp dir
+    const npmName = target;
+    const tempDir = join('/tmp', `ldm-install-npm-${Date.now()}`);
+    console.log('');
+    console.log(`  Installing ${npmName} from npm...`);
+    try {
+      mkdirSync(tempDir, { recursive: true });
+      execSync(`npm install ${npmName} --prefix "${tempDir}"`, { stdio: 'pipe' });
+      // Find the installed package in node_modules
+      const pkgName = npmName.startsWith('@') ? npmName : npmName;
+      const installed = join(tempDir, 'node_modules', pkgName);
+      if (existsSync(installed)) {
+        console.log(`  + Installed from npm`);
+        repoPath = installed;
+      } else {
+        console.error(`  x Package installed but not found at expected path`);
+        process.exit(1);
+      }
+    } catch (e) {
+      // npm failed, fall through to git clone or path resolution
+      console.log(`  npm install failed, trying other methods...`);
+      try { execSync(`rm -rf "${tempDir}"`, { stdio: 'pipe' }); } catch {}
+    }
+  }
+
+  if (!repoPath && (target.startsWith('http') || target.startsWith('git@') || target.match(/^[\w-]+\/[\w.-]+$/))) {
     const isShorthand = target.match(/^[\w-]+\/[\w.-]+$/);
     const httpsUrl = isShorthand
       ? `https://github.com/${target}.git`
@@ -332,7 +358,7 @@ async function cmdInstall() {
       console.error(`  x Clone failed: ${e.message}`);
       process.exit(1);
     }
-  } else {
+  } else if (!repoPath) {
     repoPath = resolve(target);
     if (!existsSync(repoPath)) {
       console.error(`  x Path not found: ${repoPath}`);

package/lib/deploy.mjs

@@ -159,8 +159,34 @@ function runBuildIfNeeded(repoPath) {
   }
 }
 
+// ── Version comparison (fix #7) ──
+
+function compareSemver(a, b) {
+  if (!a || !b) return 0;
+  const pa = a.split('.').map(Number);
+  const pb = b.split('.').map(Number);
+  for (let i = 0; i < 3; i++) {
+    const na = pa[i] || 0;
+    const nb = pb[i] || 0;
+    if (na > nb) return 1;
+    if (na < nb) return -1;
+  }
+  return 0;
+}
+
+// Config files that should never be overwritten during updates
+const PRESERVE_PATTERNS = [
+  'boot-config.json', '.env', '.env.local',
+  'config.local.json', 'settings.local.json',
+];
+
+function isPreservedFile(filename) {
+  return PRESERVE_PATTERNS.some(p => filename === p || filename.endsWith('.local'));
+}
+
 // ── Safe deploy (fix #7) ──
 // Deploy to temp dir first, then atomic rename. Never rm -rf the live dir.
+// Preserves config files from existing installs.
 
 function copyFiltered(src, dest) {
   cpSync(src, dest, {
@@ -169,6 +195,23 @@ function copyFiltered(src, dest) {
   });
 }
 
+function restorePreservedFiles(oldDir, newDir) {
+  if (!existsSync(oldDir)) return;
+  try {
+    const entries = readdirSync(oldDir);
+    for (const entry of entries) {
+      if (isPreservedFile(entry)) {
+        const oldPath = join(oldDir, entry);
+        const newPath = join(newDir, entry);
+        if (existsSync(oldPath) && !existsSync(newPath)) {
+          cpSync(oldPath, newPath);
+          ok(`Preserved config: ${entry}`);
+        }
+      }
+    }
+  } catch {}
+}
+
 function safeDeployDir(repoPath, destDir, name) {
   const finalPath = join(destDir, name);
   const tempPath = join(tmpdir(), `ldm-deploy-${name}-${Date.now()}`);
@@ -200,7 +243,12 @@ function safeDeployDir(repoPath, destDir, name) {
     }
     renameSync(tempPath, finalPath);
 
-    // 5. Trash the old version (never delete)
+    // 5. Restore preserved config files from old version
+    if (existsSync(backupPath)) {
+      restorePreservedFiles(backupPath, finalPath);
+    }
+
+    // 6. Trash the old version (never delete)
     if (existsSync(backupPath)) {
       const trashed = moveToTrash(backupPath);
       if (trashed) ok(`Old version moved to ${trashed}`);
@@ -337,8 +385,9 @@ function deployExtension(repoPath, name) {
   const newVersion = sourcePkg?.version;
   const currentVersion = installedPkg?.version;
 
-  if (newVersion && currentVersion && newVersion === currentVersion) {
-    skip(`LDM: ${name} already at v${currentVersion}`);
+  const cmp = compareSemver(newVersion, currentVersion);
+  if (newVersion && currentVersion && cmp <= 0) {
+    skip(`LDM: ${name} already at v${currentVersion}${cmp < 0 ? ` (source is older: v${newVersion})` : ''}`);
     // Ensure OC copy exists too
     const ocName = resolveOcPluginName(repoPath, name);
     const ocDest = join(OC_EXTENSIONS, ocName);
@@ -382,11 +431,35 @@ function deployExtension(repoPath, name) {
     fail(`OpenClaw: deploy failed for ${ocName}`);
   } else {
     ok(`OpenClaw: deployed to ${join(OC_EXTENSIONS, ocName)}`);
+    // Verify openclaw.json references match actual directory
+    verifyOcConfig(ocName);
   }
 
   return true;
 }
 
+function verifyOcConfig(pluginDirName) {
+  const ocConfigPath = join(OC_ROOT, 'openclaw.json');
+  const ocConfig = readJSON(ocConfigPath);
+  if (!ocConfig?.extensions) return;
+
+  const pluginPath = join(OC_EXTENSIONS, pluginDirName);
+  const pluginJson = readJSON(join(pluginPath, 'openclaw.plugin.json'));
+  if (!pluginJson?.id) return;
+
+  // Check if openclaw.json has an entry whose path references a different dir
+  for (const ext of ocConfig.extensions) {
+    if (ext.id === pluginJson.id) {
+      const configDir = basename(ext.path || '');
+      if (configDir && configDir !== pluginDirName) {
+        log(`Warning: openclaw.json references "${configDir}" but plugin is at "${pluginDirName}"`);
+        log(`  Update openclaw.json or rename the directory to match.`);
+      }
+      return;
+    }
+  }
+}
+
 function registerMCP(repoPath, door, toolName) {
   const rawName = toolName || door.name || basename(repoPath);
   const name = rawName.replace(/^@[\w-]+\//, '');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wipcomputer/wip-ldm-os",
-  "version": "0.2.9",
+  "version": "0.2.10",
   "type": "module",
   "description": "LDM OS: identity, memory, and sovereignty infrastructure for AI agents",
   "main": "src/boot/boot-hook.mjs",