polydev-ai 1.2.14 → 1.4.0

package/mcp/stdio-wrapper.js

@@ -3,27 +3,56 @@
 // Lightweight stdio wrapper with local CLI functionality and remote Polydev MCP server fallback
 const fs = require('fs');
 const path = require('path');
+const os = require('os');
 const { CLIManager } = require('../lib/cliManager');
 
+function ensureWritableTmpDir() {
+  const candidates = [
+    process.env.POLYDEV_TMPDIR,
+    '/tmp/polydev',
+    '/tmp',
+    path.join(os.tmpdir(), 'polydev'),
+    path.join(process.cwd(), '.polydev-tmp')
+  ].filter(Boolean);
+
+  for (const candidate of candidates) {
+    try {
+      fs.mkdirSync(candidate, { recursive: true });
+      fs.accessSync(candidate, fs.constants.W_OK);
+      return candidate;
+    } catch (error) {
+      console.warn(`[Stdio Wrapper] TMP candidate not writable (${candidate}):`, error.message);
+    }
+  }
+
+  console.error('[Stdio Wrapper] No writable TMP directory found; Codex CLI will fail.');
+  return null;
+}
+
+const writableTmp = ensureWritableTmpDir();
+if (writableTmp) {
+  process.env.TMPDIR = writableTmp;
+  process.env.TMP = writableTmp;
+  process.env.TEMP = writableTmp;
+  console.error(`[Stdio Wrapper] Using TMP directory: ${writableTmp}`);
+}
+
 class StdioMCPWrapper {
   constructor() {
     this.userToken = process.env.POLYDEV_USER_TOKEN;
-    // Don't exit immediately - wait for initialization to report error properly
-    
-    // Initialize CLI Manager for local CLI functionality (only if token exists)
-    if (this.userToken) {
-      this.cliManager = new CLIManager();
+    if (!this.userToken) {
+      console.error('POLYDEV_USER_TOKEN environment variable is required');
+      process.exit(1);
     }
     
+    // Initialize CLI Manager for local CLI functionality
+    this.cliManager = new CLIManager();
+    
     // Load manifest for tool definitions
     this.loadManifest();
     
     // Smart refresh scheduler (will be started after initialization)
     this.refreshScheduler = null;
-    
-    // Track if initial CLI detection has been done
-    this.initialDetectionDone = false;
-    this.initialDetectionPromise = null;
   }
 
   loadManifest() {
@@ -69,20 +98,14 @@ class StdioMCPWrapper {
           };
         
         case 'tools/call':
-          // Check if token is missing
-          if (!this.userToken) {
-            return {
-              jsonrpc: '2.0',
-              id,
-              error: {
-                code: -32603,
-                message: 'POLYDEV_USER_TOKEN environment variable is required. Please add it to your MCP configuration.'
-              }
-            };
+          // Handle get_perspectives with local CLIs + remote perspectives
+          const toolName = params.name;
+          
+          if (toolName === 'get_perspectives') {
+            return await this.handleGetPerspectivesWithCLIs(params, id);
           }
           
-          // Handle CLI tools locally, forward others to remote server
-          const toolName = params.name;
+          // Handle other CLI tools locally
           if (toolName.startsWith('polydev.') && this.isCliTool(toolName)) {
             return await this.handleLocalCliTool(request);
           } else {
@@ -157,6 +180,45 @@ class StdioMCPWrapper {
     return cliTools.includes(toolName);
   }
 
+  /**
+   * Handle get_perspectives with local CLIs + remote perspectives
+   */
+  async handleGetPerspectivesWithCLIs(params, id) {
+    console.error(`[Stdio Wrapper] Handling get_perspectives with local CLIs + remote`);
+    
+    try {
+      // Use existing localSendCliPrompt logic which already:
+      // 1. Checks all local CLIs
+      // 2. Calls remote perspectives
+      // 3. Combines results
+      const result = await this.localSendCliPrompt(params.arguments);
+      
+      return {
+        jsonrpc: '2.0',
+        id,
+        result: {
+          content: [
+            {
+              type: 'text',
+              text: result.content || this.formatCliResponse(result)
+            }
+          ]
+        }
+      };
+
+    } catch (error) {
+      console.error(`[Stdio Wrapper] get_perspectives error:`, error);
+      return {
+        jsonrpc: '2.0',
+        id,
+        error: {
+          code: -32603,
+          message: error.message
+        }
+      };
+    }
+  }
+
   /**
    * Handle CLI tools locally without remote server calls
    */
@@ -230,10 +292,6 @@ class StdioMCPWrapper {
       // Update database with CLI status
       await this.updateCliStatusInDatabase(results);
       
-      // Mark initial detection as done
-      this.initialDetectionDone = true;
-      this.initialDetectionPromise = null;
-      
       return {
         success: true,
         results,
@@ -244,10 +302,6 @@ class StdioMCPWrapper {
 
     } catch (error) {
       console.error('[Stdio Wrapper] Local CLI detection error:', error);
-      // Mark as done even on error to prevent blocking
-      this.initialDetectionDone = true;
-      this.initialDetectionPromise = null;
-      
       return {
         success: false,
         error: error.message,
@@ -309,16 +363,6 @@ class StdioMCPWrapper {
     console.error(`[Stdio Wrapper] Local CLI prompt sending with perspectives`);
     
     try {
-      // Ensure initial CLI detection has completed before sending prompts
-      if (!this.initialDetectionDone && this.initialDetectionPromise) {
-        console.error('[Stdio Wrapper] Waiting for initial CLI detection to complete...');
-        await this.initialDetectionPromise;
-      } else if (!this.initialDetectionDone && !this.initialDetectionPromise) {
-        console.error('[Stdio Wrapper] Running initial CLI detection before sending prompt...');
-        this.initialDetectionPromise = this.localForceCliDetection({});
-        await this.initialDetectionPromise;
-      }
-      
       let { provider_id, prompt, mode = 'args', timeout_ms = 30000 } = args;
       
       // Ensure timeout_ms is valid (not undefined, null, Infinity, or negative)
@@ -402,18 +446,7 @@ class StdioMCPWrapper {
    */
   async getAllAvailableProviders() {
     try {
-      // Try to load from cache first to avoid re-detection
-      const cachedStatus = await this.loadLocalCliStatus();
-      let results;
-      
-      if (cachedStatus && Object.keys(cachedStatus).length > 0) {
-        console.error('[Stdio Wrapper] Using cached CLI status');
-        results = cachedStatus;
-      } else {
-        console.error('[Stdio Wrapper] No cached status, running detection');
-        results = await this.cliManager.forceCliDetection();
-      }
-      
+      const results = await this.cliManager.forceCliDetection();
       const availableProviders = [];
       
       // Priority order: claude_code > codex_cli > gemini_cli
@@ -894,7 +927,19 @@ class StdioMCPWrapper {
   }
 
   async start() {
-    console.error('[Stdio Wrapper] Starting Polydev Stdio MCP Wrapper...');
+    console.log('Starting Polydev Stdio MCP Wrapper...');
+    
+    // Run initial CLI detection on startup
+    console.error('[Stdio Wrapper] Running initial CLI detection...');
+    try {
+      await this.localForceCliDetection({});
+      console.error('[Stdio Wrapper] Initial CLI detection completed');
+    } catch (error) {
+      console.error('[Stdio Wrapper] Initial CLI detection failed:', error);
+    }
+    
+    // Start smart refresh scheduler for automatic updates
+    this.startSmartRefreshScheduler();
     
     process.stdin.setEncoding('utf8');
     let buffer = '';
@@ -919,46 +964,25 @@ class StdioMCPWrapper {
     });
 
     process.stdin.on('end', () => {
-      console.error('[Stdio Wrapper] Shutting down...');
+      console.log('Stdio MCP Wrapper shutting down...');
       this.stopSmartRefreshScheduler();
       process.exit(0);
     });
 
     // Handle process signals
     process.on('SIGINT', () => {
-      console.error('[Stdio Wrapper] Received SIGINT, shutting down...');
+      console.log('Received SIGINT, shutting down...');
       this.stopSmartRefreshScheduler();
       process.exit(0);
     });
 
     process.on('SIGTERM', () => {
-      console.error('[Stdio Wrapper] Received SIGTERM, shutting down...');
+      console.log('Received SIGTERM, shutting down...');
       this.stopSmartRefreshScheduler();
       process.exit(0);
     });
 
-    console.error('[Stdio Wrapper] Ready and listening on stdin...');
-    
-    // Run initial CLI detection AFTER stdin handler is set up
-    if (this.userToken && this.cliManager) {
-      console.error('[Stdio Wrapper] Scheduling initial CLI detection...');
-      // Run asynchronously after a short delay to not block initialization
-      setTimeout(async () => {
-        console.error('[Stdio Wrapper] Running initial CLI detection...');
-        try {
-          this.initialDetectionPromise = this.localForceCliDetection({});
-          await this.initialDetectionPromise;
-          console.error('[Stdio Wrapper] Initial CLI detection completed');
-        } catch (error) {
-          console.error('[Stdio Wrapper] Initial CLI detection failed:', error);
-        }
-        
-        // Start smart refresh scheduler for automatic updates
-        this.startSmartRefreshScheduler();
-      }, 100);
-    } else {
-      console.error('[Stdio Wrapper] No token provided - CLI detection disabled');
-    }
+    console.log('Stdio MCP Wrapper ready and listening on stdin...');
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polydev-ai",
-  "version": "1.2.14",
+  "version": "1.4.0",
   "description": "Agentic workflow assistant with CLI integration - get diverse perspectives from multiple LLMs when stuck or need enhanced reasoning",
   "keywords": [
     "mcp",
@@ -31,6 +31,12 @@
     "build": "next build",
     "start": "next start",
     "lint": "next lint",
+    "test": "jest",
+    "test:watch": "jest --watch",
+    "test:coverage": "jest --coverage",
+    "test:api": "jest __tests__/api",
+    "test:vm": "jest __tests__/api/vm",
+    "test:admin": "jest __tests__/api/admin",
     "mcp": "node mcp/server.js",
     "mcp-stdio": "node mcp/stdio-wrapper.js",
     "cli-detect": "node -e \"const CLIManager = require('./lib/cliManager').default; const m = new CLIManager(); m.forceCliDetection().then(console.log);\""
@@ -44,8 +50,6 @@
     "url": "https://github.com/polydev-ai/perspectives-mcp/issues"
   },
   "dependencies": {
-    "which": "^5.0.0",
-    "shelljs": "^0.8.5",
     "@hello-pangea/dnd": "^18.0.1",
     "@radix-ui/react-dialog": "^1.1.15",
     "@radix-ui/react-progress": "^1.1.7",
@@ -69,21 +73,30 @@
     "react": "^18.3.1",
     "react-dom": "^18.3.1",
     "resend": "^6.0.2",
+    "shelljs": "^0.8.5",
     "sonner": "^2.0.7",
     "stripe": "^18.5.0",
     "tailwind-merge": "^3.3.1",
+    "tailwindcss-animate": "^1.0.7",
     "ts-node": "^10.9.2",
-    "use-debounce": "^10.0.6"
+    "use-debounce": "^10.0.6",
+    "which": "^5.0.0"
   },
   "devDependencies": {
+    "@testing-library/jest-dom": "^6.9.1",
+    "@testing-library/react": "^16.3.0",
+    "@types/jest": "^30.0.0",
     "@types/node": "^20.16.1",
     "@types/react": "^18.3.3",
     "@types/react-dom": "^18.3.0",
     "autoprefixer": "^10.4.20",
     "eslint": "^8.57.0",
     "eslint-config-next": "15.0.0",
+    "jest": "^30.2.0",
+    "jest-environment-jsdom": "^30.2.0",
     "postcss": "^8.4.41",
     "tailwindcss": "^3.4.10",
+    "ts-jest": "^29.4.4",
     "typescript": "^5.5.4"
   }
 }