dsh-mcp 1.6.0 → 1.8.0

package/CHANGELOG.en.md

@@ -7,6 +7,36 @@ All notable changes to this project are documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+
+## [1.8.0] - 2026-08-27
+
+### Added
+
+- **OAuth authorization UX**: mounts no longer open the browser (they fail with guidance when authorization is needed; only Test connection auto-opens the browser); tool calls with a missing/expired token return a **clickable authorization link** (a background callback listener on the stable port completes the flow; concurrent calls reuse the same pending flow); a global authorization queue keeps at most one flow active at a time
+- **Mount failure now carries an authorization link**: when a mount needs OAuth authorization, the failure message includes a **clickable authorization link**; once the user opens it, the tokens are stored automatically — no manual trip to the settings page
+- **New `allowBrowserOnMount` config**: `false` by default (mounts never pop the browser); set it to `true` to restore the legacy behavior (mounts open the browser for authorization). Configure it under the dsh-mcp entry in the profile's `cordis.patch.yml`
+- **stdio form argument input**: arguments now parse shell-style (space/newline separated, with quote, escape, and explicit empty-argument support) so a command line pastes directly; the args box shows a live parsed-argument preview so split mistakes surface before saving
+
+### Fixed
+
+- **OAuth authorization link was dropped from the mount failure message**: the underlying connect error (including the authorization link) used to be folded into `cause`, while the mount failure view only showed the message; the detail is now folded into the message
+- **Tools did not register automatically after OAuth authorization**: saving tokens now remounts every enabled server of that name by serverName, so tools appear without a manual refresh or restart (the listener previously subscribed to a credential event name the service never dispatches, so the remount never ran; fixed)
+- **Callback-port conflict crashed the Host**: all authorization flows now share one deduplicated per-server loopback listener and every listen has an error handler — an `EADDRINUSE` can no longer crash the DSH process
+- **Wrong `resource` parameter in the authorization link**: the link used to serialize `resource=undefined`, which the authorization server rejects; it now passes a URL object (preferring the protected-resource metadata's resource)
+- **stdio working-directory pitfall**: the form now explains that an empty cwd inherits the Host working directory and that a pnpm workspace there can make npx and similar commands resolve the wrong local package; the hint carries no concrete path, leaving the value to the user
+
+## [1.7.0] - 2026-08-21
+
+### Added
+
+- **JSON config editor switched to key-value format**: server configs are shown/edited as a JSON object keyed by server name (`{ "server": { "type": "streamable_http", "url": ..., "headers": {...}, "disabled": false } }`) instead of the previous array; `type` is `streamable_http` or `stdio`, `disabled: true` disables the server
+
+### Fixed
+
+- **MCP image admission diagnostics no longer misreport failures**: distinguish image count, batch/per-image byte, MIME, Base64, raster format, decoded-pixel, and maximum-dimension limits; unknown admission errors use a fixed diagnostic so valid-but-oversized images are not reported as invalid image data and attachment storage internals are not leaked (PR #5, thanks @coding-chong)
+- The publish workflow now runs `npm test` (image-projection regression) before syntax checks and publishing
+
 ## [1.6.0] - 2026-08-17
 
 ### Added

package/CHANGELOG.md

@@ -7,6 +7,36 @@
 格式基于 [Keep a Changelog](https://keepachangelog.com/zh-CN/1.1.0/)，
 版本号遵循 [语义化版本](https://semver.org/lang/zh-CN/)。
 
+## [Unreleased]
+
+## [1.8.0] - 2026-08-27
+
+### 新增
+
+- **OAuth 授权交互改进**：启动挂载不再自动弹浏览器（需授权时失败并提示，仅测试连接自动弹浏览器）；工具调用发现 token 缺失/过期时返回**可点击的授权链接**（后台回调监听器在稳定端口完成授权，并发请求复用同一流程）；全局授权队列保证同一时间最多一个授权流程
+- **挂载失败直接附授权链接**：挂载需要授权时，失败信息直接附带**可点击的授权链接**，打开链接完成授权后 token 自动保存，无需再手动去设置页操作
+- **新增 `allowBrowserOnMount` 配置**：默认 `false`（挂载不自动弹浏览器）；设为 `true` 可恢复旧行为（挂载时自动打开浏览器授权）。在 profile 的 `cordis.patch.yml` 中 dsh-mcp 条目下配置
+- **stdio 表单参数输入改进**：参数支持 shell 风格解析（空格/换行分隔，支持引号、转义与显式空参数），命令行可直接粘贴即用；参数框下方实时预览解析结果，保存前即可发现参数拆分问题
+
+### 修复
+
+- **OAuth 授权链接未在挂载失败信息中展示**：底层连接错误（含授权链接）此前被塞进 `cause`，挂载失败视图只显示通用文案；现已将详情并入 message
+- **OAuth 授权完成后工具不自动注册**：授权保存 token 后按 serverName 自动重挂载对应服务器，工具立即注册，无需手动刷新或重启（此前监听的事件名与凭据服务实际发出的事件不一致，自动重挂载一直未生效，现已修正）
+- **授权回调端口冲突导致进程崩溃**：统一按服务器共享回调监听器（同服务器去重），并为所有 listen 增加错误处理——`EADDRINUSE` 不再导致 DSH 进程崩溃
+- **授权链接 resource 参数错误**：链接中 `resource=undefined` 会导致授权服务器校验失败；改为传 URL 对象（优先使用 protected-resource 元数据的 resource）
+- **stdio 工作目录留空时易踩坑**：表单为 cwd 补充说明——留空会继承 Host 工作目录，若该目录是 pnpm workspace，npx 等命令可能解析到错误的本地包；提示不含具体路径，由使用者自行填写
+
+## [1.7.0] - 2026-08-21
+
+### 新增
+
+- **JSON 配置编辑器改为 key-value 格式**：服务器配置以「服务器名 → 配置对象」的 JSON 对象展示/编辑（`{ "服务器名": { "type": "streamable_http", "url": ..., "headers": {...}, "disabled": false } }`），替代原数组格式；`type` 取值 `streamable_http`/`stdio`，`disabled: true` 表示停用
+
+### 修复
+
+- **MCP 图片 admission 诊断不再误报**：区分图片数量、批量/单图字节数、MIME 类型、Base64、图片格式、解码像素数和最大边长限制；未知 admission 错误使用固定诊断，避免把有效但超尺寸的图片误报为无效图片数据，也不泄露 attachment 存储内部错误（PR #5，感谢 @coding-chong）
+- 发布流水线增加 `npm test`（图片投影回归测试）后再校验语法并发布
+
 ## [1.6.0] - 2026-08-17
 
 ### 新增

package/README.en.md

@@ -153,7 +153,7 @@ Check in order:
    the `web` profile (`dsh plugin --profile web add dsh-mcp` +
    `$DSH_HOME/profiles/web/cordis.patch.yml`); other profiles have their own settings pages.
 5. **Is it the latest version?** npm metadata caching can pin an old version; force the version
-   with `dsh plugin --profile web add dsh-mcp@latest` (or `@1.6.0`).
+   with `dsh plugin --profile web add dsh-mcp@latest` (or `@1.8.0`).
 
 **Q2: "MCP" is visible but the server list is empty or errors?**
 

package/README.md

@@ -136,7 +136,7 @@ dsh plugin --profile web add link:<本仓库绝对路径>
    （`dsh plugin --profile web add dsh-mcp` + `$DSH_HOME/profiles/web/cordis.patch.yml`）；
    装到其他 profile 则在其他 profile 的设置页查看。
 5. **是否为最新版本**：npm 元数据缓存可能导致装到旧版，可强制指定版本
-   `dsh plugin --profile web add dsh-mcp@latest`（或 `@1.6.0`）。
+   `dsh plugin --profile web add dsh-mcp@latest`（或 `@1.8.0`）。
 
 **Q2：设置页能看到「MCP」，但服务器列表为空/报错？**
 

package/lib/client.js

@@ -15041,6 +15041,175 @@ var import_react4 = require("react");
 // src/client/ServerForm.tsx
 var import_react = require("react");
 
+// src/client/mcp-store.ts
+var import_client = require("@deepseek-ai/dsh-client-runtime/client");
+var DEFAULT_TOOL_CALL_TIMEOUT_MS = 6e4;
+function emptyDraft() {
+  return {
+    id: null,
+    serverName: "",
+    transport: "stdio",
+    enabled: true,
+    command: "",
+    argsText: "",
+    cwd: "",
+    url: "",
+    headersText: "",
+    toolCallTimeoutMs: String(DEFAULT_TOOL_CALL_TIMEOUT_MS),
+    // Reject the mount by default when the startup connection fails, so a
+    // broken server never registers stale tools.
+    failOnStartupError: true
+  };
+}
+function draftFromServer(server) {
+  return {
+    id: server.id,
+    serverName: server.serverName,
+    transport: server.transport,
+    enabled: server.enabled,
+    command: server.command,
+    argsText: server.args.join("\n"),
+    cwd: server.cwd,
+    url: server.url,
+    headersText: server.headers.map((header) => `${header.name}: ${header.value}`).join("\n"),
+    toolCallTimeoutMs: String(server.toolCallTimeoutMs),
+    failOnStartupError: server.failOnStartupError
+  };
+}
+function parseLines(text) {
+  return text.split("\n").map((line) => line.trim()).filter((line) => line.length > 0);
+}
+function parseArgs(text) {
+  const args = [];
+  let current = "";
+  let quote = null;
+  let hasToken = false;
+  let i = 0;
+  while (i < text.length) {
+    const ch = text[i];
+    if (quote !== null) {
+      if (ch === quote) {
+        quote = null;
+        i += 1;
+        continue;
+      }
+      if (ch === "\\" && quote === '"' && i + 1 < text.length && (text[i + 1] === '"' || text[i + 1] === "\\")) {
+        current += text[i + 1];
+        i += 2;
+        continue;
+      }
+      current += ch;
+      i += 1;
+      continue;
+    }
+    if (ch === '"' || ch === "'") {
+      quote = ch;
+      hasToken = true;
+      i += 1;
+      continue;
+    }
+    if (ch === "\\" && i + 1 < text.length) {
+      current += text[i + 1];
+      hasToken = true;
+      i += 2;
+      continue;
+    }
+    if (ch === " " || ch === "	" || ch === "\n" || ch === "\r") {
+      if (hasToken || current.length > 0) {
+        args.push(current);
+        current = "";
+        hasToken = false;
+      }
+      i += 1;
+      continue;
+    }
+    current += ch;
+    hasToken = true;
+    i += 1;
+  }
+  if (quote !== null || hasToken || current.length > 0) {
+    args.push(current);
+  }
+  return args;
+}
+function parseHeaders(text) {
+  const headers = [];
+  for (const line of parseLines(text)) {
+    const separator = line.indexOf(":");
+    if (separator <= 0) continue;
+    const name = line.slice(0, separator).trim();
+    const value = line.slice(separator + 1).trim();
+    if (name.length > 0) headers.push({ name, value });
+  }
+  return headers;
+}
+function parseTimeout(text) {
+  const value = Number(text);
+  return Number.isInteger(value) && value > 0 ? value : DEFAULT_TOOL_CALL_TIMEOUT_MS;
+}
+function draftToSubmission(draft) {
+  return {
+    server: {
+      serverName: draft.serverName.trim(),
+      transport: draft.transport,
+      enabled: draft.enabled,
+      command: draft.command.trim(),
+      args: parseArgs(draft.argsText),
+      cwd: draft.cwd.trim(),
+      url: draft.url.trim(),
+      headers: parseHeaders(draft.headersText),
+      toolCallTimeoutMs: parseTimeout(draft.toolCallTimeoutMs),
+      failOnStartupError: draft.failOnStartupError
+    }
+  };
+}
+function createMcpManagerStore() {
+  return (0, import_client.defineStore)({
+    init: () => ({
+      loadState: "loading",
+      servers: [],
+      draft: null,
+      busy: null,
+      testRunning: false,
+      test: null
+    }),
+    actions: {
+      setLoadState: (d, state) => {
+        d.loadState = state;
+      },
+      setServers: (d, servers) => {
+        d.servers = servers;
+      },
+      beginCreate: (d) => {
+        d.draft = emptyDraft();
+        d.test = null;
+      },
+      beginEdit: (d, server) => {
+        d.draft = draftFromServer(server);
+        d.test = null;
+      },
+      cancelEdit: (d) => {
+        d.draft = null;
+        d.test = null;
+        d.busy = null;
+      },
+      updateDraft: (d, patch) => {
+        if (d.draft === null) return;
+        d.draft = { ...d.draft, ...patch };
+      },
+      setBusy: (d, busy) => {
+        d.busy = busy;
+      },
+      setTestRunning: (d, running) => {
+        d.testRunning = running;
+      },
+      setTest: (d, test) => {
+        d.test = test;
+      }
+    }
+  });
+}
+
 // src/client/ServerForm.module.css
 (function() {
   if (typeof document !== "undefined") {
@@ -15185,12 +15354,24 @@ function McpServerForm(props) {
         /* @__PURE__ */ (0, import_jsx_runtime.jsx)(
           "textarea",
           {
-            rows: 2,
+            rows: 3,
             value: draft.argsText,
             placeholder: t("argsPlaceholder"),
             onChange: (event) => update({ argsText: event.currentTarget.value })
           }
-        )
+        ),
+        (() => {
+          const parsed = parseArgs(draft.argsText);
+          if (parsed.length === 0) return /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { className: ServerForm_default.muted, children: t("argsPreviewEmpty") });
+          const shown = parsed.map((arg) => arg.length > 48 ? `${arg.slice(0, 45)}\u2026` : arg).join(" | ");
+          return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: ServerForm_default.muted, children: [
+            t("argsPreview"),
+            ": ",
+            parsed.length,
+            " \xB7 ",
+            shown
+          ] });
+        })()
       ] }),
       /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("label", { className: ServerForm_default.field, children: [
         /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { children: t("cwd") }),
@@ -15202,7 +15383,8 @@ function McpServerForm(props) {
             placeholder: t("cwdPlaceholder"),
             onChange: (event) => update({ cwd: event.currentTarget.value })
           }
-        )
+        ),
+        /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { className: ServerForm_default.muted, children: t("cwdHint") })
       ] })
     ] }) : /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(import_jsx_runtime.Fragment, { children: [
       /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("label", { className: ServerForm_default.field, children: [
@@ -15295,11 +15477,11 @@ var ServersJsonEditor_default = { "panel": "panel", "editor": "editor", "hint":
 // src/client/ServersJsonEditor.tsx
 var import_jsx_runtime2 = require("react/jsx-runtime");
 function serversToJsonText(servers) {
-  const document2 = servers.map((server) => {
+  const document2 = {};
+  for (const server of servers) {
     const entry = {
-      serverName: server.serverName,
-      transport: server.transport,
-      enabled: server.enabled
+      type: server.transport === "stdio" ? "stdio" : "streamable_http",
+      disabled: !server.enabled
     };
     if (server.transport === "stdio") {
       entry.command = server.command;
@@ -15314,8 +15496,8 @@ function serversToJsonText(servers) {
     entry.toolCallTimeoutMs = server.toolCallTimeoutMs;
     entry.failOnStartupError = server.failOnStartupError;
     entry.env = server.env.map((row) => row.secret ? { name: row.name, secret: true, configured: row.configured } : { name: row.name, secret: false, ...row.value !== void 0 && row.value.length > 0 ? { value: row.value } : {} });
-    return entry;
-  });
+    document2[server.serverName] = entry;
+  }
   return `${JSON.stringify(document2, null, 2)}
 `;
 }
@@ -15326,39 +15508,37 @@ function parseServersJson(text) {
   } catch (error51) {
     return { ok: false, error: error51 instanceof Error ? error51.message : String(error51) };
   }
-  if (!Array.isArray(value)) {
-    return { ok: false, error: "JSON \u9876\u5C42\u5FC5\u987B\u662F\u670D\u52A1\u5668\u914D\u7F6E\u6570\u7EC4\uFF1A[ { serverName, transport, ... }, ... ]" };
+  if (value === null || typeof value !== "object" || Array.isArray(value)) {
+    return { ok: false, error: 'JSON \u9876\u5C42\u5FC5\u987B\u662F\u5BF9\u8C61\uFF1A{ "\u670D\u52A1\u5668\u540D": { "type": "streamable_http", ... } }' };
   }
-  if (value.length === 0) {
+  const entries = Object.entries(value);
+  if (entries.length === 0) {
     return { ok: false, error: "\u914D\u7F6E\u5217\u8868\u4E0D\u80FD\u4E3A\u7A7A" };
   }
   const servers = [];
-  const seen = /* @__PURE__ */ new Set();
-  for (const item of value) {
-    if (item === null || typeof item !== "object" || Array.isArray(item)) {
-      return { ok: false, error: "\u6BCF\u4E2A\u6761\u76EE\u5FC5\u987B\u662F\u5BF9\u8C61\uFF1A{ serverName, transport, ... }" };
-    }
-    const server = item;
-    const serverName = typeof server.serverName === "string" ? server.serverName.trim() : "";
-    if (serverName.length === 0) return { ok: false, error: "serverName \u4E0D\u80FD\u4E3A\u7A7A" };
-    if (seen.has(serverName)) return { ok: false, error: `serverName "${serverName}" \u91CD\u590D` };
-    seen.add(serverName);
-    const transport = server.transport;
-    if (transport !== "stdio" && transport !== "streamable-http") {
-      return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}" \u7684 transport \u5FC5\u987B\u662F stdio \u6216 streamable-http` };
-    }
-    if (transport === "stdio" && (typeof server.command !== "string" || server.command.trim().length === 0)) {
+  for (const [serverName, raw] of entries) {
+    if (serverName.trim().length === 0) return { ok: false, error: "\u670D\u52A1\u5668\u540D\u4E0D\u80FD\u4E3A\u7A7A" };
+    if (raw === null || typeof raw !== "object" || Array.isArray(raw)) {
+      return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}" \u7684\u914D\u7F6E\u5FC5\u987B\u662F\u5BF9\u8C61` };
+    }
+    const cfg = raw;
+    const type = cfg.type;
+    const transport = type === "stdio" ? "stdio" : type === "streamable_http" ? "streamable-http" : void 0;
+    if (transport === void 0) {
+      return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}" \u7684 type \u5FC5\u987B\u662F streamable_http \u6216 stdio` };
+    }
+    if (transport === "stdio" && (typeof cfg.command !== "string" || cfg.command.trim().length === 0)) {
       return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}"\uFF08stdio\uFF09\u7F3A\u5C11 command` };
     }
-    if (transport === "streamable-http" && typeof server.url !== "string") {
-      return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}"\uFF08streamable-http\uFF09\u7F3A\u5C11 url` };
+    if (transport === "streamable-http" && typeof cfg.url !== "string") {
+      return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}"\uFF08streamable_http\uFF09\u7F3A\u5C11 url` };
     }
-    if (server.headers !== void 0) {
-      const headers = server.headers;
+    if (cfg.headers !== void 0) {
+      const headers = cfg.headers;
       const okHeaders = headers !== null && typeof headers === "object" && !Array.isArray(headers) || Array.isArray(headers) && headers.every((h) => h !== null && typeof h === "object" && typeof h.name === "string");
       if (!okHeaders) return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}" \u7684 headers \u5FC5\u987B\u662F { "\u540D\u79F0": "\u503C" } \u5BF9\u8C61\u6216 { name, value } \u6570\u7EC4` };
     }
-    const envValue = server.env;
+    const envValue = cfg.env;
     const env = Array.isArray(envValue) ? envValue : [];
     for (const row of env) {
       const r = row;
@@ -15372,6 +15552,18 @@ function parseServersJson(text) {
         return { ok: false, error: `\u670D\u52A1\u5668 "${serverName}" \u7684 env \u6761\u76EE "${r.name}" \u7684 value \u5FC5\u987B\u662F\u5B57\u7B26\u4E32` };
       }
     }
+    const server = {
+      serverName,
+      transport,
+      enabled: cfg.disabled !== true,
+      command: typeof cfg.command === "string" ? cfg.command : "",
+      args: Array.isArray(cfg.args) ? cfg.args : [],
+      cwd: typeof cfg.cwd === "string" ? cfg.cwd : "",
+      url: typeof cfg.url === "string" ? cfg.url : "",
+      headers: cfg.headers,
+      toolCallTimeoutMs: typeof cfg.toolCallTimeoutMs === "number" ? cfg.toolCallTimeoutMs : 6e4,
+      failOnStartupError: cfg.failOnStartupError !== false
+    };
     servers.push({ server, env });
   }
   return { ok: true, servers };
@@ -16075,9 +16267,12 @@ var zh = {
   command: "\u547D\u4EE4",
   commandPlaceholder: "\u4F8B\u5982 npx",
   args: "\u53C2\u6570",
-  argsPlaceholder: "\u6BCF\u884C\u4E00\u4E2A\u53C2\u6570",
+  argsPlaceholder: "\u7A7A\u683C/\u6362\u884C\u5206\u9694\uFF0C\u652F\u6301\u5F15\u53F7\uFF0C\u5982\uFF1A-y @modelcontextprotocol/server-filesystem /path/to/dir",
+  argsPreview: "\u53C2\u6570\u9884\u89C8",
+  argsPreviewEmpty: "\uFF08\u65E0\u53C2\u6570\uFF09",
   cwd: "\u5DE5\u4F5C\u76EE\u5F55",
   cwdPlaceholder: "\u7559\u7A7A\u7EE7\u627F Host \u5DE5\u4F5C\u76EE\u5F55",
+  cwdHint: "\u7559\u7A7A\u4F1A\u7EE7\u627F DSH \u7684\u5DE5\u4F5C\u76EE\u5F55\u3002\u82E5\u8BE5\u76EE\u5F55\u662F pnpm workspace\uFF0Cnpx \u7B49\u547D\u4EE4\u53EF\u80FD\u89E3\u6790\u5230\u9519\u8BEF\u7684\u672C\u5730\u5305\uFF1B\u5EFA\u8BAE\u586B\u4E00\u4E2A\u666E\u901A\u76EE\u5F55\u7684\u7EDD\u5BF9\u8DEF\u5F84\u3002",
   url: "\u670D\u52A1\u5668 URL",
   urlPlaceholder: "\u4F8B\u5982 http://localhost:3000/mcp",
   headers: "\u8BF7\u6C42\u5934",
@@ -16132,7 +16327,7 @@ var zh = {
   toolsCollapse: "\u6536\u8D77\u5DE5\u5177",
   serversJson: "JSON \u7EF4\u62A4\u914D\u7F6E",
   serversJsonLabel: "\u670D\u52A1\u5668\u914D\u7F6E JSON",
-  serversJsonHint: "\u6574\u4E2A MCP \u670D\u52A1\u5668\u914D\u7F6E\u5217\u8868\uFF08\u6570\u7EC4\uFF09\uFF1A\u5E94\u7528\u540E\u5C06\u6309\u6B64\u5217\u8868\u66FF\u6362\u2014\u2014\u5DF2\u5217\u51FA\u7684\u670D\u52A1\u5668\u521B\u5EFA/\u66F4\u65B0\uFF0C\u672A\u5217\u51FA\u7684\u5C06\u88AB\u5220\u9664\uFF1Bsecret \u73AF\u5883\u53D8\u91CF\u503C\u4E0D\u4F1A\u5BFC\u51FA\uFF08configured \u6807\u8BB0\uFF09\uFF0C\u7559\u7A7A\u5219\u4FDD\u7559\u539F\u503C\u3002",
+  serversJsonHint: "\u6574\u4E2A MCP \u670D\u52A1\u5668\u914D\u7F6E\uFF08key-value \u5BF9\u8C61\uFF0C\u670D\u52A1\u5668\u540D \u2192 \u914D\u7F6E\uFF09\uFF1A\u5E94\u7528\u540E\u5C06\u6309\u6B64\u66FF\u6362\u2014\u2014\u5DF2\u5217\u51FA\u7684\u670D\u52A1\u5668\u521B\u5EFA/\u66F4\u65B0\uFF0C\u672A\u5217\u51FA\u7684\u5C06\u88AB\u5220\u9664\uFF1Btype \u4E3A streamable_http \u6216 stdio\uFF0Cdisabled \u4E3A true \u8868\u793A\u505C\u7528\uFF1Bsecret \u73AF\u5883\u53D8\u91CF\u503C\u4E0D\u4F1A\u5BFC\u51FA\uFF08configured \u6807\u8BB0\uFF09\uFF0C\u7559\u7A7A\u5219\u4FDD\u7559\u539F\u503C\u3002",
   serversJsonReseed: "\u91CD\u65B0\u5BFC\u51FA",
   serversJsonApply: "\u5E94\u7528",
   serversJsonSaving: "\u5E94\u7528\u4E2D\u2026",
@@ -16173,9 +16368,12 @@ var en = {
   command: "Command",
   commandPlaceholder: "e.g. npx",
   args: "Arguments",
-  argsPlaceholder: "One argument per line",
+  argsPlaceholder: "Space/newline separated, quotes supported, e.g. -y @modelcontextprotocol/server-filesystem /path/to/dir",
+  argsPreview: "Argument preview",
+  argsPreviewEmpty: "(no arguments)",
   cwd: "Working directory",
   cwdPlaceholder: "Leave empty to inherit the Host working directory",
+  cwdHint: "An empty value inherits the DSH working directory. If that directory is a pnpm workspace, npx and similar commands may resolve to a wrong local package; prefer an absolute path to a plain directory.",
   url: "Server URL",
   urlPlaceholder: "e.g. http://localhost:3000/mcp",
   headers: "Headers",
@@ -16230,7 +16428,7 @@ var en = {
   toolsCollapse: "Collapse tools",
   serversJson: "JSON config editor",
   serversJsonLabel: "Server-config JSON",
-  serversJsonHint: "The whole MCP server list (array): applying replaces the list \u2014 listed servers are created/updated, existing servers absent from the document are removed; secret env values are never exported (a configured flag marks them), a blank value keeps the stored one.",
+  serversJsonHint: "The whole MCP server config as a key-value object (server name \u2192 config): applying replaces the list \u2014 listed servers are created/updated, servers absent from the document are removed; type is streamable_http or stdio, disabled: true disables the server; secret env values are never exported (a configured flag marks them), a blank value keeps the stored one.",
   serversJsonReseed: "Re-export",
   serversJsonApply: "Apply",
   serversJsonSaving: "Applying\u2026",
@@ -16255,122 +16453,6 @@ var en = {
   configTitle: "MCP config"
 };
 
-// src/client/mcp-store.ts
-var import_client = require("@deepseek-ai/dsh-client-runtime/client");
-var DEFAULT_TOOL_CALL_TIMEOUT_MS = 6e4;
-function emptyDraft() {
-  return {
-    id: null,
-    serverName: "",
-    transport: "stdio",
-    enabled: true,
-    command: "",
-    argsText: "",
-    cwd: "",
-    url: "",
-    headersText: "",
-    toolCallTimeoutMs: String(DEFAULT_TOOL_CALL_TIMEOUT_MS),
-    // Reject the mount by default when the startup connection fails, so a
-    // broken server never registers stale tools.
-    failOnStartupError: true
-  };
-}
-function draftFromServer(server) {
-  return {
-    id: server.id,
-    serverName: server.serverName,
-    transport: server.transport,
-    enabled: server.enabled,
-    command: server.command,
-    argsText: server.args.join("\n"),
-    cwd: server.cwd,
-    url: server.url,
-    headersText: server.headers.map((header) => `${header.name}: ${header.value}`).join("\n"),
-    toolCallTimeoutMs: String(server.toolCallTimeoutMs),
-    failOnStartupError: server.failOnStartupError
-  };
-}
-function parseLines(text) {
-  return text.split("\n").map((line) => line.trim()).filter((line) => line.length > 0);
-}
-function parseHeaders(text) {
-  const headers = [];
-  for (const line of parseLines(text)) {
-    const separator = line.indexOf(":");
-    if (separator <= 0) continue;
-    const name = line.slice(0, separator).trim();
-    const value = line.slice(separator + 1).trim();
-    if (name.length > 0) headers.push({ name, value });
-  }
-  return headers;
-}
-function parseTimeout(text) {
-  const value = Number(text);
-  return Number.isInteger(value) && value > 0 ? value : DEFAULT_TOOL_CALL_TIMEOUT_MS;
-}
-function draftToSubmission(draft) {
-  return {
-    server: {
-      serverName: draft.serverName.trim(),
-      transport: draft.transport,
-      enabled: draft.enabled,
-      command: draft.command.trim(),
-      args: parseLines(draft.argsText),
-      cwd: draft.cwd.trim(),
-      url: draft.url.trim(),
-      headers: parseHeaders(draft.headersText),
-      toolCallTimeoutMs: parseTimeout(draft.toolCallTimeoutMs),
-      failOnStartupError: draft.failOnStartupError
-    }
-  };
-}
-function createMcpManagerStore() {
-  return (0, import_client.defineStore)({
-    init: () => ({
-      loadState: "loading",
-      servers: [],
-      draft: null,
-      busy: null,
-      testRunning: false,
-      test: null
-    }),
-    actions: {
-      setLoadState: (d, state) => {
-        d.loadState = state;
-      },
-      setServers: (d, servers) => {
-        d.servers = servers;
-      },
-      beginCreate: (d) => {
-        d.draft = emptyDraft();
-        d.test = null;
-      },
-      beginEdit: (d, server) => {
-        d.draft = draftFromServer(server);
-        d.test = null;
-      },
-      cancelEdit: (d) => {
-        d.draft = null;
-        d.test = null;
-        d.busy = null;
-      },
-      updateDraft: (d, patch) => {
-        if (d.draft === null) return;
-        d.draft = { ...d.draft, ...patch };
-      },
-      setBusy: (d, busy) => {
-        d.busy = busy;
-      },
-      setTestRunning: (d, running) => {
-        d.testRunning = running;
-      },
-      setTest: (d, test) => {
-        d.test = test;
-      }
-    }
-  });
-}
-
 // src/client/index.ts
 var NS = "settings.mcp";
 var inject = ["slots", "locale", "remote"];