@cocorograph/hub-agent 0.8.93 → 0.8.94

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.8.93",
+  "version": "0.8.94",
   "description": "Hub Hosted Cockpit のローカル常駐 agent。Hub と outbound WSS で接続し、ローカルの Claude / Codex チャットと tmux セッションを中継する。",
   "type": "module",
   "license": "UNLICENSED",

package/src/claude-stream-bridge.mjs

@@ -2736,6 +2736,8 @@ export class ClaudeStreamBridge extends EventEmitter {
     this._mcpControlStarting = null
     /** listSupportedCommands() の短命キャッシュ {at:ms, commands:SlashCommand[]} (T06117)。 */
     this._supportedCommandsCache = null
+    /** listSupportedCommands() の取得中 Promise (同時呼び出しの相乗り用)。 */
+    this._supportedCommandsInflight = null
   }
 
   /** _liveBySession の複合キー (T06271): cwd を含めることで、別 workspace の同一 session_id を
@@ -3274,6 +3276,8 @@ export class ClaudeStreamBridge extends EventEmitter {
             // control query が死んだらキャッシュも破棄する。次回アクセスで新しい
             // query から取り直す (別 query の init スナップショットに更新される)。
             this._supportedCommandsCache = null
+            // 死んだ query への取得に後続の要求を相乗りさせない。
+            this._supportedCommandsInflight = null
           }
         }
       })()
@@ -3317,6 +3321,9 @@ export class ClaudeStreamBridge extends EventEmitter {
     // reload / shutdown 後も古いキャッシュが残り続け、listSupportedCommands が新しい
     // query を起動せず stale を返し続ける (ライフタイムキャッシュの永続化バグ)。
     this._supportedCommandsCache = null
+    // 畳んだ query への取得中 Promise も捨てる。残すと reload 後の要求が旧 query の取得へ
+    // 相乗りし、新しい query を起動しない。旧取得の結果は _loadSupportedCommands 側で捨てる。
+    this._supportedCommandsInflight = null
     try {
       input?.close()
     } catch {
@@ -3404,6 +3411,21 @@ export class ClaudeStreamBridge extends EventEmitter {
     // ライフタイムキャッシュ: 存在すれば (init スナップショット + commands_changed で
     // 反映済みの最新) をそのまま返す。再取得しない (上記 ⚠️)。
     if (this._supportedCommandsCache) return this._supportedCommandsCache.commands
+    // 取得中の呼び出しがあれば相乗りする。再接続直後は開いているタブの数だけ skills.request が
+    // 同時に届き、キャッシュが空のうちは全員が supportedCommands() を別々に叩いていた
+    // (2026-09-23 実測: 再接続直後に skills 15 件・agents 14 件が同時着)。
+    if (this._supportedCommandsInflight) return this._supportedCommandsInflight
+    const inflight = this._loadSupportedCommands()
+    this._supportedCommandsInflight = inflight
+    try {
+      return await inflight
+    } finally {
+      if (this._supportedCommandsInflight === inflight) this._supportedCommandsInflight = null
+    }
+  }
+
+  /** listSupportedCommands の取得本体 (相乗りの外側から 1 回だけ呼ばれる)。 */
+  async _loadSupportedCommands() {
     try {
       const q = await this._ensureMcpControlQuery()
       this._armMcpControlIdle()
@@ -3412,6 +3434,9 @@ export class ClaudeStreamBridge extends EventEmitter {
         return null
       }
       const commands = await q.supportedCommands()
+      // 取得中に query が畳まれた (idle / reload / shutdown) なら、旧 query の一覧を
+      // キャッシュへ書き戻さない。呼び出し側は null で FS 走査へフォールバックする。
+      if (this._mcpControlQuery !== q) return null
       const arr = Array.isArray(commands) ? commands : []
       // commands_changed が先に届いてキャッシュを埋めていたら、その最新を尊重する
       // (init スナップショットで上書きしない)。

package/src/dispatch-scheduler.mjs

@@ -252,12 +252,26 @@ export class DispatchScheduler {
   }
 
   _runNow(msg, lane = "bypass") {
-    return Promise.resolve().then(() => this.run(msg)).catch((err) => {
-      this.logger?.error?.(
-        { err: err?.message, type: msg?.type, lane },
-        "dispatch threw",
-      )
-    })
+    // 順番待ちが無い経路なので wait_ms は常に 0。本体が遅いときだけ記録する
+    // (skills/agents 等の読み取りをここへ移したとき、効果を測れなくならないように)。
+    const startedAt = Date.now()
+    return Promise.resolve()
+      .then(() => this.run(msg))
+      .finally(() => {
+        const runMs = Date.now() - startedAt
+        if (shouldLogDispatchTiming(msg?.type, 0, runMs)) {
+          this.logger?.info?.(
+            { type: msg?.type, lane, wait_ms: 0, run_ms: runMs },
+            "dispatch timing",
+          )
+        }
+      })
+      .catch((err) => {
+        this.logger?.error?.(
+          { err: err?.message, type: msg?.type, lane },
+          "dispatch threw",
+        )
+      })
   }
 
   /**

package/src/main.mjs

@@ -323,10 +323,60 @@ export function isReadOnlyRpc(type) {
     // T06603 (2026-07-25): キュー snapshot の pull。in-memory 配列を読むだけで副作用が無く、
     // かつ「直列キュー前方の重い処理で再同期が遅延する」と送信待ちチップの固着が長引く
     // (再同期は固着を直すための経路なので、詰まりやすい経路に置いてはならない)。
-    type === "claude.queue.request"
+    type === "claude.queue.request" ||
+    // 2026-09-23: 一覧の読み取り (FS 走査と SDK のコマンド一覧) だけで状態を変えず、request_id で
+    // 相関する。再接続直後はタブの数だけ同時に届き (実測 29 件)、control レーンで 1 件ずつ
+    // 並べていたため最後の 1 件が約 3 秒待っていた。同一要求は coalesceInflight で相乗りする。
+    type === "skills.request" ||
+    type === "agents.request"
   )
 }
 
+/**
+ * 同じ key の処理が実行中なら、その Promise を返して相乗りさせる (完了後は保持しない)。
+ * 結果をキャッシュするのではなく「同時に来た同一要求を 1 回の処理にまとめる」だけなので、
+ * 完了後に来た要求は必ず取り直す (一覧が古くならない)。
+ *
+ * @template T
+ * @param {Map<string, Promise<T>>} inflight
+ * @param {string} key
+ * @param {() => Promise<T>} fn
+ * @returns {Promise<T>}
+ */
+export function coalesceInflight(inflight, key, fn) {
+  const running = inflight.get(key)
+  if (running) return running
+  const promise = Promise.resolve().then(fn)
+  inflight.set(key, promise)
+  const clear = () => {
+    if (inflight.get(key) === promise) inflight.delete(key)
+  }
+  promise.then(clear, clear)
+  return promise
+}
+
+/**
+ * 起動時に SDK のコマンド一覧を先に取りに行く (startDaemon から呼ぶ)。完了を待たない。
+ * claude CLI が無い端末・SDK が無い端末では query を起動しない。
+ *
+ * @param {{ listSupportedCommands?: () => Promise<unknown> } | null} claudeBridge
+ * @param {string[]} availableClis
+ * @returns {boolean} 取得を開始したら true
+ */
+export function prewarmSupportedCommands(claudeBridge, availableClis) {
+  if (!claudeBridge?.listSupportedCommands || !availableClis.includes("claude")) return false
+  void claudeBridge.listSupportedCommands()
+  return true
+}
+
+/** skills.request / agents.request の実行中の処理 (coalesceInflight 用)。 */
+const listRequestsInflight = new Map()
+
+/** skills.request の相乗りの単位。結果は CLI 種別と session_name で変わる。 */
+export function skillsRequestKey(cliKind, sessionName) {
+  return `skills\u0000${cliKind}\u0000${sessionName || ""}`
+}
+
 /**
  * active プロファイルの configDir を process.env.CLAUDE_CONFIG_DIR へ反映する。
  * 既定 (`~/.claude`) のときは変数を消して従来挙動 (Claude Code の既定パス) に戻す。
@@ -726,6 +776,113 @@ export class DeferredRestartCoordinator {
   }
 }
 
+/**
+ * bridge のイベントを Hub へ転送する listener を登録する。
+ * ⚠️ 送信は必ず sendBridgeMessage を通す。会話の再点灯 (agent.sessions.restore) で使う
+ * `restore:` 購読は browser に存在しないため、ここで捨てないと Hub へ幽霊 stream が流れる。
+ * 起動処理から切り出してあるのは、この配線そのものをテストから通すため。
+ */
+export function wireBridgeEvents(client, { claudeBridge, codexBridge }) {
+  // stream-json モード (Sprint G): Claude Code を SDK 経由で起動し、stream イベントを
+  // そのまま browser に転送する。SDK 未インストール時は claudeBridge=null で全 attach が
+  // claude.error を返す経路に分岐 (dispatch 側で判定)。
+  if (claudeBridge) {
+    claudeBridge.on("event", ({ stream_id, session_id, cwd, session, event }) => {
+      // SDK の rate_limit_event を捕捉して usage の 5h/7d 実値に反映する
+      // (チャットモードでは statusLine が更新されないため、これが live 値の唯一の源)。
+      if (event?.type === "rate_limit_event" && event.rate_limit_info) {
+        try {
+          recordChatRateLimit(event.rate_limit_info)
+        } catch {
+          /* ignore */
+        }
+      }
+      resolveChatContextPct(session, event)
+      sendBridgeMessage(client, { type: "claude.event", stream_id, session_id, event })
+    })
+    claudeBridge.on("permission", ({ stream_id, session_id, cwd, request_id, tool_name, input }) => {
+      // 多端末共有: session_id を載せて backend が session group へ broadcast できるように
+      // する (全端末に許可ダイアログ → 先着 1 件採用)。
+      sendBridgeMessage(client, {
+        type: "claude.permission.request",
+        stream_id,
+        session_id,
+        request_id,
+        tool_name,
+        input,
+      })
+    })
+    claudeBridge.on(
+      "permissioncancel",
+      ({ stream_id, session_id, cwd, request_ids, reason }) => {
+        // SDK チャットの未応答 permission をターン終了/中断/close で内部 deny したとき、
+        // browser に残っている承認カードを撤去する (孤児カード防止, 2026-07-10 監査)。
+        // TUI 由来カードの撤去と同じ `claude.permission.cancel` 型を使う。
+        for (const request_id of request_ids || []) {
+          try {
+            sendBridgeMessage(client, {
+              type: "claude.permission.cancel",
+              stream_id,
+              session_id,
+              cwd,
+              request_id,
+              reason,
+            })
+          } catch {
+            /* ignore */
+          }
+        }
+      },
+    )
+    claudeBridge.on("exit", ({ stream_id, code, reason, session_id, cwd }) => {
+      sendBridgeMessage(client, { type: "claude.exit", stream_id, code, reason, session_id })
+    })
+    claudeBridge.on("sessionoptions", (message) => {
+      sendBridgeMessage(client, buildClaudeSessionOptionsPayload(message))
+    })
+    claudeBridge.on("error", ({ stream_id, session_id, error }) => {
+      sendBridgeMessage(client, { type: "claude.error", stream_id, session_id, error })
+    })
+  }
+
+  // codex 版 (Slice 3): codex-stream-bridge.mjs の生 notification/permission/exit/error を
+  // browser にそのまま転送する。claudeBridge と同じ「素通し」方針 (codex-stream-bridge.mjs
+  // 冒頭コメント参照)。claude 版にある rate_limit/MCP control query 相当の処理は無い
+  // (codex 側にまだ存在しない/後続スライス)。live usage だけを producer 単位で記録する。
+  if (codexBridge) {
+    codexBridge.on("event", ({ stream_id, producer_seq, thread_id, cwd, notification }) => {
+      if (notification.method === "thread/tokenUsage/updated") {
+        recordCodexLiveTokenUsage(producer_seq, notification.params?.tokenUsage)
+      }
+      sendBridgeMessage(client, { type: "codex.event", stream_id, thread_id, event: notification })
+    })
+    codexBridge.on("permission", ({ stream_id, thread_id, cwd, request_id, method, params }) => {
+      sendBridgeMessage(client, {
+        type: "codex.permission.request",
+        stream_id,
+        thread_id,
+        request_id,
+        method,
+        params,
+      })
+    })
+    codexBridge.on("exit", ({ stream_id, producer_seq, thread_id, code, reason, cwd }) => {
+      // T06407 増分2: この producer(session.seq) の live tokenUsage スロットを破棄する。
+      // seq は immutable なので、reattach で stream_id が付け替わっても alias slot / Map リークは
+      // 生じない (stream_id キー時代の major#1 を解消)。
+      try {
+        forgetCodexLiveTokenUsage(producer_seq)
+      } catch {
+        /* ignore */
+      }
+      sendBridgeMessage(client, { type: "codex.exit", stream_id, thread_id, code, reason })
+    })
+    codexBridge.on("error", ({ stream_id, thread_id, error }) => {
+      sendBridgeMessage(client, { type: "codex.error", stream_id, thread_id, error })
+    })
+  }
+}
+
 export async function startDaemon({ version, claudeSdk } = {}) {
   const config = await readConfig()
   if (!config) {
@@ -804,6 +961,13 @@ export async function startDaemon({ version, claudeSdk } = {}) {
     ? new ClaudeStreamBridge({ sdk: resolvedSdk, logger })
     : null
 
+  // 起動直後の skills 一覧を待たせないよう、SDK のコマンド一覧を先に取りに行く。取得には
+  // control query の起動が要り、初回は 1.5〜3 秒かかる (2026-09-23 実測: 本日の再起動 7 回とも
+  // 再接続直後の最初の skills.request が 1.5〜3.1 秒)。再接続で一斉に届く要求はこの取得に
+  // 相乗りする (listSupportedCommands の in-flight)。失敗しても null を返すだけで、要求時に
+  // FS 走査へフォールバックする。claude CLI が無い端末では query を起動しない。
+  prewarmSupportedCommands(claudeBridge, availableClis)
+
   // codex app-server ベースの chat ブリッジ。resolver は各 spawn 時に再評価するため、
   // daemon 起動時に一時的に unavailable でも bridge 自体は保持する。
   const codexBridge = new CodexStreamBridge({ logger })
@@ -879,104 +1043,7 @@ export async function startDaemon({ version, claudeSdk } = {}) {
     logger.warn({ err: err.message }, "ws client error (will retry via close handler)")
   })
 
-  // stream-json モード (Sprint G): Claude Code を SDK 経由で起動し、stream イベントを
-  // そのまま browser に転送する。SDK 未インストール時は claudeBridge=null で全 attach が
-  // claude.error を返す経路に分岐 (dispatch 側で判定)。
-  if (claudeBridge) {
-    claudeBridge.on("event", ({ stream_id, session_id, cwd, session, event }) => {
-      // SDK の rate_limit_event を捕捉して usage の 5h/7d 実値に反映する
-      // (チャットモードでは statusLine が更新されないため、これが live 値の唯一の源)。
-      if (event?.type === "rate_limit_event" && event.rate_limit_info) {
-        try {
-          recordChatRateLimit(event.rate_limit_info)
-        } catch {
-          /* ignore */
-        }
-      }
-      resolveChatContextPct(session, event)
-      sendBridgeMessage(client, { type: "claude.event", stream_id, session_id, event })
-    })
-    claudeBridge.on("permission", ({ stream_id, session_id, cwd, request_id, tool_name, input }) => {
-      // 多端末共有: session_id を載せて backend が session group へ broadcast できるように
-      // する (全端末に許可ダイアログ → 先着 1 件採用)。
-      sendBridgeMessage(client, {
-        type: "claude.permission.request",
-        stream_id,
-        session_id,
-        request_id,
-        tool_name,
-        input,
-      })
-    })
-    claudeBridge.on(
-      "permissioncancel",
-      ({ stream_id, session_id, cwd, request_ids, reason }) => {
-        // SDK チャットの未応答 permission をターン終了/中断/close で内部 deny したとき、
-        // browser に残っている承認カードを撤去する (孤児カード防止, 2026-07-10 監査)。
-        // TUI 由来カードの撤去と同じ `claude.permission.cancel` 型を使う。
-        for (const request_id of request_ids || []) {
-          try {
-            sendBridgeMessage(client, {
-              type: "claude.permission.cancel",
-              stream_id,
-              session_id,
-              cwd,
-              request_id,
-              reason,
-            })
-          } catch {
-            /* ignore */
-          }
-        }
-      },
-    )
-    claudeBridge.on("exit", ({ stream_id, code, reason, session_id, cwd }) => {
-      sendBridgeMessage(client, { type: "claude.exit", stream_id, code, reason, session_id })
-    })
-    claudeBridge.on("sessionoptions", (message) => {
-      sendBridgeMessage(client, buildClaudeSessionOptionsPayload(message))
-    })
-    claudeBridge.on("error", ({ stream_id, session_id, error }) => {
-      sendBridgeMessage(client, { type: "claude.error", stream_id, session_id, error })
-    })
-  }
-
-  // codex 版 (Slice 3): codex-stream-bridge.mjs の生 notification/permission/exit/error を
-  // browser にそのまま転送する。claudeBridge と同じ「素通し」方針 (codex-stream-bridge.mjs
-  // 冒頭コメント参照)。claude 版にある rate_limit/MCP control query 相当の処理は無い
-  // (codex 側にまだ存在しない/後続スライス)。live usage だけを producer 単位で記録する。
-  if (codexBridge) {
-    codexBridge.on("event", ({ stream_id, producer_seq, thread_id, cwd, notification }) => {
-      if (notification.method === "thread/tokenUsage/updated") {
-        recordCodexLiveTokenUsage(producer_seq, notification.params?.tokenUsage)
-      }
-      sendBridgeMessage(client, { type: "codex.event", stream_id, thread_id, event: notification })
-    })
-    codexBridge.on("permission", ({ stream_id, thread_id, cwd, request_id, method, params }) => {
-      sendBridgeMessage(client, {
-        type: "codex.permission.request",
-        stream_id,
-        thread_id,
-        request_id,
-        method,
-        params,
-      })
-    })
-    codexBridge.on("exit", ({ stream_id, producer_seq, thread_id, code, reason, cwd }) => {
-      // T06407 増分2: この producer(session.seq) の live tokenUsage スロットを破棄する。
-      // seq は immutable なので、reattach で stream_id が付け替わっても alias slot / Map リークは
-      // 生じない (stream_id キー時代の major#1 を解消)。
-      try {
-        forgetCodexLiveTokenUsage(producer_seq)
-      } catch {
-        /* ignore */
-      }
-      sendBridgeMessage(client, { type: "codex.exit", stream_id, thread_id, code, reason })
-    })
-    codexBridge.on("error", ({ stream_id, thread_id, error }) => {
-      sendBridgeMessage(client, { type: "codex.error", stream_id, thread_id, error })
-    })
-  }
+  wireBridgeEvents(client, { claudeBridge, codexBridge })
 
   // Hub からのメッセージ dispatch は、接続世代 + 機能レーン単位で直列実行する。
   //
@@ -4576,18 +4643,24 @@ export async function dispatch(msg, ctx) {
           // 添えれば Codex 向け skills (~/.codex/skills/) を返す。未指定は
           // 従来どおり claude 向け。
           const skillsCliKind = msg.cli_kind === "codex" ? "codex" : "claude"
-          // T06117: claude は SDK supportedCommands() を権威に一覧を作る (組込 +
-          // グローバル skill + plugin/dynamic を FS 走査の取りこぼしなく反映)。
-          // 取れなければ null で listSkills が FS 走査へフォールバックする。
-          let sdkCommands = null
-          if (skillsCliKind === "claude" && ctx.claudeBridge?.listSupportedCommands) {
-            sdkCommands = await ctx.claudeBridge.listSupportedCommands()
-          }
-          const skills = await listSkills({
-            sessionName: msg.session_name,
-            cliKind: skillsCliKind,
-            sdkCommands,
-          })
+          const skills = await coalesceInflight(
+            listRequestsInflight,
+            skillsRequestKey(skillsCliKind, msg.session_name),
+            async () => {
+              // T06117: claude は SDK supportedCommands() を権威に一覧を作る (組込 +
+              // グローバル skill + plugin/dynamic を FS 走査の取りこぼしなく反映)。
+              // 取れなければ null で listSkills が FS 走査へフォールバックする。
+              let sdkCommands = null
+              if (skillsCliKind === "claude" && ctx.claudeBridge?.listSupportedCommands) {
+                sdkCommands = await ctx.claudeBridge.listSupportedCommands()
+              }
+              return listSkills({
+                sessionName: msg.session_name,
+                cliKind: skillsCliKind,
+                sdkCommands,
+              })
+            },
+          )
           ctx.client.send({
             type: "skills.response",
             request_id: msg.request_id,
@@ -4607,7 +4680,8 @@ export async function dispatch(msg, ctx) {
       }
       case "agents.request": {
         try {
-          const agents = await listAgents()
+          // 引数を持たない要求なので、実行中の取得があれば全員で結果を共有する。
+          const agents = await coalesceInflight(listRequestsInflight, "agents", () => listAgents())
           ctx.client.send({
             type: "agents.response",
             request_id: msg.request_id,