synabun 2026.4.26-beta.6 → 2026.4.26-beta.7

package/neural-interface/server.js

@@ -4509,12 +4509,36 @@ function stopOpencodeServer() {
   stopOpencodeSSERelay();
   const managedPid = readOpencodeManagedPid();
   const procPid = _ocpProc?.pid || null;
+  const isWin = process.platform === 'win32';
+
+  // On Windows `process.kill(pid, 'SIGTERM')` is unreliable for non-Node
+  // children (cmd.exe shells, opencode native binaries). Use taskkill /T /F
+  // to walk the process tree and force-terminate every descendant.
+  // This is the recurring source of EBUSY errors during `npm install -g
+  // synabun@latest` — opencode-ai's `serve` process survives parent SynaBun
+  // exit and keeps file handles inside neural-interface/ open.
+  const winKill = (pid) => {
+    try {
+      spawn('taskkill', ['/pid', String(pid), '/T', '/F'], {
+        shell: true, windowsHide: true, stdio: 'ignore',
+      });
+    } catch {}
+  };
+
   if (_ocpProc && !_ocpProc.killed) {
-    try { _ocpProc.kill(); } catch {}
+    if (isWin && procPid) {
+      winKill(procPid);
+    } else {
+      try { _ocpProc.kill(); } catch {}
+    }
     _ocpProc = null;
   }
   if (managedPid && managedPid !== procPid) {
-    signalPids([managedPid], 'SIGTERM');
+    if (isWin) {
+      winKill(managedPid);
+    } else {
+      signalPids([managedPid], 'SIGTERM');
+    }
   }
   clearOpencodeManagedPid();
   _ocpReady = false;
@@ -23587,6 +23611,21 @@ function saveSessionSnapshot() {
 setInterval(saveSessionSnapshot, SESSION_SNAPSHOT_INTERVAL_MS);
 
 // Also try to save on clean shutdown (works with Ctrl+C, not window close)
-process.on('SIGINT', () => { stopOpencodeServer(); saveSessionSnapshot(); process.exit(0); });
-process.on('SIGTERM', () => { stopOpencodeServer(); saveSessionSnapshot(); process.exit(0); });
-process.on('exit', () => { stopOpencodeServer(); saveSessionSnapshot(); });
+// Manual termination paths (Ctrl+C, kill, parent shell exit) all flow through
+// gracefulShutdown — same teardown the in-app restart/shutdown/update paths
+// use. Without this, hand-killed servers still leave orphaned opencode +
+// agent children holding file handles, which blocks `npm install -g synabun`.
+process.on('SIGINT', async () => {
+  console.log('\n[server] SIGINT received.');
+  await gracefulShutdown('sigint');
+  process.exit(0);
+});
+process.on('SIGTERM', async () => {
+  console.log('\n[server] SIGTERM received.');
+  await gracefulShutdown('sigterm');
+  process.exit(0);
+});
+// `exit` fires after process.exit() — only safe for synchronous final tasks.
+// gracefulShutdown is async so we cannot await here; just snapshot + ocp stop
+// best-effort. The async paths above are the primary teardown.
+process.on('exit', () => { try { stopOpencodeServer(); } catch {} try { saveSessionSnapshot(); } catch {} });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "synabun",
-  "version": "2026.4.26-beta.6",
+  "version": "2026.4.26-beta.7",
   "description": "Persistent vector memory for AI assistants",
   "license": "Apache-2.0",
   "author": "The SynaBun Authors",

package/preuninstall.js

@@ -69,6 +69,80 @@ async function tryGracefulServerShutdown() {
   }
 }
 
+// ── Step 1b: kill any orphaned processes holding file handles ──
+//
+// Even after a graceful server shutdown (or when no server is running),
+// orphaned children can survive: opencode-ai sidecars spawned via cmd.exe,
+// MCP servers (claude-memory-mcp) spawned by Claude Code instances rather
+// than by SynaBun itself, agent processes, etc. On Windows these orphans
+// keep file handles inside the install dir open, causing EBUSY when npm
+// tries to rename it.
+//
+// We nuke any process whose command line references the SynaBun install
+// path. This catches every orphan regardless of how they got that way.
+// Skips the current PID (we are the preuninstall process itself).
+
+killOrphansInsideInstall();
+
+function killOrphansInsideInstall() {
+  const installPath = __dirname;
+  const selfPid = process.pid;
+  const plat = platform();
+
+  if (plat === 'win32') {
+    // PowerShell: Get-CimInstance returns every process with CommandLine.
+    // Filter for our install path (escape backslashes for the -like wildcard).
+    // Stop-Process -Force kills synchronously; -ErrorAction SilentlyContinue
+    // suppresses "process already exited" noise.
+    const escaped = installPath.replace(/\\/g, '\\\\').replace(/'/g, `''`);
+    const ps = `
+$ErrorActionPreference = 'SilentlyContinue';
+Get-CimInstance Win32_Process |
+  Where-Object {
+    $_.ProcessId -ne ${selfPid} -and (
+      ($_.CommandLine -and $_.CommandLine -like '*${escaped}*') -or
+      ($_.ExecutablePath -and $_.ExecutablePath -like '${escaped}*')
+    )
+  } |
+  ForEach-Object {
+    Write-Host ('  Killing orphan PID ' + $_.ProcessId + ' (' + $_.Name + ')');
+    Stop-Process -Id $_.ProcessId -Force -ErrorAction SilentlyContinue
+  }
+`.trim();
+    try {
+      execSync(`powershell -NoProfile -ExecutionPolicy Bypass -Command "${ps.replace(/"/g, '\\"').replace(/\n/g, ' ')}"`, {
+        stdio: 'inherit',
+        timeout: 8000,
+      });
+    } catch {
+      // PowerShell unavailable, blocked, or timed out. Fall through.
+    }
+  } else {
+    // Unix: pgrep -f matches against the full command line. -P 1 skips
+    // self/children of init we shouldn't touch. We just enumerate then
+    // SIGKILL each one that isn't us.
+    try {
+      const out = execSync(`pgrep -f "${installPath.replace(/[/.*?+^$|(){}\\\[\]]/g, '\\$&')}"`, {
+        encoding: 'utf8',
+        timeout: 5000,
+      }).trim();
+      const pids = out.split('\n')
+        .map(s => Number(s.trim()))
+        .filter(n => Number.isFinite(n) && n > 0 && n !== selfPid);
+      for (const pid of pids) {
+        try { process.kill(pid, 'SIGKILL'); } catch {}
+      }
+    } catch {
+      // No matches or pgrep missing — fine.
+    }
+  }
+
+  // Brief grace for OS to release file handles after killing.
+  // Synchronous busy-wait so we run before the rmSync calls below.
+  const until = Date.now() + 1500;
+  while (Date.now() < until) { /* spin */ }
+}
+
 // ── Step 2: remove nested node_modules ──
 
 const dirs = [