polydev-ai 1.8.9 → 1.8.10

package/lib/cliManager.js

@@ -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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.8.9",
+  "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",