@cocorograph/hub-agent 0.8.13 → 0.8.15

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -1408,11 +1408,20 @@ export class ClaudeStreamSession {
    *   メッセージ本文。drain 由来の emit でのみ渡す。frontend はこれを user バブルへ
    *   昇格させる。キャンセル / 追加由来の emit では空 (昇格させない)。これにより
    *   「先頭の pending をキャンセルした」のを「実行開始した」と誤認しなくなる (0.6.26)。
-   *  @param {{force?: boolean}} [opts] force=true のとき署名重複チェックを無視して必ず
-   *   emit する。再アタッチ時のキュー snapshot 再送 (キュー再表示バグ修正, 0.6.30) に使う。
-   *   署名が前回と同一でも browser がライブ配信を取りこぼしている可能性があるため。 */
+   *  @param {{force?: boolean, startedIds?: (string|null)[], discarded?: string[]}} [opts]
+   *   force=true のとき署名重複チェックを無視して必ず emit する。再アタッチ時のキュー
+   *   snapshot 再送 (キュー再表示バグ修正, 0.6.30) に使う。署名が前回と同一でも browser が
+   *   ライブ配信を取りこぼしている可能性があるため。
+   *   discarded は「実行せずに捨てた input_id」(close 時の pending 破棄)。browser はこれを
+   *   受けてチップを消さず未配信印を付ける (無言削除の根治, 2026-07-31)。 */
   _emitQueueState(started = [], opts = undefined) {
     const force = opts?.force === true
+    // 無言削除の根治 (2026-07-31): 実行せずに捨てた input_id。空配列は載せない
+    // (旧 browser との後方互換 = 未知フィールドは無視されるが、無駄なノイズも出さない)。
+    const discarded =
+      Array.isArray(opts?.discarded) && opts.discarded.length > 0
+        ? [...opts.discarded]
+        : undefined
     // 実行開始した項目の input_id (started と parallel。T06603 cross-review major#2)。
     // 本文テキストだけだと、同一本文を連投したときに 2 通目のユーザー発言バブルが
     // 「既に同じ本文のバブルがある」と誤判定されて永久に出ない。id で突合させる。
@@ -1446,6 +1455,8 @@ export class ClaudeStreamSession {
       // items は id 付きでキャンセルボタンの対象特定に使う (0.6.26)。
       this.onEvent?.({
         ...this._buildQueueSnapshot(),
+        // 実行せずに捨てた id (close 時の pending 破棄)。載る時だけキーを出す。
+        ...(discarded ? { discarded } : {}),
         started,
         // started と parallel。連携注入なら 'peer'|'orchestrator'、通常は null。
         startedSources,
@@ -2105,12 +2116,30 @@ export class ClaudeStreamSession {
   close() {
     this._closed = true
     // 改修3: 未処理の pending を破棄 (セッション終了後に drain しないため)。
+    const discardedIds = this._pendingMessages
+      .map((m) => m.id)
+      .filter((id) => typeof id === "string" && id)
     const hadPending = this._pendingMessages.length > 0
     this._pendingMessages = []
+    // 無言削除の根治 (2026-07-31): 破棄した id を **受理済みリングから外す**。
+    // browser の 2 軸判定は「キューに無い ＋ 受理済み ＝ 実行完了」なので、破棄した id を
+    // accepted に残したまま空 snapshot を送ると、**実行していない送信を実行完了と誤認して
+    // チップを無言削除**する (ユーザーが送った本文が痕跡なく消える)。accepted から外し、
+    // かつ discarded として明示して「捨てた」ことを browser に伝える。
+    for (const id of discardedIds) {
+      const at = this._acceptedInputIds.indexOf(id)
+      if (at !== -1) this._acceptedInputIds.splice(at, 1)
+    }
     // T06603 (2026-07-25): 破棄したことを必ず通知する。従来は無言で捨てていたため、
     // 「セッションが畳まれた後もチップだけが残る」表示が残留していた。破棄は状態変化なので
     // force で貫通させる (started=[] なのでバブル昇格は起きない)。
-    if (hadPending) this._emitQueueState([], { force: true })
+    if (hadPending) {
+      this.logger?.info(
+        { stream_id: this.stream_id, discarded: discardedIds.length },
+        "pending messages discarded on close",
+      )
+      this._emitQueueState([], { force: true, discarded: discardedIds })
+    }
     // 常駐 query は input キューを閉じて generator を終わらせ query を完走させる。
     if (this._inputQueue) this._inputQueue.close()
     this.abortTurn()

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