@cocorograph/hub-agent 0.8.34 → 0.8.35

package/package.json

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

package/src/codex-stream-bridge.mjs

@@ -748,8 +748,8 @@ export class CodexStreamBridge extends EventEmitter {
        *  deliveryId: string, sourceStreamId: string, deliveryPending: boolean,
        *  deliveryPromise: Promise<object>, settleDelivery: Function}>} */
       pendingInjectSources: [],
-      /** 組み込み collaboration subagent の表示用台帳。ライフサイクルの安定キーは
-       * agentThreadId（item.id は dynamicToolCall とのメタデータ結合にだけ使う）。 */
+      /** 組み込み collaboration subagent の表示用台帳。検索キーは agentThreadId、
+       * 外部へ出す安定した委譲 ID は spawn dynamicToolCall の item.id を保持する。 */
       subagents: new Map(),
       /** dynamicToolCall(spawn_agent) の item.id -> 表示メタデータ。 */
       subagentSpawnMetadata: new Map(),
@@ -1460,6 +1460,12 @@ export class CodexStreamBridge extends EventEmitter {
     const list = session?.pendingInjectSources
     if (!list || !list.length) return null
     if (msg?.method !== "item/completed") return null
+    const sourceThreadId = msg.params?.threadId
+    if (
+      typeof sourceThreadId === "string" &&
+      sourceThreadId &&
+      sourceThreadId !== session?.threadId
+    ) return null
     const item = msg.params?.item
     if (!item || item.type !== "userMessage") return null
     const content = item.content
@@ -1582,13 +1588,14 @@ export class CodexStreamBridge extends EventEmitter {
       notification: {
         method: "cockpit/subagent_state",
         params: {
-          delegationId: record.agentThreadId,
+          delegationId: record.delegationId,
           agentThreadId: record.agentThreadId,
           agentPath: record.agentPath,
           phase: record.phase,
           agent: record.agent,
           description: record.description,
           prompt: record.prompt,
+          background: record.background === true,
           atMs: record.atMs,
         },
       },
@@ -1609,6 +1616,14 @@ export class CodexStreamBridge extends EventEmitter {
         ? details.agentPath
         : previous?.agentPath || ""
     const metadata = details.metadata || null
+    const delegationId =
+      previous?.delegationId ||
+      (typeof details.delegationId === "string" && details.delegationId
+        ? details.delegationId
+        : null)
+    // raw child thread ID を委譲 ID に代用しない。spawn lifecycle 由来の安定 ID が無い
+    // 通知は台帳へ登録せず、後続 item も unknown として未タグのまま扱う。
+    if (typeof delegationId !== "string" || !delegationId) return null
     const transitionAtMs =
       previous?.phase === phase
         ? previous.atMs
@@ -1616,6 +1631,7 @@ export class CodexStreamBridge extends EventEmitter {
           ? details.atMs
           : Date.now()
     const record = {
+      delegationId,
       agentThreadId,
       agentPath,
       phase,
@@ -1638,8 +1654,20 @@ export class CodexStreamBridge extends EventEmitter {
     return record
   }
 
-  _observeSubagentItem(session, item, atMs = Date.now(), { emit = true } = {}) {
+  _observeSubagentItem(
+    session,
+    item,
+    atMs = Date.now(),
+    { emit = true, sourceThreadId = session?.threadId } = {},
+  ) {
     if (!item || typeof item !== "object") return
+    // root thread の collaboration item だけが root subagent 台帳の正本。子 thread 内の
+    // spawn/subAgentActivity（孫）を root の lifecycle/snapshot へ昇格しない。
+    if (
+      typeof sourceThreadId === "string" &&
+      sourceThreadId &&
+      sourceThreadId !== session?.threadId
+    ) return
     if (item.type === "dynamicToolCall") {
       const namespace = item.namespace
       if (namespace !== "collaboration" || item.tool !== "spawn_agent" || !item.id) return
@@ -1674,6 +1702,7 @@ export class CodexStreamBridge extends EventEmitter {
       agentThreadId,
       phase,
       {
+        delegationId: item.id,
         agentPath: item.agentPath,
         metadata,
         atMs,
@@ -1736,7 +1765,7 @@ export class CodexStreamBridge extends EventEmitter {
       const startedHistoryIndex = startedIndexByThread.get(record.agentThreadId)
       if (startedHistoryIndex === undefined) continue
       snapshots.push({
-        delegationId: record.agentThreadId,
+        delegationId: record.delegationId,
         agentThreadId: record.agentThreadId,
         agentPath: record.agentPath,
         agent: record.agent,
@@ -1880,25 +1909,46 @@ export class CodexStreamBridge extends EventEmitter {
     // ※ hub-agent 再起動 (cold resume) では historyItems が揮発するため青バブルへ戻る (claude の
     //    jsonl at-rest と同じ許容範囲。source 永続マップは TTL/ID 再利用の複雑さに見合わないため不採用)。
     const eventThreadId = msg?.params?.threadId
-    const backgroundChildItemNotification =
-      typeof eventThreadId === "string" &&
-      typeof msg?.method === "string" &&
-      msg.method.startsWith("item/") &&
-      session.subagents.get(eventThreadId)?.background === true
-    if (backgroundChildItemNotification) {
-      // 背景子の item は root の履歴・バブルへ混ぜない。子自身の lifecycle は
-      // turn 通知と root 側 subAgentActivity が正本なので、子 item から孫を登録しない。
-      // foreground / background 不明（spawn record 生成前の短い窓を含む）は fail-open で
-      // 既存どおり root へ通す。背景出力の過剰抑止より既存表示の維持を優先する。
+    const taggedParams = this._tagInjectSourceOnItem(session, msg)
+    let effectiveParams = taggedParams || msg.params
+    const childRecord =
+      typeof eventThreadId === "string" ? session.subagents.get(eventThreadId) : null
+    const childTextDelta =
+      msg.method === "item/agentMessage/delta" ||
+      msg.method === "item/reasoning/summaryTextDelta" ||
+      msg.method === "item/reasoning/textDelta"
+    if (childRecord && childTextDelta) {
+      // 子の message/reasoning 途中 delta を root liveText へ混ぜない。前景・背景とも
+      // attribution 付き item/completed が表示の正本で、root/unknown delta は従来どおり
+      // fail-open で通す。
       return
     }
-    const taggedParams = this._tagInjectSourceOnItem(session, msg)
-    const effectiveParams = taggedParams || msg.params
+    if (
+      childRecord &&
+      (msg.method === "item/started" || msg.method === "item/completed") &&
+      effectiveParams?.item &&
+      typeof effectiveParams.item === "object"
+    ) {
+      // app-server 所有の item は変更せず、既知の子 thread だけを attribution 付きで複製する。
+      // unknown/root を推測で子扱いしない。delegationId は child thread ID ではなく、spawn
+      // lifecycle が保持する安定 ID を使う。
+      const attributedItem = {
+        ...effectiveParams.item,
+        _hubSubagent: {
+          delegationId: childRecord.delegationId,
+          threadId: eventThreadId,
+          runtime: "codex",
+          background: childRecord.background === true,
+        },
+      }
+      effectiveParams = { ...effectiveParams, item: attributedItem }
+    }
     if (msg.method === "item/started" || msg.method === "item/completed") {
       this._observeSubagentItem(
         session,
         effectiveParams?.item,
         effectiveParams?.completedAtMs || Date.now(),
+        { sourceThreadId: eventThreadId },
       )
     }
     const childTurnNotification =
@@ -1968,8 +2018,8 @@ export class CodexStreamBridge extends EventEmitter {
         void this._pumpInputs(session)
         break
       case "item/completed":
-        // T06541 案1: 完了した ThreadItem を高忠実度履歴に追記する（cwd reattach で返す用）。
-        // Phase 2: タグ済み item (effectiveParams.item) を積む = reattach でも source が残る。
+        // 完了した ThreadItem を高忠実度履歴に追記する（cwd reattach で返す用）。
+        // attribution/source 済み item (effectiveParams.item) を積み、emit と同じ clone を使う。
         this._appendHistoryItem(session, effectiveParams?.item, effectiveParams?.completedAtMs)
         break
       default: