@wonderwhy-er/desktop-commander 0.2.18-alpha.4 → 0.2.18-alpha.6

package/dist/setup-claude-server.js

@@ -209,36 +209,32 @@ function detectShell() {
 // Function to get the package spec that was used to run this script
 function getPackageSpec() {
   console.log('\n[DEBUG getPackageSpec] Starting detection...');
+  console.log('[DEBUG getPackageSpec] process.argv:', process.argv);
+  console.log('[DEBUG getPackageSpec] __dirname:', __dirname);
+  console.log('[DEBUG getPackageSpec] __filename:', __filename);
   
-  // Strategy: We want to preserve the tag/version the user used
-  // - If user ran: npx @pkg@latest → keep @latest
-  // - If user ran: npx @pkg@alpha → keep @alpha  
-  // - If user ran: npx @pkg@0.2.18-alpha.1 → keep @0.2.18-alpha.1
+  // Strategy: Check multiple sources to detect the version
+  // 1. Check process.argv[1] which contains the actual script path  
+  // 2. Check package.json in the script's directory
+  // 3. Fall back to @latest for stable, keep version for pre-release
   
-  // Check npm_config_user_agent which contains the original command
-  // Example: "npm/10.2.4 node/v20.11.0 darwin arm64 workspaces/false"
-  const userAgent = process.env.npm_config_user_agent || '';
-  const execPath = process.env.npm_execpath || '';
-  console.log('[DEBUG getPackageSpec] userAgent:', userAgent);
-  console.log('[DEBUG getPackageSpec] execPath:', execPath);
+  // Method 1: Check the script path (process.argv[1] or __dirname)
+  // npx extracts packages to: ~/.npm/_npx/<hash>/node_modules/@scope/package-name/
+  // The actual script path will contain this structure
+  const scriptPath = __dirname;
+  console.log('[DEBUG getPackageSpec] Checking script path for version...');
   
-  // Check if there's a version/tag in the npx cache path
-  // npx caches packages like: ~/.npm/_npx/<hash>/node_modules/@wonderwhy-er/desktop-commander
-  const scriptPath = import.meta.url;
-  console.log('[DEBUG getPackageSpec] scriptPath:', scriptPath);
-  
-  // Look for @version in the path (npx cache includes it sometimes)
-  const versionMatch = scriptPath.match(/@wonderwhy-er\/desktop-commander@([^\/]+)/);
-  if (versionMatch) {
-    const spec = `@wonderwhy-er/desktop-commander@${versionMatch[1]}`;
-    console.log('[DEBUG getPackageSpec] Detected version from path:', spec);
-    return spec;
+  // Look for node_modules/@wonderwhy-er/desktop-commander in the path
+  // This works because npx extracts to a predictable location
+  const nodeModulesMatch = scriptPath.match(/node_modules\/@wonderwhy-er\/desktop-commander/);
+  if (nodeModulesMatch) {
+    console.log('[DEBUG getPackageSpec] Script is in node_modules, reading package.json...');
   }
   
-  // Read package.json to get the actual installed version
+  // Method 2: Read package.json to get the actual installed version
   try {
     const packageJsonPath = join(__dirname, 'package.json');
-    console.log('[DEBUG getPackageSpec] Reading package.json from:', packageJsonPath);
+    console.log('[DEBUG getPackageSpec] Trying to read:', packageJsonPath);
     
     if (existsSync(packageJsonPath)) {
       const packageJson = JSON.parse(readFileSync(packageJsonPath, 'utf8'));
@@ -246,31 +242,28 @@ function getPackageSpec() {
       console.log('[DEBUG getPackageSpec] Found version in package.json:', version);
       
       if (version) {
-        // Check if version looks like a pre-release (alpha, beta, rc)
+        // Always use the exact version if it's a pre-release
         if (version.includes('alpha') || version.includes('beta') || version.includes('rc')) {
-          // User explicitly installed a pre-release version, keep it
           const spec = `@wonderwhy-er/desktop-commander@${version}`;
-          console.log('[DEBUG getPackageSpec] Using pre-release version:', spec);
+          console.log('[DEBUG getPackageSpec] ✓ Using pre-release version:', spec);
           return spec;
         }
         
-        // For stable versions, assume user wants @latest
-        console.log('[DEBUG getPackageSpec] Stable version detected, using @latest');
+        // For stable versions, use @latest tag
+        console.log('[DEBUG getPackageSpec] ✓ Stable version, using @latest');
         return '@wonderwhy-er/desktop-commander@latest';
       }
     } else {
-      console.log('[DEBUG getPackageSpec] package.json not found at path');
+      console.log('[DEBUG getPackageSpec] ✗ package.json not found');
     }
   } catch (error) {
-    console.log('[DEBUG getPackageSpec] Error reading package.json:', error.message);
+    console.log('[DEBUG getPackageSpec] ✗ Error reading package.json:', error.message);
   }
   
-  // Fallback to @latest
-  console.log('[DEBUG getPackageSpec] Falling back to @latest');
+  // Fallback
+  console.log('[DEBUG getPackageSpec] ⚠ Falling back to @latest');
   return '@wonderwhy-er/desktop-commander@latest';
 }
-    console.log('[DEBUG] Found package spec from npm env:', spec);
-    return spec;
   }
   
   console.log('[DEBUG] Falling back to @latest');
@@ -837,12 +830,14 @@ export default async function setup() {
                 // Standard configuration without debug
                 if (isNpx) {
                     const packageSpec = getPackageSpec();
+                    console.log('\n[SETUP] Creating config with package spec:', packageSpec);
                     serverConfig = {
                         "command": isWindows ? "npx.cmd" : "npx",
                         "args": [
                             packageSpec
                         ]
                     };
+                    console.log('[SETUP] serverConfig.args:', JSON.stringify(serverConfig.args));
                     await trackEvent('npx_setup_config_standard_npx', { packageSpec });
                 } else {
                     // For local installation, use absolute path to handle Windows properly
@@ -880,8 +875,16 @@ export default async function setup() {
             // Add or update the terminal server config with the proper name "desktop-commander"
             config.mcpServers["desktop-commander"] = serverConfig;
 
+            console.log('\n[SETUP] Writing config to Claude:');
+            console.log('[SETUP] desktop-commander args:', JSON.stringify(config.mcpServers["desktop-commander"].args));
+
             // Write the updated config back
             writeFileSync(claudeConfigPath, JSON.stringify(config, null, 2), 'utf8');
+            
+            // Verify what was written
+            const writtenConfig = JSON.parse(readFileSync(claudeConfigPath, 'utf8'));
+            console.log('[SETUP] Verified written args:', JSON.stringify(writtenConfig.mcpServers["desktop-commander"].args));
+            
             updateSetupStep(updateConfigStep, 'completed');
             await trackEvent('npx_setup_update_config');
         } catch (updateError) {

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.2.18-alpha.4";
+export declare const VERSION = "0.2.18-alpha.6";

package/dist/version.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wonderwhy-er/desktop-commander",
-  "version": "0.2.18-alpha.4",
+  "version": "0.2.18-alpha.6",
   "description": "MCP server for terminal operations and file editing",
   "mcpName": "io.github.wonderwhy-er/desktop-commander",
   "license": "MIT",