@max-null/dsh-skill-mcp-center 0.6.2 → 0.6.4

package/README.md

@@ -23,6 +23,12 @@ Skill 与 MCP 管理中心：在设置里管理 skills 与 MCP 服务器，右
 
 ![设置 → Skill & MCP](docs/shots/skill-mcp-1.png)
 
+### 设置页：MCP 开关的效力边界
+
+DSH 的 `Loader.write()` 是空实现，面板上的启停只改内存树——所以 MCP 标签页顶部写明「开关只对本次运行生效，重启后恢复」；要长期禁用，给 `cordis.patch.yml` 的对应条目加 `disabled: true`。被面板关掉的条目会在卡片里标「本次运行已停用 · 重启恢复」，而 patch 里持久禁用的条目不标——两种「关」必须能区分：
+
+![MCP 开关的效力边界](docs/shots/mcp-switch-session-only.png)
+
 ### 侧栏：Skill 一览
 
 搜索 + 命名空间筛选（全部 / 全局 / 工作区），每条带开关与 `MD`（查看 SKILL.md）。**插件包内自带的 skill 同样列出**，并标出来源插件——下图里 8 个 `ssid-*` 来自 `dsh-skills`，而用户级的 `ssid-release` 没有来源标签，这正是两者的区别：

package/client.js

@@ -223,6 +223,8 @@ var zhDict = {
   provider: "provider \xB7 {name}",
   addServer: "\uFF0B \u6DFB\u52A0 server",
   hotApplied: "\u589E\u5220\u6539\u70ED\u751F\u6548\uFF0C\u514D\u91CD\u542F",
+  switchSessionOnly: "\u5F00\u5173\u53EA\u5BF9\u672C\u6B21\u8FD0\u884C\u751F\u6548\uFF0C\u91CD\u542F\u540E\u6062\u590D\uFF08\u8981\u957F\u671F\u7981\u7528\uFF0C\u7ED9\u8BE5\u6761\u76EE\u52A0 disabled: true\uFF09",
+  switchReverted: "\u672C\u6B21\u8FD0\u884C\u5DF2\u505C\u7528 \xB7 \u91CD\u542F\u6062\u590D",
   edit: "\u7F16\u8F91",
   remove: "\u5220\u9664",
   disableServer: "\u505C\u7528\uFF08\u65AD\u5F00\u5E76\u91CA\u653E context\uFF09",
@@ -288,6 +290,8 @@ var enDict = {
   provider: "provider \xB7 {name}",
   addServer: "\uFF0B Add server",
   hotApplied: "hot-applied, no restart",
+  switchSessionOnly: "switch applies to this run only, reverted after restart (to disable for good, add disabled: true to the entry)",
+  switchReverted: "disabled for this run \xB7 restored on restart",
   edit: "Edit",
   remove: "Remove",
   disableServer: "Disable (disconnect & free context)",
@@ -659,6 +663,7 @@ function McpView() {
       /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { className: "smc-spacer" }),
       /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { className: "smc-sub", children: t("hotApplied") })
     ] }),
+    /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { className: "smc-sub", style: { marginBottom: 8 }, children: t("switchSessionOnly") }),
     editing !== null && /* @__PURE__ */ (0, import_jsx_runtime.jsx)(ServerForm, { server: editing === "new" ? null : editing, onClose: () => {
       setEditing(null);
     }, onSaved: () => {
@@ -690,7 +695,8 @@ function McpView() {
           }
         )
       ] }),
-      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { className: "smc-desc", children: s.transport === "stdio" ? `${s.command ?? ""} ${(s.args ?? []).join(" ")}` : s.url })
+      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { className: "smc-desc", children: s.transport === "stdio" ? `${s.command ?? ""} ${(s.args ?? []).join(" ")}` : s.url }),
+      s.disabled && s.sessionToggled === true && /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { className: "smc-desc", children: t("switchReverted") })
     ] }, s.id))
   ] });
 }

package/dist/service.d.ts

@@ -42,6 +42,13 @@ export interface McpServer {
     url?: string;
     headers?: Record<string, string>;
     disabled: boolean;
+    /**
+     * True when the switch was changed during this run. Lets the panel say
+     * "reverted on restart" only where that is true — an entry disabled in
+     * `cordis.patch.yml` stays disabled across restarts and must not be
+     * labelled as session-only.
+     */
+    sessionToggled: boolean;
     fiberPhase: string | null;
 }
 /** One tool a server contributes, as the management surface lists it. */
