@hcu-lab.me/mcp-device 1.0.5 → 1.0.7

package/dist/server.js

@@ -1,4 +1,3 @@
-import path from 'path';
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { CallToolRequestSchema, ListToolsRequestSchema, ListResourcesRequestSchema, ReadResourceRequestSchema, ListResourceTemplatesRequestSchema, ListPromptsRequestSchema, InitializeRequestSchema, LATEST_PROTOCOL_VERSION, SUPPORTED_PROTOCOL_VERSIONS, } from "@modelcontextprotocol/sdk/types.js";
 import { zodToJsonSchema } from "zod-to-json-schema";
@@ -9,11 +8,10 @@ const OS_GUIDANCE = getOSSpecificGuidance(SYSTEM_INFO);
 const DEV_TOOL_GUIDANCE = getDevelopmentToolGuidance(SYSTEM_INFO);
 const PATH_GUIDANCE = `IMPORTANT: ${getPathGuidance(SYSTEM_INFO)} Relative paths may fail as they depend on the current working directory. Tilde paths (~/...) might not work in all contexts. Unless the user explicitly asks for relative paths, use absolute paths.`;
 const CMD_PREFIX_DESCRIPTION = `This command can be referenced as "MCP Device: ..." or "use MCP Device to ..." in your instructions.`;
-import { StartProcessArgsSchema, ReadProcessOutputArgsSchema, InteractWithProcessArgsSchema, ForceTerminateArgsSchema, ListSessionsArgsSchema, KillProcessArgsSchema, ReadFileArgsSchema, ReadMultipleFilesArgsSchema, WriteFileArgsSchema, CreateDirectoryArgsSchema, ListDirectoryArgsSchema, MoveFileArgsSchema, GetFileInfoArgsSchema, GetConfigArgsSchema, SetConfigValueArgsSchema, ListProcessesArgsSchema, EditBlockArgsSchema, GetUsageStatsArgsSchema, GiveFeedbackArgsSchema, StartSearchArgsSchema, GetMoreSearchResultsArgsSchema, StopSearchArgsSchema, ListSearchesArgsSchema, GetPromptsArgsSchema, GetRecentToolCallsArgsSchema, WritePdfArgsSchema, toolArgSchemas, } from './tools/schemas.js';
+import { StartProcessArgsSchema, ReadProcessOutputArgsSchema, InteractWithProcessArgsSchema, ForceTerminateArgsSchema, ListSessionsArgsSchema, KillProcessArgsSchema, ReadFileArgsSchema, ReadMultipleFilesArgsSchema, WriteFileArgsSchema, CreateDirectoryArgsSchema, ListDirectoryArgsSchema, MoveFileArgsSchema, GetFileInfoArgsSchema, GetConfigArgsSchema, SetConfigValueArgsSchema, ListProcessesArgsSchema, EditBlockArgsSchema, GetUsageStatsArgsSchema, StartSearchArgsSchema, GetMoreSearchResultsArgsSchema, StopSearchArgsSchema, ListSearchesArgsSchema, GetPromptsArgsSchema, GetRecentToolCallsArgsSchema, WritePdfArgsSchema, toolArgSchemas, } from './tools/schemas.js';
 import { detectUnsupportedParams, getSupportedParams, buildUnsupportedParamsWarning, } from './utils/unsupportedParams.js';
 import { getConfig, setConfigValue } from './tools/config.js';
 import { getUsageStats } from './tools/usage.js';
-import { giveFeedbackToDesktopCommander } from './tools/feedback.js';
 import { getPrompts } from './tools/prompts.js';
 import { trackToolCall } from './utils/trackTools.js';
 import { usageTracker } from './utils/usageTracker.js';
@@ -89,7 +87,7 @@ function setCurrentCallIsRemote(isRemote) {
     currentCallIsRemote = isRemote;
 }
 // The remote caller's client for the in-flight tool call (e.g. openai-mcp,
-// claude-ai). Set per CallTool when the call is remote; null for local calls.
+// remote client). Set per CallTool when the call is remote; null for local calls.
 // Mirrors currentCallIsRemote so telemetry attributes remote events to the
 // actual remote client instead of the device's own currentClient (which stays
 // LOCAL and must not be polluted by remote callers).
