@openagents-org/agent-connector 0.1.1 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openagents-org/agent-connector",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "Agent management CLI and library for OpenAgents — install, configure, and run AI coding agents",
   "main": "src/index.js",
   "bin": {

package/src/config.js

@@ -139,8 +139,10 @@ class Config {
       try {
         process.kill(pid, 0);
         return pid;
-      } catch {
-        // Stale PID file
+      } catch (e) {
+        // EPERM means the process exists but we lack permission (cross-session on Windows)
+        if (e.code === 'EPERM') return pid;
+        // ESRCH means no such process — stale PID file
         try { fs.unlinkSync(this.pidFile); } catch {}
         return null;
       }

package/src/daemon.js

@@ -481,20 +481,34 @@ class Daemon {
 
       this._log(`${agentCfg.name} CLI: ${binary} ${args.slice(0, 5).join(' ')} ...`);
 
+      const spawnEnv = { ...env };
+      if (IS_WINDOWS) {
+        const npmBin = path.join(process.env.APPDATA || '', 'npm');
+        if (npmBin && !(spawnEnv.PATH || '').includes(npmBin)) {
+          spawnEnv.PATH = npmBin + ';' + (spawnEnv.PATH || process.env.PATH || '');
+        }
+      }
+
+      // On Windows, .cmd shims need shell:true but that breaks argument
+      // quoting. Use execFileSync-style approach: resolve the .cmd to its
+      // target and run directly, or use spawn without shell and full path.
+      let spawnBinary = binary;
+      let spawnArgs = args;
       const spawnOpts = {
         stdio: ['ignore', 'pipe', 'pipe'],
-        env,
+        env: spawnEnv,
         timeout: 600000,
       };
+
       if (IS_WINDOWS) {
-        spawnOpts.shell = true;
-        const npmBin = path.join(process.env.APPDATA || '', 'npm');
-        if (npmBin && !(env.PATH || '').includes(npmBin)) {
-          spawnOpts.env = { ...env, PATH: npmBin + ';' + (env.PATH || process.env.PATH || '') };
-        }
+        // Find the actual .cmd shim and invoke via cmd.exe /C with proper quoting
+        spawnBinary = process.env.COMSPEC || 'cmd.exe';
+        // Wrap argument containing spaces in double quotes for cmd.exe
+        const quotedArgs = args.map((a) => a.includes(' ') ? `"${a}"` : a);
+        spawnArgs = ['/C', binary, ...quotedArgs];
       }
 
-      const proc = spawn(binary, args, spawnOpts);
+      const proc = spawn(spawnBinary, spawnArgs, spawnOpts);
       let stdout = '';
       let stderr = '';