@cocorograph/hub-agent 0.5.18 → 0.5.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.5.18",
+  "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()

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,