agent-comm-hub 0.8.0 → 0.8.1

package/README.md

@@ -226,19 +226,28 @@ Merge into `~/.gemini/settings.json`:
 
 Auto-configured by `agent-comm-hub setup`: it discovers
 `~/.dsh/profiles/*/cordis.patch.yml` and appends the `@deepseek-ai/dsh-mcp-client`
-row, so DSH sessions expose the tools as `mcp__agent-hub__bridge_*` after a dsh
-restart. Manual equivalent (or template for other profiles):
+row with **`serverName: agent-comm-hub`** (not the generic `agent-hub` key other
+agents use). That name is deliberate: a DSH MCP-manager plugin that dynamically
+mounts `serverName: "agent-hub"` would otherwise collide
+(`serverName "agent-hub" is already in use by another mcp-client instance`) and
+fail the whole profile load. DSH sessions expose the tools as
+`mcp__agent-comm-hub__bridge_*` after a dsh restart. Re-running `setup` rewrites
+any older `serverName: agent-hub` insert to the new key. Manual equivalent:
 
 ```yaml
 - insert:
     - id: agent-comm-hub
       name: '@deepseek-ai/dsh-mcp-client'
       config:
-        serverName: agent-hub
+        serverName: agent-comm-hub
         transport: streamable-http
         url: http://127.0.0.1:18764/mcp
 ```
 
+If your MCP manager already owns the hub mount entirely, undo the static insert
+with `agent-comm-hub setup --remove` (or only refresh skills via
+`setup --agent` for other agents) and keep the dynamic mount.
+
 ## Tools
 
 | Tool | Purpose |

package/agents/README.md

@@ -30,7 +30,7 @@ Codex（TOML 追加）/ zcode（`mcp.servers`）。Claude Code 和 DSH 需手动
 | Codex | `~/.codex/config.toml` | `codex/config.toml` | `~/.codex/skills/agent-comm-hub/SKILL.md` |
 | zcode | `~/.zcode/cli/config.json` | `zcode/config.json` | `~/.zcode/skills/agent-comm-hub/SKILL.md` |
 | Claude Code | 项目根 `.mcp.json`（手动复制；**不碰 `~/.claude.json`**——含凭据且无法安全往返） | `claude-code/.mcp.json` | `~/.claude/skills/agent-comm-hub/SKILL.md` |
-| DeepSeek Harness (DSH) | profile `cordis.patch.yml`（手动合并） | `dsh/cordis.patch.yml`（用 `@deepseek-ai/dsh-mcp-client`，工具名为 `mcp__agent-hub__bridge_*`） | `$DSH_HOME/skills/agent-comm-hub/SKILL.md` |
+| DeepSeek Harness (DSH) | profile `cordis.patch.yml`（手动合并） | `dsh/cordis.patch.yml`（`@deepseek-ai/dsh-mcp-client`，**serverName: `agent-comm-hub`**，工具名为 `mcp__agent-comm-hub__bridge_*`；刻意不用 `agent-hub`，避免与 MCP 管理器动态挂载同名冲突） | `$DSH_HOME/skills/agent-comm-hub/SKILL.md` |
 
 > 各 agent 对 streamable-http MCP 的支持随版本演进，模板里的字段以官方文档为准；
 > 不支持的版本可退化为 stdio 包装（见下）。

package/agents/dsh/cordis.patch.yml

@@ -2,6 +2,10 @@
     - id: agent-comm-hub
       name: '@deepseek-ai/dsh-mcp-client'
       config:
-        serverName: agent-hub
+        # Distinct from the generic `agent-hub` key other agents use: a DSH
+        # MCP-manager plugin that dynamically mounts serverName "agent-hub"
+        # would otherwise collide ("serverName already in use") and fail the
+        # whole profile load.
+        serverName: agent-comm-hub
         transport: streamable-http
         url: http://127.0.0.1:18764/mcp

package/agents/registry.json

