@cocorograph/hub-agent 0.8.18 → 0.8.20

package/src/tmux.mjs

@@ -23,12 +23,8 @@ import { fileURLToPath } from "node:url"
 
 import { ensureClaudeMd } from "./claude-md.mjs"
 import { syncClaudeMdWithHub } from "./claude-md-sync.mjs"
-import {
-  capturePane,
-  detectInputBoxText,
-  detectSessionState,
-  getSessionCwd,
-} from "./state.mjs"
+import { detectSessionState, getSessionCwd } from "./state.mjs"
+import { shouldLaunchPaneCli } from "./pane-cli-flag.mjs"
 import { baseSessionName, sessionNameFor } from "./session-names.mjs"
 import { boundSessionId, getSessionUsages, isFreshUnboundBind, jsonlContextForCwd } from "./usage.mjs"
 
@@ -510,25 +506,9 @@ export function composeClaudeCmdFromOptions(opts = {}) {
 
 const DEFAULT_CLAUDE_CMD = buildClaudeCmd()
 
-/**
- * tmux pane で CLI (claude / codex) の TUI を起動してよいかを判定する。
- *
- * **既定は false**（pane は素のシェルのまま）。Cockpit のターミナルモードは
- * 2026-07-03 に封印済みで、通常運用は常にチャットモード (Agent SDK / codex
- * app-server) になった。それでも createSession は pane で TUI を起動し続けていたため、
- * **誰も見ていない CLI が常駐**する。実測 (2026-08-01 中村さん端末): tmux セッション
- * 33 個に対して pane の `claude` プロセス 31 個・合計 RSS 1.7 GB。
- *
- * `HUB_AGENT_TMUX_LAUNCH_CLI=1` のときだけ従来どおり起動する（デバッグ・TUI を
- * 意図的に使いたいときの逃げ道）。呼び出し元ごとに挙動を変えないため、判定は
- * この 1 箇所 (createSession 内) だけで行い、opts では上書きさせない。
- *
- * @param {NodeJS.ProcessEnv} [env]
- * @returns {boolean}
- */
-export function shouldLaunchPaneCli(env = process.env) {
-  return env?.HUB_AGENT_TMUX_LAUNCH_CLI === "1"
-}
+// 判定の実体は ./pane-cli-flag.mjs (state.mjs との循環 import を避けるため分離)。
+// ここからの re-export は後方互換 (既存の `from "./tmux.mjs"` を壊さない)。
+export { shouldLaunchPaneCli }
 
 function tmuxBin(opts = {}) {
   return opts.tmuxBin || DEFAULT_TMUX_BIN
@@ -797,7 +777,7 @@ export async function killManySessions(names, opts = {}) {
  *
  * ⚠️ **既定では pane に CLI (claude / codex) を起動しない**（pane は素のシェルのまま）。
  * Cockpit の通常運用はチャットモードで、pane の TUI は誰も見ないまま常駐して
- * メモリを食うだけだったため (詳細は shouldLaunchPaneCli の JSDoc)。
+ * メモリを食うだけだったため (詳細は `pane-cli-flag.mjs` の shouldLaunchPaneCli の JSDoc)。
  * `HUB_AGENT_TMUX_LAUNCH_CLI=1` のときだけ従来どおり CLI を send-keys する。
  *
  * - 同名 session が既にあれば 'duplicate session' throw
@@ -1165,74 +1145,6 @@ export function injectCodexHomeEnv(codexCmd, codexHome) {
 
 const _delay = (ms) => new Promise((r) => setTimeout(r, ms))
 
-/**
- * `/model` / `/effort` 切替時に claude TUI が出す確認ダイアログを検知する正規表現。
- *
- * claude v2.1.170 実機確認 (2026-06-10): 会話に出力済みの履歴 (prior output) があると、
- * モデル/effort 切替は即時反映されず確認ダイアログを挟む。ダイアログは:
- *   - タイトル: "Switch model?" / "Change effort level?"
- *   - 本文: "This conversation is cached for the current model. Switching to … means the
- *     full history gets re-read on your next message."
- *   - 選択肢: "Yes, switch to …" (デフォルトハイライト) / "No, go back"
- * デフォルトが確定側なので Enter 1 回で承認できる。複数マーカーのいずれかで判定する
- * (文言の小変更に多少強くする)。
- */
-const SWITCH_CONFIRM_DIALOG_RE =
-  /Switch model\?|Change effort level\?|Yes, switch to|No, go back|full history gets re-read/i
-
-/**
- * `/model` / `/effort` の確認ダイアログを「描画を検知してから」Enter で畳む。
- *
- * 旧実装は本文確定 Enter の一定時間後に盲打ちで追い Enter を 1 回送っていたが、確認
- * ダイアログのレンダはコマンド確定から遅延することがあり (TUI の全画面再描画ラグ)、
- * 固定待ちの Enter がダイアログ描画前に着弾して取りこぼされ、ダイアログが残ったまま
- * 次の入力の Enter を吸収する事故になっていた。代わりにペインを短周期で読み、ダイアログ
- * が実際に描画されたのを確認してから Enter を送り、消えるまで数回リトライする。ダイアログ
- * が出ない場合 (prior output 無し = 即時反映) は余計な Enter を送らない (空 Enter が
- * 別の場面で誤確定するのを避ける)。ベストエフォート。
- *
- * タイミングは opts で上書きできる (テストで短縮するため):
- *   - confirmAppearMs: ダイアログ出現を待つ上限 (既定 2400ms)
- *   - confirmPollMs:   出現待ちのポーリング間隔 (既定 150ms)
- *   - confirmDismissMs: 承認後にダイアログ消失を待つ上限 (既定 2000ms)
- *
- * @param {string} bin tmux バイナリパス
- * @param {string} name tmux セッション名
- * @param {{logger?:object,tmuxBin?:string,confirmAppearMs?:number,confirmPollMs?:number,confirmDismissMs?:number}} opts
- * @returns {Promise<boolean>} 確認ダイアログを承認したら true
- */
-async function _confirmSwitchDialog(bin, name, opts) {
-  const appearMs = opts.confirmAppearMs ?? 2400
-  const pollMs = opts.confirmPollMs ?? 150
-  const dismissMs = opts.confirmDismissMs ?? 2000
-  // ダイアログ出現待ち: コマンド確定から最大 appearMs 観測する (レンダ遅延の上限想定)。
-  const APPEAR_DEADLINE = Date.now() + appearMs
-  let appeared = false
-  while (Date.now() < APPEAR_DEADLINE) {
-    const text = await capturePane(name, { ...opts, noCache: true })
-    if (SWITCH_CONFIRM_DIALOG_RE.test(text)) {
-      appeared = true
-      break
-    }
-    await _delay(pollMs)
-  }
-  if (!appeared) return false // prior output 無し = 即時反映済み。追い Enter 不要。
-
-  // ダイアログが消える (= 承認反映) まで Enter を送ってリトライする。
-  const DISMISS_DEADLINE = Date.now() + dismissMs
-  while (Date.now() < DISMISS_DEADLINE) {
-    await execFileP(bin, ["send-keys", "-t", name, "Enter"])
-    await _delay(180)
-    const after = await capturePane(name, { ...opts, noCache: true })
-    if (!SWITCH_CONFIRM_DIALOG_RE.test(after)) return true
-  }
-  opts.logger?.warn(
-    { session: name },
-    "switch confirm dialog still visible after retries",
-  )
-  return true
-}
-
 /**
  * 離席中キュー投入: 送信待ちメッセージ 1 件を、ブラウザがそのセッションのターミナルを
  * マウントしていなくても tmux send-keys で対話 claude TUI へ投入する (案A の agent 側実体)。