@cocorograph/hub-agent 0.7.19 → 0.7.20

package/package.json

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

package/src/extract-paragraph.mjs

@@ -86,6 +86,31 @@ export function extractLastAssistantText(paneText) {
   return text
 }
 
+/**
+ * permission/質問カードに出す「直前アシスタント説明」の採用優先順位を決める純関数。
+ *
+ * 症状4 根治: フック (bundle) 由来の context_text は PreToolUse 発火時点で jsonl が
+ * 前ターン止まり (フックがブロック中は claude が現ターンを commit しない=デッドロック)
+ * のため「前ターンの古い説明」を掴む取り違えを起こす。一方 hub-agent が tmux
+ * capture-pane で抜く liveContext は「いま描画中の現ターン本文」なので回答前に
+ * 信頼できる唯一のソース。よって **liveContext を最優先**し、フック由来は後方互換の
+ * フォールバックとしてのみ採用する (旧 bundle が context_text を送ってきても汚染しない)。
+ * 両方とも空/非文字列なら null (= 説明なしで安全縮退)。
+ *
+ * @param {unknown} hookContextText フック (bundle) が同梱した context_text (旧経路)
+ * @param {unknown} liveContext capture-pane から抽出した現ターン説明
+ * @returns {string|null}
+ */
+export function pickContextText(hookContextText, liveContext) {
+  const live =
+    typeof liveContext === "string" && liveContext.trim() ? liveContext : null
+  const hook =
+    typeof hookContextText === "string" && hookContextText.trim()
+      ? hookContextText
+      : null
+  return live || hook || null
+}
+
 export const __test = {
   _MAX_CHARS,
   _MIN_CHARS,

package/src/main.mjs

@@ -20,7 +20,7 @@ import path from "node:path"
 import pino from "pino"
 
 import { detectPlatform, readConfig, writeConfig } from "./config.mjs"
-import { extractLastAssistantText } from "./extract-paragraph.mjs"
+import { extractLastAssistantText, pickContextText } from "./extract-paragraph.mjs"
 import { loadPlugins, runHookBroadcast, runHookChain } from "./plugin-loader.mjs"
 import { WsClient } from "./ws-client.mjs"
 import { PtyBridge } from "./pty-bridge.mjs"
@@ -92,6 +92,7 @@ import {
   contextWindowSize,
   getSessionUsages,
   getUsage,
+  isFreshUnboundBind,
   recordChatRateLimit,
   turnActiveForCwd,
 } from "./usage.mjs"
@@ -706,17 +707,18 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
           /* ignore */
         }
       }
-      // ── 直前アシスタント説明の抽出 (T05xxx 根治) ────────────────────────────
+      // ── 直前アシスタント説明の抽出 (症状4 根治) ─────────────────────────────
       // PreToolUse フック発火時点で jsonl はまだ前ターン止まり (フックがブロック
-      // している間 claude は jsonl を commit しない) のため、フック側の transcript
-      // 抽出は前ターンの古い説明を掴む「取り違え」を起こす。
-      // 代わりに hub-agent がここで tmux capture-pane を打ち、TUI のペイン上に
-      // 既に描画されている「現ターンの assistant 本文段落 (⏺ プレフィックス)」を
-      // 抜き出して context_text に注入する。フックが既に context_text を
-      // 持っている場合 (旧経路) はそれを尊重し、空 / null のときだけ上書きする。
-      // 抽出失敗時は null フォールバック (= 説明なし) で安全縮退。
+      // している間 claude は現ターンを jsonl に commit しない=デッドロック構造) の
+      // ため、フック側の transcript 抽出は前ターンの古い説明を掴む「取り違え」を
+      // 起こす。代わりに hub-agent がここで tmux capture-pane を打ち、TUI のペイン上
+      // に既に描画されている「現ターンの assistant 本文段落 (⏺ プレフィックス)」を
+      // 抜き出す。これが回答前に説明を届ける唯一の信頼ソースなので、フック由来の
+      // context_text の有無に関わらず **常に** capture-pane を試み、pickContextText で
+      // liveContext を最優先する (旧 bundle が古い context_text を送ってきても汚染
+      // しない)。抽出失敗時は null フォールバック (= 説明なし) で安全縮退。
       let liveContext = null
