polydev-ai 1.5.1 → 1.5.2

package/lib/cliManager.js

@@ -379,17 +379,15 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
   }
 
   async sendCliPrompt(providerId, prompt, mode = 'args', timeoutMs = null) {
-    // Set provider-specific default timeouts
+    // Set provider-specific default timeouts (180s for all by default, complex prompts take time)
     if (timeoutMs === null) {
-      timeoutMs = providerId === 'claude_code' ? 60000 : 30000; // 60s for Claude Code, 30s for others
-    }
-    if (providerId === 'codex_cli' && timeoutMs < 90000) {
-      timeoutMs = 90000;
+      timeoutMs = 180000; // 180 seconds default for all providers
     }
     
     // Ensure timeoutMs is valid (not undefined, null, Infinity, or negative)
+    // Allow up to 300 seconds (5 minutes) for complex operations
     if (!timeoutMs || timeoutMs === Infinity || timeoutMs < 1 || timeoutMs > 300000) {
-      timeoutMs = 30000 // Default to 30 seconds
+      timeoutMs = 180000 // Default to 180 seconds
     }
     
     const startTime = Date.now();
@@ -514,10 +512,11 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
     }
   }
 
-  async executeCliCommand(command, args, mode = 'args', timeoutMs = 30000, stdinInput) {
+  async executeCliCommand(command, args, mode = 'args', timeoutMs = 180000, stdinInput) {
     // Ensure timeoutMs is valid (not undefined, null, Infinity, or negative)
+    // Allow up to 300 seconds (5 minutes) for complex operations
     if (!timeoutMs || timeoutMs === Infinity || timeoutMs < 1 || timeoutMs > 300000) {
-      timeoutMs = 30000 // Default to 30 seconds
+      timeoutMs = 180000 // Default to 180 seconds
     }
     
     return new Promise((resolve, reject) => {
@@ -672,17 +671,52 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
         stop(() => reject(new Error(`Codex exec timeout after ${timeoutMs}ms`)));
       }, timeoutMs);
 
+      // Helper function to parse Codex output robustly
+      const parseCodexOutput = (output) => {
+        if (!output || !output.trim()) return null;
+        
+        const lines = output.split('\n');
+        const contentLines = [];
+        
+        // Filter out MCP protocol noise and collect actual content
+        for (const line of lines) {
+          const trimmedLine = line.trim();
+          // Skip empty lines and MCP/protocol noise
+          if (!trimmedLine) continue;
+          if (trimmedLine.startsWith('MCP ')) continue;
+          if (trimmedLine.includes('handshake') && trimmedLine.includes('error')) continue;
+          if (trimmedLine.startsWith('[MCP]')) continue;
+          if (trimmedLine.startsWith('Error:') && trimmedLine.includes('MCP')) continue;
+          
+          // Extract bullet content if present, otherwise keep the line
+          const bulletMatch = trimmedLine.match(/^•\s*(.+)/);
+          if (bulletMatch) {
+            contentLines.push(bulletMatch[1].trim());
+          } else if (!trimmedLine.startsWith('•')) {
+            // Keep non-bullet content that isn't just whitespace
+            contentLines.push(trimmedLine);
+          }
+        }
+        
+        const result = contentLines.join('\n').trim();
+        return result.length > 0 ? result : null;
+      };
+
+      // Check if we have a complete response - look for actual content
       const flushIfComplete = () => {
-        const match = stdout.match(/•\s*(.+)/);
-        if (match && match[1]) {
+        const parsed = parseCodexOutput(stdout);
+        // Only resolve early if we have meaningful content (at least 2 chars) and output seems done
+        // This prevents cutting off multi-line responses
+        if (parsed && parsed.length >= 2 && stdout.includes('\n') && !stdout.endsWith('...')) {
           clearTimeout(timeoutHandle);
-          stop(() => resolve(match[1].trim()));
+          stop(() => resolve(parsed));
         }
       };
 
       child.stdout?.on('data', (data) => {
         stdout += data.toString();
-        flushIfComplete();
+        // Don't flush too eagerly - wait a bit for more data
+        // flushIfComplete(); // Disabled: let the process complete naturally
       });
 
       child.stderr?.on('data', (data) => {
@@ -698,12 +732,22 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
         const trimmedStderr = stderr.trim();
 
         if (code === 0 && trimmedStdout) {
-          const match = trimmedStdout.match(/•\s*(.+)/);
-          if (match && match[1]) {
-            resolve(match[1].trim());
+          // Use the robust parser for consistent output handling
+          const parsed = parseCodexOutput(trimmedStdout);
+          if (parsed) {
+            resolve(parsed);
             return;
           }
+          // Fallback to raw output if parsing yields nothing
           resolve(trimmedStdout);
+        } else if (code === 0) {
+          // Code 0 but no stdout - check if stderr has useful info
+          const parsedStderr = parseCodexOutput(trimmedStderr);
+          if (parsedStderr) {
+            resolve(parsedStderr);
+            return;
+          }
+          reject(new Error('Codex completed but produced no output'));
         } else {
           reject(new Error(trimmedStderr || trimmedStdout || `Codex exited with code ${code}`));
         }

package/mcp/stdio-wrapper.js

@@ -449,8 +449,8 @@ class StdioMCPWrapper {
         throw new Error('prompt is required');
       }
 
-      // Use reasonable timeout for CLI responses (15 seconds instead of 5)
-      const gracefulTimeout = Math.min(timeout_ms, 15000);
+      // Use reasonable timeout for CLI responses (180 seconds for complex prompts)
+      const gracefulTimeout = Math.min(timeout_ms, 180000);
 
       let localResults = [];
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.5.1",
+  "version": "1.5.2",
   "description": "Agentic workflow assistant with CLI integration - get diverse perspectives from multiple LLMs when stuck or need enhanced reasoning",
   "keywords": [
     "mcp",