@openagents-org/agent-connector 0.1.7 → 0.1.8

package/package.json

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

package/src/daemon.js

@@ -475,6 +475,35 @@ class Daemon {
    * Build workspace context preamble for CLI agents.
    * Teaches the agent about shared workspace APIs (browser, files).
    */
+  /**
+   * On Windows, resolve a .cmd shim to the underlying node script
+   * so we can spawn directly with node (avoiding cmd.exe argument limits).
+   */
+  _resolveWindowsBinary(binary, env) {
+    const { execSync } = require('child_process');
+    try {
+      // Find the .cmd shim
+      const cmdPath = execSync(`where ${binary}`, {
+        encoding: 'utf-8', timeout: 5000, env,
+      }).split(/\r?\n/)[0].trim();
+
+      if (cmdPath.endsWith('.cmd')) {
+        // Read the .cmd file to find the target JS script
+        const cmdContent = fs.readFileSync(cmdPath, 'utf-8');
+        // npm .cmd shims have: "%~dp0\node_modules\...\bin\cli.js" %*
+        // or: @IF EXIST "%~dp0\node.exe" ... "%~dp0\node_modules\...\cli.js" %*
+        const match = cmdContent.match(/"([^"]+\.js)"/);
+        if (match) {
+          const jsPath = match[1].replace('%~dp0\\', path.dirname(cmdPath) + '\\');
+          return { binary: process.execPath, prefix: [jsPath] };
+        }
+      }
+    } catch {}
+
+    // Fallback: use cmd.exe /C (may truncate long args)
+    return { binary: process.env.COMSPEC || 'cmd.exe', prefix: ['/C', binary] };
+  }
+
   _buildWorkspaceContext(agentCfg, network) {
     const baseUrl = 'https://workspace-endpoint.openagents.org';
     const h = `Authorization: Bearer ${network.token}`;
@@ -534,9 +563,6 @@ class Daemon {
         }
       }
 
-      // 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 = {
@@ -546,11 +572,11 @@ class Daemon {
       };
 
       if (IS_WINDOWS) {
-        // 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];
+        // Resolve .cmd shim to actual node script to avoid cmd.exe
+        // argument mangling and length limits
+        const resolved = this._resolveWindowsBinary(binary, spawnEnv);
+        spawnBinary = resolved.binary;
+        spawnArgs = [...resolved.prefix, ...args];
       }
 
       const proc = spawn(spawnBinary, spawnArgs, spawnOpts);