@cocorograph/hub-agent 0.8.88 → 0.8.90

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.8.88",
+  "version": "0.8.90",
   "description": "Hub Hosted Cockpit のローカル常駐 agent。Hub と outbound WSS で接続し、ローカルの Claude / Codex チャットと tmux セッションを中継する。",
   "type": "module",
   "license": "UNLICENSED",

package/src/chat-sessions-snapshot.mjs

@@ -50,6 +50,29 @@ function idleRank(item) {
   return typeof item?.idle_ms === "number" && item.idle_ms >= 0 ? item.idle_ms : Infinity
 }
 
+/**
+ * この記述子が、その cwd の表示信号の持ち主 (= 代表会話) か。
+ *
+ * ⚠️ 畳み込みで信号の持ち主を落とすと、frontend (chatSessionsAdapter.ts の
+ * signalBelongsTo) が「別の会話の信号」として捨て、生成中でも turn_active=false の赤に
+ * なる。idle_ms は attach 中の端末の活性からしか作られず (claude-stream-bridge.mjs
+ * `_describeCandidate`)、ワークスペースを離れると -1 になるので、idle_ms だけで選ぶと
+ * 離れた後の生成中の会話がほぼ必ず負ける (2026-09-22 に実測)。
+ *
+ * 会話 ID が片方でも未確定なら持ち主とは言えないので false (従来の idle_ms 規則に委ねる)。
+ * 信号の取得失敗も false。失敗の記録は attachChatSignal が行うので、ここでは黙る。
+ */
+function ownsChatSignal(item, cwd, getChatSignal) {
+  const id = conversationIdOf(item)
+  if (!id || !cwd) return false
+  try {
+    const sig = getChatSignal(cwd, item.kind)
+    return typeof sig?.sessionId === "string" && sig.sessionId === id
+  } catch {
+    return false
+  }
+}
+
 /**
  * 記述子の会話 ID。claude は `session_id`、codex は `thread_id` に入る。
  * まだ確定していない会話 (初回入力前の attach 等) は両方とも欠けるので null を返す。
@@ -107,6 +130,23 @@ export async function buildChatSessionsSnapshot({
   const descriptors = describeLiveSessionsUnion(bridges)
   const { targets, errors } = await listChatTargets({ logger })
   const targetByCwd = chatTargetsByCwd(targets)
+  // 畳み込みでは同じ (cwd, kind) の信号を何度も引く (所有者判定 ×2 + 付与)。readRowChatState は
+  // ライブ会話の走査を伴うので、この同期区間の中だけで結果 (例外も含む) を共有する。
+  // ⚠️ await をまたいで使わない。送信直前の付け直しは refreshChatSessionsSignals が行う。
+  const signalMemo = new Map()
+  const signalFor = (cwd, kind) => {
+    const key = `${kind}\u0000${cwd}`
+    if (!signalMemo.has(key)) {
+      try {
+        signalMemo.set(key, { value: getChatSignal(cwd, kind) })
+      } catch (error) {
+        signalMemo.set(key, { error })
+      }
+    }
+    const hit = signalMemo.get(key)
+    if (hit.error) throw hit.error
+    return hit.value
+  }
   const named = descriptors.map((descriptor) => {
     const item = { ...descriptor }
     const cwd = typeof item.cwd === "string" ? item.cwd : ""
@@ -122,10 +162,12 @@ export async function buildChatSessionsSnapshot({
   // 成立していた (同名 worktree は片方しか起動できなかった)。会話を tmux から切り離すと
   // 両方ライブになれるため、agent 側で畳んで不変条件を維持する。
   //
-  // 残す 1 本は「直近に動いた方」。frontend の adaptChatSessions が同名を畳むときと
-  // 同じ規則 (idle_ms が小さい方、同値なら先勝ち) にして、どちらで畳んでも同じ行が残る。
-  // ⚠️ 表示シグナルは cwd キーなので、畳んで残った会話のものが正しく付く (畳み込みは
-  // 行の一意性のためで、帰属のためではない)。
+  // 残す 1 本は、まず「その cwd の表示信号の持ち主 (代表会話)」、次に「直近に動いた方」
+  // (idle_ms が小さい方、同値なら先勝ち)。
+  // ⚠️ 表示シグナルは cwd キーで代表会話の値なので、代表でない会話を残すと frontend が
+  // 帰属不一致で信号を捨てて赤に落ちる (ownsChatSignal 参照)。同じ cwd に会話が複数
+  // 生きるのは通常運用 (warm 境界は cwd あたり K=3) なので、ここが実際の分岐になる。
+  // frontend の adaptChatSessions も同名を畳むが、それは畳まない旧 agent 向けの互換経路。
   const live = []
   const indexByName = new Map()
   for (const { item, cwd, sessionName } of named) {
@@ -137,11 +179,13 @@ export async function buildChatSessionsSnapshot({
     const prior = indexByName.get(sessionName)
     if (prior === undefined) {
       indexByName.set(sessionName, live.length)
-      live.push(attachChatSignal(item, cwd, getChatSignal, logger))
+      live.push(attachChatSignal(item, cwd, signalFor, logger))
       continue
     }
     const kept = live[prior]
-    const replace = idleRank(item) < idleRank(kept)
+    const itemOwns = ownsChatSignal(item, cwd, signalFor)
+    const keptOwns = ownsChatSignal(kept, kept.cwd, signalFor)
+    const replace = itemOwns !== keptOwns ? itemOwns : idleRank(item) < idleRank(kept)
     const winner = replace ? item : kept
     const loser = replace ? kept : item
     // ⚠️ cwd だけでは衝突の正体が分からない。想定していたのは「別 workspace 配下の同名
@@ -163,7 +207,7 @@ export async function buildChatSessionsSnapshot({
       },
       "chat.sessions: session name collision (folded)",
     )
-    if (replace) live[prior] = attachChatSignal(item, cwd, getChatSignal, logger)
+    if (replace) live[prior] = attachChatSignal(item, cwd, signalFor, logger)
   }
   // chat-targets.mjs の fs.readdir(hubProjectsBase()) と同じ絶対パスに揃える（~ は展開しない）。
   const projects_base = path.resolve(hubProjectsBase())

package/src/dispatch-scheduler.mjs

@@ -43,7 +43,7 @@ const SLOW_DISPATCH_MS = 1000
  * 調査対象そのものなので、速かった場合も含めて毎回記録する
  * (速いことの記録が無いと「詰まっていない」を示せない)。
  */
