@cocorograph/hub-agent 0.6.46 → 0.6.47

package/package.json

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

package/src/main.mjs

@@ -452,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)
@@ -1100,18 +1105,28 @@ async function dispatch(msg, ctx) {
             })
           }
           // 3) tmux claude を --resume で載せ替え (targetId があるときだけ)。
-          let rebind = { ok: false }
+          //    冪等性: 同じ session を同じ id へ既に載せ替え済みなら respawn しない
+          //    (TUI ビューは remount のたびに bind を送るため、毎回 respawn すると
+          //     claude が再起動ループになり送信不能になる。2026-06-06 leader 実機)。
+          let rebind = { ok: false, skipped: false }
           if (targetId && sessionName) {
-            rebind = await rebindClaudeSession(sessionName, targetId, {
-              model: ctx.config?.claude_model || "",
-              permissionMode: ctx.config?.claude_permission_mode || "",
-              logger,
-            })
+            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) {

package/src/tmux.mjs

@@ -716,45 +716,29 @@ export function buildResumeCmd(sessionId, opts = {}) {
 
 const _delay = (ms) => new Promise((r) => setTimeout(r, ms))
 
-/** pane_current_command が claude でなくなる (= シェルに戻った) のを待つ。 */
-async function _waitPaneLeftClaude(name, opts = {}, timeoutMs = 4000) {
-  const bin = tmuxBin(opts)
-  const start = Date.now()
-  while (Date.now() - start < timeoutMs) {
-    try {
-      const { stdout } = await execFileP(bin, [
-        "display-message",
-        "-p",
-        "-t",
-        name,
-        "#{pane_current_command}",
-      ])
-      const cmd = (stdout || "").trim().toLowerCase()
-      if (cmd && !cmd.includes("claude")) return true
-    } catch {
-      /* ignore */
-    }
-    await _delay(200)
-  }
-  return false
-}
-
 /**
  * tmux セッションの対話 claude を、指定 session_id に `--resume` で載せ替える
  * (T04784 TUI resume binding)。
  *
- * 手順 (in-pane 再起動。PTY/attach は維持するので browser の隠し端末は繋がったまま):
- *   1. Ctrl-C を数回送り、生成中断 → claude REPL を抜けてシェルへ戻す
- *      (claude は空入力 Ctrl-C 2 回で exit する)。
- *   2. pane_current_command が claude でなくなるのを待つ (最大 timeoutMs)。
- *   3. 行頭をクリア (C-u) してから `claude --resume <id> [flags]` を送って起動。
+ * 旧実装 (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 {{model?:string,permissionMode?:string,logger?:object,tmuxBin?:string}} [opts]
- * @returns {Promise<{ok:boolean, leftClaude?:boolean, cmd?:string, error?:string}>}
+ * @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
@@ -764,25 +748,21 @@ export async function rebindClaudeSession(name, sessionId, opts = {}) {
     return { ok: false, error: err?.message || String(err) }
   }
   const bin = tmuxBin(opts)
-  const send = (...keys) =>
-    execFileP(bin, ["send-keys", "-t", name, ...keys]).catch(() => {})
   try {
-    // 1) 生成中断 + REPL 終了 (Ctrl-C 連打)。
-    for (let i = 0; i < 4; i++) {
-      await send("C-c")
-      await _delay(180)
-    }
-    // 2) シェルに戻るのを待つ。
-    const leftClaude = await _waitPaneLeftClaude(name, opts, 4000)
-    // 3) 行をクリアして resume コマンドを起動。
-    await send("C-u")
-    await _delay(80)
+    // 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, leftClaude },
-      "tui rebind: relaunched claude with --resume",
+      { session: name, resume: sessionId },
+      "tui rebind: respawned pane shell + claude --resume",
     )
-    return { ok: true, leftClaude, cmd }
+    return { ok: true, cmd }
   } catch (err) {
     opts.logger?.warn(
       { session: name, resume: sessionId, err: err?.message },