polydev-ai 1.8.79 → 1.8.81

package/mcp/stdio-wrapper.js

@@ -479,16 +479,12 @@ class StdioMCPWrapper {
 
         console.error(`[Polydev] Opening browser for authentication: ${authUrl}`);
         
-        // Open browser - use spawn with detached for better reliability in stdio context
-        const platform = process.platform;
-        
-        if (platform === 'darwin') {
-          spawn('open', [authUrl], { detached: true, stdio: 'ignore' }).unref();
-        } else if (platform === 'win32') {
-          spawn('cmd', ['/c', 'start', '', authUrl], { detached: true, stdio: 'ignore', shell: true }).unref();
-        } else {
-          spawn('xdg-open', [authUrl], { detached: true, stdio: 'ignore' }).unref();
-        }
+        // Best-in-class browser opening using 'open' package (cross-platform)
+        // Falls back to platform-specific commands if package fails
+        this.openBrowser(authUrl).catch(() => {
+          console.error('[Polydev] All browser open methods failed');
+          console.error('[Polydev] Please open this URL manually:', authUrl);
+        });
 
         // Timeout after 5 minutes
         setTimeout(() => {
@@ -595,6 +591,64 @@ class StdioMCPWrapper {
     }
   }
 
+  /**
+   * Open URL in browser - best-in-class cross-platform implementation
+   * Uses 'open' npm package with fallbacks for reliability in MCP/stdio context
+   */
+  async openBrowser(url) {
+    // Try the 'open' package first (ESM, requires dynamic import)
+    try {
+      const open = await import('open');
+      await open.default(url);
+      console.error('[Polydev] Browser opened successfully');
+      return;
+    } catch (openError) {
+      console.error('[Polydev] open package failed:', openError.message);
+    }
+
+    // Fallback 1: Platform-specific commands with exec
+    const { exec } = require('child_process');
+    const platform = process.platform;
+    
+    return new Promise((resolve, reject) => {
+      let command;
+      if (platform === 'darwin') {
+        // macOS: try osascript first (more reliable in headless contexts)
+        command = `osascript -e 'open location "${url}"'`;
+      } else if (platform === 'win32') {
+        command = `start "" "${url}"`;
+      } else {
+        command = `xdg-open "${url}"`;
+      }
+
+      exec(command, (error) => {
+        if (error) {
+          console.error('[Polydev] exec command failed:', error.message);
+          
+          // Fallback 2: spawn with detached
+          const { spawn } = require('child_process');
+          try {
+            if (platform === 'darwin') {
+              spawn('open', [url], { detached: true, stdio: 'ignore' }).unref();
+            } else if (platform === 'win32') {
+              spawn('cmd', ['/c', 'start', '', url], { detached: true, stdio: 'ignore', shell: true }).unref();
+            } else {
+              spawn('xdg-open', [url], { detached: true, stdio: 'ignore' }).unref();
+            }
+            console.error('[Polydev] Browser opened via spawn fallback');
+            resolve();
+          } catch (spawnError) {
+            console.error('[Polydev] spawn fallback failed:', spawnError.message);
+            reject(new Error('All browser open methods failed'));
+          }
+        } else {
+          console.error('[Polydev] Browser opened via exec');
+          resolve();
+        }
+      });
+    });
+  }
+
   /**
    * Save token to shell config files
    */
@@ -2098,21 +2152,12 @@ class StdioMCPWrapper {
         console.error(authUrl);
         console.error('');
 
-        // Open browser - use spawn with detached for better reliability in stdio context
-        const platform = process.platform;
-        let cmd, args;
-        if (platform === 'darwin') {
-          cmd = 'open';
-          args = [authUrl];
-        } else if (platform === 'win32') {
-          cmd = 'cmd';
-          args = ['/c', 'start', '', authUrl];
-        } else {
-          cmd = 'xdg-open';
-          args = [authUrl];
-        }
-
-        spawn(cmd, args, { detached: true, stdio: 'ignore' }).unref();
+        // Best-in-class browser opening using 'open' package (cross-platform)
+        // Falls back to platform-specific commands if package fails
+        this.openBrowser(authUrl).catch(() => {
+          console.error('[Polydev] All browser open methods failed');
+          console.error('[Polydev] Please open this URL manually:', authUrl);
+        });
 
         // Don't block forever - resolve after timeout
         setTimeout(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.8.79",
+  "version": "1.8.81",
   "engines": {
     "node": ">=20.x <=22.x"
   },
@@ -73,6 +73,7 @@
     "lucide-react": "^0.542.0",
     "marked": "^16.2.1",
     "next": "^15.5.7",
+    "open": "^11.0.0",
     "polydev-ai": "^1.8.42",
     "posthog-js": "^1.157.2",
     "prismjs": "^1.30.0",