@cocorograph/hub-agent 0.6.99 → 0.7.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.6.99",
+  "version": "0.7.1",
   "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"
@@ -68,6 +69,7 @@ import {
   listWorktreeNameHistory,
   listWorktreeStubs,
   rebindClaudeSession,
+  shouldSkipRebindRespawn,
   recoverTuiInput,
   removeWorktree as removeWorktreeDir,
   resumeWithMessage,
@@ -593,6 +595,9 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
   // 終わって真に入力可能になった」エッジを検出する (proc-introspect.mjs)。capture-pane は
   // Stop フック実行中を観測できないため、これが「stop の早期消灯/誤フラッシュ」の根治。
   const readinessTracker = new ReadinessTracker()
+  // claude.tui.bind ハンドラが「生成中か (isArmed)」を参照して、生成中 claude を rebind の
+  // respawn-pane -k で kill する「謎停止」を防ぐ (生成中ガード)。
+  ctx.readinessTracker = readinessTracker
   const stateLoop = startStateLoop({
     client,
     plugins,
@@ -891,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,
@@ -1037,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) がこの時間継続して観測
@@ -1175,10 +1189,10 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             proc_busy: procBusy,
             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,
@@ -1193,6 +1207,7 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             },
             "session.state push",
           )
+          }
           client.send({
             type: "session.state",
             session_name: s.session_name,
@@ -1204,6 +1219,22 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             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")
@@ -1471,13 +1502,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
@@ -1688,20 +1732,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)
@@ -1962,12 +2023,16 @@ async function dispatch(msg, ctx) {
           // resume せず resume 無しの claude を起動する。session_id は起動後の初回送信で
           // 生成され、frontend は回転検知 / sessions 応答で拾う。
           const fresh = msg.fresh === true
-          // 1) 載せ替え先 session_id を確定 (fresh は解決しない = 新規起動)。
+          // 1) 載せ替え先 session_id を確定 (fresh は解決しない = 新規起動)。あわせて
+          //    現在動いているセッション (= cwd の最新 jsonl = newestId) も解決する。生成中ガードで
+          //    「再接続/remount による同一セッションへの再 bind」か「別 session への明示 switch」かを
+          //    区別するのに使う (前者は respawn を抑止して生成中 claude を温存する)。
           let targetId =
             typeof msg.session_id === "string" && msg.session_id
               ? msg.session_id
               : null
-          if (!fresh && !targetId && cwd) {
+          let newestId = null
+          if (!fresh && cwd) {
             const projectsRoot = await getActiveProjectsRoot()
             const { sessions } = await listSessions({
               cwd,
@@ -1975,7 +2040,20 @@ async function dispatch(msg, ctx) {
               limit: 1,
               logger,
             })
-            targetId = sessions?.[0]?.session_id || null
+            newestId = sessions?.[0]?.session_id || null
+            if (!targetId) targetId = newestId
+          }
+          // 生成中ガード (謎停止対策): respawn-pane -k は生成中 claude を強制 kill し in-flight 応答を失う。
+          // hub-agent 再起動直後は tuiReboundSessions が空で冪等ガードが効かず、ブラウザ再接続の bind が
+          // 生成中 claude を kill する。armed (ターン進行中) か pane=processing なら生成中とみなす。
+          let generating = ctx.readinessTracker?.isArmed?.(sessionName) === true
+          if (!generating && sessionName) {
+            try {
+              const snap = await detectSessionState(sessionName, {})
+              if (snap?.status === "processing") generating = true
+            } catch {
+              // capture 失敗時は据え置き (生成中とみなさない = 従来挙動)
+            }
           }
           // 2) SDK を停止 (cwd 全体 + 対象 id)。
           let stoppedSdk = 0
@@ -2002,6 +2080,18 @@ async function dispatch(msg, ctx) {
           if (bindKey && sessionName && (fresh || targetId)) {
             if (ctx.tuiReboundSessions.get(sessionName) === bindKey) {
               rebind = { ok: true, skipped: true }
+            } else if (
+              shouldSkipRebindRespawn({ generating, fresh, targetId, newestId })
+            ) {
+              // 生成中 claude を再接続/remount の bind で kill しない (謎停止対策)。respawn を抑止して
+              // 既存の生成中 claude を温存する。bindKey は記録して以降の remount を冪等化する
+              // (targetId === newestId = 動いているセッション本人なので記録は正しい)。
+              rebind = { ok: true, skipped: true }
+              ctx.tuiReboundSessions.set(sessionName, bindKey)
+              logger?.info(
+                { session: sessionName, session_id: targetId },
+                "tui rebind: skipped respawn (turn in progress on the running session)",
+              )
             } else {
               rebind = await rebindClaudeSession(sessionName, targetId, {
                 cwd,

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)
 }
 
 /**

package/src/tmux.mjs

@@ -1220,6 +1220,29 @@ export async function recoverTuiInput(name, expectText, opts = {}) {
  * @param {{cwd?:string,model?:string,permissionMode?:string,fresh?:boolean,logger?:object,tmuxBin?:string}} [opts]
  * @returns {Promise<{ok:boolean, cmd?:string, error?:string}>}
  */
+/**
+ * rebind (respawn-pane -k + claude --resume) を抑止すべきか判定する純ロジック。
+ *
+ * 背景 (謎停止対策): claude.tui.bind は TUI ビューの mount / 再接続のたびに来る。respawn-pane -k は
+ * 現 claude を強制 kill するため、生成中に走ると in-flight の応答が失われる (jsonl 未確定)。冪等ガード
+ * (tuiReboundSessions) は hub-agent 再起動直後は空で効かず、ブラウザ再接続の bind が生成中 claude を
+ * kill する事故 (謎停止) を起こす。
+ *
+ * 抑止条件: 生成中 (generating) かつ「今動いているセッション (= 最新 jsonl = newestId) への再 bind」。
+ * これは再接続/remount であり respawn は純粋に破壊的。逆に:
+ *  - fresh (+新規セッション要求) は明示操作なので respawn する。
+ *  - targetId !== newestId (別 session への明示 switch) は respawn する (生成中セッションは別ペイン文脈で温存)。
+ *  - 非生成中は従来どおり respawn する (idle claude の --resume 載せ替えは非破壊)。
+ *
+ * @param {{generating?: boolean, fresh?: boolean, targetId?: string|null, newestId?: string|null}} a
+ * @returns {boolean}
+ */
+export function shouldSkipRebindRespawn({ generating, fresh, targetId, newestId } = {}) {
+  if (fresh) return false
+  if (!generating) return false
+  return !!targetId && targetId === newestId
+}
+
 export async function rebindClaudeSession(name, sessionId, opts = {}) {
   let cmd
   try {