@cocorograph/hub-agent 0.7.56 → 0.7.58

package/package.json

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

package/src/hub-bundle.mjs

@@ -300,6 +300,15 @@ function runOnce(argv, { logger } = {}) {
  * (Cockpit 用途では install.sh が事前に claude をインストールしている
  * 想定だが、独自運用環境への配慮で fail-soft にする)
  */
+/**
+ * ホスト資格ヘッダ（`X-Cockpit-Agent`）を付与する Hub の MCP サーバー名。
+ *
+ * Hub backend の acting tenant はこのヘッダで**ホストのテナントに固定**される。両方に
+ * 付けるのが正（`hub` だけだと、社内が実際に使う `hub_staff` が固定されず、フロントの
+ * ワークスペース切替で実行中セッションがデータを読めなくなる）。
+ */
+export const HUB_MCP_NAMES = new Set(["hub", "hub_staff"])
+
 export async function applyMcpServers(bundle, { agentId, agentToken, logger } = {}) {
   const servers = Array.isArray(bundle?.mcp_servers) ? bundle.mcp_servers : []
   if (servers.length === 0) return { added: [], skipped: [] }
@@ -320,29 +329,54 @@ 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) {
+
+    // 越境制御 (2026-07-14 訂正): **Hub の MCP（`hub` / `hub_staff`）の両方**に、この
+    // ホストの資格 `X-Cockpit-Agent: <agentId>:<agentToken>` をヘッダで付与する。Hub
+    // backend はこれを検証してホストのテナントを解決し、acting tenant をそのテナントに
+    // **固定**する（フロントエンドのワークスペース切替より優先。端末とワークスペースを
+    // 紐づけるのが設計意図）。外部 MCP には付けない。
+    //
+    // ⚠️ かつて `hub` にしか付けていなかったが、**設計意図が未達だった**。社内の横断作業は
+    // decision B により `hub_staff` を使うため、**実際に使う経路が固定されず**、フロントを
+    // 個人WSに切り替えると実行中の Cockpit セッションがココログラフのデータを読めなくなる
+    // 回帰が残っていた（本番実測 2026-07-14）。
+    //
+    // `hub_staff` を固定しても社内スタッフの横断は死なない: 提供元（ココログラフ）ホストは
+    // ココログラフテナントに固定され、backend の `is_unrestricted` はそこで真になるため
+    // （固定 ≠ 横断不可。固定されたテナントが提供元かどうかで決まる）。
+    const wantHeader = HUB_MCP_NAMES.has(s.name) && 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 は冪等)。ログだけ残して継続。
+      // ヘッダ不要の外部 MCP は既存なら "already exists" で失敗する。これは「既に正しく
+      // 登録済み」なので実害なし。ログだけ残して継続する。
       logger?.warn({ name: s.name, err: err.message }, "claude mcp add failed")
       skipped.push(s.name)
     }