@cocorograph/hub-agent 0.7.30 → 0.7.32

package/package.json

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

package/src/claude-history.mjs

@@ -29,6 +29,50 @@ export const MAX_HISTORY_LINES = 500
 /** UI 表示対象の SDK message type (それ以外は jsonl 内部メタなので除外)。 */
 export const DISPLAY_TYPES = new Set(["user", "assistant", "system", "result"])
 
+/**
+ * hydrate 結果のパース済みキャッシュ (2026-07-03: Cockpit セッション切替パフォーマンス改善)。
+ *
+ * 背景: CockpitChatView は resume セッション切替のたびに毎回 readSessionHistory を呼ぶが
+ * (window/page.tsx の `key={selected}` 再マウント設計)、同一ファイルが未変更のまま
+ * 数秒〜数分内に再 hydrate されるケースが大半 (行ったり来たりのセッション切替、複数タブ)。
+ * readFile → split → 全行 JSON.parse (+ MASK_ENABLED 時は tool_use 走査) は jsonl が
+ * 大きいほど無視できないコストになるため、`mtimeMs + size` が前回と一致する間は
+ * パース結果をそのまま再利用し、ディスク読み直し・再パースを丸ごとスキップする。
+ * ファイルが変わっていれば (ライブ会話が進行 / 別ターミナルから追記等) 通常どおり再読み込み
+ * して上書きするので、常に最新のファイル内容を正しく反映する (stale 表示のリスクは無い)。
+ *
+ * key: filePath, value: { mtimeMs, size, maxLines, result }。maxLines もキーの一部として
+ * 比較する (異なる maxLines でのリクエストはキャッシュを再利用しない)。
+ */
+const HYDRATE_CACHE_MAX_ENTRIES = 200
+const hydrateCache = new Map()
+
+function getHydrateCache(filePath, mtimeMs, size, maxLines) {
+  const entry = hydrateCache.get(filePath)
+  if (!entry) return null
+  if (entry.mtimeMs !== mtimeMs || entry.size !== size || entry.maxLines !== maxLines) return null
+  // LRU: 再利用したエントリを Map の末尾へ移し直す (再挿入で挿入順を更新)。
+  hydrateCache.delete(filePath)
+  hydrateCache.set(filePath, entry)
+  return entry.result
+}
+
+function setHydrateCache(filePath, mtimeMs, size, maxLines, result) {
+  hydrateCache.delete(filePath)
+  hydrateCache.set(filePath, { mtimeMs, size, maxLines, result })
+  // 上限超過分は挿入順が古いものから GC (長時間稼働の hub-agent プロセスでメモリが
+  // 際限なく増えないようにする)。
+  while (hydrateCache.size > HYDRATE_CACHE_MAX_ENTRIES) {
+    const oldestKey = hydrateCache.keys().next().value
+    hydrateCache.delete(oldestKey)
+  }
+}
+
+/** テスト用: hydrate キャッシュを全消去する。 */
+export function clearHydrateCacheForTest() {
+  hydrateCache.clear()
+}
+
 /**
  * jsonl の生 object を、Browser が SDK message として受け取れる表示用 shape に正規化する。
  * 余分な内部メタは落とし、表示・集計に必要なフィールドだけを superset で拾う。
@@ -97,6 +141,21 @@ export function jsonlPath({ cwd, session_id, projectsRoot }) {
  * @returns {Promise<{events: object[], total_lines: number, truncated: boolean}>}
  */
 export async function readSessionHistory(filePath, { maxLines = MAX_HISTORY_LINES, logger } = {}) {
+  // stat は readFile よりずっと軽い (メタデータのみ)。mtimeMs+size が前回と同じなら
+  // ファイル内容も同じとみなし、以降の readFile + 全行 parse をまるごとスキップする。
+  let st
+  try {
+    st = await stat(filePath)
+  } catch (err) {
+    if (err.code === "ENOENT") {
+      return { events: [], total_lines: 0, truncated: false }
+    }
+    logger?.warn({ err: err.message, filePath }, "claude history stat failed")
+    throw err
+  }
+  const cached = getHydrateCache(filePath, st.mtimeMs, st.size, maxLines)
+  if (cached) return cached
+
   let text
   try {
     text = await readFile(filePath, "utf-8")
@@ -143,7 +202,9 @@ export async function readSessionHistory(filePath, { maxLines = MAX_HISTORY_LINE
     if (MASK_ENABLED) event = maskMessageObject(event, toolNames)
     events.push(event)
   }
-  return { events, total_lines, truncated }
+  const result = { events, total_lines, truncated }
+  setHydrateCache(filePath, st.mtimeMs, st.size, maxLines, result)
+  return result
 }
 
 /**

package/src/claude-stream-bridge.mjs

@@ -1647,7 +1647,12 @@ export class ClaudeStreamBridge extends EventEmitter {
       for (const t of session.lastActivityByStream.values()) {
         if (t > at) at = t
       }
-      if (at > bestAt) {
+      // `>=` (2026-07-03): Date.now() はミリ秒精度のため、短時間に複数セッションへ
+      // attach+input すると同一ミリ秒でタイが起き得る (テストや高速な連続操作で実測)。
+      // `this.sessions` は Map で挿入順を保持するため、タイ時は後から attach された
+      // (= イテレーション順で後に来る) セッションを採用するのが「最終アクティビティが
+      // 最も新しい」の意図に合う。厳密な `>` だと先勝ちになり意図と逆転する。
+      if (at >= bestAt) {
         bestAt = at
         best = session.sessionId
       }

package/src/codex-stream-bridge.mjs

@@ -23,6 +23,13 @@
  *   - 画像/添付入力 (テキストのみ)。
  *   - acceptForSession や execpolicy/network amendment 等の高度な承認 decision
  *     (permissionReply は decision をそのまま素通しするので呼び出し側が組み立てる)。
+ *
+ * Slice 6 (履歴 hydrate) で追加: `thread/resume` のレスポンスは既定で
+ * `thread.turns[].items[]` (= ThreadItem の時系列) を含む (公式スキーマ
+ * `ThreadReadResponse.json` の Thread.turns 説明: "Only populated on thread/resume,
+ * thread/rollback, thread/fork, and thread/read (when includeTurns is true)")。
+ * 追加の `thread/read` 往復は不要で、attach() の戻り値に `history` として平坦化して
+ * 返す (browser 側での re-mount 白紙状態を解消する)。
  */
 import { EventEmitter } from "node:events"
 
@@ -39,6 +46,17 @@ const IDLE_DETACH_TTL_MS = 5 * 60 * 1000
 
 const CLIENT_INFO = { name: "cockpit-hub-agent", title: "Cockpit", version: "1.0.0" }
 
+// Slice 6: thread/resume が返す Thread.turns[].items[] を単一の時系列配列へ平坦化する。
+// turns は開始ターン順の配列、各 turn.items も時系列順のため単純結合で会話順を保てる。
+function flattenThreadItems(thread) {
+  const turns = Array.isArray(thread?.turns) ? thread.turns : []
+  const items = []
+  for (const turn of turns) {
+    if (Array.isArray(turn?.items)) items.push(...turn.items)
+  }
+  return items
+}
+
 function userInputFromMessage(message) {
   const content = message?.content
   if (typeof content === "string") {
@@ -65,7 +83,7 @@ export class CodexStreamBridge extends EventEmitter {
   }
 
   /**
-   * @returns {Promise<{resuming: boolean, threadId: string}>}
+   * @returns {Promise<{resuming: boolean, threadId: string, history: object[]}>}
    */
   async attach({ stream_id, cwd, model, effort, approvalPolicy, sandbox, resumeThreadId }) {
     if (this.sessions.has(stream_id)) {
@@ -96,7 +114,10 @@ export class CodexStreamBridge extends EventEmitter {
         ? await client.request("thread/resume", { ...startParams, threadId: resumeThreadId })
         : await client.request("thread/start", startParams)
       session.threadId = result?.thread?.id
-      return { resuming, threadId: session.threadId }
+      // 新規スレッド (thread/start) は履歴が無いので空配列。resume 時のみ平坦化する
+      // (thread/start のレスポンスは turns を持たない仕様のため呼んでも常に空)。
+      const history = resuming ? flattenThreadItems(result?.thread) : []
+      return { resuming, threadId: session.threadId, history }
     } catch (err) {
       this.sessions.delete(stream_id)
       client.stop()

package/src/config.mjs

@@ -10,7 +10,7 @@
 import { promises as fs, constants as fsConsts, readFileSync } from "node:fs"
 import path from "node:path"
 import os from "node:os"
-import { execFile } from "node:child_process"
+import { execFile, execFileSync } from "node:child_process"
 import { promisify } from "node:util"
 
 const _execFileP = promisify(execFile)
@@ -21,9 +21,20 @@ const _execFileP = promisify(execFile)
  * 重要: Windows のエージェントは WSL2 内で動くため `process.platform` は "linux" を
  * 返す。素の Linux と区別するため、WSL を検出して "wsl" を返す。
  *
+ * 判定は 3 段階のフォールバック:
+ * 1. `WSL_DISTRO_NAME` / `WSL_INTEROP` 環境変数 (対話シェルなら通常付く)
+ * 2. `/proc/version` の内容 (microsoft/wsl 文字列)
+ * 3. (2026-07-03 追加) `uname -r` のカーネルリリース文字列。systemd サービス等
+ *    非対話起動では 1 の環境変数が unit に伝播しないケースがある (WSL2 + systemd の
+ *    典型的な罠。対話シェルにしか WSL_DISTRO_NAME が無く、hub-agent をサービス化
+ *    すると引き継がれない)。`uname -r` は環境変数に依存せずカーネル自体から
+ *    "5.15.x-microsoft-standard-WSL2" のような値を返すため、より確実な最終フォールバック
+ *    として使う。
+ *
+ * @param {{execImpl?: (cmd: string, args: string[]) => string}} [opts] テスト用の exec 差し替え
  * @returns {"darwin"|"wsl"|"linux"|"win32"|string}
  */
-export function detectPlatform() {
+export function detectPlatform({ execImpl } = {}) {
   if (process.platform === "darwin") return "darwin"
   if (process.platform === "win32") return "win32"
   if (process.platform === "linux") {
@@ -32,7 +43,14 @@ export function detectPlatform() {
       const v = readFileSync("/proc/version", "utf-8").toLowerCase()
       if (v.includes("microsoft") || v.includes("wsl")) return "wsl"
     } catch {
-      /* /proc/version が読めない環境は素の linux 扱い */
+      /* /proc/version が読めない環境は次のフォールバックへ */
+    }
+    try {
+      const run = execImpl || ((cmd, args) => execFileSync(cmd, args, { encoding: "utf-8" }))
+      const release = run("uname", ["-r"]).toLowerCase()
+      if (release.includes("microsoft") || release.includes("wsl")) return "wsl"
+    } catch {
+      /* uname が無い/失敗する環境は素の linux 扱い */
     }
     return "linux"
   }

package/src/main.mjs

@@ -3093,6 +3093,10 @@ async function dispatch(msg, ctx) {
             stream_id,
             resuming: info.resuming,
             thread_id: info.threadId,
+            // Slice 6: 履歴 hydrate。resume 時のみ非空 (thread/start は常に [])。
+            // consumers.py の per-stream relay は content-blind なので backend 側の
+            // 変更は不要 (_AGENT_PER_STREAM_TYPES の codex.ready がそのまま運ぶ)。
+            history: info.history || [],
           })
         } catch (err) {
           ctx.client.send({

package/src/usage.mjs

@@ -1152,9 +1152,13 @@ export async function getUsage() {
   // チャット(SDK) の rate_limit_event が statusLine cache より新しければ、5h/7d を
   // その実値 (utilization) で上書きする。ターミナルが statusLine を更新していない
   // チャットモードでも 5h/7d がライブで動くようになる。
+  // `>=` (2026-07-03): ファイル mtime はミリ秒未満の粒度が丸められる環境があり、
+  // 同一ミリ秒 (またはそれ以下) の解像度で statusLine 書き込みと rate_limit_event が
+  // 揃うとタイになり得る。プロセス内の直近イベント (chatRateLimits) の方が
+  // ディスク read-back より鮮度の情報として確実なため、タイ時は上書きを優先する。
   if (
     chatRateLimits.updatedAtMs > 0 &&
-    (cacheMtime === null || chatRateLimits.updatedAtMs > cacheMtime)
+    (cacheMtime === null || chatRateLimits.updatedAtMs >= cacheMtime)
   ) {
     if (chatRateLimits.five_hour) {
       result.last5h = {
@@ -1183,7 +1187,9 @@ export async function getUsage() {
   const chatSnap = await readChatUsageSnapshot()
   if (chatSnap && chatSnap.updatedAtMs > 0) {
     const baseline = Math.max(cacheMtime ?? 0, chatRateLimits.updatedAtMs || 0)
-    if (chatSnap.updatedAtMs > baseline) {
+    // `>=` (上と同じ理由): ファイル mtime とプロセス内タイムスタンプがタイになり得る。
+    // chatSnap は最も直接的な情報源 (SDK control request 実値) なのでタイ時は優先する。
+    if (chatSnap.updatedAtMs >= baseline) {
       if (chatSnap.five_hour) {
         result.last5h = {
           ...result.last5h,