@cocorograph/hub-agent 0.7.57 → 0.7.59

package/package.json

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

package/src/codex-mcp-control.mjs

@@ -0,0 +1,313 @@
+/**
+ * Codex の MCP サーバー状態 control (agent-global, チャットセッション非依存)。
+ *
+ * Claude 側 (claude-stream-bridge.mjs の「MCP control 専用 query」) と同じ役割を Codex で
+ * 果たす。MCP サーバーは `~/.codex/config.toml` = アカウント単位の設定で全セッション共有
+ * のため、チャットの stream_id / thread に紐付けず agent-global な control クライアントで
+ * 扱い、アイドルで破棄する。
+ *
+ * ── 2026-07-14 の実機検証で確定した Codex app-server の仕様 ──────────────────
+ *
+ * 1. `mcpServerStatus/list` は **生存状態を返さない**。返るのは
+ *    `{name, serverInfo, tools, resources, resourceTemplates, authStatus}` で、
+ *    `authStatus` は「認証方式」(unsupported | notLoggedIn | bearerToken | oAuth) であって
+ *    「今つながっているか」ではない。実際、OAuth 期限切れで接続に失敗している hub も
+ *    `authStatus: "oAuth"` を返す (成功している hub_staff と同値) ため、これ単独では
+ *    Claude の `status` 相当を作れない。
+ *
+ * 2. 生存状態は `mcpServer/startupStatus/updated` 通知
+ *    (`{name, status: starting|ready|failed|cancelled, error, threadId}`) でのみ得られる。
+ *    ⚠️ この通知は **thread を start しないと一切飛ばない** (initialize だけでは無音)。
+ *    そのため control クライアントは「捨てスレッド」を 1 本張って MCP 起動を誘発し、
+ *    通知を集約する。turn は開始しないのでトークン消費は無い。
+ *
+ * 3. `detail: "toolsAndAuthOnly"` でも応答は 1MB 超になりうる (tools の inputSchema が
+ *    そのまま載る。実測 1.2MB / 421 tools)。**WS に載せる前に必ず要約する** こと。
+ *
+ * 4. `mcpServer/oauth/login` は認可 URL を即返す (実測 0.2s)。ブラウザは codex 側では
+ *    開かず、クライアント (Cockpit) が開く前提。コールバックは codex がホスト上の
+ *    ループバック (127.0.0.1:<port>/callback/<id>) で待ち受け、完了は
+ *    `mcpServer/oauthLogin/completed` 通知で push される。**Claude と違いコールバック URL の
+ *    貼り付けは不要** なので、UI にはブローカー型 (callbackExpected: false) として返す。
+ *    ⚠️ 裏を返すと、Cockpit をスマホで開いている場合はループバックに着地できないため
+ *    デスクトップからの再認証のみ成立する (既知の制約)。
+ */
+import os from "node:os"
+
+import { CodexAppServerClient } from "./codex-appserver-client.mjs"
+
+/** control クライアントのアイドル破棄時間 (Claude 側の MCP_CONTROL_IDLE_MS と対称)。 */
+const IDLE_MS = Number(process.env.HUB_AGENT_MCP_CONTROL_IDLE_MS || 5 * 60 * 1000)
+
+/** 起動通知が落ち着くまでの静穏時間 / 全体の上限。 */
+const SETTLE_QUIET_MS = 1_200
+const SETTLE_MAX_MS = 25_000
+
+/**
+ * Codex の startup 状態 + list エントリを Cockpit の `McpServerStatus` へ正規化する。
+ *
+ * フロントの型 (`useCockpitMcpServers.ts` の McpServerStatus) は Claude SDK 由来だが、
+ * 同じ形に寄せることで Popover / 再認証 UI をそのまま再利用できる。
+ *
+ * `needs-auth` の判定が肝: Codex は「OAuth が要る」を専用の状態では返さず、
+ * `failed` + エラー文言 ("OAuth authorization required" 等) で表現する。ここを
+ * `failed` のままにすると UI が「再接続」ボタンを出してしまい、何度押しても直らない
+ * (OAuth フローに入れない) ため、認証由来の失敗は `needs-auth` に倒す。
+ *
+ * @param {{name: string, serverInfo?: object|null, tools?: object|null, authStatus?: string}} entry
+ * @param {{status?: string, error?: string|null}|undefined} startup
+ * @returns {{name: string, status: string, serverInfo?: object, error?: string, tools: {name: string}[]}}
+ */
+export function normalizeCodexMcpServer(entry, startup) {
+  const name = entry?.name
+  const authStatus = entry?.authStatus
+  const startupState = startup?.status
+  const startupError = startup?.error || undefined
+
+  // tools は {toolName: {name, description, inputSchema...}} のマップで来る。
+  // inputSchema を載せると WS が 1MB 超になるため、名前だけに落とす。
+  const toolNames = entry?.tools && typeof entry.tools === "object" ? Object.keys(entry.tools) : []
+
+  let status
+  if (startupState === "ready") {
+    status = "connected"
+  } else if (startupState === "starting") {
+    status = "pending"
+  } else if (startupState === "failed" || startupState === "cancelled") {
+    status = isAuthFailure(startupError, authStatus) ? "needs-auth" : "failed"
+  } else if (authStatus === "notLoggedIn") {
+    // 通知が来ていない (thread 起動前 / 取りこぼし) が、未ログインは確実に再認証が要る。
+    status = "needs-auth"
+  } else if (entry?.serverInfo) {
+    // 通知を取りこぼしても serverInfo が埋まっていれば handshake は成立している。
+    status = "connected"
+  } else {
+    status = "pending"
+  }
+
+  return {
+    name,
+    status,
+    ...(entry?.serverInfo ? { serverInfo: entry.serverInfo } : {}),
+    ...(startupError ? { error: truncateError(startupError) } : {}),
+    tools: toolNames.map((n) => ({ name: n })),
+  }
+}
+
+/** 起動失敗が「認証が要る」由来か。OAuth フロー (再認証ボタン) に倒してよいかの判定。 */
+function isAuthFailure(error, authStatus) {
+  if (authStatus === "notLoggedIn") return true
+  if (!error) return false
+  // 実測のエラー文言:
+  //   "MCP client for `hub` failed to start: ... Auth error: OAuth authorization required,
+  //    when send initialize request"
+  return /oauth|authorization required|unauthorized|401/i.test(error)
+}
+
+/** エラー文言は UI の 1 行に収まらないため要約する (全文は agent のログに残る)。 */
+function truncateError(error) {
+  const s = String(error)
+  if (/oauth|authorization required/i.test(s)) return "OAuth 認証が必要です"
+  return s.length > 160 ? `${s.slice(0, 157)}…` : s
+}
+
+export class CodexMcpControl {
+  /**
+   * @param {{logger?: object, codexCmd?: string, cwd?: string, idleMs?: number,
+   *          clientFactory?: (opts: object) => CodexAppServerClient}} [opts]
+   */
+  constructor({ logger, codexCmd, cwd, idleMs, clientFactory } = {}) {
+    this.logger = logger
+    this.codexCmd = codexCmd
+    // control は特定のプロジェクトに属さない。MCP 設定はグローバル (~/.codex/config.toml)
+    // なので home を cwd にする (プロジェクト固有の設定に引きずられないため)。
+    this.cwd = cwd || os.homedir()
+    this.idleMs = typeof idleMs === "number" ? idleMs : IDLE_MS
+    this._clientFactory =
+      clientFactory || ((o) => new CodexAppServerClient(o))
+
+    this.client = null
+    /** @type {Map<string, {status: string, error: string|null}>} name → 起動状態。 */
+    this.startup = new Map()
+    this._starting = null
+    this._idleTimer = null
+    /** OAuth 完了通知の待受 (name → resolve)。 */
+    this._oauthWaiters = new Map()
+  }
+
+  /** control クライアントを起動し、捨てスレッドで MCP 起動を誘発して通知を集める。 */
+  async _ensure() {
+    if (this.client && !this.client.closed) return this.client
+    if (this._starting) return this._starting
+
+    this._starting = (async () => {
+      const client = this._clientFactory({
+        cwd: this.cwd,
+        codexCmd: this.codexCmd,
+        logger: this.logger,
+      })
+      client.on("notification", (msg) => this._onNotification(msg))
+      client.on("request", (msg) => {
+        // control では承認要求は起こり得ない (turn を開始しないため) が、万一来ても
+        // ハングさせないよう即エラーを返す。
+        client.respondError(msg.id, {
+          code: -32601,
+          message: "codex mcp control does not handle server requests",
+        })
+      })
+      client.on("exit", () => {
+        this.client = null
+        this.startup.clear()
+      })
+      client.start()
+
+      await client.request("initialize", {
+        clientInfo: { name: "hub-agent", title: "Hub Agent", version: "mcp-control" },
+      })
+      client.notify("initialized", {})
+
+      // ⚠️ thread を張らないと mcpServer/startupStatus/updated が一切飛ばない
+      // (2026-07-14 実機検証)。turn は開始しないのでトークン消費は無い。
+      await client.request("thread/start", { cwd: this.cwd })
+
+      this.client = client
+      return client
+    })()
+
+    try {
+      return await this._starting
+    } finally {
+      this._starting = null
+    }
+  }
+
+  _onNotification(msg) {
+    if (msg?.method === "mcpServer/startupStatus/updated") {
+      const p = msg.params || {}
+      if (p.name) {
+        this.startup.set(p.name, { status: p.status, error: p.error ?? null })
+      }
+      return
+    }
+    if (msg?.method === "mcpServer/oauthLogin/completed") {
+      const p = msg.params || {}
+      const waiter = this._oauthWaiters.get(p.name)
+      if (waiter) {
+        this._oauthWaiters.delete(p.name)
+        waiter({ success: p.success === true, error: p.error ?? null })
+      }
+      this.logger?.info?.(
+        { server: p.name, success: p.success },
+        "codex mcp oauth login completed",
+      )
+    }
+  }
+
+  /**
+   * 起動通知が出揃うまで待つ。全サーバーが終端状態 (ready/failed/cancelled) になり、
+   * かつ静穏時間 (新規通知が来ない) を満たしたら抜ける。上限で打ち切る。
+   */
+  async _waitForStartupSettle({ quietMs = SETTLE_QUIET_MS, maxMs = SETTLE_MAX_MS } = {}) {
+    const deadline = Date.now() + maxMs
+    let lastSize = -1
+    let lastChange = Date.now()
+    for (;;) {
+      const terminal = [...this.startup.values()].every(
+        (s) => s.status === "ready" || s.status === "failed" || s.status === "cancelled",
+      )
+      const size = this.startup.size
+      if (size !== lastSize) {
+        lastSize = size
+        lastChange = Date.now()
+      }
+      if (size > 0 && terminal && Date.now() - lastChange >= quietMs) return
+      if (Date.now() >= deadline) return
+      await new Promise((r) => setTimeout(r, 150))
+    }
+  }
+
+  /** アイドルタイマーを引き直す (触られなくなって idleMs 経過したら破棄)。 */
+  _touch() {
+    if (this._idleTimer) clearTimeout(this._idleTimer)
+    if (this.idleMs <= 0) return
+    this._idleTimer = setTimeout(() => this.stop(), this.idleMs)
+    this._idleTimer.unref?.()
+  }
+
+  /**
+   * MCP サーバー一覧を Cockpit 用に正規化して返す。
+   * @param {{reload?: boolean}} [opts] reload=true で control を作り直し設定を読み直す。
+   */
+  async status({ reload = false } = {}) {
+    if (reload) this.stop()
+    const client = await this._ensure()
+    await this._waitForStartupSettle()
+
+    const res = await client.request("mcpServerStatus/list", {
+      // full だと resources / resourceTemplates まで載って更に膨らむ。
+      detail: "toolsAndAuthOnly",
+    })
+    const rows = res?.data ?? []
+    this._touch()
+    return rows.map((entry) => normalizeCodexMcpServer(entry, this.startup.get(entry?.name)))
+  }
+
+  /**
+   * 設定を読み直して MCP を張り直す。
+   *
+   * Codex には Claude の `reconnectMcpServer(name)` に相当する「1 サーバーだけ再接続」が
+   * 無く、`config/mcpServer/reload` は設定全体の再読込 (params 無し) となる。個別指定が
+   * 効かない以上、control を作り直して全サーバーを張り直すのが最も確実なので、
+   * server_name は「どのサーバーの操作としてログに残すか」だけに使う。
+   */
+  async reconnect(serverName) {
+    const client = await this._ensure()
+    try {
+      await client.request("config/mcpServer/reload", null)
+    } catch (err) {
+      // reload が無い / 失敗しても、control 作り直しで実質同じ効果が得られる。
+      this.logger?.warn?.(
+        { err: err?.message, server: serverName },
+        "codex config/mcpServer/reload failed; falling back to control restart",
+      )
+    }
+    // 設定再読込後の状態を取り直す (control ごと作り直して startup 通知を採り直す)。
+    const servers = await this.status({ reload: true })
+    return { servers, targeted: 0 }
+  }
+
+  /**
+   * OAuth 認証を開始し、認可 URL を返す。
+   *
+   * codex がホスト上のループバックでコールバックを待ち受けるため、Cockpit 側での
+   * コールバック URL 貼り付けは不要 (callbackExpected: false)。
+   */
+  async authenticate(serverName) {
+    const client = await this._ensure()
+    const res = await client.request("mcpServer/oauth/login", { name: serverName })
+    this._touch()
+    return {
+      authorizationUrl: res?.authorizationUrl,
+      // Claude の localhost コールバック型と違い、貼り付け UI は出さない。
+      callbackExpected: false,
+      requiresUserAction: true,
+    }
+  }
+
+  /** control クライアントを停止する (アイドル破棄 / reload 用)。二重呼び出しは無害。 */
+  stop() {
+    if (this._idleTimer) {
+      clearTimeout(this._idleTimer)
+      this._idleTimer = null
+    }
+    if (this.client) {
+      try {
+        this.client.stop()
+      } catch {
+        // 既に落ちている場合等は無視して良い (stop の冪等性を優先)。
+      }
+      this.client = null
+    }
+    this.startup.clear()
+  }
+}

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)
     }

