@cocorograph/hub-agent 0.6.52 → 0.6.53

package/package.json

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

package/src/claude-history.mjs

@@ -256,22 +256,50 @@ export async function listSessions({ cwd, projectsRoot, limit = 30, logger }) {
  * - `viewingSessionId`: フロントが今表示している (閲覧ハートビートが運ぶ) session_id。
  * - `newestSessionId`: cwd 配下で実際に最新 (mtime 降順先頭) の session_id。
  * - `lastNotifiedNewId`: 同一 new への多重通知を防ぐため、最後に通知した new_session_id。
+ * - `lastNotifiedAt` / `now`: 再通知スロットリング用のタイムスタンプ (ms)。両方与えられた
+ *   ときだけ時間判定が効く (無ければ従来どおり「同一 new は一度きり」)。
+ * - `reNotifyMs`: 同一 new でも再通知を許す最小間隔 (既定 4000ms)。
+ *
+ * 回転とみなす条件:
+ *   最新が存在し、閲覧中 id と異なり、かつ次のいずれか:
+ *     (a) 最後に通知した new と異なる (= さらに別 new へ回転した) → 即通知。
+ *     (b) 最後に通知した new と同じだが、`reNotifyMs` 以上経過している → **再通知**。
+ *
+ * (b) が肝。回転通知は one-shot broadcast で、多タブ・再接続・hidden タブ等で取りこぼすと
+ * クライアントが旧 session に**恒久固着**する (ゾンビ閲覧者バグ)。viewer が依然 stale な
+ * session_id を報告し続けている (= newest と不一致) という事実は「まだ切り替わっていない」
+ * 証拠なので、スロットリングしつつ繰り返し通知して救済する。切り替わった viewer は次の
+ * ハートビートで viewingSessionId === newest を報告し、条件 (最新≠閲覧中) を満たさなくなる
+ * ため自然に再通知が止まる。`reNotifyMs` をハートビート間隔 (5s) よりやや短くしておくと、
+ * 固着している間は毎ハートビートで 1 回ずつ通知が飛ぶ。
  *
- * 回転とみなす条件: 最新が存在し、閲覧中 id と異なり、かつ直近で同じ new を通知済みでない。
  * 注意: 過去セッションを意図的に開いている (= newest 非追従) ビューでは呼び出し側が
  * `follow_newest=false` でこの判定自体をスキップすること (ピン留め閲覧を勝手に最新へ
  * 引きずらないため)。本関数は「追従中ビュー」前提で newest とのズレだけを見る。
  *
- * @param {{viewingSessionId?: string|null, newestSessionId?: string|null, lastNotifiedNewId?: string|null}} args
+ * @param {{viewingSessionId?: string|null, newestSessionId?: string|null, lastNotifiedNewId?: string|null, lastNotifiedAt?: number|null, now?: number|null, reNotifyMs?: number}} args
  * @returns {{rotated: boolean, newSessionId?: string}}
  */
 export function decideSessionRotation({
   viewingSessionId,
   newestSessionId,
   lastNotifiedNewId,
+  lastNotifiedAt,
+  now,
+  reNotifyMs = 4000,
 } = {}) {
   if (!newestSessionId || !viewingSessionId) return { rotated: false }
   if (newestSessionId === viewingSessionId) return { rotated: false }
-  if (newestSessionId === lastNotifiedNewId) return { rotated: false }
+  if (newestSessionId === lastNotifiedNewId) {
+    // 同一 new。時刻情報が揃っていて再通知間隔を超えていれば、固着救済のため再通知する。
+    if (
+      typeof now === "number" &&
+      typeof lastNotifiedAt === "number" &&
+      now - lastNotifiedAt >= reNotifyMs
+    ) {
+      return { rotated: true, newSessionId: newestSessionId }
+    }
+    return { rotated: false }
+  }
   return { rotated: true, newSessionId: newestSessionId }
 }

package/src/main.mjs

@@ -1148,13 +1148,22 @@ async function dispatch(msg, ctx) {
               })
               const newestId = sessions?.[0]?.session_id || null
               const key = viewName || viewCwd
+              const prev = ctx.tuiRotationNotified.get(key)
+              const now = Date.now()
               const { rotated, newSessionId } = decideSessionRotation({
                 viewingSessionId: viewSid,
                 newestSessionId: newestId,
-                lastNotifiedNewId: ctx.tuiRotationNotified.get(key),
+                // 旧形式 (文字列) と新形式 ({newId, ts}) の両対応。
+                lastNotifiedNewId:
+                  prev && typeof prev === "object" ? prev.newId : prev,
+                lastNotifiedAt:
+                  prev && typeof prev === "object" ? prev.ts : null,
+                now,
               })
               if (!rotated) return
-              ctx.tuiRotationNotified.set(key, newSessionId)
+              // {newId, ts} で保存し、同一 new への再通知をスロットリングする
+              // (固着した viewer は throttle 間隔ごとに 1 回ずつ救済通知を受ける)。
+              ctx.tuiRotationNotified.set(key, { newId: newSessionId, ts: now })
               // 冪等マップも新 id へ更新し、将来の remount bind が旧 id へ
               // respawn (claude 再起動) してしまうのを防ぐ。
               if (viewName) ctx.tuiReboundSessions.set(viewName, newSessionId)