@cocorograph/hub-agent 0.6.45 → 0.6.47

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -1136,6 +1136,51 @@ export class ClaudeStreamBridge extends EventEmitter {
     }
   }
 
+  /**
+   * 指定 cwd / session_id に紐づく SDK セッションを**即停止**する (T04784 TUI bind 用)。
+   *
+   * TUI モードに切り替えた session は対話 TUI (tmux/PTY) の claude が write を担うため、
+   * SDK セッションを生かしたままだと同一 cwd の jsonl を二重ホストして競合する
+   * (DESIGN.md C7)。soft-detach (7 日 resident) では止まらないので、ここで明示 close する。
+   *
+   * @param {{cwd?: string, session_id?: string}} sel
+   * @returns {number} 停止したセッション数
+   */
+  stopSessionsFor({ cwd, session_id } = {}) {
+    const victims = new Set()
+    if (session_id) {
+      const s = this._liveBySession.get(session_id)
+      if (s) victims.add(s)
+    }
+    if (cwd) {
+      for (const s of this._liveBySession.values()) {
+        if (s.cwd === cwd) victims.add(s)
+      }
+      // session_id 未確定 (起動直後) の端末は sessions Map 側で cwd 一致を拾う。
+      for (const s of this.sessions.values()) {
+        if (s.cwd === cwd) victims.add(s)
+      }
+    }
+    let stopped = 0
+    for (const s of victims) {
+      if (s._closed) continue
+      this._dropSessionMappings(s)
+      try {
+        s.close()
+      } catch {
+        /* ignore */
+      }
+      this.emit("exit", {
+        stream_id: s.stream_id,
+        code: 0,
+        reason: "tui-bind-stop",
+        session_id: s.sessionId,
+      })
+      stopped++
+    }
+    return stopped
+  }
+
   /**
    * 新しい Claude セッションを開始する。
    *

package/src/main.mjs

@@ -47,6 +47,7 @@ import {
   listSessions as listTmuxSessions,
   listWorktreeNameHistory,
   listWorktreeStubs,
+  rebindClaudeSession,
   removeWorktree as removeWorktreeDir,
 } from "./tmux.mjs"
 import { TuiPermissionBridge } from "./tui-permission-bridge.mjs"
@@ -451,6 +452,11 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
   jsonlLiveWatchers.start()
   ctx.jsonlLiveWatchers = jsonlLiveWatchers
 
+  // T04784 会話継続バインディングの冪等性ガード: tmux セッション名 → 最後に --resume で
+  // 載せ替えた Claude session_id。TUI ビューが remount のたびに claude.tui.bind を送っても、
+  // 同じ id への再 respawn (= claude 再起動) を抑止する。
+  ctx.tuiReboundSessions = new Map()
+
   const shutdown = async (signal) => {
     logger.info({ signal }, "shutting down")
     await runHookBroadcast(plugins, "onAgentStop", ctx)
@@ -1058,6 +1064,80 @@ async function dispatch(msg, ctx) {
         // jsonl tail も即停止 (閲覧していない session を追従し続けない)。
         ctx.jsonlLiveWatchers?.unwatch({ session_id: msg.session_id })
         return
+      case "claude.tui.bind": {
+        // T04784 会話継続バインディング: TUI モード突入時に 1 回送られる。
+        //   1. 対象 cwd の最新 session_id を確定 (browser 指定があれば優先)。
+        //   2. その session の SDK を完全停止 (二重ホスト/枠流出を防ぐ)。
+        //   3. 裏の tmux claude を `--resume <id>` で同じ会話に載せ替える (read=write 一致)。
+        // ベストエフォート。失敗時は frontend が従来 newest 表示に degrade する。
+        const request_id = msg.request_id
+        const cwd = typeof msg.cwd === "string" ? msg.cwd : ""
+        const sessionName =
+          typeof msg.session_name === "string" ? msg.session_name : ""
+        const reply = (payload) =>
+          ctx.client.send({
+            type: "claude.tui.bind.result",
+            request_id,
+            ...payload,
+          })
+        try {
+          // 1) 載せ替え先 session_id を確定。
+          let targetId =
+            typeof msg.session_id === "string" && msg.session_id
+              ? msg.session_id
+              : null
+          if (!targetId && cwd) {
+            const projectsRoot = await getActiveProjectsRoot()
+            const { sessions } = await listSessions({
+              cwd,
+              projectsRoot,
+              limit: 1,
+              logger,
+            })
+            targetId = sessions?.[0]?.session_id || null
+          }
+          // 2) SDK を停止 (cwd 全体 + 対象 id)。
+          let stoppedSdk = 0
+          if (ctx.claudeBridge) {
+            stoppedSdk = ctx.claudeBridge.stopSessionsFor({
+              cwd,
+              session_id: targetId || undefined,
+            })
+          }
+          // 3) tmux claude を --resume で載せ替え (targetId があるときだけ)。
+          //    冪等性: 同じ session を同じ id へ既に載せ替え済みなら respawn しない
+          //    (TUI ビューは remount のたびに bind を送るため、毎回 respawn すると
+          //     claude が再起動ループになり送信不能になる。2026-06-06 leader 実機)。
+          let rebind = { ok: false, skipped: false }
+          if (targetId && sessionName) {
+            if (ctx.tuiReboundSessions.get(sessionName) === targetId) {
+              rebind = { ok: true, skipped: true }
+            } else {
+              rebind = await rebindClaudeSession(sessionName, targetId, {
+                cwd,
+                model: ctx.config?.claude_model || "",
+                permissionMode: ctx.config?.claude_permission_mode || "",
+                logger,
+              })
+              if (rebind.ok) ctx.tuiReboundSessions.set(sessionName, targetId)
+            }
+          }
+          reply({
+            session_id: targetId,
+            stopped_sdk: stoppedSdk,
+            rebound: !!rebind.ok,
+            skipped: !!rebind.skipped,
+            error: rebind.ok ? undefined : rebind.error,
+          })
+        } catch (err) {
+          logger?.warn(
+            { err: err?.message, sessionName, cwd },
+            "claude.tui.bind failed",
+          )
+          reply({ session_id: null, error: err?.message || String(err) })
+        }
+        return
+      }
       case "claude.mcp.status":
       case "claude.mcp.reconnect":
       case "claude.mcp.authenticate":

package/src/tmux.mjs

@@ -679,3 +679,95 @@ export async function createSession(name, cwd, opts = {}) {
     await execFileP(tmuxBin(opts), ["send-keys", "-t", name, claudeCmd, "Enter"])
   }
 }
+
+/** session_id として安全な形だけ許可する (send-keys へ流すためコマンド注入を防ぐ)。
+ *  Claude の session_id は UUID (ASCII 英数 + ハイフン) なので、それ以外は弾く。 */
+export function isSafeSessionId(sessionId) {
+  return typeof sessionId === "string" && /^[A-Za-z0-9_-]{8,128}$/.test(sessionId)
+}
+
+/** agent 設定 (model / permissionMode) を claude のフラグ文字列に変換する ("" or "--model X ...")。 */
+export function composeClaudeFlags(opts = {}) {
+  const model = (opts.model || "").trim()
+  const mode = (opts.permissionMode || "").trim()
+  const flags = []
+  if (model) flags.push(`--model ${model}`)
+  if (mode) flags.push(`--permission-mode ${mode}`)
+  return flags.join(" ")
+}
+
+/**
+ * TUI 載せ替え用の `claude --resume <id> [flags]` コマンド文字列を組み立てる (純粋関数)。
+ * session_id を明示 resume することで、表示(read)と書込(write)の jsonl を一致させる
+ * (`--continue` は解決後 id 不明で read/write が乖離しうるため不可)。
+ *
+ * @param {string} sessionId
+ * @param {{model?:string,permissionMode?:string}} [opts]
+ * @returns {string}
+ * @throws session_id が安全形でなければ throw (コマンド注入防止)
+ */
+export function buildResumeCmd(sessionId, opts = {}) {
+  if (!isSafeSessionId(sessionId)) {
+    throw new Error(`unsafe session_id for resume: ${String(sessionId)}`)
+  }
+  const flags = composeClaudeFlags(opts)
+  return `claude --resume ${sessionId}${flags ? " " + flags : ""}`.trim()
+}
+
+const _delay = (ms) => new Promise((r) => setTimeout(r, ms))
+
+/**
+ * tmux セッションの対話 claude を、指定 session_id に `--resume` で載せ替える
+ * (T04784 TUI resume binding)。
+ *
+ * 旧実装 (Ctrl-C 連打で claude REPL を抜ける) は不安定だった: `claude --continue || claude`
+ * の `|| claude` フォールバックや生成中の状態で REPL を抜けきれず、resume コマンドが
+ * **動作中の claude の入力欄に文字列として打ち込まれて会話に混入**する事故が出た
+ * (2026-06-06 finance 実機 / leftClaude:false でも送ってしまっていた)。
+ *
+ * 新実装は `tmux respawn-pane -k` でペインのプロセスを**原子的に kill → 既定シェルへ置換**し、
+ * クリーンなシェルに `claude --resume <id> [flags]` を送る。文字列混入が起きず決定的。
+ * respawn はペイン自体を残すので browser の隠し端末 (tmux attach) は繋がったまま、
+ * かつ claude 終了後もシェルが残る (ペイン消滅でセッションが落ちるのを防ぐ)。
+ *
+ * ⚠️ 冪等性 (remount のたびに再起動しない) は呼び出し側 (main.mjs の
+ * tuiReboundSessions マップ) で担保する。本関数は呼ばれたら必ず respawn する。
+ *
+ * ベストエフォート。失敗しても throw せず {ok:false} を返し、上位は newest 表示に degrade する。
+ *
+ * @param {string} name tmux セッション名
+ * @param {string} sessionId 載せ替え先 Claude session_id
+ * @param {{cwd?:string,model?:string,permissionMode?:string,logger?:object,tmuxBin?:string}} [opts]
+ * @returns {Promise<{ok:boolean, cmd?:string, error?:string}>}
+ */
+export async function rebindClaudeSession(name, sessionId, opts = {}) {
+  let cmd
+  try {
+    cmd = buildResumeCmd(sessionId, opts) // session_id 検証もここで実施
+  } catch (err) {
+    return { ok: false, error: err?.message || String(err) }
+  }
+  const bin = tmuxBin(opts)
+  try {
+    // 1) ペインを既定シェルへ原子置換 (-k で現 claude を強制 kill)。コマンド未指定なので
+    //    クリーンなシェルが起動する。-c でセッションの cwd を維持。
+    const respArgs = ["respawn-pane", "-k", "-t", name]
+    if (opts.cwd) respArgs.push("-c", opts.cwd)
+    await execFileP(bin, respArgs)
+    // 2) シェル準備待ち (rc 読み込み等)。send-keys は pty にバッファされるので過敏でなくてよい。
+    await _delay(300)
+    // 3) クリーンなシェルに resume コマンドを送って claude を起動。
+    await execFileP(bin, ["send-keys", "-t", name, cmd, "Enter"])
+    opts.logger?.info(
+      { session: name, resume: sessionId },
+      "tui rebind: respawned pane shell + claude --resume",
+    )
+    return { ok: true, cmd }
+  } catch (err) {
+    opts.logger?.warn(
+      { session: name, resume: sessionId, err: err?.message },
+      "tui rebind failed",
+    )
+    return { ok: false, error: err?.message || String(err) }
+  }
+}