@cocorograph/hub-agent 0.8.85 → 0.8.86

package/package.json

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

package/src/chat-sessions-snapshot.mjs

@@ -73,6 +73,31 @@ function attachedOf(item) {
   return null
 }
 
+/** listChatTargets の戻り値を cwd 引きの Map にする。 */
+export function chatTargetsByCwd(targets) {
+  return new Map((targets ?? []).map((target) => [target.cwd, target]))
+}
+
+/**
+ * cwd から一覧の行キー (session_name) を解決する。**行名の正本はここ**。
+ *
+ * ⚠️ sessionNameFor は `__codex` を付け外しするだけで basename 化しない。cwd を直接
+ * 渡すと絶対パスがそのまま名前になり、一覧の行にも通知の宛先にも一致しない。
+ *
+ * `targetByCwd` は任意。現在 listChatTargets が返す名前は workspace / worktree とも
+ * `sanitizeTmuxName(basename(cwd))` と同値なので、渡しても渡さなくても結果は変わらない
+ * (この同値は test/chat-targets-naming.test.mjs で固定している)。target 固有の名前を
+ * 将来入れるなら、targets を引かない呼び出し元 (state loop の通知) も同時に直すこと。
+ *
+ * ⚠️ cwd が無い記述子で path.basename に undefined を渡すと throw し、呼び出し元の map ごと
+ * 失敗して会話一覧が丸ごと空になる。名前を解決できない 1 本のために全部を失わない。
+ */
+export function chatRowSessionName(cwd, cliKind, targetByCwd) {
+  const key = typeof cwd === "string" ? cwd : ""
+  const dir = targetByCwd?.get(key)?.name ?? (key ? sanitizeTmuxName(path.basename(key)) : "")
+  return dir ? sessionNameFor(dir, cliKind) : ""
+}
+
 /** ローカル検出した宛先と、毎回取得するライブ会話を返す。 */
 export async function buildChatSessionsSnapshot({
   bridges,
@@ -81,15 +106,11 @@ export async function buildChatSessionsSnapshot({
 }) {
   const descriptors = describeLiveSessionsUnion(bridges)
   const { targets, errors } = await listChatTargets({ logger })
-  const targetByCwd = new Map(targets.map((target) => [target.cwd, target]))
+  const targetByCwd = chatTargetsByCwd(targets)
   const named = descriptors.map((descriptor) => {
     const item = { ...descriptor }
-    // ⚠️ cwd が無い記述子で path.basename に undefined を渡すと throw し、map ごと失敗して
-    // 会話一覧が丸ごと空になる (呼び出し側の catch が targets:[] live:[] を返す)。名前を
-    // 解決できない 1 本のために全部を失わない。
     const cwd = typeof item.cwd === "string" ? item.cwd : ""
-    const dir = targetByCwd.get(cwd)?.name ?? (cwd ? sanitizeTmuxName(path.basename(cwd)) : "")
-    return { item, cwd, sessionName: dir ? sessionNameFor(dir, item.kind) : "" }
+    return { item, cwd, sessionName: chatRowSessionName(cwd, item.kind, targetByCwd) }
   })
 
   // ⚠️ session_name は dir 名から作るので、別 workspace 配下の同名 worktree は同じ名前へ

package/src/main.mjs

@@ -94,7 +94,7 @@ import {
 } from "./claude-credentials.mjs"
 import { createWorktreeDir, listWorktreeNameHistory, removeWorktree as removeWorktreeDir } from "./tmux.mjs"
 import { expandTilde, prepareChatWorkspace, isCwdUnavailableError } from "./chat-workspace.mjs"
-import { buildChatSessionsSnapshot, refreshChatSessionsSignals } from "./chat-sessions-snapshot.mjs"
+import { buildChatSessionsSnapshot, chatRowSessionName, refreshChatSessionsSignals } from "./chat-sessions-snapshot.mjs"
 import { chatTargetsCache } from "./chat-targets.mjs"
 import { ensureChatSession } from "./chat-session-starter.mjs"
 import { beginRemoval } from "./chat-removal-guard.mjs"
@@ -1881,12 +1881,17 @@ export function filterCodexSessionsByAuthority(codexSessions, producerByCwd, seq
  *
  * @returns {{ev: string, at: number}|null}
  */
-export function chatSessionEventForTick({ chat, prevEv, prevTurnAt, directAt }) {
-  const ev = chat.inputPending
+/** 現在のチャット状態から境界イベント名を導く。初回観測の基準値もここから取る。 */
+export function deriveChatEvent(chat) {
+  return chat.inputPending
     ? "input_required"
     : chat.status === "processing"
       ? "prompt_submit"
       : "stop"
+}
+
+export function chatSessionEventForTick({ chat, prevEv, prevTurnAt, directAt }) {
+  const ev = deriveChatEvent(chat)
   if (!(chat.turnAt > prevTurnAt && ev !== prevEv)) return null
   const at = ev === "prompt_submit" && typeof directAt === "number" && directAt > 0
     ? directAt
@@ -1966,6 +1971,7 @@ export function startStateLoop({ client, logger, intervalMs, claudeBridge, codex
   // 保護されていて刈れない」tick は 5 秒ごとに同じ行を出し続けるので、状態が変わったときと
   // 一定間隔でだけ記録する（warm-bounds-total.mjs の _shouldLog）。
   const warmBoundsLogState = { lastUnresolved: 0, lastLoggedAtMs: 0 }
+  const previousChatEvents = new Map()
   let stopped = false
 
   const tick = async () => {
@@ -2012,6 +2018,66 @@ export function startStateLoop({ client, logger, intervalMs, claudeBridge, codex
       // 実コンテキスト窓サイズ (1M ベータ等) を反映。contextPctFromUsage の分母を
       // 5s ごとに最新化し、ドーナツが 200k 固定で振り切れるのを防ぐ。
       await refreshContextWindow()
+      // 配送前に停止契約を取り直す。タイマー解除は進行中の tick を止めないので、
+      // await をまたいだ後の観測で通知を出さないのはここだけが担保する。
+      if (stopped) return
+      try {
+        // ⚠️ ここで listChatTargets を引かないこと。TTL は 3 秒 (chat-targets.mjs) で 5 秒 tick では
+        // 毎回失効し、全 worktree に対して `git branch --show-current` をタイムアウト無しで
+        // 直列実行する (tmux.mjs)。無関係な worktree の Git 処理待ちで全会話の通知観測が止まる。
+        // 現在の target 名は workspace / worktree とも sanitizeTmuxName(basename(cwd)) と同値
+        // なので、cwd から直接導出する。この同値は test/chat-targets-naming.test.mjs で固定する。
+        for (const kind of ["claude", "codex"]) {
+          for (const cwd of representativeRows(kind)) {
+            // 観測キーは行名でなく cwd+kind。同名 worktree は 1 行へ畳まれるため、
+            // 行名で観測を共用すると別 cwd の状態が混ざる。
+            const key = `${kind}\u0000${cwd}`
+            const chat = readRowChatState(cwd, kind)
+            const sessionName = chat ? chatRowSessionName(cwd, kind) : ""
+            if (!chat || !sessionName) {
+              previousChatEvents.delete(key)
+              continue
+            }
+            const directAt = getPromptAcceptedAt(cwd, kind)
+            const previous = previousChatEvents.get(key)
+            // 起動時の過去状態は通知せず、次の変化を比較する基準だけを保存する。
+            if (!previous) {
+              previousChatEvents.set(key, { ev: deriveChatEvent(chat), turnAt: chat.turnAt, status: chat.status })
+              continue
+            }
+            const event = chatSessionEventForTick({
+              chat, prevEv: previous.ev, prevTurnAt: previous.turnAt, directAt,
+            })
+            // ⚠️ status は event の有無に関わらず毎 tick 更新する。イベントが出たときだけ
+            // 更新すると、代表の終了が「イベントなし」で素通りしたときに古い status が残る。
+            if (!event) {
+              previousChatEvents.set(key, { ...previous, status: chat.status })
+              continue
+            }
+            // 代表会話が終了して実体が戻ってきた遷移。ターンの完了ではない。
+            const revived = previous.status === null && chat.status !== null
+            previousChatEvents.set(key, { ev: event.ev, turnAt: chat.turnAt, status: chat.status })
+            // ⚠️ 行は代表会話が終了しても消えず、status だけが null になる。これを "完了" と
+            // して送ると、処理中の会話を閉じただけで完了通知が出る。復帰も同じで、終了中に
+            // 承認待ちを送った後に別の会話が waiting で代表になると、ev が
+            // input_required → stop と変わって偽の完了になる。
+            // 一方 inputPending は同 cwd の生存会話すべてから集約されるので、代表終了後に
+            // 背景の会話が承認待ちになったときは送る必要がある。stop だけを絞る。
+            // prompt_submit はこの経路の対象外 (確定仕様として stop / input_required のみ通知する)。
+            const deliverable = event.ev === "input_required"
+              || (event.ev === "stop" && chat.status !== null && !revived)
+            if (!deliverable) continue
+            // ⚠️ chat の読み取りから publish まで await を挟まない。
+            // 既知の取りこぼし: 前回 tick も今回 tick も waiting で、その間にターンが
+            // 開始・完了すると ev が stop のまま変わらないため、通知は遅延ではなく欠落する。
+            // 5 秒 tick を前提にした仕様。
+            publishSessionEvent({ client, logger }, sessionName, event.ev, event.at, { arm: false })
+              .catch(err => logger?.warn({ err: err?.message }, "chat session.event publish failed"))
+          }
+        }
+      } catch (err) {
+        logger?.warn({ err: err?.message }, "chat session.event tick failed")
+      }
     } catch (err) {
       logger?.warn({ err: err.message }, "state loop tick failed")
     }