@cocorograph/hub-agent 0.6.94 → 0.6.96

package/package.json

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

package/src/main.mjs

@@ -1021,9 +1021,26 @@ function startReadinessLoop({ tracker, logger, intervalMs = 700 }) {
  *   pty.exit 受信時に処理する)
  * - tmux 自体が動いてない場合 (listSessionStates → []) は何も push しない
  */
+// (i) 出力フロー検知の有効窓。ペイン出力領域の署名 (output_sig) が直近この時間内に変化して
+// いれば「出力ストリーミング中」とみなし 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)
+
+// 権威的な「応答停止 (ハング)」判定の無進捗しきい値。ターン進行中 (armed) なのに、実際の
+// 進捗 (子プロセス実行中 / 出力フロー / ペインのライブ processing) がこの時間継続して観測
+// されなければ stalled とする。frontend 側の jsonl 無音タイマー (長時間ツールで誤検知し、
+// 本当のハングでは turnActive クリアにより見逃す) を置き換える正本。frontend の
+// TURN_STALL_WARN_MS と揃える。
+const STALL_WARN_MS = Number(process.env.HUB_AGENT_STALL_WARN_MS ?? 90000)
+
 function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, readinessTracker }) {
   const lastByName = new Map() // session_name → {status, context_pct}
   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()
   let stopped = false
 
   // RC-8: WS 再接続のたびに差分送信の基準 (lastByName) をクリアし、次 tick で全 session.state を
@@ -1093,7 +1110,37 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
         // プロセス内省由来の busy レベル (Stop フック/ツール実行中=true)。frontend は
         // これでペイン由来の早期消灯を抑止し、tool/Stop フック中もスピナーを保つ。
         // 高速 readiness loop が維持する値をそのまま相乗せする (この loop では計算しない)。
-        const procBusy = readinessTracker ? readinessTracker.getBusy(s.session_name) : false
+        const childBusy = readinessTracker ? readinessTracker.getBusy(s.session_name) : false
+        // (i) 出力フロー検知: proc 内省は pane_pid 子孫の「プロセス数」で busy を見るため、
+        // ツールを起動しない純テキスト生成 (モデルが出力を流すだけ) を busy として検出できない。
+        // その間に capture-pane のスピナー検出が 1 フレーム取りこぼすと frontend がアイドルと
+        // 誤読してスピナーを消す (生成中なのに消える=症状B)。出力領域の署名が直近変化していれば
+        // 「出力中」とみなし proc_busy に OR する。childBusy=false の純テキスト生成中も
+        // proc_busy=true となり frontend のペイン由来消灯が抑止される。出力が止まれば署名が
+        // 不変になり窓経過後に false へ落ちるので、ターン終了後のスピナー固着は悪化させない。
+        let outputActive = false
+        if (typeof s.output_sig === "string") {
+          const prevFlow = outputFlowByName.get(s.session_name)
+          let changedAt = prevFlow ? prevFlow.changedAt : 0
+          if (!prevFlow || prevFlow.sig !== s.output_sig) changedAt = Date.now()
+          outputFlowByName.set(s.session_name, { sig: s.output_sig, changedAt })
+          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 はこれを応答停止バナーの正本に使う。
+        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 prev = lastByName.get(s.session_name)
         if (
           !prev ||
@@ -1101,7 +1148,8 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
           prev.context_pct !== contextPct ||
           prev.permission_mode !== permissionMode ||
           prev.stable !== stable ||
-          prev.proc_busy !== procBusy
+          prev.proc_busy !== procBusy ||
+          prev.stalled !== stalled
         ) {
           lastByName.set(s.session_name, {
             status,
@@ -1109,6 +1157,7 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             permission_mode: permissionMode,
             stable,
             proc_busy: procBusy,
+            stalled,
           })
           client.send({
             type: "session.state",
@@ -1118,6 +1167,7 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge, rea
             permission_mode: permissionMode,
             stable,
             proc_busy: procBusy,
+            stalled,
           })
         }
       }

package/src/proc-introspect.mjs

@@ -283,7 +283,18 @@ export class ReadinessTracker {
       s.grace = this.graceSamples
       s.clearStreak = 0
       s.activeStreak = 0
-      return { busy: false, ready: false }
+      // cap = 140s armed のまま settle しない異常状態の後始末。stopSeen 済 (= モデルのターンは
+      // 終わっている) なら、ここで ready を 1 回だけ出して frontend のスピナー固着とキュー詰まり
+      // (ready 不発で turnOverride=true が畳めず drain されない) を解く。ユーザーが Claude 経由で
+      // dev server 等を起動しっぱなしにし、その子プロセスで count>baseline が永続して busy が
+      // 解けないケース (症状A の主因) を救済する。stopSeen=false (stop 取りこぼし or 本当に 140s
+      // 走行中) のときは出さない (生成中への誤フラッシュ防止)。
+      let ready = false
+      if (s.stopSeen && !s.emitted) {
+        ready = true
+        s.emitted = true
+      }
+      return { busy: false, ready }
     }
     // ターン中: baseline は上げない (一過性ツールを新 idle に化けさせない)。より低い床は学習。
     if (count < s.baseline) s.baseline = count
@@ -313,6 +324,12 @@ export class ReadinessTracker {
     return this.byName.get(name)?.busy ?? false
   }
 
+  /** ターン進行中か (prompt_submit で arm 後、ready/cap で disarm するまで true)。
+   *  生成中・Stop フック中の両方を含む。state loop が権威的な stall 判定に使う。 */
+  isArmed(name) {
+    return this.byName.get(name)?.armed ?? false
+  }
+
   /** readiness loop が観測対象に含めるべきか (armed / busy / grace 中)。 */
   isActive(name) {
     const s = this.byName.get(name)

package/src/state.mjs

@@ -531,7 +531,7 @@ const _stabilityByName = new Map() // session_name → { sig, since }
 
 /** ペインの出力領域 (入力欄・揮発フッターを除いた上側) の軽量シグネチャ (FNV-1a 32bit)。
  *  末尾 8 行 (入力欄 + スピナー経過秒/ヒント等) を除くので、idle 中は連続 capture が一致する。 */
-function _outputSignature(text) {
+export function _outputSignature(text) {
   const lines = text.split("\n")
   const body = lines.slice(0, Math.max(0, lines.length - 8)).join("\n")
   let h = 0x811c9dc5
@@ -600,6 +600,10 @@ export async function detectSessionState(sessionName, opts = {}) {
         permission_mode:
           hookResult.result.permission_mode ?? defaultPermissionMode,
         stable: _computeStable(sessionName, status, text),
+        // 出力領域 (揮発フッター除外) の署名。state loop が tick 間で比較し、純テキスト
+        // 生成 (子プロセスを spawn しないため proc 内省が busy=false と見るケース) を
+        // 「出力フロー中=busy」として検出するのに使う。
+        output_sig: _outputSignature(text),
       }
     }
   }
@@ -610,6 +614,7 @@ export async function detectSessionState(sessionName, opts = {}) {
     context_pct: defaultContextPct,
     permission_mode: defaultPermissionMode,
     stable: _computeStable(sessionName, status, text),
+    output_sig: _outputSignature(text),
   }
 }