dsh-mcp 1.2.0 → 1.3.0

package/CHANGELOG.en.md

@@ -7,6 +7,18 @@ 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).
 
+## [1.3.0] - 2026-08-16
+
+### Added
+
+- **Windows working-directory support**: stdio servers now accept drive-letter absolute paths for `cwd` (e.g. `C:\Users\...`, `C:/...`), consistent with POSIX `/` and UNC `\\` paths (PR #2, thanks @coding-chong)
+
+### Fixed
+
+- Form operations now surface the real error: save / delete / test-connection failures show `code: message` (e.g. `MCP_SERVER_NAME_CONFLICT: serverName "x" is already used...`) instead of a generic message, making failures diagnosable
+- Refresh is decoupled from save/delete: a `refresh()` failure no longer misreports the save/delete outcome — the editor stays open and shows the refresh failure reason (`refresh()` keeps its try/catch and returns a result)
+- Removed the now-unused `failureLocaleKey` dead code (error display shows `code: message` directly)
+
 ## [1.2.0] - 2026-08-16
 
 ### Added

package/CHANGELOG.md

@@ -7,6 +7,18 @@
 格式基于 [Keep a Changelog](https://keepachangelog.com/zh-CN/1.1.0/)，
 版本号遵循 [语义化版本](https://semver.org/lang/zh-CN/)。
 
+## [1.3.0] - 2026-08-16
+
+### 新增
+
+- **Windows 工作目录支持**：stdio 服务器的 cwd 接受 Windows 盘符绝对路径（如 `C:\Users\...`、`C:/...`），与 POSIX `/`、UNC `\\` 路径一致（PR #2，感谢 @coding-chong）
+
+### 修复
+
+- 表单操作失败时展示真实错误信息：保存 / 删除 / 测试连接失败不再只显示笼统文案，直接展示 `code: message`（如 `MCP_SERVER_NAME_CONFLICT: serverName "x" is already used...`），便于定位问题
+- 刷新与保存/删除解耦：`refresh()` 失败不再误报保存结果，停留编辑页并显示刷新失败原因（`refresh()` 保留 try/catch 并返回结果）
+- 清理死代码：移除已无引用的 `failureLocaleKey`（错误展示改为直接显示 `code: message`）
+
 ## [1.2.0] - 2026-08-16
 
 ### 新增

package/lib/client.js

@@ -14971,20 +14971,6 @@ function draftToSubmission(draft) {
     }))
   };
 }
