@cocorograph/hub-agent 0.7.0 → 0.7.2

package/package.json

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

package/src/claude-history.mjs

@@ -321,13 +321,22 @@ export function decideSessionRotation({
   newestSessionId,
   lastNotifiedNewId,
   lastNotifiedAt,
+  lastNotifiedCount = 0,
   now,
   reNotifyMs = 4000,
+  maxReNotify = 5,
 } = {}) {
   if (!newestSessionId || !viewingSessionId) return { rotated: false }
   if (newestSessionId === viewingSessionId) return { rotated: false }
   if (newestSessionId === lastNotifiedNewId) {
     // 同一 new。時刻情報が揃っていて再通知間隔を超えていれば、固着救済のため再通知する。
+    // ただし安全弁: 同一 (viewing→newest) ペアを maxReNotify 回再通知してもフロントが
+    // 採用しない場合は打ち切る (capped)。フロントが viewing を更新しない=外来 id 汚染や
+    // 別 cwd の rotated 誤採用の兆候で、これを無制限に再通知すると rotated 無限ピンポンに
+    // なる (perf監査: cockpit↔cocomiru で rotated 244件/h を実測)。
+    if (lastNotifiedCount >= maxReNotify) {
+      return { rotated: false, capped: true }
+    }
     if (
       typeof now === "number" &&
       typeof lastNotifiedAt === "number" &&

package/src/main.mjs

@@ -37,6 +37,7 @@ import {
   capturePane,
   detectPermissionModeFromText,
   detectSessionState,
+  invalidateSessionCache,
   listSessionStates,
   StallTracker,
 } from "./state.mjs"
@@ -895,17 +896,18 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
           readinessTracker.arm(sessionName)
         }
       }
-      // 計装 (2026-06-19): ターン境界イベントと arm を成功送信時に記録する。従来は session.event
-      // が WS 未接続スキップ時しかログに残らず、arm→ターン進行→ready の追跡が事後にできなかった
-      // (生成中/停止判定の障害が「ログから盲目」だった主因)。armed は prompt_submit/stop/idle_hint で立つ。
-      logger?.info(
-        {
-          session_name: sessionName,
-          event: data.event,
-          armed: readinessTracker ? readinessTracker.isArmed(sessionName) : undefined,
-        },
-        "session.event push",
-      )
+      // 計装 (2026-06-19, 既定 OFF=2026-06-20): ターン境界イベントと arm を記録する。常時 info は
+      // ログ肥大の主因のため HUB_AGENT_STATE_TRACE=1 のときだけ出す(障害解析時に有効化)。
+      if (STATE_TRACE) {
+        logger?.info(
+          {
+            session_name: sessionName,
+            event: data.event,
+            armed: readinessTracker ? readinessTracker.isArmed(sessionName) : undefined,
+          },
+          "session.event push",
+        )
+      }
       client.send({
         type: "session.event",
         session_name: sessionName,
@@ -1041,7 +1043,15 @@ function startReadinessLoop({ tracker, logger, intervalMs = 700 }) {
 // いれば「出力ストリーミング中」とみなし proc_busy に OR する。state loop は 5s 周期なので、
 // 連続生成中は毎 tick 署名が変わり常時 true、出力が止まれば次 tick (≤5s) で署名不変になり
 // 窓経過後に false へ落ちる (ターン終了後のスピナー固着=症状A を悪化させない短さ)。
-const OUTPUT_ACTIVE_MS = Number(process.env.HUB_AGENT_OUTPUT_ACTIVE_MS ?? 8000)
+// 出力フロー検知窓。state loop tick(5000ms)の整数倍にして、出力が止まった後の outputActive が
+// 5s tick 境界で true↔false フラッピングするのを防ぐ(8000 は非整数倍で proc_busy 単独反転 push が
+// 多発していた=perf監査。10000=2tick で安定側、影響は『出力停止とみなす猶予 8s→10s』のみ)。
+const OUTPUT_ACTIVE_MS = Number(process.env.HUB_AGENT_OUTPUT_ACTIVE_MS ?? 10000)
+
+// 状態信号の計装ログ(session.state/event push)を出すか。既定 OFF。障害解析時に
+// HUB_AGENT_STATE_TRACE=1 で一時的に有効化する(常時 info 出力はログ肥大の主因=perf監査で
+// 直近1h ログの約半分を占めると実測。挙動には一切影響しない純ログ)。
+const STATE_TRACE = process.env.HUB_AGENT_STATE_TRACE === "1"
 
 // 権威的な「応答停止 (ハング)」判定の無進捗しきい値。ターン進行中 (armed) なのに、実際の
 // 進捗 (子プロセス実行中 / 出力フロー / ペインのライブ processing) がこの時間継続して観測
@@ -1169,6 +1179,7 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
           prev.permission_mode !== permissionMode ||
           prev.stable !== stable ||
           prev.proc_busy !== procBusy ||
+          prev.child_busy !== childBusy ||
           prev.stalled !== stalled
         ) {
           lastByName.set(s.session_name, {
@@ -1177,12 +1188,13 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             permission_mode: permissionMode,
             stable,
             proc_busy: procBusy,
+            child_busy: childBusy,
             stalled,
           })
-          // 計装 (2026-06-19): 状態の「変化時」(差分送信なので低頻度) に値を記録する。従来は
-          // session.state が WS 未接続スキップ時しかログに残らず、proc_busy/stalled/status の実値が
-          // 事後追跡できなかった (生成中/停止判定の障害がログから検証不能だった主因)。childBusy と
-          // outputActive を分けて出し、proc 内省と出力フローのどちらが busy を立てたか切り分け可能にする。
+          // 計装 (2026-06-19, 既定 OFF=2026-06-20): 状態変化時に値を記録する。常時 info はログ肥大の
+          // 主因のため HUB_AGENT_STATE_TRACE=1 のときだけ出す。childBusy/outputActive を分けて出し、
+          // proc 内省と出力フローのどちらが busy を立てたか切り分け可能にする(障害解析用)。
+          if (STATE_TRACE) {
           logger?.info(
             {
               session_name: s.session_name,
@@ -1197,6 +1209,7 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             },
             "session.state push",
           )
+          }
           client.send({
             type: "session.state",
             session_name: s.session_name,
@@ -1205,9 +1218,28 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             permission_mode: permissionMode,
             stable,
             proc_busy: procBusy,
+            // child_busy: outputActive を含まない真の子プロセス busy。frontend の shouldQueue が
+            // 出力残り火 (proc_busy 残存) でアイドルなのにキューへ逃がす誤キュー (症状A) を断つために使う。
+            child_busy: childBusy,
             stalled,
           })
         }
+        // 消滅セッションの GC (perf監査/メモリリーク対策): tmux から消えたセッションの状態 Map を
+        // まとめて解放する。clear() は再接続時(onReopen)のみのため、worktree 案件の生成/破棄で
+        // 各 per-session Map が線形増加し RSS が無入力時も成長していた。tmux が一時的に空応答
+        // (states=[]) のときは全消去しないよう states.length>0 をガードにする。
+        if (states.length > 0) {
+          const liveNames = new Set(states.map((s) => s.session_name))
+          for (const name of [...lastByName.keys()]) {
+            if (liveNames.has(name)) continue
+            invalidateSessionCache(name) // capture/cwd/statusGate/spinnerFreeze/stability (state.mjs)
+            stallTracker.forget(name) // StallTracker.byName
+            readinessTracker?.forget?.(name) // ReadinessTracker.byName
+            lastByName.delete(name)
+            outputFlowByName.delete(name)
+            lastTurnAtByName.delete(name)
+          }
+        }
       }
     } catch (err) {
       logger?.warn({ err: err.message }, "state loop tick failed")
@@ -1475,13 +1507,26 @@ async function dispatch(msg, ctx) {
         }
         handleUntrackedPtyData(msg, ctx)
         return
-      case "pty.resize":
-        ctx.ptyBridge.resize({
+      case "pty.resize": {
+        const resized = ctx.ptyBridge.resize({
           stream_id: msg.stream_id,
           cols: msg.cols,
           rows: msg.rows,
         })
+        // 自己修復 (perf監査 Phase3): stream 不在で resize できなかった (reap 済み等) ときは
+        // pty.error(stream_missing) を返し browser を再 attach させる。keepalive resize が dead
+        // stream を即検知して 4 分単位の「pty.resize but stream missing」churn を自己修復する
+        // (handleTrackedPtyData/handleUntrackedPtyData と同じパターン)。resize 例外 (pty 存在) も
+        // false だが、その場合の再 attach は既存 stream への無害な再接続。
+        if (!resized && msg.stream_id) {
+          ctx.client.send({
+            type: "pty.error",
+            stream_id: msg.stream_id,
+            error: "stream_missing",
+          })
+        }
         return
+      }
       case "pty.detach":
         ctx.ptyBridge.detach({ stream_id: msg.stream_id })
         return
@@ -1692,20 +1737,37 @@ async function dispatch(msg, ctx) {
               const key = viewName || viewCwd
               const prev = ctx.tuiRotationNotified.get(key)
               const now = Date.now()
-              const { rotated, newSessionId } = decideSessionRotation({
+              const prevNewId =
+                prev && typeof prev === "object" ? prev.newId : prev
+              const prevCount =
+                prev && typeof prev === "object" ? (prev.count ?? 0) : 0
+              const { rotated, newSessionId, capped } = decideSessionRotation({
                 viewingSessionId: viewSid,
                 newestSessionId: newestId,
-                // 旧形式 (文字列) と新形式 ({newId, ts}) の両対応。
-                lastNotifiedNewId:
-                  prev && typeof prev === "object" ? prev.newId : prev,
+                // 旧形式 (文字列) と新形式 ({newId, ts, count}) の両対応。
+                lastNotifiedNewId: prevNewId,
                 lastNotifiedAt:
                   prev && typeof prev === "object" ? prev.ts : null,
+                lastNotifiedCount: prevCount,
                 now,
               })
+              if (capped) {
+                // 安全弁発火: フロントが採用しないまま maxReNotify 回再通知済み。これ以上の
+                // rotated 連発を止め、無限ピンポンを断つ (フロント側 cwd 必須化で通常は到達しない)。
+                logger.warn(
+                  { session: viewName, cwd: viewCwd, view: viewSid, newest: newestId },
+                  "tui session rotation: capped (frontend not adopting; stopped re-notify)",
+                )
+                return
+              }
               if (!rotated) return
-              // {newId, ts} で保存し、同一 new への再通知をスロットリングする
-              // (固着した viewer は throttle 間隔ごとに 1 回ずつ救済通知を受ける)。
-              ctx.tuiRotationNotified.set(key, { newId: newSessionId, ts: now })
+              // {newId, ts, count} で保存。同一 new への再通知をスロットリング+回数キャップする。
+              // newId が変われば count を 1 にリセット、同一なら加算する。
+              ctx.tuiRotationNotified.set(key, {
+                newId: newSessionId,
+                ts: now,
+                count: newSessionId === prevNewId ? prevCount + 1 : 1,
+              })
               // 冪等マップも新 id へ更新し、将来の remount bind が旧 id へ
               // respawn (claude 再起動) してしまうのを防ぐ。
               if (viewName) ctx.tuiReboundSessions.set(viewName, newSessionId)

package/src/state.mjs

@@ -47,12 +47,15 @@ export function invalidateSessionCache(sessionName) {
     _cwdCache.clear()
     _statusGate.clear()
     _spinnerFreezeByName.clear()
+    _stabilityByName.clear()
     return
   }
   _captureCache.delete(sessionName)
   _cwdCache.delete(sessionName)
   _statusGate.delete(sessionName)
   _spinnerFreezeByName.delete(sessionName)
+  // _stabilityByName も per-session で消す (消滅セッションの GC 漏れ=メモリリーク対策, perf監査)。
+  _stabilityByName.delete(sessionName)
 }
 
 /**