@cocorograph/hub-agent 0.8.23 → 0.8.25

package/src/plugin-loader.mjs

@@ -13,7 +13,45 @@ import { pathToFileURL } from "node:url"
 
 import { paths } from "./config.mjs"
 
-const PLUGIN_DIR_REGEX = /^(\d+)-(.+)$/
+/**
+ * プラグインディレクトリ名の形 (`<prio>-<name>`)。
+ *
+ * ⚠️ これは形の一部でしかない。受理判定は必ず `parsePluginDirName()` を使うこと
+ * (正規表現だけを共有しても「読める形」と「消せる形」は揃わない)。
+ */
+export const PLUGIN_DIR_REGEX = /^(\d+)-(.+)$/
+
+/**
+ * プラグインディレクトリ名を検証し、`{ prio, name }` に分解する。受理しない名前は null。
+ *
+ * ⚠️ **受理集合の定義はこの 1 関数だけに置く**。discovery (loadPlugins) と削除
+ * (plugin-uninstall.mjs) が同じ集合を使わないと「置けるのに消せない」「読めるのに
+ * 拒否される」ディレクトリが生まれる。以前は正規表現だけを共有し、uninstall 側に
+ * `name.includes("..")` を足していたため、`10-my..plugin` を loader は読み込むのに
+ * uninstall は拒否していた。
+ *
+ * 判定順:
+ *   1. 文字列でない / 空 → 拒否
+ *   2. `/` または `\` を含む → 拒否 (パス区切りの持ち込みを許さない。POSIX / WSL では
+ *      `\` は正当なファイル名文字なので、loader 側もここで揃えて読み飛ばす)
+ *   3. `.` / `..` そのもの → 拒否
+ *   4. `PLUGIN_DIR_REGEX` に一致しない → 拒否
+ *
+ * ⚠️ セグメント内部の連続ドット (`10-my..plugin`) は **許可** する。区切り文字を
+ * 既に拒否済みなので単一セグメントであり、上位への移動にはならない
+ * (`path.resolve` は `10-..` を親参照として扱わない)。
+ *
+ * @param {unknown} name
+ * @returns {{ prio: number, name: string } | null}
+ */
+export function parsePluginDirName(name) {
+  if (typeof name !== "string" || name === "") return null
+  if (name.includes("/") || name.includes("\\")) return null
+  if (name === "." || name === "..") return null
+  const m = PLUGIN_DIR_REGEX.exec(name)
+  if (!m) return null
+  return { prio: Number(m[1]), name: m[2] }
+}
 
 /**
  * Plugins discovery。読み込み順は dirname の数値昇順。
@@ -33,9 +71,16 @@ export async function loadPlugins(logger) {
   const candidates = entries
     .filter((d) => d.isDirectory())
     .map((d) => {
-      const m = PLUGIN_DIR_REGEX.exec(d.name)
-      if (!m) return null
-      return { dir: d.name, prio: Number(m[1]), name: m[2] }
+      const parsed = parsePluginDirName(d.name)
+      if (!parsed) {
+        // 読み飛ばしを黙って隠すと、利用者は「置いたのに動かない」理由に辿り着けない。
+        logger?.warn(
+          { dir: d.name },
+          "plugin dir name rejected (skipped): expected <prio>-<name> without path separators (e.g. 10-my-plugin)",
+        )
+        return null
+      }
+      return { dir: d.name, prio: parsed.prio, name: parsed.name }
     })
     .filter(Boolean)
     .sort((a, b) => a.prio - b.prio)

package/src/plugin-uninstall.mjs

@@ -0,0 +1,61 @@
+/**
+ * `hub-agent plugins uninstall <name>` の実装。
+ *
+ * プラグインは利用者が `~/.hub/plugins/<prio>-<name>/` へ自分で配置する運用なので、
+ * CLI が提供するのは「読み込み済み一覧 (list)」と「配置済みディレクトリの削除
+ * (uninstall)」だけ。リポ同梱 example を配る install 系は持たない。
+ *
+ * ⚠️ セキュリティ: 引数の `name` はそのままパス結合すると `../` で plugins ディレクトリの
+ * 外を再帰削除できてしまう (旧 plugin-install.mjs の uninstallPlugin はこれを塞いでいな
+ * かった)。次の 2 段で塞ぐ:
+ *   1. ディレクトリ名を `plugin-loader.mjs` の `parsePluginDirName()` で検証する
+ *      (discovery 側と **同じ 1 つの受理集合** を再利用する。判定を書き写すと
+ *      「読める形」と「消せる形」がずれ、置けるのに消せないディレクトリが生まれる)
+ *   2. 削除直前に `path.resolve` した結果が plugins ディレクトリ配下に収まっているか
+ *      を再確認する (形の検証をすり抜けた場合の最後の砦)
+ */
+import { promises as fs } from "node:fs"
+import path from "node:path"
+
+import { paths as configPaths } from "./config.mjs"
+import { parsePluginDirName } from "./plugin-loader.mjs"
+
+/**
+ * `~/.hub/plugins/<name>/` を再帰削除する。存在しない場合も成功扱い (冪等)。
+ *
+ * @param {string} name プラグインディレクトリ名 (`<prio>-<name>` 形式)
+ * @returns {Promise<{ dest: string, removed: boolean }>}
+ */
+export async function uninstallPlugin(name) {
+  if (!name || typeof name !== "string") {
+    throw new Error("plugin name is required")
+  }
+  // ⚠️ 受理集合は loader と共有する 1 つの validator が決める (ここで条件を足さない)。
+  if (!parsePluginDirName(name)) {
+    throw new Error(
+      `invalid plugin name "${name}": expected <prio>-<name> (e.g. 10-my-plugin)`,
+    )
+  }
+
+  const pluginsRoot = path.resolve(configPaths.pluginsDir)
+  const dest = path.resolve(path.join(pluginsRoot, name))
+  // 封じ込めの再確認。ここに来る時点で形は検証済みだが、削除は不可逆なので結合結果も見る。
+  // ⚠️ ここで見ているのは「**設定された論理 root** (configPaths.pluginsDir) 配下に収まって
+  // いるか」であり、実 (real) パスではない。`path.resolve` は字句正規化で symlink を
+  // 解決しないため、root 自体が別ボリュームへの symlink ならその先が正式な plugins root と
+  // して扱われる (loader も同じ論理 root を読むので一貫する)。`realpath` による拒否は
+  // 入れない — `.hub` や plugins を別ボリュームへ置く環境を壊し、かつ不存在時に ENOENT で
+  // 「存在しなくても成功 (冪等)」の契約も壊れるため。
+  if (!dest.startsWith(pluginsRoot + path.sep)) {
+    throw new Error(`refusing to remove "${dest}": outside of ${pluginsRoot}`)
+  }
+
+  try {
+    await fs.rm(dest, { recursive: true, force: true })
+  } catch (err) {
+    throw new Error(`failed to remove ${dest}: ${err.message}`)
+  }
+  return { dest, removed: true }
+}
+
+export default { uninstallPlugin }

