@cocorograph/hub-agent 0.7.49 → 0.7.50

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -209,6 +209,24 @@ function extractPromptText(message) {
   return ""
 }
 
+/**
+ * SDK-P0-3 (2026-07-10): SDK の権威的ターン終了シグナル。
+ * sdk.d.ts (SDKSessionStateChangedMessage) いわく
+ *   "'idle' fires after heldBackResult flushes and the bg-agent do-while exits
+ *    — authoritative turn-over signal."
+ * つまり result が流れるターンでは `result` → `idle` の順で届き、result が届かない
+ * ターン (held back のまま終わる等) でも idle は届く。frontend は SDK-P0-1 で既に
+ * idle をターン終端として扱っているが、agent 側 (resident query) は result しか見て
+ * おらず _busy が固着していた。その非対称を解消するための判定。
+ */
+function isSessionIdleEvent(msg) {
+  return (
+    msg?.type === "system" &&
+    msg?.subtype === "session_state_changed" &&
+    msg?.state === "idle"
+  )
+}
+
 /**
  * 1 stream に対応する Claude セッション (per-message query モデル)。
  */
@@ -331,6 +349,18 @@ class ClaudeStreamSession {
     this._permissionResolvers = new Map()
     /** 現在ターン実行中か (多重 query 防止) */
     this._busy = false
+    /** SDK-P0-3 (2026-07-10): 現ターンが既に終端処理済みか (result / idle / abort の
+     *  いずれか 1 回だけ _busy=false + drain させるための冪等ガード)。resident query は
+     *  1 ターンにつき `result` と `session_state_changed:idle` の**両方**が流れうるため、
+     *  素朴に両方で drain すると二重 drain (FIFO 崩れ) を起こす。ターン開始時
+     *  (_beginResidentTurn) に false へ戻す。 */
+    this._turnSettled = false
+    /** SDK-P0-3: 現ターンが 1 件でもストリームイベントを出したか。
+     *  「前ターンの遅延 idle が、drain で始まったばかりの次ターンを誤終了させる」レースの
+     *  ガードに使う。generator のイベント順序は保証される (turn N の idle は turn N+1 の
+     *  最初のイベントより必ず先に届く) ため、`idle 到着時に現ターンがまだ何も出していない`
+     *  = その idle は前ターンのもの、と判定できる。ターン開始時に false へ戻す。 */
+    this._turnProgressed = false
     /** detach 済みフラグ (新規ターン受付停止) */
     this._closed = false
     /** 実行中ターンの完走を待ってからクローズする予約フラグ (graceful detach 用)。
@@ -721,7 +751,7 @@ class ClaudeStreamSession {
         this._emitQueueState()
         return { accepted: true }
       }
-      this._busy = true
+      this._beginResidentTurn()
       // 改修4 (A): 死亡ガード。常駐 query が未起動 or (エラー等で) 終了済み (_residentQuery=null)
       // なら (再)起動する。_runResidentQuery は起動時 options.resume=this.sessionId で文脈を
       // 復元するため、途中死からの復活でも過去コンテキストは失われない。
@@ -1067,6 +1097,40 @@ class ClaudeStreamSession {
     }
   }
 
+  /** SDK-P0-3 (2026-07-10): resident の「ターン開始」を 1 箇所に集約する。
+   *  _busy を立てるのと同時に、そのターンの終端フラグ (_turnSettled) と進行フラグ
+   *  (_turnProgressed) を必ずリセットする。ターン開始は 3 箇所ある
+   *  (sendMessage の直接 push / _drainResidentPending / _runResidentQuery finally の
+   *  異常再起動 push) ので、リセット漏れを構造的に防ぐためのヘルパー。 */
+  _beginResidentTurn() {
+    this._busy = true
+    this._turnSettled = false
+    this._turnProgressed = false
+  }
+
+  /** SDK-P0-3 (2026-07-10): resident の「ターン終端」を 1 箇所に集約する (result / idle 共通)。
+   *  同一ターンで 2 回以上呼ばれても最初の 1 回しか作用しない (_turnSettled ガード)。
+   *  これにより `result` → `idle` の順で両方届く通常ケースでも drain は 1 回だけになる。
+   *  @param {string} reason "result" | "idle" — ログ用
+   *  @param {(msg: string) => void} [denyPending] 未解決 permission を閉じるクロージャ
+   */
+  _settleResidentTurn(reason, denyPending) {
+    if (this._turnSettled) return
+    this._turnSettled = true
+    denyPending?.("turn ended")
+    this._busy = false
+    // 改修4 (A): シリアライズした pending があれば次の 1 件を InputQueue へ流す。
+    // ultracode (0.6.28): _drainResidentPending は applyFlagSettings を await するため
+    // async。for await ループ内からは fire-and-forget で呼ぶ (最初の await より前に
+    // _beginResidentTurn が同期実行されるので、以降の sendMessage は busy を正しく観測する)。
+    this._drainResidentPending().catch((err) =>
+      this.logger?.warn(
+        { stream_id: this.stream_id, err: err?.message, reason },
+        "drainResidentPending threw",
+      ),
+    )
+  }
+
   /** 改修4 (A): ターン完了時に pending の先頭 1 件を InputQueue へ流す (ターンのシリアライズ)。
    *  queue_state を更新して送信待ちチップを drain させる (frontend がバブル昇格する)。
    *  ultracode (0.6.28): 次ターンの目標 ultracode 状態へ reconcile してから push する
@@ -1082,7 +1146,7 @@ class ClaudeStreamSession {
     // _busy=true のまま残り、以降の drain が全て弾かれて pending が永久に積み上がる
     // デッドロックになる。失敗時は取り出したメッセージを先頭へ戻し _busy を解除して、
     // 次の drain がやり直せる状態に必ず回復する (例外は既存方針通り warn で握る)。
-    this._busy = true
+    this._beginResidentTurn()
     try {
       // ultracode reconcile (0.6.28) も await で例外を投げうるので try の中に含める。
       await this._reconcileResidentUltracode(next.ultracode === true)
@@ -1100,8 +1164,10 @@ class ClaudeStreamSession {
 
   /** 改修2+4: 常駐 query を 1 回だけ起動し、streaming input キューから複数ターンを処理する。
    *  system/init は最初の 1 回のみ (毎ターン init = 文脈/キャッシュ温め直しを解消)。各ターンは
-   *  result で _busy=false にするが query は継続し次 input を待つ。close (inputQueue.close) で
-   *  generator が終わり query が完走する。
+   *  終端シグナル (result / session_state_changed:idle) で _busy=false にするが query は継続し
+   *  次 input を待つ。close (inputQueue.close) で generator が終わり query が完走する。
+   *  SDK-P0-3 (2026-07-10): 終端を result だけに頼ると、SDK が idle でターンを終えたときに
+   *  _busy が固着し、以降の送信が pending へ永久滞留する (T06052)。
    *  改修4: resume セッションも対象。query 起動時 options.resume に過去 session_id を渡して
    *  文脈を引き継ぐ (init は初回 1 回のみ)。continue は使わない (自動継続の暴走回避)。初回 input は
    *  sendMessage が積むユーザーの実メッセージなので「Continue from where you left off」は発生しない。
@@ -1153,17 +1219,34 @@ class ClaudeStreamSession {
         if (msg?.type === "result") {
           if (typeof msg.session_id === "string") this.sessionId = msg.session_id
           // ターン完了: 未解決 permission を閉じ、次 input 受付へ (query は継続)。
-          denyPending("turn ended")
-          this._busy = false
-          // 改修4 (A): シリアライズした pending があれば次の 1 件を InputQueue へ流す。
-          // ultracode (0.6.28): _drainResidentPending は applyFlagSettings を await するため
-          // async。result ハンドラ (for await ループ内) からは fire-and-forget で呼ぶ。
-          this._drainResidentPending().catch((err) =>
-            this.logger?.warn(
-              { stream_id: this.stream_id, err: err?.message },
-              "drainResidentPending threw",
-            ),
-          )
+          this._settleResidentTurn("result", denyPending)
+        } else if (isSessionIdleEvent(msg)) {
+          // SDK-P0-3 (2026-07-10): result が届かずに idle でターンが終わるケースの救済。
+          // これを見ていなかったため _busy が true で固着し、以降の送信が
+          // _pendingMessages に退避されたまま (drain 契機の result が永久に来ないので)
+          // 永久滞留していた ("resident busy, message queued" が出続ける / リロードで消える)。
+          //
+          // ガード: 「前ターンの遅延 idle」で新ターンを誤終了させない。result で settle →
+          // drain → 次ターン開始 (_busy=true) の直後に、同じターンの idle が続けて届く
+          // (SDK は result → idle の順に流す) ため、素朴に _busy=false にすると新ターンを
+          // 撃ち抜き二重 drain / FIFO 崩れを起こす。generator のイベント順序は保証されるので
+          // 「現ターンがまだ 1 件もイベントを出していない (_turnProgressed=false)」なら
+          // その idle は前ターンのものと確定できる。
+          if (this._busy && this._turnProgressed) {
+            this._settleResidentTurn("idle", denyPending)
+            // per-message の RC-4 と対称に、チャット信号 (サイドバードット) を
+            // processing→waiting へ落とす。result 経由なら main.mjs が result を見て
+            // waiting にするが、idle 単独ではその契機が無く 90s TTL まで固着するため。
+            try {
+              this.onTurnSettled?.()
+            } catch {
+              /* ignore */
+            }
+          }
+        } else {
+          // 現ターンが「進行した」証跡。init / assistant / stream_event / user、および
+          // session_state_changed の running / requires_action もここに含む。
+          this._turnProgressed = true
         }
         try {
           this._emit(msg)
@@ -1207,7 +1290,7 @@ class ClaudeStreamSession {
         // close() するだけで参照を持ち越さないので、ここで差し替えても整合する。
         this._inputQueue = new InputQueue()
         const next = this._pendingMessages.shift()
-        this._busy = true
+        this._beginResidentTurn()
         this._residentStarted = true
         // ultracode (0.6.29): query を先に起動して _residentQuery を確定させてから
         // reconcile → push する (sendMessage の通常経路と同順)。push してから起動すると
@@ -1237,6 +1320,11 @@ class ClaudeStreamSession {
       } catch {
         /* ignore */
       }
+      // SDK-P0-3 (2026-07-10): 中断も「このターンの終端」なので settled を立てる。
+      // interrupt() 後に SDK が session_state_changed:idle を流しても、_turnSettled=true
+      // (pending 無しで drain が早期 return するケース) か _turnProgressed=false
+      // (drain が次ターンを開始したケース) のどちらかで必ず無視される。
+      this._turnSettled = true
       this._busy = false
       // SDK-P0-2 (2026-07-07): resident 経路のターン終端イベント非対称の解消。
       // per-message の abort (_runPerMessage finally) は合成 result (aborted_by_user) を
@@ -1248,8 +1336,9 @@ class ClaudeStreamSession {
       // - 実行中ターンがあった (wasBusy) ときだけ emit する (未走行での空 abort に
       //   「中断しました」フッターを出さない)。
       // - close()/reap 経由の abort (_closed=true) では browser は既に離脱しているため抑止する。
-      // - drain タイミングは変更しない (既存の同期 drain を維持し、result+idle 二重 drain や
-      //   中断ターンの遅延 idle が新ターンを誤終了させる新種レースを持ち込まない)。
+      // - drain タイミングは変更しない (既存の同期 drain を維持する)。result+idle の二重 drain と
+      //   中断ターンの遅延 idle による新ターン誤終了は、SDK-P0-3 の _turnSettled /
+      //   _turnProgressed ガードで潰してある (上の _turnSettled=true もその一部)。
       // - 合成 result の uuid は重複排除 (isDuplicateEvent の result 署名判定) 回避のため必須。
       if (wasBusy && !this._closed) {
         try {