@wonderwhy-er/desktop-commander 0.2.42 → 0.2.43

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

@@ -146,7 +146,7 @@ export function attachDirectoryHandlers(options) {
             loadMoreBtn.querySelector('.dir-warning-text').textContent = 'Loading…';
             loadMoreBtn.disabled = true;
             try {
-                const result = await options.callTool?.('list_directory', { path: loadPath, depth: 1 });
+                const result = await options.callTool?.('list_directory', { path: loadPath, depth: 1, origin: 'ui' });
                 const text = extractToolText(result) ?? '';
                 if (text) {
                     const parsed = parseDirectoryEntries(text);
@@ -194,7 +194,7 @@ export function attachDirectoryHandlers(options) {
             if (chevron)
                 chevron.textContent = '⏳';
             try {
-                const result = await options.callTool?.('list_directory', { path: fullPath, depth: 2 });
+                const result = await options.callTool?.('list_directory', { path: fullPath, depth: 2, origin: 'ui' });
                 const text = extractToolText(result) ?? '';
                 if (text) {
                     target.dataset.loaded = 'true';
@@ -222,7 +222,7 @@ export function attachDirectoryHandlers(options) {
         if (target.classList.contains('dir-row-file')) {
             target.classList.add('dir-loading');
             try {
-                const result = await options.callTool?.('read_file', { path: fullPath });
+                const result = await options.callTool?.('read_file', { path: fullPath, origin: 'ui' });
                 if (!result || typeof result !== 'object' || result === null) {
                     return;
                 }

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

@@ -16,13 +16,13 @@ function renderImageBody(payload) {
             html: '<div class="panel-content source-content"></div>',
         };
     }
-    if (!payload.imageData || payload.imageData.trim().length === 0) {
+    if (!payload.content || payload.content.trim().length === 0) {
         return {
             notice: 'Preview is unavailable because image data is missing.',
             html: '<div class="panel-content source-content"></div>',
         };
     }
-    const src = `data:${mimeType};base64,${payload.imageData}`;
+    const src = `data:${mimeType};base64,${payload.content}`;
     return {
         html: `<div class="panel-content image-content"><div class="image-preview"><img src="${escapeHtml(src)}" alt="${escapeHtml(payload.fileName)}" loading="eager" decoding="async"></div></div>`,
     };

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

@@ -281,6 +281,7 @@ export function createMarkdownController(dependencies) {
         const rawResult = await dependencies.callTool?.('read_file', {
             path: filePath,
             ...(typeof length === 'number' ? { offset: offset ?? 0, length } : {}),
+            origin: 'ui',
         });
         return { rawResult, payload: extractRenderPayload(rawResult) ?? null };
     }
@@ -360,7 +361,7 @@ export function createMarkdownController(dependencies) {
         const markers = new Set(['[DIR] .git', '[DIR] .obsidian', '[FILE] package.json', '[FILE] pnpm-workspace.yaml', '[FILE] turbo.json']);
         for (const ancestor of ancestors) {
             try {
-                const result = await dependencies.callTool?.('list_directory', { path: ancestor, depth: 1 });
+                const result = await dependencies.callTool?.('list_directory', { path: ancestor, depth: 1, origin: 'ui' });
                 const text = extractToolText(result) ?? '';
                 const entries = splitListingLines(text);
                 if (entries.some((entry) => markers.has(entry))) {
@@ -681,6 +682,7 @@ export function createMarkdownController(dependencies) {
                         old_string: block.old_string,
                         new_string: block.new_string,
                         expected_replacements: 1,
+                        origin: 'ui',
                     });
                     assertSuccessfulEditBlockResult(editResult);
                     appliedCount++;

package/dist/ui/file-preview/src/panel-actions.js

@@ -144,8 +144,8 @@ export function attachPanelActions(options) {
         });
         try {
             const readArgs = direction === 'before'
-                ? { path: options.payload.filePath, offset: 0, length: range.fromLine - 1 }
-                : { path: options.payload.filePath, offset: range.toLine };
+                ? { path: options.payload.filePath, offset: 0, length: range.fromLine - 1, origin: 'ui' }
+                : { path: options.payload.filePath, offset: range.toLine, origin: 'ui' };
             const result = await options.callTool?.('read_file', readArgs);
             const newText = extractToolText(result);
             if (newText && typeof newText === 'string') {

package/dist/utils/capture.js

@@ -1,7 +1,7 @@
 import { platform } from 'os';
 import * as https from 'https';
 import { configManager, isTelemetryDisabledValue } from '../config-manager.js';
-import { currentClient, currentCallIsRemote } from '../server.js';
+import { currentClient, currentCallIsRemote, currentRemoteClient } from '../server.js';
 let VERSION = 'unknown';
 try {
     const versionModule = await import('../version.js');
@@ -89,11 +89,14 @@ export const captureBase = async (captureURL, event, properties) => {
             uniqueUserId = await configManager.getOrCreateClientId();
         }
         // Get current client information for all events
+        // For remote calls, attribute to the originating remote client (carried on
+        // the tool call) instead of the device's local currentClient.
+        const effectiveClient = currentCallIsRemote && currentRemoteClient ? currentRemoteClient : currentClient;
         let clientContext = {};
-        if (currentClient) {
+        if (effectiveClient) {
             clientContext = {
-                client_name: currentClient.name,
-                client_version: currentClient.version,
+                client_name: effectiveClient.name,
+                client_version: effectiveClient.version,
             };
         }
         // Track if user saw onboarding page
@@ -265,11 +268,14 @@ const buildEventProperties = async (properties) => {
     if (uniqueUserId === 'unknown') {
         uniqueUserId = await configManager.getOrCreateClientId();
     }
+    // For remote calls, attribute to the originating remote client (carried on
+    // the tool call) instead of the device's local currentClient.
+    const effectiveClient = currentCallIsRemote && currentRemoteClient ? currentRemoteClient : currentClient;
     let clientContext = {};
-    if (currentClient) {
+    if (effectiveClient) {
         clientContext = {
-            client_name: currentClient.name,
-            client_version: currentClient.version,
+            client_name: effectiveClient.name,
+            client_version: effectiveClient.version,
         };
     }
     const sawOnboardingPage = await configManager.getValue('sawOnboardingPage');

package/dist/utils/unsupportedParams.d.ts

@@ -0,0 +1,24 @@
+/**
+ * Utility for detecting parameters a caller sent that a tool's Zod schema does
+ * not accept.
+ *
+ * Zod's default `z.object()` silently strips unknown keys, so a model that
+ * invents a parameter (e.g. `view_range`) gets a normal-looking response with no
+ * signal that its input was ignored. These helpers let the dispatcher surface a
+ * corrective warning instead.
+ *
+ * Detection is top-level only. That is sufficient here because none of the tool
+ * arg schemas use a top-level `.passthrough()`/catchall — free-form fields
+ * (`options`, `params`, `pdfOptions`) are named keys whose contents are
+ * intentionally unvalidated.
+ */
+/** List the parameter names a tool schema accepts (empty array if it takes none). */
+export declare function getSupportedParams(schema: unknown): string[];
+/**
+ * Return the names of top-level keys in `args` that the schema does not accept.
+ * Returns [] when args is not a plain object or the schema's params can't be
+ * determined (so we never warn on something we can't reason about).
+ */
+export declare function detectUnsupportedParams(args: unknown, schema: unknown): string[];
+/** Build the corrective warning string shown to the model. */
+export declare function buildUnsupportedParamsWarning(toolName: string, unsupported: string[], supported: string[]): string;

package/dist/utils/unsupportedParams.js

@@ -0,0 +1,66 @@
+/**
+ * Utility for detecting parameters a caller sent that a tool's Zod schema does
+ * not accept.
+ *
+ * Zod's default `z.object()` silently strips unknown keys, so a model that
+ * invents a parameter (e.g. `view_range`) gets a normal-looking response with no
+ * signal that its input was ignored. These helpers let the dispatcher surface a
+ * corrective warning instead.
+ *
+ * Detection is top-level only. That is sufficient here because none of the tool
+ * arg schemas use a top-level `.passthrough()`/catchall — free-form fields
+ * (`options`, `params`, `pdfOptions`) are named keys whose contents are
+ * intentionally unvalidated.
+ */
+/**
+ * Resolve a Zod schema down to the field map of its underlying object schema,
+ * unwrapping wrappers (effects/optional/default/nullable/branded) defensively.
+ * Returns null when the schema is not (and does not wrap) a plain object schema,
+ * in which case we cannot determine the supported parameters and must not warn.
+ */
+function getObjectShape(schema) {
+    let current = schema;
+    for (let i = 0; current && current._def && i < 10; i++) {
+        const typeName = current._def.typeName;
+        if (typeName === 'ZodObject') {
+            const shape = typeof current.shape === 'function' ? current.shape() : current.shape;
+            return shape ?? null;
+        }
+        if (typeName === 'ZodEffects')
+            current = current._def.schema;
+        else if (typeName === 'ZodOptional' || typeName === 'ZodNullable' || typeName === 'ZodDefault') {
+            current = typeof current._def.innerType === 'function' ? current._def.innerType() : current._def.innerType;
+        }
+        else if (typeName === 'ZodBranded')
+            current = current._def.type;
+        else
+            return null;
+    }
+    return null;
+}
+/** List the parameter names a tool schema accepts (empty array if it takes none). */
+export function getSupportedParams(schema) {
+    const shape = getObjectShape(schema);
+    return shape ? Object.keys(shape) : [];
+}
+/**
+ * Return the names of top-level keys in `args` that the schema does not accept.
+ * Returns [] when args is not a plain object or the schema's params can't be
+ * determined (so we never warn on something we can't reason about).
+ */
+export function detectUnsupportedParams(args, schema) {
+    if (!args || typeof args !== 'object' || Array.isArray(args))
+        return [];
+    const shape = getObjectShape(schema);
+    if (shape === null)
+        return [];
+    const supported = new Set(Object.keys(shape));
+    return Object.keys(args).filter((k) => !supported.has(k));
+}
+/** Build the corrective warning string shown to the model. */
+export function buildUnsupportedParamsWarning(toolName, unsupported, supported) {
+    const ignored = unsupported.join(', ');
+    const supportedList = supported.length > 0 ? supported.join(', ') : '(none)';
+    return `You sent parameters not supported by this tool, which were ignored: ${ignored}. `
+        + `Supported parameters for ${toolName}: ${supportedList}.`;
+}

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.2.42";
+export declare const VERSION = "0.2.43";

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = '0.2.42';
+export const VERSION = '0.2.43';

package/package.json

@@ -1,11 +1,15 @@
 {
   "name": "@wonderwhy-er/desktop-commander",
-  "version": "0.2.42",
+  "version": "0.2.43",
   "description": "MCP server for terminal operations and file editing",
   "mcpName": "io.github.wonderwhy-er/desktop-commander",
   "license": "MIT",
   "author": "Eduards Ruzga",
   "homepage": "https://github.com/wonderwhy-er/DesktopCommanderMCP",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/wonderwhy-er/DesktopCommanderMCP.git"
+  },
   "bugs": "https://github.com/wonderwhy-er/DesktopCommanderMCP/issues",
   "type": "module",
   "engines": {