@cocorograph/hub-agent 0.7.69 → 0.7.70

package/package.json

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

package/src/codex-stream-bridge.mjs

@@ -11,6 +11,7 @@
  *   claude.input              → input()    (turn/start、ターン中は turn/steer)
  *   claude.interrupt          → interrupt() (turn/interrupt)
  *   claude.detach              → detach()   (TTL 付き graceful close)
+ *   (codex 固有)               → keepalive() (閲覧中の生存シグナル。pty.resize 相当)
  *   claude.permission.reply    → permissionReply() (client.respond/respondError)
  *   claude.event (生 SDK イベント素通し) → 'event' (生 notification 素通し、同じ思想)
  *   claude.permission.request  → 'permission' (承認/入力要求の生 method+params 素通し)
@@ -509,6 +510,26 @@ export class CodexStreamBridge extends EventEmitter {
     session.client.respond(request_id, result)
   }
 
+  /**
+   * ブラウザ閲覧中の生存シグナル (codex.keepalive)。frontend が Codex チャットを開いて
+   * いる間だけ定期送信するため、これが届く = 「人が見ている」の観測可能な証跡になる。
+   * gcAbandonedSessions は lastActivityMs しか見ないので、touch するだけで
+   * 「開いたまま無入力でも 1h で殺される」ギャップが閉じる (TTL の数字を増やす必要は無い)。
+   *
+   * ⚠️ 意図的に `_cancelIdleReap` を **呼ばない**。detach 済みセッションの 5 分 TTL は
+   * 「ブラウザが去った」という別事実に基づく予約であり、detach と in-flight keepalive が
+   * 競合しても予約を解除しない (= 明示 detach は必ず 5 分で回収される) 方が意味論が明快。
+   * 閲覧を再開したブラウザは必ず attach を送り直すので、再養子はそちらの経路で足りる。
+   *
+   * @returns {boolean} 生きているセッションを touch できたら true (未知 stream は false)。
+   */
+  keepalive({ stream_id }) {
+    const session = this.sessions.get(stream_id)
+    if (!session || session.dead) return false
+    this._touch(session)
+    return true
+  }
+
   /**
    * グレースフル切断。ターン実行中は完走を待ち、非 busy ならアイドル TTL 後に
    * プロセスを reap する (claude-stream-bridge.mjs の softDetach と同じ思想。
@@ -744,17 +765,17 @@ export class CodexStreamBridge extends EventEmitter {
    * 進行中の正常なターンは app-server 通知で都度 touch されるので回収されない。
    * main.mjs の state loop から claudeBridge.gcDeadTerminals と並べて定期実行する想定。
    *
-   * ⚠️ 既知のギャップ (2026-07-17、次段の keepalive PR で解消予定): この reaper は
-   * `detachRequested` を参照せず、`_touch()` は input() と app-server 通知でしか呼ばれない。
-   * そのため **ブラウザで Codex チャットを開いたままでも、1 時間まったく入力も生成も
-   * 無ければ回収される**。アイドル TTL(5 分) の誤爆を直しても、この 1h が次の上限になる。
-   * TTL の数字を大きくするのではなく「見ている」を観測可能にするのが正しい解:
-   * frontend/components/cockpit/CockpitTerminal.tsx の PTY keepalive (閲覧中は定期的に
-   * pty.resize で lastSeenAt を触り、ブラウザが離れれば keepalive も止まるので orphan GC が
-   * 正しく効く) と同じ構造を codex チャットへ移植する (codex.keepalive を新設し、backend
-   * consumers.py の中継テーブルと main.mjs の dispatch をセットで変更する)。そうすれば
-   * 「ブラウザ開 = 無期限 / 明示 detach = 5 分 / ブラウザ落ち = 1h reaper」になり、
-   * 「N 時間放置しても生きていてほしい」という数字合わせ自体が不要になる。
+   * この reaper は `detachRequested` を **意図的に参照しない**。detach を送れずに消えた
+   * ブラウザ (タブごとクラッシュ / 端末スリープ) は detachRequested が立たないまま残るため、
+   * ここでフラグを見ると回収不能になりリークする。「無通信が続いた事実」だけで回収する。
+   *
+   * 閲覧中の誤爆は keepalive で防ぐ (2026-07-17 に解消。旧: 開いたままでも 1h 無入力で
+   * 回収されていた)。frontend は Codex チャットを開いている間 codex.keepalive を定期送信し、
+   * keepalive() が lastActivityMs を touch する。ブラウザが閉じれば keepalive も止まるので
+   * この reaper は本来の orphan GC として正しく効く。CockpitTerminal.tsx の PTY keepalive
+   * (pty.resize で lastSeenAt を触る) と同じ構造。結果として TTL の意味は
+   * 「ブラウザ開 = 無期限 / 明示 detach = 5 分 / ブラウザ落ち = 1h reaper」に整理され、
+   * 「N 時間放置しても生きていてほしい」という数字合わせ自体が不要になった。
    *
    * @param {number} [ttlMs] 無通信とみなす閾値 (テスト用に上書き可)。
    * @returns {number} 回収したセッション数。

package/src/main.mjs

@@ -4335,6 +4335,13 @@ async function dispatch(msg, ctx) {
         if (!ctx.codexBridge) return
         ctx.codexBridge.detach({ stream_id: msg.stream_id })
         return
+      case "codex.keepalive":
+        // ブラウザが Codex チャットを開いている間の生存シグナル (CockpitTerminal.tsx の
+        // PTY keepalive と同じ構造)。閲覧中に gcAbandonedSessions (既定 1h 無通信) が
+        // 誤爆するのを防ぐ。未知 stream_id は false が返るだけ (無害な no-op)。
+        if (!ctx.codexBridge) return
+        ctx.codexBridge.keepalive({ stream_id: msg.stream_id })
+        return
       case "codex.sessions.request": {
         // cwd 配下の Codex thread 一覧を返す (claude.sessions.request と対称。
         // Cockpit のセッションピッカー/最新セッション自動選択用)。