@wipcomputer/wip-ldm-os 0.4.10 → 0.4.12

package/SKILL.md

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

package/bin/ldm.js

@@ -64,6 +64,9 @@ const LOCK_PATH = join(LDM_ROOT, 'state', '.ldm-install.lock');
 
 function acquireInstallLock() {
   try {
+    // Child processes spawned by `ldm install` inherit this env var
+    if (process.env.LDM_INSTALL_LOCK_PID) return true;
+
     if (existsSync(LOCK_PATH)) {
       const lock = JSON.parse(readFileSync(LOCK_PATH, 'utf8'));
       // Re-entrant: if we already hold the lock, allow it
@@ -82,6 +85,7 @@ function acquireInstallLock() {
     }
     mkdirSync(dirname(LOCK_PATH), { recursive: true });
     writeFileSync(LOCK_PATH, JSON.stringify({ pid: process.pid, started: new Date().toISOString() }));
+    process.env.LDM_INSTALL_LOCK_PID = String(process.pid);
 
     // Clean up on exit
     const cleanup = () => { try { if (existsSync(LOCK_PATH)) { const l = JSON.parse(readFileSync(LOCK_PATH, 'utf8')); if (l.pid === process.pid) unlinkSync(LOCK_PATH); } } catch {} };

package/bin/wip-install.js

@@ -4,5 +4,21 @@
 // If not, installs LDM OS from npm, then delegates.
 // Replaces the standalone 700-line install.js from wip-ai-devops-toolbox.
 
+// Handle --version directly to prevent recursive spawn loop (#70)
+// detectCLIBinaries() calls wip-install --version which would trigger
+// ldm install which triggers npm checks which spawn more processes.
+const args = process.argv.slice(2);
+if (args.includes('--version') || args.includes('-v')) {
+  const { readFileSync } = await import('node:fs');
+  const { join, dirname } = await import('node:path');
+  const { fileURLToPath } = await import('node:url');
+  const __dirname = dirname(fileURLToPath(import.meta.url));
+  try {
+    const pkg = JSON.parse(readFileSync(join(__dirname, '..', 'package.json'), 'utf8'));
+    console.log(pkg.version);
+  } catch { console.log('unknown'); }
+  process.exit(0);
+}
+
 import { run } from '../lib/bootstrap.mjs';
-run(process.argv.slice(2));
+run(args);

package/lib/deploy.mjs

@@ -137,7 +137,7 @@ function findExistingInstalls(toolName, pkg, ocPluginConfig) {
 
 function runBuildIfNeeded(repoPath) {
   const pkg = readJSON(join(repoPath, 'package.json'));
-  if (!pkg) return;
+  if (!pkg) return true;
 
   const hasBuildScript = !!pkg.scripts?.build;
   const hasTsConfig = existsSync(join(repoPath, 'tsconfig.json'));
@@ -155,8 +155,10 @@ function runBuildIfNeeded(repoPath) {
       ok(`Build complete`);
     } catch (e) {
       fail(`Build failed: ${e.stderr?.toString()?.slice(0, 200) || e.message}`);
+      return false;
     }
   }
+  return true;
 }
 
 // ── Version comparison (fix #7) ──
@@ -229,13 +231,21 @@ function safeDeployDir(repoPath, destDir, name) {
       } catch {}
     }
 
-    // 3. Verify temp has package.json (basic sanity)
+    // 3. Verify staged copy is valid
     if (!existsSync(join(tempPath, 'package.json'))) {
       fail(`Deploy verification failed: no package.json in staged copy`);
       rmSync(tempPath, { recursive: true, force: true });
       return false;
     }
 
+    // 3b. If source has a build script, verify dist/ exists (#69)
+    const stagePkg = readJSON(join(tempPath, 'package.json'));
+    if (stagePkg?.scripts?.build && !existsSync(join(tempPath, 'dist'))) {
+      fail(`Deploy aborted: ${name} has a build script but no dist/. Build failed or was skipped.`);
+      rmSync(tempPath, { recursive: true, force: true });
+      return false;
+    }
+
     // 4. Swap: old -> backup, temp -> final
     mkdirSync(destDir, { recursive: true });
     if (existsSync(finalPath)) {

package/package.json

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