@cocorograph/hub-agent 0.7.67 → 0.7.69

package/src/main.mjs

@@ -20,6 +20,17 @@ import path from "node:path"
 import pino from "pino"
 
 import { detectAvailableClis, detectPlatform, readConfig, writeConfig } from "./config.mjs"
+import {
+  codexTerminalSkipVersion,
+  compareBundleVersions,
+  readCliBundleVersionSync,
+  readCliInstalledVersionSync,
+  readCodexApprovalStatusSync,
+  readCodexConfigText,
+  readHubAiState,
+  resolveSyncTargets,
+  syncBundleTargets,
+} from "./hub-bundle.mjs"
 import { extractLastAssistantText, pickContextText } from "./extract-paragraph.mjs"
 import { loadPlugins, runHookBroadcast, runHookChain } from "./plugin-loader.mjs"
 import { WsClient } from "./ws-client.mjs"
@@ -496,6 +507,152 @@ export async function kickSetupHubAiForDetectedClis({
   return targets
 }
 
+// ── bundle.sync.request（backend server-driven 自動同期）の状態管理 ──────────────
+//
+// backend が heartbeat の `bundle_version` とリモート最新版の差分を検知すると agent へ
+// `bundle.sync.request` を送る。agent はそれを受けて **非対話** で syncBundleTargets を実行する
+// （files/hooks/setup + codex 承認モード補完。codex mcp add / OAuth は呼ばない）。
+//
+// 連続 heartbeat ごとに full bundle を取りに行かないよう、以下のガードを重ねる:
+//  - running: 同期中の再入を弾く
+//  - debounce: 直近 _BUNDLE_SYNC_MIN_INTERVAL_MS 以内の要求は間引く
+//  - version-guard: 要求 version が既に files+承認とも反映済みなら fetch せず skip（冪等・無駄取得回避）
+//  - backoff: 失敗が続くほど次回まで待つ（指数、上限あり）
+//  - auto_update:false（.hub_ai_state.json）を尊重し自動同期しない
+const _BUNDLE_SYNC_MIN_INTERVAL_MS = 60_000
+const _BUNDLE_SYNC_BACKOFF_BASE_MS = 30_000
+const _BUNDLE_SYNC_BACKOFF_MAX_MS = 10 * 60_000
+const _bundleSyncState = { running: false, lastAttemptAt: 0, lastFailureAt: 0, failureCount: 0 }
+
+/** テスト用: 自動同期のガード状態をリセットする。 */
+export function _resetBundleSyncStateForTest() {
+  _bundleSyncState.running = false
+  _bundleSyncState.lastAttemptAt = 0
+  _bundleSyncState.lastFailureAt = 0
+  _bundleSyncState.failureCount = 0
+}
+
+/**
+ * ある target（cli_kind）について、要求 version が既に反映済みかを判定する。
+ *  - codex: mcp_approval_applied_version（無ければ installed_version）が要求以上
+ *  - claude: installed_version が要求以上
+ * auto_update:false のときは「対象外（＝要求は満たされたものとして扱う＝同期しない）」を返す。
+ */
+async function _bundleTargetSatisfied(cliKind, requestedVersion, readStateImpl, readConfigImpl) {
+  const st = (await (readStateImpl || readHubAiState)(cliKind)) || {}
+  if (st.auto_update === false) return true // opt-out は「同期不要」扱い
+  if (!requestedVersion) return false
+  const filesVer = st.installed_version || "0.0.0"
+  if (compareBundleVersions(filesVer, requestedVersion) < 0) return false
+  if (cliKind === "codex") {
+    const approvalVer = st.mcp_approval_applied_version || "0.0.0"
+    const approved = compareBundleVersions(approvalVer, requestedVersion) >= 0
+    // 恒久 skip は heartbeat と同一の共通判定（codexTerminalSkipVersion）で **fingerprint 検証済み**
+    // のときだけ有効。config が変わっている / io にリセットされた skip は無効化され、再試行される。
+    const configText = await (readConfigImpl || readCodexConfigText)()
+    const skipV = codexTerminalSkipVersion(st, configText)
+    const terminallySkipped = skipV ? compareBundleVersions(skipV, requestedVersion) >= 0 : false
+    if (!approved && !terminallySkipped) return false
+  }
+  return true
+}
+
+/**
+ * `bundle.sync.request` を処理する。非対話 syncBundleTargets を debounce/version-guard/backoff 付きで実行。
+ * 主経路（daemon による事前同期）。テスト注入点: syncTargetsImpl / detectImpl / readStateImpl / nowMs。
+ */
+export async function handleBundleSyncRequest(msg, ctx, opts = {}) {
+  const logger = ctx?.logger
+  const requestedVersion = typeof msg?.version === "string" ? msg.version : null
+  const now = typeof opts.nowMs === "number" ? opts.nowMs : Date.now()
+
+  // ⚠️ 再入防止フラグは **await をまたぐ前に同期的に** claim する。await を挟んでから running を
+  // 立てると、fire-and-forget で並んだ 2 本目が running を false のまま通過して二重 sync する
+  // （dispatch の bundle.sync.request は直列キューを塞がない fire-and-forget のため実際に競合しうる）。
+  if (_bundleSyncState.running) {
+    logger?.info?.("bundle.sync.request: 既に同期中のため skip")
+    return { skipped: "running" }
+  }
+  _bundleSyncState.running = true
+  try {
+    if (now - _bundleSyncState.lastAttemptAt < _BUNDLE_SYNC_MIN_INTERVAL_MS) {
+      logger?.debug?.("bundle.sync.request: debounce により skip")
+      return { skipped: "debounced" }
+    }
+    if (_bundleSyncState.failureCount > 0) {
+      const backoff = Math.min(
+        _BUNDLE_SYNC_BACKOFF_BASE_MS * 2 ** (_bundleSyncState.failureCount - 1),
+        _BUNDLE_SYNC_BACKOFF_MAX_MS,
+      )
+      if (now - _bundleSyncState.lastFailureAt < backoff) {
+        logger?.debug?.({ backoff }, "bundle.sync.request: backoff 中のため skip")
+        return { skipped: "backoff" }
+      }
+    }
+
+    const config = ctx?.config
+    if (!config?.hub_url || !config?.agent_id || !config?.agent_token) {
+      logger?.warn?.("bundle.sync.request: agent 設定が不足しているため skip")
+      return { skipped: "no_config" }
+    }
+
+    const detect = opts.detectImpl || detectAvailableClis
+    const readStateImpl = opts.readStateImpl || readHubAiState
+    let targets
+    try {
+      targets = resolveSyncTargets(await detect())
+    } catch {
+      targets = ["claude"]
+    }
+
+    // version-guard: 全 target が要求 version 反映済みなら fetch せず skip（冪等・無駄取得回避）。
+    const satisfied = await Promise.all(
+      targets.map((c) => _bundleTargetSatisfied(c, requestedVersion, readStateImpl, opts.readConfigImpl)),
+    )
+    if (satisfied.every(Boolean)) {
+      logger?.info?.({ requestedVersion }, "bundle.sync.request: 既に反映済みのため skip（fetch しない）")
+      // 反映済みなら試行時刻だけ進めて過剰 debounce 判定を安定させる（失敗ではないので backoff は触らない）。
+      _bundleSyncState.lastAttemptAt = now
+      return { skipped: "already_satisfied" }
+    }
+
+    _bundleSyncState.lastAttemptAt = now
+    try {
+      const syncTargets = opts.syncTargetsImpl || syncBundleTargets
+      const { results, failures } = await syncTargets({
+        targets,
+        hubUrl: config.hub_url,
+        agentId: config.agent_id,
+        agentToken: config.agent_token,
+        logger,
+        nonInteractive: true, // ⚠️ 自動同期は完全非対話（codex mcp add / OAuth を呼ばない）
+      })
+      if (failures.length > 0) {
+        _bundleSyncState.failureCount += 1
+        _bundleSyncState.lastFailureAt = Date.now()
+      } else {
+        _bundleSyncState.failureCount = 0
+      }
+      // 観測用に結果を backend へ返す（次回 heartbeat で bundle_version も更新される）。
+      ctx?.client?.send?.({
+        type: "bundle.sync.result",
+        version: requestedVersion,
+        ok: failures.length === 0,
+        failures,
+      })
+      logger?.info?.({ requestedVersion, failures }, "bundle.sync.request 完了")
+      return { results, failures }
+    } catch (err) {
+      _bundleSyncState.failureCount += 1
+      _bundleSyncState.lastFailureAt = Date.now()
+      logger?.warn?.({ err: err?.message }, "bundle.sync.request 実行に失敗")
+      return { error: err?.message || String(err) }
+    }
+  } finally {
+    _bundleSyncState.running = false
+  }
+}
+
 export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
   const config = await readConfig()
   if (!config) {
@@ -597,6 +754,27 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     logger.info({ bundleVersion }, "hub bundle version detected")
   }
 
+  // CLI 別（claude / codex）の bundle version + codex 承認反映状況を毎 heartbeat 報告する provider。
+  // backend が per-CLI に stale / 承認未反映を判定して bundle.sync.request を送る（Codex-only 環境や
+  // 「Claude 最新・Codex 承認未反映」でも正しく検知させるため。~/.claude の manifest だけに依存しない）。
+  const bundleStatusProvider = () => {
+    const codexApproval = readCodexApprovalStatusSync()
+    return {
+      // ⚠️ backend の「反映済み」判定に使う version は **setup 完了済み** の installed_version（manifest
+      // ではない。manifest は setup 前に配置されるため、setup 失敗を「適用済み」と誤報告してしまう）。
+      bundle_versions: {
+        claude: readCliInstalledVersionSync("claude"),
+        codex: readCliInstalledVersionSync("codex"),
+      },
+      // 参考: 配置済み（setup 完了とは限らない）manifest version。判定には使わない（観測用）。
+      downloaded_versions: {
+        claude: readCliBundleVersionSync("claude"),
+        codex: readCliBundleVersionSync("codex"),
+      },
+      codex_approval: codexApproval,
+    }
+  }
+
   const client = new WsClient(config, {
     logger,
     version,
@@ -604,6 +782,7 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     availableClis,
     bundleVersion,
     bundleVersionProvider: readBundleVersionSync,
+    bundleStatusProvider,
     bundleManifestPath: BUNDLE_MANIFEST_PATH,
   })
 
@@ -2757,6 +2936,15 @@ async function dispatch(msg, ctx) {
       case "error":
         ctx.logger.warn({ msg }, "hub error")
         return
+      case "bundle.sync.request": {
+        // backend が heartbeat の bundle_version 差分を検知して送る自動同期要求。
+        // 非対話 syncBundleTargets を debounce/version-guard/backoff/lock 付きで実行する
+        // （codex mcp add / OAuth は呼ばない）。fire-and-forget で dispatch を塞がない。
+        handleBundleSyncRequest(msg, ctx).catch((err) => {
+          ctx.logger?.warn?.({ err: err?.message }, "handleBundleSyncRequest threw")
+        })
+        return
+      }
       case "agent.shutdown": {
         // Cockpit の「エージェント削除 (ゴミ箱)」由来。この端末をエージェントとして
         // 撤去する: サービス停止・無効化・autostart 撤去のみ。ユーザーデータ

package/src/sibling-inject.mjs

@@ -112,9 +112,13 @@ export class SiblingInjectBridge {
       const sent = bridge?.sendToCwd?.(cwd, text) ?? false
       result = sent
         ? { ok: true }
-        : { ok: false, error: "no_live_session_for_cwd" }
+        : { ok: false, error: "no_live_session_for_cwd", ...this._diagnose(cwd, target_cli_kind) }
     }
 
+    this.logger?.[result.ok ? "info" : "warn"]?.(
+      { request_id, cwd, target_cli_kind, ...(result.ok ? {} : { error: result.error, live_sessions: result.live_sessions }) },
+      result.ok ? "sibling-inject delivered" : "sibling-inject failed",
+    )
     await this._writeResult(request_id, result)
     // 要求ファイルは処理後に必ず削除する (成否問わず、再処理させないため)。
     try {
@@ -124,6 +128,32 @@ export class SiblingInjectBridge {
     }
   }
 
+  /**
+   * `no_live_session_for_cwd` に「一時的か恒久的か」「TUI と混同していないか」を
+   * 切り分ける材料を添える (2026-07-17: エラー文字列だけでは (a) 1 回失敗 → 再送で成功
+   * (b) 3 回失敗 → 本当に消滅 の区別が付かず誤判断した)。
+   *
+   * ⚠️ ここに出るのは hub-agent が spawn した **チャットセッションのみ**。tmux 配下で
+   * 動く TUI codex は ask-sibling の宛先にならないため、一覧が空でも「codex は画面上
+   * 動いている」ことは普通に起きる。その旨を hint に明記する。
+   * @param {string} cwd
+   * @param {string} targetKind
+   */
+  _diagnose(cwd, targetKind) {
+    const all = [
+      ...(this.claudeBridge?.describeLiveSessions?.() ?? []),
+      ...(this.codexBridge?.describeLiveSessions?.() ?? []),
+    ]
+    const sameCwd = all.filter((s) => s.cwd === cwd)
+    const hint = sameCwd.length
+      ? `この cwd には ${sameCwd.map((s) => s.kind).join("/")} のチャットセッションはあるが ${targetKind} は無い。`
+      : `この cwd には hub-agent 管理下のチャットセッションが 1 本も無い。` +
+        `tmux の TUI codex は ask-sibling の宛先にならない (hub-agent が spawn した` +
+        ` Cockpit のチャットセッションのみが対象)。Cockpit のブラウザで対象 cwd の` +
+        ` ${targetKind} チャットを開き直してから再送すること。`
+    return { live_sessions: all, hint }
+  }
+
   /** 結果ファイルを atomic write する (呼び出し側 CLI がポーリングする)。 */
   async _writeResult(request_id, result) {
     const fp = path.join(this.dir, `${request_id}.result.json`)

package/src/ws-client.mjs

@@ -119,6 +119,12 @@ export class WsClient extends EventEmitter {
       typeof opts.bundleVersionProvider === "function"
         ? opts.bundleVersionProvider
         : null
+    // CLI 別（claude / codex）の bundle version と codex 承認反映状況を返す provider。
+    // backend が per-CLI に stale/承認未反映を判定して bundle.sync.request を送るために報告する。
+    // 返り値例: { bundle_versions: {claude, codex}, codex_approval: {applied, skipped} }
+    this.bundleStatusProvider =
+      typeof opts.bundleStatusProvider === "function" ? opts.bundleStatusProvider : null
+    this.bundleStatus = {}
     this.bundleManifestPath = opts.bundleManifestPath || null
     this.bundleWatcher = null
     this.bundleWatchDebounceTimer = null
@@ -230,8 +236,9 @@ export class WsClient extends EventEmitter {
     // 同期は backend が承認するため hello 送信後の同期 send_json で問題ない。
     // ただし flush は hello 直後ではなく streams.sync.request の後に置く方が
     // 安全 (sync 完了で stream_id がサーバに復元される想定)。
-    // hello 前に最新の bundle version を読み直す (再接続時の鮮度確保)
+    // hello 前に最新の bundle version / CLI 別ステータスを読み直す (再接続時の鮮度確保)
     this._refreshBundleVersion()
+    this._refreshBundleStatus()
     this._sendJson({
       type: "hello",
       agent_id: this.config.agent_id,
@@ -240,6 +247,7 @@ export class WsClient extends EventEmitter {
       bundle_version: this.bundleVersion,
       platform: this.platform,
       available_clis: this.availableClis,
+      ...this.bundleStatus,
     })
     // Plan ε: 毎回 reconnect 直後に backend ↔ agent の stream_id を能動同期する
     // (orphan stream の即時 kill 用)。response は main.mjs の dispatch が拾い、
@@ -446,9 +454,10 @@ export class WsClient extends EventEmitter {
         this._forceReconnect()
         return
       }
-      // heartbeat 都度 provider 経由で bundle version を最新化する。
+      // heartbeat 都度 provider 経由で bundle version / CLI 別ステータスを最新化する。
       // fs.watch を取り逃した変更 (atomic rename / 別マウント越し等) への保険。
       this._refreshBundleVersion()
+      this._refreshBundleStatus()
       // 送信失敗 (= ws not OPEN) を検知したら即時 close + reconnect 発火。
       // setInterval を放置すると死んだコネクションに heartbeat を投げ続けて
       // 30s 間気付かないので、close を強制トリガーする。
@@ -456,6 +465,7 @@ export class WsClient extends EventEmitter {
         type: "heartbeat",
         uptime_sec: Math.floor((Date.now() - this.startedAt) / 1000),
         bundle_version: this.bundleVersion,
+        ...this.bundleStatus,
       })
       if (!ok) {
         this.logger?.warn("heartbeat send failed, forcing reconnect")
@@ -495,6 +505,21 @@ export class WsClient extends EventEmitter {
     }
   }
 
+  /**
+   * CLI 別 bundle version + codex 承認状況を provider から読み直して `this.bundleStatus` を更新する。
+   * hello / heartbeat に spread して backend へ報告する（backend は per-CLI に stale 判定する）。
+   * 失敗しても空オブジェクトのまま（旧 backend / provider 未設定でも従来どおり動く）。
+   */
+  _refreshBundleStatus() {
+    if (!this.bundleStatusProvider) return
+    try {
+      const s = this.bundleStatusProvider()
+      this.bundleStatus = s && typeof s === "object" ? s : {}
+    } catch (err) {
+      this.logger?.warn({ err: err.message }, "bundle status refresh failed")
+    }
+  }
+
   /**
    * `bundleManifestPath` を fs.watch で監視し、変更時に即時 heartbeat を送る。
    * Cockpit UI への反映ラグを 30s heartbeat 周期から ~15s (UI poll) に短縮する。
@@ -553,10 +578,12 @@ export class WsClient extends EventEmitter {
       this.bundleWatchDebounceTimer = null
       const changed = this._refreshBundleVersion()
       if (!changed) return
+      this._refreshBundleStatus()
       const ok = this._sendJson({
         type: "heartbeat",
         uptime_sec: Math.floor((Date.now() - this.startedAt) / 1000),
         bundle_version: this.bundleVersion,
+        ...this.bundleStatus,
       })
       if (!ok) {
         this.logger?.warn("bundle-change heartbeat send failed, forcing reconnect")