@cocorograph/hub-agent 0.8.12 → 0.8.14

package/bin/hub-agent.mjs

@@ -11,6 +11,12 @@
  *   - stop             : daemon 停止 (Sprint J で launchd / systemd と併用)
  *   - plugins list     : プラグイン一覧
  */
+// R3(d): libuv スレッドプールを拡張する。サイドバー snapshot の全セッション jsonl 読み込みが、
+// attach / history の fs と既定 4 スレッドのプールを奪い合って切替が体感で遅くなる。libuv は
+// プール初回利用時に UV_THREADPOOL_SIZE を読むため、いかなる fs/dns/crypto 非同期処理より前
+// (= プロセス最初期・import より前) に設定する。既に設定済みなら運用者の指定を尊重する。
+if (!process.env.UV_THREADPOOL_SIZE) process.env.UV_THREADPOOL_SIZE = "16"
+
 import { readFileSync } from "node:fs"
 import { fileURLToPath } from "node:url"
 import path from "node:path"

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -1720,6 +1720,12 @@ export class ClaudeStreamSession {
     // 正規の中断は interrupt() だが、control request が失敗/無応答のときに generator を畳む
     // 最終手段が無いと、close()/shutdown() で CLI 子プロセスが残留する。
     this._residentAbortController = new AbortController()
+    // R1(a): この run の AbortController を捕捉する。finally で「常駐 generator が返却された理由」を
+    // 判別するのに使う。我々が意図的に畳んだ (close / interrupt フォールバックの keep-alive
+    // hard-abort) ときだけ signal.aborted=true になる。子プロセスが自然終了 (クラッシュ/異常終了)
+    // したときは我々は abort していないので false。this.* 参照だと後続 run が再代入した別
+    // controller を読んでしまうため、必ずこの run 固有のローカル参照で判定する。
+    const myAbort = this._residentAbortController
     const options = {
       cwd: this.cwd,
       env: siblingEnvFor(this.cwd),
@@ -1882,6 +1888,40 @@ export class ClaudeStreamSession {
           next.ultracode === true,
         )
         this._emitQueueState([next.text], { startedIds: [next.id] })
+      } else if (
+        !this._closed &&
+        !this._residentSendInFlight &&
+        !myAbort?.signal?.aborted
+      ) {
+        // R1(a) 孤児根絶: detach-reap でも pending 再起動でもなく、我々の意図的 fold
+        // (close / interrupt フォールバックの keep-alive hard-abort = myAbort.signal.aborted)
+        // でもないのに常駐 generator が返却された = 子プロセスが自然終了した (クラッシュ/異常終了)。
+        // _residentSendInFlight 中 (ABA/FIFO: 起動済み query の送信が reconcile を await 中) は
+        // await 復帰後の送信が fresh-start で回復させるため除外済み。
+        if (this._detached) {
+          // browser が離れた warm セッションで子が死亡 = 真の孤児。誰も見ておらず、死んだ query は
+          // R1(b) で reattach 対象にもならないため warm 保持する意味が無い。即 close + onReap で
+          // _liveBySession から evict し browser へ exit を通知する (既存 reap 経路と同じ冪等処理)。
+          // 従来はこの経路が無く、warm 中に子が死ぬと _closed=false のまま索引に残って孤児化し、
+          // 枠を占有し続け reattach が死んだ session を引き当てていた。
+          try {
+            this.close()
+          } finally {
+            this.onReap?.({ reason: "child-exited" })
+          }
+        } else {
+          // まだ browser attach 中: 既存の「次 input で resume 付き再起動」recovery (改修4/B6) を
+          // 壊さないため evict しない (即 evict すると同一 stream への継続入力が stream_missing で
+          // 落ちる)。ただし終端シグナルが無いとサイドバーの処理中ドット / UI「停止中…」が固着する
+          // ため、abort 経路と対称に onTurnSettled を出して即時解除する。次 input が来なければ
+          // detach 時の softDetach idle-timer / warm-bounds が回収し、R1(b) が死んだ session への
+          // reattach を防ぐ。
+          try {
+            this.onTurnSettled?.()
+          } catch {
+            /* ignore */
+          }
+        }
       }
       // マスク (2026-06-23): 常駐 query 終了後 (= CLI が jsonl を掴んでいない) のこの地点で、
       // closed 済みならオンディスク jsonl をマスクする。reap 経路は close() 内で実行済み。
@@ -2049,6 +2089,18 @@ export class ClaudeStreamSession {
     return true
   }
 
+  /** R1(b): このセッションへ resume 再アタッチしてよいか。close 済み、または常駐 generator が
+   *  返却済み (_residentQueryReturned) で再走行していない (_residentQuery=null) = 子プロセスが
+   *  死亡した状態には reattach しない (attach 側が新規 spawn へフォールバックする)。R1(a) で
+   *  死亡時は同期 close されるので通常は _closed で弾かれるが、close 到達前の窓や将来経路でも
+   *  死んだ query へ再アタッチしない二重ガードとして効かせる。まだ一度も query を起動していない
+   *  idle セッション (_residentQueryReturned=false) は生存中なので reattach 可。 */
+  _isReattachable() {
+    if (this._closed) return false
+    if (this._residentQueryReturned && !this._residentQuery) return false
+    return true
+  }
+
   /** セッション終了。新規ターンを止め、実行中なら中断し、watcher も停止する。 */
   close() {
     this._closed = true
@@ -2261,7 +2313,9 @@ export class ClaudeStreamBridge extends EventEmitter {
       //    attach しても引き当たらず、越境 reattach が原理的に起きない (session_id は本来
       //    ユニークだが異常系でも cwd を跨がない)。
       const live = this._liveBySession.get(this._liveKeyFor(scwd, resumeSessionId))
-      if (live && !live._closed) {
+      // R1(b): !_closed だけでなく「常駐子プロセスが実生存しているか」も確認する。子が死んで
+      // いれば reattach せず下の新規 spawn 経路へフォールバックし、死んだ session を引き当てない。
+      if (live && live._isReattachable()) {
         // 多端末共有 (T06542 で常時): 旧 stream_id を Map から消さない。同一セッションを複数端末が
         // 購読する状態を作り、どの端末の input/permission/interrupt も解決できるようにする
         // (旧端末は reattach.streamIds に足され、backend が session group へ fan-out する)。
@@ -2366,14 +2420,15 @@ export class ClaudeStreamBridge extends EventEmitter {
           error: err?.message || String(err),
         })
       },
-      onReap: () => {
-        // ターン完走後の遅延クローズ。Map / 索引から撤去し exit を emit する。
-        // 多端末共有では複数端末キーが Map に載っているのでまとめて外す。
+      onReap: (info) => {
+        // ターン完走後の遅延クローズ、または R1(a) 子プロセス死亡時の即 evict。Map / 索引から
+        // 撤去し exit を emit する。多端末共有では複数端末キーが Map に載っているのでまとめて外す。
+        // reason は既定「detached-after-turn」。R1(a) は「child-exited」を渡して観測可能にする。
         this._dropSessionMappings(session)
         this.emit("exit", {
           stream_id: session.stream_id,
           code: 0,
-          reason: "detached-after-turn",
+          reason: info?.reason || "detached-after-turn",
           session_id: session.sessionId,
         })
       },
@@ -3252,13 +3307,32 @@ export class ClaudeStreamBridge extends EventEmitter {
     return at
   }
 
+  /** R1(c): このセッションを現在ライブに購読している端末数 (sessions Map で自分を指す stream key 数)。
+   *  正規の多端末共有 (T06542) では 1 つの session を複数 stream_id が指す。単端末なら 1、
+   *  soft-detach 済み warm なら 0。 */
+  _liveTerminalCount(session) {
+    if (!session) return 0
+    let n = 0
+    for (const sid of session.streamIds || []) {
+      if (this.sessions.get(sid) === session) n += 1
+    }
+    return n
+  }
+
   /** 子⑤ round5 Finding 2 対応: warm 境界で reap してはいけない「in-flight」セッション。
    *  走行中ターン（_busy）に加え、送信待ちの pending がある（現ターン完了時に drain される）
-   *  セッションも保護する（reap すると未処理入力が失われる）。 */
+   *  セッションも保護する（reap すると未処理入力が失われる）。
+   *  R1(c) 多ウィンドウ相互 reap 抑止: 2 端末以上が現在ライブ購読中の多端末共有セッション
+   *  (T06542) も保護する。同一 cwd で別 session_id を開いた新規 1 本を残すために、複数
+   *  ウィンドウが見ている共有セッションを per-cwd=1 / cwd-LRU が刈ってしまう事故を防ぐ
+   *  (単端末の重複セッション collapse は従来どおり働く: 単端末は count=1 で非保護)。 */
   _isSessionProtected(session) {
     if (!session) return false
     if (session._busy) return true
-    return Array.isArray(session._pendingMessages) && session._pendingMessages.length > 0
+    if (Array.isArray(session._pendingMessages) && session._pendingMessages.length > 0) {
+      return true
+    }
+    return this._liveTerminalCount(session) >= 2
   }
 
   /** 子⑤: warm 境界超過セッションを reap する（detach の softDetach onTimeout と同じ撤去処理）。 */

package/src/codex-stream-bridge.mjs

@@ -391,13 +391,59 @@ export class CodexStreamBridge extends EventEmitter {
     if (this.sessions.has(stream_id)) {
       throw new Error(`stream_id already attached: ${stream_id}`)
     }
+    // 計測 (2026-07-31): attach 1 回 = 1 行の内訳ログ (`codex attach timing`)。
+    // frontend の attach_sent→ready_received は 1 つの数字でしか見えず、その中身が
+    // 「app-server の cold spawn + initialize」なのか「thread/resume の rollout 読み込み」なのか、
+    // そもそも **cold spawn だったのか warm reattach だったのか**すら判別できなかった。
+    // ここで mode / warm フラグと各フェーズの ms を **同じ 1 行**に載せる。attach 時のみ出すので
+    // ログ量は既存の "codex session attached" と同オーダー (ターン毎には出ない)。
+    const attachStartedAt = Date.now()
+    let lapMark = attachStartedAt
+    /** 直前のマークからの経過 ms を返し、基準点を進める。 */
+    const lap = () => {
+      const now = Date.now()
+      const delta = now - lapMark
+      lapMark = now
+      return delta
+    }
+    /** フェーズ別内訳。未実行フェーズは未設定のまま (ログでは null)。 */
+    const timing = {}
     // #1: 同じ attach_id のライブ session があれば in-memory reattach を試みる。
     if (attachId) {
       const existing = this._findLiveSessionByAttachId(attachId)
       if (existing && existing.stream_id !== stream_id) {
         const r = await this._tryReattach(existing, stream_id)
-        if (r.status === "reattached") return r.info
+        timing.reattach_wait_ms = (timing.reattach_wait_ms ?? 0) + lap()
+        if (r.status === "reattached") {
+          this._logAttachTiming({
+            stream_id,
+            cwd,
+            thread_id: r.info.threadId || null,
+            mode: "reattach_attach_id",
+            warm: true,
+            ok: true,
+            resuming: true,
+            thread_op: null,
+            history_items: 0,
+            total_ms: Date.now() - attachStartedAt,
+            timing,
+          })
+          return r.info
+        }
         if (r.status === "superseded") {
+          this._logAttachTiming({
+            stream_id,
+            cwd,
+            thread_id: null,
+            mode: "reattach_attach_id",
+            warm: true,
+            ok: false,
+            resuming: true,
+            thread_op: null,
+            failed_stage: "reattach_superseded",
+            total_ms: Date.now() - attachStartedAt,
+            timing,
+          })
           // 別 reattach が勝って session を奪った (concurrent reconnect)。**通常経路へ落とさない**
           // — 落とすと敗者が別 thread を thread/start してしまい I2 (fresh 最大1回) を再び破る /
           // 同一 thread の app-server が複数化する (major #1 ラウンド5)。二重 thread 副作用を作らず
@@ -424,16 +470,45 @@ export class CodexStreamBridge extends EventEmitter {
       const existing = this._findLiveSessionByCwd(cwd, resumeThreadId)
       if (existing && existing.stream_id !== stream_id) {
         const r = await this._tryReattach(existing, stream_id)
+        timing.reattach_wait_ms = (timing.reattach_wait_ms ?? 0) + lap()
         if (r.status === "reattached") {
+          const history = Array.isArray(existing.historyItems) ? existing.historyItems.slice() : []
+          this._logAttachTiming({
+            stream_id,
+            cwd,
+            thread_id: r.info.threadId || null,
+            mode: "reattach_cwd",
+            warm: true,
+            ok: true,
+            resuming: true,
+            thread_op: null,
+            history_items: history.length,
+            history_truncated: existing.historyTruncated === true,
+            total_ms: Date.now() - attachStartedAt,
+            timing,
+          })
           return {
             ...r.info,
             // 案1: reattach は履歴を返さない設計 (r.info.history=[]) だが、cwd reattach は remount
             // 由来で表示がリセットされ得るため、live セッションが保持する高忠実度履歴を返す。
-            history: Array.isArray(existing.historyItems) ? existing.historyItems.slice() : [],
+            history,
             historyTruncated: existing.historyTruncated === true,
           }
         }
         if (r.status === "superseded") {
+          this._logAttachTiming({
+            stream_id,
+            cwd,
+            thread_id: null,
+            mode: "reattach_cwd",
+            warm: true,
+            ok: false,
+            resuming: true,
+            thread_op: null,
+            failed_stage: "reattach_superseded",
+            total_ms: Date.now() - attachStartedAt,
+            timing,
+          })
           const err = new Error("attach superseded by a concurrent reattach")
           err.attachOperation = "reattach"
           err.attachReason = "superseded"
@@ -549,6 +624,9 @@ export class CodexStreamBridge extends EventEmitter {
     this.sessions.set(stream_id, session)
     this._wireClient(session)
     client.start()
+    // app-server の spawn は同期呼び出し (fork/exec のみ)。プロセス起動の実コストは
+    // 直後の initialize RPC 応答待ちに現れるので、spawn_ms と initialize_ms は必ずセットで読む。
+    timing.spawn_ms = lap()
 
     const resuming = !!resumeThreadId
     // #3: initialize と thread op を分離し、thread op に到達したかを記録する。initialize / config
@@ -560,6 +638,8 @@ export class CodexStreamBridge extends EventEmitter {
         "codex app-server session initialization",
       )
       await client.request("initialize", { clientInfo: CLIENT_INFO })
+      // ここまでが「app-server プロセスが起動して JSON-RPC に応答できるようになるまで」。
+      timing.initialize_ms = lap()
       // 公式 App Server handshake: initialize 応答後、同じ transport 上で initialized を
       // 通知してから thread operation を開始する。現行 server の寛容動作に依存しない。
       client.notify("initialized", {})
@@ -584,6 +664,9 @@ export class CodexStreamBridge extends EventEmitter {
       const result = resuming
         ? await client.request("thread/resume", { ...startParams, threadId: resumeThreadId })
         : await client.request("thread/start", startParams)
+      // thread/resume は app-server 側で rollout(JSONL) を読み直すため、resume の
+      // thread_op_ms が大きい = rollout 読み込みが支配的、という読み方になる。
+      timing.thread_op_ms = lap()
       session.threadId = result?.thread?.id
       // thread/start は一覧そのものを変更するため、pre-attach で得た短期キャッシュを破棄する。
       // resume/reattach は既存 thread の再閲覧なのでキャッシュを維持し、ready 後の重複走査を防ぐ。
@@ -604,6 +687,8 @@ export class CodexStreamBridge extends EventEmitter {
       const flat = resuming
         ? flattenThreadItems(result?.thread, this._historyLimits)
         : { items: [], truncated: false, total: 0 }
+      // hub-agent 側の履歴整形コスト (巨大スレッドで効く)。thread_op_ms とは別勘定にする。
+      timing.history_flatten_ms = lap()
       const history = flat.items
       const historyTruncated = flat.truncated
       // T06541 案1: cold attach で得た高忠実度履歴をセッションに保持する。以降ライブ item
@@ -643,6 +728,21 @@ export class CodexStreamBridge extends EventEmitter {
         },
         "codex session attached",
       )
+      this._logAttachTiming({
+        stream_id,
+        cwd,
+        thread_id: session.threadId || null,
+        mode: resuming ? "cold_resume" : "cold_start",
+        warm: false,
+        ok: true,
+        resuming,
+        thread_op: resuming ? "thread/resume" : "thread/start",
+        history_items: history.length,
+        history_total: flat.total,
+        history_truncated: historyTruncated,
+        total_ms: Date.now() - attachStartedAt,
+        timing,
+      })
       // 子⑤ (T06544): genuine attach で 1 本増えたので warm 境界（per-cwd=1 / N-cwd LRU）を即時
       // 適用する（次の reconnect まで ~5s 待たず、同一 cwd の重複や cwd 数超過が居座るのを防ぐ）。
       // 新セッションはたった今 touch 済み = 各判定で最新なので退避対象にならず、最も古い非 protected
@@ -661,12 +761,60 @@ export class CodexStreamBridge extends EventEmitter {
       this.sessions.delete(stream_id)
       client.stop()
       settleReady(null) // reattach 待ちへ「失敗」を通知 (reject しない)
+      // 失敗も 1 行に落とす。**_classifyAttachFailure より前**に出す (分類は rollout の disk 走査を
+      // 伴うことがあり、その待ち時間を attach レイテンシに混ぜないため)。
+      this._logAttachTiming({
+        stream_id,
+        cwd,
+        thread_id: null,
+        mode: resuming ? "cold_resume" : "cold_start",
+        warm: false,
+        ok: false,
+        resuming,
+        thread_op: reachedThreadOp ? (resuming ? "thread/resume" : "thread/start") : null,
+        failed_stage: reachedThreadOp ? "thread_op" : "initialize",
+        total_ms: Date.now() - attachStartedAt,
+        timing,
+      })
       // #3/#4: thread op に到達した失敗だけ分類する (initialize 失敗は分類しない)。
       if (reachedThreadOp) await this._classifyAttachFailure(err, { resuming, resumeThreadId })
       throw err
     }
   }
 
+  /**
+   * attach 1 回分の内訳を **1 行** の構造化ログに落とす (`msg="codex attach timing"`)。
+   *
+   * ## なぜ必要か (WHY)
+   * frontend の switch-metrics は `attach_sent → ready_received` を 1 つの数字でしか持たず
+   * (本番 p50 ~436ms)、その中身が app-server の cold spawn なのか thread/resume の rollout
+   * 読み込みなのか切り分けられなかった。さらに致命的なのは **warm 再利用 (in-memory reattach) と
+   * cold spawn を区別する材料がログのどこにも無かった**こと (frontend の temp=cold|warm は
+   * ブラウザ内キャッシュの命中/非命中であって hub-agent の warm 再利用とは無関係)。
+   *
+   * ## 読み方
+   * - `warm=true` / `mode=reattach_*` … app-server を立てていない (所要は `reattach_wait_ms` だけ)
+   * - `warm=false` / `mode=cold_*` … `spawn_ms`(同期 spawn) + `initialize_ms`(プロセス起動待ち)
+   *   + `thread_op_ms`(thread/start|resume。resume は rollout 読み込み込み) + `history_flatten_ms`
+   *
+   * ログ量: attach 時のみ (ターン毎には出ない)。既存の "codex session attached" と同オーダー。
+   */
+  _logAttachTiming(fields) {
+    const { timing = {}, ...rest } = fields
+    this.logger?.info?.(
+      {
+        ...rest,
+        // 未実行フェーズは null (0ms と「通っていない」を混同させない)。
+        reattach_wait_ms: timing.reattach_wait_ms ?? null,
+        spawn_ms: timing.spawn_ms ?? null,
+        initialize_ms: timing.initialize_ms ?? null,
+        thread_op_ms: timing.thread_op_ms ?? null,
+        history_flatten_ms: timing.history_flatten_ms ?? null,
+      },
+      "codex attach timing",
+    )
+  }
+
   /** attach_id に一致するライブ session を返す (in-memory reattach 用。無ければ null)。 */
   _findLiveSessionByAttachId(attachId) {
     if (!attachId) return null

package/src/sidebar-message-snapshot.mjs

@@ -182,7 +182,7 @@ async function findCodexRolloutFile(threadId, sessionsDir) {
   return null
 }
 
-async function readCached(filePath, extract, { maxAgeMs = Infinity } = {}) {
+async function readCached(filePath, extract) {
   if (!filePath) return { user: "", assistant: "" }
   let fileStat
   try {
@@ -192,11 +192,15 @@ async function readCached(filePath, extract, { maxAgeMs = Infinity } = {}) {
   }
   const version = `${fileStat.mtimeMs}:${fileStat.size}:${fileStat.ctimeMs}`
   const cached = textCache.get(filePath)
-  if (
-    cached &&
-    cached.version === version &&
-    Date.now() - cached.checkedAt < maxAgeMs
-  ) {
+  if (cached && cached.version === version) {
+    // mtime/size/ctime 未変化 = jsonl 本文不変。tail 読み + パースを丸ごとスキップし、
+    // 前回抽出済みテキストをそのまま返す (R3(a) mtime 差分キャッシュ)。stat 1 発だけで済む。
+    // 旧実装は maxAgeMs=2s の age ゲートで、8s 周期のサイドバー snapshot が毎回 (2s 超過で)
+    // 未変化ファイルまで再抽出し、attach/history の fs と libuv プールを奪い合っていた。
+    // LRU 順を保つため hot エントリを末尾へ入れ直す (最古から evict される側から救う)。
+    cached.checkedAt = Date.now()
+    textCache.delete(filePath)
+    textCache.set(filePath, cached)
     return cached.texts
   }
 
@@ -287,10 +291,8 @@ export async function readSidebarMessageSnapshot(
           session_id: target.conversationId,
           projectsRoot,
         })
-        texts = await readCached(
-          filePath,
-          (fp) => extractClaude(fp, { isInjected }),
-          { maxAgeMs: 2_000 },
+        texts = await readCached(filePath, (fp) =>
+          extractClaude(fp, { isInjected }),
         )
         return {
           session_name: target.sessionName,
@@ -309,10 +311,8 @@ export async function readSidebarMessageSnapshot(
       const { sessionsDir, profileId } = await resolveCodexRoot(target.profileId)
       if (!texts.user || !texts.assistant) {
         const filePath = await findCodexRolloutFile(target.conversationId, sessionsDir)
-        const diskTexts = await readCached(
-          filePath,
-          (fp) => extractCodex(fp, { isInjected }),
-          { maxAgeMs: 2_000 },
+        const diskTexts = await readCached(filePath, (fp) =>
+          extractCodex(fp, { isInjected }),
         )
         texts = {
           user: texts.user || diskTexts.user,