@wonderwhy-er/desktop-commander 0.2.17 → 0.2.18-alpha.0

package/dist/server.js

@@ -591,12 +591,20 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                         
                         STATES DETECTED:
                         Process waiting for input (shows prompt)
-                        Process finished execution  
+                        Process finished execution
                         Process running (use read_process_output)
-                        
+
+                        PERFORMANCE DEBUGGING (verbose_timing parameter):
+                        Set verbose_timing: true to get detailed timing information including:
+                        - Exit reason (early_exit_quick_pattern, early_exit_periodic_check, process_exit, timeout)
+                        - Total duration and time to first output
+                        - Complete timeline of all output events with timestamps
+                        - Which detection mechanism triggered early exit
+                        Use this to identify missed optimization opportunities and improve detection patterns.
+
                         ALWAYS USE FOR: Local file analysis, CSV processing, data exploration, system commands
                         NEVER USE ANALYSIS TOOL FOR: Local file access (analysis tool is browser-only and WILL FAIL)
-                        
+
                         ${PATH_GUIDANCE}
                         ${CMD_PREFIX_DESCRIPTION}`,
                 inputSchema: zodToJsonSchema(StartProcessArgsSchema),
@@ -630,7 +638,15 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                         Process waiting for input (ready for interact_with_process)
                         Process finished execution
                         Timeout reached (may still be running)
-                        
+
+                        PERFORMANCE DEBUGGING (verbose_timing parameter):
+                        Set verbose_timing: true to get detailed timing information including:
+                        - Exit reason (early_exit_quick_pattern, early_exit_periodic_check, process_finished, timeout)
+                        - Total duration and time to first output
+                        - Complete timeline of all output events with timestamps
+                        - Which detection mechanism triggered early exit
+                        Use this to identify when timeouts could be reduced or detection patterns improved.
+
                         ${CMD_PREFIX_DESCRIPTION}`,
                 inputSchema: zodToJsonSchema(ReadProcessOutputArgsSchema),
                 annotations: {
@@ -681,12 +697,21 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                         - input: Code/command to execute
                         - timeout_ms: Max wait (default: 8000ms)
                         - wait_for_prompt: Auto-wait for response (default: true)
-                        
+                        - verbose_timing: Enable detailed performance telemetry (default: false)
+
                         Returns execution result with status indicators.
-                        
+
+                        PERFORMANCE DEBUGGING (verbose_timing parameter):
+                        Set verbose_timing: true to get detailed timing information including:
+                        - Exit reason (early_exit_quick_pattern, early_exit_periodic_check, process_finished, timeout, no_wait)
+                        - Total duration and time to first output
+                        - Complete timeline of all output events with timestamps
+                        - Which detection mechanism triggered early exit
+                        Use this to identify slow interactions and optimize detection patterns.
+
                         ALWAYS USE FOR: CSV analysis, JSON processing, file statistics, data visualization prep, ANY local file work
                         NEVER USE ANALYSIS TOOL FOR: Local file access (it cannot read files from disk and WILL FAIL)
-                        
+
                         ${CMD_PREFIX_DESCRIPTION}`,
                 inputSchema: zodToJsonSchema(InteractWithProcessArgsSchema),
                 annotations: {

package/dist/setup-claude-server.js

@@ -44,7 +44,7 @@ async function initConfigFile() {
         // Ensure config directory exists
         const configDir = path.dirname(CONFIG_FILE);
         if (!existsSync(configDir)) {
-            await mkdir(configDir, { recursive: true });
+            mkdirSync(configDir, { recursive: true });
         }
 
         // Check if config file exists
@@ -206,6 +206,28 @@ function detectShell() {
   return 'unknown-shell';
 }
 
+// Function to get the package spec that was used to run this script
+function getPackageSpec() {
+  // Check if running via npx - look for the package spec in process.argv
+  // e.g., npx @wonderwhy-er/desktop-commander@0.2.18-alpha setup
+  const argv = process.argv;
+  
+  // Look for the package name in argv
+  for (let i = 0; i < argv.length; i++) {
+    const arg = argv[i];
+    if (arg.includes('@wonderwhy-er/desktop-commander')) {
+      // Extract just the package spec (e.g., @wonderwhy-er/desktop-commander@0.2.18-alpha)
+      const match = arg.match(/(@wonderwhy-er\/desktop-commander(@[^\/\s]+)?)/);
+      if (match) {
+        return match[1];
+      }
+    }
+  }
+  
+  // Fallback to @latest if we can't detect
+  return '@wonderwhy-er/desktop-commander@latest';
+}
+
 // Function to determine execution context
 function getExecutionContext() {
   // Check if running from npx
@@ -728,6 +750,7 @@ export default async function setup() {
                         "DEBUG": "*"
                     };
 
+                    const packageSpec = getPackageSpec();
                     serverConfig = {
                         "command": isWindows ? "node.exe" : "node",
                         "args": [
@@ -735,11 +758,11 @@ export default async function setup() {
                             isWindows ?
                                 join(process.env.APPDATA || '', "npm", "npx.cmd").replace(/\\/g, '\\\\') :
                                 "$(which npx)",
-                            "@wonderwhy-er/desktop-commander@latest"
+                            packageSpec
                         ],
                         "env": debugEnv
                     };
-                    await trackEvent('npx_setup_config_debug_npx');
+                    await trackEvent('npx_setup_config_debug_npx', { packageSpec });
                 } else {
                     // Debug with local installation path
                     const indexPath = join(__dirname, 'dist', 'index.js');
@@ -763,13 +786,14 @@ export default async function setup() {
             } else {
                 // Standard configuration without debug
                 if (isNpx) {
+                    const packageSpec = getPackageSpec();
                     serverConfig = {
                         "command": isWindows ? "npx.cmd" : "npx",
                         "args": [
-                            "@wonderwhy-er/desktop-commander@latest"
+                            packageSpec
                         ]
                     };
-                    await trackEvent('npx_setup_config_standard_npx');
+                    await trackEvent('npx_setup_config_standard_npx', { packageSpec });
                 } else {
                     // For local installation, use absolute path to handle Windows properly
                     const indexPath = join(__dirname, 'dist', 'index.js');

package/dist/terminal-manager.d.ts

@@ -16,7 +16,7 @@ export declare class TerminalManager {
      * @returns Whether input was successfully sent
      */
     sendInputToProcess(pid: number, input: string): boolean;
-    executeCommand(command: string, timeoutMs?: number, shell?: string): Promise<CommandExecutionResult>;
+    executeCommand(command: string, timeoutMs?: number, shell?: string, collectTiming?: boolean): Promise<CommandExecutionResult>;
     getNewOutput(pid: number): string | null;
     /**
    * Get a session by PID

package/dist/terminal-manager.js

@@ -33,7 +33,7 @@ export class TerminalManager {
             return false;
         }
     }
-    async executeCommand(command, timeoutMs = DEFAULT_COMMAND_TIMEOUT, shell) {
+    async executeCommand(command, timeoutMs = DEFAULT_COMMAND_TIMEOUT, shell, collectTiming = false) {
         // Get the shell from config if not specified
         let shellToUse = shell;
         if (!shellToUse) {
@@ -81,6 +81,12 @@ export class TerminalManager {
             startTime: new Date()
         };
         this.sessions.set(childProcess.pid, session);
+        // Timing telemetry
+        const startTime = Date.now();
+        let firstOutputTime;
+        let lastOutputTime;
+        const outputEvents = [];
+        let exitReason = 'timeout';
         return new Promise((resolve) => {
             let resolved = false;
             let periodicCheck = null;
@@ -92,15 +98,47 @@ export class TerminalManager {
                 resolved = true;
                 if (periodicCheck)
                     clearInterval(periodicCheck);
+                // Add timing info if requested
+                if (collectTiming) {
+                    const endTime = Date.now();
+                    result.timingInfo = {
+                        startTime,
+                        endTime,
+                        totalDurationMs: endTime - startTime,
+                        exitReason,
+                        firstOutputTime,
+                        lastOutputTime,
+                        timeToFirstOutputMs: firstOutputTime ? firstOutputTime - startTime : undefined,
+                        outputEvents: outputEvents.length > 0 ? outputEvents : undefined
+                    };
+                }
                 resolve(result);
             };
             childProcess.stdout.on('data', (data) => {
                 const text = data.toString();
+                const now = Date.now();
+                if (!firstOutputTime)
+                    firstOutputTime = now;
+                lastOutputTime = now;
                 output += text;
                 session.lastOutput += text;
+                // Record output event if collecting timing
+                if (collectTiming) {
+                    outputEvents.push({
+                        timestamp: now,
+                        deltaMs: now - startTime,
+                        source: 'stdout',
+                        length: text.length,
+                        snippet: text.slice(0, 50).replace(/\n/g, '\\n')
+                    });
+                }
                 // Immediate check for obvious prompts
                 if (quickPromptPatterns.test(text)) {
                     session.isBlocked = true;
+                    exitReason = 'early_exit_quick_pattern';
+                    if (collectTiming && outputEvents.length > 0) {
+                        outputEvents[outputEvents.length - 1].matchedPattern = 'quick_pattern';
+                    }
                     resolveOnce({
                         pid: childProcess.pid,
                         output,
@@ -110,8 +148,22 @@ export class TerminalManager {
             });
             childProcess.stderr.on('data', (data) => {
                 const text = data.toString();
+                const now = Date.now();
+                if (!firstOutputTime)
+                    firstOutputTime = now;
+                lastOutputTime = now;
                 output += text;
                 session.lastOutput += text;
+                // Record output event if collecting timing
+                if (collectTiming) {
+                    outputEvents.push({
+                        timestamp: now,
+                        deltaMs: now - startTime,
+                        source: 'stderr',
+                        length: text.length,
+                        snippet: text.slice(0, 50).replace(/\n/g, '\\n')
+                    });
+                }
             });
             // Periodic comprehensive check every 100ms
             periodicCheck = setInterval(() => {
@@ -119,6 +171,7 @@ export class TerminalManager {
                     const processState = analyzeProcessState(output, childProcess.pid);
                     if (processState.isWaitingForInput) {
                         session.isBlocked = true;
+                        exitReason = 'early_exit_periodic_check';
                         resolveOnce({
                             pid: childProcess.pid,
                             output,
@@ -130,6 +183,7 @@ export class TerminalManager {
             // Timeout fallback
             setTimeout(() => {
                 session.isBlocked = true;
+                exitReason = 'timeout';
                 resolveOnce({
                     pid: childProcess.pid,
                     output,
@@ -153,6 +207,7 @@ export class TerminalManager {
                     }
                     this.sessions.delete(childProcess.pid);
                 }
+                exitReason = 'process_exit';
                 resolveOnce({
                     pid: childProcess.pid,
                     output,

package/dist/tools/config.js

@@ -2,6 +2,7 @@ import { configManager } from '../config-manager.js';
 import { SetConfigValueArgsSchema } from './schemas.js';
 import { getSystemInfo } from '../utils/system-info.js';
 import { currentClient } from '../server.js';
+import { featureFlagManager } from '../utils/feature-flags.js';
 /**
  * Get the entire config including system information
  */
@@ -23,6 +24,7 @@ export async function getConfig() {
         const configWithSystemInfo = {
             ...config,
             currentClient,
+            featureFlags: featureFlagManager.getAll(),
             systemInfo: {
                 ...systemInfo,
                 memory

package/dist/tools/feedback.js

@@ -73,7 +73,7 @@ export async function giveFeedbackToDesktopCommander(params = {}) {
             content: [{
                     type: "text",
                     text: `❌ **Error opening feedback form**: ${error instanceof Error ? error.message : String(error)}\n\n` +
-                        `You can still access our feedback form directly at: https://tally.so/r/mYB6av\n\n` +
+                        `You can still access our feedback form directly at: https://tally.so/r/mKqoKg\n\n` +
                         `We appreciate your willingness to provide feedback!`
                 }],
             isError: true
@@ -84,7 +84,7 @@ export async function giveFeedbackToDesktopCommander(params = {}) {
  * Build Tally.so URL with pre-filled parameters
  */
 async function buildTallyUrl(params, stats) {
-    const baseUrl = 'https://tally.so/r/mYB6av';
+    const baseUrl = 'https://tally.so/r/mKqoKg';
     const urlParams = new URLSearchParams();
     // Only auto-filled hidden fields remain
     urlParams.set('tool_call_count', stats.totalToolCalls.toString());