@cocorograph/hub-agent 0.5.17 → 0.5.19

package/package.json

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

package/src/main.mjs

@@ -137,8 +137,34 @@ export async function startDaemon({ version, ptyModule } = {}) {
     client.send({ type: "pty.exit", stream_id, code })
   })
 
-  client.on("message", async (msg) => {
-    await dispatch(msg, { ...ctx, client, ptyBridge })
+  // Hub からのメッセージ dispatch は **直列実行** する。
+  //
+  // `EventEmitter.emit` は async リスナーの完了を待たないため、`async (msg) =>
+  // await dispatch(msg, ...)` の形だと複数 WS メッセージが連続して届いたとき
+  // それぞれの dispatch が並列実行される。`tmux.exec` の `await execTmux`
+  // (subprocess 待ち) 中に後続 `pty.data` の同期 `ptyBridge.write` が先行する
+  // と、ブラウザ→Hub では順序が保証されていても agent 内部で逆転する。
+  //
+  // 典型的な被害: cockpit ChatInput の `await onCancelTmuxMode()` → paste
+  // の流れで、ブラウザ側が cancel の応答を待ってから paste を送っても、
+  // 別経路の `tmux.exec` (state loop の polling 等) で agent が busy だと、
+  // cancel の execTmux が完了する前に paste が ptyBridge.write される。
+  // 結果として paste が tmux 内部状態 (copy mode 等) を抜ける前に届き、
+  // claude TUI に正しく入らない事象が散発する。
+  //
+  // Promise chain で前の dispatch 完了を待ってから次を始めることで、WS
+  // 受信順 = pane 反映順を保証する。pty 出力 (agent → browser) は別経路
+  // (`ptyBridge.on("output")`) なのでこの直列化の影響を受けない。
+  let dispatchChain = Promise.resolve()
+  client.on("message", (msg) => {
+    dispatchChain = dispatchChain
+      .then(() => dispatch(msg, { ...ctx, client, ptyBridge }))
+      .catch((err) => {
+        logger.error(
+          { err: err.message, type: msg?.type },
+          "dispatch threw in serial queue",
+        )
+      })
   })
 
   client.connect()
@@ -169,6 +195,40 @@ export async function startDaemon({ version, ptyModule } = {}) {
 const SESSION_EVENTS_DIR =
   process.env.COCKPIT_SESSION_EVENTS_DIR || "/tmp/cockpit_session_events"
 
+/**
+ * `/tmp/cockpit_session_events/` 全 .json を読んで session_name → {event, at}
+ * の Map を返す (Phase 4: tmux.list_sessions レスポンスに含めて配信する用)。
+ *
+ * fs.watch 由来の `session.event` push はファイル変更時のみのため、frontend が
+ * 新規マウント (ハードリロード等) すると過去 event を取得できず CockpitStatusDot
+ * が全グレー表示になる問題があった。これを list 同梱で復元する。
+ */
+async function readAllSessionEvents() {
+  const out = new Map()
+  let files
+  try {
+    files = await readdir(SESSION_EVENTS_DIR)
+  } catch {
+    return out
+  }
+  for (const f of files) {
+    if (!f.endsWith(".json")) continue
+    const sessionName = f.slice(0, -5)
+    try {
+      const text = await readFile(path.join(SESSION_EVENTS_DIR, f), "utf-8")
+      const data = JSON.parse(text)
+      if (!data || typeof data.event !== "string") continue
+      out.set(sessionName, {
+        event: data.event,
+        at: typeof data.at === "number" ? data.at : Date.now(),
+      })
+    } catch {
+      // 一時欠如 / parse 失敗は無視
+    }
+  }
+  return out
+}
+
 /**
  * `/tmp/cockpit_session_events/<tmux_session>.json` を fs.watch して
  * UserPromptSubmit / Stop の event を `session.event` 型で Hub に push する。
@@ -437,10 +497,19 @@ async function dispatch(msg, ctx) {
       case "tmux.list_sessions": {
         try {
           const sessions = await listTmuxSessions({ plugins: ctx.plugins, logger: ctx.logger })
+          // 各 session の最新 hook event を /tmp/cockpit_session_events/ から読んで
+          // レスポンスに含める (Phase 4: ハードリロード時の lastEvent 復元用)。
+          // fs.watch 由来の session.event push は揮発性のため、frontend が新規
+          // マウントすると過去 event を取れず全グレー表示になる事象を解消する。
+          const lastEventByName = await readAllSessionEvents()
+          const enriched = sessions.map((s) => ({
+            ...s,
+            last_event: lastEventByName.get(s.name) || null,
+          }))
           ctx.client.send({
             type: "tmux.sessions",
             request_id: msg.request_id,
-            sessions,
+            sessions: enriched,
           })
         } catch (err) {
           ctx.client.send({

package/src/service-install.mjs

@@ -9,7 +9,7 @@
  * テンプレ内の __HUB_AGENT_BIN__ / __HOME__ / __PATH__ を実行時に置換する。
  * 仕様書: ナレッジ/インフラ/cockpit-hub-hosted-integration-spec (id=6080)
  */
-import { promises as fs } from "node:fs"
+import { promises as fs, existsSync, realpathSync } from "node:fs"
 import os from "node:os"
 import path from "node:path"
 import { spawnSync } from "node:child_process"
@@ -63,10 +63,59 @@ function run(cmd, args, opts = {}) {
   }
 }
 
+/**
+ * `which hub-agent` の結果を、launchd / systemd から長期間 exec 可能な
+ * 安定パスに正規化する。pure 関数。`opts` で fs を差し替えてテスト可能。
+ *
+ * 背景: fnm 利用者の `which hub-agent` は現在のシェル専用の一時パス
+ * (`~/.local/state/fnm_multishells/<PID>_<ts>/bin/hub-agent`) を返す。
+ * それを plist に書き込むと、install-service を叩いたシェル終了後に
+ * symlink が消えて launchd が exec できず、agent が無音で停止する事故が
+ * 発生する (2026-05-19, シェル PID 4683 で install したまま放置されて
+ * 検知が遅れた)。
+ *
+ * 優先順:
+ *   1. fnm default alias `~/.local/share/fnm/aliases/default/bin/hub-agent`
+ *      が存在すればそれを返す (fnm 利用者にとって最も安定)
+ *   2. whichPath が `/fnm_multishells/` を含むなら realpath で
+ *      `~/.local/share/fnm/node-versions/vX.Y.Z/.../bin/hub-agent.mjs`
+ *      に展開 (node アップグレード前提でも install-service 再実行で済む)
+ *   3. それ以外 (brew / nvm / 通常 PATH) は whichPath をそのまま返す
+ */
+function normalizeBinPath(whichPath, opts = {}) {
+  const home = opts.home ?? os.homedir()
+  const fileExists = opts.fileExists ?? existsSync
+  const resolveReal = opts.realpath ?? realpathSync
+
+  const fnmDefaultBin = path.join(
+    home,
+    ".local",
+    "share",
+    "fnm",
+    "aliases",
+    "default",
+    "bin",
+    "hub-agent",
+  )
+  if (fileExists(fnmDefaultBin)) return fnmDefaultBin
+
+  if (!whichPath) return null
+  if (whichPath.includes("/fnm_multishells/")) {
+    try {
+      return resolveReal(whichPath)
+    } catch {
+      // realpath 失敗時は whichPath をそのまま返す (既存挙動の保持)
+    }
+  }
+  return whichPath
+}
+
 /** インストール先の hub-agent CLI のフルパスを返す。`hub-agent` が PATH にある前提。 */
 function detectHubAgentBin() {
   const r = spawnSync("/usr/bin/which", ["hub-agent"], { encoding: "utf-8" })
-  if (r.status === 0 && r.stdout.trim()) return r.stdout.trim()
+  const whichPath = r.status === 0 && r.stdout ? r.stdout.trim() : null
+  const normalized = normalizeBinPath(whichPath)
+  if (normalized) return normalized
   // fallback: node $repo/bin/hub-agent.mjs
   return path.join(path.dirname(fileURLToPath(import.meta.url)), "..", "bin", "hub-agent.mjs")
 }
@@ -179,6 +228,7 @@ export async function uninstallService() {
 export const _internal = {
   expandTemplate,
   detectHubAgentBin,
+  normalizeBinPath,
   macPlistPath,
   linuxUnitPath,
   repoTemplatesDir,