package/src/positive-int-env.mjs

@@ -0,0 +1,81 @@
+/**
+ * 上限 / TTL 系の env を **1 箇所で検証**する共通ヘルパー（cross-review minor 6 対応）。
+ *
+ * 【なぜ必要か】旧実装は `Number(process.env.X) || <既定>` で読み、`Math.max(1, v)` しか
+ * 掛けていなかった。`Number("1.5")` は 1.5 を通し `Math.max(1, 1.5) === 1.5` なので、
+ * その値が **for ループの開始添字**（`for (let i = keepPerCwd; ...)`）に入ると実害が出る:
+ *   - codex: `list[1.5]` が undefined のまま `_isProtected(undefined, now)` へ渡り
+ *     `session.pumping` の参照で TypeError
+ *   - claude: undefined の reap が no-op なのに `reaped` だけ加算され、
+ *     **実際には畳んでいないのに畳んだと報告する**
+ * `Infinity` は `Number("Infinity")` が通るため実質「上限なし」になる。
+ *
+ * 【なぜ Math.floor で丸めないか】設定ミスを黙って別の値に読み替えると、運用者は
+ * 「設定したつもりの値」と「効いている値」がズレたまま気づけない。**既定値へ戻して warn**
+ * に統一し、必ず痕跡が残るようにする。
+ *
+ * 【なぜ「正の安全整数」か】上限（K / M / N）は「何本残すか」の個数、TTL は「何ミリ秒」で、
+ * いずれも 1 未満・非整数・非有限に意味が無い。`Number.isSafeInteger` は NaN / Infinity /
+ * 小数 をまとめて弾ける唯一の述語なので、これ 1 本で判定する。
+ */
+
+/** 正の安全整数（>= 1）か。NaN / Infinity / 小数 / 0 / 負数 はすべて false。 */
+export function isPositiveInt(value) {
+  return Number.isSafeInteger(value) && value >= 1
+}
+
+/** warn の出力先。logger が無い（モジュール読み込み時など）場合は stderr へ落とす。 */
+function warn(logger, fields, message) {
+  if (typeof logger?.warn === "function") {
+    logger.warn(fields, message)
+    return
+  }
+  // eslint-disable-next-line no-console
+  console.warn(`[hub-agent] ${message}`, fields)
+}
+
+/**
+ * env から正の整数を読む。**不正値は既定値へ戻して warn**（黙って丸めない）。
+ *
+ * @param {string|string[]} names 環境変数名。配列なら「先に設定されている方を優先」して順に評価し、
+ *   不正値だった名前は warn したうえで次の候補へ落ちる（後方互換 env を持つ設定のため）。
+ * @param {number} fallback 既定値（正の安全整数であること）
+ * @param {object} [logger]
+ * @returns {number}
+ */
+export function readPositiveIntEnv(names, fallback, logger) {
+  const list = Array.isArray(names) ? names : [names]
+  for (const name of list) {
+    const raw = process.env[name]
+    if (raw === undefined || raw === null || String(raw).trim() === "") continue
+    const parsed = Number(raw)
+    if (isPositiveInt(parsed)) return parsed
+    warn(
+      logger,
+      { env: name, value: String(raw), fallback },
+      "invalid env value (expected a positive integer); falling back to the default",
+    )
+  }
+  return fallback
+}
+
+/**
+ * 実行時に注入された上限値（テスト / 呼び出し側 opts）を検証する。env と同じ規則で、
+ * 不正なら **既定値へ戻して warn** する（env だけ守っても opts 経由で同じ穴が開くため）。
+ *
+ * @param {*} value
+ * @param {number} fallback
+ * @param {{ logger?: object, label?: string }} [opts]
+ * @returns {number}
+ */
+export function sanitizePositiveInt(value, fallback, { logger, label = "bound" } = {}) {
+  if (value === undefined || value === null) return fallback
+  const parsed = typeof value === "number" ? value : Number(value)
+  if (isPositiveInt(parsed)) return parsed
+  warn(
+    logger,
+    { setting: label, value: String(value), fallback },
+    "invalid bound value (expected a positive integer); falling back to the default",
+  )
+  return fallback
+}