package/src/main.mjs

@@ -25,6 +25,7 @@ import { loadPlugins, runHookBroadcast, runHookChain } from "./plugin-loader.mjs
 import { WsClient } from "./ws-client.mjs"
 import { PtyBridge } from "./pty-bridge.mjs"
 import { ClaudeStreamBridge } from "./claude-stream-bridge.mjs"
+import { CodexMcpControl } from "./codex-mcp-control.mjs"
 import { CodexStreamBridge, classifyCodexInputError } from "./codex-stream-bridge.mjs"
 import { UploadManager } from "./claude-upload.mjs"
 import { requestSelfUninstall } from "./service-install.mjs"
@@ -541,6 +542,14 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     ? new CodexStreamBridge({ logger })
     : null
 
+  // Codex の MCP 状態 control (agent-global, セッション非依存)。MCP は ~/.codex/config.toml
+  // = アカウント単位の設定で全セッション共有のため、claudeBridge の「MCP control 専用
+  // query」と同じ流儀で、チャットの thread に紐付けない専用クライアントで扱う。
+  // codex バイナリが無ければ null (codex.mcp.* は cli_unavailable を返す)。
+  const codexMcpControl = availableClis.includes("codex")
+    ? new CodexMcpControl({ logger })
+    : null
+
   // Cockpit チャットモードの添付ファイル受信器 (browser → agent のチャンク送信を
   // ローカル FS に保存し、保存パスを返す)。SDK 有無に関わらず生成してよいが、添付は
   // チャットモード専用機能なので claudeBridge と同じく stream モード前提で使う。
