@cocorograph/hub-agent 0.7.102 → 0.7.103

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -530,15 +530,16 @@ export class ClaudeStreamSession {
   /** マスク済みイベントを onEvent へ流す共通経路。秘匿情報 / 非公開スキルプロンプト本文を
    *  表示・WS 配信前に伏せる。マスク失敗時は素のイベントを流す (表示が止まるより素通し優先)。 */
   /** 連携カード種別 (Phase 2): 注入本文と source を FIFO に控える。sendToCwd/sendToStream が
-   *  session.sendMessage() を呼ぶ直前に記録し、直後の user echo を _tagInjectSource が拾う。
+   *  session.sendMessage() を呼ぶ直前に記録し、**drain 昇格時に _emitQueueState が started 本文
+   *  一致で消費**して startedSources として frontend へ載せる。
    *
-   *  ⚠️ 相関は「注入本文の完全一致」で行う (cross-review round1 major#1)。ID をメタデータで
-   *  伝播する方が堅牢だが、**Claude Agent SDK の user echo には隠しメタを載せる経路が無い**
-   *  (echo は本文そのもの)。cockpit_orch.py が注入本文末尾に付ける返信 footer で本文の一意性が
-   *  高く、人間が同一本文を打つ衝突は実質起きない。残る衝突リスクは①送信失敗時の stale entry
-   *  → _removeInjectSource で掃除、②未 echo の無限成長 → 32 件上限、で最小化する。衝突時も
-   *  誤タグ (青バブルが稀にカード化) に留まりクラッシュしない。
-   *  @param {string} text 注入本文 (SDK が echo する user メッセージ本文と一致する)
+   *  ⚠️ claude は SDK が user メッセージを live echo しない (reducer set_queue コメント参照)。
+   *  user バブルは queue_state の started 昇格で作られるため、当初の _emit(user echo) タグ方式は
+   *  claude で発火せず注入が青バブルのままだった (2026-07-23 実機で判明)。消費点を _emitQueueState
+   *  に移した。相関は「注入本文の完全一致」(footer 込みで一意性が高い)。ID メタ伝播が堅牢だが
+   *  started は本文しか持たないため本文相関を採る。衝突リスクは①送信失敗時の stale → _removeInjectSource
+   *  で handle 同定除去、②未 drain の無限成長 → 32 件上限、で最小化。衝突時も誤タグ止まりで安全。
+   *  @param {string} text 注入本文 (drain 時の started 本文と一致する)
    *  @param {'peer'|'orchestrator'|undefined} source
    *  @returns {object|null} この entry の固有 handle (identity)。配送失敗時に _removeInjectSource へ
    *    渡して**その entry だけ**を除去する。無効な source/text では null。 */
@@ -563,30 +564,6 @@ export class ClaudeStreamSession {
     if (idx !== -1) this._pendingInjectSources.splice(idx, 1)
   }
 
-  /** user echo イベントの本文が控えた注入本文と一致すれば source タグを付けて返す (それ以外は
-   *  無変更)。frontend はこの source があれば専用の連携依頼カードへ分岐する。共有/at-rest の
-   *  イベントを壊さないよう shallow-clone に付ける (top-level に source を足すだけ)。 */
-  _tagInjectSource(event) {
-    if (!this._pendingInjectSources.length) return event
-    if (!event || event.type !== "user" || !event.message) return event
-    const content = event.message.content
-    const text =
-      typeof content === "string"
-        ? content
-        : Array.isArray(content)
-          ? content
-              .filter((b) => b && b.type === "text" && typeof b.text === "string")
-              .map((b) => b.text)
-              .join("\n")
-          : ""
-    if (!text) return event
-    const idx = this._pendingInjectSources.findIndex((p) => p.text === text)
-    if (idx === -1) return event
-    const { source } = this._pendingInjectSources[idx]
-    this._pendingInjectSources.splice(idx, 1)
-    return { ...event, source }
-  }
-
   _emit(event) {
     if (!this.onEvent) return
     let out = event
@@ -601,7 +578,6 @@ export class ClaudeStreamSession {
         out = event
       }
     }
-    out = this._tagInjectSource(out)
     this.onEvent(out)
   }
 
