polydev-ai 1.8.45 → 1.8.50

package/mcp/server.js

@@ -595,13 +595,13 @@ class MCPServer {
     console.log('[MCP Server] Send CLI prompt requested');
     
     try {
-      const { provider_id, prompt, mode = 'args', timeout_ms = 90000, user_id } = args;
+      const { provider_id, prompt, mode = 'args', timeout_ms = 240000, user_id } = args;
 
       // Ensure timeout_ms is valid (not undefined, null, Infinity, or negative)
-      // 90 seconds default for CLI-within-CLI scenarios (Claude Code calling Claude Code)
+      // 240 seconds (4 minutes) default for CLI-within-CLI scenarios (Claude Code calling Claude Code)
       let validTimeout = timeout_ms;
       if (!validTimeout || validTimeout === Infinity || validTimeout < 1 || validTimeout > 600000) {
-        validTimeout = 90000 // Default to 90 seconds for CLI responses
+        validTimeout = 240000 // Default to 240 seconds (4 minutes) for CLI responses
       }
       
       if (!provider_id || !prompt) {

package/mcp/stdio-wrapper.js

@@ -602,12 +602,12 @@ class StdioMCPWrapper {
     console.error(`[Stdio Wrapper] Local CLI prompt sending with perspectives`);
     
     try {
-      let { provider_id, prompt, mode = 'args', timeout_ms = 300000 } = args;
+      let { provider_id, prompt, mode = 'args', timeout_ms = 240000 } = args;
       
       // Ensure timeout_ms is valid (not undefined, null, Infinity, or negative)
-      // Default to 5 minutes (300 seconds) for complex CLI responses
+      // Default to 4 minutes (240 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
+        timeout_ms = 240000; // Default to 4 minutes for CLI responses
       }
       
       if (!prompt) {
@@ -733,14 +733,19 @@ class StdioMCPWrapper {
       const successfulLocalCount = localResults.filter(r => r.success).length;
       const failedCliCount = localResults.filter(r => !r.success).length;
       
-      // Need API for: API-only providers + failed CLIs
+      // Need API for: remaining perspectives to reach maxPerspectives
+      // OLD (WRONG): const remainingPerspectives = apiOnlyCount + failedCliCount;
+      // This was wrong because if allProviders was empty, apiOnlyCount=0 and we'd get too few perspectives
+      // NEW (CORRECT): Always try to fill up to maxPerspectives from remote API
       const apiOnlyCount = (this._apiOnlyProviders || []).length;
-      const remainingPerspectives = apiOnlyCount + failedCliCount;
+      const remainingPerspectives = Math.max(0, maxPerspectives - successfulLocalCount);
+      
+      console.error(`[Stdio Wrapper] Perspectives breakdown: ${successfulLocalCount} successful CLI, ${failedCliCount} failed CLI, ${apiOnlyCount} API-only, need ${remainingPerspectives} more from remote (target: ${maxPerspectives})`);
 
       // Get remote perspectives for API-only providers and failed CLIs
       let perspectivesResult;
       if (remainingPerspectives > 0) {
-        console.error(`[Stdio Wrapper] Need ${remainingPerspectives} perspectives from remote API (${apiOnlyCount} API-only + ${failedCliCount} failed CLIs)`);
+        console.error(`[Stdio Wrapper] Calling remote API for ${remainingPerspectives} more perspectives (have ${successfulLocalCount}/${maxPerspectives})`);
         
         // Pass API-only provider info to help remote API choose correct models
         const apiProvidersInfo = (this._apiOnlyProviders || []).map(p => ({
@@ -752,7 +757,7 @@ class StdioMCPWrapper {
         // We do NOT call reportCliResultsToServer() here to avoid duplicate logs
         perspectivesResult = await this.callPerspectivesForCli(args, localResults, remainingPerspectives, apiProvidersInfo);
       } else {
-        console.error(`[Stdio Wrapper] Already have ${successfulLocalCount} perspectives from CLIs, skipping remote call`);
+        console.error(`[Stdio Wrapper] Already have ${successfulLocalCount} perspectives (max: ${maxPerspectives})`);
         
         // ONLY report CLI results when NOT calling remote API
         // (When we call remote API, it handles logging CLI + API results together)

package/package.json

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