@cocorograph/hub-agent 0.8.78 → 0.8.80

package/package.json

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

package/src/chat-sessions-snapshot.mjs

@@ -48,6 +48,29 @@ function idleRank(item) {
   return typeof item?.idle_ms === "number" && item.idle_ms >= 0 ? item.idle_ms : Infinity
 }
 
+/**
+ * 記述子の会話 ID。claude は `session_id`、codex は `thread_id` に入る。
+ * まだ確定していない会話 (初回入力前の attach 等) は両方とも欠けるので null を返す。
+ */
+function conversationIdOf(item) {
+  if (typeof item?.session_id === "string" && item.session_id) return item.session_id
+  if (typeof item?.thread_id === "string" && item.thread_id) return item.thread_id
+  return null
+}
+
+/**
+ * 記述子が示す接続状態。⚠️ **2 つの CLI で持ち方が逆**で、claude は `attached`
+ * (claude-stream-bridge.mjs `_describeCandidate`)、codex は `detached`
+ * (codex-stream-bridge.mjs `_describe` = `!!session.detachRequested`) を返す。
+ * 片方だけを見ると、もう片方は接続中でも常に「非接続」として記録され、旧インスタンスの
+ * 残留調査を誤らせる。どちらも無ければ **false ではなく null** (不明) を返す。
+ */
+function attachedOf(item) {
+  if (typeof item?.attached === "boolean") return item.attached
+  if (typeof item?.detached === "boolean") return !item.detached
+  return null
+}
+
 /** ローカル検出した宛先と、毎回取得するライブ会話を返す。 */
 export async function buildChatSessionsSnapshot({
   bridges,
@@ -96,11 +119,24 @@ export async function buildChatSessionsSnapshot({
     }
     const kept = live[prior]
     const replace = idleRank(item) < idleRank(kept)
+    const winner = replace ? item : kept
+    const loser = replace ? kept : item
+    // ⚠️ cwd だけでは衝突の正体が分からない。想定していたのは「別 workspace 配下の同名
+    // worktree」(cwd が違う) だが、実際に観測されたのは **cwd が同じ** 衝突で、これは
+    // 同名 worktree では説明できない。会話 ID を出さないと「同じ会話が二重に生きている」
+    // (bridge に旧インスタンスが残る) のか「同じフォルダで別会話が 2 本走っている」のかを
+    // 区別できず、2026-09-17 に実際にログだけでは判別できずに詰まった。attached は
+    // 「bridge から見て今ブラウザが繋がっているか」で、旧インスタンスの残骸は false に
+    // なりやすい (ブラウザ側の実際の生存を保証する値ではない)。
     logger?.warn?.(
       {
         session_name: sessionName,
         kept_cwd: replace ? cwd : kept.cwd,
         dropped_cwd: replace ? kept.cwd : cwd,
+        kept_conversation_id: conversationIdOf(winner),
+        dropped_conversation_id: conversationIdOf(loser),
+        kept_attached: attachedOf(winner),
+        dropped_attached: attachedOf(loser),
       },
       "chat.sessions: session name collision (folded)",
     )

package/src/claude-stream-bridge.mjs

@@ -2870,12 +2870,20 @@ export class ClaudeStreamBridge extends EventEmitter {
     // 新規 query を起動せずそのセッションへ再アタッチする。これにより端末スリープ/再接続でも
     // ターンは落ちず、以降のイベントは新しい stream_id 経由でライブに流れる
     // (= 通常の生成中表示と同じ)。再接続前の確定分は browser の jsonl hydrate で復元する。
+    // ⚠️ 新規生成へ落ちた理由を 1 語で残す。ここを通ると同じ cwd に会話オブジェクトが 1 つ増え、
+    // 古い方は close されないので一覧に残り続ける (chat.sessions の名前衝突になる)。理由が
+    // `index_miss` なら「browser が持っている resume id が索引に無い」= /clear・compaction での
+    // session_id 回転後に古い id で再接続した疑い (下の onEvent で旧キーを delete しているため)、
+    // `not_reattachable` なら「索引にはあるが常駐 query が終了済み」= 古い session の掃除漏れで、
+    // 直し方が正反対になる。2026-09-17 にログからこの 2 つを区別できず判別に詰まった。
+    let reattachSkip = "no_resume"
     if (resumeSessionId) {
       // ⚠️ cwd 境界 (T06271): _liveBySession は **複合キー (cwd+session_id)** なので、別 cwd の
       //    同一 session_id は別エントリに分離される。よって cwd=A の resume を cwd=B から
       //    attach しても引き当たらず、越境 reattach が原理的に起きない (session_id は本来
       //    ユニークだが異常系でも cwd を跨がない)。
       const live = this._liveBySession.get(this._liveKeyFor(scwd, resumeSessionId))
+      reattachSkip = live ? "not_reattachable" : "index_miss"
       // R1(b): !_closed だけでなく「常駐子プロセスが実生存しているか」も確認する。子が死んで
       // いれば reattach せず下の新規 spawn 経路へフォールバックし、死んだ session を引き当てない。
       if (live && live._isReattachable()) {
@@ -3026,7 +3034,16 @@ export class ClaudeStreamBridge extends EventEmitter {
       this._liveBySession.set(key, session)
     }
     this.logger?.info(
-      { stream_id, cwd: session.cwd, model: session.model, resume: !!resumeSessionId },
+      {
+        stream_id,
+        cwd: session.cwd,
+        model: session.model,
+        // ⚠️ ブール値ではなく id そのものを出す。reattach 側 (上) は id を出しているので、
+        // 同じ値で突き合わせて「その id が過去に live だったか」を追える。ブールだけだと
+        // 「古い id で来た」と「知らない id で来た」を区別できない。
+        resume: resumeSessionId || null,
+        reattach_skip: reattachSkip,
+      },
       "claude stream attached",
     )
     // per-message モデルでは attach 時点では query() を起動しない (即 ready)。