@@ -110,12 +110,14 @@
     {
       "id": "dsh",
       "probe": [],
+      "serverName": "agent-comm-hub",
       "configs": [
         {
           "file": "~/.dsh/profiles/*/cordis.patch.yml",
           "section": null,
           "strategy": "dsh",
-          "entry": null
+          "entry": null,
+          "serverName": "agent-comm-hub"
         }
       ],
       "skill": "~/.dsh/skills",

package/lib/cli.js

@@ -2512,7 +2512,7 @@ var SQLiteStateStore = class {
 
 // src/index.ts
 var SERVER_NAME = "agent-comm-hub";
-var SERVER_VERSION = "0.8.0";
+var SERVER_VERSION = "0.8.1";
 var DEFAULT_HOST = "127.0.0.1";
 var DEFAULT_PORT = 18764;
 var DEFAULT_PATH = "/mcp";
@@ -2883,6 +2883,9 @@ function validateRegistry(registry) {
     if (agent.skill !== null && (typeof agent.skill !== "string" || !agent.skill.startsWith("~/"))) {
       throw new Error(`registry: ${agent.id}: skill must be '~'-relative or null`);
     }
+    if (agent.serverName !== void 0 && (typeof agent.serverName !== "string" || agent.serverName === "")) {
+      throw new Error(`registry: ${agent.id}: serverName must be a non-empty string`);
+    }
     if (agent.os !== void 0 && (!Array.isArray(agent.os) || agent.os.some((os2) => !["win32", "darwin", "linux"].includes(os2)))) {
       throw new Error(`registry: ${agent.id}: invalid os list`);
     }
@@ -3134,7 +3137,7 @@ async function mergeDshPatch(file, opts) {
     return "removed";
   }
   const ranges = hubEntryRanges();
-  if (ranges.length === 1 && lines.slice(ranges[0].start, ranges[0].end).some((line) => line.includes(`url: ${opts.url}`))) {
+  if (ranges.length === 1 && lines.slice(ranges[0].start, ranges[0].end).some((line) => line.includes(`url: ${opts.url}`)) && lines.slice(ranges[0].start, ranges[0].end).some((line) => line.includes(`serverName: ${opts.serverName}`))) {
     return "unchanged";
   }
   await backup(file);
@@ -3205,9 +3208,10 @@ async function runSetup(options = {}) {
   }
   for (const agent of targetAgents) {
     for (const config of agent.configs) {
+      const nameFor = config.serverName ?? agent.serverName ?? serverName;
       for (const file of expandConfigFile(config.file, home)) {
         try {
-          const status = config.strategy === "json" ? await mergeJsonServer(file, config.section, substitute(config.entry, { url, serverName }), { serverName, url, remove }) : config.strategy === "toml" ? await mergeTomlSection(file, { serverName, url, remove }) : await mergeDshPatch(file, { serverName, url, remove });
+          const status = config.strategy === "json" ? await mergeJsonServer(file, config.section, substitute(config.entry, { url, serverName: nameFor }), { serverName: nameFor, url, remove }) : config.strategy === "toml" ? await mergeTomlSection(file, { serverName: nameFor, url, remove }) : await mergeDshPatch(file, { serverName: nameFor, url, remove });
           record(status, agent.id, file);
         } catch (error) {
           summary.errors.push(`${agent.id}: ${file} \u2014 ${error.message}`);

package/lib/index.js

@@ -2518,7 +2518,7 @@ var SQLiteStateStore = class {
 
 // src/index.ts
 var SERVER_NAME = "agent-comm-hub";
-var SERVER_VERSION = "0.8.0";
+var SERVER_VERSION = "0.8.1";
 var DEFAULT_HOST = "127.0.0.1";
 var DEFAULT_PORT = 18764;
 var DEFAULT_PATH = "/mcp";

package/lib/setup.js

@@ -62,6 +62,9 @@ function validateRegistry(registry) {
     if (agent.skill !== null && (typeof agent.skill !== "string" || !agent.skill.startsWith("~/"))) {
       throw new Error(`registry: ${agent.id}: skill must be '~'-relative or null`);
     }
+    if (agent.serverName !== void 0 && (typeof agent.serverName !== "string" || agent.serverName === "")) {
+      throw new Error(`registry: ${agent.id}: serverName must be a non-empty string`);
+    }
     if (agent.os !== void 0 && (!Array.isArray(agent.os) || agent.os.some((os) => !["win32", "darwin", "linux"].includes(os)))) {
       throw new Error(`registry: ${agent.id}: invalid os list`);
     }
@@ -302,7 +305,7 @@ async function mergeDshPatch(file, opts) {
     return "removed";
   }
   const ranges = hubEntryRanges();
-  if (ranges.length === 1 && lines.slice(ranges[0].start, ranges[0].end).some((line) => line.includes(`url: ${opts.url}`))) {
+  if (ranges.length === 1 && lines.slice(ranges[0].start, ranges[0].end).some((line) => line.includes(`url: ${opts.url}`)) && lines.slice(ranges[0].start, ranges[0].end).some((line) => line.includes(`serverName: ${opts.serverName}`))) {
     return "unchanged";
   }
   await backup(file);
@@ -373,9 +376,10 @@ async function runSetup(options = {}) {
   }
   for (const agent of targetAgents) {
     for (const config of agent.configs) {
+      const nameFor = config.serverName ?? agent.serverName ?? serverName;
       for (const file of expandConfigFile(config.file, home)) {
         try {
-          const status = config.strategy === "json" ? await mergeJsonServer(file, config.section, substitute(config.entry, { url, serverName }), { serverName, url, remove }) : config.strategy === "toml" ? await mergeTomlSection(file, { serverName, url, remove }) : await mergeDshPatch(file, { serverName, url, remove });
+          const status = config.strategy === "json" ? await mergeJsonServer(file, config.section, substitute(config.entry, { url, serverName: nameFor }), { serverName: nameFor, url, remove }) : config.strategy === "toml" ? await mergeTomlSection(file, { serverName: nameFor, url, remove }) : await mergeDshPatch(file, { serverName: nameFor, url, remove });
           record(status, agent.id, file);
         } catch (error) {
           summary.errors.push(`${agent.id}: ${file} \u2014 ${error.message}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-comm-hub",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "description": "Generic multi-peer MCP hub: any MCP-capable agent (MiniMax Code, Claude Code, opencode, Codex, Gemini CLI, DSH, ...) connects to one local streamable-http endpoint and they chat, delegate tasks, and acknowledge in real time",
   "keywords": [
     "mcp",