@cocorograph/hub-agent 0.8.66 → 0.8.67

package/bin/hub-agent.mjs

@@ -38,6 +38,7 @@ import { loadPlugins } from "../src/plugin-loader.mjs"
 import { uninstallPlugin } from "../src/plugin-uninstall.mjs"
 import {
   canSuggestForcedRestart,
+  formatInstallServiceReport,
   installService,
   restartService,
   restartWithPolicy,
@@ -215,16 +216,12 @@ program
   .action(async (opts) => {
     try {
       const r = await installService({ bin: opts.bin })
-      console.log(`installed: ${r.platform} ${r.path}`)
-      console.log(`bin: ${r.bin}`)
-      if (r.platform === "darwin") {
-        console.log(`label: ${r.label}`)
-        console.log("確認: launchctl list | grep co.cocorograph.hub-agent")
-      } else {
-        console.log(`unit: ${r.unit}`)
-        console.log("確認: systemctl --user status hub-agent.service")
+      // 表示と警告の判断は formatInstallServiceReport (テスト可能な純関数) が持つ。
+      // 順序を崩さないよう、種別つきの 1 本の配列をそのまま流す。
+      for (const { stream, text } of formatInstallServiceReport(r)) {
+        if (stream === "warn") console.warn(text)
+        else console.log(text)
       }
-      console.log("ログ: ~/.hub/agent.log")
     } catch (err) {
       console.error(`install-service failed: ${err.message}`)
       process.exit(1)

package/package.json

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

package/scripts/install.sh

@@ -1347,8 +1347,11 @@ verify_setup() {
   done < <(_hub_target_clis)
 
   # 4. OS サービス起動 (Darwin 限定の確認。Linux は systemctl --user で別途)
+  # ⚠️ 判定は `launchctl print` で行う。install-service 側が登録の出現を待つのに使う
+  # 道具と揃えるため (src/service-install.mjs の waitUntilBootedIn)。`launchctl list | grep`
+  # に戻すと、待つ側と確認する側で別の状態を見ることになり誤検知が復活する。
   if [[ "$(uname)" == "Darwin" ]]; then
-    if launchctl list 2>/dev/null | grep -q "co.cocorograph.hub-agent"; then
+    if launchctl print "gui/$(id -u)/co.cocorograph.hub-agent" >/dev/null 2>&1; then
       color_ok "launchd service 起動中 (co.cocorograph.hub-agent)"
     else
       color_warn "launchd service が見つかりません"

package/src/service-install.mjs

@@ -154,6 +154,29 @@ const sleep = (ms) => new Promise((r) => setTimeout(r, ms))
  * @returns {Promise<boolean>} 消えたら true / timeout なら false
  */
 async function waitUntilBootedOut(uid, opts = {}) {
+  // 非 0 = 登録に存在しない = bootout 完了
+  return waitForRegistration(uid, false, opts)
+}
+
+/**
+ * launchd の登録状態が `want` になるまで polling する。`waitUntilBootedOut` /
+ * `waitUntilBootedIn` の共通実装。
+ *
+ * ⚠️ 判定は `launchctl print` の終了コードで行う。`launchctl list | grep` は同じ状態を
+ * 別の道具で見ることになり、待つ側と確認する側がずれると誤検知が復活するため、
+ * hub-agent 側と `scripts/install.sh` の verify_setup で道具を揃えている。
+ *
+ * ⚠️ 成功条件は `(status === 0) === want` の 1 箇所だけで表す。`status` は spawn 自体が
+ * 失敗すると `null` になり得るので、「0 か否か」に正規化してから比較する
+ * (bootout 待ちでは null は「登録に存在しない」＝完了、bootin 待ちでは「まだ現れない」
+ *  ＝継続、という従来の挙動がそのまま保たれる)。
+ *
+ * @param {number} uid
+ * @param {boolean} want true = 登録に現れるまで待つ / false = 消えるまで待つ
+ * @param {{ timeoutMs?: number, intervalMs?: number, spawn?: Function, sleep?: Function, now?: Function }} [opts]
+ * @returns {Promise<boolean>} 期待状態になったら true / timeout なら false
+ */
+async function waitForRegistration(uid, want, opts = {}) {
   const timeoutMs = opts.timeoutMs ?? 5000
   const intervalMs = opts.intervalMs ?? 200
   const spawn = opts.spawn ?? spawnSync
@@ -164,13 +187,33 @@ async function waitUntilBootedOut(uid, opts = {}) {
     const r = spawn("launchctl", ["print", `gui/${uid}/${PLIST_LABEL}`], {
       stdio: "ignore",
     })
-    // 非 0 = 登録に存在しない = bootout 完了
-    if (r.status !== 0) return true
+    if ((r.status === 0) === want) return true
     await sleepFn(intervalMs)
   }
   return false
 }
 
+/**
+ * `launchctl bootstrap` / `kickstart -k` の直後は、launchd 側の登録が見えるまでに
+ * わずかな遅れがある。install-service がそこで返ってしまうと、直後に登録を確認する
+ * 呼び出し側 (`scripts/install.sh` の verify_setup 等) が「サービスが見つからない」と
+ * 誤検知する。本関数は対象 service が launchd の登録に現れるまで polling して待つ。
+ * `waitUntilBootedOut` の対称形。
+ *
+ * 判定手段の統一 (`launchctl print`) は `waitForRegistration` の注意書きを参照。
+ *
+ * (2026-09-09: ワンライナー完走時に毎回「launchd service が見つかりません」が出ていた
+ *  誤検知の修正。install-service は成功しており、agent も起動していた)
+ *
+ * @param {number} uid
+ * @param {{ timeoutMs?: number, intervalMs?: number }} [opts]
+ * @returns {Promise<boolean>} 現れたら true / timeout なら false
+ */
+async function waitUntilBootedIn(uid, opts = {}) {
+  // 0 = 登録に存在する
+  return waitForRegistration(uid, true, opts)
+}
+
 /**
  * bootout 後の bootstrap を、過渡的失敗 (旧サービスの抜け残り等) に対して
  * リトライする。`run` と違い最終的に失敗したときだけ throw する。
@@ -257,7 +300,52 @@ function detectHubAgentBin() {
   return path.join(path.dirname(fileURLToPath(import.meta.url)), "..", "bin", "hub-agent.mjs")
 }
 
-export async function installService({ bin } = {}) {
+/**
+ * darwin の launchd 登録シーケンス:
+ * bootout → (登録が消えるまで待機) → bootstrap (リトライ) → kickstart → (登録が現れるまで待機)。
+ *
+ * - bootout は非同期なので、待たずに bootstrap するとレースで失敗し offline 化する
+ *   (2026-05-30 の事故)。`waitUntilBootedOut` で確実に抜けてから bootstrap する。
+ * - kickstart は「既に走っている daemon を再起動」する用途。bootstrap + RunAtLoad で
+ *   既に起動済みのことがあり、その場合 kickstart 失敗は致命的でないので throw しない。
+ * - kickstart もまた launchd 側の登録が見えるまで待たない。ここで待たずに返すと、直後に
+ *   登録を確認する呼び出し側 (`scripts/install.sh` の verify_setup 等) が誤検知する
+ *   (2026-09-09 の誤検知)。現れなかった場合も throw しない: bootstrap は成功しているので
+ *   致命ではなく、判断は呼び出し側の確認へ委ねる。
+ *   ⚠️ ただし「確認できなかった」事実は捨てない。非 0 で終わると `scripts/install.sh` の
+ *   `retry 2 hub-agent install-service` が再インストール + 再起動を走らせてしまうので、
+ *   exit code は変えず `registered` で返し、CLI 側は警告だけ出す。
+ *
+ * ⚠️ この順序をソース文字列の走査で固定するのは**やめた** (2026-09-09 cross-review 3 巡目)。
+ * コメント化・別オブジェクトへの退避など、文字列一致では追い切れない後退が巡ごとに出たため、
+ * 依存を差し替えて**実際に順序と返り値を確認できる**形へ切り出してある。同ファイルの
+ * `restartWithPolicy` と同じ考え方。`deps` は本番では既定値のまま使う。
+ *
+ * @param {{uid: number, dest: string, bin: string}} target
+ * @param {{spawn?: Function, bootedOut?: Function, bootstrap?: Function, bootedIn?: Function}} [deps]
+ */
+export async function activateDarwinService({ uid, dest, bin }, deps = {}) {
+  const spawn = deps.spawn ?? spawnSync
+  const bootedOut = deps.bootedOut ?? waitUntilBootedOut
+  const bootstrap = deps.bootstrap ?? bootstrapWithRetry
+  const bootedIn = deps.bootedIn ?? waitUntilBootedIn
+
+  spawn("launchctl", ["bootout", `gui/${uid}`, dest], { stdio: "ignore" })
+  await bootedOut(uid)
+  await bootstrap(uid, dest)
+  spawn("launchctl", ["kickstart", "-k", `gui/${uid}/${PLIST_LABEL}`], {
+    stdio: "ignore",
+  })
+  const registered = await bootedIn(uid)
+  return { platform: "darwin", path: dest, label: PLIST_LABEL, bin, registered }
+}
+
+/**
+ * @param {{bin?: string, activate?: Function}} [opts]
+ *   `activate` は darwin の launchd 登録シーケンス。テストから差し替えるための口で、
+ *   本番では既定 (`activateDarwinService`) のまま使う。
+ */
+export async function installService({ bin, activate = activateDarwinService } = {}) {
   const hubAgentBin = bin || detectHubAgentBin()
   await ensureLogFile()
 
@@ -269,18 +357,7 @@ export async function installService({ bin } = {}) {
     await fs.writeFile(dest, expanded, { mode: 0o644 })
 
     const uid = ensureUnixUid()
-    // 既存ロード解除 → (登録が消えるまで待機) → bootstrap (リトライ) → kickstart。
-    // bootout は非同期なので待たずに bootstrap するとレースで失敗し offline 化する
-    // (2026-05-30 の事故)。waitUntilBootedOut で確実に抜けてから bootstrap する。
-    spawnSync("launchctl", ["bootout", `gui/${uid}`, dest], { stdio: "ignore" })
-    await waitUntilBootedOut(uid)
-    await bootstrapWithRetry(uid, dest)
-    // kickstart は「既に走っている daemon を再起動」する用途。bootstrap + RunAtLoad で
-    // 既に起動済みのことがあり、その場合 kickstart 失敗は致命的でないので throw しない。
-    spawnSync("launchctl", ["kickstart", "-k", `gui/${uid}/${PLIST_LABEL}`], {
-      stdio: "ignore",
-    })
-    return { platform: "darwin", path: dest, label: PLIST_LABEL, bin: hubAgentBin }
+    return activate({ uid, dest, bin: hubAgentBin })
   }
 
   if (process.platform === "linux") {
@@ -389,6 +466,50 @@ export async function restartWithPolicy({
   return { mode: "deferred", result: await deferredRestart(), service }
 }
 
+/**
+ * install-service の結果を CLI 表示へ変換する。`restartWithPolicy` と同じく、
+ * CLI 側の判断をテスト可能な純関数へ切り出したもの (bin/hub-agent.mjs から使う)。
+ *
+ * ⚠️ 確認コマンドは launchd の登録を待つ側 (`waitForRegistration`) と
+ * `scripts/install.sh` の verify_setup に揃える。`launchctl list | grep` へ戻すと、
+ * 同じ状態を 3 箇所が別々の道具で見ることになり、誤検知が復活する。
+ *
+ * ⚠️ 警告は `registered === false` のときだけ。bootstrap 自体は成功しているので
+ * exit code は変えない (非 0 にすると `scripts/install.sh` の `retry 2` が
+ * 再インストール + 再起動を走らせてしまう)。linux は登録待ちを持たず `registered` を
+ * 返さないので、`undefined` で警告しないこと。
+ *
+ * @param {{platform: string, path: string, bin: string, label?: string, unit?: string, registered?: boolean}} result
+ * @returns {{stream: 'log'|'warn', text: string}[]} 出力すべき行を順序どおりに
+ */
+export function formatInstallServiceReport(result) {
+  const out = [
+    { stream: "log", text: `installed: ${result.platform} ${result.path}` },
+    { stream: "log", text: `bin: ${result.bin}` },
+  ]
+  if (result.platform === "darwin") {
+    out.push({ stream: "log", text: `label: ${result.label}` })
+    out.push({
+      stream: "log",
+      text: "確認: launchctl print gui/$(id -u)/co.cocorograph.hub-agent",
+    })
+    if (result.registered === false) {
+      // ⚠️ 警告は確認コマンドの直後に置く (読む人が対処法とセットで見るため)。
+      // 種別つきの 1 本の配列で返すのは、この順序を stdout / stderr の分離と
+      // 両立させるため。呼び出し側で 2 本のリストへ分けると順序が崩れる。
+      out.push({
+        stream: "warn",
+        text: "warn: launchd への登録を確認できませんでした (待機 timeout)。上のコマンドで確認してください",
+      })
+    }
+  } else {
+    out.push({ stream: "log", text: `unit: ${result.unit}` })
+    out.push({ stream: "log", text: "確認: systemctl --user status hub-agent.service" })
+  }
+  out.push({ stream: "log", text: "ログ: ~/.hub/agent.log" })
+  return out
+}
+
 export async function uninstallService() {
   if (process.platform === "darwin") {
     const uid = ensureUnixUid()
@@ -498,6 +619,10 @@ export const _internal = {
   linuxUnitPath,
   repoTemplatesDir,
   waitUntilBootedOut,
+  waitUntilBootedIn,
+  waitForRegistration,
+  formatInstallServiceReport,
+  activateDarwinService,
   bootstrapWithRetry,
   currentUsername,
   enableLinger,