@cocorograph/hub-agent 0.6.47 → 0.6.49

package/package.json

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

package/src/claude-history.mjs

@@ -241,3 +241,37 @@ export async function listSessions({ cwd, projectsRoot, limit = 30, logger }) {
   }
   return { sessions }
 }
+
+/**
+ * TUI チャット閲覧中に裏の対話 claude が新しい session_id へ「回転」したか
+ * (= `/clear` で新 jsonl が生まれたか) を判定する純関数。
+ *
+ * 背景: TUI チャットは bind 時に決めた session_id を固定して jsonl を読む。ユーザーが
+ * `/clear` すると裏の claude は**新しい session_id (新 jsonl)** に切り替わるが、フロントは
+ * 旧 jsonl を読み続けるため「読み出し先が切り替わらない」「生成中で固まる」「新セッションへの
+ * 送信が表示されない」という不整合が起きる (T04786 後続調査)。これを agent 側で検知して
+ * `claude.session.rotated` を push し、フロントに読み出し先の切替を促すための判定ロジック。
+ *
+ * 判定材料:
+ * - `viewingSessionId`: フロントが今表示している (閲覧ハートビートが運ぶ) session_id。
+ * - `newestSessionId`: cwd 配下で実際に最新 (mtime 降順先頭) の session_id。
+ * - `lastNotifiedNewId`: 同一 new への多重通知を防ぐため、最後に通知した new_session_id。
+ *
+ * 回転とみなす条件: 最新が存在し、閲覧中 id と異なり、かつ直近で同じ new を通知済みでない。
+ * 注意: 過去セッションを意図的に開いている (= newest 非追従) ビューでは呼び出し側が
+ * `follow_newest=false` でこの判定自体をスキップすること (ピン留め閲覧を勝手に最新へ
+ * 引きずらないため)。本関数は「追従中ビュー」前提で newest とのズレだけを見る。
+ *
+ * @param {{viewingSessionId?: string|null, newestSessionId?: string|null, lastNotifiedNewId?: string|null}} args
+ * @returns {{rotated: boolean, newSessionId?: string}}
+ */
+export function decideSessionRotation({
+  viewingSessionId,
+  newestSessionId,
+  lastNotifiedNewId,
+} = {}) {
+  if (!newestSessionId || !viewingSessionId) return { rotated: false }
+  if (newestSessionId === viewingSessionId) return { rotated: false }
+  if (newestSessionId === lastNotifiedNewId) return { rotated: false }
+  return { rotated: true, newSessionId: newestSessionId }
+}

package/src/main.mjs

@@ -24,7 +24,11 @@ import { WsClient } from "./ws-client.mjs"
 import { PtyBridge } from "./pty-bridge.mjs"
 import { ClaudeStreamBridge } from "./claude-stream-bridge.mjs"
 import { UploadManager } from "./claude-upload.mjs"
-import { fetchSessionHistory, listSessions } from "./claude-history.mjs"
+import {
+  decideSessionRotation,
+  fetchSessionHistory,
+  listSessions,
+} from "./claude-history.mjs"
 import { listAgents } from "./agents.mjs"
 import { listSkills } from "./skills.mjs"
 import { listSessionStates } from "./state.mjs"
@@ -457,6 +461,12 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
   // 同じ id への再 respawn (= claude 再起動) を抑止する。
   ctx.tuiReboundSessions = new Map()
 
