@wonderwhy-er/desktop-commander 0.2.40 → 0.2.42

package/dist/ui/file-preview/src/app.js

@@ -11,7 +11,7 @@ import { attachDirectoryHandlers } from './directory-controller.js';
 import { buildDocumentLayout } from './document-layout.js';
 import { getDocumentFullscreenAvailability, parseReadRange, stripReadStatusLine } from './document-workspace.js';
 import { getFileTypeCapabilities, renderPayloadBody } from './file-type-handlers.js';
-import { buildOpenInEditorCommand, buildOpenInFolderCommand, detectDefaultMarkdownEditor, renderMarkdownEditorAppIcon } from './host/external-actions.js';
+import { buildOpenInEditorCommand, buildOpenInFolderCommand, renderMarkdownEditorAppIcon } from './host/external-actions.js';
 import { attachSelectionContext } from './host/selection-context.js';
 import { createMarkdownController } from './markdown/controller.js';
 import { createConflictDialogController, renderConflictDialogMarkup, } from './markdown/conflict-dialog.js';
@@ -40,7 +40,9 @@ let inlinePayloadBeforeFullscreen;
 let directoryBackPayload;
 let selectionAbortController = null;
 const markdownEditorAppCache = new Map();
-const markdownEditorAppPending = new Set();
+function getTelemetryToolName(payload) {
+    return typeof payload?.sourceTool === 'string' ? payload.sourceTool : 'read_file';
+}
 async function callToolIfReady(name, args) {
     return rpcCallTool ? rpcCallTool(name, args) : undefined;
 }