@@ -1026,6 +1002,9 @@ export class ClaudeStreamSession {
       // 初回ターンの ultracode は起動時 options.settings に焼き込むため、起動前 reconcile
       // (applyFlagSettings) のための空窓も不要。
       if (!this._residentQuery || this._residentQueryReturned) {
+        // Phase 2 (major#1): resident fresh start は started を出さないため、注入なら source 付きで
+        // 昇格バブルを作るよう started を先出しする (通常メッセージは no-op)。
+        this._maybeEmitInjectStarted(prompt)
         this._startResidentTurnWithInput(toSDKUserMessage(prompt), ultracode)
         return { accepted: true }
       }
@@ -1038,6 +1017,11 @@ export class ClaudeStreamSession {
         ultracode,
       )
       if (outcome === "closed") return { accepted: false, reason: "queue_dead" }
+      // Phase 2 (round2 major): 起動済み・idle な resident query への delivery は
+      // _reconcileThenDeliver 内で pushed (inputQueue.push) / restarted
+      // (_startResidentTurnWithInput) のどちらも started を出さない。注入なら source 付き
+      // started をここで先出しして昇格バブルをカード化する (通常メッセージは no-op)。
+      this._maybeEmitInjectStarted(prompt)
       return { accepted: true }
     }
 
@@ -1058,6 +1042,9 @@ export class ClaudeStreamSession {
       this._emitQueueState()
       return { accepted: true }
     }
+    // Phase 2 (major#1): per-message idle 直接実行も started を出さないため、注入なら source 付きで
+    // 昇格バブルを作るよう started を先出しする (通常メッセージは no-op)。
+    this._maybeEmitInjectStarted(prompt)
     // ack はターン完走を待たず「起動できたか」で返すため、フル実行の Promise を
     // 呼び出し元へ伝播させない (fire-and-forget)。例外は _runPerMessage 内部で
     // 握っているため通常は reject しないが、念のため warn だけ残す。
@@ -1304,6 +1291,20 @@ export class ClaudeStreamSession {
     // force のときは再アタッチ snapshot なので重複チェックを完全にバイパスする。
     if (!force && started.length === 0 && sig === this._lastEmittedQueueSig) return
     this._lastEmittedQueueSig = sig
+    // 連携カード種別 (Phase 2, 2026-07-23 修正): **claude は SDK が user メッセージを live echo
+    // しない** (reducer set_queue のコメント参照)。ゆえに user バブルは queue_state の started
+    // (drain 昇格) から作られる。当初 _emit の user echo にタグする実装は claude では発火せず
+    // (dead code)、注入が青バブルのままだった (実機確認)。正しくは started 各テキストの source を
+    // ここで _pendingInjectSources から引いて startedSources として載せ、frontend が昇格バブルへ
+    // source を付ける。started は全 drain 経路 (_drainPending / resident / per-message finally) が
+    // 通る funnel なのでここ 1 箇所で全経路をカバーする。注入本文 (footer 込み) の完全一致で消費。
+    const startedSources = started.map((text) => {
+      const idx = this._pendingInjectSources.findIndex((p) => p.text === text)
+      if (idx === -1) return null
+      const { source } = this._pendingInjectSources[idx]
+      this._pendingInjectSources.splice(idx, 1)
+      return source
+    })
     try {
       // messages は全文を載せる。frontend は実行開始 (drain) 時にこれを user バブルへ
       // 昇格させるため、ここで切り詰めると本文が欠ける。チップの省略表示は CSS 側で行う。
@@ -1314,12 +1315,25 @@ export class ClaudeStreamSession {
         messages: this._pendingMessages.map((m) => m.text),
         items: this._pendingMessages.map((m) => ({ id: m.id, text: m.text })),
         started,
+        // started と parallel。連携注入なら 'peer'|'orchestrator'、通常は null。
+        startedSources,
       })
     } catch {
       /* ignore */
     }
   }
 