-function failureLocaleKey(code) {
-  switch (code) {
-    case "MCP_SERVER_NAME_CONFLICT":
-      return "serverNameConflict";
-    case "MCP_SERVER_NOT_FOUND":
-      return "notFound";
-    case "MCP_INVALID_SPEC":
-      return "invalidSpec";
-    case "MCP_MOUNT_FAILED":
-      return "mountFailed";
-    default:
-      return "failureTitle";
-  }
-}
 function createMcpManagerStore() {
   return (0, import_client.defineStore)({
     init: () => ({
@@ -15065,13 +15051,16 @@ function McpServerForm(props) {
   const addEnvRow = () => {
     update({ env: [...draft.env, createEnvRowDraft()] });
   };
+  const errorText = (error51) => error51 instanceof Error ? error51.message : String(error51);
   const runTest = async () => {
     setSaveError(null);
     actions.setTestRunning(true);
     try {
       actions.setTest(await injected.test(draft));
-    } catch {
+    } catch (error51) {
+      console.error("[dsh-mcp] test failed:", error51);
       actions.setTest(null);
+      setSaveError(errorText(error51));
     } finally {
       actions.setTestRunning(false);
     }
@@ -15082,14 +15071,15 @@ function McpServerForm(props) {
     try {
       const failure = await injected.save(draft);
       if (failure !== null) {
-        setSaveError(t(failureLocaleKey(failure.code)));
+        setSaveError(`${failure.code}: ${failure.message}`);
         return;
       }
-      await refresh();
+      if (!await refresh()) return;
       actions.cancelEdit();
       onSaved?.();
-    } catch {
-      setSaveError(t("failureTitle"));
+    } catch (error51) {
+      console.error("[dsh-mcp] save failed:", error51);
+      setSaveError(errorText(error51));
     } finally {
       actions.setBusy(null);
     }
@@ -15101,13 +15091,14 @@ function McpServerForm(props) {
     try {
       const failure = await injected.remove(draft.id);
       if (failure !== null) {
-        setSaveError(failure.message);
+        setSaveError(`${failure.code}: ${failure.message}`);
         return;
       }
-      await refresh();
+      if (!await refresh()) return;
       actions.cancelEdit();
-    } catch {
-      setSaveError(t("failureTitle"));
+    } catch (error51) {
+      console.error("[dsh-mcp] remove failed:", error51);
+      setSaveError(errorText(error51));
     } finally {
       actions.setBusy(null);
     }
@@ -15115,7 +15106,11 @@ function McpServerForm(props) {
   const refresh = async () => {
     try {
       actions.setServers(await injected.list());
-    } catch {
+      return true;
+    } catch (error51) {
+      console.error("[dsh-mcp] refresh failed:", error51);
+      setSaveError(errorText(error51));
+      return false;
     }
   };
   const stdio = draft.transport === "stdio";

package/lib/index.js

@@ -105,7 +105,7 @@ function validateServerInput(server, env) {
 	if (!serverNameSchema.safeParse(server.serverName).success) throw new McpServerValidationError("serverName must match [A-Za-z0-9_-]{1,32}");
 	if (server.transport === "stdio") {
 		if (server.command.trim().length === 0) throw new McpServerValidationError("stdio transport requires a command");
-		if (server.cwd.length > 0 && !/^[/\\]/.test(server.cwd)) throw new McpServerValidationError("cwd must be an absolute path or empty");
+		if (server.cwd.length > 0 && !/^(?:[A-Za-z]:[\\/]|[/\\])/.test(server.cwd)) throw new McpServerValidationError("cwd must be an absolute path or empty");
 	} else try {
 		const parsed = new URL(server.url);
 		if (parsed.protocol !== "http:" && parsed.protocol !== "https:") throw new Error("non-http protocol");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-mcp",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "MCP 管理界面 + tool search：可视化配置/启停/刷新 MCP 服务器与工具勾选，按需检索（tool search）热注入；工具列表稳定命中缓存、不撑爆上下文。DeepSeek Harness plugin: MCP management UI + tool search — stable tool list, cache-friendly, no context bloat.",
   "type": "module",
   "main": "lib/index.js",

package/src/client/ServerForm.tsx

@@ -1,16 +1,16 @@
 import { useState, type ReactNode } from 'react'
-import type { McpServerId, McpServerView } from './types.ts'
+import type { McpManagerFailure, McpServerId, McpServerView } from './types.ts'
 import type {
   McpDraft, McpEnvRowDraft, McpTestOutcome,
 } from './mcp-store.ts'
-import { createEnvRowDraft, failureLocaleKey } from './mcp-store.ts'
+import { createEnvRowDraft } from './mcp-store.ts'
 import type { McpSettingsLocaleKey } from './locales.ts'
 import css from './ServerForm.module.css'
 
 /** The Remote verbs the editor needs; structurally the section's inject face. */
 export interface McpServerFormRemote {
-  save: (draft: McpDraft) => Promise<{ message: string } | null>
-  remove: (id: McpServerId) => Promise<{ message: string } | null>
+  save: (draft: McpDraft) => Promise<McpManagerFailure | null>
+  remove: (id: McpServerId) => Promise<McpManagerFailure | null>
   test: (draft: McpDraft) => Promise<McpTestOutcome>
   list: () => Promise<readonly McpServerView[]>
 }
@@ -60,13 +60,18 @@ export function McpServerForm(props: McpServerFormProps): ReactNode {
     update({ env: [...draft.env, createEnvRowDraft()] })
   }
 
+  const errorText = (error: unknown): string =>
+    error instanceof Error ? error.message : String(error)
+
   const runTest = async (): Promise<void> => {
     setSaveError(null)
     actions.setTestRunning(true)
     try {
       actions.setTest(await injected.test(draft))
-    } catch {
+    } catch (error) {
+      console.error('[dsh-mcp] test failed:', error)
       actions.setTest(null)
+      setSaveError(errorText(error))
     } finally {
       actions.setTestRunning(false)
     }
@@ -78,14 +83,15 @@ export function McpServerForm(props: McpServerFormProps): ReactNode {
     try {
       const failure = await injected.save(draft)
       if (failure !== null) {
-        setSaveError(t(failureLocaleKey(failure.code)))
+        setSaveError(`${failure.code}: ${failure.message}`)
         return
       }
-      await refresh()
+      if (!await refresh()) return
       actions.cancelEdit()
       onSaved?.()
-    } catch {
-      setSaveError(t('failureTitle'))
+    } catch (error) {
+      console.error('[dsh-mcp] save failed:', error)
+      setSaveError(errorText(error))
     } finally {
       actions.setBusy(null)
     }
@@ -98,24 +104,27 @@ export function McpServerForm(props: McpServerFormProps): ReactNode {
     try {
       const failure = await injected.remove(draft.id)
       if (failure !== null) {
-        setSaveError(failure.message)
+        setSaveError(`${failure.code}: ${failure.message}`)
         return
       }
-      await refresh()
+      if (!await refresh()) return
       actions.cancelEdit()
-    } catch {
-      setSaveError(t('failureTitle'))
+    } catch (error) {
+      console.error('[dsh-mcp] remove failed:', error)
+      setSaveError(errorText(error))
     } finally {
       actions.setBusy(null)
     }
   }
 
-  const refresh = async (): Promise<void> => {
+  const refresh = async (): Promise<boolean> => {
     try {
       actions.setServers(await injected.list())
-    } catch {
-      // The save/remove outcome is the actionable one; a failed refresh is
-      // re-run on the next page visit.
+      return true
+    } catch (error) {
+      console.error('[dsh-mcp] refresh failed:', error)
+      setSaveError(errorText(error))
+      return false
     }
   }
 

package/src/client/mcp-store.ts

@@ -188,17 +188,6 @@ export function draftToSubmission(draft: McpDraft): McpSubmission {
   }
 }
 
-/** Map a Remote failure code to a locale key for user-facing copy. */
-export function failureLocaleKey(code: string): string {
-  switch (code) {
-    case 'MCP_SERVER_NAME_CONFLICT': return 'serverNameConflict'
-    case 'MCP_SERVER_NOT_FOUND': return 'notFound'
-    case 'MCP_INVALID_SPEC': return 'invalidSpec'
-    case 'MCP_MOUNT_FAILED': return 'mountFailed'
-    default: return 'failureTitle'
-  }
-}
-
 /**
  * Declares the MCP management page state and write surface.
  * @returns the store handle.