@cocorograph/hub-agent 0.8.87 → 0.8.89

package/package.json

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

package/src/main.mjs

@@ -1572,7 +1572,10 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
       const data = JSON.parse(text)
       if (!data || typeof data.event !== "string") return
       const at = typeof data.at === "number" ? data.at : Date.now()
-      await publishSessionEvent({ client, readinessTracker, logger }, sessionName, data.event, at)
+      // チャットの行は state loop が返答付きの stop を送る。ここでも送ると Hub が完了通知を
+      // 2 通 (定型文だけ + 返答付き) 出すため、送信だけ省く (並び順・readiness は取り込む)。
+      const send = !(data.event === "stop" && chatStopNotifyingSessionNames.has(sessionName))
+      await publishSessionEvent({ client, readinessTracker, logger }, sessionName, data.event, at, { send })
     } catch {
       // 一時欠如 / parse 失敗等は無視 (次の変更で読み直す)
     }
@@ -1613,13 +1616,20 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
   }
 }
 
+/**
+ * state loop が直近の tick で観測したチャット行の session 名。これらの行の stop は
+ * state loop が返答付きで送るので、bundle hook 由来の stop は WS へ送らない。
+ * 行を観測する前 (起動直後の最初の tick まで) は空なので、hook の stop がそのまま送られる。
+ */
+export const chatStopNotifyingSessionNames = new Set()
+
 /** session-event をプロセス内で取り込み、並び順を更新して WS push する。 */
 export async function publishSessionEvent(
   { client, readinessTracker, logger },
   sessionName,
   event,
   at,
-  { fresh = false, arm = !fresh, text = "" } = {},
+  { fresh = false, arm = !fresh, text = "", send = true } = {},
 ) {
   // readiness (proc 内省の ready 判定) を動かすのはファイル監視経路だけ。直接 publish (fresh) は
   // 「人間が送った」事実を並び順へ流すだけで、ターンの開始を意味しない (bridge が拒否すれば
@@ -1651,7 +1661,7 @@ export async function publishSessionEvent(
   }
   // 通知本文に添える最後の返答。空なら項目ごと送らない (Hub は固定文だけにする)。
   if (typeof text === "string" && text) message.text = text
-  client.send(message)
+  if (send) client.send(message)
   return { sortAffecting }
 }
 
@@ -2031,6 +2041,7 @@ export function startStateLoop({ client, logger, intervalMs, claudeBridge, codex
         // 直列実行する (tmux.mjs)。無関係な worktree の Git 処理待ちで全会話の通知観測が止まる。
         // 現在の target 名は workspace / worktree とも sanitizeTmuxName(basename(cwd)) と同値
         // なので、cwd から直接導出する。この同値は test/chat-targets-naming.test.mjs で固定する。
+        const observedChatNames = new Set()
         for (const kind of ["claude", "codex"]) {
           for (const cwd of representativeRows(kind)) {
             // 観測キーは行名でなく cwd+kind。同名 worktree は 1 行へ畳まれるため、
@@ -2042,6 +2053,7 @@ export function startStateLoop({ client, logger, intervalMs, claudeBridge, codex
               previousChatEvents.delete(key)
               continue
             }
+            observedChatNames.add(sessionName)
             const directAt = getPromptAcceptedAt(cwd, kind)
             const previous = previousChatEvents.get(key)
             // 起動時の過去状態は通知せず、次の変化を比較する基準だけを保存する。
@@ -2084,6 +2096,9 @@ export function startStateLoop({ client, logger, intervalMs, claudeBridge, codex
               .catch(err => logger?.warn({ err: err?.message }, "chat session.event publish failed"))
           }
         }
+        // 走査を完走した tick の観測だけで置き換える (途中で例外なら前回の集合を保つ)。
+        chatStopNotifyingSessionNames.clear()
+        for (const name of observedChatNames) chatStopNotifyingSessionNames.add(name)
       } catch (err) {
         logger?.warn({ err: err?.message }, "chat session.event tick failed")
       }