@cocorograph/hub-agent 0.8.13 → 0.8.14

package/package.json

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

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