package/src/session-seq.mjs

@@ -0,0 +1,22 @@
+/**
+ * **プロセス内で 1 本**のセッション生成順シーケンス（cross-review minor 4 対応）。
+ *
+ * 【なぜ CLI ごとに持ってはいけないか】warm 境界の recency が同値になったときのタイブレークに
+ * seq を使う。cwd 単位の境界（per-cwd=K / N-cwd LRU）は同一 CLI 内でしか比較しないので
+ * CLI ごとのカウンタでも成立したが、**総 live 上限 M（warm-bounds-total.mjs）は claude と codex を
+ * 合算して「古い方から刈る」**ため、別カウンタだと CLI をまたいだ比較が成立しない。
+ * 例: 古い codex が seq=100 / 新しい claude が seq=2 のとき、recency 同値だと **新しい claude が
+ * 先に刈られる**。プロセス内で単一の単調増加にすれば、この比較が常に生成順と一致する。
+ *
+ * cwd 内のタイブレークは「単調増加であること」しか使っていないため、共有化しても挙動は変わらない
+ * （値の絶対値には意味が無い。⚠️ そのためテストで絶対値を assert しないこと。module-global は
+ * テスト間で値が持ち越される）。
+ */
+
+let SESSION_SEQ = 0
+
+/** 次のセッション seq（1 始まりの単調増加）を返す。 */
+export function nextSessionSeq() {
+  SESSION_SEQ += 1
+  return SESSION_SEQ
+}

package/src/tmux.mjs

@@ -588,7 +588,7 @@ export async function setTmuxGlobalEnv(name, value, opts = {}) {
 }
 
 /**
- * 汎用 tmux 実行。`tmux.exec` メッセージから呼び出される。
+ * 汎用 tmux 実行 (raw)。任意の tmux サブコマンドを実行して結果を返す低レベル API。
  *
  * @returns {Promise<{stdout: string, stderr: string, exit_code: number}>}
  */
@@ -913,16 +913,9 @@ export async function createSession(name, cwd, opts = {}) {
     // has-session が非 0 = セッション無し
   }
   await execFileP(tmuxBin(opts), ["new-session", "-d", "-s", name, "-c", resolvedCwd])
-  // 既定では mouse mode を ON にする (スマホの touch swipe → SGR wheel escape を
+  // mouse mode を ON にする (スマホの touch swipe → SGR wheel escape を
   // tmux が copy-mode スクロールとして拾えるようにするため)。tmux 2.1+ で `mouse`
   // option は session-scoped、`-t <session>` 指定で他 session への副作用なし。
