@cocorograph/hub-agent 0.7.57 → 0.7.58

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.7.57",
+  "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: [] }
@@ -321,12 +330,21 @@ export async function applyMcpServers(bundle, { agentId, agentToken, logger } =
     const transport = s.transport || "http"
     const scope = s.scope || "user"
 
-    // 越境制御 (2026-07): 一般向け `hub` MCP には、このホストの資格
-    // `X-Cockpit-Agent: <agentId>:<agentToken>` をヘッダで付与する。Hub backend は
-    // これを検証してホストのテナントを解決し、`hub` の acting tenant をそのテナントに
-    // **固定**する（クライアントホストは自テナントの外を触れない）。`hub_staff`（社内
-    // 横断）や外部 MCP には付けない。
-    const wantHeader = 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、モックでは絶対に出ない）:
     //
@@ -357,8 +375,8 @@ export async function applyMcpServers(bundle, { agentId, agentToken, logger } =
       await runOnce(argv, { logger })
       added.push(s.name)
     } catch (err) {
-      // `hub_staff` 等（ヘッダ不要）は既存なら "already exists" で失敗する。これは
-      // 「既に正しく登録済み」なので実害なし。ログだけ残して継続する。
+      // ヘッダ不要の外部 MCP は既存なら "already exists" で失敗する。これは「既に正しく
+      // 登録済み」なので実害なし。ログだけ残して継続する。
       logger?.warn({ name: s.name, err: err.message }, "claude mcp add failed")
       skipped.push(s.name)
     }