-const ALWAYS_TIMED_TYPES = new Set(["tmux.kill_session"])
+const ALWAYS_TIMED_TYPES = new Set(["chat.session.stop"])
 
 export function isConnectionLifecycleMessage(type) {
   return LIFECYCLE_TYPES.has(type)
@@ -184,7 +184,7 @@ export class DispatchScheduler {
     const mutationMode = globalMutationMode(type)
     if (mutationMode) return `${mutationMode}:global`
 
-    // usage build は最大 25 秒かかり、同じ control lane の tmux.list_sessions が
+    // usage build は最大 25 秒かかり、同じ control lane の一覧要求 (当時は tmux.list_sessions) が
     // frontend の 10 秒打ち切りで捨てられていたため分離する (2026-09-04 本番計測)。
     if (type === "usage.request") return "usage"
 
@@ -203,8 +203,7 @@ export class DispatchScheduler {
       }
       return "control"
     }
-    // chat.stop_all は全レーンに跨るため control に置く。その時点で live な会話だけを
-    // 止める契約であり、接続処理中のまだ live でない会話は対象外。sessions.request も control。
+    // chat.sessions.request は control。
     const mode = chatMode(type)
     if (!mode) return "control"
 

package/src/main.mjs

@@ -4398,43 +4398,6 @@ export async function dispatch(msg, ctx) {
         ctx.client.send(result)
         return
       }
-      case "chat.stop_all": {
-        // exit に載る停止理由。bridge 側の既定 ("tui-bind-stop" / "explicit-stop") は
-        // それぞれ TUI 切替・個別停止の名前なので、全停止はここで明示して区別する。
-        const STOP_ALL_REASON = "chat-stop-all"
-        const result = { type: "chat.stop_all.result", request_id: msg.request_id, stopped: 0, failed: [] }
-        try {
-          const live = describeLiveSessionsUnion({ claudeBridge: ctx.claudeBridge, codexBridge: ctx.codexBridge })
-          const claudeCwds = new Set()
-          const codexStreams = new Set()
-          for (const session of live) {
-            let target
-            try {
-              if (session.kind === "claude") {
-                const cwd = session.cwd
-                if (claudeCwds.has(cwd)) continue
-                claudeCwds.add(cwd)
-                // ⚠️ reason を省くと stopSessionsFor の既定 "tui-bind-stop" が exit に載り、
-                // 画面から止めた会話が「tmux へ切り替えたせい」で消えたように見える。
-                target = { cwd, reason: STOP_ALL_REASON }
-                result.stopped += await ctx.claudeBridge.stopSessionsFor(target)
-              } else if (session.kind === "codex") {
-                const stream_id = session.stream_id
-                if (codexStreams.has(stream_id)) continue
-                codexStreams.add(stream_id)
-                target = { stream_id, reason: STOP_ALL_REASON }
-                result.stopped += await ctx.codexBridge.stopSessionForStream(target)
-              }
-            } catch (err) {
-              result.failed.push({ ...target, reason: err?.message || String(err) })
-            }
-          }
-        } catch (err) {
-          result.error = err?.message || String(err)
-        }
-        ctx.client.send(result)
-        return
-      }
       case "chat.sessions.request": {
         try {
           const snapshot = await buildChatSessionsSnapshot({