@cocorograph/hub-agent 0.7.66 → 0.7.67

package/package.json

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

package/src/codex-appserver-client.mjs

@@ -74,6 +74,12 @@ export class CodexAppServerClient extends EventEmitter {
       cwd: this.cwd,
       env: this.env || process.env,
       stdio: ["pipe", "pipe", "pipe"],
+      // detached にして子 (node ランチャー codex.js) を新しいプロセスグループの leader に
+      // する。こうすると stop() で `process.kill(-pid)` によりグループごと落とせ、node
+      // ランチャーが spawn する native codex バイナリ (孫) まで確実に終了できる。detached
+      // 単体では親終了時に孫が生き残るが、stop()/shutdown() で必ずグループ kill するため
+      // orphan 化しない (逆に child.kill() 単体だと node だけ落ちて native が orphan 化する)。
+      detached: true,
     })
     this.child.stdout.on("data", (chunk) => this._onStdout(chunk))
     this.child.stderr.on("data", (chunk) => {
@@ -224,15 +230,32 @@ export class CodexAppServerClient extends EventEmitter {
     if (!this.child || this.closed) return
     this.closed = true
     this._settleAllPending(new Error("codex app-server client stopped"))
+    const child = this.child
     try {
-      this.child.stdin.end()
+      child.stdin.end()
     } catch {
       // stdin が既に閉じている等は無視して良い (stop の冪等性を優先)。
     }
-    try {
-      this.child.kill()
-    } catch {
-      // プロセスが既に無い場合等は無視して良い。
+    // detached spawn によりプロセスグループ leader なので、グループごと (孫の native
+    // codex バイナリ含む) 落とす。グループが無い/権限エラー時は単体 kill にフォールバック。
+    const killGroup = (signal) => {
+      try {
+        if (typeof child.pid === "number") {
+          process.kill(-child.pid, signal)
+          return
+        }
+      } catch {
+        // グループが既に消えている / 未 detached 等 → 単体 kill にフォールバック。
+      }
+      try {
+        child.kill(signal)
+      } catch {
+        // プロセスが既に無い場合等は無視して良い。
+      }
     }
+    killGroup("SIGTERM")
+    // SIGTERM で落ちない (ハング等) 場合に備え、猶予後 SIGKILL でグループごと強制終了。
+    const t = setTimeout(() => killGroup("SIGKILL"), 3000)
+    t.unref?.()
   }
 }

package/src/codex-stream-bridge.mjs

@@ -64,6 +64,16 @@ function siblingEnvFor(cwd) {
 // 関わらず完走まで絶対に落とさない (_maybeReapAfterIdle が busy 中は予約しない)。
 const IDLE_DETACH_TTL_MS = 5 * 60 * 1000
 
+// 放棄セッションの reaper 閾値。claude-stream-bridge.mjs の gcDeadTerminals と同じ思想・
+// 同じ env (HUB_AGENT_DEAD_TERMINAL_TTL_MS、既定 1 時間)。detach を送らず消えた端末や、
+// codex エラーで turn/completed が来ず activeTurnId が固着したスタックセッションは、
+// detach→TTL(5分) の graceful reap 経路に乗らないため app-server が永久に残る (リーク)。
+// これを補い、「一定時間まったく通信 (browser 入力 / app-server 通知) が無い」セッションを
+// activeTurnId の有無に関わらず定期回収する。進行中の正常なターンは app-server 通知で
+// 都度 touch されるため回収されない (通知が途絶えたスタックターンのみ回収される)。
+const ABANDONED_SESSION_TTL_MS =
+  Number(process.env.HUB_AGENT_DEAD_TERMINAL_TTL_MS) || 60 * 60 * 1000
+
 const CLIENT_INFO = { name: "cockpit-hub-agent", title: "Cockpit", version: "1.0.0" }
 
 // Slice 6: thread/resume が返す Thread.turns[].items[] を単一の時系列配列へ平坦化する。
@@ -191,6 +201,9 @@ export class CodexStreamBridge extends EventEmitter {
       pendingApprovals: new Map(), // request_id -> { turnId }
       detachTimer: null,
       dead: false,
+      // 放棄セッション reaper (gcAbandonedSessions) 用の最終活性時刻。browser 入力と
+      // app-server 通知 (ターン進捗) の両方で更新する。Date.now() は hub-agent 実行環境で利用可。
+      lastActivityMs: Date.now(),
     }
     this.sessions.set(stream_id, session)
     this._wireClient(session)
@@ -284,8 +297,17 @@ export class CodexStreamBridge extends EventEmitter {
     })
   }
 
+  /** gcAbandonedSessions 用の最終活性更新。browser 入力 / app-server 通知の両方で呼ぶ。 */
+  _touch(session) {
+    if (session) session.lastActivityMs = Date.now()
+  }
+
   _onNotification(session, msg) {
     const { stream_id } = session
+    // app-server からの通知 = ターンが進行している証跡。これで touch することで、
+    // 正常に走っているターンは reaper に回収されず、通知が途絶えたスタックターンだけが
+    // 放棄セッションとして回収される。
+    this._touch(session)
     switch (msg.method) {
       case "turn/started":
         session.activeTurnId = msg.params?.turn?.id || null
@@ -361,6 +383,7 @@ export class CodexStreamBridge extends EventEmitter {
    */
   async input({ stream_id, message, model, effort }) {
     const session = this._require(stream_id)
+    this._touch(session)
     const input = userInputFromMessage(message)
     if (session.activeTurnId) {
       try {
@@ -408,6 +431,7 @@ export class CodexStreamBridge extends EventEmitter {
   /** @returns {Promise<void>} */
   async interrupt({ stream_id }) {
     const session = this._require(stream_id)
+    this._touch(session)
     if (!session.activeTurnId) return
     await session.client.request("turn/interrupt", {
       threadId: session.threadId,
@@ -418,6 +442,7 @@ export class CodexStreamBridge extends EventEmitter {
   /** サーバー発の承認/入力要求への応答。result の shape は method ごとに呼び出し側が組み立てる。 */
   permissionReply({ stream_id, request_id, result }) {
     const session = this._require(stream_id)
+    this._touch(session)
     if (!session.pendingApprovals.has(request_id)) {
       this.logger?.warn?.({ stream_id, request_id }, "codex permissionReply: unknown request_id (ignored)")
       return
@@ -508,6 +533,49 @@ export class CodexStreamBridge extends EventEmitter {
     return session
   }
 
+  /**
+   * 放棄セッションの reaper。claude-stream-bridge.mjs の gcDeadTerminals と同じ思想:
+   * 一定時間まったく通信 (browser 入力 / app-server 通知) が無いセッションの app-server を
+   * 停止して回収する。detach を送らず消えた端末や、codex エラーで turn/completed が来ず
+   * activeTurnId が固着したスタックターンは detach→TTL の graceful reap 経路に乗らないため、
+   * これが唯一の回収経路になる (プロセスリーク / 生成中固着の根治)。
+   *
+   * 進行中の正常なターンは app-server 通知で都度 touch されるので回収されない。
+   * main.mjs の state loop から claudeBridge.gcDeadTerminals と並べて定期実行する想定。
+   *
+   * @param {number} [ttlMs] 無通信とみなす閾値 (テスト用に上書き可)。
+   * @returns {number} 回収したセッション数。
+   */
+  gcAbandonedSessions(ttlMs = ABANDONED_SESSION_TTL_MS) {
+    const now = Date.now()
+    let reaped = 0
+    for (const session of Array.from(this.sessions.values())) {
+      if (session.dead) continue
+      const last = session.lastActivityMs ?? 0
+      if (now - last < ttlMs) continue
+      this._clearDetachTimer(session)
+      session.dead = true
+      this.sessions.delete(session.stream_id)
+      try {
+        session.client.stop()
+      } catch (err) {
+        this.logger?.warn?.({ stream_id: session.stream_id, err: err?.message }, "gcAbandonedSessions: stop threw")
+      }
+      reaped += 1
+      this.logger?.info?.(
+        { stream_id: session.stream_id, thread_id: session.threadId, idle_ms: now - last },
+        "codex abandoned session reaped (no activity past TTL)",
+      )
+      this.emit("exit", {
+        stream_id: session.stream_id,
+        thread_id: session.threadId,
+        code: 0,
+        reason: "idle-reaped",
+      })
+    }
+    return reaped
+  }
+
   /** 全セッションを即座に終了する (agent shutdown 用。ターン完走は待たない)。 */
   shutdown() {
     for (const session of this.sessions.values()) {

package/src/main.mjs

@@ -1871,6 +1871,14 @@ export function startStateLoop({ client, plugins, logger, intervalMs, claudeBrid
       } catch (err) {
         logger?.warn({ err: err?.message }, "gcDeadTerminals failed")
       }
+      // Codex 版 (claude の gcDeadTerminals と対): detach を送らず消えた / turn/completed が
+      // 来ず activeTurnId が固着したスタックセッションの app-server を回収する (プロセス
+      // リーク・生成中固着の根治)。進行中の正常ターンは app-server 通知で touch され残る。
+      try {
+        codexBridge?.gcAbandonedSessions?.()
+      } catch (err) {
+        logger?.warn({ err: err?.message }, "gcAbandonedSessions failed")
+      }
       // 実コンテキスト窓サイズ (1M ベータ等) を反映。contextPctFromUsage の分母を
       // 5s ごとに最新化し、ドーナツが 200k 固定で振り切れるのを防ぐ。
       await refreshContextWindow()