@cocorograph/hub-agent 0.8.14 → 0.8.16

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -1408,11 +1408,20 @@ export class ClaudeStreamSession {
    *   メッセージ本文。drain 由来の emit でのみ渡す。frontend はこれを user バブルへ
    *   昇格させる。キャンセル / 追加由来の emit では空 (昇格させない)。これにより
    *   「先頭の pending をキャンセルした」のを「実行開始した」と誤認しなくなる (0.6.26)。
-   *  @param {{force?: boolean}} [opts] force=true のとき署名重複チェックを無視して必ず
-   *   emit する。再アタッチ時のキュー snapshot 再送 (キュー再表示バグ修正, 0.6.30) に使う。
-   *   署名が前回と同一でも browser がライブ配信を取りこぼしている可能性があるため。 */
+   *  @param {{force?: boolean, startedIds?: (string|null)[], discarded?: string[]}} [opts]
+   *   force=true のとき署名重複チェックを無視して必ず emit する。再アタッチ時のキュー
+   *   snapshot 再送 (キュー再表示バグ修正, 0.6.30) に使う。署名が前回と同一でも browser が
+   *   ライブ配信を取りこぼしている可能性があるため。
+   *   discarded は「実行せずに捨てた input_id」(close 時の pending 破棄)。browser はこれを
+   *   受けてチップを消さず未配信印を付ける (無言削除の根治, 2026-07-31)。 */
   _emitQueueState(started = [], opts = undefined) {
     const force = opts?.force === true
+    // 無言削除の根治 (2026-07-31): 実行せずに捨てた input_id。空配列は載せない
+    // (旧 browser との後方互換 = 未知フィールドは無視されるが、無駄なノイズも出さない)。
+    const discarded =
+      Array.isArray(opts?.discarded) && opts.discarded.length > 0
+        ? [...opts.discarded]
+        : undefined
     // 実行開始した項目の input_id (started と parallel。T06603 cross-review major#2)。
     // 本文テキストだけだと、同一本文を連投したときに 2 通目のユーザー発言バブルが
     // 「既に同じ本文のバブルがある」と誤判定されて永久に出ない。id で突合させる。
@@ -1446,6 +1455,8 @@ export class ClaudeStreamSession {
       // items は id 付きでキャンセルボタンの対象特定に使う (0.6.26)。
       this.onEvent?.({
         ...this._buildQueueSnapshot(),
+        // 実行せずに捨てた id (close 時の pending 破棄)。載る時だけキーを出す。
+        ...(discarded ? { discarded } : {}),
         started,
         // started と parallel。連携注入なら 'peer'|'orchestrator'、通常は null。
         startedSources,
@@ -2105,12 +2116,30 @@ export class ClaudeStreamSession {
   close() {
     this._closed = true
     // 改修3: 未処理の pending を破棄 (セッション終了後に drain しないため)。
+    const discardedIds = this._pendingMessages
+      .map((m) => m.id)
+      .filter((id) => typeof id === "string" && id)
     const hadPending = this._pendingMessages.length > 0
     this._pendingMessages = []
+    // 無言削除の根治 (2026-07-31): 破棄した id を **受理済みリングから外す**。
+    // browser の 2 軸判定は「キューに無い ＋ 受理済み ＝ 実行完了」なので、破棄した id を
+    // accepted に残したまま空 snapshot を送ると、**実行していない送信を実行完了と誤認して
+    // チップを無言削除**する (ユーザーが送った本文が痕跡なく消える)。accepted から外し、
+    // かつ discarded として明示して「捨てた」ことを browser に伝える。
+    for (const id of discardedIds) {
+      const at = this._acceptedInputIds.indexOf(id)
+      if (at !== -1) this._acceptedInputIds.splice(at, 1)
+    }
     // T06603 (2026-07-25): 破棄したことを必ず通知する。従来は無言で捨てていたため、
     // 「セッションが畳まれた後もチップだけが残る」表示が残留していた。破棄は状態変化なので
     // force で貫通させる (started=[] なのでバブル昇格は起きない)。
-    if (hadPending) this._emitQueueState([], { force: true })
+    if (hadPending) {
+      this.logger?.info(
+        { stream_id: this.stream_id, discarded: discardedIds.length },
+        "pending messages discarded on close",
+      )
+      this._emitQueueState([], { force: true, discarded: discardedIds })
+    }
     // 常駐 query は input キューを閉じて generator を終わらせ query を完走させる。
     if (this._inputQueue) this._inputQueue.close()
     this.abortTurn()

package/src/dispatch-scheduler.mjs

@@ -23,10 +23,44 @@ const LIFECYCLE_TYPES = new Set([
 
 const MAX_STREAM_LANE_KEYS = 4096
 
+/**
+ * dispatch の所要時間ログを出す閾値 (ms)。
+ *
+ * ⚠️ これは計測のために足したもので、性能を変えるものではない。
+ * 「セッションの停止に 10 秒経っても応答が返らない」の中身が **一切観測できなかった**
+ * ため入れた (2026-07-31)。kill 実体は実測 0.00 秒・レーン内の tmux 処理も 0.1 秒未満
+ * なので、遅延の候補は「レーンの順番待ち」か「そもそも届いていない」に絞られるが、
+ * どちらなのかを判別する材料がログに 1 行も無かった。
+ *
+ * 全 dispatch を出すとログが膨れるので、閾値超えのものだけ出す。
+ */
+const SLOW_DISPATCH_MS = 1000
+
+/**
+ * 閾値に関係なく必ず所要時間を出す type。
+ *
+ * 停止 (kill) は低頻度なのでログ量の心配が無く、かつ「遅い / 返ってこない」の
+ * 調査対象そのものなので、速かった場合も含めて毎回記録する
+ * (速いことの記録が無いと「詰まっていない」を示せない)。
+ */
+const ALWAYS_TIMED_TYPES = new Set(["tmux.kill_session"])
+
 export function isConnectionLifecycleMessage(type) {
   return LIFECYCLE_TYPES.has(type)
 }
 
+/**
+ * 所要時間ログを出すべきか。テストから直接検査できるよう export する。
+ *
+ * @param {string} type    メッセージ種別
+ * @param {number} waitMs  レーンで順番待ちしていた時間
+ * @param {number} runMs   ハンドラ本体の実行時間
+ */
+export function shouldLogDispatchTiming(type, waitMs, runMs) {
+  if (ALWAYS_TIMED_TYPES.has(type)) return true
+  return waitMs >= SLOW_DISPATCH_MS || runMs >= SLOW_DISPATCH_MS
+}
+
 function chatMode(type) {
   if (typeof type !== "string") return null
   if (type === "claude.attach" || type.startsWith("claude.")) return "claude"
@@ -179,6 +213,8 @@ export class DispatchScheduler {
   enqueue(msg) {
     const type = msg?.type
     const sequence = ++this.sequence
+    // 順番待ち時間の起点。bypass 経路は待たないので測らない。
+    const enqueuedAt = Date.now()
     if (this.shouldBypass(type) || isConnectionLifecycleMessage(type)) {
       return this._runNow(msg)
     }
@@ -217,7 +253,21 @@ export class DispatchScheduler {
           )
           return
         }
-        await this.run(msg)
+        // レーンの順番待ち時間と、ハンドラ本体の実行時間を分けて測る。
+        // この 2 つを分けないと「遅いのは順番待ちか処理そのものか」が判別できない。
+        const startedAt = Date.now()
+        const waitMs = startedAt - enqueuedAt
+        try {
+          await this.run(msg)
+        } finally {
+          const runMs = Date.now() - startedAt
+          if (shouldLogDispatchTiming(type, waitMs, runMs)) {
+            this.logger?.info?.(
+              { type, lane, wait_ms: waitMs, run_ms: runMs },
+              "dispatch timing",
+            )
+          }
+        }
       })
       .catch((err) => {
         this.logger?.error?.(

package/src/main.mjs

@@ -295,6 +295,36 @@ async function loadClaudeSdk(logger) {
   }
 }
 
+/** アカウント利用量取得の時間制限 (ms)。超えたらレーンを解放して次へ進む。 */
+export const ACCOUNT_USAGE_TIMEOUT_MS = 8_000
+
+/**
+ * `promise` に時間制限を設ける。制限を超えたら `null` で解決し `onTimeout` を呼ぶ。
+ *
+ * ⚠️ **元の promise はキャンセルできない**ので、裏では走り続ける。ここでの目的は
+ * 「返ってこない処理に dispatch レーンを占有させ続けない」ことであり、処理そのものを
+ * 止めることではない。時間切れを黙って握りつぶさず呼び出し側でログに残すこと
+ * (時間切れの発生回数と頻度が、詰まりの実測値そのものになる)。
+ *
+ * テストから直接呼べるよう export する。
+ */
+export function withDeadline(promise, timeoutMs, onTimeout) {
+  let timer
+  return Promise.race([
+    Promise.resolve(promise).finally(() => clearTimeout(timer)),
+    new Promise((resolve) => {
+      timer = setTimeout(() => {
+        try {
+          onTimeout?.()
+        } catch {
+          /* ログ失敗で本筋を止めない */
+        }
+        resolve(null)
+      }, timeoutMs)
+    }),
+  ])
+}
+
 /**
  * B7: 直列 dispatchChain をバイパスして即時処理してよい高頻度・低レイテンシ経路かを判定する。
  * - pty 出力データ (pty.data) と resize (pty.resize): 高頻度・順序保証不要。
@@ -3242,13 +3272,33 @@ export async function handleKillSession(msg, ctx) {
     })
     return
   }
+  // ⚠️ kill の受信・完了は必ずログに残す (2026-07-31 追加)。
+  // 「停止に 10 秒経っても応答が返らない」の調査で、agent 側に kill の痕跡が
+  // 1 行も無く、遅いのか・届いていないのか・返事が捨てられたのかを**判別できなかった**。
+  // kill は低頻度なのでログ量の懸念は無い。速かった場合も記録する
+  // (速いことの記録が無いと「agent は詰まっていない」を示せない)。
+  ctx.logger?.info?.({ names, request_id: msg.request_id }, "tmux kill_session received")
+  const killStartedAt = Date.now()
   const r = await killFn(names)
-  ctx.client.send({
+  const killMs = Date.now() - killStartedAt
+  const sent = ctx.client.send({
     type: "tmux.kill_session.result",
     request_id: msg.request_id,
     killed: r.killed,
     failed: r.failed,
   })
+  // 送信できたか自体を残す。ws が閉じていると result は黙って捨てられるため、
+  // 「kill は成功したのに要求元へ返っていない」を後から特定できるようにする。
+  ctx.logger?.info?.(
+    {
+      request_id: msg.request_id,
+      killed: r.killed.length,
+      failed: r.failed.length,
+      kill_ms: killMs,
+      result_sent: sent === true,
+    },
+    "tmux kill_session done",
+  )
   for (const name of r.killed) {
     try {
       ctx.client.send({ type: "session.gone", session_name: name })
@@ -4926,7 +4976,20 @@ async function dispatch(msg, ctx) {
           // は no-op) ので、既存の statusLine/transcript フォールバックは無傷で残る。
           if (ctx.claudeBridge) {
             try {
-              const acct = await ctx.claudeBridge.getAccountUsage()
+              // ⚠️ getAccountUsage は SDK のサブプロセスを spawn する。**時間制限が無いと
+              // 返ってこないときに control レーン全体が止まる** (usage.request は 60 秒周期で
+              // control レーンに乗り、そこにセッション停止も並ぶ)。同種の head-of-line
+              // blocking で cold p95 40〜54 秒を計測した記録が isReadOnlyRpc の注記にある。
+              // 時間切れは握りつぶさず warn で残す (これ自体が「詰まりの実測値」になる)。
+              const acct = await withDeadline(
+                ctx.claudeBridge.getAccountUsage(),
+                ACCOUNT_USAGE_TIMEOUT_MS,
+                () =>
+                  ctx.logger?.warn?.(
+                    { timeout_ms: ACCOUNT_USAGE_TIMEOUT_MS },
+                    "account usage timed out (dispatch lane released)",
+                  ),
+              )
               if (acct) recordChatUsageSnapshot(acct, Date.now(), activeClaudeProfile.id)
             } catch (err) {
               ctx.logger?.debug?.({ err: err?.message }, "account usage snapshot skipped")