@cocorograph/hub-agent 0.7.86 → 0.7.87

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -2401,31 +2401,76 @@ export class ClaudeStreamBridge extends EventEmitter {
    * @param {string} cwd
    * @returns {string|null}
    */
-  liveSessionIdForCwd(cwd) {
+  /**
+   * cwd 内で「最終アクティビティが最も新しいライブセッション」を返す共通スキャン。
+   *
+   * ⚠️ this.sessions **と** _liveBySession の union を走査する (P5 / T06570 症状3): detach で
+   * セッションは this.sessions から外れるが _liveBySession に warm 保持される (_softDetachSession)。
+   * this.sessions だけを見ると「browser を離れた稼働中セッション」を取りこぼし、bound id /
+   * turn_active が落ちて state loop が jsonl を mtime 最新で誤バインドし、非選択行のドットが
+   * 稼働中でも赤へ誤落ちする。reconcileWarmBounds / shutdown と同じ union 走査に揃える。
+   *
+   * @param {string} cwd
+   * @param {boolean} [requireSessionId=true] sessionId 未確定の fresh を除外するか (jsonl バインド用途は true)。
+   * @returns {ClaudeStreamSession|null}
+   */
+  _liveSessionForCwd(cwd, requireSessionId = true) {
     if (!cwd) return null
     let best = null
     let bestAt = -1
     const seen = new Set()
-    for (const session of this.sessions.values()) {
-      if (seen.has(session) || session.cwd !== cwd || !session.sessionId) continue
-      seen.add(session)
-      let at = 0
-      for (const t of session.lastActivityByStream.values()) {
-        if (t > at) at = t
-      }
-      // `>=` (2026-07-03): Date.now() はミリ秒精度のため、短時間に複数セッションへ
-      // attach+input すると同一ミリ秒でタイが起き得る (テストや高速な連続操作で実測)。
-      // `this.sessions` は Map で挿入順を保持するため、タイ時は後から attach された
-      // (= イテレーション順で後に来る) セッションを採用するのが「最終アクティビティが
-      // 最も新しい」の意図に合う。厳密な `>` だと先勝ちになり意図と逆転する。
-      if (at >= bestAt) {
-        bestAt = at
-        best = session.sessionId
+    // this.sessions (attached) を先に走査してから _liveBySession (soft-detached の warm) を走査し
+    // union にする。同一セッションは両 Map に載り得るため seen で識別重複を排除する。
+    const scan = (iter) => {
+      for (const session of iter) {
+        if (seen.has(session) || session.cwd !== cwd) continue
+        if (requireSessionId && !session.sessionId) continue
+        seen.add(session)
+        let at = 0
+        for (const t of session.lastActivityByStream.values()) {
+          if (t > at) at = t
+        }
+        // `>=` (2026-07-03): Date.now() はミリ秒精度のため、短時間に複数セッションへ
+        // attach+input すると同一ミリ秒でタイが起き得る。挿入順/走査順で後に来るものを採用する
+        // のが「最終アクティビティが最も新しい」の意図に合う (attached を先に走査するため、
+        // attached と warm がタイなら warm を採る=どちらも同一 cwd の生存 session で無害)。
+        if (at >= bestAt) {
+          bestAt = at
+          best = session
+        }
       }
     }
+    scan(this.sessions.values())
+    scan(this._liveBySession.values())
     return best
   }
 
+  /**
+   * main.mjs (startStateLoop / tmux.list_sessions ハンドラ) が使うチャット版 bound session 解決。
+   * cwd 内で最終活性が最も新しいライブセッションの sessionId を返す。背景は _liveSessionForCwd 参照。
+   * @param {string} cwd
+   * @returns {string|null}
+   */
+  liveSessionIdForCwd(cwd) {
+    return this._liveSessionForCwd(cwd, true)?.sessionId ?? null
+  }
+
+  /**
+   * P5 (T06570 症状3): cwd のライブチャットセッションの **agent 側ターン進行フラグ** (_busy) を返す。
+   * _busy は SDK query ライフサイクル (ターン開始で true / result・idle・exit で同期 false) 由来で、
+   * browser の attach/detach やストリーム tick に依存しない権威信号。state loop の turn_active に
+   * これを採ることで、browser を離れた稼働中セッションでもドットが青のまま追従し、ターン終了時は
+   * 次 tick で即座に赤へ落ちる (誤青を生まない)。Codex の codexTurnActiveForThread と対称。
+   * fresh (sessionId 未確定) の初回ターンも稼働中として拾うため requireSessionId=false。
+   * @param {string} cwd
+   * @returns {boolean|null} true=ターン進行中 / false=idle / null=このcwdにライブチャットセッション無し (純TUI等)
+   */
+  chatTurnActiveForCwd(cwd) {
+    const s = this._liveSessionForCwd(cwd, false)
+    if (!s) return null
+    return s._busy === true
+  }
+
   /**
    * 指定 cwd でライブなセッションへメッセージを直接注入する (sibling-inject 用、
    * 2026-07-04)。同一マシン内の別プロセス (ask-sibling スキル経由の cockpit_orch.py)
@@ -2445,21 +2490,10 @@ export class ClaudeStreamBridge extends EventEmitter {
    */
   sendToCwd(cwd, text) {
     if (!cwd || !text) return false
-    let best = null
-    let bestAt = -1
-    const seen = new Set()
-    for (const session of this.sessions.values()) {
-      if (seen.has(session) || session.cwd !== cwd) continue
-      seen.add(session)
-      let at = 0
-      for (const t of session.lastActivityByStream.values()) {
-        if (t > at) at = t
-      }
-      if (at >= bestAt) {
-        bestAt = at
-        best = session
-      }
-    }
+    // P5 (T06570): warm (soft-detached) セッションも対象にするため union 走査へ委譲する
+    // (従来は this.sessions のみで、detach 済みだが生きているセッションへ sibling-inject できず
+    // no_live_session_for_cwd になっていた)。sessionId 未確定の fresh も対象にするため false。
+    const best = this._liveSessionForCwd(cwd, false)
     if (!best) return false
     best.sendMessage({ role: "user", content: text }).catch((err) => {
       this.logger?.error(

package/src/main.mjs

@@ -2579,14 +2579,27 @@ export function startStateLoop({ client, plugins, logger, intervalMs, claudeBrid
           case "codex":
             turnActive = await codexTurnActiveForThread(codexThreadId)
             break
-          case "claude":
-            turnActive = await turnActiveForCwd(
-              s.cwd,
-              resolveStateLoopBoundSessionId(boundSessions, s.session_name, s.cwd, claudeBridge),
-              s.session_name,
-              { paneAliveProbe: paneClaudeAliveOrUnknown },
-            )
+          case "claude": {
+            // P5 (T06570 症状3): チャットセッションは、そのセッションの SDK query ライフサイクル由来の
+            // 権威フラグ _busy を最優先する (Codex の rollout flag 優先と対称)。これにより browser を
+            // 離れて detach され this.sessions から外れた warm セッションでも、稼働中ターンの turn_active
+            // が維持され、非選択行のドットが赤へ誤落ちしない。加えて bare-shell pane を dead 判定して
+            // 赤へ倒す paneAliveProbe 経路 (チャットは実体が SDK 子プロセスで pane は素のシェル) も回避する。
+            // chatTurnActiveForCwd が null (= この cwd にライブチャットセッション無し = 純 TUI / bare pane)
+            // のときだけ、従来の jsonl 末尾 + pane probe 経路にフォールバックする (TUI 挙動は不変)。
+            const chatActive = claudeBridge?.chatTurnActiveForCwd?.(s.cwd)
+            if (chatActive === true || chatActive === false) {
+              turnActive = chatActive
+            } else {
+              turnActive = await turnActiveForCwd(
+                s.cwd,
+                resolveStateLoopBoundSessionId(boundSessions, s.session_name, s.cwd, claudeBridge),
+                s.session_name,
+                { paneAliveProbe: paneClaudeAliveOrUnknown },
+              )
+            }
             break
+          }
           default:
             turnActive = null
         }