@cocorograph/hub-agent 0.7.56 → 0.7.57

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.7.56",
+  "version": "0.7.57",
   "description": "Hub Hosted Cockpit のローカル常駐 agent。Hub と outbound WSS で接続し、ローカルの tmux/pty を中継する。",
   "type": "module",
   "license": "UNLICENSED",

package/src/hub-bundle.mjs

@@ -320,29 +320,45 @@ export async function applyMcpServers(bundle, { agentId, agentToken, logger } =
     }
     const transport = s.transport || "http"
     const scope = s.scope || "user"
-    const argv = [
-      claudeBin,
-      "mcp",
-      "add",
-      "--transport",
-      transport,
-      "--scope",
-      scope,
-    ]
+
     // 越境制御 (2026-07): 一般向け `hub` MCP には、このホストの資格
     // `X-Cockpit-Agent: <agentId>:<agentToken>` をヘッダで付与する。Hub backend は
     // これを検証してホストのテナントを解決し、`hub` の acting tenant をそのテナントに
     // **固定**する（クライアントホストは自テナントの外を触れない）。`hub_staff`（社内
     // 横断）や外部 MCP には付けない。
-    if (s.name === "hub" && agentId && agentToken) {
+    const wantHeader = s.name === "hub" && agentId && agentToken
+
+    // ⚠️ 実 CLI で確認した 2 つの罠（claude-code、モックでは絶対に出ない）:
+    //
+    // 1. `--header` は **可変長引数** (`-H, --header <header...>`)。オプションを
+    //    位置引数より前に置くと、後続の `<name> <url>` まで header 値として飲み込み、
+    //    `error: missing required argument 'name'` で落ちる。
+    //    → **必ず `<name> <url>` の後ろに置く**（実測: 後置なら headers に正しく入る）。
+    //
+    // 2. `claude mcp add` は **冪等ではない**。既存の同名 server があると
+    //    `MCP server <name> already exists in <scope> config` を返して失敗し、
+    //    **ヘッダは永久に適用されない**（＝publish しても pin が発火しない）。
+    //    → ヘッダを付ける対象は **remove してから add** して確実に再登録する。
+    //    remove は「存在しなければ失敗する」ので、失敗は握り潰して続行する。
+    const argv = [claudeBin, "mcp", "add", "--transport", transport, "--scope", scope, s.name, s.url]
+    if (wantHeader) {
       argv.push("--header", `X-Cockpit-Agent: ${agentId}:${agentToken}`)
     }
-    argv.push(s.name, s.url)
+
     try {
+      if (wantHeader) {
+        // 既存があると add が弾かれるため、先に落とす（無ければ失敗するので無視）。
+        try {
+          await runOnce([claudeBin, "mcp", "remove", s.name, "-s", scope], { logger })
+        } catch {
+          // 未登録 → remove は失敗するが正常系。
+        }
+      }
       await runOnce(argv, { logger })
       added.push(s.name)
     } catch (err) {
-      // 既存のため fail することはほぼ無い (CLI は冪等)。ログだけ残して継続。
+      // `hub_staff` 等（ヘッダ不要）は既存なら "already exists" で失敗する。これは
+      // 「既に正しく登録済み」なので実害なし。ログだけ残して継続する。
       logger?.warn({ name: s.name, err: err.message }, "claude mcp add failed")
       skipped.push(s.name)
     }