@cocorograph/hub-agent 0.6.98 → 0.6.99

package/package.json

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

package/src/main.mjs

@@ -38,6 +38,7 @@ import {
   detectPermissionModeFromText,
   detectSessionState,
   listSessionStates,
+  StallTracker,
 } from "./state.mjs"
 import {
   DEFAULT_PROFILE_ID,
@@ -1050,8 +1051,9 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
   const lastTurnAtByName = new Map() // session_name → 最後に event ファイル化した turnAt
   // (i) session_name → {sig, changedAt}: 出力領域署名の最終変化時刻 (出力フロー検知用)。
   const outputFlowByName = new Map()
-  // 権威的 stall 判定: session_name → 最後に「進捗あり」と観測した時刻 (ms)。
-  const lastProgressAtByName = new Map()
+  // 権威的 stall 判定 (arm 非依存・cap 非依存)。state loop が観測する signal だけで turn の開閉と
+  // 無進捗を判定する。旧 lastProgressAtByName + armed ゲートの「フック依存 + cap 140s で消える」死角を解消。
+  const stallTracker = new StallTracker({ stallWarnMs: STALL_WARN_MS })
   let stopped = false
 
   // RC-8: WS 再接続のたびに差分送信の基準 (lastByName) をクリアし、次 tick で全 session.state を
@@ -1138,20 +1140,23 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
           outputActive = Date.now() - changedAt < OUTPUT_ACTIVE_MS
         }
         const procBusy = childBusy || outputActive
-        // 権威的 stall 判定。armed (ターン進行中=prompt_submit 受信後 stop/ready 未) なのに
-        // 進捗 (procBusy=子プロセス/出力フロー or ペインの live processing) が STALL_WARN_MS
-        // 継続して無いとき stalled=true。armed でない / 進捗ありのときは進捗時刻を更新して
-        // 無進捗カウントをリセットする。frontend はこれを応答停止バナーの正本に使う。
+        // 権威的 stall 判定 (StallTracker, arm 非依存・cap 非依存)。armed (バンドルフックが効く
+        // セッション) は OR で常に turn 開扱い (従来挙動の上位互換)。フック未発火/旧世代hookセッション
+        // でも観測した processing で turnOpen を立て、frozen スピナー (spinner_present) を維持して proc 子も
+        // 出力も止まったハングを拾う。proc 内省の cap(140s) disarm に依存しないので 140s 以後も消えない。
+        // 旧実装の「armed 依存 + cap で消える」二重死角を解消。frontend はこれを応答停止バナーの正本に使う。
         const armed = readinessTracker
           ? readinessTracker.isArmed(s.session_name)
           : false
         const nowMs = Date.now()
