polydev-ai 1.8.35 → 1.8.37

package/lib/cliManager.js

@@ -584,9 +584,10 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
             // Claude Code uses --model flag
             args = ['--model', model, ...args, prompt];
           } else if (providerId === 'gemini_cli') {
-            // Gemini CLI: -m for model, -p for prompt (headless mode)
-            // Format: gemini -m gemini-2.0-flash -p "prompt"
-            args = ['-m', model, '-p', prompt];
+            // Gemini CLI: -s for sandbox (no tools), -m for model, -p for prompt (headless mode)
+            // Format: gemini -s -m gemini-2.0-flash -p "prompt"
+            // -s flag disables agentic tool use which causes truncated "I will search..." outputs
+            args = ['-s', '-m', model, '-p', prompt];
           } else {
             // Default: just append prompt
             args = [...args, prompt];
@@ -594,8 +595,8 @@ This is a known issue with @google/gemini-cli@0.3.4 and older Node.js versions.`
         } else {
           // No model specified
           if (providerId === 'gemini_cli') {
-            // Gemini CLI still needs -p flag for headless mode
-            args = ['-p', prompt];
+            // Gemini CLI: -s for sandbox (no tools), -p for headless mode
+            args = ['-s', '-p', prompt];
           } else {
             args = [...args, prompt];
           }

package/mcp/README.md

@@ -57,12 +57,44 @@ export POLYDEV_USER_TOKEN="pd_your_token_here"
 Add to your Claude Code MCP configuration:
 
 ```json
- "polydev": {
-          "disabled": false,
-          "timeout": 120,
-          "type": "http",
-          "url": "https://www.polydev.ai/api/mcp"
-        },
+"polydev": {
+  "disabled": false,
+  "timeout": 120,
+  "type": "http",
+  "url": "https://www.polydev.ai/api/mcp"
+}
+```
+
+### Codex CLI Integration (OpenAI)
+
+Add to `~/.codex/config.toml`:
+
+```toml
+[mcp_servers.polydev]
+command = "/path/to/node"
+args = ["/path/to/node_modules/polydev-ai/mcp/stdio-wrapper.js"]
+env = { POLYDEV_USER_TOKEN = "pd_your_token_here" }
+
+[mcp_servers.polydev.timeouts]
+tool_timeout = 180
+session_timeout = 600
+```
+
+**Important for Codex CLI:**
+- Use direct `node` path instead of `npx` for faster startup
+- Find your node path with: `which node`
+- Find polydev path with: `npm root -g`
+
+Example with typical paths:
+```toml
+[mcp_servers.polydev]
+command = "/Users/yourname/.nvm/versions/node/v22.20.0/bin/node"
+args = ["/Users/yourname/.nvm/versions/node/v22.20.0/lib/node_modules/polydev-ai/mcp/stdio-wrapper.js"]
+env = { POLYDEV_USER_TOKEN = "pd_your_token_here" }
+
+[mcp_servers.polydev.timeouts]
+tool_timeout = 180
+session_timeout = 600
 ```
 
 ## Available Tools
@@ -129,6 +161,34 @@ The MCP server automatically detects CLI status changes:
 
 ## Troubleshooting
 
+### Codex CLI: "Transport closed" Error
+
+This error occurs when the MCP handshake fails. Common causes and fixes:
+
+1. **Use direct node path instead of npx**
+   ```toml
+   # Wrong - npx has startup overhead
+   command = "npx"
+   args = ["-y", "polydev-ai"]
+
+   # Correct - direct node path
+   command = "/path/to/node"
+   args = ["/path/to/polydev-ai/mcp/stdio-wrapper.js"]
+   ```
+
+2. **Use inline env format**
+   ```toml
+   # Correct format
+   env = { POLYDEV_USER_TOKEN = "pd_xxx" }
+   ```
+
+3. **Protocol version mismatch**: Ensure polydev-ai version >= 1.8.36 which supports MCP protocol 2025-06-18
+
+4. **Check Codex logs** for details:
+   ```bash
+   cat ~/.codex/log/codex-tui.log | tail -50
+   ```
+
 ### CLI Not Detected
 
 1. Ensure the CLI is installed and in your PATH
@@ -149,6 +209,16 @@ npm config set prefix '~/.npm-global'
 export PATH=~/.npm-global/bin:$PATH
 ```
 
+### Debugging MCP Connection
+
+Test the wrapper manually:
+```bash
+echo '{"jsonrpc":"2.0","id":1,"method":"initialize","params":{}}' | \
+  POLYDEV_USER_TOKEN="pd_xxx" node /path/to/stdio-wrapper.js
+```
+
+You should see a JSON response with `protocolVersion: "2025-06-18"`
+
 ## Support
 
 - 📧 Email: [support@polydev.ai](mailto:support@polydev.ai)

package/mcp/stdio-wrapper.js

@@ -292,7 +292,7 @@ class StdioMCPWrapper {
             jsonrpc: '2.0',
             id,
             result: {
-              protocolVersion: '2024-11-05',
+              protocolVersion: '2025-06-18',
               capabilities: { tools: {} },
               serverInfo: {
                 name: this.manifest.name,
@@ -718,11 +718,6 @@ class StdioMCPWrapper {
         }
       }
 
-      // Report CLI results to server for dashboard storage (non-blocking)
-      this.reportCliResultsToServer(prompt, localResults, args).catch(err => {
-        console.error('[Stdio Wrapper] CLI results reporting failed (non-critical):', err.message);
-      });
-
       // Calculate how many successful local perspectives we got
       const successfulLocalCount = localResults.filter(r => r.success).length;
       const failedCliCount = localResults.filter(r => !r.success).length;
@@ -742,9 +737,18 @@ class StdioMCPWrapper {
           model: p.model
         }));
         
+        // Note: Remote API will handle logging of CLI + API results together
+        // 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`);
+        
+        // ONLY report CLI results when NOT calling remote API
+        // (When we call remote API, it handles logging CLI + API results together)
+        this.reportCliResultsToServer(prompt, localResults, args).catch(err => {
+          console.error('[Stdio Wrapper] CLI results reporting failed (non-critical):', err.message);
+        });
+        
         perspectivesResult = {
           success: true,
           content: '',
@@ -955,6 +959,22 @@ class StdioMCPWrapper {
       .map(cli => cliToApiProvider[cli])
       .filter(Boolean);
     
+    // Get failed CLI providers and map them to their API equivalents
+    const failedCliProviders = localResults
+      .filter(r => !r.success && r.provider_id)
+      .map(r => r.provider_id);
+    
+    const failedCliApiProviders = failedCliProviders
+      .map(cli => {
+        const apiProvider = cliToApiProvider[cli];
+        if (apiProvider) {
+          console.error(`[Stdio Wrapper] CLI ${cli} failed, will request API fallback: ${apiProvider}`);
+          return { provider: apiProvider, model: null };  // Let server choose model
+        }
+        return null;
+      })
+      .filter(Boolean);
+    
     // If we don't need any perspectives, skip remote call
     if (maxPerspectives <= 0) {
       console.error(`[Stdio Wrapper] Max perspectives is 0, skipping remote perspectives`);
@@ -967,12 +987,16 @@ class StdioMCPWrapper {
       };
     }
     
-    // Build list of specific providers to request (from API-only providers)
-    const requestProviders = apiProvidersInfo.map(p => ({
+    // Build list of specific providers to request (from API-only providers + failed CLI fallbacks)
+    const apiOnlyProvidersList = apiProvidersInfo.map(p => ({
       provider: p.provider,
       model: p.model
     }));
     
+    // Combine API-only providers with failed CLI fallback providers
+    // Failed CLI providers get priority (they're the ones user expected to work)
+    const requestProviders = [...failedCliApiProviders, ...apiOnlyProvidersList];
+    
     console.error(`[Stdio Wrapper] Calling remote perspectives (excluding: ${excludeProviders.join(', ') || 'none'}, requesting: ${requestProviders.map(p => p.provider).join(', ') || 'any'}, max: ${maxPerspectives})`);
     
     // Format ONLY SUCCESSFUL CLI responses for logging on the server
@@ -1577,14 +1601,19 @@ class StdioMCPWrapper {
     
     process.stdin.on('data', async (chunk) => {
       buffer += chunk;
-      
+
       const lines = buffer.split('\n');
       buffer = lines.pop() || '';
-      
+
       for (const line of lines) {
         if (line.trim()) {
           try {
             const request = JSON.parse(line);
+            // Notifications have no id - don't respond to them (JSON-RPC spec)
+            if (request.id === undefined) {
+              console.error(`[Stdio Wrapper] Received notification: ${request.method}`);
+              continue;
+            }
             const response = await this.handleRequest(request);
             process.stdout.write(JSON.stringify(response) + '\n');
           } catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.8.35",
+  "version": "1.8.37",
   "description": "Agentic workflow assistant with CLI integration - get diverse perspectives from multiple LLMs when stuck or need enhanced reasoning",
   "keywords": [
     "mcp",
@@ -16,7 +16,7 @@
   "author": "Polydev AI",
   "license": "MIT",
   "private": false,
-  "main": "mcp/server.js",
+  "main": "mcp/stdio-wrapper.js",
   "bin": {
     "polydev-stdio": "mcp/stdio-wrapper.js"
   },
@@ -67,7 +67,7 @@
     "lucide-react": "^0.542.0",
     "marked": "^16.2.1",
     "next": "^15.5.7",
-    "polydev-ai": "^1.8.34",
+    "polydev-ai": "^1.8.36",
     "posthog-js": "^1.157.2",
     "prismjs": "^1.30.0",
     "react": "^18.3.1",