@@ -820,7 +829,7 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     // permission / cancel→paste 等) は WS 受信順 = pane 反映順を守るため dispatchChain に残す。
     if (isFastPathMessage(msg?.type)) {
       Promise.resolve(
-        dispatch(msg, { ...ctx, client, ptyBridge, claudeBridge, codexBridge, uploadManager }),
+        dispatch(msg, { ...ctx, client, ptyBridge, claudeBridge, codexBridge, codexMcpControl, uploadManager }),
       ).catch((err) => {
         logger.error(
           { err: err.message, type: msg?.type },
@@ -830,7 +839,7 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
       return
     }
     dispatchChain = dispatchChain
-      .then(() => dispatch(msg, { ...ctx, client, ptyBridge, claudeBridge, codexBridge, uploadManager }))
+      .then(() => dispatch(msg, { ...ctx, client, ptyBridge, claudeBridge, codexBridge, codexMcpControl, uploadManager }))
       .catch((err) => {
         logger.error(
           { err: err.message, type: msg?.type },
@@ -1121,7 +1130,7 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     )
   })
 
-  return { client, plugins, ptyBridge, claudeBridge, codexBridge, uploadManager }
+  return { client, plugins, ptyBridge, claudeBridge, codexBridge, codexMcpControl, uploadManager }
 }
 
 const SESSION_EVENTS_DIR =
@@ -3833,6 +3842,71 @@ async function dispatch(msg, ctx) {
         }
         return
       }
+      case "codex.mcp.status":
+      case "codex.mcp.reconnect":
+      case "codex.mcp.authenticate": {
+        // Codex 版 MCP control (agent-global, セッション非依存) — 2026-07-14。
+        // claude.mcp.* と同じ request_id 相関・同じ応答形 (servers[] は Cockpit の
+        // McpServerStatus に正規化済み) にすることで、フロントの Popover / 再認証 UI を
+        // そのまま再利用する。
+        //
+        // Claude との差分 (Codex 側に対応物が無いもの):
+        //   - callback : codex がホスト上のループバックでコールバックを受けるため不要
+        //                (authenticate の応答で callbackExpected: false を返す)
+        //   - clearauth: 対応する RPC が無い (CLI の `codex mcp logout` のみ)
+        const request_id = msg.request_id
+        if (!ctx.codexMcpControl) {
+          ctx.client.send({
+            type: "codex.mcp.response",
+            request_id,
+            action: msg.type,
+            error: "codex_cli_unavailable",
+          })
+          return
+        }
+        try {
+          let result
+          switch (msg.type) {
+            case "codex.mcp.status":
+              result = {
+                servers: await ctx.codexMcpControl.status({
+                  reload: msg.reload === true,
+                }),
+              }
+              break
+            case "codex.mcp.reconnect": {
+              const r = await ctx.codexMcpControl.reconnect(msg.server_name)
+              result = { servers: r.servers, targeted: r.targeted }
+              break
+            }
+            case "codex.mcp.authenticate":
+              result = {
+                auth: await ctx.codexMcpControl.authenticate(msg.server_name),
+              }
+              break
+          }
+          ctx.client.send({
+            type: "codex.mcp.response",
+            request_id,
+            action: msg.type,
+            server_name: msg.server_name,
+            ...result,
+          })
+        } catch (err) {
+          ctx.logger?.warn(
+            { err: err?.message, action: msg.type, server: msg.server_name },
+            "codex.mcp control request failed",
+          )
+          ctx.client.send({
+            type: "codex.mcp.response",
+            request_id,
+            action: msg.type,
+            server_name: msg.server_name,
+            error: err?.message || "mcp_control_failed",
+          })
+        }
+        return
+      }
       case "claude.queue.cancel":
         // 送信待ち (pending) メッセージを id 指定で取り消す (0.6.26)。
         if (!ctx.claudeBridge) return