v0.2.18

Author: wonderwhy-er

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@wonderwhy-er/desktop-commander",
-  "version": "0.2.18-alpha.6",
+  "version": "0.2.18",
   "description": "MCP server for terminal operations and file editing",
   "mcpName": "io.github.wonderwhy-er/desktop-commander",
   "license": "MIT",
@@ -39,6 +39,7 @@
     "clean": "shx rm -rf dist",
     "test": "npm run build && node test/run-all-tests.js",
     "test:debug": "node --inspect test/run-all-tests.js",
+    "validate:tools": "npm run build && node scripts/validate-tools-sync.js",
     "link:local": "npm run build && npm link",
     "unlink:local": "npm unlink",
     "inspector": "npx @modelcontextprotocol/inspector dist/index.js",

package.json

@@ -0,0 +1,170 @@
+#!/usr/bin/env node
+/**
+ * Validates that the tools listed in mcpb-bundle/manifest.json match
+ * the tools actually provided by the running MCP server
+ * 
+ * This uses JSON-RPC to query the server directly, avoiding fragile regex parsing.
+ */
+
+import { readFile } from 'fs/promises';
+import { join, dirname } from 'path';
+import { fileURLToPath } from 'url';
+import { spawn } from 'child_process';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const rootDir = join(__dirname, '..');
+
+// ANSI color codes for pretty output
+const colors = {
+  reset: '\x1b[0m',
+  red: '\x1b[31m',
+  green: '\x1b[32m',
+  yellow: '\x1b[33m',
+  blue: '\x1b[34m',
+  cyan: '\x1b[36m'
+};
+
+async function extractToolsFromManifest() {
+  const manifestPath = join(rootDir, 'mcpb-bundle', 'manifest.json');
+  const content = await readFile(manifestPath, 'utf-8');
+  const manifest = JSON.parse(content);
+  
+  return manifest.tools.map(tool => tool.name).sort();
+}
+
+async function extractToolsFromServer() {
+  return new Promise((resolve, reject) => {
+    // Start the MCP server
+    const serverPath = join(rootDir, 'dist', 'index.js');
+    const server = spawn('node', [serverPath], {
+      stdio: ['pipe', 'pipe', 'pipe']
+    });
+
+    let output = '';
+    let errorOutput = '';
+    const messages = [];
+
+    server.stdout.on('data', (data) => {
+      output += data.toString();
+      
+      // Try to parse each line as JSON-RPC message
+      const lines = output.split('\n');
+      output = lines.pop() || ''; // Keep incomplete line
+      
+      for (const line of lines) {
+        if (line.trim()) {
+          try {
+            messages.push(JSON.parse(line));
+          } catch (e) {
+            // Not JSON, might be debug output
+          }
+        }
+      }
+    });
+
+    server.stderr.on('data', (data) => {
+      errorOutput += data.toString();
+    });
+
+    // Step 1: Send initialize request
+    const initRequest = {
+      jsonrpc: '2.0',
+      id: 1,
+      method: 'initialize',
+      params: {
+        protocolVersion: '2024-11-05',
+        capabilities: {},
+        clientInfo: {
+          name: 'validate-tools-sync',
+          version: '1.0.0'
+        }
+      }
+    };
+
+    server.stdin.write(JSON.stringify(initRequest) + '\n');
+
+    // Wait for initialize response, then send tools/list
+    setTimeout(() => {
+      // Step 2: Send tools/list request
+      const toolsRequest = {
+        jsonrpc: '2.0',
+        id: 2,
+        method: 'tools/list',
+        params: {}
+      };
+
+      server.stdin.write(JSON.stringify(toolsRequest) + '\n');
+
+      // Wait for tools/list response
+      setTimeout(() => {
+        server.kill();
+
+        // Find the tools/list response
+        const toolsResponse = messages.find(msg => msg.id === 2 && msg.result);
+
+        if (!toolsResponse) {
+          reject(new Error('No tools/list response received'));
+          return;
+        }
+
+        const tools = toolsResponse.result.tools.map(tool => tool.name).sort();
+        resolve(tools);
+      }, 1000);
+    }, 500);
+
+    server.on('error', (error) => {
+      reject(new Error(`Failed to start server: ${error.message}`));
+    });
+  });
+}
+
+async function main() {
+  console.log(`${colors.cyan}🔍 Validating tool synchronization...${colors.reset}\n`);
+  
+  try {
+    const manifestTools = await extractToolsFromManifest();
+    const serverTools = await extractToolsFromServer();
+    
+    console.log(`${colors.blue}📋 Manifest tools (${manifestTools.length}):${colors.reset}`);
+    manifestTools.forEach(tool => console.log(`   - ${tool}`));
+    
+    console.log(`\n${colors.blue}⚙️  Server tools (${serverTools.length}):${colors.reset}`);
+    serverTools.forEach(tool => console.log(`   - ${tool}`));
+    
+    // Find differences
+    const missingInManifest = serverTools.filter(t => !manifestTools.includes(t));
+    const missingInServer = manifestTools.filter(t => !serverTools.includes(t));
+    
+    console.log('\n' + '='.repeat(60));
+    
+    if (missingInManifest.length === 0 && missingInServer.length === 0) {
+      console.log(`${colors.green}✅ SUCCESS: All tools are in sync!${colors.reset}`);
+      console.log(`${colors.green}   Both manifest.json and server.ts have ${manifestTools.length} tools.${colors.reset}`);
+      process.exit(0);
+    } else {
+      console.log(`${colors.red}❌ MISMATCH DETECTED!${colors.reset}\n`);
+      
+      if (missingInManifest.length > 0) {
+        console.log(`${colors.yellow}⚠️  Tools in server.ts but NOT in manifest.json:${colors.reset}`);
+        missingInManifest.forEach(tool => console.log(`   ${colors.red}✗${colors.reset} ${tool}`));
+        console.log();
+      }
+      
+      if (missingInServer.length > 0) {
+        console.log(`${colors.yellow}⚠️  Tools in manifest.json but NOT in server.ts:${colors.reset}`);
+        missingInServer.forEach(tool => console.log(`   ${colors.red}✗${colors.reset} ${tool}`));
+        console.log();
+      }
+      
+      console.log(`${colors.red}Please update the files to match!${colors.reset}`);
+      process.exit(1);
+    }
+    
+  } catch (error) {
+    console.error(`${colors.red}❌ Error:${colors.reset}`, error.message);
+    process.exit(1);
+  }
+}
+
+main();

