@cocorograph/hub-agent 0.7.35 → 0.7.37

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -1134,6 +1134,24 @@ class ClaudeStreamSession {
         /* ignore */
       }
       this._busy = false
+      // T05699 残タスク #2-1 (2026-07-04): interrupt() は現ターンの result を発火しない
+      // (query は継続、次 input 受付継続のみ)。そのため abort 時点で _pendingMessages に
+      // 溜まっている送信待ちメッセージは、result 経由の drain (_runResidentQuery 内の
+      // result ハンドラ) を永久に迎えられず、_busy=false のまま取り残されていた。この
+      // 状態で直後に次の sendMessage が届くと (_busy===false のため) pending をすっ飛ばして
+      // _inputQueue へ直接 push してしまい、先に積んだメッセージより後着メッセージが
+      // 先に処理される FIFO 順序崩れが起きる。ここで明示的に drain し、pending があれば
+      // 同一 query 内で継続処理させる (無ければ _emitQueueState() のみで no-op)。
+      // _drainResidentPending は async だが最初の await 前に _busy=true を同期的に
+      // セットするため、この関数の呼び出し (await しない fire-and-forget) が完了した
+      // 時点 (= abortTurn() の同期実行が終わるまでの同一 tick 内) で以降の sendMessage は
+      // 正しく busy 状態を観測できる。
+      this._drainResidentPending().catch((err) =>
+        this.logger?.warn(
+          { stream_id: this.stream_id, err: err?.message },
+          "drainResidentPending (abort) threw",
+        ),
+      )
       return
     }
     if (this._abortController) {

package/src/codex-stream-bridge.mjs

@@ -300,6 +300,23 @@ export class CodexStreamBridge extends EventEmitter {
     this._maybeReapAfterIdle(session)
   }
 
+  /**
+   * 指定 cwd に一致するライブ (attach 済み) codex セッションの threadId を返す。
+   * Claude 版 `liveSessionIdForCwd` の対 (main.mjs の state loop / terminal mirror から
+   * cwd → threadId 解決に使う)。複数一致時は先に見つかったものを返す (5h/7d はアカウント
+   * 全体の値のためどのセッションから読んでも同じ契約の残量、代表 1 件で十分)。
+   *
+   * @param {string} cwd
+   * @returns {string|null}
+   */
+  threadIdForCwd(cwd) {
+    if (!cwd) return null
+    for (const session of this.sessions.values()) {
+      if (session.cwd === cwd && session.threadId) return session.threadId
+    }
+    return null
+  }
+
   _maybeReapAfterIdle(session) {
     if (session.activeTurnId) return // 完走まで待つ (turn/completed で再評価される)
     this._clearDetachTimer(session)

package/src/enroll.mjs

@@ -66,6 +66,9 @@ export async function enroll(enrollmentToken, opts = {}) {
     claude_model: typeof data.claude_model === "string" ? data.claude_model : "",
     claude_permission_mode:
       typeof data.claude_permission_mode === "string" ? data.claude_permission_mode : "",
+    // Codex CLI 設定 (2026-07-04 追加、claude_model/claude_permission_mode と対称)。
+    codex_model: typeof data.codex_model === "string" ? data.codex_model : "",
+    codex_effort: typeof data.codex_effort === "string" ? data.codex_effort : "",
   })
 
   // Hub AI bundle を続けて同期する (CLAUDE.md / scripts / hooks / projects.json

package/src/main.mjs

@@ -96,6 +96,7 @@ import { JsonlLiveWatchers } from "./jsonl-live-watchers.mjs"
 import {
   boundSessionId,
   codexSessionUsages,
+  codexTurnActiveForThread,
   codexUsageForThread,
   contextWindowSize,
   getSessionUsages,
@@ -173,6 +174,23 @@ function claudeCmdFromAgentConfig(config) {
   return cmd
 }
 
+/**
+ * `~/.hub/agent.json` の Codex CLI 設定 (codex_model / codex_effort) から
+ * createTmuxSession の model/effort opts へのデフォルト値を取得する
+ * (claudeCmdFromAgentConfig の Codex 版)。
+ *
+ * tmux.create_session / worktree.create の browser 側 msg.model / msg.effort が
+ * 明示指定されていればそちらを優先し (呼び出し側で判定済み)、未指定のときだけ
+ * このデフォルトにフォールバックする。空文字は「未指定」として undefined を返す
+ * (buildCodexCmd 側は undefined を渡されても既定値へフォールバックするため安全)。
+ */
+function codexDefaultsFromAgentConfig(config) {
+  return {
+    model: (config?.codex_model || "").trim() || undefined,
+    effort: (config?.codex_effort || "").trim() || undefined,
+  }
+}
+
 /**
  * プロファイル切替用: 全 tmux セッションを kill → 同名・同 cwd で再作成する。
  *
@@ -749,6 +767,10 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     logger,
     intervalMs: 5_000,
     claudeBridge,
+    // T05699 残タスク #1 (2026-07-04): 背景 Codex チャットセッションの turn_active を
+    // rollout ベースで解決するために渡す (main.mjs startStateLoop 内の codexBridge.threadIdForCwd
+    // 参照)。
+    codexBridge,
     readinessTracker,
     // tmux session_name → bind 中の session_id。turn_active を「そのペインが実際に走らせている
     // セッションの jsonl」で判定するために渡す (cwd dir 内の別 jsonl 取り違え=メタ欠陥#1 の根治)。
@@ -1408,7 +1430,31 @@ export function resolveStateLoopBoundSessionId(boundSessions, sessionName, cwd,
   return null
 }
 
-export function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, readinessTracker, boundSessions }) {
+/**
+ * turn_active をどの情報源で判定するかを決める純関数 (main.mjs startStateLoop から抽出、
+ * T05699 残タスク #1)。実際の非同期解決 (turnActiveForCwd / codexTurnActiveForThread の
+ * 呼び出し) は呼び出し側 (tick) が行う。優先順位:
+ *
+ *  1. chat 信号が processing → "chat-active" (true)
+ *  2. chat 信号が waiting/idle → "chat-inactive" (false)
+ *  3. fresh unbound bind (adopt 前の bind プレースホルダ) → "fresh-unbound" (null=不明、安全側)
+ *  4. cwd に一致するライブ codexBridge セッションがある → "codex" (rollout ベース判定)
+ *  5. cwd があれば → "claude" (jsonl ベース判定、TUI/bound session フォールバック)
+ *  6. cwd も無ければ → "none" (null=不明)
+ *
+ * @param {{chatStatus?: string|null, cwd?: string|null, isFreshUnbound?: boolean, codexThreadId?: string|null}} params
+ * @returns {'chat-active'|'chat-inactive'|'fresh-unbound'|'codex'|'claude'|'none'}
+ */
+export function decideTurnActiveSource({ chatStatus, cwd, isFreshUnbound, codexThreadId }) {
+  if (chatStatus === "processing") return "chat-active"
+  if (chatStatus === "waiting" || chatStatus === "idle") return "chat-inactive"
+  if (cwd && isFreshUnbound) return "fresh-unbound"
+  if (codexThreadId) return "codex"
+  if (cwd) return "claude"
+  return "none"
+}
+
+export function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, codexBridge, readinessTracker, boundSessions }) {
   const lastByName = new Map() // session_name → {status, context_pct}
   const lastTurnAtByName = new Map() // session_name → 最後に event ファイル化した turnAt
   // (i) session_name → {sig, changedAt}: 出力領域署名の最終変化時刻 (出力フロー検知用)。
@@ -1536,23 +1582,50 @@ export function startStateLoop({ client, plugins, logger, intervalMs, claudeBrid
         //    null を返し mtime 最新 jsonl へフォールバックしていた (chat 版の cross-activity
         //    汚染, 2026-07 発覚)。resolveStateLoopBoundSessionId が claudeBridge のライブ
         //    セッション id へフォールバックすることで TUI と同じ方式で根治する。
+        //  - Codex (T05699 残タスク #1, 2026-07-04): この cwd にライブな codexBridge セッション
+        //    (chat モード) があれば、chat 信号の 90s TTL に依存しない rollout ベースの判定
+        //    (codexTurnActiveForThread, task_started/task_complete イベント対) を使う。長時間
+        //    ターン中に chat 信号が stale 化してドットが消える欠落を根治する。Claude 版のような
+        //    paneAliveProbe は渡さない (chat モードのペインは素のシェルのまま = 実処理は別
+        //    プロセスのため、pane 生存判定を適用すると誤って dead 扱いになる)。cwd がライブな
+        //    codex セッションと一致しなければ従来通り Claude jsonl (turnActiveForCwd) にフォール
+        //    バックする (TUI/bare codex pane は現状 bound thread 解決が無いため、これまで通り
+        //    null=不明のまま、退行はしない)。
+        //    分岐の優先順位は decideTurnActiveSource (純関数、単体テスト済み) が権威。
         let turnActive = null
-        if (chat?.status === "processing") {
-          turnActive = true
-        } else if (chat?.status === "waiting" || chat?.status === "idle") {
-          turnActive = false
-        } else if (s.cwd && isFreshUnboundBind(boundSessions?.get(s.session_name))) {
-          // 症状2b 根治: fresh bind プレースホルダの間は、別アクティビティ jsonl を mtime 最新で
-          // 取り違える汚染を避けて turn_active を出さない (null=不明)。実セッション id へ adopt
-          // されれば下の通常経路に乗る。青を出さない安全側にのみ倒すため固着青は生まない。
-          turnActive = null
-        } else if (s.cwd) {
-          turnActive = await turnActiveForCwd(
-            s.cwd,
-            resolveStateLoopBoundSessionId(boundSessions, s.session_name, s.cwd, claudeBridge),
-            s.session_name,
-            { paneAliveProbe: paneClaudeAliveOrUnknown },
-          )
+        const codexThreadId = s.cwd ? codexBridge?.threadIdForCwd?.(s.cwd) || null : null
+        const turnActiveSource = decideTurnActiveSource({
+          chatStatus: chat?.status,
+          cwd: s.cwd,
+          isFreshUnbound: isFreshUnboundBind(boundSessions?.get(s.session_name)),
+          codexThreadId,
+        })
+        switch (turnActiveSource) {
+          case "chat-active":
+            turnActive = true
+            break
+          case "chat-inactive":
+            turnActive = false
+            break
+          case "fresh-unbound":
+            // 症状2b 根治: fresh bind プレースホルダの間は、別アクティビティ jsonl を mtime 最新で
+            // 取り違える汚染を避けて turn_active を出さない (null=不明)。実セッション id へ adopt
+            // されれば下の通常経路に乗る。青を出さない安全側にのみ倒すため固着青は生まない。
+            turnActive = null
+            break
+          case "codex":
+            turnActive = await codexTurnActiveForThread(codexThreadId)
+            break
+          case "claude":
+            turnActive = await turnActiveForCwd(
+              s.cwd,
+              resolveStateLoopBoundSessionId(boundSessions, s.session_name, s.cwd, claudeBridge),
+              s.session_name,
+              { paneAliveProbe: paneClaudeAliveOrUnknown },
+            )
+            break
+          default:
+            turnActive = null
         }
 
         const prev = lastByName.get(s.session_name)
@@ -1687,27 +1760,46 @@ export function startStateLoop({ client, plugins, logger, intervalMs, claudeBrid
  * 渡す予定なので、現状は plugin hook chain だけ走らせて log に残す。
  */
 /**
- * Hub から `ready` / `ack` で配信された Claude CLI 設定を agent.json と
+ * Hub から `ready` / `ack` で配信された Claude/Codex CLI 設定を agent.json と
  * in-memory config に反映する (Phase 3 live-sync)。
  *
  * 旧設計 (Phase 2) では enrollment 時のみ取得していたため、cockpit 管理画面で
  * 設定を変更しても既存 agent には再 enroll しない限り反映されなかった。Phase 3 で
  * 全 WS 接続/heartbeat 毎に最新値が同期されるようにした。
  *
+ * 2026-07-04: Codex turn/start 経由の model/effort 切替実装に伴い、codex_model/
+ * codex_effort も同じ経路で同期する (claude_model/claude_permission_mode と対称)。
+ *
  * 値が変化していない場合は writeConfig を呼ばない (書込負荷軽減)。
  * 失敗しても agent は止めず log だけ出して継続。
  */
 export async function maybeSyncClaudeSettings(msg, ctx) {
-  if (typeof msg?.claude_model !== "string" && typeof msg?.claude_permission_mode !== "string") {
-    return
-  }
+  const hasClaudeFields =
+    typeof msg?.claude_model === "string" || typeof msg?.claude_permission_mode === "string"
+  const hasCodexFields =
+    typeof msg?.codex_model === "string" || typeof msg?.codex_effort === "string"
+  if (!hasClaudeFields && !hasCodexFields) return
+
   const nextModel = typeof msg.claude_model === "string" ? msg.claude_model : ctx.config.claude_model || ""
   const nextMode = typeof msg.claude_permission_mode === "string" ? msg.claude_permission_mode : ctx.config.claude_permission_mode || ""
+  const nextCodexModel = typeof msg.codex_model === "string" ? msg.codex_model : ctx.config.codex_model || ""
+  const nextCodexEffort = typeof msg.codex_effort === "string" ? msg.codex_effort : ctx.config.codex_effort || ""
   const prevModel = ctx.config.claude_model || ""
   const prevMode = ctx.config.claude_permission_mode || ""
-  if (nextModel === prevModel && nextMode === prevMode) return
+  const prevCodexModel = ctx.config.codex_model || ""
+  const prevCodexEffort = ctx.config.codex_effort || ""
+  if (
+    nextModel === prevModel &&
+    nextMode === prevMode &&
+    nextCodexModel === prevCodexModel &&
+    nextCodexEffort === prevCodexEffort
+  ) {
+    return
+  }
   ctx.config.claude_model = nextModel
   ctx.config.claude_permission_mode = nextMode
+  ctx.config.codex_model = nextCodexModel
+  ctx.config.codex_effort = nextCodexEffort
   try {
     await writeConfig({
       agent_id: ctx.config.agent_id,
@@ -1715,13 +1807,20 @@ export async function maybeSyncClaudeSettings(msg, ctx) {
       hub_url: ctx.config.hub_url,
       claude_model: nextModel,
       claude_permission_mode: nextMode,
+      codex_model: nextCodexModel,
+      codex_effort: nextCodexEffort,
     })
     ctx.logger.info(
-      { claude_model: nextModel, claude_permission_mode: nextMode },
-      "claude settings synced from hub",
+      {
+        claude_model: nextModel,
+        claude_permission_mode: nextMode,
+        codex_model: nextCodexModel,
+        codex_effort: nextCodexEffort,
+      },
+      "claude/codex settings synced from hub",
     )
   } catch (err) {
-    ctx.logger.warn({ err: err.message }, "failed to persist claude settings to agent.json")
+    ctx.logger.warn({ err: err.message }, "failed to persist claude/codex settings to agent.json")
   }
 }
 
@@ -3579,11 +3678,17 @@ async function dispatch(msg, ctx) {
                 : claudeCmdFromAgentConfig(ctx.config),
             codexCmd: typeof msg.codex_cmd === "string" ? msg.codex_cmd : undefined,
             // codex 用設定 (claude 経路では claudeCmd を直接使うため無視される)。
-            model: typeof msg.model === "string" ? msg.model : undefined,
+            model:
+              typeof msg.model === "string"
+                ? msg.model
+                : codexDefaultsFromAgentConfig(ctx.config).model,
             sandbox: typeof msg.sandbox === "string" ? msg.sandbox : undefined,
             approvalPolicy:
               typeof msg.approval_policy === "string" ? msg.approval_policy : undefined,
-            effort: typeof msg.effort === "string" ? msg.effort : undefined,
+            effort:
+              typeof msg.effort === "string"
+                ? msg.effort
+                : codexDefaultsFromAgentConfig(ctx.config).effort,
             initialPrompt:
               typeof msg.initial_prompt === "string" ? msg.initial_prompt : undefined,
             logger: ctx.logger,
@@ -3639,11 +3744,17 @@ async function dispatch(msg, ctx) {
                 : claudeCmdFromAgentConfig(ctx.config),
             codexCmd: typeof msg.codex_cmd === "string" ? msg.codex_cmd : undefined,
             // codex 用設定 (claude 経路では無視される)。
-            model: typeof msg.model === "string" ? msg.model : undefined,
+            model:
+              typeof msg.model === "string"
+                ? msg.model
+                : codexDefaultsFromAgentConfig(ctx.config).model,
             sandbox: typeof msg.sandbox === "string" ? msg.sandbox : undefined,
             approvalPolicy:
               typeof msg.approval_policy === "string" ? msg.approval_policy : undefined,
-            effort: typeof msg.effort === "string" ? msg.effort : undefined,
+            effort:
+              typeof msg.effort === "string"
+                ? msg.effort
+                : codexDefaultsFromAgentConfig(ctx.config).effort,
             initialPrompt:
               typeof msg.initial_prompt === "string" ? msg.initial_prompt : undefined,
             logger: ctx.logger,

package/src/usage.mjs

@@ -1450,3 +1450,123 @@ export async function codexSessionUsages(liveSessions) {
   )
   return out
 }
+
+// ---------------------------------------------------------------------------
+// Codex 版 turn_active 判定 (Cockpit Codex 両対応 T05699 残タスク #1, 2026-07-04)。
+//
+// 背景: サイドバー背景セッションのステータスドット (turn_active) は state loop
+// (main.mjs startStateLoop) が権威。Codex はこれまで in-memory チャット信号
+// (chat-signals.mjs, TTL 90s) だけに依存しており、長時間ターン中に 90s 無通知の
+// 間隔が空くと stale 化しドットが消える欠落があった (main.mjs:1556-1573 の
+// else-if 分岐は Claude jsonl 専用 turnActiveForCwd しか知らない)。
+//
+// 実機の rollout jsonl (2026-07-04, 実データ 2444 ファイル走査) で `event_msg` の
+// `task_started`/`task_complete` が turn の開始/終了を示す明確なペアであることを
+// 確認済み (全ファイルで started==completed の対応が崩れた例は無し)。B3
+// (codexUsageForThread) と同じ「threadId → rollout ファイル特定 → 末尾 tail 走査」
+// パターンを踏襲し、チャット信号の TTL に依存しない独立した真実源にする。
+// ---------------------------------------------------------------------------
+
+// task_started の記録から task_complete が届かないまま経過した場合の猶予。TUI (bare pane)
+// 経由の bound thread 解決は未対応 (2026-07-04 時点、`codex.tui.bind` 相当が無い) のため、
+// 現状 paneAliveProbe を渡すのは将来の TUI 対応時のみ想定。チャットモード (app-server 駆動)
+// はペインが素のシェルのまま (実処理は別プロセス) なので pane 生存判定の対象外。
+const CODEX_STALE_TASK_MS = 5_000
+
+/** P5(perf): codex turn_active 判定結果を mtime+size でメモ化。 */
+const _codexTurnActiveMemo = new Map() // fp → { mtimeMs, size, result }
+
+/**
+ * rollout jsonl テキストを走査して「最終 turn_active 状態 + 直近 task_started の時刻」を
+ * 返す内部ヘルパー。Claude の `_scanTurnState` と対だが、Codex は `task_started`/
+ * `task_complete` という明示イベントペアがあるため stop_reason 解釈のステートマシンは不要。
+ *
+ * @param {string} text
+ * @returns {{active: boolean, activeSinceTs: number|null}}
+ */
+export function _scanCodexTurnState(text) {
+  const empty = { active: false, activeSinceTs: null }
+  if (!text) return empty
+  let active = false
+  let activeSinceTs = null
+  const lines = text.split("\n")
+  for (const line of lines) {
+    if (!line) continue
+    let d
+    try {
+      d = JSON.parse(line)
+    } catch {
+      continue
+    }
+    if (d.type !== "event_msg") continue
+    const pt = d.payload?.type
+    if (pt === "task_started") {
+      active = true
+      activeSinceTs = parseEntryTs(d.timestamp)
+    } else if (pt === "task_complete") {
+      active = false
+      activeSinceTs = null
+    }
+  }
+  return { active, activeSinceTs }
+}
+
+/**
+ * 指定 threadId の Codex セッションが現在ターン進行中かを rollout jsonl から判定する。
+ * `turnActiveForCwd` (Claude) の対。チャット信号 (in-memory, TTL 90s) が stale なときの
+ * フォールバックとして main.mjs の state loop から呼ばれる想定。
+ *
+ * @param {string} threadId
+ * @param {{sessionName?: string, paneAliveProbe?: (name:string)=>Promise<'alive'|'dead'|'unknown'>, now?: number}} [opts]
+ *   sessionName/paneAliveProbe: 将来の TUI (bare pane) 対応用。チャットモードはペインが
+ *   素のシェルのまま (実処理は別プロセス) のため渡さないこと (誤って dead 判定され得る)。
+ * @returns {Promise<boolean|null>} true=生成中 / false=待機中 / null=rollout 不明
+ */
+export async function codexTurnActiveForThread(threadId, opts = {}) {
+  if (!threadId) return null
+  const fp = await findCodexRolloutFile(threadId)
+  if (!fp) return null
+  let st
+  try {
+    st = await fs.stat(fp)
+  } catch {
+    return null
+  }
+
+  const memo = _codexTurnActiveMemo.get(fp)
+  if (memo && memo.mtimeMs === st.mtimeMs && memo.size === st.size) {
+    return memo.result
+  }
+
+  const tail = await readTail(fp, 256 * 1024)
+  let state = { active: false, activeSinceTs: null }
+  if (tail != null) state = _scanCodexTurnState(tail)
+  // tail で active=false の場合のみ、巨大 response_item で先頭の task_started が
+  // tail 窓から落ちていないか全文で確認する (Claude 版 turnActiveForCwd と同じ方針)。
+  if (state.active === false) {
+    const full = await readOrNull(fp)
+    if (full != null) state = _scanCodexTurnState(full)
+  }
+
+  let active = state.active
+  const now = typeof opts.now === "number" ? opts.now : Date.now()
+  if (
+    active &&
+    state.activeSinceTs !== null &&
+    now - state.activeSinceTs > CODEX_STALE_TASK_MS &&
+    opts.sessionName &&
+    typeof opts.paneAliveProbe === "function"
+  ) {
+    const verdict = await opts.paneAliveProbe(opts.sessionName)
+    if (verdict === "dead") active = false
+  }
+
+  const result = active
+  _codexTurnActiveMemo.set(fp, { mtimeMs: st.mtimeMs, size: st.size, result })
+  return result
+}
+
+/** テスト用: codex turn_active メモをクリアする。 */
+export function _resetCodexTurnActiveMemo() {
+  _codexTurnActiveMemo.clear()
+}