@cocorograph/hub-agent 0.7.63 → 0.7.64

package/bin/hub-agent.mjs

@@ -27,7 +27,7 @@ if (!process.env.LC_ALL) process.env.LC_ALL = "C.UTF-8"
 
 import { hasConfig, readConfig } from "../src/config.mjs"
 import { enroll } from "../src/enroll.mjs"
-import { resolveSyncTargets, syncBundle } from "../src/hub-bundle.mjs"
+import { resolveSyncTargets, syncBundleTargets } from "../src/hub-bundle.mjs"
 import {
   installPlugin,
   listInstallableExamples,
@@ -115,29 +115,32 @@ program
         targets = resolveSyncTargets(await detectAvailableClis())
       }
 
-      // 各 CLI を **setup 完了まで同期的に**処理し、失敗は CLI 名付きで報告して continue
-      // （一方の失敗で他方を止めない）。1 つでも失敗したら非 0 で終える。
-      const failures = []
+      // 各 CLI を **setup 完了まで同期的に**処理する（純関数 syncBundleTargets に分離）。
+      // 一方の失敗で他方を止めず、失敗した CLI は成功メッセージを出さない。
+      const { results, failures } = await syncBundleTargets({
+        targets,
+        hubUrl: cfg.hub_url,
+        agentId: cfg.agent_id,
+        agentToken: cfg.agent_token,
+      })
       for (const cliKind of targets) {
-        try {
-          const r = await syncBundle({
-            hubUrl: cfg.hub_url,
-            agentId: cfg.agent_id,
-            agentToken: cfg.agent_token,
-            cliKind,
-          })
-          const mcpSummary = r.mcp
-            ? ` mcp_added=${r.mcp.added.length}${r.mcp.skipped.length > 0 ? ` mcp_skipped=${r.mcp.skipped.length}` : ""}`
-            : ""
-          console.log(
-            `hub bundle synced (${cliKind}): version=${r.version} written=${r.written.length} skipped_same=${r.skipped_same.length}${mcpSummary}`,
+        const r = results[cliKind]
+        if (!r || r.error) {
+          // 失敗した CLI は success 行を出さない（reject が正しく届いている証跡）。
+          console.error(
+            `sync-bundle failed (${cliKind}): ${r?.error ?? "unknown"}` +
+            (r?.exitCode != null ? ` (exit ${r.exitCode})` : ""),
           )
-          if (r.mcp && r.mcp.added.length > 0) {
-            console.log(`  mcp servers added: ${r.mcp.added.join(", ")}`)
-          }
-        } catch (err) {
-          console.error(`sync-bundle failed (${cliKind}): ${err.message}`)
-          failures.push(cliKind)
+          continue
+        }
+        const mcpSummary = r.mcp
+          ? ` mcp_added=${r.mcp.added.length}${r.mcp.skipped.length > 0 ? ` mcp_skipped=${r.mcp.skipped.length}` : ""}`
+          : ""
+        console.log(
+          `hub bundle synced (${cliKind}): version=${r.version} written=${r.written.length} skipped_same=${r.skipped_same.length}${mcpSummary}`,
+        )
+        if (r.mcp && r.mcp.added.length > 0) {
+          console.log(`  mcp servers added: ${r.mcp.added.join(", ")}`)
         }
       }
       if (failures.length > 0) {

package/package.json

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

package/scripts/install.sh

@@ -651,12 +651,22 @@ do_enroll() {
   _hub_reconcile_bundles
 }
 
-# 検証対象の CLI を列挙する。claude は常に対象、codex は PATH にある時だけ対象。
-# 各行 "cli_kind:dir:config_md:hooks_file"（Codex 未インストール環境では codex を要求しない）。
+# 検証対象の CLI を列挙する。**PATH で検出された CLI だけ**を対象にし、どちらも未検出の
+# ときのみ claude にフォールバックする（JS 側 resolveSyncTargets と一致。2026-07-15 差し戻し対応。
+# 旧実装は claude を常に対象にしていたため、Codex-only 環境で不要な Claude bundle 欠損警告・
+# bootstrap が走っていた）。各行 "cli_kind:dir:config_md:hooks_file"。
 _hub_target_clis() {
-  echo "claude:.claude:CLAUDE.md:settings.json"
+  local any=0
+  if present claude; then
+    echo "claude:.claude:CLAUDE.md:settings.json"
+    any=1
+  fi
   if present codex; then
     echo "codex:.codex:AGENTS.md:hooks.json"
+    any=1
+  fi
+  if [ "$any" -eq 0 ]; then
+    echo "claude:.claude:CLAUDE.md:settings.json"
   fi
 }
 

package/src/hub-bundle.mjs

@@ -538,15 +538,60 @@ export async function syncBundle({ hubUrl, agentId, agentToken, logger, fetchImp
   try {
     await runPostInstall(bundle, { logger })
   } catch (err) {
-    logger?.warn({ cliKind, err: err.message }, "post_install/setup partially failed (continuing)")
+    // ⚠️ post_install(=setup_hub_ai.py) 失敗は **bundle 同期失敗として呼び出し元へ伝播** する
+    // （2026-07-15 差し戻し対応）。旧実装は warn で握りつぶして成功を返していたため、hooks 未登録 /
+    // CONFIG_MD 未生成 / installed_version 未記録 / token 検証失敗 / setup 内 ping 失敗 / self-reexec
+    // 失敗があっても「同期成功」と誤報告していた（syncBundle の「成功を返す時点で setup 完了済み」
+    // という docstring 契約と実装が不一致だった）。MCP 登録の fail-soft（下記）とは明確に分ける。
+    // 元の exit code / 原因を失わないよう cause と cliKind を付けて再 throw する。
+    logger?.error?.({ cliKind, err: err.message }, "post_install/setup failed")
+    const wrapped = new Error(`bundle setup failed (${cliKind}): ${err.message}`)
+    wrapped.cause = err
+    wrapped.cliKind = cliKind
+    throw wrapped
   }
+  // MCP 登録は fail-soft を維持（setup 完了後の付随処理。CLI が未インストール等でも同期自体は成功扱い）。
   let mcp = { added: [], skipped: [] }
   try {
     mcp = cliKind === "codex"
       ? await applyCodexMcpServers(bundle, { logger })
       : await applyMcpServers(bundle, { agentId, agentToken, logger })
   } catch (err) {
-    logger?.warn({ err: err.message }, "applyMcpServers failed (continuing)")
+    logger?.warn({ cliKind, err: err.message }, "applyMcpServers failed (continuing)")
   }
   return { version: bundle.version, ...result, mcp }
 }
+
+/**
+ * 複数 CLI の syncBundle を順に実行し、**一方が失敗しても他方を試行**する純関数
+ * （bin/hub-agent.mjs の sync-bundle が process.exit する前段をテスト可能にするため分離。
+ * 2026-07-15）。
+ *
+ * @returns {{results: Record<string, any>, failures: string[]}}
+ *   results[cliKind] は成功時 syncBundle の戻り、失敗時 `{ error, exitCode? }`。
+ *   failures は失敗した cliKind の配列（1 件以上なら呼び出し側が exit 非 0 にする）。
+ */
+export async function syncBundleTargets({
+  targets,
+  hubUrl,
+  agentId,
+  agentToken,
+  fetchImpl,
+  logger,
+  syncImpl,
+} = {}) {
+  const sync = syncImpl || syncBundle
+  const results = {}
+  const failures = []
+  for (const cliKind of Array.isArray(targets) ? targets : []) {
+    try {
+      results[cliKind] = await sync({ hubUrl, agentId, agentToken, fetchImpl, cliKind, logger })
+    } catch (err) {
+      // exit code を message から救い出す（runOnce は "... (exit N): ..." を投げる）。
+      const m = /exit (\d+)/.exec(err?.cause?.message || err?.message || "")
+      results[cliKind] = { error: err.message, exitCode: m ? Number(m[1]) : null }
+      failures.push(cliKind)
+    }
+  }
+  return { results, failures }
+}

package/src/main.mjs

@@ -437,8 +437,9 @@ export function kickSetupHubAi({
     const setupScript = path.join(home, dirName, "scripts", "setup_hub_ai.py")
     const exists = existsImpl || existsSync
     if (!exists(setupScript)) {
+      // 全ログに cliKind を含める（Claude/Codex 2 行を識別できるように。2026-07-15 差し戻し対応）。
       logger?.info?.(
-        { setupScript },
+        { cliKind, setupScript },
         "kickSetupHubAi: setup_hub_ai.py 不在 - skip",
       )
       return false
@@ -449,13 +450,16 @@ export function kickSetupHubAi({
       stdio: "ignore",
     })
     child?.on?.("error", (e) =>
-      logger?.warn?.({ err: e?.message }, "kickSetupHubAi: spawn error"),
+      logger?.warn?.({ cliKind, setupScript, err: e?.message }, "kickSetupHubAi: spawn error"),
     )
     child?.unref?.()
-    logger?.info?.({}, "kickSetupHubAi: setup_hub_ai --silent を起動 (hooks 同期)")
+    logger?.info?.(
+      { cliKind, setupScript },
+      "kickSetupHubAi: setup_hub_ai --silent を起動 (hooks 同期)",
+    )
     return true
   } catch (e) {
-    logger?.warn?.({ err: e?.message }, "kickSetupHubAi: 失敗 (無視)")
+    logger?.warn?.({ cliKind, err: e?.message }, "kickSetupHubAi: 失敗 (無視)")
     return false
   }
 }