@cocorograph/hub-agent 0.8.88 → 0.8.89

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.8.88",
+  "version": "0.8.89",
   "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())