-  //
-  // ⚠️ ただしこれは「初期既定値」にすぎない。実際の値は接続してきた端末に応じて
-  // pty.attach 時に setSessionMouse() で上書きされる: デスクトップ attach は off
-  // (= マウスのドラッグを xterm のローカル選択にして、コピー奪取・button release
-  // 取りこぼしによる操作不能を防ぐ。0.5.11 で全 session を mouse on にしたことが
-  // デスクトップのドラッグ選択を壊した回帰の修正)。スマホ attach は on のまま。
-  // 旧フロント (mouse フィールド未送信) はこの既定値 on がそのまま使われる。
   try {
     await execFileP(tmuxBin(opts), ["set-option", "-t", name, "mouse", "on"])
   } catch (err) {
@@ -995,27 +988,6 @@ export async function createSession(name, cwd, opts = {}) {
   }
 }
 
-/**
- * 指定 session の tmux mouse mode を on/off する。pty.attach 時に接続端末の
- * 種別に応じて呼ぶ (デスクトップ=off / スマホ=on)。失敗してもセッションの
- * 基本機能は動くので warn のみで握り潰す。
- * @param {string} name session 名
- * @param {boolean} on true=mouse on / false=mouse off
- * @param {{logger?: any, tmuxBin?: string}} [opts]
- */
-export async function setSessionMouse(name, on, opts = {}) {
-  if (!name) return
-  const value = on ? "on" : "off"
-  try {
-    await execFileP(tmuxBin(opts), ["set-option", "-t", name, "mouse", value])
-  } catch (err) {
-    opts.logger?.warn?.(
-      { session: name, mouse: value, err: err?.message || String(err) },
-      "tmux set-option mouse failed",
-    )
-  }
-}
-
 /** session_id として安全な形だけ許可する (send-keys へ流すためコマンド注入を防ぐ)。
  *  Claude の session_id は UUID (ASCII 英数 + ハイフン) なので、それ以外は弾く。 */
 export function isSafeSessionId(sessionId) {
@@ -1172,8 +1144,8 @@ const _delay = (ms) => new Promise((r) => setTimeout(r, ms))
  * resumeWithMessage は改行を空白へ畳む単一行用 (誤確定防止) でチャット本文の複数行を壊すため、
  * こちらは **bracketed paste** (ESC[200~ … ESC[201~) で本文を「貼り付け」として送り、改行を
  * 保ったまま入力欄へ入れてから、確定の Enter (CR) を別途送る。これは frontend の通常送信
- * (`\x1b[200~${text}\x1b[201~` → CR) と同じ手順を agent 側で再現したもの。PTY 非依存
- * (tmux send-keys) なので、当該セッションのターミナルを誰も開いていなくても届く。
+ * (`\x1b[200~${text}\x1b[201~` → CR) と同じ手順を agent 側で再現したもの。tmux send-keys
+ * で直接ペインへ届くので、ブラウザ側が何も開いていなくても投入できる。
  *
  * @param {string} name tmux セッション名
  * @param {string} text 投入する本文 (複数行可)

package/src/warm-bounds-total.mjs

@@ -0,0 +1,150 @@
+/**
+ * 総 live 上限 M（既定 20）: claude / codex を **合算** した live セッション数の天井。
+ *
+ * 【なぜ必要か】per-cwd の上限を 1 → K（既定 3）へ緩めたので、cwd 単位の境界（per-cwd=K ×
+ * N-cwd LRU）だけではマシン全体の総量が K 倍まで膨らみうる。方針は「総量は増やさず配分の
+ * 自由度だけ上げる」なので、cwd をまたいだ総量の天井をここで別に持つ。
+ *
+ * 【なぜ 1 モジュールで合算できるか】claude / codex の両ブリッジは同一プロセスに同居している
+ * （src/main.mjs の agent 起動時に両方 new される）ため、プロセス内で足し合わせれば実際の
+ * リソース消費（1 セッション ≈ 120-140MB）と一致する。
+ *
+ * 【CLI 分岐を持たない設計】各ブリッジが `listWarmCandidates(now)` という **CLI 非依存の共通
+ * アダプタ** を提供し、本モジュールはその配列だけを見る。ここに `kind === 'claude'` のような
+ * 分岐を入れると、CLI が増えるたびに刈り取りロジックが分岐して腐るため入れない。
+ * テストはアダプタ形状の stub 2 本で書ける。
+ */
+import { readPositiveIntEnv, sanitizePositiveInt } from "./positive-int-env.mjs"
+
+/** 総 live 上限 M（claude + codex 合算）。env で調整可能。ブリッジ側に重複定義しないこと。
+ *
+ *  【なぜ 20 か】2026-08-03 の実測で、同時 live は **17 本**（claude 13 / codex 4）だった。
+ *  内訳は distinct な (cwd, 種別) 16 バケットに対しほぼ 1 本ずつで、2 本あるのは 1 バケットのみ。
+ *  つまり資源を消費しているのは「1 cwd に何本開くか（深さ）」ではなく「同時に開いている cwd の
+ *  数（広さ）」である。したがって:
+ *    - K=3（深さ）を許しても今日の使い方の総量は変わらない
+ *    - M=10 にすると **広さを切る**ことになり、導入した瞬間に 7 本が畳まれる（当初案を却下）
+ *  構造上の天井は、**旧 per-cwd=1 時代**が 40 本（cwd 数 N=20 × per-cwd 1 × ブリッジ 2）、
+ *  **本 PR の per-cwd=K=3 では M が無ければ 120 本**（N=20 × K=3 × ブリッジ 2）。M=20 はその
+ *  どちらより小さく、旧天井 40 の半分でありながら実測 17 本を切らない値になる（K を緩めても
+ *  総量は増やさない、という方針がこの M で担保される）。
+ *  ⚠️ 畳んでも会話は消えない（resume で復帰する）が、温まった状態を失って切替が遅くなる。
+ *  この待ち時間は 2026-07-31 に課題として挙がっている論点なので、安易に下げないこと。 */
+export const WARM_TOTAL_MAX = readPositiveIntEnv("HUB_AGENT_WARM_TOTAL_MAX", 20)
+
+/** 「超過しているが保護されていて刈れない」状態を info で出す最小間隔（既定 5 分）。
+ *  state loop は 5 秒周期なので、抑制しないと同じ 1 行が 1 時間に約 720 件出る。 */
+export const UNRESOLVED_LOG_INTERVAL_MS = 5 * 60 * 1000
+
+/**
+ * @typedef {object} WarmCandidate
+ * @property {string} kind    'claude' | 'codex' 等（ログ用。刈り取り判断には使わない）
+ * @property {string} cwd     作業ディレクトリ（ログ用）
+ * @property {number} recency 最終活性（epoch ms。小さいほど古い = 先に刈る）
+ * @property {number} seq     生成順の単調増加値（recency 同値のタイブレーク。小さいほど古い）
+ * @property {boolean} protected 走行中ターン / pending / タスク実行中 等で不可侵か
+ * @property {(reason: string) => void} reap 実際に撤去する
+ */
+
+/**
+ * 総 live 上限を適用する。**保護されているものは刈らない**（全部保護なら上限超過を許容する
+ * = 走行中を殺さない方を優先する。上限は次の tick で改めて試みられる）。
+ *
+ * ⚠️ **M はハードな資源上限ではない**（cross-review round2 minor）。保護（走行中ターン / 実行中の
+ * 背景タスク・コマンド）は必ず skip するので、保護が同時に多数付けば live 総数は M を超えたまま
+ * 増え続けうる。有界なのは **1 エントリあたりの保護時間**（各ブリッジの TTL）だけで、**同時に
+ * 保護されうる件数には K も N も M も効かない**。ここは「一時的な超過を明示的に受容する」設計で、
+ * 実際の超過量は戻り値の `liveTotal` / `unresolved`（と対応する info ログ）で観測する。
+ * 上限を守れているかの運用判断は、M という設定値ではなくその実測値を見ること。
+ *
+ * ⚠️ 将来「保護中でも刈る」ハード上限を足すなら、**実行中の作業を中断するという新しい失敗モード**
+ * が生まれる。ここに暗黙に足さないこと。足すなら victim の選び方・利用者への通知・再開可否を
+ * 別仕様として設計する。
+ *
+ * 【ログの抑制】state loop は 5 秒周期で回るため、「超過しているが全部保護されていて刈れない」
+ * 状態が続くと同じ info が 1 時間に約 720 件出る（旧実装は `reaped > 0 || overflow > 0` という
+ * **常に真の死んだ条件**だったので毎 tick 出ていた。overflow <= 0 は手前で return 済み）。
+ * そこで **実際に reap した tick は従来どおり info**、**unresolved だけの tick は「状態が変わった
+ * とき」か「一定間隔（既定 5 分）」でのみ**出す。
+ *
+ * ⚠️ 抑制の状態は **module-global に置かない**（複数インスタンス・並行テストが互いのログを
+ * 抑制してしまう）。呼び出し側が持つ `logState` オブジェクトに書く。渡さなければ抑制しない。
+ *
+ * @param {{ bridges?: Array<{ listWarmCandidates?: (now: number) => WarmCandidate[] }>,
+ *           max?: number, now?: number, logger?: object,
+ *           logState?: {lastUnresolved?: number, lastLoggedAtMs?: number} }} [opts]
+ * @returns {{reaped: number, unresolved: number, liveTotal: number}}
+ *   reaped=実際に刈った数 / unresolved=保護されて刈れなかった超過ぶん / liveTotal=候補総数
+ */
+export function enforceTotalLiveBound({
+  bridges = [],
+  max,
+  now = Date.now(),
+  logger,
+  logState,
+} = {}) {
+  // ⚠️ 上限は正の整数（1.5 / Infinity / NaN は既定へ戻して warn）。positive-int-env.mjs 参照。
+  const limit = sanitizePositiveInt(max, WARM_TOTAL_MAX, {
+    logger,
+    label: "warmBounds.totalMax",
+  })
+  /** @type {WarmCandidate[]} */
+  const candidates = []
+  for (const bridge of bridges) {
+    if (!bridge || typeof bridge.listWarmCandidates !== "function") continue
+    try {
+      const list = bridge.listWarmCandidates(now)
+      if (Array.isArray(list)) {
+        for (const c of list) if (c) candidates.push(c)
+      }
+    } catch (err) {
+      logger?.warn?.({ err: err?.message }, "listWarmCandidates threw")
+    }
+  }
+  const overflow = candidates.length - limit
+  if (overflow <= 0) {
+    // 超過が解消した = 抑制状態をリセットする。次に新しく超過したときは即座に記録される。
+    if (logState) {
+      logState.lastUnresolved = 0
+      logState.lastLoggedAtMs = 0
+    }
+    return { reaped: 0, unresolved: 0, liveTotal: candidates.length }
+  }
+  // 古い順（recency 昇順、同値は seq 昇順）に、非保護だけを刈る。
+  const evictable = candidates
+    .filter((c) => !c.protected)
+    .sort((a, b) => (a.recency ?? 0) - (b.recency ?? 0) || (a.seq ?? 0) - (b.seq ?? 0))
+  let reaped = 0
+  for (const c of evictable) {
+    if (reaped >= overflow) break
+    try {
+      c.reap("warm-bounds:total")
+      reaped += 1
+    } catch (err) {
+      logger?.warn?.({ err: err?.message, kind: c.kind, cwd: c.cwd }, "total warm bound reap threw")
+    }
+  }
+  // 保護されて刈れなかったぶん（>0 なら一時的に上限超過を許容している）。
+  const unresolved = overflow - reaped
+  if (_shouldLog({ reaped, unresolved, now, logState })) {
+    logger?.info?.(
+      { live_total: candidates.length, max: limit, overflow, reaped, unresolved },
+      "warm bounds: total live limit applied",
+    )
+    if (logState) {
+      logState.lastUnresolved = unresolved
+      logState.lastLoggedAtMs = now
+    }
+  }
+  return { reaped, unresolved, liveTotal: candidates.length }
+}
+
+/** ログを出すか。reap した tick は常に出す。unresolved だけの tick は、状態が変わったときか
+ *  一定間隔が空いたときだけ出す（logState 未指定なら抑制しない）。 */
+function _shouldLog({ reaped, unresolved, now, logState }) {
+  if (reaped > 0) return true
+  if (unresolved <= 0) return false
+  if (!logState) return true
+  if (logState.lastUnresolved !== unresolved) return true
+  return now - (logState.lastLoggedAtMs ?? 0) >= UNRESOLVED_LOG_INTERVAL_MS
+}

package/src/ws-client.mjs

@@ -49,36 +49,17 @@ const STABLE_CONNECTION_MS = 10_000
 // 当たって連発失敗するため、最低 5s 待つ。30s リトライまで段階的に伸びる。
 const MIN_BACKOFF_AFTER_5XX_MS = 5_000
 
-// outbound pty.data buffer のサイズ上限。
-// WS not OPEN 中に pty.data を捨てると Cockpit 上のターミナル描画が欠落して
-// 「動かない / 一部だけ表示される」体感を生むため、reconnect 後に flush する。
-// pty.data は冪等で順序維持できれば xterm 側で正しく再現できる。
-// 1 frame は典型的に 数十〜数百 bytes (claude TUI の partial redraw)。
-// 500 frame ≒ 数十KB を 1 切断あたりの上限とする。
-const PTY_BUFFER_MAX_FRAMES = 500
-// outbound buffer に残った pty.data が古すぎると、ターミナルの履歴として
-// 再生する意味が薄れる (ユーザーが視覚的に「過去のもの」として無視する範囲)。
-// 30 秒以上経過した pty.data は flush 時に破棄する。
-const PTY_BUFFER_MAX_AGE_MS = 30_000
-
 // agent→browser の制御/イベント系メッセージのうち、WS not OPEN 中に捨てると
 // Cockpit の挙動が壊れるものを reconnect 越しに保持する (機構③: 本番 agent.log で
 // `claude.jsonl.event ... "ws send skipped (not open)"` のドロップを観測)。
-//   - pty.input.ack: 喪失すると browser の pendingAcks が永久滞留し「送信待ち(未配信)」が
-//     消えない (S1/S3)。再接続 flush で遅れて届けば即解消する。
-//   - claude/codex の event / ready / exit / error / permission / input ack,nack:
+//   - claude/codex の input ack,nack: 喪失すると browser の pendingAcks が永久滞留し
+//     「送信待ち(未配信)」が消えない (S1/S3)。再接続 flush で遅れて届けば即解消する。
+//   - claude/codex の event / ready / exit / error / permission:
 //     喪失すると新規セッションの楽観バブルが本物へ昇格できず、再 attach も固着する。
 //   - claude.jsonl.event / session.event: TUI と一覧の確定状態を復元する。
-//   - pty.exit / pty.ready / pty.error: stream ライフサイクルの節目。落とすと browser の
-//     再 attach/復帰判断が狂う。
-// pty.data とは別バッファにするのは、高頻度な pty.data の洪水で低頻度な ack/event が
-// リング evict されるのを防ぐため (pty.data は冪等で最新フレームだけ届けば良いが、
-// ack/event は 1 件ずつ意味を持つ)。
+// 高頻度の delta 系はここに積まず (下の isEphemeral*DeltaEvent で除外)、低頻度な
+// ack/event がリング evict されるのを防ぐ (ack/event は 1 件ずつ意味を持つ)。
 const BUFFERED_CTRL_TYPES = new Set([
-  "pty.input.ack",
-  "pty.exit",
-  "pty.ready",
-  "pty.error",
   "claude.jsonl.event",
   "claude.event",
   "claude.ready",
@@ -97,7 +78,7 @@ const BUFFERED_CTRL_TYPES = new Set([
   "codex.input.nack",
   "session.event",
 ])
-// 制御/イベントバッファの上限。低頻度なので frames は小さめ、age は pty と揃える。
+// 制御/イベントバッファの上限。低頻度なので frames は小さめ。
 const CTRL_BUFFER_MAX_FRAMES = 200
 const CTRL_BUFFER_MAX_AGE_MS = 30_000
 // reconnect 後の buffered event は backend が browser の active stream 集合を
@@ -168,11 +149,8 @@ export class WsClient extends EventEmitter {
     this.backoff = MIN_BACKOFF_MS
     this.stopped = false
     this.startedAt = Date.now()
-    // pty.data の outbound buffer (WS not OPEN 中に積み、open 後 flush)。
-    // entry: { obj, ts }
-    this.ptyOutboundBuffer = []
-    // 制御/イベント系 (ack / jsonl.event 等) の outbound buffer。pty.data とは別管理で、
-    // 高頻度 pty.data の洪水に evict されないようにする (機構③)。entry: { obj, ts }
+    // 制御/イベント系 (ack / jsonl.event 等) の outbound buffer
+    // (WS not OPEN 中に積み、open 後 flush)。entry: { obj, ts }
     this.ctrlOutboundBuffer = []
     // reconnect 世代ごとの stream sync barrier。古い request_id の遅延応答で
     // 現接続の buffer を flush しないよう request_id を照合する。
@@ -282,11 +260,10 @@ export class WsClient extends EventEmitter {
       available_clis: this.availableClis,
       ...this.bundleStatus,
     })
-    // Plan ε: 毎回 reconnect 直後に backend ↔ agent の stream_id を能動同期する
-    // (orphan stream の即時 kill 用)。response は main.mjs の dispatch が拾い、
-    // ptyBridge.list() と差分を取って kill する。request_id は uuid で
-    // 重複排除。backend が古い (= Plan ε 未対応) なら "unknown_type" error が
-    // 返るだけで動作には影響しない。
+    // Plan ε: 毎回 reconnect 直後に backend ↔ agent の stream_id を能動同期する。
+    // response は main.mjs の dispatch が拾い、mode 別 snapshot から外れた
+    // chat subscriber を掃除する。request_id は uuid で重複排除。backend が
+    // 古い (= Plan ε 未対応) なら "unknown_type" error が返るだけで動作には影響しない。
     const streamsSyncRequestId = randomUUID()
     this._sendJson({
       type: "agent.streams.sync.request",
@@ -298,11 +275,11 @@ export class WsClient extends EventEmitter {
     this.emit("open")
   }
 
-  /** メッセージを送る。未接続時は pty.data と制御/イベント系 (ack / jsonl.event 等) を
+  /** メッセージを送る。未接続時は制御/イベント系 (ack / jsonl.event 等) を
    * buffer に積み、reconnect 後に flush する。
    *
    * heartbeat / hello / agent.streams.sync.* など「古くなると意味が無い」制御系は
-   * buffer しない (warn のみ)。ack / event 系は 1 件ずつ意味を持つので別バッファに保持する。
+   * buffer しない (warn のみ)。ack / event 系は 1 件ずつ意味を持つのでバッファに保持する。
    */
   send(obj) {
     const type = obj?.type
@@ -310,12 +287,8 @@ export class WsClient extends EventEmitter {
     // sync barrier 中に新しく発生した frame も同じ buffer へ積み、切断中 frame より
     // 先に送って時系列を逆転させない。heartbeat 等の非 buffer 制御系は通常どおり送る。
     const holdForStreamsSync = Boolean(this.pendingStreamsSyncRequestId)
-      && (type === "pty.data" || BUFFERED_CTRL_TYPES.has(type))
+      && BUFFERED_CTRL_TYPES.has(type)
     if (!wsOpen || holdForStreamsSync) {
-      if (type === "pty.data") {
-        this._bufferPtyData(obj)
-        return false
-      }
       if (BUFFERED_CTRL_TYPES.has(type)) {
         // SDK-P2 (2026-07-08): ephemeral な stream_event の content_block_delta (text/thinking
         // delta) は ctrl バッファに積まない。切断中は生成が続くと大量の delta が流れ、200 frame
@@ -340,50 +313,6 @@ export class WsClient extends EventEmitter {
     return this._sendJson(obj)
   }
 
-  /** pty.data を outbound buffer に積む。リング (drop oldest on overflow)。 */
-  _bufferPtyData(obj) {
-    this.ptyOutboundBuffer.push({ obj, ts: Date.now() })
-    if (this.ptyOutboundBuffer.length > PTY_BUFFER_MAX_FRAMES) {
-      const dropped = this.ptyOutboundBuffer.length - PTY_BUFFER_MAX_FRAMES
-      this.ptyOutboundBuffer.splice(0, dropped)
-      this.logger?.warn(
-        { dropped, kept: this.ptyOutboundBuffer.length },
-        "pty outbound buffer overflow (oldest dropped)"
-      )
-    }
-  }
-
-  /** open 直後に buffer を flush。古すぎる entry は破棄する。 */
-  _flushPtyBuffer() {
-    if (this.ptyOutboundBuffer.length === 0) return
-    const now = Date.now()
-    // B10: バッファを退避してから処理する。送信失敗 (_sendJson が false) で break
-    // した場合、未送信の残りフレームを ptyOutboundBuffer の先頭へ戻して順序を保持する
-    // (空配列で上書きしたまま break すると残りが欠落する)。
-    const buf = this.ptyOutboundBuffer
-    this.ptyOutboundBuffer = []
-    let sent = 0
-    let expired = 0
-    for (let i = 0; i < buf.length; i++) {
-      const entry = buf[i]
-      if (now - entry.ts > PTY_BUFFER_MAX_AGE_MS) {
-        expired += 1
-        continue
-      }
-      const ok = this._sendJson(entry.obj)
-      if (!ok) {
-        // 未送信分 (現エントリ含む i 以降) を先頭へ戻す。次回 open / flush で再送する。
-        this.ptyOutboundBuffer = buf.slice(i).concat(this.ptyOutboundBuffer)
-        break
-      }
-      sent += 1
-    }
-    this.logger?.info(
-      { sent, expired, total: buf.length, requeued: this.ptyOutboundBuffer.length },
-      "pty outbound buffer flushed"
-    )
-  }
-
   /** 制御/イベント系 (ack / jsonl.event 等) を outbound buffer に積む。リング (drop oldest)。 */
   _bufferCtrl(obj) {
     this.ctrlOutboundBuffer.push({ obj, ts: Date.now() })
@@ -398,7 +327,10 @@ export class WsClient extends EventEmitter {
   }
 
   /** open 直後に制御/イベントバッファを flush。古すぎる entry は破棄する。
-   * 構造は _flushPtyBuffer と同じ (退避 → 期限切れ skip → 送信失敗で残りを順序保持で戻す)。
+   *
+   * B10: バッファを退避してから処理する。送信失敗 (_sendJson が false) で break した
+   * 場合、未送信の残りフレームを ctrlOutboundBuffer の先頭へ戻して順序を保持する
+   * (空配列で上書きしたまま break すると残りが欠落する)。
    */
   _flushCtrlBuffer() {
     if (this.ctrlOutboundBuffer.length === 0) return
@@ -443,7 +375,6 @@ export class WsClient extends EventEmitter {
     // frame を失うため、2s compatibility timeout まで barrier を維持する。
     if (activeIds.size === 0) return false
     this._clearStreamsSyncBarrier()
-    this._flushPtyBuffer()
     this._flushCtrlBuffer()
     return true
   }
@@ -459,7 +390,6 @@ export class WsClient extends EventEmitter {
         { request_id: requestId },
         "streams sync barrier timed out; flushing buffered frames for legacy backend",
       )
-      this._flushPtyBuffer()
       this._flushCtrlBuffer()
     }, STREAMS_SYNC_BARRIER_TIMEOUT_MS)
     this.streamsSyncBarrierTimer.unref?.()