@cocorograph/hub-agent 0.8.0 → 0.8.2

package/src/main.mjs

@@ -327,7 +327,11 @@ export function isReadOnlyRpc(type) {
   return (
     type === "claude.history.request" ||
     type === "claude.sessions.request" ||
-    type === "codex.sessions.request"
+    type === "codex.sessions.request" ||
+    // T06603 (2026-07-25): キュー snapshot の pull。in-memory 配列を読むだけで副作用が無く、
+    // かつ「直列キュー前方の重い処理で再同期が遅延する」と送信待ちチップの固着が長引く
+    // (再同期は固着を直すための経路なので、詰まりやすい経路に置いてはならない)。
+    type === "claude.queue.request"
   )
 }
 
@@ -2859,6 +2863,13 @@ const seenCodexChatInputIds = makeSeenInputIdRegistry()
 // Promise へ合流させる (settle 時に元の .then/.catch が 1 回だけ ack/nack を出す)。
 const codexInflightByInputId = new Map()
 
+// T06603 (2026-07-25): claude.input の **in-flight** dedup 用レジストリ (input_id -> Promise)。
+// 上の codex 版と同じ理由・同じ流儀。claude 側は sendMessage が resident query への
+// reconcile (applyFlagSettings の control request 往復) を await するため settle まで数秒
+// かかることがあり、その窓で ackWatchdog(5s) の再送が入ると `seen` 未登録のまま二重投入され、
+// 同一 input_id がキューに 2 件積まれて drain 後も 1 件残る (送信待ちチップが消えない)。
+const claudeInflightByInputId = new Map()
+
 /**
  * input_id 付き pty.data (tracked 入力) の処理。write 成功で pty.input.ack を返し、
  * stream 不在なら pty.error を返して browser の再 attach → 再送に繋げる。
@@ -2933,7 +2944,20 @@ export function handleClaudeChatInput(msg, ctx) {
     ctx.client.send({ type: "claude.input.ack", stream_id, input_id })
     return undefined
   }
-  return ctx.claudeBridge
+  if (input_id && claudeInflightByInputId.has(input_id)) {
+    // T06603 (2026-07-25): **処理中** の同一 input_id (browser の ackWatchdog が 5s で
+    // 再送してくるが、こちらはまだ settle していない窓)。codex 側は round4 major#1 で
+    // 対処済みだったが claude 側だけ欠けており、`seen` は settle 後にしか登録されないため
+    // 再送がそのまま二重投入されていた (同じ本文が 2 回実行され、かつ同一 id の項目が
+    // キューに 2 つ積まれて drain 後も 1 つ残る = 「実行されたのにキューから消えない」)。
+    // 既存 Promise へ合流させ、ack/nack は元の 1 回だけ出す。
+    ctx.logger?.info?.(
+      { stream_id, input_id },
+      "claude.input in-flight dedup (join, no re-dispatch)",
+    )
+    return claudeInflightByInputId.get(input_id)
+  }
+  const p = ctx.claudeBridge
     .input({
       stream_id,
       message: msg.message,
@@ -2972,6 +2996,13 @@ export function handleClaudeChatInput(msg, ctx) {
         })
       }
     })
+    .finally(() => {
+      // in-flight レジストリから外す (成功時は seen が以降の再送を dedup、
+      // 失敗時は再送で正当にリトライさせる)。codex 版と同じ流儀。
+      if (input_id) claudeInflightByInputId.delete(input_id)
+    })
+  if (input_id) claudeInflightByInputId.set(input_id, p)
+  return p
 }
 
 /**
@@ -3386,6 +3417,12 @@ async function dispatch(msg, ctx) {
             // 旧 agent にはこのフラグが無く、browser は楽観アイテムの×を出さない (ゴースト
             // 実行防止のフォールバック)。
             supports_queue_input_id: true,
+            // pull 型再同期 capability (T06603, 2026-07-25): この agent は
+            // claude.queue.request に claude.queue.response で答え、queue_state に
+            // 版番号 (seq) と受理済み input_id (accepted) を載せることを通知する。
+            // 旧 agent はこのフラグが無く、browser は従来の「配信到達に依存する」
+            // 挙動へフォールバックする (混在期間も安全に段階デプロイできる)。
+            supports_queue_snapshot: true,
           })
           // dispatch全体はWSメッセージ順を守る直列キュー上にある。ここで履歴I/Oをawaitすると
           // 同一接続のpermission/input等まで停止するため、readyの同期送信後は非同期でpushする。
@@ -3770,11 +3807,67 @@ async function dispatch(msg, ctx) {
         }
         return
       }
-      case "claude.queue.cancel":
+      case "claude.queue.cancel": {
         // 送信待ち (pending) メッセージを id 指定で取り消す (0.6.26)。
-        if (!ctx.claudeBridge) return
-        ctx.claudeBridge.cancelQueued({ stream_id: msg.stream_id, id: msg.id })
+        // T06603 cross-review 4 巡目: 取り消せなかった (stream 不在) ときは browser へ
+        // 失敗を返す。browser は要求時にチップを消さず「取消中」印を付けて権威の確認を
+        // 待つため、無応答だと印が外れず固着する。request_id をエコーして相関させる。
+        // T06603 cross-review 5 巡目: 結末 (cancelled / already_started / stream_missing) を
+        // **必ず** 返す。「pending から消えた」だけでは取消成功と実行開始済みを区別できず、
+        // 特に drain の started フレームを取りこぼしていると、browser は取消成功に見せかけた
+        // まま実行を続行してしまう (発言バブルも欠けたまま)。request_id + input_id で相関させる。
+        const cancelRes = {
+          type: "claude.queue.response",
+          stream_id: msg.stream_id,
+          request_id: msg.request_id || null,
+          input_id: msg.id || null,
+        }
+        if (!ctx.claudeBridge) {
+          ctx.client.send({ ...cancelRes, outcome: "stream_missing" })
+          return
+        }
+        const canceled = ctx.claudeBridge.cancelQueued({
+          stream_id: msg.stream_id,
+          id: msg.id,
+        })
+        // ⚠️ T06603 cross-review 6 巡目: **outcome を先に送る**。キュー snapshot の再表明を
+        // 先に流すと、backend は到着順に relay するため browser は空 snapshot で
+        // 「キューに無い ＋ 受理済み = 実行完了」と判定してチップを消し、後続の outcome は
+        // 対象不在で no-op になる (already_started のバブル補償が発火しない)。
+        ctx.client.send({
+          ...cancelRes,
+          outcome: canceled?.outcome || "stream_missing",
+        })
+        canceled?.flush?.()
+        return
+      }
+      case "claude.queue.request": {
+        // pull 型再同期 (T06603, 2026-07-25): browser が権威キュー snapshot を引きにくる。
+        // 従来キュー表示の是正手段は「agent が turn settle 毎に force 再送する queue_state」
+        // だけで、その配信が 1 通落ちると browser には自己修復の手段が無く送信待ちチップが
+        // 恒久固着した。ここで「いつでも引き直せる」経路を与えるのが根治の中核。
+        // 応答は要求元 stream へ返す (backend が cockpit_stream_<stream_id> へ relay)。
+        const queueReq = {
+          type: "claude.queue.response",
+          stream_id: msg.stream_id,
+          request_id: msg.request_id || null,
+        }
+        if (!ctx.claudeBridge) {
+          ctx.client.send({ ...queueReq, error: "stream_missing" })
+          return
+        }
+        const got = ctx.claudeBridge.queueSnapshot({ stream_id: msg.stream_id })
+        if (!got) {
+          ctx.client.send({ ...queueReq, error: "stream_missing" })
+          return
+        }
+        ctx.client.send({
+          ...queueReq,
+          session_id: got.session_id,
+          event: got.snapshot,
+        })
         return
+      }
       case "claude.detach":
         if (!ctx.claudeBridge) return
         ctx.claudeBridge.detach({ stream_id: msg.stream_id })

package/src/sibling-inject.mjs

@@ -45,6 +45,16 @@ import { verifySiblingTokenAny, siblingTokenCwd } from "./sibling-token.mjs"
 export const SIBLING_REQUESTS_DIR =
   process.env.COCKPIT_SIBLING_REQUESTS_DIR || "/tmp/cockpit_sibling_requests"
 
+const DELIVERY_UNCONFIRMED_HINT =
+  "turn RPC は受理されましたが userMessage echo を確認できませんでした。既に届いている可能性があり、" +
+  "再送すると重複配送になります。再送前に対象セッションの受信状態または rollout 監査を確認してください。"
+const DELIVERY_FAILED_HINT =
+  "turn RPC の受付前または受付中に失敗しました。対象セッションを確認して再送してください。"
+
+function deliveredStateFor(kind) {
+  return kind === "codex" ? "delivered" : "dispatched"
+}
+
 export class SiblingInjectBridge {
   /**
    * @param {{dir?: string, claudeBridge?: object, codexBridge?: object, logger?: object}} [opts]
@@ -186,11 +196,34 @@ export class SiblingInjectBridge {
       } else if (isExplicit) {
         // 明示宛先: session_id (claude) / stream_id|thread_id (codex) で cwd 一致下に確定解決。
         // sendToStream は {status:"delivered"|"not_found"|"ambiguous"} を返す。
-        const r = bridge?.sendToStream?.(cwd, target_session_id, message, injectSource) ?? {
-          status: "not_found",
-        }
+        const r = bridge?.sendToStream
+          ? await bridge.sendToStream(cwd, target_session_id, message, injectSource)
+          : { status: "not_found" }
         if (r.status === "delivered") {
-          result = { ok: true, delivered_to: r.session }
+          result = {
+            ok: true,
+            delivered_to: r.session,
+            delivery_id: r.delivery_id,
+            delivery_state: deliveredStateFor(target_cli_kind),
+            durable_state: r.durable_state,
+            accepted_via: r.accepted_via,
+            item_id: r.item_id,
+          }
+        } else if (r.status === "delivery_unconfirmed" || r.status === "delivery_failed") {
+          result = {
+            ok: false,
+            error: r.status,
+            delivered_to: r.session,
+            delivery_id: r.delivery_id,
+            delivery_state:
+              r.status === "delivery_unconfirmed" ? "accepted_unconfirmed" : "failed",
+            accepted_via: r.accepted_via,
+            reason: r.reason,
+            hint:
+              r.status === "delivery_unconfirmed"
+                ? DELIVERY_UNCONFIRMED_HINT
+                : DELIVERY_FAILED_HINT,
+          }
         } else if (r.status === "ambiguous") {
           // 宛先 id が同一 cwd の複数 live セッションに一致し一意に定まらない。
           // ⚠️ 解消手段は CLI で異なる (T06271 レビュー4 minor):
@@ -235,9 +268,34 @@ export class SiblingInjectBridge {
         //    (サイレント誤配送 + delivered_to に実配送先でない sameCwd[0] を報告する成功偽装) が
         //    あった。sendToCwd を status 返却にして解決と配送を 1 呼び出しに統合し、delivered_to は
         //    **実配送先 (r.session)** を返す。
-        const r = bridge?.sendToCwd?.(cwd, text, injectSource) ?? { status: "not_found" }
+        const r = bridge?.sendToCwd
+          ? await bridge.sendToCwd(cwd, text, injectSource)
+          : { status: "not_found" }
         if (r.status === "delivered") {
-          result = { ok: true, delivered_to: r.session ?? null }
+          result = {
+            ok: true,
+            delivered_to: r.session ?? null,
+            delivery_id: r.delivery_id,
+            delivery_state: deliveredStateFor(target_cli_kind),
+            durable_state: r.durable_state,
+            accepted_via: r.accepted_via,
+            item_id: r.item_id,
+          }
+        } else if (r.status === "delivery_unconfirmed" || r.status === "delivery_failed") {
+          result = {
+            ok: false,
+            error: r.status,
+            delivered_to: r.session ?? null,
+            delivery_id: r.delivery_id,
+            delivery_state:
+              r.status === "delivery_unconfirmed" ? "accepted_unconfirmed" : "failed",
+            accepted_via: r.accepted_via,
+            reason: r.reason,
+            hint:
+              r.status === "delivery_unconfirmed"
+                ? DELIVERY_UNCONFIRMED_HINT
+                : DELIVERY_FAILED_HINT,
+          }
         } else if (r.status === "ambiguous") {
           result = {
             ok: false,
@@ -258,10 +316,24 @@ export class SiblingInjectBridge {
         target_cli_kind,
         op: op || "normal",
         ...(result.ok
-          ? { delivered_to: result.delivered_to }
-          : { error: result.error, live_sessions: result.live_sessions }),
+          ? {
+              delivered_to: result.delivered_to,
+              delivery_id: result.delivery_id,
+              delivery_state: result.delivery_state,
+              durable_state: result.durable_state,
+            }
+          : {
+              error: result.error,
+              delivery_id: result.delivery_id,
+              delivery_state: result.delivery_state,
+              live_sessions: result.live_sessions,
+            }),
       },
-      result.ok ? "sibling-inject delivered" : "sibling-inject failed",
+      result.ok
+        ? result.delivery_state === "delivered"
+          ? "sibling-inject delivered"
+          : "sibling-inject dispatched"
+        : "sibling-inject failed",
     )
     await this._writeResult(request_id, result)
     // 要求ファイルは処理後に必ず削除する (成否問わず、再処理させないため)。