@cocorograph/hub-agent 0.8.99 → 0.8.101

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -20,7 +20,8 @@ import { recordConversationFocus } from "./chat-signals.mjs"
  */
 import { EventEmitter } from "node:events"
 import { randomUUID } from "node:crypto"
-import { readFileSync } from "node:fs"
+import { readFileSync, statSync } from "node:fs"
+import { open } from "node:fs/promises"
 import os from "node:os"
 import path from "node:path"
 import { isUnderRemoval } from "./chat-removal-guard.mjs"
@@ -323,11 +324,12 @@ export function _resetModelCapabilities() {
  *  SDKUserMessage は parent_tool_use_id: string|null が必須フィールド。現行 SDK は入力側で
  *  寛容なので省略しても動くが、将来の型厳格化に備えて明示する。トップレベルのユーザー入力
  *  なので親 tool_use は無く null。 */
-function toSDKUserMessage(text) {
+function toSDKUserMessage(text, uuid) {
   return {
     type: "user",
     message: { role: "user", content: text },
     parent_tool_use_id: null,
+    ...(uuid === undefined ? {} : { uuid }),
   }
 }
 
@@ -427,6 +429,14 @@ function isSessionRunningEvent(msg) {
  *  正しく終端として扱う (ガードが固着して _busy が永久 true になる = T06052 の再発を防ぐ安全弁)。 */
 const STALE_IDLE_GUARD_MS = 5000
 
+// sibling CLI の既定待ち 10 秒より短く受理状態を待つ。超過は失敗と断定しない。
+export const SIBLING_ACCEPT_WAIT_MS = 1000
+export const SIBLING_CONFIRM_WAIT_MS = 2500
+const SIBLING_CONFIRM_POLL_MS = 50
+// 注入本文の user 行はこれより十分小さい。改行の無い巨大行 (大きな tool result 等) を
+// 抱え込んで再コピー・再検索し続けないよう、超えた行は次の改行まで読み飛ばす。
+const SIBLING_CONFIRM_MAX_LINE_BYTES = 1024 * 1024
+
 /** per-message の title control request を待って InputQueue を閉じる上限。 */
 const SESSION_TITLE_CLOSE_WAIT_MS = 15_000
 
@@ -1408,7 +1418,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)
@@ -1425,7 +1435,7 @@ export class ClaudeStreamSession {
         // 昇格バブルを作るよう started を先出しする (通常メッセージは no-op)。
         this._maybeEmitInjectStarted(prompt)
         this._pendingTitleSeed = prompt
-        this._startResidentTurnWithInput(toSDKUserMessage(prompt), ultracode)
+        this._startResidentTurnWithInput(toSDKUserMessage(prompt, opts.message_uuid), ultracode)
         return { accepted: true }
       }
       // 起動済みの常駐 query へ次ターンを流す。init 済みで空窓が無いため、ultracode は走行中
@@ -1433,7 +1443,7 @@ export class ClaudeStreamSession {
       // 旧 query 終了 × 後着 pending の ABA/FIFO 逆転は _reconcileThenDeliver が直列化する
       // (drain 経路 _drainResidentPending と共通)。
       const outcome = await this._reconcileThenDeliver(
-        toSDKUserMessage(prompt),
+        toSDKUserMessage(prompt, opts.message_uuid),
         ultracode,
       )
       if (outcome === "closed") return { accepted: false, reason: "queue_dead" }
@@ -1460,7 +1470,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)。
@@ -1469,7 +1479,7 @@ export class ClaudeStreamSession {
     // ack はターン完走を待たず「起動できたか」で返すため、フル実行の Promise を
     // 呼び出し元へ伝播させない (fire-and-forget)。例外は _runPerMessage 内部で
     // 握っているため通常は reject しないが、念のため warn だけ残す。
-    this._runPerMessage(prompt, { ultracode, injectSource }).catch((err) => {
+    this._runPerMessage(prompt, { ultracode, injectSource, message_uuid: opts.message_uuid }).catch((err) => {
       this.logger?.error(
         { stream_id: this.stream_id, err: err?.message },
         "per-message run threw unexpectedly",
@@ -1538,7 +1548,7 @@ export class ClaudeStreamSession {
       // per-message でも単発 InputQueue (1 件 push → 即 close) を prompt に渡し、1 query=1 ターンの
       // resume チェーン挙動は維持しつつ、文字列 prompt + canUseTool の SDK 制約リスクを回避する。
       const inputQueue = new InputQueue()
-      inputQueue.push(toSDKUserMessage(prompt))
+      inputQueue.push(toSDKUserMessage(prompt, opts.message_uuid))
       const q = this.sdk.query({ prompt: inputQueue, options })
       const shouldWaitForTitle =
         isFreshStart && this._canGenerateSessionTitle(q, prompt)
@@ -2705,11 +2715,12 @@ export class ClaudeStreamBridge extends EventEmitter {
    *   (テストでは `{ query: stubQuery }` を渡す)
    * @param {import('pino').Logger} [opts.logger]
    */
-  constructor({ sdk, logger, readNeedsAuthCacheNames: readNeedsAuth } = {}) {
+  constructor({ sdk, logger, readNeedsAuthCacheNames: readNeedsAuth, projectsRoot } = {}) {
     super()
     if (!sdk || typeof sdk.query !== "function") {
       throw new TypeError("ClaudeStreamBridge requires { sdk: { query } }")
     }
+    this._projectsRoot = projectsRoot
     this.sdk = sdk
     this.logger = logger
     /** needs-auth キャッシュ名読み取り (テスト差し替え用。既定は実ファイル読み込み)。 */
@@ -3695,7 +3706,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 +3723,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 +3747,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 +3766,127 @@ export class ClaudeStreamBridge extends EventEmitter {
       }
     }
     const { session, attached } = matches[0]
-    // 連携カード種別 (Phase 2): 宛先確定後に記録 (not_found/ambiguous では記録しない)。
+    return this._sendSiblingMessage(cwd, text, source, session, attached)
+  }
+
+  /** 追記された完全な JSONL 行だけを確認する。パス変更時は新ファイルの先頭へ戻す。 */
+  async _confirmSiblingTranscript(session, cwd, uuid, cursor, startedAt) {
+    const deadline = startedAt + SIBLING_CONFIRM_WAIT_MS
+    try {
+      while (Date.now() < deadline) {
+        const file = session.sessionId
+          ? jsonlPath({ cwd, session_id: session.sessionId, projectsRoot: this._projectsRoot })
+          : null
+        if (file !== cursor.file) Object.assign(cursor, { file, offset: 0, pending: Buffer.alloc(0), skipping: false })
+        if (file) {
+          let handle
+          try {
+            handle = await open(file, "r")
+            const { size } = await handle.stat()
+            while (cursor.offset < size && Date.now() < deadline) {
+              const buffer = Buffer.alloc(Math.min(64 * 1024, size - cursor.offset))
+              const { bytesRead } = await handle.read(buffer, 0, buffer.length, cursor.offset)
+              if (!bytesRead) break
+              cursor.offset += bytesRead
+              const data = Buffer.concat([cursor.pending, buffer.subarray(0, bytesRead)])
+              let start = 0
+              if (cursor.skipping) {
+                // 上限超えの行の続き。途中から JSON として読まず、次の改行の後から再開する。
+                const newline = data.indexOf(10)
+                if (newline === -1) {
+                  cursor.pending = Buffer.alloc(0)
+                  continue
+                }
+                start = newline + 1
+                cursor.skipping = false
+              }
+              for (let end; (end = data.indexOf(10, start)) !== -1; start = end + 1) {
+                try {
+                  const row = JSON.parse(data.subarray(start, end).toString("utf8"))
+                  if (row?.type === "user" && row.uuid === uuid) return true
+                } catch { /* 壊れた行は確認の根拠にしない。 */ }
+              }
+              cursor.pending = data.subarray(start)
+              if (cursor.pending.length > SIBLING_CONFIRM_MAX_LINE_BYTES) {
+                cursor.pending = Buffer.alloc(0)
+                cursor.skipping = true
+              }
+            }
+          } catch (err) {
+            if (err.code !== "ENOENT") return false
+          } finally {
+            await handle?.close()
+          }
+        }
+        const remaining = deadline - Date.now()
+        if (remaining > 0) await new Promise((resolve) => setTimeout(resolve, Math.min(SIBLING_CONFIRM_POLL_MS, remaining)))
+      }
+      this.logger?.info(
+        { cwd, session_id: session.sessionId, uuid, elapsed_ms: Date.now() - startedAt },
+        "sibling transcript confirmation timed out; delivery remains unconfirmed",
+      )
+    } catch { /* 確認不能を配送失敗にしない。 */ }
+    return false
+  }
+
+  /** 受理だけを1秒待ち、直接受理された入力は送信開始から2.5秒まで会話記録を確認する。 */
+  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) {
+    const message_uuid = randomUUID()
+    const transcriptCwd = session.cwd || cwd
+    const startedAt = Date.now()
+    const cursor = { file: null, offset: 0, pending: Buffer.alloc(0), skipping: false }
+    try {
+      if (session.sessionId) {
+        cursor.file = jsonlPath({ cwd: transcriptCwd, session_id: session.sessionId, projectsRoot: this._projectsRoot })
+        cursor.offset = statSync(cursor.file).size
+      }
+    } catch { /* 新規ファイル・読み取り不能なら offset 0。 */ }
+    let timer
+    const outcome = (async () => {
+      try {
+        const result = await session.sendMessage({ role: "user", content: text }, { message_uuid })
+        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 result?.accepted === true ? { ...delivered, confirm: true } : 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
+      }
+    })()
+    let accepted
+    try {
+      accepted = 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)
+    }
+    if (!accepted.confirm) return accepted
+    const confirmed = await this._confirmSiblingTranscript(session, transcriptCwd, message_uuid, cursor, startedAt)
+    return confirmed ? { ...delivered, confirmed_via: "transcript" } : delivered
   }
 
   /**

package/src/sibling-inject.mjs

@@ -49,6 +49,7 @@
  * 書き、不正値でも property があれば明示エラーを書く。未指定の旧 producer は fire-and-forget の
  * ままなので、新旧 bundle / hub-agent のローリング互換を保つ。
  *
+ * Claude の `delivery_state: "delivered"` は会話記録の user 行の uuid で確認した場合だけ。
  * ⚠️ codex の `delivery_state` は 4 値ある (2026-08-05 に `queued` を追加):
  *   - `delivered`            … userMessage echo まで確認済み (ok:true)
  *   - `queued`               … **相手が応答生成中**。次ターンのキューに受理済みで、echo は次の
@@ -397,8 +398,9 @@ function ambiguousTargetOutcome(cliKind, targetSessionId, candidates) {
   }
 }
 
-function deliveredStateFor(kind) {
-  return kind === "codex" ? "delivered" : "dispatched"
+function deliveredStateFor(kind, result) {
+  return kind === "codex" || (kind === "claude" && result.confirmed_via === "transcript")
+    ? "delivered" : "dispatched"
 }
 
 /**
@@ -1085,7 +1087,8 @@ export class SiblingInjectBridge {
             ok: true,
             delivered_to: r.session,
             delivery_id: r.delivery_id,
-            delivery_state: deliveredStateFor(target_cli_kind),
+            delivery_state: deliveredStateFor(target_cli_kind, r),
+            ...(r.confirmed_via ? { confirmed_via: r.confirmed_via } : {}),
             durable_state: r.durable_state,
             accepted_via: r.accepted_via,
             item_id: r.item_id,
@@ -1180,7 +1183,8 @@ export class SiblingInjectBridge {
             ok: true,
             delivered_to: r.session ?? null,
             delivery_id: r.delivery_id,
-            delivery_state: deliveredStateFor(target_cli_kind),
+            delivery_state: deliveredStateFor(target_cli_kind, r),
+            ...(r.confirmed_via ? { confirmed_via: r.confirmed_via } : {}),
             durable_state: r.durable_state,
             accepted_via: r.accepted_via,
             item_id: r.item_id,