scripts/validate-tools-sync.js

@@ -7,13 +7,13 @@
     "url": "https://github.com/wonderwhy-er/DesktopCommanderMCP",
     "source": "github"
   },
-  "version": "0.2.18-alpha.0",
+  "version": "0.2.18",
   "packages": [
     {
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "@wonderwhy-er/desktop-commander",
-      "version": "0.2.18-alpha.0",
+      "version": "0.2.18",
       "transport": {
         "type": "stdio"
       }

server.json

@@ -234,13 +234,16 @@ function getPackageSpec(versionArg = null) {
   return '@wonderwhy-er/desktop-commander@latest';
 }
 
+function isNPX() {
+    return process.env.npm_lifecycle_event === 'npx' ||
+        process.env.npm_execpath?.includes('npx') ||
+        process.env._?.includes('npx') ||
+        import.meta.url.includes('node_modules');
+}
 // Function to determine execution context
 function getExecutionContext() {
   // Check if running from npx
-  const isNpx = process.env.npm_lifecycle_event === 'npx' ||
-                process.env.npm_execpath?.includes('npx') ||
-                process.env._?.includes('npx') ||
-                import.meta.url.includes('node_modules');
+  const isNpx = isNPX();
 
   // Check if installed globally
   const isGlobal = process.env.npm_config_global === 'true' ||
@@ -641,11 +644,8 @@ async function restartClaude() {
 // Main function to export for ESM compatibility
 export default async function setup() {
     // Parse command line arguments for version/tag
-    // Usage: npx @wonderwhy-er/desktop-commander setup alpha
-    //        npx @wonderwhy-er/desktop-commander setup latest
-    //        npx @wonderwhy-er/desktop-commander setup 0.2.18
-    const versionArg = process.argv[3]; // argv[0]=node, argv[1]=script, argv[2]=setup, argv[3]=version/tag
-    
+    const versionArg = process.argv[3]; // argv[0]=node, argv[1]=script, argv[2]=version/tag
+
     // Add tracking for setup function entry
     await trackEvent('npx_setup_function_started');
 
@@ -744,7 +744,7 @@ export default async function setup() {
         const configPrepStep = addSetupStep('prepare_server_config');
 
         // Determine if running through npx or locally
-        const isNpx = import.meta.url.includes('node_modules');
+        const isNpx = isNPX();
         await trackEvent('npx_setup_execution_mode', { isNpx });
 
         // Fix Windows path handling for npx execution

setup-claude-server.js

@@ -13,7 +13,7 @@ class FeatureFlagManager {
   private flags: Record<string, any> = {};
   private lastFetch: number = 0;
   private cachePath: string;
-  private cacheMaxAge: number = 30 * 60 * 1000; // 5 minutes - hardcoded refresh interval
+  private cacheMaxAge: number = 30 * 60 * 1000;
   private flagUrl: string;
   private refreshInterval: NodeJS.Timeout | null = null;
 

src/utils/feature-flags.ts

@@ -1 +1 @@
-export const VERSION = '0.2.18-alpha.6';
+export const VERSION = '0.2.18';