@@ -146,6 +153,8 @@ export declare function deleteTargetOf(path: string, roots: readonly string[]):
 export declare class SkillMcpService extends Service {
     static inject: string[];
     private readonly officialSkillDirs;
+    /** Ids toggled by the panel during this run (in-memory only, by design: the host does not persist it). */
+    private readonly sessionToggled;
     constructor(ctx: Context, config?: SkillConfig);
     /**
      * Skill roots that live **inside loaded plugin packages**.
@@ -207,7 +216,7 @@ export declare class SkillMcpService extends Service {
     updateMcpServer(id: string, config: McpConfig): Promise<void>;
     /** Remove one server — disconnects and unregisters its tools. */
     removeMcpServer(id: string): Promise<void>;
-    /** Enable/disable one server without deleting its config. */
+    /** Enable/disable one server without deleting its config. The host does not persist this. */
     setMcpServerEnabled(id: string, enabled: boolean): Promise<void>;
     /** Runtime status per server: upstream `mcpStatus` seam when present, else derived. */
     mcpStatus(): Promise<McpServerStatus[]>;

package/dist/service.js

@@ -232,6 +232,8 @@ export function deleteTargetOf(path, roots) {
 export class SkillMcpService extends Service {
     static inject = ['loader', 'tools'];
     officialSkillDirs;
+    /** Ids toggled by the panel during this run (in-memory only, by design: the host does not persist it). */
+    sessionToggled = new Set();
     constructor(ctx, config = {}) {
         super(ctx, 'skillMcp');
         this.officialSkillDirs = config.officialSkillDirs ?? [];
@@ -404,17 +406,23 @@ export class SkillMcpService extends Service {
             // plugin `name` ('mcp-client'); one entry = one MCP server.
             if (entry.options.name !== '@deepseek-ai/dsh-mcp-client')
                 continue;
-            const cfg = entry.options.config;
+            // `!!js` 表达式在 options.config 里保留为 { __jsExpr } 节点，求值结果在 fiber.config 上
+            // （与本仓库 pluginSkillRoots 同一个坑）。照 options 读会把表达式当值：
+            // command 渲染成 [object Object]、args 变成一串 JSON 转义。fiber 缺失时退回 options。
+            const evaluated = entry.fiber?.config;
+            const cfg = evaluated ?? entry.options.config;
+            const text = (value) => typeof value === 'string' ? value : value === undefined ? undefined : JSON.stringify(value);
             servers.push({
                 id: entry.id,
                 serverName: cfg?.serverName ?? entry.id,
                 transport: cfg?.transport === 'streamable-http' ? 'streamable-http' : 'stdio',
-                command: cfg?.command,
-                args: cfg?.args,
-                cwd: cfg?.cwd,
-                url: cfg?.url,
+                command: text(cfg?.command),
+                args: cfg?.args?.map(arg => text(arg) ?? ''),
+                cwd: text(cfg?.cwd),
+                url: text(cfg?.url),
                 headers: cfg?.headers,
                 disabled: entry.disabled,
+                sessionToggled: this.sessionToggled.has(entry.id),
                 fiberPhase: entry.fiber === undefined ? null : FIBER_PHASE[entry.fiber.state],
             });
         }
@@ -436,9 +444,12 @@ export class SkillMcpService extends Service {
     async removeMcpServer(id) {
         await this.ctx.loader.remove(id);
     }
-    /** Enable/disable one server without deleting its config. */
+    /** Enable/disable one server without deleting its config. The host does not persist this. */
     async setMcpServerEnabled(id, enabled) {
         await this.ctx.loader.update(id, { disabled: enabled ? null : true });
+        // 记「本次运行改过」：Loader 根树的 write 是 no-op（loader lib/index.js 的 `write() {}`），
+        // 所以这里改的只是内存态；面板据此显示“重启后恢复”，而不是假装它是持久设置。
+        this.sessionToggled.add(id);
     }
     /** Runtime status per server: upstream `mcpStatus` seam when present, else derived. */
     async mcpStatus() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@max-null/dsh-skill-mcp-center",
-  "version": "0.6.2",
+  "version": "0.6.4",
   "description": "Skill & MCP management center for DeepSeek Harness — manage skills and MCP servers in Settings, live MCP status in the sidebar",
   "license": "MIT",
   "publishConfig": {