@cocorograph/hub-agent 0.8.98 → 0.8.100

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.8.98",
+  "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)
+    }
   }
 
   /**

package/src/main.mjs

@@ -1411,7 +1411,7 @@ export function updateSawUserTurn(name, event) {
  * @param {number} at epoch ms
  * @returns {{ sortAffecting: boolean, sortAffectingAt: number|undefined, stale: boolean }}
  */
-export function ingestSessionEventForSort(name, event, at, { fresh = false } = {}) {
+export function ingestSessionEventForSort(name, event, at, { fresh = false, sortable = true } = {}) {
   const prevAt = lastEventAtByName.get(name) ?? 0
   // stale (この session で処理済みの at より古い) read は状態を汚さない。arm や status は
   // 呼び出し側で従来通り処理してよいが、sort へは一切効かせない。
@@ -1437,7 +1437,9 @@ export function ingestSessionEventForSort(name, event, at, { fresh = false } = {
   }
   if (typeof at === "number") lastEventAtByName.set(name, Math.max(prevAt, at))
   updateSawUserTurn(name, event)
-  const sortAffecting = computeSortAffecting(event, sawUserTurnByName.get(name) === true)
+  // `sortable=false`: 並び順の供給源ではない経路 (bundle hook のイベントファイル)。状態・readiness には
+  // 使うが sort 時刻は動かさない (publishChatPromptSubmit / publishHookSessionEvent 参照)。
+  const sortAffecting = sortable && computeSortAffecting(event, sawUserTurnByName.get(name) === true)
   // sort_affecting=true で前進したときだけ sort 時刻を更新し、sidecar に永続化する (major#1)。
   // event ファイルは揮発 (最新イベントで上書き) のため、再起動後に「最後の実ターン時刻」を
   // 復元できるよう別ファイルに残す。冪等な再取込 (at 同値) では書かない。
@@ -1457,6 +1459,26 @@ export function getSortAffectingAt(name) {
   return lastSortAffectingAtByName.get(name)
 }
 
+/**
+ * `chat.sessions` の各 live に、その行の「最後にユーザーが送信した時刻」(`sort_affecting_at`) を載せる。
+ *
+ * サイドバーの並び順の正本は送信時刻だけ (2026-07-29 決定)。frontend は `session.event` の push で
+ * これを受け取るが、push はページを開いている間しか届かないため、再読み込み後に復元する手段が
+ * 一覧応答しかない。値は sidecar で永続化済みのもの (seedSortAtFromSidecars) をそのまま返す。
+ * 送信履歴の無い行はキーごと付けない (0 を入れると「送信済み」と区別できない)。
+ */
+export function withSortAffectingAt(snapshot, getSortAt = getSortAffectingAt) {
+  const live = Array.isArray(snapshot?.live) ? snapshot.live : []
+  return {
+    ...snapshot,
+    live: live.map((item) => {
+      const name = typeof item?.session_name === "string" ? item.session_name : ""
+      const at = name ? getSortAt(name) : undefined
+      return typeof at === "number" && at > 0 ? { ...item, sort_affecting_at: at } : item
+    }),
+  }
+}
+
 // sort 時刻 sidecar の接尾辞。`.json` にしない (readAllSessionEvents / pushFile が
 // `.endsWith(".json")` で event ファイルと誤認しないように)。
 const SORT_AT_SUFFIX = ".sortat"
@@ -1639,6 +1661,24 @@ async function readAllSessionEvents() {
   return out
 }
 
+/**
+ * bundle hook が書いたイベントファイル 1 件 (`{ event, at }`) を取り込んで push する。
+ *
+ * ⚠️ hook のイベントは並び順に使わない (sortable: false)。並び順の正本は「ユーザーが送信した
+ * 時刻」で、その供給源は agent の入力受付 (publishChatPromptSubmit) だけにする。hook の
+ * prompt_submit は (1) 実行を始めた時刻なので、キュー待ちの送信では受付より後ろへずれて
+ * 送信順を逆転させ、(2) 行名を cwd の basename から作るので、Cockpit 外で同じフォルダの
+ * claude (headless `claude -p` 等) が動いただけでも行を浮かせる。状態・readiness には従来どおり使う。
+ */
+export async function publishHookSessionEvent(ctx, sessionName, data) {
+  if (!data || typeof data.event !== "string") return undefined
+  const at = typeof data.at === "number" ? data.at : Date.now()
+  // チャットの行は state loop が返答付きの stop を送る。ここでも送ると Hub が完了通知を
+  // 2 通 (定型文だけ + 返答付き) 出すため、送信だけ省く (readiness は取り込む)。
+  const send = !(data.event === "stop" && chatStopNotifyingSessionNames.has(sessionName))
+  return publishSessionEvent(ctx, sessionName, data.event, at, { send, sortable: false })
+}
+
 /**
  * `/tmp/cockpit_session_events/<tmux_session>.json` を fs.watch して
  * UserPromptSubmit / Stop の event を `session.event` 型で Hub に push する。
@@ -1668,13 +1708,7 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
     const full = path.join(SESSION_EVENTS_DIR, filename)
     try {
       const text = await readFile(full, "utf-8")
-      const data = JSON.parse(text)
-      if (!data || typeof data.event !== "string") return
-      const at = typeof data.at === "number" ? data.at : Date.now()
-      // チャットの行は state loop が返答付きの stop を送る。ここでも送ると Hub が完了通知を
-      // 2 通 (定型文だけ + 返答付き) 出すため、送信だけ省く (並び順・readiness は取り込む)。
-      const send = !(data.event === "stop" && chatStopNotifyingSessionNames.has(sessionName))
-      await publishSessionEvent({ client, readinessTracker, logger }, sessionName, data.event, at, { send })
+      await publishHookSessionEvent({ client, readinessTracker, logger }, sessionName, JSON.parse(text))
     } catch {
       // 一時欠如 / parse 失敗等は無視 (次の変更で読み直す)
     }
@@ -1722,20 +1756,39 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
  */
 export const chatStopNotifyingSessionNames = new Set()
 
+/**
+ * チャット入力の受付を並び順へ反映する (`prompt_submit`, 受付時刻, fresh)。
+ *
+ * 並び順の正本は「ユーザーが送信した時刻」だけ (2026-07-29 決定)。**これが唯一の供給源**で、
+ * bundle hook のイベントファイルは並び順に使わない (publishHookSessionEvent の sortable: false)。
+ * この直接 publish は tmux 撤去 (189c555) で、tmux 行を走査する旧経路ごと消えていた。行名は
+ * cwd から直接解決する (chatRowSessionName が行名の正本)。publish の失敗で入力の配送を止めない。
+ */
+export function publishChatPromptSubmit(ctx, cwd, cliKind, at) {
+  const name = chatRowSessionName(cwd, cliKind)
+  if (!name) return
+  const publish = ctx._publishSessionEvent ?? publishSessionEvent
+  Promise.resolve()
+    .then(() => publish({ client: ctx.client, logger: ctx.logger }, name, "prompt_submit", at, { fresh: true }))
+    .catch((err) => {
+      ctx.logger?.warn?.({ err: err?.message, cwd, cli_kind: cliKind, session_name: name }, "chat prompt_submit publish failed")
+    })
+}
+
 /** session-event をプロセス内で取り込み、並び順を更新して WS push する。 */
 export async function publishSessionEvent(
   { client, readinessTracker, logger },
   sessionName,
   event,
   at,
-  { fresh = false, arm = !fresh, text = "", send = true } = {},
+  { fresh = false, arm = !fresh, text = "", send = true, sortable = true } = {},
 ) {
   // readiness (proc 内省の ready 判定) を動かすのはファイル監視経路だけ。直接 publish (fresh) は
   // 「人間が送った」事実を並び順へ流すだけで、ターンの開始を意味しない (bridge が拒否すれば
   // ターンは始まらないし、走査が逆順に完了した後着の古い方は既に終わったターン)。ここで arm すると
   // 次の stop が来ない限り busy が残る。ターン境界の arm は従来どおり state loop / bundle hook が
   // 書く event ファイル経由で行う。
-  const { sortAffecting } = ingestSessionEventForSort(sessionName, event, at, { fresh })
+  const { sortAffecting } = ingestSessionEventForSort(sessionName, event, at, { fresh, sortable })
   if (arm) {
     if (event === "prompt_submit") {
       readinessTracker?.arm(sessionName, { newTurn: true })
@@ -2510,6 +2563,7 @@ export function handleClaudeChatInput(msg, ctx) {
   const cwdSnapshot = ctx.claudeBridge.getCwd?.(stream_id) ?? null
   if (cwdSnapshot) {
     recordPromptAccepted(cwdSnapshot, "claude", acceptedAt, ctx.claudeBridge.sessions?.get(stream_id), ctx.claudeBridge)
+    publishChatPromptSubmit(ctx, cwdSnapshot, "claude", acceptedAt)
   }
   const p = ctx.claudeBridge
     .input({
@@ -2725,6 +2779,7 @@ export function handleCodexChatInput(msg, ctx) {
   const cwdSnapshot = ctx.codexBridge.getCwd?.(stream_id) ?? null
   if (cwdSnapshot) {
     recordPromptAccepted(cwdSnapshot, "codex", acceptedAt, ctx.codexBridge.sessions?.get(stream_id), ctx.codexBridge)
+    publishChatPromptSubmit(ctx, cwdSnapshot, "codex", acceptedAt)
   }
   const p = ctx.codexBridge
     .input({
@@ -4583,7 +4638,7 @@ export async function dispatch(msg, ctx) {
             bridges: { claudeBridge: ctx.claudeBridge, codexBridge: ctx.codexBridge },
             logger: ctx.logger,
           })
-          ctx.client.send({ type: "chat.sessions", request_id: msg.request_id, ...refreshChatSessionsSignals(snapshot, ctx.logger) })
+          ctx.client.send({ type: "chat.sessions", request_id: msg.request_id, ...withSortAffectingAt(refreshChatSessionsSignals(snapshot, ctx.logger)) })
         } catch (err) {
           ctx.client.send({
             type: "chat.sessions", request_id: msg.request_id, targets: [], live: [],