polydev-ai 1.9.14 → 1.9.15

package/mcp/stdio-wrapper.js

@@ -1769,9 +1769,10 @@ Error: ${error.message}`
         const finalProviders = [];
         const usedProviderNames = new Set();
         
-        // STEP 1: Add ALL available CLIs first (in priority order) - they're FREE
+        // STEP 1: Add ALL available CLIs (in priority order) - they're FREE
+        // Don't limit to maxPerspectives here — we run all CLIs in parallel
+        // and take the first maxPerspectives successes (fast-collect pattern)
         for (const cliId of cliPriorityOrder) {
-          if (finalProviders.length >= maxPerspectives) break;
           if (!availableClis.includes(cliId)) continue;
           
           const providerName = cliToProviderMap[cliId];
@@ -1817,7 +1818,7 @@ Error: ${error.message}`
           console.error(`[Stdio Wrapper] [CLI-FIRST] Added API/credits: ${normalizedProvider} (${p.model})${p.tier ? ` [${p.tier}]` : ''}`);
         }
         
-        console.error(`[Stdio Wrapper] Final provider list (${finalProviders.length}/${maxPerspectives}): ${finalProviders.map(p => `${p.provider}[${p.source}]`).join(', ')}`);
+        console.error(`[Stdio Wrapper] Final provider list (${finalProviders.length}, need ${maxPerspectives}): ${finalProviders.map(p => `${p.provider}[${p.source}]`).join(', ')}`);
         
         // Separate into CLI entries (local execution) vs API entries (remote execution)
         const cliProviderEntries = finalProviders.filter(p => p.source === 'cli');
@@ -1825,7 +1826,8 @@ Error: ${error.message}`
         
         console.error(`[Stdio Wrapper] Provider breakdown: CLI=${cliProviderEntries.map(p => p.cliId).join(', ') || 'none'}, API-only=${apiOnlyProviders.map(p => p.provider).join(', ') || 'none'}`);
         
-        // Run all CLI prompts concurrently
+        // Run ALL CLI prompts concurrently with fast-collect pattern
+        // Resolves once we have maxPerspectives successes (don't wait for slow CLIs)
         if (cliProviderEntries.length > 0) {
           const cliPromises = cliProviderEntries.map(async (providerEntry) => {
             try {
@@ -1866,7 +1868,10 @@ Error: ${error.message}`
               };
             }
           });
-          localResults = await Promise.all(cliPromises);
+          
+          // Fast-collect: resolve once we have maxPerspectives successes OR all complete
+          localResults = await this.collectFirstNSuccesses(cliPromises, maxPerspectives);
+          console.error(`[Stdio Wrapper] Fast-collect: got ${localResults.filter(r => r.success).length} successful, ${localResults.filter(r => !r.success).length} failed out of ${cliPromises.length} CLIs`);
         }
         
         // Store API-only providers info for remote API call
@@ -1941,6 +1946,42 @@ Error: ${error.message}`
     }
   }
 
+  /**
+   * Collect results from parallel promises, resolving early once we have N successes
+   * This avoids waiting for slow/timed-out CLIs when we already have enough results
+   * @param {Promise[]} promises - Array of promises to collect from
+   * @param {number} needed - Number of successful results needed
+   * @returns {Promise<Array>} Array of results (may include failures if not enough successes)
+   */
+  collectFirstNSuccesses(promises, needed) {
+    return new Promise((resolve) => {
+      const results = [];
+      let successCount = 0;
+      let completedCount = 0;
+      let resolved = false;
+      
+      if (promises.length === 0) {
+        resolve([]);
+        return;
+      }
+      
+      for (const promise of promises) {
+        promise.then(result => {
+          if (resolved) return;
+          results.push(result);
+          if (result.success) successCount++;
+          completedCount++;
+          
+          // Resolve early if we have enough successes OR all promises done
+          if (successCount >= needed || completedCount >= promises.length) {
+            resolved = true;
+            resolve([...results]); // Copy to prevent mutation from late arrivals
+          }
+        });
+      }
+    });
+  }
+
   /**
    * Get all available and authenticated CLI providers
    * Uses user's provider order from dashboard (display_order) instead of hardcoded order
@@ -2278,54 +2319,6 @@ Error: ${error.message}`
   combineAllCliAndPerspectives(localResults, perspectivesResult, args) {
     // Ensure perspectivesResult is always an object to prevent undefined errors
     const safePersp = perspectivesResult || { success: false, error: 'No response from perspectives server' };
-
-    const combinedResult = {
-      success: true,
-      timestamp: new Date().toISOString(),
-      mode: args.mode,
-      local_cli_count: localResults.length,
-      sections: {
-        local: localResults,
-        remote: safePersp
-      }
-    };
-
-    // Check if any local CLIs succeeded
-    const successfulClis = localResults.filter(result => result.success);
-    const hasSomeLocalSuccess = successfulClis.length > 0;
-
-    // Determine overall success and content
-    if (hasSomeLocalSuccess && safePersp.success) {
-      combinedResult.content = this.formatMultipleCliResponse(localResults, safePersp, false);
-      combinedResult.tokens_used = successfulClis.reduce((total, cli) => total + (cli.tokens_used || 0), 0);
-      combinedResult.latency_ms = Math.max(...successfulClis.map(cli => cli.latency_ms || 0));
-    } else if (!hasSomeLocalSuccess && safePersp.success) {
-      // Complete fallback case - no local CLIs worked
-      combinedResult.content = this.formatMultipleCliResponse(localResults, safePersp, true);
-      combinedResult.fallback_used = true;
-      combinedResult.tokens_used = 0; // No local tokens used
-    } else if (hasSomeLocalSuccess && !safePersp.success) {
-      // Local CLIs succeeded, remote failed
-      combinedResult.content = this.formatMultipleCliResponse(localResults, safePersp, false);
-      combinedResult.tokens_used = successfulClis.reduce((total, cli) => total + (cli.tokens_used || 0), 0);
-      combinedResult.latency_ms = Math.max(...successfulClis.map(cli => cli.latency_ms || 0));
-    } else {
-      // Both failed
-      combinedResult.success = false;
-      const cliErrors = localResults.map(cli => `${cli.provider_id}: ${cli.error || 'Unknown error'}`).join('; ');
-      const perspectivesError = safePersp.error || 'Unknown remote error';
-      combinedResult.error = `All local CLIs failed: ${cliErrors}; Perspectives also failed: ${perspectivesError}`;
-    }
-
-    return combinedResult;
-  }
-
-  /**
-   * Format multiple CLI responses with remote perspectives
-   */
-  formatMultipleCliResponse(localResults, perspectivesResult, isFallback) {
-    // Safety check - ensure perspectivesResult is always an object
-    const safePersp = perspectivesResult || { success: false, error: 'No perspectives data' };
     let formatted = '';
 
     // Show all local CLI responses

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.9.14",
+  "version": "1.9.15",
   "engines": {
     "node": ">=20.x <=22.x"
   },