+  /** 連携カード種別 (Phase 2, cross-review major#1): **idle 直接送信経路**用。busy→queue→drain は
+   *  _emitQueueState([text]) を通るが、idle 直接実行 (resident fresh start / per-message 即時) は
+   *  started を出さないため、注入の source が startedSources に載らずカード化されない。ここで
+   *  「その本文が注入 (source 記録済み) のときだけ」started を 1 件 emit して昇格バブルを source 付き
+   *  で作る。通常メッセージ (source 無し) は emit せず従来経路 (optimistic 等) に任せる = 挙動不変。 */
+  _maybeEmitInjectStarted(prompt) {
+    if (this._pendingInjectSources.some((p) => p.text === prompt)) {
+      this._emitQueueState([prompt])
+    }
+  }
+
   /** 空 InputQueue 窓封鎖 (T06118): fresh start する resume 常駐 query を「入力を先に積んで
    *  から起動」する (push 先行)。fresh な InputQueue を作り、実ユーザーメッセージを push して
    *  から query を起動するため、「resume + 空入力」の窓で SDK が自動継続ターン (Continue from
@@ -2697,14 +2711,22 @@ export class ClaudeStreamBridge extends EventEmitter {
     // 連携カード種別 (Phase 2): 配送先が確定した後に記録する (ambiguous/not_found では記録しない)。
     // handle を控え、失敗時にその entry だけを除去する (round2: text 除去は並行同一本文で誤消費)。
     const injectHandle = session._recordInjectSource?.(text, source)
-    session.sendMessage({ role: "user", content: text }).catch((err) => {
-      // Phase 2: 配送失敗なら echo は来ないので stale な source entry を掃除する (誤タグ防止)。
-      session._removeInjectSource?.(injectHandle)
-      this.logger?.error(
-        { cwd, err: err?.message },
-        "sendToCwd: sendMessage threw unexpectedly",
-      )
-    })
+    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 source entry を掃除する (誤タグ防止)。
+        if (r && r.accepted === false) session._removeInjectSource?.(injectHandle)
+      })
+      .catch((err) => {
+        // 配送失敗 (throw) でも echo は来ないので stale な source entry を掃除する。
+        session._removeInjectSource?.(injectHandle)
+        this.logger?.error(
+          { cwd, err: err?.message },
+          "sendToCwd: sendMessage threw unexpectedly",
+        )
+      })
     return { status: "delivered", session: this._describeCandidate(session, att) }
   }
 
@@ -2750,14 +2772,20 @@ export class ClaudeStreamBridge extends EventEmitter {
     const { session, attached } = matches[0]
     // 連携カード種別 (Phase 2): 宛先確定後に記録 (not_found/ambiguous では記録しない)。
     const injectHandle = session._recordInjectSource?.(text, source)
-    session.sendMessage({ role: "user", content: text }).catch((err) => {
-      // Phase 2: 配送失敗なら echo は来ないので stale な source entry を掃除する (誤タグ防止)。
-      session._removeInjectSource?.(injectHandle)
-      this.logger?.error(
-        { session_id: sessionId, err: err?.message },
-        "sendToStream: sendMessage threw unexpectedly",
-      )
-    })
+    session
+      .sendMessage({ role: "user", content: text })
+      .then((r) => {
+        // Phase 2 (major#3): {accepted:false} (queue_dead / empty_message / pre-canceled) も
+        // echo/started は来ないので stale source entry を掃除する (誤タグ防止)。
+        if (r && r.accepted === false) session._removeInjectSource?.(injectHandle)
+      })
+      .catch((err) => {
+        session._removeInjectSource?.(injectHandle)
+        this.logger?.error(
+          { session_id: sessionId, err: err?.message },
+          "sendToStream: sendMessage threw unexpectedly",
+        )
+      })
     return { status: "delivered", session: this._describeCandidate(session, attached) }
   }