polydev-ai 1.8.42 → 1.8.44

package/lib/cliManager.js

@@ -585,8 +585,9 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
             args = ['--model', model, ...args, prompt];
           } else if (providerId === 'gemini_cli') {
             // Gemini CLI: -m for model, -p for prompt (headless mode)
-            // Format: gemini -m gemini-2.0-flash -p "prompt"
-            args = ['-m', model, '-p', prompt];
+            // Add prompt prefix to prevent tool planning in non-interactive mode
+            const geminiPrompt = `Answer directly without using any tools, file operations, or searches. Do not say "I will search" or "I will look up". Provide your analysis immediately.\n\n${prompt}`;
+            args = ['-m', model, '-p', geminiPrompt];
           } else {
             // Default: just append prompt
             args = [...args, prompt];
@@ -594,8 +595,10 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
         } else {
           // No model specified
           if (providerId === 'gemini_cli') {
-            // Gemini CLI still needs -p flag for headless mode
-            args = ['-p', prompt];
+            // Gemini CLI: -p for headless mode
+            // Add prompt prefix to prevent tool planning in non-interactive mode
+            const geminiPrompt = `Answer directly without using any tools, file operations, or searches. Do not say "I will search" or "I will look up". Provide your analysis immediately.\n\n${prompt}`;
+            args = ['-p', geminiPrompt];
           } else {
             args = [...args, prompt];
           }
@@ -752,7 +755,13 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
       const child = spawn(command, args, {
         stdio: ['pipe', 'pipe', 'pipe'],
         shell: process.platform === 'win32',
-        timeout: timeoutMs
+        timeout: timeoutMs,
+        // Explicitly pass environment to ensure HOME is available for CLI tools
+        // that read config from ~/.config or similar paths (e.g., Gemini CLI)
+        env: {
+          ...process.env,
+          HOME: process.env.HOME || os.homedir()
+        }
       });
 
       if (child.stdin) {

package/mcp/stdio-wrapper.js

@@ -265,7 +265,14 @@ class StdioMCPWrapper {
     // Smart refresh scheduler (will be started after initialization)
     this.refreshScheduler = null;
     
-    // Cache for user model preferences (provider -> model)
+    // CLI detection readiness tracking - requests will wait for this
+    // Create the promise NOW in constructor so it exists before any requests arrive
+    this._cliDetectionComplete = false;
+    this._cliDetectionReady = new Promise((resolve) => {
+      this._cliDetectionResolver = resolve;
+    });
+    
+    // Cache for user model preferences
     this.userModelPreferences = null;
     this.perspectivesPerMessage = 2; // Default to 2 perspectives
     this.modelPreferencesCacheTime = null;
@@ -514,6 +521,7 @@ class StdioMCPWrapper {
       
       // Force detection using CLI Manager (no remote API calls)
       const results = await this.cliManager.forceCliDetection(providerId);
+      console.error(`[Stdio Wrapper] CLI detection results:`, JSON.stringify(results, null, 2));
       
       // Save status locally to file-based cache
       await this.saveLocalCliStatus(results);
@@ -651,7 +659,10 @@ class StdioMCPWrapper {
         if (allProviders.length === 0) {
           // Fallback: use legacy CLI-only flow
           console.error(`[Stdio Wrapper] No allProviders, using legacy CLI-only flow`);
-          const userOrder = this.userProviderOrder || ['claude_code', 'codex_cli', 'gemini_cli'];
+          // NOTE: Use length check because empty array [] is truthy in JS
+          const userOrder = (this.userProviderOrder && this.userProviderOrder.length > 0) 
+            ? this.userProviderOrder 
+            : ['claude_code', 'codex_cli', 'gemini_cli'];
           const cliProviders = userOrder.slice(0, maxPerspectives).filter(p => availableClis.includes(p));
           
           const cliPromises = cliProviders.map(async (providerId) => {
@@ -788,13 +799,24 @@ class StdioMCPWrapper {
    */
   async getAllAvailableProviders() {
     try {
+      // Wait for initial CLI detection to complete if it's still running
+      if (this._cliDetectionReady && !this._cliDetectionComplete) {
+        console.error('[Stdio Wrapper] Waiting for initial CLI detection to complete...');
+        await this._cliDetectionReady;
+        console.error('[Stdio Wrapper] CLI detection ready, proceeding with request');
+      }
+      
       const results = await this.cliManager.forceCliDetection();
+      console.error(`[Stdio Wrapper] CLI detection results:`, JSON.stringify(results, null, 2));
       const availableProviders = [];
       const unavailableProviders = [];
       
       // Use user's provider order from dashboard (fetched via model-preferences API)
-      // Falls back to default order if not yet loaded
-      const priorityOrder = this.userProviderOrder || ['claude_code', 'codex_cli', 'gemini_cli'];
+      // Falls back to default order if not yet loaded or empty
+      // NOTE: Use length check because empty array [] is truthy in JS
+      const priorityOrder = (this.userProviderOrder && this.userProviderOrder.length > 0) 
+        ? this.userProviderOrder 
+        : ['claude_code', 'codex_cli', 'gemini_cli'];
       console.error(`[Stdio Wrapper] Using provider order: ${priorityOrder.join(' > ')}`);
       
       for (const providerId of priorityOrder) {
@@ -1541,7 +1563,10 @@ class StdioMCPWrapper {
         // Cache provider order from user's dashboard (respects display_order)
         // This determines which CLIs/APIs to use first
         // IMPORTANT: This is cached alongside modelPreferences and restored when cache is used
-        this.userProviderOrder = result.providerOrder || ['claude_code', 'codex_cli', 'gemini_cli'];
+        // NOTE: Use length check because empty array [] is truthy in JS
+        this.userProviderOrder = (result.providerOrder && result.providerOrder.length > 0) 
+          ? result.providerOrder 
+          : ['claude_code', 'codex_cli', 'gemini_cli'];
         
         // NEW: Cache full list of all providers (CLI + API-only) from dashboard
         // Format: [{ provider: 'openai', model: 'gpt-52-codex', cliId: 'codex_cli' }, { provider: 'x-ai', model: 'grok-4', cliId: null }, ...]
@@ -1645,15 +1670,23 @@ class StdioMCPWrapper {
     console.error('Stdio MCP Wrapper ready and listening on stdin...');
 
     // Run initial CLI detection in the background so MCP handshake isn't blocked.
+    // The promise was already created in constructor so requests will wait for it
     console.error('[Stdio Wrapper] Running initial CLI detection...');
+    
     this.localForceCliDetection({})
       .then(() => {
         console.error('[Stdio Wrapper] Initial CLI detection completed');
+        this._cliDetectionComplete = true;
       })
       .catch((error) => {
         console.error('[Stdio Wrapper] Initial CLI detection failed:', error);
+        this._cliDetectionComplete = true; // Mark complete even on failure
       })
       .finally(() => {
+        // Resolve the readiness promise so waiting requests can proceed
+        if (this._cliDetectionResolver) {
+          this._cliDetectionResolver();
+        }
         // Start smart refresh scheduler after initial detection attempt
         this.startSmartRefreshScheduler();
       });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.8.42",
+  "version": "1.8.44",
   "mcpName": "io.github.backspacevenkat/perspectives",
   "description": "Agentic workflow assistant with CLI integration - get diverse perspectives from multiple LLMs when stuck or need enhanced reasoning",
   "keywords": [
@@ -68,7 +68,7 @@
     "lucide-react": "^0.542.0",
     "marked": "^16.2.1",
     "next": "^15.5.7",
-    "polydev-ai": "^1.8.40",
+    "polydev-ai": "^1.8.42",
     "posthog-js": "^1.157.2",
     "prismjs": "^1.30.0",
     "react": "^18.3.1",