-      if (!context_text && cwd) {
+      if (cwd) {
         try {
           const sessions = await listTmuxSessions({ logger })
           const match = Array.isArray(sessions)
@@ -742,9 +744,9 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
         tool_name,
         input,
         // 質問/承認カードの直前アシスタント説明 (browser がカード上部に表示)。
-        // 優先順位: (1) フックが既に同梱した context_text、(2) hub-agent が
-        // tmux capture-pane で抽出した liveContext、(3) null。
-        context_text: context_text || liveContext || null,
+        // 優先順位: (1) capture-pane で抽出した現ターンの liveContext、(2) フック由来
+        // の context_text (後方互換フォールバック)、(3) null。pickContextText 参照。
+        context_text: pickContextText(context_text, liveContext),
       })
     },
   )
@@ -1366,6 +1368,11 @@ export function startStateLoop({ client, plugins, logger, intervalMs, claudeBrid
           turnActive = true
         } else if (chat?.status === "waiting" || chat?.status === "idle") {
           turnActive = false
+        } else if (s.cwd && isFreshUnboundBind(boundSessions?.get(s.session_name))) {
+          // 症状2b 根治: fresh bind プレースホルダの間は、別アクティビティ jsonl を mtime 最新で
+          // 取り違える汚染を避けて turn_active を出さない (null=不明)。実セッション id へ adopt
+          // されれば下の通常経路に乗る。青を出さない安全側にのみ倒すため固着青は生まない。
+          turnActive = null
         } else if (s.cwd) {
           turnActive = await turnActiveForCwd(
             s.cwd,
@@ -1442,6 +1449,15 @@ export function startStateLoop({ client, plugins, logger, intervalMs, claudeBrid
           const liveNames = new Set(states.map((s) => s.session_name))
           for (const name of [...lastByName.keys()]) {
             if (liveNames.has(name)) continue
+            // 終端を能動 broadcast (症状3 根治): tmux から消えた = crash/外部 kill/別経路
+            // 終了。explicit kill は kill_session ハンドラが session.gone を出すが、それ以外
+            // の消滅はここが唯一の検知点。frontend が states/lastEvent を忘れてドットを
+            // 'down' に倒せるよう、ローカル Map を forget する前に通知する。
+            try {
+              client.send({ type: "session.gone", session_name: name })
+            } catch {
+              /* ignore */
+            }
             invalidateSessionCache(name) // capture/cwd/statusGate/spinnerFreeze/stability (state.mjs)
             stallTracker.forget(name) // StallTracker.byName
             readinessTracker?.forget?.(name) // ReadinessTracker.byName
@@ -1752,6 +1768,51 @@ export async function handleQueueFlush(msg, ctx) {
   })
 }
 
+/**
+ * tmux.kill_session の処理 (症状3 根治)。要求された session を kill し、要求元へ
+ * tmux.kill_session.result を返したうえで、kill できた各 session について
+ * **session.gone を能動 broadcast** する。
+ *
+ * 不変条件: session の終端 (kill/crash/外部終了) は、同一ホストで実体を観測できる
+ * agent が正本として能動 push する。これが無いと frontend は kill 時点の turn_active=true
+ * を keep-last したままステータスドットが青で固着する。最後の 1 セッションを kill した
+ * 場合 state loop の GC は states.length>0 ガードで発火しないため、explicit kill 経路の
+ * ここから session.gone を出すのが確実 (GC は crash/外部 kill の補完)。
+ *
+ * テストから直接呼べるよう export。kill 実体 (killManySessions) は ctx で差し替え可能。
+ */
+export async function handleKillSession(msg, ctx) {
+  const killFn = ctx.killManySessions || killManySessions
+  const names = Array.isArray(msg.session_names)
+    ? msg.session_names
+    : msg.session_name
+      ? [msg.session_name]
+      : []
+  if (names.length === 0) {
+    ctx.client.send({
+      type: "tmux.kill_session.result",
+      request_id: msg.request_id,
+      killed: [],
+      failed: [{ name: "", reason: "session_name(s) required" }],
+    })
+    return
+  }
+  const r = await killFn(names)
+  ctx.client.send({
+    type: "tmux.kill_session.result",
+    request_id: msg.request_id,
+    killed: r.killed,
+    failed: r.failed,
+  })
+  for (const name of r.killed) {
+    try {
+      ctx.client.send({ type: "session.gone", session_name: name })
+    } catch {
+      /* ignore */
+    }
+  }
+}
+
 /**
  * claude.tui.interrupt の処理 (確認付き中断 = Phase3)。生 ESC を tracked pty.data で best-effort
  * 送出する旧経路 (stream 欠落で無言ドロップ + ESC 到達でも止まったか未検証) を、agent が ESC を
@@ -3191,27 +3252,7 @@ async function dispatch(msg, ctx) {
         return
       }
       case "tmux.kill_session": {
-        const names = Array.isArray(msg.session_names)
-          ? msg.session_names
-          : msg.session_name
-            ? [msg.session_name]
-            : []
-        if (names.length === 0) {
-          ctx.client.send({
-            type: "tmux.kill_session.result",
-            request_id: msg.request_id,
-            killed: [],
-            failed: [{ name: "", reason: "session_name(s) required" }],
-          })
-          return
-        }
-        const r = await killManySessions(names)
-        ctx.client.send({
-          type: "tmux.kill_session.result",
-          request_id: msg.request_id,
-          killed: r.killed,
-          failed: r.failed,
-        })
+        await handleKillSession(msg, ctx)
         return
       }
       case "skills.request": {

package/src/usage.mjs

@@ -603,6 +603,28 @@ export function boundSessionId(boundSessions, name) {
   return typeof v === "string" && v && !v.startsWith("fresh:") ? v : null
 }
 
+/**
+ * その tmux セッションが「fresh bind プレースホルダ (`fresh:<req>`)」の状態か判定する
+ * (症状2b 根治)。
+ *
+ * fresh bind は「新規セッションを開始する」意思表示で、まだ実セッション id へ adopt されて
+ * おらず、このペインが走らせている jsonl が確定していない。この状態で turn_active を
+ * `resolveTargetJsonl` の mtime 最新フォールバックから導くと、同一 cwd-encode dir に堆積した
+ * 別アクティビティ (headless `claude -p` / subagent / 並走セッション / 旧 jsonl) を「最新」と
+ * 取り違えて **偽の生成中 (青ドット) を出し、新規セッションで無操作なのに三点リーダーが固着** する。
+ *
+ * よって呼び出し側 (state loop) は fresh プレースホルダの間 turn_active を null (不明) に倒す。
+ * これは安全側 (青を出さない方向) にしか倒さないため、偽陽性の青固着を生むことはない。生成中の
+ * in-chat スピナーは frontend が turnOverride / session.event(prompt_submit) で別途駆動するため、
+ * fresh セッションの実生成中表示は保たれる (本判定はサイドバー/ヘッダーのドット権威のみに効く)。
+ *
+ * @param {string|null|undefined} boundVal `boundSessions.get(session_name)` の生値
+ * @returns {boolean}
+ */
+export function isFreshUnboundBind(boundVal) {
+  return typeof boundVal === "string" && boundVal.startsWith("fresh:")
+}
+
 /**
  * 指定 cwd の Claude セッション jsonl から context% (USED %) を算出する。
  *