@cocorograph/hub-agent 0.7.65 → 0.7.67

package/package.json

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

package/scripts/install.ps1

@@ -240,23 +240,23 @@ function Set-Systemd($distro) {
 # Register a logon task that keeps the distro ALIVE (so its systemd --user
 # hub-agent stays online).
 #
-# 旧実装は `wsl ... /bin/true` で distro を一瞬起こすだけだった。これだと WSL2 の
-# アイドルシャットダウンが数十秒で distro ごと畳んでしまい (agent.log に
-# `SIGTERM shutting down` が出て `wsl -l -v` が Stopped になる)、hub-agent も停止して
-# オフラインに戻る。systemd --user + linger は「distro が起動している間」しか効かない
-# ため、distro 自体を起こし続ける常駐プロセスが必要。
+# The old impl used `wsl ... /bin/true` to wake the distro only briefly. WSL2's
+# idle shutdown then collapsed the whole distro within tens of seconds (agent.log
+# shows `SIGTERM shutting down` and `wsl -l -v` goes Stopped), so hub-agent also
+# stopped and went offline. systemd --user + linger only works "while the distro
+# is running", so we need a resident process that keeps the distro awake.
 #
-# 対策: ログオン中ずっと `sleep infinity` を WSL 内で走らせて distro を起動状態に保つ。
-# 併せて hub-agent.service を start しておく (enable + linger 済みなら冪等)。
+# Fix: run `sleep infinity` inside WSL for the whole logon session to keep the
+# distro up. Also start hub-agent.service (idempotent when already enabled+linger).
 function Register-BootTask($distro, $user) {
   Write-Step "Registering autostart on logon (Task Scheduler, keep-alive)"
-  # distro を起動し続ける keep-alive。systemctl start は冪等 (既に active なら no-op)。
+  # keep-alive that holds the distro up. systemctl start is idempotent (no-op if active).
   $keepalive = "systemctl --user start hub-agent.service 2>/dev/null; exec sleep infinity"
   $action  = New-ScheduledTaskAction -Execute "wsl.exe" `
     -Argument "-d $distro -u $user -- bash -lc `"$keepalive`""
   $trigger = New-ScheduledTaskTrigger -AtLogOn
-  # ExecutionTimeLimit=0 で無制限 (既定 72h で停止されると distro が落ちる)。
-  # keep-alive が万一終了したら 1 分後に最大 3 回まで再起動。多重起動は抑止。
+  # ExecutionTimeLimit=0 for unlimited (default 72h would stop it and drop the distro).
+  # If the keep-alive ever exits, restart after 1 min up to 3 times. Multiple instances suppressed.
   $set     = New-ScheduledTaskSettingsSet -AllowStartIfOnBatteries -DontStopIfGoingOnBatteries `
     -StartWhenAvailable -MultipleInstances IgnoreNew `
     -RestartInterval (New-TimeSpan -Minutes 1) -RestartCount 3 `
@@ -333,14 +333,14 @@ function Main {
   # --- autostart ---
   Register-BootTask $distro $user
 
-  # セットアップ完了時点でオンライン化まで完結させる: keep-alive タスクを今すぐ起動し、
-  # distro を起動状態に保つ (再ログオンを待たずに hub-agent が常駐する)。
+  # Finish onlining at setup time: start the keep-alive task now and keep the
+  # distro up (hub-agent goes resident without waiting for re-logon).
   Write-Step "Starting keep-alive now (online without re-logon)"
   try {
     Start-ScheduledTask -TaskName $TASK_NAME
     Write-Ok "Keep-alive started; hub-agent should come online shortly"
   } catch {
-    Write-Warn "Start-ScheduledTask failed ($($_.Exception.Message)); ログオンし直すと起動します"
+    Write-Warn "Start-ScheduledTask failed ($($_.Exception.Message)); re-logon to start it"
   }
 
   Write-Host ""

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()