@wonderwhy-er/desktop-commander 0.2.26 → 0.2.28

package/dist/config-manager.d.ts

@@ -62,6 +62,10 @@ declare class ConfigManager {
      * Check if this is the first run (config file was just created)
      */
     isFirstRun(): boolean;
+    /**
+     * Get or create a persistent client ID for analytics and A/B tests
+     */
+    getOrCreateClientId(): Promise<string>;
 }
 export declare const configManager: ConfigManager;
 export {};

package/dist/config-manager.js

@@ -116,7 +116,8 @@ class ConfigManager {
             allowedDirectories: [],
             telemetryEnabled: true, // Default to opt-out approach (telemetry on by default)
             fileWriteLineLimit: 50, // Default line limit for file write operations (changed from 100)
-            fileReadLineLimit: 1000 // Default line limit for file read operations (changed from character-based)
+            fileReadLineLimit: 1000, // Default line limit for file read operations (changed from character-based)
+            pendingWelcomeOnboarding: true // New install flag - triggers A/B test for welcome page
         };
     }
     /**
@@ -193,6 +194,18 @@ class ConfigManager {
     isFirstRun() {
         return this._isFirstRun;
     }
+    /**
+     * Get or create a persistent client ID for analytics and A/B tests
+     */
+    async getOrCreateClientId() {
+        let clientId = await this.getValue('clientId');
+        if (!clientId) {
+            const { randomUUID } = await import('crypto');
+            clientId = randomUUID();
+            await this.setValue('clientId', clientId);
+        }
+        return clientId;
+    }
 }
 // Export singleton instance
 export const configManager = new ConfigManager();

package/dist/utils/ab-test.js

@@ -28,7 +28,7 @@ async function getVariant(experimentName) {
         return existing;
     }
     // New assignment based on clientId
-    const clientId = await configManager.getValue('clientId') || '';
+    const clientId = await configManager.getOrCreateClientId();
     const hash = hashCode(clientId + experimentName);
     const variantIndex = hash % experiment.variants.length;
     const variant = experiment.variants[variantIndex];

package/dist/utils/capture.js

@@ -1,5 +1,4 @@
 import { platform } from 'os';
-import { randomUUID } from 'crypto';
 import * as https from 'https';
 import { configManager } from '../config-manager.js';
 import { currentClient } from '../server.js';
@@ -13,23 +12,6 @@ catch {
 }
 // Will be initialized when needed
 let uniqueUserId = 'unknown';
-// Function to get or create a persistent UUID
-async function getOrCreateUUID() {
-    try {
-        // Try to get the UUID from the config
-        let clientId = await configManager.getValue('clientId');
-        // If it doesn't exist, create a new one and save it
-        if (!clientId) {
-            clientId = randomUUID();
-            await configManager.setValue('clientId', clientId);
-        }
-        return clientId;
-    }
-    catch (error) {
-        // Fallback to a random UUID if config operations fail
-        return randomUUID();
-    }
-}
 /**
  * Sanitizes error objects to remove potentially sensitive information like file paths
  * @param error Error object or string to sanitize
@@ -76,7 +58,7 @@ export const captureBase = async (captureURL, event, properties) => {
         }
         // Get or create the client ID if not already initialized
         if (uniqueUserId === 'unknown') {
-            uniqueUserId = await getOrCreateUUID();
+            uniqueUserId = await configManager.getOrCreateClientId();
         }
         // Get current client information for all events
         let clientContext = {};

package/dist/utils/welcome-onboarding.d.ts

@@ -2,7 +2,7 @@
  * Handle welcome page display for new users (A/B test controlled)
  *
  * Only shows to:
- * 1. New users (first run - config was just created)
+ * 1. New users (pendingWelcomeOnboarding flag set when config created)
  * 2. Users in the 'showOnboardingPage' A/B variant
  * 3. Haven't seen it yet
  */

package/dist/utils/welcome-onboarding.js

@@ -3,29 +3,42 @@ import { hasFeature } from './ab-test.js';
 import { featureFlagManager } from './feature-flags.js';
 import { openWelcomePage } from './open-browser.js';
 import { logToStderr } from './logger.js';
+import { capture } from './capture.js';
 /**
  * Handle welcome page display for new users (A/B test controlled)
  *
  * Only shows to:
- * 1. New users (first run - config was just created)
+ * 1. New users (pendingWelcomeOnboarding flag set when config created)
  * 2. Users in the 'showOnboardingPage' A/B variant
  * 3. Haven't seen it yet
  */
 export async function handleWelcomePageOnboarding() {
-    // Only for brand new users (config just created)
-    if (!configManager.isFirstRun()) {
-        return;
+    // Check if this is a new install pending A/B decision
+    // This flag is set when config is first created and survives process restarts
+    const pending = await configManager.getValue('pendingWelcomeOnboarding');
+    if (!pending) {
+        return; // Existing user or already processed
     }
+    // Track that we have a first-run user attempting onboarding
+    const loadedFromCache = featureFlagManager.wasLoadedFromCache();
     // For new users, we need to wait for feature flags to load from network
     // since they won't have a cache file yet. Without this, hasFeature() would
     // return false (no experiments defined) and all new users go to control.
-    if (!featureFlagManager.wasLoadedFromCache()) {
+    if (!loadedFromCache) {
         logToStderr('debug', 'Waiting for feature flags to load...');
         await featureFlagManager.waitForFreshFlags();
     }
     // Check A/B test assignment
     const shouldShow = await hasFeature('showOnboardingPage');
+    // Track the A/B decision
+    capture('server_welcome_page_ab_decision', {
+        variant: shouldShow ? 'treatment' : 'control',
+        loaded_from_cache: loadedFromCache
+    });
     if (!shouldShow) {
+        // Mark as control group for analytics - this will be sent with all future events
+        await configManager.setValue('sawOnboardingPage', false);
+        await configManager.setValue('pendingWelcomeOnboarding', false);
         logToStderr('debug', 'Welcome page skipped (A/B: noOnboardingPage)');
         return;
     }
@@ -37,9 +50,14 @@ export async function handleWelcomePageOnboarding() {
     try {
         await openWelcomePage();
         await configManager.setValue('sawOnboardingPage', true);
+        await configManager.setValue('pendingWelcomeOnboarding', false);
+        capture('server_welcome_page_opened', { success: true });
         logToStderr('info', 'Welcome page opened');
     }
     catch (e) {
+        // Still clear the pending flag even on failure - don't retry forever
+        await configManager.setValue('pendingWelcomeOnboarding', false);
+        capture('server_welcome_page_opened', { success: false, error: e instanceof Error ? e.message : String(e) });
         logToStderr('warning', `Failed to open welcome page: ${e instanceof Error ? e.message : e}`);
     }
 }

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.2.26";
+export declare const VERSION = "0.2.28";

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = '0.2.26';
+export const VERSION = '0.2.28';

package/package.json

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