@cocorograph/hub-agent 0.8.99 → 0.8.100

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -427,6 +427,9 @@ function isSessionRunningEvent(msg) {
  *  正しく終端として扱う (ガードが固着して _busy が永久 true になる = T06052 の再発を防ぐ安全弁)。 */
 const STALE_IDLE_GUARD_MS = 5000
 
+// sibling CLI の既定待ち 10 秒より短く受理状態を待つ。超過は失敗と断定しない。
+export const SIBLING_ACCEPT_WAIT_MS = 1000
+
 /** per-message の title control request を待って InputQueue を閉じる上限。 */
 const SESSION_TITLE_CLOSE_WAIT_MS = 15_000
 
@@ -1408,7 +1411,7 @@ export class ClaudeStreamSession {
           "resident busy, message queued",
         )
         this._emitQueueState()
-        return { accepted: true }
+        return { accepted: true, queued: true }
       }
       if (this._closed) return { accepted: false, reason: "queue_dead" }
       this._beginResidentTurn(prompt)
@@ -1460,7 +1463,7 @@ export class ClaudeStreamSession {
         "claude busy, message queued",
       )
       this._emitQueueState()
-      return { accepted: true }
+      return { accepted: true, queued: true }
     }
     // Phase 2 (major#1): per-message idle 直接実行も started を出さないため、注入なら source 付きで
     // 昇格バブルを作るよう started を先出しする (通常メッセージは no-op)。
@@ -3695,7 +3698,7 @@ export class ClaudeStreamBridge extends EventEmitter {
    *
    * @param {string} cwd
    * @param {string} text
-   * @returns {{status:"delivered", session:object} | {status:"not_found"} | {status:"ambiguous", candidates:object[]}}
+   * @returns {object | Promise<object>} 配送先なし/曖昧は同期、配送先確定後は受理結果を待つ。
    */
   sendToCwd(cwd, text, source) {
     if (!cwd || !text) return { status: "not_found" }
@@ -3712,32 +3715,7 @@ export class ClaudeStreamBridge extends EventEmitter {
       }
     }
     const { session, attached: att } = tier[0]
-    // 連携カード種別 (Phase 2): 配送先が確定した後に記録する (ambiguous/not_found では記録しない)。
-    // handle を控え、失敗時にその entry だけを除去する (round2: text 除去は並行同一本文で誤消費)。
-    const injectHandle = session._recordInjectSource?.(text, source)
-    session
-      .sendMessage({ role: "user", content: text })
-      .then((r) => {
-        // Phase 2 (major#3): sendMessage は throw せず {accepted:false} で失敗を返す経路がある
-        // (queue_dead / empty_message / pre-canceled)。この場合は echo/started は来ないので
-        // stale な live source entry を掃除し、**永続化もしない** (persist round1 major#1: 失敗を
-        // 永続化すると 30 日 stale が残り後続の同文 human を誤カード化する)。
-        if (r && r.accepted === false) {
-          session._removeInjectSource?.(injectHandle)
-          return
-        }
-        // 受理された (queued/started) ときだけ hydrate 用に永続化する (best-effort、live と独立)。
-        recordInjectSourcePersistent(cwd, text, source).catch(() => {})
-      })
-      .catch((err) => {
-        // 配送失敗 (throw) でも echo は来ないので stale な live entry を掃除する (永続化はしない)。
-        session._removeInjectSource?.(injectHandle)
-        this.logger?.error(
-          { cwd, err: err?.message },
-          "sendToCwd: sendMessage threw unexpectedly",
-        )
-      })
-    return { status: "delivered", session: this._describeCandidate(session, att) }
+    return this._sendSiblingMessage(cwd, text, source, session, att)
   }
 
   /**
@@ -3761,7 +3739,7 @@ export class ClaudeStreamBridge extends EventEmitter {
    * @param {string} cwd 呼び出し元がトークン検証した cwd (この cwd のセッションにのみ注入)
    * @param {string} sessionId 宛先 session_id (describeLiveSessions の session_id)
    * @param {string} text
-   * @returns {{status: "delivered", session: object} | {status: "not_found"} | {status: "ambiguous", candidates: object[]}}
+   * @returns {object | Promise<object>} 配送先なし/曖昧は同期、配送先確定後は受理結果を待つ。
    */
   sendToStream(cwd, sessionId, text, source) {
     if (!cwd || !sessionId || !text) return { status: "not_found" }
@@ -3780,28 +3758,53 @@ export class ClaudeStreamBridge extends EventEmitter {
       }
     }
     const { session, attached } = matches[0]
-    // 連携カード種別 (Phase 2): 宛先確定後に記録 (not_found/ambiguous では記録しない)。
+    return this._sendSiblingMessage(cwd, text, source, session, attached)
+  }
+
+  /** 受理結果だけを有限時間待つ。遅れて確定した結果も source の後始末には使う。 */
+  async _sendSiblingMessage(cwd, text, source, session, attached) {
+    const target = this._describeCandidate(session, attached)
+    const delivered = { status: "delivered", session: target }
     const injectHandle = session._recordInjectSource?.(text, source)
-    session
-      .sendMessage({ role: "user", content: text })
-      .then((r) => {
-        // Phase 2 (major#3 / persist major#1): {accepted:false} は echo/started 来ないので live を
-        // 掃除し永続化もしない (失敗を永続化すると 30 日 stale で後続同文 human を誤カード化)。
-        if (r && r.accepted === false) {
+    let timer
+    const outcome = (async () => {
+      try {
+        const result = await session.sendMessage({ role: "user", content: text })
+        if (result?.accepted === false) {
           session._removeInjectSource?.(injectHandle)
-          return
+          return { status: "delivery_failed", session: target, reason: result.reason }
         }
-        // 受理時のみ hydrate 用に永続化 (best-effort)。
+        // 受理時のみ永続化する。timeout 後に受理が確定した場合も維持する。
         recordInjectSourcePersistent(cwd, text, source).catch(() => {})
-      })
-      .catch((err) => {
+        if (result?.queued === true) {
+          return { status: "delivery_queued", session: target, reason: "queued_behind_active_turn" }
+        }
+        return delivered
+      } catch (err) {
         session._removeInjectSource?.(injectHandle)
         this.logger?.error(
-          { session_id: sessionId, err: err?.message },
-          "sendToStream: sendMessage threw unexpectedly",
+          { cwd, session_id: session.sessionId, err: err?.message },
+          "sibling sendMessage threw unexpectedly; delivery remains unconfirmed",
         )
-      })
-    return { status: "delivered", session: this._describeCandidate(session, attached) }
+        return delivered
+      }
+    })()
+    try {
+      return await Promise.race([
+        outcome,
+        new Promise((resolve) => {
+          timer = setTimeout(() => {
+            this.logger?.warn(
+              { cwd, session_id: session.sessionId, timeout_ms: SIBLING_ACCEPT_WAIT_MS },
+              "sibling sendMessage acceptance timed out; delivery remains unconfirmed",
+            )
+            resolve(delivered)
+          }, SIBLING_ACCEPT_WAIT_MS)
+        }),
+      ])
+    } finally {
+      clearTimeout(timer)
+    }
   }
 
   /**