@@ -104,11 +102,10 @@ function setCurrentRemoteClient(clientInfo) {
 /**
  * True when this server instance is serving remote services rather than a
  * local MCP client. The device wrapper marks the server it spawns with
- * MCP_DEVICE_REMOTE=true (see device/execution-engine.ts);
- * the client-name check covers older wrappers that predate the env marker.
+ * MCP_DEVICE_REMOTE=true (see device/execution-engine.ts).
  */
 function isRemoteClientContext(clientName) {
-    return process.env.MCP_DEVICE_REMOTE === 'true' || clientName === 'desktop-commander-client';
+    return process.env.MCP_DEVICE_REMOTE === 'true';
 }
 /**
  * Unified way to update client information
@@ -138,40 +135,19 @@ server.setRequestHandler(InitializeRequestSchema, async (request) => {
         const clientInfo = request.params?.clientInfo;
         if (clientInfo) {
             await updateCurrentClient(clientInfo);
-            // Welcome page for new users (A/B test controlled) — all clients except
-            // the Desktop Commander app and remote contexts. Further exclusions are
-            // flag-served via welcome_page_excluded_clients (e.g. claude-code, which
-            // covers Claude Code and Cowork plugin sessions — both identify as
-            // `claude-code` and provide their own onboarding surface).
-            const isWelcomePageEligibleClient = currentClient.name !== 'desktop-commander-app'
-                && currentClient.name !== 'desktop-commander'
-                && !isRemoteClientContext(currentClient.name)
+            const isWelcomePageEligibleClient = !isRemoteClientContext(currentClient.name)
                 && !global.disableOnboarding;
-            if (isWelcomePageEligibleClient) {
+            if (isRemoteClientContext(currentClient.name)) {
+                // Remote child engines do not mutate local onboarding state.
+            }
+            else if (isWelcomePageEligibleClient) {
                 await handleWelcomePageOnboarding(currentClient.name);
             }
             else {
-                // Do not carry a first-run page over to a client that is made
-                // eligible in a later release.
                 await skipWelcomePageOnboarding();
             }
+            capture('run_server_mcp_initialized');
         }
-        // Raw host environment signals (no PII, undefined when absent). Some
-        // hosts share a clientInfo name — Claude Code CLI, Claude Code inside
-        // the Claude Desktop app, and Cowork all report 'claude-code' — and
-        // these let analytics tell them apart without client-specific
-        // branching in code. Verified signatures: CLI → entrypoint 'cli';
-        // CC-in-desktop → entrypoint 'claude-desktop'; Cowork → no
-        // entrypoint/agent, plugin id 'desktop-commander-inline'.
-        // Values truncated to GA4's 100-char param limit (same convention as
-        // containerName/containerImage) so an oversized value can never get
-        // the whole event rejected.
-        capture('run_server_mcp_initialized', {
-            host_entrypoint: process.env.CLAUDE_CODE_ENTRYPOINT?.substring(0, 100),
-            host_agent: process.env.AI_AGENT?.substring(0, 100),
-            host_plugin_id: process.env.CLAUDE_PLUGIN_DATA
-                ? path.basename(process.env.CLAUDE_PLUGIN_DATA).substring(0, 100) : undefined
-        });
         // Negotiate protocol version with client
         const requestedVersion = request.params?.protocolVersion;
         const protocolVersion = (requestedVersion && SUPPORTED_PROTOCOL_VERSIONS.includes(requestedVersion))
@@ -203,15 +179,7 @@ deferLog('info', 'Setting up request handlers...');
 /**
  * Check if a tool should be included based on current client
  */
-function shouldIncludeTool(toolName) {
-    // Exclude these tools for desktop-commander client (DC-specific meta-tools not useful when DC itself is the client)
-    if (currentClient?.name === 'desktop-commander-app') {
-        if (toolName === 'give_feedback_to_desktop_commander' || toolName === 'get_prompts') {
-            return false;
-        }
-    }
-    // Add more conditional tool logic here as needed
-    // Example: if (toolName === 'some_tool' && currentClient?.name === 'some_client') return false;
+function shouldIncludeTool(_toolName) {
     return true;
 }
 server.setRequestHandler(ListToolsRequestSchema, async () => {
@@ -1069,48 +1037,6 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                     readOnlyHint: true,
                 },
             },
-            {
-                name: "give_feedback_to_desktop_commander",
-                description: `
-                        Open feedback form in browser to provide feedback about MCP Device.
-                        
-                        IMPORTANT: This tool simply opens the feedback form - no pre-filling available.
-                        The user will fill out the form manually in their browser.
-                        
-                        WORKFLOW:
-                        1. When user agrees to give feedback, just call this tool immediately
-                        2. No need to ask questions or collect information
-                        3. Tool opens form with only usage statistics pre-filled automatically:
-                           - tool_call_count: Number of commands they've made
-                           - days_using: How many days they've used MCP Device
-                           - platform: Their operating system (Mac/Windows/Linux)
-                           - client_id: Analytics identifier
-                        
-                        All survey questions will be answered directly in the form:
-                        - Job title and technical comfort level
-                        - Company URL for industry context
-                        - Other AI tools they use
-                        - MCP Device's biggest advantage
-                        - How they typically use it
-                        - Recommendation likelihood (0-10)
-                        - User study participation interest
-                        - Email and any additional feedback
-                        
-                        EXAMPLE INTERACTION:
-                        User: "sure, I'll give feedback"
-                        Claude: "Perfect! Let me open the feedback form for you."
-                        [calls tool immediately]
-                        
-                        No parameters are needed - just call the tool to open the form.
-                        
-                        ${CMD_PREFIX_DESCRIPTION}`,
-                inputSchema: zodToJsonSchema(GiveFeedbackArgsSchema),
-                annotations: {
-                    title: "Give Feedback",
-                    readOnlyHint: false,
-                    openWorldHint: true,
-                },
-            },
             {
                 name: "get_prompts",
                 description: `
@@ -1190,7 +1116,7 @@ async function handleCallToolRequest(request) {
             // add remote flag (convert to string for telemetry)
             telemetryData.remote = String(metadata.remote);
             // Remote calls carry the originating MCP client (e.g. openai-mcp,
-            // claude-ai) in _meta.clientInfo. Attribute this call to that remote
+            // remote client) in _meta.clientInfo. Attribute this call to that remote
             // client — NOT the device's own currentClient. Fall back to a sentinel
             // when it's absent so the call is visibly remote-but-unattributed
             // rather than masquerading as the local device client. We deliberately
@@ -1332,18 +1258,6 @@ async function handleCallToolRequest(request) {
                     };
                 }
                 break;
-            case "give_feedback_to_desktop_commander":
-                try {
-                    result = await giveFeedbackToDesktopCommander(args);
-                }
-                catch (error) {
-                    capture('server_request_error', { message: `Error in give_feedback_to_desktop_commander handler: ${error}` });
-                    result = {
-                        content: [{ type: "text", text: `Error: Failed to open feedback form` }],
-                        isError: true,
-                    };
-                }
-                break;
             // Terminal tools
             case "start_process":
                 result = await handlers.handleStartProcess(args);
@@ -1422,7 +1336,7 @@ async function handleCallToolRequest(request) {
             'get_recent_tool_calls',
             'track_ui_event'
         ];
-        if (!EXCLUDED_TOOLS.includes(name)) {
+        if (!currentCallIsRemote && process.env.MCP_DEVICE_REMOTE !== 'true' && !EXCLUDED_TOOLS.includes(name)) {
             toolHistory.addCall(name, args, result, duration);
         }
         // Track success or failure based on result
@@ -1436,77 +1350,79 @@ async function handleCallToolRequest(request) {
         else {
             await usageTracker.trackSuccess(name);
             console.log(`[FEEDBACK DEBUG] Tool ${name} succeeded, checking feedback...`);
-            // Check if should show onboarding (before feedback - first-time users are priority)
-            const shouldShowOnboarding = await usageTracker.shouldShowOnboarding();
-            console.log(`[ONBOARDING DEBUG] Should show onboarding: ${shouldShowOnboarding}`);
-            if (shouldShowOnboarding) {
-                console.log(`[ONBOARDING DEBUG] Generating onboarding message...`);
-                const onboardingResult = await usageTracker.getOnboardingMessage();
-                console.log(`[ONBOARDING DEBUG] Generated variant: ${onboardingResult.variant}`);
-                // Capture onboarding prompt injection event
-                const stats = await usageTracker.getStats();
-                await capture('server_onboarding_shown', {
-                    trigger_tool: name,
-                    total_calls: stats.totalToolCalls,
-                    successful_calls: stats.successfulCalls,
-                    days_since_first_use: Math.floor((Date.now() - stats.firstUsed) / (1000 * 60 * 60 * 24)),
-                    total_sessions: stats.totalSessions,
-                    message_variant: onboardingResult.variant
-                });
-                // Inject onboarding message for the LLM
-                if (result.content && result.content.length > 0 && result.content[0].type === "text") {
-                    const currentContent = result.content[0].text || '';
-                    result.content[0].text = `${currentContent}${onboardingResult.message}`;
-                }
-                else {
-                    result.content = [
-                        ...(result.content || []),
-                        {
-                            type: "text",
-                            text: onboardingResult.message
-                        }
-                    ];
-                }
-                // Mark that we've shown onboarding (to prevent spam)
-                await usageTracker.markOnboardingShown(onboardingResult.variant);
-            }
-            // Check if should prompt for feedback (only on successful operations)
-            const shouldPrompt = await usageTracker.shouldPromptForFeedback();
-            console.log(`[FEEDBACK DEBUG] Should prompt for feedback: ${shouldPrompt}`);
-            if (shouldPrompt) {
-                console.log(`[FEEDBACK DEBUG] Generating feedback message...`);
-                const feedbackResult = await usageTracker.getFeedbackPromptMessage();
-                console.log(`[FEEDBACK DEBUG] Generated variant: ${feedbackResult.variant}`);
-                // Capture feedback prompt injection event
-                const stats = await usageTracker.getStats();
-                await capture('feedback_prompt_injected', {
-                    trigger_tool: name,
-                    total_calls: stats.totalToolCalls,
-                    successful_calls: stats.successfulCalls,
-                    failed_calls: stats.failedCalls,
-                    days_since_first_use: Math.floor((Date.now() - stats.firstUsed) / (1000 * 60 * 60 * 24)),
-                    total_sessions: stats.totalSessions,
-                    message_variant: feedbackResult.variant
-                });
-                // Inject feedback instruction for the LLM
-                if (result.content && result.content.length > 0 && result.content[0].type === "text") {
-                    const currentContent = result.content[0].text || '';
-                    result.content[0].text = `${currentContent}${feedbackResult.message}`;
+            if (!currentCallIsRemote && !isRemoteClientContext(currentClient.name)) {
+                // Check if should show onboarding (before feedback - first-time users are priority)
+                const shouldShowOnboarding = await usageTracker.shouldShowOnboarding();
+                console.log(`[ONBOARDING DEBUG] Should show onboarding: ${shouldShowOnboarding}`);
+                if (shouldShowOnboarding) {
+                    console.log(`[ONBOARDING DEBUG] Generating onboarding message...`);
+                    const onboardingResult = await usageTracker.getOnboardingMessage();
+                    console.log(`[ONBOARDING DEBUG] Generated variant: ${onboardingResult.variant}`);
+                    // Capture onboarding prompt injection event
+                    const stats = await usageTracker.getStats();
+                    await capture('server_onboarding_shown', {
+                        trigger_tool: name,
+                        total_calls: stats.totalToolCalls,
+                        successful_calls: stats.successfulCalls,
+                        days_since_first_use: Math.floor((Date.now() - stats.firstUsed) / (1000 * 60 * 60 * 24)),
+                        total_sessions: stats.totalSessions,
+                        message_variant: onboardingResult.variant
+                    });
+                    // Inject onboarding message for the LLM
+                    if (result.content && result.content.length > 0 && result.content[0].type === "text") {
+                        const currentContent = result.content[0].text || '';
+                        result.content[0].text = `${currentContent}${onboardingResult.message}`;
+                    }
+                    else {
+                        result.content = [
+                            ...(result.content || []),
+                            {
+                                type: "text",
+                                text: onboardingResult.message
+                            }
+                        ];
+                    }
+                    // Mark that we've shown onboarding (to prevent spam)
+                    await usageTracker.markOnboardingShown(onboardingResult.variant);
                 }
-                else {
-                    result.content = [
-                        ...(result.content || []),
-                        {
-                            type: "text",
-                            text: feedbackResult.message
-                        }
-                    ];
+                // Check if should prompt for feedback (only on successful operations)
+                const shouldPrompt = await usageTracker.shouldPromptForFeedback();
+                console.log(`[FEEDBACK DEBUG] Should prompt for feedback: ${shouldPrompt}`);
+                if (shouldPrompt) {
+                    console.log(`[FEEDBACK DEBUG] Generating feedback message...`);
+                    const feedbackResult = await usageTracker.getFeedbackPromptMessage();
+                    console.log(`[FEEDBACK DEBUG] Generated variant: ${feedbackResult.variant}`);
+                    // Capture feedback prompt injection event
+                    const stats = await usageTracker.getStats();
+                    await capture('feedback_prompt_injected', {
+                        trigger_tool: name,
+                        total_calls: stats.totalToolCalls,
+                        successful_calls: stats.successfulCalls,
+                        failed_calls: stats.failedCalls,
+                        days_since_first_use: Math.floor((Date.now() - stats.firstUsed) / (1000 * 60 * 60 * 24)),
+                        total_sessions: stats.totalSessions,
+                        message_variant: feedbackResult.variant
+                    });
+                    // Inject feedback instruction for the LLM
+                    if (result.content && result.content.length > 0 && result.content[0].type === "text") {
+                        const currentContent = result.content[0].text || '';
+                        result.content[0].text = `${currentContent}${feedbackResult.message}`;
+                    }
+                    else {
+                        result.content = [
+                            ...(result.content || []),
+                            {
+                                type: "text",
+                                text: feedbackResult.message
+                            }
+                        ];
+                    }
+                    // Mark that we've prompted (to prevent spam)
+                    await usageTracker.markFeedbackPrompted();
                 }
-                // Mark that we've prompted (to prevent spam)
-                await usageTracker.markFeedbackPrompted();
+                // Check if should prompt about Docker environment
+                result = await processDockerPrompt(result, name);
             }
-            // Check if should prompt about Docker environment
-            result = await processDockerPrompt(result, name);
         }
         // If the caller sent parameters this tool does not support, Zod silently
         // strips them. Prepend a corrective warning so the model knows they were

package/dist/terminal-manager.js

@@ -8,7 +8,7 @@ import { analyzeProcessState } from './utils/process-detection.js';
  * Standard Windows PATHEXT value, used to repair a corrupted PATHEXT before
  * spawning child shells.
  *
- * On some Windows Claude Desktop / DXT launches the server process inherits a
+ * On some Windows host launches the server process inherits a
  * broken PATHEXT (observed as ".CPL" only). Because we build the child env from
  * { ...process.env }, that broken value would propagate into every spawned
  * shell, stripping ".EXE" and breaking resolution of git / node / python / rg /
@@ -205,6 +205,21 @@ export class TerminalManager {
         // Spawn the process with appropriate arguments
         const childProcess = spawn(spawnConfig.executable, spawnConfig.args, spawnOptions);
         let output = '';
+        // spawn() reports resolution/launch failures asynchronously via 'error'.
+        // A listener-less ChildProcess error becomes an uncaughtException and kills
+        // the inherited MCP server. Attach before any early-return path and keep
+        // the listener for the process lifetime.
+        let forwardProcessError = null;
+        let pendingProcessError = null;
+        childProcess.on('error', (err) => {
+            if (forwardProcessError) {
+                forwardProcessError(err);
+            }
+            else {
+                pendingProcessError = err;
+                console.error(`Process error for "${command}": ${err.message}`);
+            }
+        });
         // Ensure childProcess.pid is defined before proceeding
         if (!childProcess.pid) {
             // Return a consistent error object instead of throwing
@@ -259,6 +274,31 @@ export class TerminalManager {
                 }
                 resolve(result);
             };
+            // Once resolveOnce exists, route runtime/spawn errors into the normal
+            // command result path so callers do not hang and the session is retired.
+            forwardProcessError = (err) => {
+                // A late ChildProcess 'error' can arrive after the caller has already
+                // received a live/background session. Do not silently retire that
+                // session; the normal 'exit' event remains authoritative for cleanup.
+                if (resolved) {
+                    this.appendToLineBuffer(session, `\nProcess error: ${err.message}\n`);
+                    console.error(`Process error for "${command}": ${err.message}`);
+                    return;
+                }
+                if (childProcess.pid)
+                    this.sessions.delete(childProcess.pid);
+                exitReason = 'process_exit';
+                resolveOnce({
+                    pid: childProcess.pid ?? -1,
+                    output: output + `\nProcess error: ${err.message}`,
+                    isBlocked: false
+                });
+            };
+            // Spawn failures normally arrive on the next tick, but replay any error
+            // emitted between spawn() and resolver initialization.
+            if (pendingProcessError) {
+                forwardProcessError(pendingProcessError);
+            }
             childProcess.stdout.on('data', (data) => {
                 const text = data.toString();
                 const now = Date.now();

package/dist/tools/config.d.ts

@@ -65,7 +65,7 @@ export declare function getConfig(): Promise<{
             availableShells: string[];
         };
         entries: {
-            key: "defaultShell" | "blockedCommands" | "allowedDirectories" | "telemetryEnabled" | "fileReadLineLimit" | "fileWriteLineLimit";
+            key: "blockedCommands" | "defaultShell" | "allowedDirectories" | "telemetryEnabled" | "fileWriteLineLimit" | "fileReadLineLimit";
             value: unknown;
             valueType: "string" | "number" | "boolean" | "array";
             editable: boolean;

package/dist/tools/filesystem.js

@@ -84,10 +84,10 @@ function buildPermissionError(filePath, errCode) {
         `  5. The app does not have permission to access this location (macOS Full Disk Access).`,
     ];
     if (isMac) {
-        lines.push(`       → Go to System Settings → Privacy & Security → Full Disk Access and enable Claude.`);
+        lines.push(`       → Go to System Settings → Privacy & Security → Full Disk Access and enable the terminal or runtime hosting MCP Device.`);
         lines.push(`       → To open that pane directly, run in terminal:`);
         lines.push(`           open "x-apple.systempreferences:com.apple.preference.security?Privacy_AllFiles"`);
-        lines.push(`         Then find "Claude" in the list and enable the toggle next to it.`);
+        lines.push(`         Then enable the toggle for the application that launches MCP Device.`);
     }
     else {
         lines.push(`       → Check that the app has permission to access this file location.`);
@@ -265,7 +265,7 @@ export async function validatePath(requestedPath) {
         const pathForNextCheck = resolvedRealPath ?? absoluteOriginal;
         // Direct Gateway device workers are already authenticated and policy-gated by
         // the central gateway. In that dedicated remote context, do not inherit a
-        // workstation's historical Desktop Commander directory allowlist. An optional
+        // workstation's historical directory allowlist. An optional
         // MCP_GATEWAY_ALLOWED_ROOTS restriction is enforced by GatewayToolAdapter before
         // calls reach this filesystem layer.
         const remoteGatewayContext = process.env.MCP_DEVICE_REMOTE === 'true';

package/dist/tools/pdf/markdown.js

@@ -9,7 +9,7 @@ const isUrl = (source) => source.startsWith('http://') || source.startsWith('htt
 let cachedChromePath = null; // null = not checked yet
 let chromeCheckPromise = null;
 /**
- * Get Desktop Commander's private Puppeteer cache directory.
+ * Get the local engine's private Puppeteer cache directory.
  */
 function getPuppeteerCacheDir() {
     return join(dirname(CONFIG_FILE), 'puppeteer-cache');

package/dist/tools/schemas.d.ts

@@ -117,15 +117,15 @@ export declare const WriteFileArgsSchema: z.ZodObject<{
     mode: z.ZodDefault<z.ZodEnum<["rewrite", "append"]>>;
     origin: z.ZodOptional<z.ZodEnum<["ui", "llm"]>>;
 }, "strip", z.ZodTypeAny, {
+    mode: "rewrite" | "append";
     path: string;
     content: string;
-    mode: "rewrite" | "append";
     origin?: "ui" | "llm" | undefined;
 }, {
     path: string;
     content: string;
-    origin?: "ui" | "llm" | undefined;
     mode?: "rewrite" | "append" | undefined;
+    origin?: "ui" | "llm" | undefined;
 }>;
 export declare const PdfInsertOperationSchema: z.ZodObject<{
     type: z.ZodLiteral<"insert">;
@@ -348,7 +348,6 @@ export declare const InteractWithProcessArgsSchema: z.ZodObject<{
     wait_for_prompt?: boolean | undefined;
 }>;
 export declare const GetUsageStatsArgsSchema: z.ZodObject<{}, "strip", z.ZodTypeAny, {}, {}>;
-export declare const GiveFeedbackArgsSchema: z.ZodObject<{}, "strip", z.ZodTypeAny, {}, {}>;
 export declare const StartSearchArgsSchema: z.ZodObject<{
     path: z.ZodString;
     pattern: z.ZodString;

package/dist/tools/schemas.js

@@ -153,15 +153,6 @@ export const InteractWithProcessArgsSchema = z.object({
 });
 // Usage stats schema
 export const GetUsageStatsArgsSchema = z.object({});
-// Feedback tool schema - no pre-filled parameters, all user input
-export const GiveFeedbackArgsSchema = z.object({
-// No parameters needed - form will be filled manually by user
-// Only auto-filled hidden fields remain:
-// - tool_call_count (auto)
-// - days_using (auto) 
-// - platform (auto)
-// - client_id (auto)
-});
 // Search schemas (renamed for natural language)
 export const StartSearchArgsSchema = z.object({
     path: z.string(),
@@ -236,7 +227,6 @@ export const toolArgSchemas = {
     kill_process: KillProcessArgsSchema,
     get_usage_stats: GetUsageStatsArgsSchema,
     get_recent_tool_calls: GetRecentToolCallsArgsSchema,
-    give_feedback_to_desktop_commander: GiveFeedbackArgsSchema,
     get_prompts: GetPromptsArgsSchema,
     track_ui_event: TrackUiEventArgsSchema,
 };

package/dist/ui/config-editor/index.html

@@ -3,7 +3,7 @@
 <head>
   <meta charset="utf-8" />
   <meta name="viewport" content="width=device-width, initial-scale=1" />
-  <title>Desktop Commander Config Editor</title>
+  <title>MCP Device Config Editor</title>
   <link rel="stylesheet" href="./styles.css" />
 </head>
 <body>

package/dist/ui/file-preview/index.html

@@ -6,7 +6,7 @@
 <head>
   <meta charset="utf-8" />
   <meta name="viewport" content="width=device-width, initial-scale=1" />
-  <title>Desktop Commander File Preview</title>
+  <title>MCP Device File Preview</title>
   <link rel="stylesheet" href="./styles.css" />
 </head>
 <body>

package/dist/ui/file-preview/styles.css

@@ -141,7 +141,7 @@ body.dc-ready {
 
 
 /*
- * App-specific styling for File Preview — tuned to blend with Claude's UI.
+ * App-specific styling for File Preview — tuned for the MCP Device UI.
  */
 :root {
   --code-bg: var(--panel);
@@ -166,7 +166,7 @@ body.dc-ready {
   --toc-rail-width: 320px;
 }
 
-/* ── Panel (Claude-style card) ── */
+/* ── Panel (compact card) ── */
 
 .panel {
   margin-top: 6px;

package/dist/utils/capture.d.ts

@@ -1,35 +1,12 @@
 export declare function runInUiOriginCallContext<T>(fn: () => T): T;
 export declare function isInsideUiOriginCall(): boolean;
-/**
- * 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
- * @returns An object with sanitized message and optional error code
- */
 export declare function sanitizeError(error: any): {
     message: string;
     code?: string;
 };
-/**
- * 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: (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 device telemetry
- * Also adds additional privacy filtering to remove sensitive identity information
- * @param event Event name
- * @param properties Optional event properties
- */
-export declare const captureRemote: (event: string, properties?: any) => Promise<void>;
+export declare const captureBase: (_captureURL: string, _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>;
+export declare const captureRemote: (_event: string, _properties?: any) => Promise<void>;