@cocorograph/hub-agent 0.7.97 → 0.7.98

package/package.json

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

package/src/hub-bundle.mjs

@@ -471,7 +471,7 @@ export async function applyMcpServers(
  */
 export async function applyCodexMcpServers(
   bundle,
-  { logger, nonInteractive = false, resolveBinImpl, runImpl } = {},
+  { agentId, agentToken, logger, nonInteractive = false, resolveBinImpl, runImpl } = {},
 ) {
   const servers = Array.isArray(bundle?.mcp_servers) ? bundle.mcp_servers : []
   if (servers.length === 0) {
@@ -512,25 +512,34 @@ export async function applyCodexMcpServers(
     }
   }
 
-  // MCP 承認モードを `~/.codex/config.toml` に反映する（2026-07-15）。
+  // MCP 承認モードと Hub のホスト資格ヘッダを `~/.codex/config.toml` に反映する。
   // ⚠️ `codex mcp add` には承認モードの永続設定オプションが無いため、Bundle が宣言した
   // `default_tools_approval_mode` を config.toml の `[mcp_servers.<name>]` テーブルへ直接
-  // マージする。対話 add の成否に関わらず全 target に対して実行する（add が「already exists」で
-  // 失敗しても承認設定は更新する要件）。非対話モードでは onlyExisting=true で「既存テーブルの承認モード
-  // 補完」だけを行い、未登録 server には新規テーブルを作らず pending 報告する。I/O・ネットワーク失敗
-  // （transient）は approvalError として上位（syncBundle）へ返し、state に「再試行すべき失敗」として
-  // 記録させる。config 破壊は _atomicWriteFilePreserving が防ぐ。
+  // マージする。また Codex CLI 0.145.0 時点では `mcp add --header` が無いため、Claude と同じ
+  // `X-Cockpit-Agent` を公式設定 `http_headers` へ直接マージする。ヘッダは `hub` / `hub_staff`
+  // の両方だけに付け、外部 MCP には付けない。
+  //
+  // 対話 add の成否に関わらず全 target に対して実行する（add が「already exists」で失敗しても
+  // 設定は更新する要件）。非対話モードでは onlyExisting=true で「既存テーブルの補完」だけを行い、
+  // 未登録 server には新規テーブルを作らず pending 報告する。I/O・ネットワーク失敗（transient）は
+  // approvalError として上位（syncBundle）へ返し、state に「再試行すべき失敗」として記録させる。
+  // config 破壊は _atomicWriteFilePreserving が防ぐ。
   let approvalUpdated = []
   let approvalPending = []
   let approvalSkipped = []
   let approvalError = null
   try {
-    const res = await applyCodexMcpApprovalModes(servers, { logger, onlyExisting: nonInteractive })
+    const res = await applyCodexMcpApprovalModes(servers, {
+      agentId,
+      agentToken,
+      logger,
+      onlyExisting: nonInteractive,
+    })
     approvalUpdated = res.updated
     approvalPending = res.pending || []
     approvalSkipped = res.skipped || []
   } catch (err) {
-    logger?.warn({ err: err.message }, "codex MCP approval mode の config.toml 反映に失敗")
+    logger?.warn({ err: err.message }, "codex MCP 設定の config.toml 反映に失敗")
     approvalError = err?.message || String(err)
   }
   return { added, skipped, approvalUpdated, approvalPending, approvalSkipped, approvalError }
@@ -554,7 +563,8 @@ function resolveCodexConfigPath() {
 
 /**
  * config.toml テキストへ、`[mcp_servers.<name>]` テーブルの `url` /
- * `default_tools_approval_mode` を**テーブル単位でサージカルに** upsert して返す（純関数）。
+ * `http_headers.X-Cockpit-Agent` / `default_tools_approval_mode` を**テーブル単位で
+ * サージカルに** upsert して返す（純関数）。
  *
  * 設計（config 全体を再生成しない・既存を壊さない）:
  *  - 対象テーブルの行範囲（ヘッダ行の次〜次の `[...]` テーブルヘッダ直前）だけを編集する。
@@ -567,18 +577,24 @@ function resolveCodexConfigPath() {
  *  - 不正な approvalMode は無視（url のみ・または no-op）。改行コード（LF/CRLF）は維持する。
  *
  * @param {string} configText
- * @param {{name: string, url?: string, approvalMode?: string, onlyExisting?: boolean}} opts
+ * @param {{name: string, url?: string, approvalMode?: string, hostCredential?: string, onlyExisting?: boolean}} opts
  *   onlyExisting=true のとき、標準テーブルが**まだ無い** server には新規テーブルを追記しない
  *   （config を無変更で返す）。自動同期（非対話）で使う: 未登録 MCP に承認モードだけ書くと
  *   OAuth 未認証の「半登録テーブル」を作ってしまうため、登録は対話経路（`codex mcp add`）に
  *   限定し、自動同期は既存テーブルの承認モード補完だけを行う。
  * @returns {string}
  */
-export function upsertCodexMcpApprovalMode(configText, { name, url, approvalMode, onlyExisting = false } = {}) {
+export function upsertCodexMcpApprovalMode(
+  configText,
+  { name, url, approvalMode, hostCredential, onlyExisting = false } = {},
+) {
   if (!name) return configText
   const mode = _CODEX_MCP_APPROVAL_MODES.has(approvalMode) ? approvalMode : null
   const kv = []
   if (typeof url === "string" && url) kv.push(["url", JSON.stringify(url)])
+  if (typeof hostCredential === "string" && hostCredential) {
+    kv.push(["http_headers", `{ "X-Cockpit-Agent" = ${JSON.stringify(hostCredential)} }`])
+  }
   if (mode) kv.push(["default_tools_approval_mode", JSON.stringify(mode)])
   if (kv.length === 0) return configText
 
@@ -634,8 +650,22 @@ export function upsertCodexMcpApprovalMode(configText, { name, url, approvalMode
       }
     }
     if (found >= 0) {
-      // 値だけ差し替え、インデント・キー左辺・**行末インラインコメント**を保持する。
-      lines[found] = _replaceTomlStringValue(lines[found], key, valStr)
+      if (key === "http_headers") {
+        // `http_headers` はユーザーが別のヘッダを持ち得る。行全体を置換せず、管理対象の
+        // X-Cockpit-Agent だけを inline table へマージする。安全に解釈できない形式なら
+        // config 全体を無変更で返し、呼び出し側の検証で skipped 扱いにする。
+        const merged = _upsertTomlInlineStringMapValue(
+          lines[found],
+          "http_headers",
+          "X-Cockpit-Agent",
+          JSON.stringify(hostCredential),
+        )
+        if (merged === null) return configText
+        lines[found] = merged
+      } else {
+        // 値だけ差し替え、インデント・キー左辺・**行末インラインコメント**を保持する。
+        lines[found] = _replaceTomlStringValue(lines[found], key, valStr)
+      }
       if (key === "url") urlLineIdx = found
     } else {
       toInsert.push(`${key} = ${valStr}`)
@@ -651,8 +681,8 @@ export function upsertCodexMcpApprovalMode(configText, { name, url, approvalMode
 }
 
 /**
- * Bundle の mcp_servers のうち承認モードを宣言したものについて、`~/.codex/config.toml` の
- * 該当テーブルへ `default_tools_approval_mode`（と url）を安全マージする。
+ * Bundle の mcp_servers のうち承認モードを宣言したもの、またはホスト資格を付ける Hub MCP
+ * について、`~/.codex/config.toml` の該当テーブルへ承認モード・url・http_headers を安全マージする。
  *
  *  - 承認モードが許可値でない server はスキップ（不正値を端末設定へ流さない）。
  *  - 変更が無ければ書き込まない（完全冪等）。
@@ -660,7 +690,7 @@ export function upsertCodexMcpApprovalMode(configText, { name, url, approvalMode
  *    残さない）。rename が失敗した場合は破壊的フォールバックを行わず中止し、既存 config を無変更で維持する。
  *
  * @param {Array<{name?: string, url?: string, default_tools_approval_mode?: string}>} servers
- * @param {{configPath?: string, logger?: any, fsImpl?: any, isWindows?: boolean, onlyExisting?: boolean}} [opts]
+ * @param {{configPath?: string, logger?: any, fsImpl?: any, isWindows?: boolean, onlyExisting?: boolean, agentId?: string, agentToken?: string}} [opts]
  *   configPath 省略時は resolveCodexConfigPath()。fsImpl / isWindows はテスト用の注入点（既定は実 fs / 実 OS）。
  *   onlyExisting=true（自動同期・非対話）のとき、標準テーブルが**まだ無い** server には新規
  *   テーブルを追記せず `pending` として報告する（未登録 MCP に承認モードだけ書いて OAuth 未認証の
@@ -670,10 +700,22 @@ export function upsertCodexMcpApprovalMode(configText, { name, url, approvalMode
  */
 export async function applyCodexMcpApprovalModes(
   servers,
-  { configPath, logger, fsImpl = fs, isWindows = process.platform === "win32", onlyExisting = false } = {},
+  {
+    configPath,
+    logger,
+    fsImpl = fs,
+    isWindows = process.platform === "win32",
+    onlyExisting = false,
+    agentId,
+    agentToken,
+  } = {},
 ) {
+  const hostCredential = agentId && agentToken ? `${agentId}:${agentToken}` : null
   const targets = (Array.isArray(servers) ? servers : []).filter(
-    (s) => s?.name && _CODEX_MCP_APPROVAL_MODES.has(s?.default_tools_approval_mode),
+    (s) =>
+      s?.name &&
+      (_CODEX_MCP_APPROVAL_MODES.has(s?.default_tools_approval_mode) ||
+        (hostCredential && HUB_MCP_NAMES.has(s.name))),
   )
   if (targets.length === 0) return { updated: [], skipped: [], pending: [] }
 
@@ -696,6 +738,7 @@ export async function applyCodexMcpApprovalModes(
   const skipped = []
   const pending = []
   for (const s of targets) {
+    const serverHostCredential = hostCredential && HUB_MCP_NAMES.has(s.name) ? hostCredential : null
     // 自動同期（onlyExisting）で標準テーブルが無い server は、新規テーブルを作らない。
     // 非標準形（空白入り/ドット記法/インライン）で既に定義されていれば「安全に編集できない」
     // → skipped、真に未登録なら「登録が必要」→ pending として区別する（ブラウザは開かない）。
@@ -719,12 +762,13 @@ export async function applyCodexMcpApprovalModes(
       name: s.name,
       url: s.url,
       approvalMode: s.default_tools_approval_mode,
+      hostCredential: serverHostCredential,
       onlyExisting,
     })
     if (candidate === next) {
       // 変化なし。原因が「既に正しく反映済み（＝ no-op）」か「非標準形で upsert が編集を拒否した」
       // かを検証で区別する。標準テーブルが健全なら no-op、そうでなければ適用不能としてスキップ報告。
-      if (!_verifyCodexServerTable(next, s.name, s.default_tools_approval_mode)) {
+      if (!_verifyCodexServerTable(next, s.name, s.default_tools_approval_mode, serverHostCredential)) {
         logger?.warn(
           { name: s.name },
           "codex config.toml の該当テーブルを安全に編集できないためスキップ（既存 config は不変）",
@@ -737,7 +781,7 @@ export async function applyCodexMcpApprovalModes(
     // 期待値でちょうど 1 つ・url は高々 1 つ・他形式（空白入り/インライン）の重複定義が無い」
     // ことを検証する。素の文字列編集が非標準 config で誤動作しても、ここで弾いて **既存 config を
     // 一切書き換えない**（worst case = 承認設定が付かない＝現状維持）。
-    if (!_verifyCodexServerTable(candidate, s.name, s.default_tools_approval_mode)) {
+    if (!_verifyCodexServerTable(candidate, s.name, s.default_tools_approval_mode, serverHostCredential)) {
       logger?.warn(
         { name: s.name },
         "codex config.toml の該当テーブルを安全に編集できないためスキップ（既存 config は不変）",
@@ -752,7 +796,7 @@ export async function applyCodexMcpApprovalModes(
 
   // ⚠️ 失敗しても既存 config.toml を必ず維持する fail-safe な atomic write。
   await _atomicWriteFilePreserving(cfgPath, next, fileMode, { fsImpl, isWindows, logger })
-  logger?.info({ updated, skipped, pending }, "codex MCP approval mode を config.toml へ反映")
+  logger?.info({ updated, skipped, pending }, "codex MCP 設定を config.toml へ反映")
   return { updated, skipped, pending }
 }
 
@@ -862,6 +906,81 @@ function _replaceTomlStringValue(line, key, quotedVal) {
   return `${indent}${key} = ${quotedVal}`
 }
 
+/**
+ * TOML inline table の string entry を 1 件だけ upsert する。
+ *
+ * 例: `http_headers = { "Authorization" = "..." } # keep` に
+ * `"X-Cockpit-Agent" = "ag:token"` を追加する。既存 entry・行末コメント・他ヘッダは保持する。
+ * inline table として安全に読めない場合は null（呼び出し側は config を変更しない）。
+ */
+function _upsertTomlInlineStringMapValue(line, key, entryKey, quotedVal) {
+  const keyRe = new RegExp(`^(\\s*${_escapeRegExp(key)}\\s*=\\s*)\\{`)
+  const prefixMatch = line.match(keyRe)
+  if (!prefixMatch) return null
+
+  // `#` は TOML 文字列の外側だけコメント開始。コメント内の `}` を inline table の終端と
+  // 誤認しないよう、quote / escape を追跡してコード部分を切り出す。
+  let inDouble = false
+  let inSingle = false
+  let escaped = false
+  let commentIdx = line.length
+  for (let i = prefixMatch[0].length; i < line.length; i++) {
+    const ch = line[i]
+    if (escaped) {
+      escaped = false
+      continue
+    }
+    if (inDouble && ch === "\\") {
+      escaped = true
+      continue
+    }
+    if (!inSingle && ch === '"') inDouble = !inDouble
+    else if (!inDouble && ch === "'") inSingle = !inSingle
+    else if (!inDouble && !inSingle && ch === "#") {
+      commentIdx = i
+      break
+    }
+  }
+  if (inDouble || inSingle || escaped) return null
+
+  const code = line.slice(0, commentIdx)
+  const openIdx = prefixMatch[0].lastIndexOf("{")
+  const closeIdx = code.lastIndexOf("}")
+  if (closeIdx <= openIdx || code.slice(closeIdx + 1).trim() !== "") return null
+
+  const body = code.slice(openIdx + 1, closeIdx)
+  const entryRe = new RegExp(
+    `((?:"${_escapeRegExp(entryKey)}"|${_escapeRegExp(entryKey)})\\s*=\\s*)"(?:[^"\\\\]|\\\\.)*"`,
+  )
+  let nextBody
+  if (entryRe.test(body)) {
+    nextBody = body.replace(entryRe, (_whole, lhs) => `${lhs}${quotedVal}`)
+  } else {
+    const trailing = body.match(/\\s*$/)?.[0] || ""
+    const content = body.slice(0, body.length - trailing.length)
+    const entry = `${JSON.stringify(entryKey)} = ${quotedVal}`
+    nextBody = content.trim() ? `${content}, ${entry}${trailing}` : ` ${entry} `
+  }
+  return `${line.slice(0, openIdx + 1)}${nextBody}${line.slice(closeIdx)}`
+}
+
+/** 対象 inline table 内の string entry の raw quoted value と出現数を返す。 */
+function _readTomlInlineStringMapValue(line, key, entryKey) {
+  const keyRe = new RegExp(`^\\s*${_escapeRegExp(key)}\\s*=\\s*\\{(.*)\\}\\s*(?:#.*)?$`)
+  const m = line.match(keyRe)
+  if (!m) return { count: 0, value: null, valid: false }
+  const entryRe = new RegExp(
+    `(?:"${_escapeRegExp(entryKey)}"|${_escapeRegExp(entryKey)})\\s*=\\s*("(?:[^"\\\\]|\\\\.)*")`,
+    "g",
+  )
+  const matches = [...m[1].matchAll(entryRe)]
+  return {
+    count: matches.length,
+    value: matches.length === 1 ? matches[0][1] : null,
+    valid: true,
+  }
+}
+
 /**
  * マージ結果の config.toml テキストで、`name` の MCP サーバーテーブルが**安全に**編集された
  * 状態か検証する（書き込み前のフェイルセーフ）。以下を全て満たすときのみ true:
@@ -872,7 +991,7 @@ function _replaceTomlStringValue(line, key, quotedVal) {
  *  - `url` は高々 1 つ（重複していない）。
  * いずれか外れたら false（＝呼び出し側は書き込まず既存 config を保持する）。
  */
-function _verifyCodexServerTable(text, name, mode) {
+function _verifyCodexServerTable(text, name, mode, hostCredential = null) {
   const n = _escapeRegExp(name)
   const lines = text.split(/\r?\n/)
   const bracketRe = new RegExp(`^\\s*\\[\\s*mcp_servers\\s*\\.\\s*"?${n}"?\\s*\\]\\s*$`)
@@ -904,8 +1023,10 @@ function _verifyCodexServerTable(text, name, mode) {
   }
   let modeCount = 0
   let urlCount = 0
+  let httpHeadersCount = 0
   let modeOk = false
-  const expectedVal = JSON.stringify(mode) // 例: "approve"
+  const expectedVal = _CODEX_MCP_APPROVAL_MODES.has(mode) ? JSON.stringify(mode) : null // 例: "approve"
+  let hostHeaderOk = !hostCredential
   // 値（クォート文字列）だけを解析し、行末インラインコメント（` # ...`）は許容する。
   // 行全体の厳密一致にすると `default_tools_approval_mode = "approve" # keep` を弾いてしまい、
   // upsert が正しく更新しても書き込みがスキップされて古い値が残る（2026-07-15 レビュー指摘）。
@@ -917,8 +1038,20 @@ function _verifyCodexServerTable(text, name, mode) {
       if (m && m[1] === expectedVal) modeOk = true
     }
     if (/^\s*url\s*=/.test(lines[i])) urlCount++
+    if (/^\s*http_headers\s*=/.test(lines[i])) {
+      httpHeadersCount++
+      if (hostCredential) {
+        const parsed = _readTomlInlineStringMapValue(lines[i], "http_headers", "X-Cockpit-Agent")
+        hostHeaderOk =
+          parsed.valid && parsed.count === 1 && parsed.value === JSON.stringify(hostCredential)
+      }
+    }
   }
-  return modeCount === 1 && modeOk && urlCount <= 1
+  const modeValid = expectedVal ? modeCount === 1 && modeOk : modeCount <= 1
+  const headersValid = hostCredential
+    ? httpHeadersCount === 1 && hostHeaderOk
+    : httpHeadersCount <= 1
+  return modeValid && headersValid && urlCount <= 1
 }
 
 /**
@@ -1384,7 +1517,7 @@ async function _syncBundleLocked({ hubUrl, agentId, agentToken, logger, fetchImp
   let mcp = { added: [], skipped: [] }
   try {
     mcp = cliKind === "codex"
-      ? await applyCodexMcpServers(bundle, { logger, nonInteractive })
+      ? await applyCodexMcpServers(bundle, { agentId, agentToken, logger, nonInteractive })
       : await applyMcpServers(bundle, { agentId, agentToken, logger, nonInteractive })
   } catch (err) {
     logger?.warn({ cliKind, err: err.message }, "applyMcpServers failed (continuing)")