@cocorograph/hub-agent 0.7.100 → 0.7.102

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -314,7 +314,9 @@ const STALE_IDLE_GUARD_MS = 5000
 /**
  * 1 stream に対応する Claude セッション (per-message query モデル)。
  */
-class ClaudeStreamSession {
+// export: 単体テストから直接インスタンス化して純ロジック (連携カード source タグの
+// record/remove/tag 等) を検証するため。実運用の生成は ClaudeStreamBridge が行う。
+export class ClaudeStreamSession {
   constructor({
     stream_id,
     cwd,
@@ -431,6 +433,14 @@ class ClaudeStreamSession {
     /** 改修3: 直近 browser へ通知した queue 署名 (件数 + id 列)。変化時のみ emit する。
      *  空キューの署名 ("0:") で初期化し、空→空の冗長 emit を抑止する。 */
     this._lastEmittedQueueSig = "0:"
+
+    /** 連携カード種別タグ (Phase 2, 2026-07-23): ask-sibling/orchestrate 等の **別セッションからの
+     *  注入** を、受け手画面で「青い自分バブル」ではなく専用の連携依頼カードで表示させるため、
+     *  注入本文 (text) を相関キーに source ('peer'|'orchestrator') を stash する FIFO。
+     *  sendToCwd/sendToStream が注入直前に記録し、直後に SDK が返す user echo (type:"user") の
+     *  本文一致で _emit が消費してタグ付けする。人間の実入力や tool_result は本 set に無いので
+     *  タグされない (誤タグ防止)。echo が来ない稀ケースの無限成長は上限で古い順に切る。 */
+    this._pendingInjectSources = []
     /** ultracode (0.6.28): 常駐 query へ現在適用済みの ultracode 状態。ターン単位の
      *  ワンショット適用を applyFlagSettings で reconcile する際の差分判定に使う。
      *  query を (再)起動すると flag settings は既定に戻るため、_runResidentQuery 冒頭で
@@ -519,6 +529,64 @@ class ClaudeStreamSession {
 
   /** マスク済みイベントを onEvent へ流す共通経路。秘匿情報 / 非公開スキルプロンプト本文を
    *  表示・WS 配信前に伏せる。マスク失敗時は素のイベントを流す (表示が止まるより素通し優先)。 */
+  /** 連携カード種別 (Phase 2): 注入本文と source を FIFO に控える。sendToCwd/sendToStream が
+   *  session.sendMessage() を呼ぶ直前に記録し、直後の user echo を _tagInjectSource が拾う。
+   *
+   *  ⚠️ 相関は「注入本文の完全一致」で行う (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 メッセージ本文と一致する)
+   *  @param {'peer'|'orchestrator'|undefined} source
+   *  @returns {object|null} この entry の固有 handle (identity)。配送失敗時に _removeInjectSource へ
+   *    渡して**その entry だけ**を除去する。無効な source/text では null。 */
+  _recordInjectSource(text, source) {
+    if ((source !== "peer" && source !== "orchestrator") || !text) return null
+    const entry = { text, source }
+    this._pendingInjectSources.push(entry)
+    // echo が来ない稀ケース (注入直後にセッション消失等) の無限成長を防ぐ。連携注入は低頻度
+    // なので 32 件で十分 (古い順に切る)。
+    if (this._pendingInjectSources.length > 32) this._pendingInjectSources.shift()
+    return entry
+  }
+
+  /** 連携カード種別 (Phase 2, cross-review round1 major#1 / round2 major): 注入 (sendMessage) が
+   *  配送失敗したときに、record が返した **その entry だけ**を identity (indexOf) で取り消す。
+   *  ⚠️ text で消してはいけない (round2 指摘): 同一本文を並行注入し後発だけ失敗すると、text 先頭
+   *  一致は先発の成功 entry を誤って消し、先発 echo が後発 source で誤タグされる。handle 同定なら
+   *  失敗した当該 entry のみ除去される。既に消費/cap 済みなら indexOf=-1 で no-op。 */
+  _removeInjectSource(handle) {
+    if (!handle) return
+    const idx = this._pendingInjectSources.indexOf(handle)
+    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
@@ -533,6 +601,7 @@ class ClaudeStreamSession {
         out = event
       }
     }
+    out = this._tagInjectSource(out)
     this.onEvent(out)
   }
 
@@ -2610,7 +2679,7 @@ export class ClaudeStreamBridge extends EventEmitter {
    * @param {string} text
    * @returns {{status:"delivered", session:object} | {status:"not_found"} | {status:"ambiguous", candidates:object[]}}
    */
-  sendToCwd(cwd, text) {
+  sendToCwd(cwd, text, source) {
     if (!cwd || !text) return { status: "not_found" }
     // sessionId 未確定の fresh も配送対象にするため requireSessionId=false。
     const cands = this._liveCandidatesForCwd(cwd, false)
@@ -2625,7 +2694,12 @@ 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 }).catch((err) => {
+      // Phase 2: 配送失敗なら echo は来ないので stale な source entry を掃除する (誤タグ防止)。
+      session._removeInjectSource?.(injectHandle)
       this.logger?.error(
         { cwd, err: err?.message },
         "sendToCwd: sendMessage threw unexpectedly",
@@ -2657,7 +2731,7 @@ export class ClaudeStreamBridge extends EventEmitter {
    * @param {string} text
    * @returns {{status: "delivered", session: object} | {status: "not_found"} | {status: "ambiguous", candidates: object[]}}
    */
-  sendToStream(cwd, sessionId, text) {
+  sendToStream(cwd, sessionId, text, source) {
     if (!cwd || !sessionId || !text) return { status: "not_found" }
     // union 走査 (2026-07-23, 統一再設計 1-1): warm (soft-detached) セッションも session_id
     // ピンの対象にする。従来は this.sessions のみで、detach 済みだが生きているセッションを
@@ -2674,7 +2748,11 @@ 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",

package/src/codex-appserver-client.mjs

@@ -65,8 +65,40 @@ export class CodexAppServerClient extends EventEmitter {
     // サブコマンド (`app-server`) より前に置く必要がある（`codex app-server --help`
     // にはこのフラグが出ない = サブコマンド固有オプションではないため）。
     const bypassHookTrust = process.env.HUB_CODEX_BYPASS_HOOK_TRUST !== "0"
+    // sibling 連携 env 注入 (2026-07-23): app-server プロセス env には siblingEnvFor が
+    // COCKPIT_SIBLING_TOKEN/CWD/SELF_KIND を載せている (codex-stream-bridge.mjs)。だが codex の
+    // **code-mode exec (tools.exec_command)** がモデル生成コマンドを実行する際、これらは
+    // シェルに渡らない (2026-07-23 実測: rollout で `env|grep COCKPIT_SIBLING` が空・SELF=[])。
+    // classic `codex exec` は inherit=core でも通すが、app-server チャットが使う code-mode 経路は
+    // 継承ベースが異なり COCKPIT_* を落とす。結果 codex-as-reviewer の返信 `cockpit_orch.py send`
+    // が「SELF_KIND 環境変数が無い」で fail-closed になり cross-review の返信が不達になっていた。
+    //
+    // 修正: `shell_environment_policy.set.<VAR>=<値>` で 3 変数を **無条件注入** する
+    // (inherit/exclude フィルタや code-mode の env 剥がしに関係なく command env に入る。
+    //  2026-07-23 実測で **code-mode exec (custom_tool_call/exec_command) 経路でも** 注入値が
+    //  コマンドシェルに出現することを確認済み = 実際に壊れていた経路で有効)。値は this.env (= siblingEnvFor が
+    // 生成した本セッションの実値) から取るので静的化されず正しい per-session token/cwd になる。
+    // inherit=all を選ばない理由: code-mode に効く保証が無い (core 継承下で全消しを実測) ため。
+    // token が argv (ps) に載るが、既に app-server/code-mode-host の env に同値があり同一ユーザー
+    // 可視・cwd スコープ・短命なので露出クラスは増えない。HUB_CODEX_SIBLING_ENV_PASSTHROUGH=0 で opt-out。
+    const siblingEnvPassthrough = process.env.HUB_CODEX_SIBLING_ENV_PASSTHROUGH !== "0"
+    const spawnEnv = this.env || process.env
+    const siblingSetArgs = []
+    if (siblingEnvPassthrough) {
+      for (const key of [
+        "COCKPIT_SIBLING_SELF_KIND",
+        "COCKPIT_SIBLING_CWD",
+        "COCKPIT_SIBLING_TOKEN",
+      ]) {
+        const val = spawnEnv[key]
+        // TOML 文字列としてクォート (パスの空白/特殊文字対策)。値に " は含まれない
+        // (hex token / パス / "codex")。1 つの argv 要素として渡すのでシェル再解釈は無い。
+        if (val) siblingSetArgs.push("-c", `shell_environment_policy.set.${key}="${val}"`)
+      }
+    }
     const args = [
       ...(bypassHookTrust ? ["--dangerously-bypass-hook-trust"] : []),
+      ...siblingSetArgs,
       "app-server",
       "--stdio",
     ]

package/src/codex-stream-bridge.mjs

@@ -497,6 +497,12 @@ export class CodexStreamBridge extends EventEmitter {
       // major#2: ライブ追記でも cold flatten と同じ「item 数＋概算 byte」二重上限を課すため、
       // 蓄積済み履歴の概算 byte 合計を保持する (再計算を避け O(1) で trim 判定)。
       historyBytes: 0,
+      // 連携カード種別タグ (Phase 2, 2026-07-23): ask-sibling/orchestrate 等の別セッションからの
+      // 注入本文 (text) → source ('peer'|'orchestrator') を控える FIFO。sendToCwd/sendToStream が
+      // 注入直前に積み、直後に app-server が返す userMessage の item/completed で本文一致により
+      // _onNotification が消費して params.item.source を付ける (frontend が専用カードへ分岐)。
+      /** @type {Array<{text: string, source: 'peer'|'orchestrator'}>} */
+      pendingInjectSources: [],
     }
     session.ready = new Promise((resolve) => {
       settleReady = resolve
@@ -849,6 +855,62 @@ export class CodexStreamBridge extends EventEmitter {
     return Array.from(new Set(this.sessions.values())).filter((s) => !s.dead)
   }
 
+  /** 連携カード種別 (Phase 2): 注入本文 → source を session の FIFO に控える。sendToCwd/
+   *  sendToStream が input() を呼ぶ直前に記録し、直後の userMessage echo を
+   *  _tagInjectSourceOnItem が拾う。
+   *  @param {object} session
+   *  @param {string} text
+   *  @param {'peer'|'orchestrator'|undefined} source */
+  _recordInjectSource(session, text, source) {
+    if (!session || (source !== "peer" && source !== "orchestrator") || !text) return null
+    const list = session.pendingInjectSources || (session.pendingInjectSources = [])
+    const entry = { text, source }
+    list.push(entry)
+    // echo が来ない稀ケースの無限成長を防ぐ (連携注入は低頻度、32 件で十分)。
+    if (list.length > 32) list.shift()
+    return entry
+  }
+
+  /** 連携カード種別 (Phase 2, cross-review round1 major#1 / round2 major): 注入 (input) が配送失敗
+   *  したときに、record が返した **その entry だけ**を identity (indexOf) で取り消す。
+   *  ⚠️ text で消してはいけない (round2 指摘): 同一本文を並行注入し後発だけ失敗すると、text 先頭
+   *  一致は先発の成功 entry を誤って消し、先発 echo が後発 source で誤タグされる。handle 同定なら
+   *  失敗した当該 entry のみ除去される。既に消費/cap 済みなら indexOf=-1 で no-op。 */
+  _removeInjectSource(session, handle) {
+    const list = session?.pendingInjectSources
+    if (!list || !handle) return
+    const idx = list.indexOf(handle)
+    if (idx !== -1) list.splice(idx, 1)
+  }
+
+  /** userMessage の echo (item/completed) が控えた注入本文と一致すれば、params を **クローン**して
+   *  item.source を付けたものを返す (元の app-server オブジェクトは変更しない)。一致しなければ null。
+   *  frontend の userMessageItemToEvent がこの item.source を読み専用カードへ分岐する。 */
+  _tagInjectSourceOnItem(session, msg) {
+    const list = session?.pendingInjectSources
+    if (!list || !list.length) return null
+    if (msg?.method !== "item/completed") return null
+    const item = msg.params?.item
+    if (!item || item.type !== "userMessage") return null
+    const content = item.content
+    const text =
+      typeof content === "string"
+        ? content
+        : Array.isArray(content)
+          ? content
+              .map((b) =>
+                b && typeof b === "object" && typeof b.text === "string" ? b.text : "",
+              )
+              .join("")
+          : ""
+    if (!text) return null
+    const idx = list.findIndex((p) => p.text === text)
+    if (idx === -1) return null
+    const { source } = list[idx]
+    list.splice(idx, 1)
+    return { ...msg.params, item: { ...item, source } }
+  }
+
   /** 案B: この session を指す全購読 stream_id キーを sessions Map から外す（reap / exit 共通）。 */
   _dropAllStreamKeys(session) {
     const ids = session.streamIds ? Array.from(session.streamIds) : []
@@ -865,6 +927,15 @@ export class CodexStreamBridge extends EventEmitter {
     // 正常に走っているターンは reaper に回収されず、通知が途絶えたスタックターンだけが
     // 放棄セッションとして回収される。
     this._touch(session)
+    // 連携カード種別 (Phase 2, cross-review round1 major#2): タグ判定を switch より前に行い、
+    // **history にもタグ済み item を積む**。旧実装は未タグの msg.params.item を history へ入れてから
+    // ライブ emit だけタグしていたため、cwd reattach で historyItems が返ると連携カードが通常の青い
+    // 自分バブルへ戻っていた。同一プロセス内 reattach で表示が維持されるよう、以降 effectiveParams
+    // (タグ済み or 素) を history と emit の両方で使う。一致しなければ null → 素の params のまま。
+    // ※ hub-agent 再起動 (cold resume) では historyItems が揮発するため青バブルへ戻る (claude の
+    //    jsonl at-rest と同じ許容範囲。source 永続マップは TTL/ID 再利用の複雑さに見合わないため不採用)。
+    const taggedParams = this._tagInjectSourceOnItem(session, msg)
+    const effectiveParams = taggedParams || msg.params
     switch (msg.method) {
       case "turn/started":
         session.activeTurnId = msg.params?.turn?.id || null
@@ -887,8 +958,8 @@ export class CodexStreamBridge extends EventEmitter {
         break
       case "item/completed":
         // T06541 案1: 完了した ThreadItem を高忠実度履歴に追記する（cwd reattach で返す用）。
-        // ライブ表示は下の emit がそのまま担う（本 case は履歴蓄積を足すだけで emit は変えない）。
-        this._appendHistoryItem(session, msg.params?.item, msg.params?.completedAtMs)
+        // Phase 2: タグ済み item (effectiveParams.item) を積む = reattach でも source が残る。
+        this._appendHistoryItem(session, effectiveParams?.item, effectiveParams?.completedAtMs)
         break
       default:
         break
@@ -902,7 +973,7 @@ export class CodexStreamBridge extends EventEmitter {
       producer_seq: session.seq,
       thread_id: session.threadId,
       cwd: session.cwd,
-      notification: { method: msg.method, params: msg.params },
+      notification: { method: msg.method, params: effectiveParams },
     })
   }
 
@@ -1416,7 +1487,7 @@ export class CodexStreamBridge extends EventEmitter {
    * @param {string} text
    * @returns {{status:"delivered", session:object} | {status:"not_found"}}
    */
-  sendToCwd(cwd, text) {
+  sendToCwd(cwd, text, source) {
     if (!cwd || !text) return { status: "not_found" }
     // 案B: distinct session で走査（多対一 Map の重複を除く）。
     const cands = this._liveSessions().filter((s) => s.cwd === cwd)
@@ -1449,6 +1520,8 @@ export class CodexStreamBridge extends EventEmitter {
     // 明示しておく。ターン実行中は healthy busy = protected なので reconcile で kill されない。
     best.detachRequested = false
     this._touch(best)
+    // 連携カード種別 (Phase 2): 宛先確定後に記録 (not_found では記録しない)。handle を控え失敗時に除去。
+    const injectHandle = this._recordInjectSource(best, text, source)
     this.logger?.info?.(
       {
         cwd,
@@ -1461,6 +1534,8 @@ export class CodexStreamBridge extends EventEmitter {
     )
     this.input({ stream_id: best.stream_id, message: { role: "user", content: text } }).catch(
       (err) => {
+        // Phase 2: 配送失敗なら echo は来ないので stale な source entry を取り消す (誤タグ防止)。
+        this._removeInjectSource(best, injectHandle)
         this.logger?.error(
           { cwd, stream_id: best.stream_id, err: err?.message },
           "sendToCwd: input threw unexpectedly",
@@ -1498,7 +1573,7 @@ export class CodexStreamBridge extends EventEmitter {
    * @param {string} text
    * @returns {{status: "delivered", session: object} | {status: "not_found"} | {status: "ambiguous", candidates: object[]}}
    */
-  sendToStream(cwd, id, text) {
+  sendToStream(cwd, id, text, source) {
     if (!cwd || !id || !text) return { status: "not_found" }
     let bySid = null
     const byThread = []
@@ -1524,8 +1599,12 @@ export class CodexStreamBridge extends EventEmitter {
     // (sendToCwd と同じ扱い)。
     target.detachRequested = false
     this._touch(target)
+    // 連携カード種別 (Phase 2): 宛先確定後に記録 (not_found/ambiguous では記録しない)。handle を控える。
+    const injectHandle = this._recordInjectSource(target, text, source)
     this.input({ stream_id: target.stream_id, message: { role: "user", content: text } }).catch(
       (err) => {
+        // Phase 2: 配送失敗なら echo は来ないので stale な source entry を取り消す (誤タグ防止)。
+        this._removeInjectSource(target, injectHandle)
         this.logger?.error(
           { id, stream_id: target.stream_id, err: err?.message },
           "sendToStream: input threw unexpectedly",

package/src/sibling-inject.mjs

@@ -123,7 +123,11 @@ export class SiblingInjectBridge {
     //    (T06271 レビュー major①): 旧 hub-agent は `!text` で invalid_request を返すため、
     //    未知フィールドを無視して sendToCwd 推測注入する事故が **単一リクエストで原理的に
     //    起きない** (probe→注入の TOCTOU を排除)。
-    const { op, cwd, target_cli_kind, text, message, token, target_session_id } = body || {}
+    const { op, cwd, target_cli_kind, text, message, token, target_session_id, source } = body || {}
+    // 連携カード種別 (Phase 2, 2026-07-23): 注入を受け手画面で専用カード表示させるためのタグ。
+    // 既知値 ('peer'|'orchestrator') のみ通し、未知値/未指定は undefined に落とす (未知の source を
+    // frontend へ流して分岐を汚さない fail-closed)。bridge が user echo にこれを付ける。
+    const injectSource = source === "peer" || source === "orchestrator" ? source : undefined
     const isListAll = op === "list-all"
     const isList = op === "list"
     const isExplicit = op === "inject"
@@ -182,7 +186,9 @@ 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) ?? { status: "not_found" }
+        const r = bridge?.sendToStream?.(cwd, target_session_id, message, injectSource) ?? {
+          status: "not_found",
+        }
         if (r.status === "delivered") {
           result = { ok: true, delivered_to: r.session }
         } else if (r.status === "ambiguous") {
@@ -229,7 +235,7 @@ export class SiblingInjectBridge {
         //    (サイレント誤配送 + delivered_to に実配送先でない sameCwd[0] を報告する成功偽装) が
         //    あった。sendToCwd を status 返却にして解決と配送を 1 呼び出しに統合し、delivered_to は
         //    **実配送先 (r.session)** を返す。
-        const r = bridge?.sendToCwd?.(cwd, text) ?? { status: "not_found" }
+        const r = bridge?.sendToCwd?.(cwd, text, injectSource) ?? { status: "not_found" }
         if (r.status === "delivered") {
           result = { ok: true, delivered_to: r.session ?? null }
         } else if (r.status === "ambiguous") {