@cocorograph/hub-agent 0.7.81 → 0.7.82

package/package.json

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

package/src/main.mjs

@@ -1515,6 +1515,230 @@ export function _resetContextWindowLearning() {
   lastAssistantModelByChatKey.clear()
 }
 
+/**
+ * session_event の「サイドバー並び順に効くか (sort_affecting)」判定用の per-session 状態。
+ * 「その session-run で実ユーザーターン (prompt_submit) を見たか」を持つ。
+ *
+ * ## なぜ必要か (2026-07-20 実測で確定した現象B の根治)
+ * 死んだ/寝ているセッションを開く=hub-agent が claude を respawn/resume すると、その
+ * 起動・resume の落ち着きで Stop フック (`stop`) や readiness (`ready`)、SessionStart
+ * (`session_start`) が near-now で発火する。frontend は session.event を種別問わず
+ * eventAt の sort key に使うため、ユーザーが 1 ターンも回していないのに「開いた/起こした
+ * だけ」でサイドバー最上位へ浮上していた。
+ *
+ * 対処療法 (フロントで focus 直後の bump を数秒抑制) ではなく、発火源で「ソート信号」と
+ * 「状態信号」を分離する: **並び順は実ユーザーターン (prompt_submit と、それに続く
+ * stop/ready) だけで動かす**。ライフサイクル (session_start/session_end) と、先行する
+ * prompt_submit の無い stop/ready (=起動/resume の落ち着き) は sort_affecting=false とし、
+ * frontend は status ドット用途にのみ使い eventAt は動かさない。
+ */
+const sawUserTurnByName = new Map()
+
+// session 単位で「最後に sort_affecting=true だった event の at」を保持する。
+// event ファイルは 1 session 1 ファイルの上書きで、最新が sort_affecting=false
+// (ready 等) だと直前の実ターン時刻が失われる。これを別途保持し poll (ハードリロード
+// 復元) で配信することで、リロード後に created 順へ退行する劣化を防ぐ (codex-review major#1)。
+const lastSortAffectingAtByName = new Map()
+
+// session 単位で「最後に処理した event の at」。fs.watch の pushFile は session 単位で
+// 直列化されず、古い prompt_submit の readFile が新しい session_start の後に完了して
+// sawUserTurn=true を復活させうる (out-of-order)。at 単調ガードで stale な状態更新を弾く
+// (codex-review major#2)。
+const lastEventAtByName = new Map()
+
+/**
+ * session-event の種別 (と「実ユーザーターンを見たか」) から sort_affecting を導出する純関数。
+ * - prompt_submit: 常に true (ユーザーがターンを開始した)
+ * - session_start / session_end: 常に false (ライフサイクル、ターンではない)
+ * - stop / ready / idle_hint: 直前に prompt_submit があった (sawUserTurn) ときだけ true
+ *   (=起動/resume の落ち着きで出た stop/ready は false)
+ * - 未知種別: 後方互換で true (並び順に効かせる)
+ * @param {string} event
+ * @param {boolean} sawUserTurn
+ * @returns {boolean}
+ */
+export function computeSortAffecting(event, sawUserTurn) {
+  if (event === "prompt_submit") return true
+  if (event === "session_start" || event === "session_end") return false
+  if (event === "stop" || event === "ready" || event === "idle_hint") {
+    return sawUserTurn === true
+  }
+  return true
+}
+
+/**
+ * session-event 種別に応じて sawUserTurnByName を更新する。
+ * - prompt_submit: 実ユーザーターン開始 → true
+ * - session_start / session_end: session-run の境界 → false にリセット
+ *   (次に実 prompt_submit を見るまで stop/ready は sort_affecting=false)
+ * その他の種別ではフラグを変えない (stop/ready は直前の prompt_submit 状態を引き継ぐ)。
+ * @param {string} name tmux session 名
+ * @param {string} event
+ */
+export function updateSawUserTurn(name, event) {
+  if (!name) return
+  if (event === "prompt_submit") sawUserTurnByName.set(name, true)
+  else if (event === "session_start" || event === "session_end") {
+    sawUserTurnByName.set(name, false)
+  }
+}
+
+/**
+ * session-event 1 件を sort_affecting 状態機械に取り込む。at 単調ガードで out-of-order な
+ * stale read を弾き (major#2)、sort_affecting を導出し、true なら sort 時刻を更新する (major#1)。
+ * @param {string} name tmux session 名
+ * @param {string} event 種別
+ * @param {number} at epoch ms
+ * @returns {{ sortAffecting: boolean, sortAffectingAt: number|undefined, stale: boolean }}
+ */
+export function ingestSessionEventForSort(name, event, at) {
+  const prevAt = lastEventAtByName.get(name) ?? 0
+  // stale (この session で処理済みの at より古い) read は状態を汚さない。arm や status は
+  // 呼び出し側で従来通り処理してよいが、sort へは一切効かせない。
+  if (typeof at === "number" && at < prevAt) {
+    return {
+      sortAffecting: false,
+      sortAffectingAt: lastSortAffectingAtByName.get(name),
+      stale: true,
+    }
+  }
+  if (typeof at === "number") lastEventAtByName.set(name, at)
+  updateSawUserTurn(name, event)
+  const sortAffecting = computeSortAffecting(event, sawUserTurnByName.get(name) === true)
+  // sort_affecting=true で前進したときだけ sort 時刻を更新し、sidecar に永続化する (major#1)。
+  // event ファイルは揮発 (最新イベントで上書き) のため、再起動後に「最後の実ターン時刻」を
+  // 復元できるよう別ファイルに残す。冪等な再取込 (at 同値) では書かない。
+  if (sortAffecting && typeof at === "number" && at > (lastSortAffectingAtByName.get(name) ?? 0)) {
+    lastSortAffectingAtByName.set(name, at)
+    writeSortAtSidecar(name, at) // fire-and-forget (エラーは握りつぶす)
+  }
+  return {
+    sortAffecting,
+    sortAffectingAt: lastSortAffectingAtByName.get(name),
+    stale: false,
+  }
+}
+
+/** poll (readAllSessionEvents) 用: この session の「最後に sort に効いた at」を返す (無ければ undefined)。 */
+export function getSortAffectingAt(name) {
+  return lastSortAffectingAtByName.get(name)
+}
+
+// sort 時刻 sidecar の接尾辞。`.json` にしない (readAllSessionEvents / pushFile が
+// `.endsWith(".json")` で event ファイルと誤認しないように)。
+const SORT_AT_SUFFIX = ".sortat"
+
+export function sortAtSidecarPath(name) {
+  return path.join(SESSION_EVENTS_DIR, `${name}${SORT_AT_SUFFIX}`)
+}
+
+// sidecar の write / delete を **session 単位で直列化** する op キュー。write と delete が
+// 双方 fire-and-forget だと「実ターン直後に kill → delete が先に走り (ENOENT)、その後 write の
+// rename が dead sidecar を復活」する競合が起きる (codex-review minor)。enqueue 順 = 実行順を
+// 保証すれば、ingest(write) → forget(delete) の順で delete が後になり sidecar は確実に消える。
+const sidecarOpTailByName = new Map()
+
+function enqueueSidecarOp(name, op) {
+  const prev = sidecarOpTailByName.get(name) ?? Promise.resolve()
+  const next = prev.then(op, op).catch(() => {})
+  sidecarOpTailByName.set(name, next)
+  // 末尾が settle したら Map エントリを掃除 (自分が最新末尾のときだけ; リーク防止)。
+  next.finally(() => {
+    if (sidecarOpTailByName.get(name) === next) sidecarOpTailByName.delete(name)
+  })
+  return next
+}
+
+/**
+ * 「最後の実ターン時刻」を sidecar に atomic 書き出す (major#1 の再起動永続化)。
+ * fire-and-forget だが session 単位で直列化する。event ファイルと別ファイルなので
+ * Python hook の上書きとは競合しない。
+ */
+export function writeSortAtSidecar(name, at) {
+  if (!name || /[/\\]/.test(name) || typeof at !== "number") return Promise.resolve()
+  return enqueueSidecarOp(name, async () => {
+    const fp = sortAtSidecarPath(name)
+    const tmp = `${fp}.tmp.${randomUUID()}`
+    try {
+      await mkdir(SESSION_EVENTS_DIR, { recursive: true })
+      await writeFile(tmp, JSON.stringify({ sort_at: at }))
+      await rename(tmp, fp)
+    } catch {
+      try {
+        await unlink(tmp)
+      } catch {
+        /* ignore */
+      }
+    }
+  })
+}
+
+/**
+ * forget 時に sidecar を消す (死名が再起動シードで復活しないように; codex-review minor)。
+ * write と同じ session 単位キューに載せ、先行 write の後に必ず delete が走るようにする。
+ */
+export function deleteSortAtSidecar(name) {
+  if (!name || /[/\\]/.test(name)) return Promise.resolve()
+  return enqueueSidecarOp(name, async () => {
+    try {
+      await unlink(sortAtSidecarPath(name))
+    } catch {
+      /* 無ければ無視 */
+    }
+  })
+}
+
+/**
+ * 起動時シード: sidecar から「最後の実ターン時刻」を復元する。揮発 event ファイルではなく
+ * sidecar を正本にすることで、最新イベントが session_start/resume 由来 (sort_affecting=false)
+ * でもその near-now at で現象B を再発させない (codex-review major#1 の再репро対策)。
+ */
+export async function seedSortAtFromSidecars() {
+  let files
+  try {
+    files = await readdir(SESSION_EVENTS_DIR)
+  } catch {
+    return
+  }
+  for (const f of files) {
+    if (!f.endsWith(SORT_AT_SUFFIX)) continue
+    const name = f.slice(0, -SORT_AT_SUFFIX.length)
+    try {
+      const data = JSON.parse(await readFile(path.join(SESSION_EVENTS_DIR, f), "utf-8"))
+      if (data && typeof data.sort_at === "number") seedSortAffectingAt(name, data.sort_at)
+    } catch {
+      /* 壊れた sidecar は無視 */
+    }
+  }
+}
+
+/**
+ * sort 時刻の初期値を設定する (sidecar シード専用)。既存値は上書きしない。
+ * @param {string} name
+ * @param {number} at
+ */
+export function seedSortAffectingAt(name, at) {
+  if (!name || typeof at !== "number") return
+  if (!lastSortAffectingAtByName.has(name)) lastSortAffectingAtByName.set(name, at)
+  if ((lastEventAtByName.get(name) ?? 0) < at) lastEventAtByName.set(name, at)
+}
+
+/** session 終了 (session.gone / kill / GC) 時に sort_affecting 状態を破棄する (リーク防止)。 */
+export function forgetSortAffectingState(name) {
+  if (!name) return
+  sawUserTurnByName.delete(name)
+  lastSortAffectingAtByName.delete(name)
+  lastEventAtByName.delete(name)
+  deleteSortAtSidecar(name) // 永続 sidecar も消す (再起動シードでの死名復活防止)
+}
+
+/** テスト用: sort_affecting 状態を全リセットする。 */
+export function _resetSawUserTurn() {
+  sawUserTurnByName.clear()
+  lastSortAffectingAtByName.clear()
+  lastEventAtByName.clear()
+}
+
 /**
  * sort 用の session-event ファイルをチャットのターン境界で書き出す
  * (`/tmp/cockpit_session_events/<tmux名>.json`)。bundle hook が TUI で書くものと
@@ -1565,6 +1789,15 @@ async function readAllSessionEvents() {
       out.set(sessionName, {
         event: data.event,
         at: typeof data.at === "number" ? data.at : Date.now(),
+        // ポーリング経路 (ハードリロード時の eventAt 復元) でも push 経路と同じ
+        // sort_affecting を frontend へ伝える。map (watcher が最新化) を参照する。
+        sort_affecting: computeSortAffecting(
+          data.event,
+          sawUserTurnByName.get(sessionName) === true,
+        ),
+        // major#1: 最新 event が sort_affecting=false でも、直前の実ターン時刻を
+        // frontend へ渡して created 順への退行を防ぐ。frontend はこの値で eventAt を復元する。
+        sort_affecting_at: getSortAffectingAt(sessionName),
       })
     } catch {
       // 一時欠如 / parse 失敗は無視
@@ -1641,6 +1874,11 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
           readinessTracker.arm(sessionName)
         }
       }
+      // sort_affecting 判定を状態機械へ委譲する: at 単調ガードで out-of-order な stale read を
+      // 弾き (major#2)、prompt_submit/session_start で状態遷移し、sort_affecting=true なら
+      // sort 時刻を更新する (major#1)。arm や status push は従来通り (stale でも status は最新化)。
+      const at = typeof data.at === "number" ? data.at : Date.now()
+      const { sortAffecting } = ingestSessionEventForSort(sessionName, data.event, at)
       // 計装 (2026-06-19, 既定 OFF=2026-06-20): ターン境界イベントと arm を記録する。常時 info は
       // ログ肥大の主因のため HUB_AGENT_STATE_TRACE=1 のときだけ出す(障害解析時に有効化)。
       if (STATE_TRACE) {
@@ -1649,6 +1887,7 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
             session_name: sessionName,
             event: data.event,
             armed: readinessTracker ? readinessTracker.isArmed(sessionName) : undefined,
+            sort_affecting: sortAffecting,
           },
           "session.event push",
         )
@@ -1657,13 +1896,20 @@ async function startSessionEventWatcher({ client, logger, readinessTracker }) {
         type: "session.event",
         session_name: sessionName,
         event: data.event,
-        at: typeof data.at === "number" ? data.at : Date.now(),
+        at,
+        // false=ライフサイクル/resume 落ち着き由来 → frontend は status のみ更新し並び順は動かさない。
+        sort_affecting: sortAffecting,
       })
     } catch {
       // 一時欠如 / parse 失敗等は無視 (次の変更で読み直す)
     }
   }
 
+  // major#1 再起動ケア: pushFile より前に、sidecar から「最後の実ターン時刻」をシードする。
+  // 揮発 event ファイル (最新イベントで上書き) からはシードしない。最新が session_start/resume
+  // 由来 (sort_affecting=false) の near-now at をシードすると現象B が再発するため (codex-review)。
+  await seedSortAtFromSidecars()
+
   // 起動時に既存ファイルを 1 回読んで初期 push (再起動後に取りこぼし防止)
   try {
     const files = await readdir(SESSION_EVENTS_DIR)
@@ -2343,6 +2589,7 @@ export function startStateLoop({ client, plugins, logger, intervalMs, claudeBrid
             invalidateSessionCache(name) // capture/cwd/statusGate/spinnerFreeze/stability (state.mjs)
             stallTracker.forget(name) // StallTracker.byName
             readinessTracker?.forget?.(name) // ReadinessTracker.byName
+            forgetSortAffectingState(name) // sawUserTurn / sort_affecting_at / lastEventAt (リーク防止)
             lastByName.delete(name)
             outputFlowByName.delete(name)
             lastTurnAtByName.delete(name)
@@ -2897,6 +3144,7 @@ export async function handleKillSession(msg, ctx) {
     } catch {
       /* ignore */
     }
+    forgetSortAffectingState(name) // sort_affecting 状態のリーク防止 (kill 経路)
   }
 }
 
@@ -3901,6 +4149,9 @@ async function dispatch(msg, ctx) {
                 event: lastEvent.event,
                 at: lastEvent.at,
                 replay: true,
+                // この再配信は bind(=フォーカス) 契機の status 復旧専用。並び順は絶対に
+                // 動かさない (フォーカスで浮上させない現象B 根治の一部)。
+                sort_affecting: false,
               })
             }
           }