polydev-ai 1.8.8 → 1.8.10

package/lib/cliManager.js

@@ -379,15 +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, model = null) {
-    // Set provider-specific default timeouts (180s for all by default, complex prompts take time)
+    // Set provider-specific default timeouts (5 minutes for all by default, complex prompts take time)
     if (timeoutMs === null) {
-      timeoutMs = 180000; // 180 seconds default for all providers
+      timeoutMs = 300000; // 300 seconds (5 minutes) 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 = 180000 // Default to 180 seconds
+    // Allow up to 600 seconds (10 minutes) for very complex operations
+    if (!timeoutMs || timeoutMs === Infinity || timeoutMs < 1 || timeoutMs > 600000) {
+      timeoutMs = 300000 // Default to 5 minutes
     }
     
     const startTime = Date.now();
@@ -644,9 +644,50 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
 
       let stdout = '';
       let stderr = '';
+      let resolved = false;
+      let debounceTimer = null;
+
+      // Helper to check if output looks complete (for JSON output from claude code)
+      const looksComplete = () => {
+        const trimmed = stdout.trim();
+        // For JSON output, check if it's valid JSON
+        if (trimmed.startsWith('{') || trimmed.startsWith('[')) {
+          try {
+            JSON.parse(trimmed);
+            return true; // Valid JSON means complete
+          } catch {
+            return false;
+          }
+        }
+        // For text output, check if we have substantial content
+        return trimmed.length > 50 && trimmed.includes('\n');
+      };
+
+      const doResolve = () => {
+        if (resolved) return;
+        resolved = true;
+        if (debounceTimer) clearTimeout(debounceTimer);
+        if (timeoutId) clearTimeout(timeoutId);
+        if (!child.killed) {
+          try { child.kill('SIGTERM'); } catch(_) {}
+        }
+        resolve({ stdout, stderr });
+      };
+
+      // Schedule early return check after debounce period
+      const scheduleEarlyReturn = () => {
+        if (debounceTimer) clearTimeout(debounceTimer);
+        debounceTimer = setTimeout(() => {
+          if (!resolved && looksComplete()) {
+            console.log('[CLI Debug] Early return - output looks complete');
+            doResolve();
+          }
+        }, 2000); // Wait 2 seconds after last data for Claude Code (faster JSON parsing)
+      };
 
       child.stdout?.on('data', (data) => {
         stdout += data.toString();
+        scheduleEarlyReturn();
       });
 
       child.stderr?.on('data', (data) => {
@@ -659,6 +700,11 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
       }
 
       child.on('close', (code) => {
+        if (resolved) return;
+        resolved = true;
+        if (debounceTimer) clearTimeout(debounceTimer);
+        if (timeoutId) clearTimeout(timeoutId);
+        
         if (process.env.POLYDEV_CLI_DEBUG) {
           console.log(`[CLI Debug] Command finished with code ${code}`);
         }
@@ -679,6 +725,10 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
       });
 
       child.on('error', (error) => {
+        if (resolved) return;
+        resolved = true;
+        if (debounceTimer) clearTimeout(debounceTimer);
+        if (timeoutId) clearTimeout(timeoutId);
         if (process.env.POLYDEV_CLI_DEBUG) {
           console.log(`[CLI Debug] Command error:`, error);
         }
@@ -686,15 +736,11 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
       });
 
       let timeoutId;
-      const cleanup = () => {
-        if (timeoutId) {
-          clearTimeout(timeoutId);
-          timeoutId = null;
-        }
-      };
 
       timeoutId = setTimeout(() => {
-        cleanup();
+        if (resolved) return;
+        resolved = true;
+        if (debounceTimer) clearTimeout(debounceTimer);
         if (!child.killed) {
           child.kill('SIGTERM');
           // Force kill after 2 seconds if still running
@@ -707,12 +753,8 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
         reject(new Error(`Command timeout after ${timeoutMs}ms`));
       }, timeoutMs);
 
-      child.on('close', () => {
-        cleanup();
-      });
-
       child.on('exit', () => {
-        cleanup();
+        if (debounceTimer) clearTimeout(debounceTimer);
       });
     });
   }
@@ -888,10 +930,12 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
       let stdout = '';
       let stderr = '';
       let resolved = false;
+      let debounceTimer = null; // Smart debounce timer for early return
 
       const stop = (handler) => {
         if (!resolved) {
           resolved = true;
+          if (debounceTimer) clearTimeout(debounceTimer);
           try { child.kill('SIGTERM'); } catch (_) {}
           handler();
         }
@@ -981,18 +1025,31 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
       // Check if we have a complete response - look for actual content
       const flushIfComplete = () => {
         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('...')) {
+        // Only resolve early if we have meaningful content (at least 20 chars) and output looks complete
+        // Look for signs that Codex has finished outputting (tokens used, empty lines at end, etc.)
+        if (parsed && parsed.length >= 20) {
+          const detectedModel = this.detectModelFromOutput('codex_cli', stdout, stderr);
           clearTimeout(timeoutHandle);
-          stop(() => resolve(parsed));
+          if (debounceTimer) clearTimeout(debounceTimer);
+          stop(() => resolve({ content: parsed, detectedModel, rawStdout: stdout.trim(), rawStderr: stderr.trim() }));
         }
       };
 
+      // Smart debounce: wait 3 seconds after last data received before checking for early return
+      const scheduleEarlyReturn = () => {
+        if (debounceTimer) clearTimeout(debounceTimer);
+        debounceTimer = setTimeout(() => {
+          if (!resolved) {
+            console.log('[CLI Debug] Checking for early return after debounce...');
+            flushIfComplete();
+          }
+        }, 3000); // Wait 3 seconds after last data received
+      };
+
       child.stdout?.on('data', (data) => {
         stdout += data.toString();
-        // Don't flush too eagerly - wait a bit for more data
-        // flushIfComplete(); // Disabled: let the process complete naturally
+        // Schedule early return check after debounce period
+        scheduleEarlyReturn();
       });
 
       child.stderr?.on('data', (data) => {

package/mcp/stdio-wrapper.js

@@ -443,19 +443,20 @@ class StdioMCPWrapper {
     console.error(`[Stdio Wrapper] Local CLI prompt sending with perspectives`);
     
     try {
-      let { provider_id, prompt, mode = 'args', timeout_ms = 180000 } = args;
+      let { provider_id, prompt, mode = 'args', timeout_ms = 300000 } = args;
       
       // Ensure timeout_ms is valid (not undefined, null, Infinity, or negative)
-      if (!timeout_ms || timeout_ms === Infinity || timeout_ms < 1 || timeout_ms > 300000) {
-        timeout_ms = 180000; // Default to 180 seconds for CLI responses
+      // Default to 5 minutes (300 seconds) for complex CLI responses
+      if (!timeout_ms || timeout_ms === Infinity || timeout_ms < 1 || timeout_ms > 600000) {
+        timeout_ms = 300000; // Default to 5 minutes for CLI responses
       }
       
       if (!prompt) {
         throw new Error('prompt is required');
       }
 
-      // Use configured timeout but cap at 5 minutes max
-      const gracefulTimeout = Math.min(timeout_ms, 300000);
+      // Use configured timeout but cap at 10 minutes max for safety
+      const gracefulTimeout = Math.min(timeout_ms, 600000);
 
       // Fetch user's model preferences (cached, non-blocking on failure)
       let modelPreferences = {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.8.8",
+  "version": "1.8.10",
   "description": "Agentic workflow assistant with CLI integration - get diverse perspectives from multiple LLMs when stuck or need enhanced reasoning",
   "keywords": [
     "mcp",
@@ -67,7 +67,7 @@
     "lucide-react": "^0.542.0",
     "marked": "^16.2.1",
     "next": "^15.5.7",
-    "polydev-ai": "^1.8.7",
+    "polydev-ai": "^1.8.9",
     "posthog-js": "^1.157.2",
     "prismjs": "^1.30.0",
     "react": "^18.3.1",