polydev-ai 1.8.77 → 1.8.79

package/mcp/stdio-wrapper.js

@@ -400,7 +400,7 @@ class StdioMCPWrapper {
    */
   async handleLoginTool(params, id) {
     const http = require('http');
-    const { execFile } = require('child_process');
+    const { spawn } = require('child_process');
     
     return new Promise((resolve) => {
       // Check if already authenticated
@@ -479,26 +479,17 @@ class StdioMCPWrapper {
 
         console.error(`[Polydev] Opening browser for authentication: ${authUrl}`);
         
-        // Open browser
+        // 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];
+          spawn('open', [authUrl], { detached: true, stdio: 'ignore' }).unref();
         } else if (platform === 'win32') {
-          cmd = 'cmd';
-          args = ['/c', 'start', '', authUrl];
+          spawn('cmd', ['/c', 'start', '', authUrl], { detached: true, stdio: 'ignore', shell: true }).unref();
         } else {
-          cmd = 'xdg-open';
-          args = [authUrl];
+          spawn('xdg-open', [authUrl], { detached: true, stdio: 'ignore' }).unref();
         }
 
-        execFile(cmd, args, (err) => {
-          if (err) {
-            console.error('[Polydev] Could not open browser:', err.message);
-          }
-        });
-
         // Timeout after 5 minutes
         setTimeout(() => {
           server.close();
@@ -1984,25 +1975,32 @@ class StdioMCPWrapper {
     // Run the onboarding/status check
     await this.runStartupFlow();
 
-    // Run CLI detection in background
-    console.error('[Polydev] Detecting local CLI tools...');
-    
-    this.localForceCliDetection({})
-      .then(async () => {
-        this._cliDetectionComplete = true;
-        // Display CLI status after detection
-        await this.displayCliStatus();
-      })
-      .catch((error) => {
-        console.error('[Polydev] CLI detection failed:', error.message);
-        this._cliDetectionComplete = true;
-      })
-      .finally(() => {
-        if (this._cliDetectionResolver) {
-          this._cliDetectionResolver();
-        }
-        this.startSmartRefreshScheduler();
-      });
+    // Only run CLI detection here if we already have a token
+    // (If no token, CLI detection runs after login completes in runStartupFlow)
+    if (this.userToken) {
+      console.error('[Polydev] Detecting local CLI tools...');
+      
+      this.localForceCliDetection({})
+        .then(async () => {
+          this._cliDetectionComplete = true;
+          // Display CLI status after detection
+          await this.displayCliStatus();
+        })
+        .catch((error) => {
+          console.error('[Polydev] CLI detection failed:', error.message);
+          this._cliDetectionComplete = true;
+        })
+        .finally(() => {
+          if (this._cliDetectionResolver) {
+            this._cliDetectionResolver();
+          }
+          this.startSmartRefreshScheduler();
+        });
+    } else {
+      // No token - CLI detection will run after login completes
+      // Just start the scheduler
+      this.startSmartRefreshScheduler();
+    }
   }
 
   /**
@@ -2010,16 +2008,28 @@ class StdioMCPWrapper {
    */
   async runStartupFlow() {
     if (!this.userToken) {
-      // No token - auto-open browser for login
+      // No token - auto-open browser for login (NON-BLOCKING)
       console.error('─'.repeat(50));
       console.error('Polydev - First Time Setup');
       console.error('─'.repeat(50));
       console.error('Opening browser for authentication...\n');
       
-      await this.autoLoginOnStartup();
+      // Run auto-login in background (don't block MCP server startup)
+      this.autoLoginOnStartup().then(() => {
+        console.error('[Polydev] Login completed, running CLI detection...');
+        // After login, run CLI detection
+        return this.localForceCliDetection({});
+      }).then(() => {
+        this._cliDetectionComplete = true;
+        return this.displayCliStatus();
+      }).catch((error) => {
+        console.error('[Polydev] Auto-login flow error:', error.message);
+      });
     } else {
-      // Has token - verify it
-      await this.verifyAndDisplayAuth();
+      // Has token - verify it (also non-blocking)
+      this.verifyAndDisplayAuth().catch((error) => {
+        console.error('[Polydev] Auth verification error:', error.message);
+      });
     }
   }
 
@@ -2028,7 +2038,7 @@ class StdioMCPWrapper {
    */
   async autoLoginOnStartup() {
     const http = require('http');
-    const { execFile } = require('child_process');
+    const { spawn } = require('child_process');
 
     return new Promise((resolve) => {
       const server = http.createServer((req, res) => {
@@ -2088,7 +2098,7 @@ class StdioMCPWrapper {
         console.error(authUrl);
         console.error('');
 
-        // Open browser
+        // Open browser - use spawn with detached for better reliability in stdio context
         const platform = process.platform;
         let cmd, args;
         if (platform === 'darwin') {
@@ -2102,11 +2112,7 @@ class StdioMCPWrapper {
           args = [authUrl];
         }
 
-        execFile(cmd, args, (err) => {
-          if (err) {
-            console.error('[!] Could not open browser automatically.');
-          }
-        });
+        spawn(cmd, args, { detached: true, stdio: 'ignore' }).unref();
 
         // Don't block forever - resolve after timeout
         setTimeout(() => {
@@ -2212,16 +2218,16 @@ class StdioMCPWrapper {
         console.error('Opening Polydev Models page for API key configuration...');
         console.error('Add your API keys to unlock more models!\n');
         
-        const { execFile } = require('child_process');
+        const { spawn } = require('child_process');
         const modelsUrl = 'https://polydev.ai/dashboard/models';
         const platform = process.platform;
         
         if (platform === 'darwin') {
-          execFile('open', [modelsUrl]);
+          spawn('open', [modelsUrl], { detached: true, stdio: 'ignore' }).unref();
         } else if (platform === 'win32') {
-          execFile('cmd', ['/c', 'start', '', modelsUrl]);
+          spawn('cmd', ['/c', 'start', '', modelsUrl], { detached: true, stdio: 'ignore', shell: true }).unref();
         } else {
-          execFile('xdg-open', [modelsUrl]);
+          spawn('xdg-open', [modelsUrl], { detached: true, stdio: 'ignore' }).unref();
         }
       }
     } catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.8.77",
+  "version": "1.8.79",
   "engines": {
     "node": ">=20.x <=22.x"
   },