@cocorograph/hub-agent 0.7.77 → 0.7.79

package/package.json

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

package/src/codex-stream-bridge.mjs

@@ -177,6 +177,15 @@ function flattenThreadItems(thread, { maxItems = HISTORY_MAX_ITEMS, maxBytes = H
   return { items: kept, truncated, total }
 }
 
+/** 履歴 item の概算 byte 数 (flattenThreadItems の budget 判定と同じ JSON 長ベース)。 */
+function estimateItemBytes(item) {
+  try {
+    return JSON.stringify(item)?.length || 0
+  } catch {
+    return 0
+  }
+}
+
 /**
  * resume 応答から Codex の「権威的なターン進行状態」を取り出す。
  *
@@ -332,6 +341,35 @@ export class CodexStreamBridge extends EventEmitter {
       }
     }
 
+    // T06541 案1: attach_id が無くても、同一 (cwd, resume_thread_id) のライブ session があれば
+    // in-memory reattach する。frontend は attach_id を送らないため上の attachId 経路は発火せず、
+    // これが実質のメイン reattach 経路になる。新プロセス thread/resume を起動せず所有権を付け替え、
+    // **蓄積済みの高忠実度履歴を返す**（frontend は remount で白紙化するため履歴が必要。reducer は
+    // uuid dedup で二重描画しない）。これにより症状⑦（生成中に移動→戻ると turn_active=false）が
+    // 解消し、停止/完了通知も同一プロセスに揃う。同一会話 (threadId 一致) のみ対象。
+    if (resumeThreadId) {
+      const existing = this._findLiveSessionByCwd(cwd, resumeThreadId)
+      if (existing && existing.stream_id !== stream_id) {
+        const r = await this._tryReattach(existing, stream_id)
+        if (r.status === "reattached") {
+          return {
+            ...r.info,
+            // 案1: reattach は履歴を返さない設計 (r.info.history=[]) だが、cwd reattach は remount
+            // 由来で表示がリセットされ得るため、live セッションが保持する高忠実度履歴を返す。
+            history: Array.isArray(existing.historyItems) ? existing.historyItems.slice() : [],
+            historyTruncated: existing.historyTruncated === true,
+          }
+        }
+        if (r.status === "superseded") {
+          const err = new Error("attach superseded by a concurrent reattach")
+          err.attachOperation = "reattach"
+          err.attachReason = "superseded"
+          throw err
+        }
+        // r.status === "unavailable" → 通常経路 (thread/resume) へフォールスルー。
+      }
+    }
+
     const client = this._clientFactory({
       cwd,
       codexCmd: this.codexCmd,
@@ -349,8 +387,15 @@ export class CodexStreamBridge extends EventEmitter {
       attachId: attachId || null,
       client,
       threadId: null,
+      // T06541 major#3: cwd reattach の in-flight 引き当てキー。thread/resume 完了まで threadId は
+      // null なので、並行 cold attach が「同一 (cwd, resumeThreadId)」を検出できるよう resume 対象の
+      // thread id を **client 生成前 (同期)** に持たせる。これで後続 attach は _findLiveSessionByCwd で
+      // この session を引き当て _tryReattach の session.ready 合流に入り、二重 app-server を作らない。
+      resumeThreadId: resumeThreadId || null,
       activeTurnId: null,
-      pendingApprovals: new Map(), // request_id -> { turnId }
+      // request_id -> { turnId, method, params }。major#1: reattach で新 stream へ未応答の承認要求を
+      // 再送できるよう、元の method/params も保持する (Claude の _permissionResolvers と同型)。
+      pendingApprovals: new Map(),
       // ブラウザから codex.detach を受け取った (= 画面の持ち主が居なくなった) か。診断・ログ用。
       // Stage 5: 回収の可否は detachRequested ではなく lastActivityMs（LRU）と activeTurnId
       // （busy/stuck）で決める。detach してもすぐには殺さず warm 保持する。
@@ -361,7 +406,22 @@ export class CodexStreamBridge extends EventEmitter {
       // 通知 (ターン進捗) の両方で更新する。LRU は「最終活性が最も古い非 protected」を退避する。
       // Date.now() は hub-agent 実行環境で利用可。
       lastActivityMs: Date.now(),
+      // 相方選定・一覧の「新しさ」用の会話ターン境界時刻。lastActivityMs（keepalive 閲覧で
+      // 更新される warm/reap 保護用）と役割を分離し、**turn/started・turn/completed のときだけ**
+      // 更新する。これにより「見ているだけ（keepalive）」で並び順・ask-sibling 宛先選定が動く
+      // 問題を根治する（症状⑨と同じ「read/閲覧時刻を鮮度信号に流用しない」原則）。初期値は
+      // attach 時刻＝新規セッションは「たった今作った」で上位に来る（閲覧中の古い会話に負けない）。
+      lastTurnAtMs: Date.now(),
       ready: null,
+      // T06541 案1（高忠実度履歴の蓄積）: cold attach 時の flatten 済み ThreadItem 履歴を
+      // 保持し、ライブ item/completed ごとに追記する。cwd reattach（新プロセスを立てない
+      // 復帰）時にこれを返すことで、frontend の remount 白紙化を防ぎつつ、rollout JSONL の
+      // 生データ（ツールカードが粗い）ではなく app-server 整形済みの高忠実度履歴を維持する。
+      historyItems: [],
+      historyTruncated: false,
+      // major#2: ライブ追記でも cold flatten と同じ「item 数＋概算 byte」二重上限を課すため、
+      // 蓄積済み履歴の概算 byte 合計を保持する (再計算を避け O(1) で trim 判定)。
+      historyBytes: 0,
     }
     session.ready = new Promise((resolve) => {
       settleReady = resolve
@@ -392,6 +452,12 @@ export class CodexStreamBridge extends EventEmitter {
         : { items: [], truncated: false, total: 0 }
       const history = flat.items
       const historyTruncated = flat.truncated
+      // T06541 案1: cold attach で得た高忠実度履歴をセッションに保持する。以降ライブ item
+      // 完了ごとに追記し、cwd reattach 時にこれを返す（新プロセス thread/resume を再実行しない）。
+      // major#2: byte 予算判定を O(1) にするため概算 byte 合計も持つ (flatten は既に byte 上限適用済み)。
+      session.historyItems = history.slice()
+      session.historyBytes = history.reduce((n, it) => n + estimateItemBytes(it), 0)
+      session.historyTruncated = historyTruncated
       // 案C: 権威的なターン状態を resume 応答から取り出してフロントへ渡す。
       // これによりフロントは会話末尾からの推測をやめ、Thread.status で turnActive を
       // 決められる (生成中スピナー固着バグの根治)。新規スレッドは常に idle 相当。
@@ -455,6 +521,58 @@ export class CodexStreamBridge extends EventEmitter {
     return null
   }
 
+  /**
+   * T06541: 同一 (cwd, threadId) のライブ session を返す (cwd ベース in-memory reattach 用)。
+   * frontend は attach_id を送らないため attachId 経路は発火せず、resume は毎回 thread/resume で
+   * 新プロセスを起動していた（別プロセスで生成中のターンを Thread.status で観測できず turn_active
+   * が false になる症状⑦の主機序）。resume_thread_id 一致で生きた session を引き当て所有権を付け替え
+   * れば、生成中ターン・停止・完了通知・高忠実度履歴が 1 本に揃う。
+   * ⚠️ **同一会話 (threadId 一致) のみ** を対象にし、別会話 (threadId 不一致) を誤って奪わない。
+   * major#3: thread/resume 完了前 (threadId=null) の in-flight session も、resume 対象 thread が
+   * 一致すれば引き当てる。これで並行 cold attach の後続が _tryReattach の session.ready 合流に入り、
+   * 同一 thread の app-server が二重生成されるのを防ぐ (Claude の同期索引登録と同じ「空の窓」対策)。
+   */
+  _findLiveSessionByCwd(cwd, threadId) {
+    if (!cwd || !threadId) return null
+    for (const s of this.sessions.values()) {
+      if (s.dead || s.cwd !== cwd) continue
+      // 確定済み thread 一致、または in-flight (未確定) で resume 対象が一致。
+      if (s.threadId === threadId) return s
+      if (s.threadId === null && s.resumeThreadId === threadId) return s
+    }
+    return null
+  }
+
+  /**
+   * T06541 案1: 完了 ThreadItem を高忠実度履歴 (session.historyItems) に追記する。
+   * 表示用バブル時刻 at_ms は completedAtMs (ms epoch) を採る（flattenThreadItems と同じ ms 単位。
+   * userMessage はライブでも completedAtMs=ターン冒頭で届くため整合する）。長い会話で無限成長させ
+   * ないよう cold flatten と同じ **item 数 (maxItems) ＋ 概算 byte 数 (maxBytes) の二重上限** を課し、
+   * 超過分は先頭 (最古) から捨て truncated を立てる (major#2: 巨大ツール出力で reattach payload が
+   * 肥大化し T06499 で防いだメモリ増・WS 転送・ブラウザフリーズが再発するのを防ぐ)。
+   */
+  _appendHistoryItem(session, item, completedAtMs) {
+    if (!session || !item || typeof item !== "object") return
+    if (!Array.isArray(session.historyItems)) session.historyItems = []
+    if (typeof session.historyBytes !== "number") session.historyBytes = 0
+    const entry = typeof completedAtMs === "number" ? { ...item, at_ms: completedAtMs } : item
+    session.historyItems.push(entry)
+    session.historyBytes += estimateItemBytes(entry)
+    const maxItems = this._historyLimits.maxItems
+    const maxBytes = this._historyLimits.maxBytes
+    // 末尾 1 件は必ず残す (flatten と同じ方針)。item 数超過 or byte 超過の間、先頭から捨てる。
+    while (
+      session.historyItems.length > 1 &&
+      ((typeof maxItems === "number" && session.historyItems.length > maxItems) ||
+        (typeof maxBytes === "number" && session.historyBytes > maxBytes))
+    ) {
+      const dropped = session.historyItems.shift()
+      session.historyBytes -= estimateItemBytes(dropped)
+      if (session.historyBytes < 0) session.historyBytes = 0
+      session.historyTruncated = true
+    }
+  }
+
   /**
    * 既存ライブ session を新 stream_id へ付け替える (in-memory reattach)。thread が未確定
    * (thread/start in-flight) なら session.ready を await する (成功=threadId / 失敗=null)。
@@ -488,6 +606,11 @@ export class CodexStreamBridge extends EventEmitter {
       { stream_id: newStreamId, thread_id: threadId, cwd: session.cwd, reattached: true },
       "codex session reattached (in-memory)",
     )
+    // major#1: 新 stream へ未応答の承認要求を再送する (Claude の _replayPendingPermissions と同型)。
+    // 承認要求は app-server との in-flight callback で、履歴 hydrate には含まれない。旧 stream 宛に
+    // 出したカードは remount/端末切替で失われ、応答が来ないとターンが永久 pending になる。新 stream へ
+    // 再 emit してカードを復元させる (frontend は request_id で重複排除するので二重化しない)。
+    this._replayPendingApprovals(session)
     // 履歴は返さない: reconnect では frontend が表示を保持している (resetDisplay=false) ため
     // 再 hydrate 不要。turnActive は session が保持する activeTurnId から権威的に決める。
     return {
@@ -610,13 +733,21 @@ export class CodexStreamBridge extends EventEmitter {
     switch (msg.method) {
       case "turn/started":
         session.activeTurnId = msg.params?.turn?.id || null
+        // 会話ターン境界で「新しさ」を更新（閲覧 keepalive では更新しない）。
+        session.lastTurnAtMs = Date.now()
         break
       case "turn/completed":
         session.activeTurnId = null
+        session.lastTurnAtMs = Date.now()
         this._autoResolveStalePendingApprovals(session, msg.params?.turn?.id)
         // Stage 5: turn 完了後の reap 予約は不要。以降 idle になれば周期 reconcile が
         // 長 idle TTL / LRU で扱う（detach 済みでも 7 日 warm 保持）。
         break
+      case "item/completed":
+        // T06541 案1: 完了した ThreadItem を高忠実度履歴に追記する（cwd reattach で返す用）。
+        // ライブ表示は下の emit がそのまま担う（本 case は履歴蓄積を足すだけで emit は変えない）。
+        this._appendHistoryItem(session, msg.params?.item, msg.params?.completedAtMs)
+        break
       default:
         break
     }
@@ -630,7 +761,12 @@ export class CodexStreamBridge extends EventEmitter {
 
   _onServerRequest(session, msg) {
     const { stream_id } = session
-    session.pendingApprovals.set(msg.id, { turnId: session.activeTurnId })
+    // major#1: reattach 時に新 stream へ再送できるよう method/params も保持する。
+    session.pendingApprovals.set(msg.id, {
+      turnId: session.activeTurnId,
+      method: msg.method,
+      params: msg.params,
+    })
     this.emit("permission", {
       stream_id,
       thread_id: session.threadId,
@@ -641,6 +777,25 @@ export class CodexStreamBridge extends EventEmitter {
     })
   }
 
+  /**
+   * major#1: 未応答の承認要求を現在の stream へ再送する (reattach から呼ぶ)。Claude の
+   * _replayPendingPermissions と同型。frontend は request_id で重複排除するため、既に表示済みの
+   * 端末でも二重化しない。method/params は _onServerRequest 受信時に保存済み。
+   */
+  _replayPendingApprovals(session) {
+    if (!session || session.pendingApprovals.size === 0) return
+    for (const [request_id, info] of session.pendingApprovals) {
+      this.emit("permission", {
+        stream_id: session.stream_id,
+        thread_id: session.threadId,
+        cwd: session.cwd,
+        request_id,
+        method: info.method,
+        params: info.params,
+      })
+    }
+  }
+
   // ターン完了時、そのターンに紐づく未解決の承認要求は自動 decline する
   // (browser が応答する前にターンが打ち切られた/クラッシュ相当のケース。
   // claude-stream-bridge.mjs の「ターン終了時に未解決分は自動 deny」と同じ設計)。
@@ -833,7 +988,9 @@ export class CodexStreamBridge extends EventEmitter {
         thread_id: session.threadId || null,
         busy: !!session.activeTurnId,
         detached: !!session.detachRequested,
-        idle_ms: now - (session.lastActivityMs ?? now),
+        // idle_ms は「新しさ」の指標（一覧の並び・宛先選定に使う）なので、閲覧 keepalive で
+        // 動く lastActivityMs ではなく **会話ターン境界時刻 lastTurnAtMs** を基準にする。
+        idle_ms: now - (session.lastTurnAtMs ?? session.lastActivityMs ?? now),
       })
     }
     return out
@@ -879,7 +1036,9 @@ export class CodexStreamBridge extends EventEmitter {
     let bestAt = -1
     for (const session of this.sessions.values()) {
       if (session.dead || session.cwd !== cwd) continue
-      const at = session.lastActivityMs ?? 0
+      // 宛先選定の「最新」は会話ターン境界時刻で判定する（閲覧 keepalive で動く
+      // lastActivityMs を使うと「見ているだけの古い会話」が本命を追い越す）。
+      const at = session.lastTurnAtMs ?? session.lastActivityMs ?? 0
       if (at >= bestAt) {
         bestAt = at
         best = session