+  // session 回転通知 (T04786 後続) の多重送信ガード: 回転キー (tmux 名 or cwd) →
+  // 最後に push した new_session_id。閲覧ハートビートは 5s 間隔で来るため、フロントが
+  // 切替を完了し new id でハートビートを送り直すまでの間に来る余分なハートビートで
+  // claude.session.rotated を連発しないようにする。
+  ctx.tuiRotationNotified = new Map()
+
   const shutdown = async (signal) => {
     logger.info({ signal }, "shutting down")
     await runHookBroadcast(plugins, "onAgentStop", ctx)
@@ -1028,6 +1038,7 @@ async function dispatch(msg, ctx) {
               msg.request_id,
               !!msg.allow,
               msg.deny_message,
+              msg.updated_input,
             )
           : false
         if (handledByTui) return
@@ -1045,7 +1056,7 @@ async function dispatch(msg, ctx) {
         if (!ctx.claudeBridge) return
         ctx.claudeBridge.interrupt({ stream_id: msg.stream_id })
         return
-      case "claude.tui.viewing":
+      case "claude.tui.viewing": {
         // T04778 watcher ゲート: browser が TUI チャットでこの session を閲覧中。
         // 閲覧マーカーを更新し、承認フックが「閲覧者あり」と判定できるようにする。
         ctx.tuiViewerRegistry
@@ -1055,7 +1066,59 @@ async function dispatch(msg, ctx) {
         ctx.jsonlLiveWatchers
           ?.note({ session_id: msg.session_id, cwd: msg.cwd })
           .catch(() => {})
+        // T04786 後続: session 回転検知 (/clear で裏 claude が新 session_id へ切替)。
+        // 追従中ビュー (follow_newest 既定 true) のときだけ、cwd の最新 session_id と
+        // 閲覧中 id を突き合わせ、ズレていれば claude.session.rotated を push してフロントに
+        // 読み出し先の切替を促す。過去セッションを意図的に開いているビュー (follow_newest=false)
+        // では検知しない (ピン留め閲覧を最新へ引きずらない)。
+        const viewSid = msg.session_id
+        const viewCwd = msg.cwd
+        const viewName =
+          typeof msg.session_name === "string" ? msg.session_name : ""
+        const followNewest = msg.follow_newest !== false
+        if (followNewest && viewSid && viewCwd) {
+          ;(async () => {
+            try {
+              const projectsRoot = await getActiveProjectsRoot()
+              const { sessions } = await listSessions({
+                cwd: viewCwd,
+                projectsRoot,
+                limit: 1,
+                logger,
+              })
+              const newestId = sessions?.[0]?.session_id || null
+              const key = viewName || viewCwd
+              const { rotated, newSessionId } = decideSessionRotation({
+                viewingSessionId: viewSid,
+                newestSessionId: newestId,
+                lastNotifiedNewId: ctx.tuiRotationNotified.get(key),
+              })
+              if (!rotated) return
+              ctx.tuiRotationNotified.set(key, newSessionId)
+              // 冪等マップも新 id へ更新し、将来の remount bind が旧 id へ
+              // respawn (claude 再起動) してしまうのを防ぐ。
+              if (viewName) ctx.tuiReboundSessions.set(viewName, newSessionId)
+              ctx.client.send({
+                type: "claude.session.rotated",
+                cwd: viewCwd,
+                session_name: viewName || undefined,
+                old_session_id: viewSid,
+                new_session_id: newSessionId,
+              })
+              logger.info(
+                { session: viewName, cwd: viewCwd, old: viewSid, new: newSessionId },
+                "tui session rotated (/clear) → notified browser",
+              )
+            } catch (err) {
+              logger.warn(
+                { err: err?.message, cwd: viewCwd },
+                "tui session rotation check failed",
+              )
+            }
+          })()
+        }
         return
+      }
       case "claude.tui.unviewing":
         // 閲覧終了。session_id マーカーを即失効させ、以降の Bash を即 ask に倒す。
         ctx.tuiViewerRegistry

package/src/tui-permission-bridge.mjs

@@ -114,9 +114,12 @@ export class TuiPermissionBridge extends EventEmitter {
    * @param {string} request_id
    * @param {boolean} allow
    * @param {string} [denyMessage]
+   * @param {unknown} [updatedInput] AskUserQuestion 等の回答 ({questions, answers})。
+   *   フックが PreToolUse の hookSpecificOutput.updatedInput として注入し、claude が
+   *   対話 UI を出さず確定する。Bash 等の allow/deny だけで済むツールでは undefined。
    * @returns {Promise<boolean>} この bridge の管理下だったか
    */
-  async resolve(request_id, allow, denyMessage) {
+  async resolve(request_id, allow, denyMessage, updatedInput) {
     if (!this._pending.has(request_id)) return false
     this._pending.delete(request_id)
     const decision = allow ? "allow" : "deny"
@@ -125,7 +128,13 @@ export class TuiPermissionBridge extends EventEmitter {
     try {
       await writeFile(
         tmp,
-        JSON.stringify({ decision, deny_message: denyMessage ?? null }),
+        JSON.stringify({
+          decision,
+          deny_message: denyMessage ?? null,
+          // 回答 (AskUserQuestion の {questions, answers} 等)。フックが
+          // updatedInput として注入する。無ければ null (Bash 等)。
+          updated_input: updatedInput ?? null,
+        }),
       )
       await rename(tmp, fp)
     } catch (err) {