@cocorograph/hub-agent 0.7.104 → 0.7.105

package/package.json

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

package/src/codex-stream-bridge.mjs

@@ -334,6 +334,10 @@ export class CodexStreamBridge extends EventEmitter {
     /** @type {Map<string, object>} stream_id -> session record */
     this.sessions = new Map()
     this._accountRateLimitsByProfile = new Map()
+    // Phase 3 (Read-plane): listThreads の single-flight。key(`cwd::limit`) -> in-flight Promise。
+    // codex.sessions.request を直列キューから外したため、同一 cwd への同時リクエストが
+    // app-server の同時 spawn (storm) を起こさないよう 1 本に畳む。
+    this._listThreadsInFlight = new Map()
   }
 
   /**
@@ -762,6 +766,19 @@ export class CodexStreamBridge extends EventEmitter {
    * @returns {Promise<Array<{thread_id: string, cwd: string, preview: string, updated_at_ms: number}>>}
    */
   async listThreads({ cwd, limit } = {}) {
+    // Phase 3 (Read-plane): single-flight。同一 (cwd, limit) への同時リクエストを 1 本の
+    // in-flight Promise に畳み、app-server の同時 spawn (storm) を防ぐ。異なる cwd は個別に走る。
+    const key = `${cwd || ""}::${limit ?? ""}`
+    const existing = this._listThreadsInFlight.get(key)
+    if (existing) return existing
+    const p = this._doListThreads({ cwd, limit }).finally(() => {
+      this._listThreadsInFlight.delete(key)
+    })
+    this._listThreadsInFlight.set(key, p)
+    return p
+  }
+
+  async _doListThreads({ cwd, limit } = {}) {
     const client = this._clientFactory({ cwd, codexCmd: this.codexCmd, logger: this.logger })
     client.start()
     try {

package/src/main.mjs

@@ -342,6 +342,33 @@ export function isFastPathMessage(type) {
   )
 }
 
+/**
+ * Phase 3 (Read-plane 2026-07-24): タイムライン/セッション一覧の「読み取り専用 RPC」を判定する。
+ * これらは disk (jsonl) / codex rollout を読んで自前の response を送るだけで pane・入力・bridge の
+ * 状態を一切変更しない (ハンドラ本体で確認済み: claude.history.request=fetchSessionHistory /
+ * claude.sessions.request=listSessions / codex.sessions.request=listThreads、いずれも純粋読み取り)。
+ * メッセージ自身が cwd/session_id/stream_id を持ち、frontend も stream_id/cwd 照合で stale 応答を
+ * 捨てるため、WS 受信順に対する順序依存が無い。
+ *
+ * これを直列 dispatchChain から外す理由 (view/spawn 分離の核): 直列キューは「WS 受信順 = pane 反映順」
+ * を守るためのものだが、その代償として前方の重い処理 (codex.sessions.request/codex.attach の
+ * app-server spawn = 数秒、claude.sessions.request の全ファイル読み) が後続の claude.attach を
+ * head-of-line blocking し、attach→ready を数十秒遅延させていた (本番計測 cold p95 40〜54s)。
+ * claude/codex が同一 dispatchChain を共有するため codex の spawn が claude の切替まで巻き込む。
+ * 読み取り RPC を並行レーンへ逃がすことで、タイムライン閲覧が live 接続 (spawn) の完了を待たなくなる。
+ *
+ * ⚠️ codex.sessions.request は listThreads が app-server を spawn するため、off-queue 化の副作用で
+ * 同一 cwd への同時リクエストが spawn storm を起こしうる。CodexStreamBridge.listThreads 側に
+ * single-flight ガードを併設して同時 spawn を 1 本に畳む (B で rollout disk 走査へ置換すれば spawn 自体が消える)。
+ */
+export function isReadOnlyRpc(type) {
+  return (
+    type === "claude.history.request" ||
+    type === "claude.sessions.request" ||
+    type === "codex.sessions.request"
+  )
+}
+
 /**
  * active プロファイルの configDir を process.env.CLAUDE_CONFIG_DIR へ反映する。
  * 既定 (`~/.claude`) のときは変数を消して従来挙動 (Claude Code の既定パス) に戻す。
@@ -1122,6 +1149,20 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
       })
       return
     }
+    // Phase 3 (Read-plane): 読み取り専用 RPC (history/sessions) も直列キューから外して並行処理する。
+    // 直列キュー前方の重い処理 (codex app-server spawn 等) が claude.attach→ready を HoL blocking
+    // するのを防ぐ (view/spawn 分離)。純粋読み取りで順序依存が無いことは isReadOnlyRpc の説明参照。
+    if (isReadOnlyRpc(msg?.type)) {
+      Promise.resolve(
+        dispatch(msg, { ...ctx, client, ptyBridge, claudeBridge, codexBridge, codexMcpControl, uploadManager }),
+      ).catch((err) => {
+        logger.error(
+          { err: err.message, type: msg?.type },
+          "dispatch threw (bypassed read-only RPC lane)",
+        )
+      })
+      return
+    }
     dispatchChain = dispatchChain
       .then(() => dispatch(msg, { ...ctx, client, ptyBridge, claudeBridge, codexBridge, codexMcpControl, uploadManager }))
       .catch((err) => {