@@ -147,7 +149,10 @@ async function readAndResolvePayload(payload, onReady) {
     try {
         const freshPayload = await markdownController.readPayload(payload.filePath);
         if (freshPayload) {
-            onReady(freshPayload);
+            onReady({
+                ...freshPayload,
+                sourceTool: payload.sourceTool ?? freshPayload.sourceTool,
+            });
             if (freshPayload.fileType === 'markdown') {
                 void markdownController.refreshFromDisk(freshPayload);
             }
@@ -212,21 +217,15 @@ export function renderApp(container, payload, htmlMode = 'rendered', expandedSta
     const canGoFullscreen = !isFullscreen && getDocumentFullscreenAvailability({
         availableDisplayModes: getAvailableDisplayModes(),
     }).canFullscreen;
+    if (payload.fileType === 'markdown' && payload.defaultEditorName) {
+        markdownEditorAppCache.set(payload.filePath, {
+            appName: payload.defaultEditorName,
+            appPath: payload.defaultEditorPath,
+        });
+    }
     const defaultMarkdownEditor = payload.fileType === 'markdown'
         ? markdownEditorAppCache.get(payload.filePath)
         : undefined;
-    if (payload.fileType === 'markdown' && !defaultMarkdownEditor) {
-        void detectDefaultMarkdownEditor({
-            filePath: payload.filePath,
-            editorAppCache: markdownEditorAppCache,
-            editorAppPending: markdownEditorAppPending,
-            callTool: callToolIfReady,
-            extractToolText,
-            onDetected: () => {
-                rerenderCurrent?.();
-            },
-        });
-    }
     const layout = buildDocumentLayout({
         payload,
         body,
@@ -415,9 +414,12 @@ export function bootstrapApp() {
         const result = await app.requestDisplayMode({ mode });
         return typeof result.mode === 'string' ? result.mode : null;
     };
-    trackUiEvent = createUiEventTracker((name, args) => app.callServerTool({ name, arguments: args }), {
+    const filePreviewUiEvent = createUiEventTracker((name, args) => app.callServerTool({ name, arguments: args }), {
         component: 'file_preview',
-        baseParams: { tool_name: 'read_file' },
+    });
+    trackUiEvent = (event, params = {}) => filePreviewUiEvent(event, {
+        tool_name: getTelemetryToolName(currentPayload ?? hostPayload),
+        ...params,
     });
     app.ontoolinput = (params) => {
         const requestedPath = typeof params.arguments?.path === 'string' ? params.arguments.path : undefined;
@@ -548,11 +550,6 @@ export function bootstrapApp() {
         onConnected: () => {
             currentHostContext = app.getHostContext();
             pendingCachedPayload = widgetState.read() ?? undefined;
-            window.setTimeout(() => {
-                if (!initialStateResolved) {
-                    resolveInitialState(undefined, 'Preview unavailable after page refresh. Switch threads or re-run the tool.');
-                }
-            }, 8000);
         },
     }).catch(() => {
         renderStatusState(container, 'Failed to connect to host.');

package/dist/ui/file-preview/src/directory-controller.js

@@ -5,7 +5,7 @@ function parseDirectoryEntries(content) {
     const hintLines = [];
     const entryLines = [];
     for (const line of lines) {
-        if (/^\[(DIR|FILE|DENIED|WARNING)\]/.test(line.trim())) {
+        if (/^\[(DIR|FILE|DENIED|NOT_FOUND|WARNING)\]/.test(line.trim())) {
             entryLines.push(line.trim());
         }
         else if (entryLines.length === 0) {
@@ -25,6 +25,7 @@ function parseDirectoryEntries(content) {
                 fullPath: dirName,
                 isDir: false,
                 isDenied: false,
+                isNotFound: false,
                 isWarning: true,
                 warningText: msg,
                 depth: parts.length,
@@ -33,13 +34,15 @@ function parseDirectoryEntries(content) {
         }
         const isDir = line.startsWith('[DIR]');
         const isDenied = line.startsWith('[DENIED]');
-        const name = line.replace(/^\[(DIR|FILE|DENIED)\]\s*/, '');
+        const isNotFound = line.startsWith('[NOT_FOUND]');
+        const name = line.replace(/^\[(DIR|FILE|DENIED|NOT_FOUND)\]\s*/, '');
         const parts = name.replace(/\\/g, '/').split('/');
         flat.push({
             name,
             fullPath: name,
             isDir,
             isDenied,
+            isNotFound,
             isWarning: false,
             warningText: '',
             depth: parts.length - 1,
@@ -53,6 +56,7 @@ function parseDirectoryEntries(content) {
             name: baseName,
             isDir: item.isDir,
             isDenied: item.isDenied,
+            isNotFound: item.isNotFound,
             isWarning: item.isWarning,
             warningText: item.warningText,
             children: [],
@@ -85,6 +89,9 @@ function renderDirTree(entries, rootPath) {
             if (item.isDenied) {
                 return `<div class="dir-entry"><span class="dir-icon">🚫</span> <span class="dir-name-denied">${escapeHtml(item.name)}</span></div>`;
             }
+            if (item.isNotFound) {
+                return `<div class="dir-entry"><span class="dir-icon">❓</span> <span class="dir-name-denied">${escapeHtml(item.name)}</span></div>`;
+            }
             if (item.isDir) {
                 const hasChildren = item.children.length > 0;
                 const chevron = `<span class="dir-chevron${hasChildren ? ' expanded' : ''}">${hasChildren ? '▼' : '▶'}</span>`;

package/dist/ui/file-preview/src/file-type-handlers.js

@@ -74,15 +74,26 @@ const handlerRegistry = {
         },
     },
     unsupported: {
-        getCapabilities: () => ({
-            supportsPreview: false,
-            canCopy: false,
-            canOpenInFolder: true,
-        }),
-        renderBody: () => ({
-            notice: 'Preview is not available for this file type.',
-            html: '<div class="panel-content source-content"></div>',
-        }),
+        getCapabilities: (payload) => {
+            const hasRawContent = stripReadStatusLine(payload.content).trim().length > 0;
+            return {
+                supportsPreview: hasRawContent,
+                canCopy: hasRawContent,
+                canOpenInFolder: !isLikelyUrl(payload.filePath),
+            };
+        },
+        renderBody: ({ payload }) => {
+            const rawContent = stripReadStatusLine(payload.content);
+            if (rawContent.trim().length === 0) {
+                return {
+                    notice: 'Preview is not available for this file type.',
+                    html: '<div class="panel-content source-content"></div>',
+                };
+            }
+            return {
+                html: `<div class="panel-content source-content">${renderRawFallback(rawContent)}</div>`,
+            };
+        },
     },
 };
 export function getFileTypeCapabilities(payload) {

package/dist/ui/file-preview/src/host/external-actions.d.ts

@@ -5,15 +5,4 @@ export declare function buildOpenInEditorCommand(filePath: string, isLikelyUrl:
     appName: string;
     appPath?: string;
 }>): string | undefined;
-export declare function detectDefaultMarkdownEditor(options: {
-    filePath: string;
-    editorAppCache: Map<string, {
-        appName: string;
-        appPath?: string;
-    }>;
-    editorAppPending: Set<string>;
-    callTool?: (name: string, args: Record<string, unknown>) => Promise<unknown | undefined>;
-    extractToolText: (value: unknown) => string | undefined;
-    onDetected?: () => void;
-}): Promise<void>;
 export declare function renderMarkdownEditorAppIcon(): string;

package/dist/ui/file-preview/src/host/external-actions.js

@@ -50,45 +50,6 @@ export function buildOpenInEditorCommand(filePath, isLikelyUrl, editorAppCache)
     }
     return `xdg-open ${shellQuote(trimmedPath)}`;
 }
-export async function detectDefaultMarkdownEditor(options) {
-    const trimmedPath = options.filePath.trim();
-    if (!trimmedPath || options.editorAppCache.has(trimmedPath) || options.editorAppPending.has(trimmedPath)) {
-        return;
-    }
-    const userAgent = navigator.userAgent.toLowerCase();
-    if (!userAgent.includes('mac')) {
-        return;
-    }
-    options.editorAppPending.add(trimmedPath);
-    try {
-        const detectCommand = `osascript -e ${shellQuote(`set appAlias to default application of (info for POSIX file "${trimmedPath.replace(/"/g, '\\"')}")
-return (name of (info for appAlias)) & linefeed & POSIX path of appAlias`)}`;
-        const detectResult = await options.callTool?.('start_process', {
-            command: detectCommand,
-            timeout_ms: 12000,
-        });
-        const text = options.extractToolText(detectResult) ?? '';
-        if (!text || text.toLowerCase().includes('error') || text.toLowerCase().includes('execution')) {
-            return;
-        }
-        const lines = text.split('\n').map((line) => line.trim()).filter(Boolean);
-        const appName = lines[lines.length - 2]?.replace(/\.app$/i, '') ?? '';
-        const appPath = lines[lines.length - 1] ?? '';
-        if (appName && appPath.startsWith('/')) {
-            options.editorAppCache.set(trimmedPath, {
-                appName,
-                appPath,
-            });
-            options.onDetected?.();
-        }
-    }
-    catch {
-        // Fall back to generic editor label.
-    }
-    finally {
-        options.editorAppPending.delete(trimmedPath);
-    }
-}
 export function renderMarkdownEditorAppIcon() {
     return '<svg width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round" aria-hidden="true"><path d="M12 20h9"/><path d="M16.5 3.5a2.1 2.1 0 1 1 3 3L7 19l-4 1 1-4Z"/></svg>';
 }

package/dist/ui/file-preview/src/payload-utils.js

@@ -41,6 +41,12 @@ export function extractToolText(value) {
     }
     return undefined;
 }
+function extractStructuredContentText(value) {
+    if (!isObjectRecord(value)) {
+        return undefined;
+    }
+    return typeof value.content === 'string' ? value.content : undefined;
+}
 export function extractRenderPayload(value) {
     if (!isObjectRecord(value)) {
         return undefined;
@@ -52,7 +58,10 @@ export function extractRenderPayload(value) {
             : null;
     if (!meta)
         return undefined;
-    const text = extractToolText(value) ?? extractToolText(value.structuredContent) ?? '';
+    const text = extractStructuredContentText(value.structuredContent)
+        ?? extractToolText(value)
+        ?? extractToolText(value.structuredContent)
+        ?? '';
     return buildRenderPayload(meta, text);
 }
 export function assertSuccessfulEditBlockResult(result) {

package/dist/uninstall-claude-server.js

@@ -10,10 +10,9 @@ import { version as nodeVersion } from 'process';
 import * as https from 'https';
 import { randomUUID } from 'crypto';
 
-// Google Analytics configuration
-const GA_MEASUREMENT_ID = 'G-NGGDNL0K4L'; // Replace with your GA4 Measurement ID
-const GA_API_SECRET = '5M0mC--2S_6t94m8WrI60A';   // Replace with your GA4 API Secret
-const GA_BASE_URL = `https://www.google-analytics.com/mp/collect?measurement_id=${GA_MEASUREMENT_ID}&api_secret=${GA_API_SECRET}`;
+// Telemetry proxy configuration
+const TELEMETRY_PROXY_URL = 'https://telemetry.desktopcommander.app/mp/collect';
+const TELEMETRY_PROXY_FALLBACK_URL = 'https://dc-telemetry-proxy-83847352264.europe-west1.run.app/mp/collect';
 
 // Read clientId and telemetry settings from existing config
 let uniqueUserId = 'unknown';
@@ -222,11 +221,6 @@ async function trackEvent(eventName, additionalProps = {}) {
         return true; // Return success since this is expected behavior
     }
 
-    if (!GA_MEASUREMENT_ID || !GA_API_SECRET) {
-        updateUninstallStep(trackingStep, 'skipped', new Error('GA not configured'));
-        return;
-    }
-
     const maxRetries = 2;
     let attempt = 0;
     let lastError = null;
@@ -257,43 +251,8 @@ async function trackEvent(eventName, additionalProps = {}) {
                 }
             };
 
-            const result = await new Promise((resolve, reject) => {
-                const req = https.request(GA_BASE_URL, options);
-
-                const timeoutId = setTimeout(() => {
-                    req.destroy();
-                    reject(new Error('Request timeout'));
-                }, 5000);
-
-                req.on('error', (error) => {
-                    clearTimeout(timeoutId);
-                    reject(error);
-                });
-
-                req.on('response', (res) => {
-                    clearTimeout(timeoutId);
-                    let data = '';
-
-                    res.on('data', (chunk) => {
-                        data += chunk;
-                    });
-
-                    res.on('error', (error) => {
-                        reject(error);
-                    });
-
-                    res.on('end', () => {
-                        if (res.statusCode >= 200 && res.statusCode < 300) {
-                            resolve({ success: true, data });
-                        } else {
-                            reject(new Error(`HTTP error ${res.statusCode}: ${data}`));
-                        }
-                    });
-                });
-
-                req.write(postData);
-                req.end();
-            });
+            const result = await postTelemetryPayload(postData, options);
+            if (!result.success) throw new Error('Telemetry proxy request failed');
 
             updateUninstallStep(trackingStep, 'completed');
             return result;
@@ -309,6 +268,54 @@ async function trackEvent(eventName, additionalProps = {}) {
     updateUninstallStep(trackingStep, 'failed', lastError);
     return false;
 }
+
+async function postTelemetryPayload(postData, options) {
+    for (const endpoint of [TELEMETRY_PROXY_URL, TELEMETRY_PROXY_FALLBACK_URL]) {
+        const result = await new Promise((resolve) => {
+            let settled = false;
+            let timeoutId;
+            const finish = (result) => {
+                if (settled) return;
+                settled = true;
+                clearTimeout(timeoutId);
+                resolve(result);
+            };
+            const req = https.request(endpoint, options);
+
+            timeoutId = setTimeout(() => {
+                req.destroy();
+                finish({ success: false, data: '' });
+            }, 5000);
+
+            req.on('error', () => {
+                finish({ success: false, data: '' });
+            });
+
+            req.on('response', (res) => {
+                let data = '';
+                res.on('data', (chunk) => {
+                    data += chunk;
+                });
+                res.on('error', () => {
+                    finish({ success: false, data: '' });
+                });
+                res.on('end', () => {
+                    finish({ success: res.statusCode >= 200 && res.statusCode < 300, data });
+                });
+                res.on('close', () => {
+                    finish({ success: false, data: '' });
+                });
+            });
+
+            req.write(postData);
+            req.end();
+        });
+
+        if (result.success) return result;
+    }
+
+    return { success: false, data: '' };
+}
 // Ensure tracking completes before process exits
 async function ensureTrackingCompleted(eventName, additionalProps = {}, timeoutMs = 6000) {
     return new Promise(async (resolve) => {
@@ -750,4 +757,4 @@ if (process.argv.length >= 2 && process.argv[1] === fileURLToPath(import.meta.ur
         logToFile(`Fatal error: ${error}`, true);
         process.exit(1);
     });
-}
+}

package/dist/utils/ab-test.d.ts

@@ -1,3 +1,7 @@
+/**
+ * Get the exact assigned variant for a named experiment.
+ */
+export declare function getABTestVariant(experimentName: string): Promise<string | null>;
 /**
  * Check if a feature (variant name) is enabled for current user
  */

package/dist/utils/ab-test.js

@@ -57,6 +57,12 @@ async function getVariant(experimentName) {
     variantCache[experimentName] = variant;
     return variant;
 }
+/**
+ * Get the exact assigned variant for a named experiment.
+ */
+export async function getABTestVariant(experimentName) {
+    return getVariant(experimentName);
+}
 /**
  * Check if a feature (variant name) is enabled for current user
  */

package/dist/utils/capture.d.ts

@@ -1,3 +1,11 @@
+/**
+ * Hard kill-switch for telemetry via environment variable.
+ *
+ * Independent of the persisted `telemetryEnabled` config so that tests, CI and
+ * one-off runs can suppress all analytics without mutating the user's config.
+ * Set DESKTOP_COMMANDER_DISABLE_TELEMETRY to 1/true/yes/on to disable.
+ */
+export declare function isTelemetryDisabledByEnv(): boolean;
 /**
  * Sanitizes error objects to remove potentially sensitive information like file paths
  * @param error Error object or string to sanitize
@@ -8,13 +16,13 @@ export declare function sanitizeError(error: any): {
     code?: string;
 };
 /**
- * Send an event to Google Analytics
+ * Send an event to telemetry
  * @param event Event name
  * @param properties Optional event properties
  */
 export declare const captureBase: (captureURL: string, event: string, properties?: any) => Promise<void>;
-export declare const capture_call_tool: (event: string, properties?: any) => Promise<void>;
 export declare const capture: (event: string, properties?: any) => Promise<void>;
+export declare const capture_call_tool: (event: string, properties?: any) => Promise<void>;
 export declare const capture_ui_event: (event: string, properties?: any) => Promise<void>;
 /**
  * Wrapper for capture() that automatically adds remote flag for remote-device telemetry