-        const progressing = procBusy || status === "processing"
-        if (progressing || !armed) {
-          lastProgressAtByName.set(s.session_name, nowMs)
-        }
-        const lastProgressAt = lastProgressAtByName.get(s.session_name) ?? nowMs
-        const stalled = armed && nowMs - lastProgressAt >= STALL_WARN_MS
+        const { stalled } = stallTracker.observe(s.session_name, {
+          armed,
+          processing: status === "processing",
+          procBusy,
+          spinnerPresent: s.spinner_present === true,
+          stable,
+          now: nowMs,
+        })
         const prev = lastByName.get(s.session_name)
         if (
           !prev ||

package/src/state.mjs

@@ -597,6 +597,10 @@ export async function detectSessionState(sessionName, opts = {}) {
   }
   const defaultContextPct = detectContextPctFromText(text)
   const defaultPermissionMode = detectPermissionModeFromText(text)
+  // スピナー行 (ライブ or 凍結) が画面に在るか。frozen 降格で status が waiting に落ちても、
+  // スピナーが画面に残っている = ターンはクリーンに終わっていない (ハング候補)。StallTracker が
+  // 「クリーン idle (スピナー無し) でターンを閉じる」判定に使い、frozen スピナーのハングを拾う。
+  const spinnerPresent = workingSpinnerLine(text) !== null
 
   if (opts.plugins && opts.plugins.length) {
     const hookResult = await runHookChain(opts.plugins, "transformStatusDetection", {
@@ -616,6 +620,7 @@ export async function detectSessionState(sessionName, opts = {}) {
         permission_mode:
           hookResult.result.permission_mode ?? defaultPermissionMode,
         stable: _computeStable(sessionName, status, text),
+        spinner_present: spinnerPresent,
         // 出力領域 (揮発フッター除外) の署名。state loop が tick 間で比較し、純テキスト
         // 生成 (子プロセスを spawn しないため proc 内省が busy=false と見るケース) を
         // 「出力フロー中=busy」として検出するのに使う。
@@ -630,6 +635,7 @@ export async function detectSessionState(sessionName, opts = {}) {
     context_pct: defaultContextPct,
     permission_mode: defaultPermissionMode,
     stable: _computeStable(sessionName, status, text),
+    spinner_present: spinnerPresent,
     output_sig: _outputSignature(text),
   }
 }
@@ -648,4 +654,67 @@ export async function listSessionStates(opts = {}) {
   )
 }
 
+/**
+ * arm 非依存・cap 非依存の「応答停止 (stall)」判定ステート。
+ *
+ * 背景: 旧実装の stalled は (a) proc 内省の armed (= バンドルフックの prompt_submit 由来) に全依存し、
+ * フック未発火/旧世代hookセッションでは永久に false、(b) readiness loop の cap(≈140s)で disarm されると
+ * stalled が消える (90〜140s だけ表示で以後恒久消失)、という二重の死角があった。本トラッカーは state loop が
+ * 観測する signal だけで turn の開閉と無進捗を判定し、arm にも cap にも依存しない:
+ *   - turnOpen: processing/procBusy を観測したら true。クリーン idle (非progressing かつ スピナー行が画面に
+ *     無く stable) で false。armed (フックが効くセッション) は OR で常に開扱い (従来挙動の上位互換)。
+ *   - 進捗 (processing or procBusy) があれば lastProgressAt を更新。turn が閉じている間も更新し、次ターンの
+ *     クロックを毎 tick リセットする。
+ *   - stalled = turnOpen かつ 無進捗が stallWarnMs 継続。frozen スピナー (画面に在るが status は frozen 降格で
+ *     waiting) は spinnerPresent=true で turnOpen を維持するため、proc 子が無く出力も止まったハングを拾える。
+ *     cap に依存しないため 140s 以後も消えない。
+ */
+export class StallTracker {
+  constructor({ stallWarnMs = 90_000 } = {}) {
+    this.stallWarnMs = Math.max(1000, stallWarnMs)
+    /** @type {Map<string, {turnOpen: boolean, lastProgressAt: number|null}>} */
+    this.byName = new Map()
+  }
+
+  _get(name) {
+    let s = this.byName.get(name)
+    if (!s) {
+      s = { turnOpen: false, lastProgressAt: null }
+      this.byName.set(name, s)
+    }
+    return s
+  }
+
+  /**
+   * @param {string} name
+   * @param {{armed?: boolean, processing?: boolean, procBusy?: boolean,
+   *          spinnerPresent?: boolean, stable?: boolean, now: number}} sig
+   * @returns {{stalled: boolean, turnOpen: boolean}}
+   */
+  observe(name, { armed = false, processing = false, procBusy = false, spinnerPresent = false, stable = false, now }) {
+    const s = this._get(name)
+    const progressing = processing || procBusy
+    if (progressing) {
+      s.turnOpen = true
+    } else if (!spinnerPresent && stable) {
+      // クリーン idle: 生成中でなく proc 非busy、スピナーも画面に無く出力安定 = ターン正常終了。
+      // frozen スピナー (spinnerPresent=true) や stable 未確定の間は閉じない (ハング候補を維持)。
+      s.turnOpen = false
+    }
+    const turnOpen = armed || s.turnOpen
+    if (progressing || !turnOpen) {
+      // 進捗あり、または turn が閉じている間はクロックを前進させ続ける (次ターンのクロックを毎 tick 0 に保つ)。
+      s.lastProgressAt = now
+    } else if (s.lastProgressAt == null) {
+      s.lastProgressAt = now
+    }
+    const stalled = turnOpen && now - s.lastProgressAt >= this.stallWarnMs
+    return { stalled, turnOpen }
+  }
+
+  forget(name) {
+    this.byName.delete(name)
+  }
+}
+
 export { STATUSES }