@cocorograph/hub-agent 0.5.29 → 0.5.31

package/bin/hub-agent.mjs

@@ -95,7 +95,15 @@ program
         agentId: cfg.agent_id,
         agentToken: cfg.agent_token,
       })
-      console.log(`hub bundle synced: version=${r.version} written=${r.written.length} skipped_same=${r.skipped_same.length}`)
+      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: 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(", ")}`)
+      }
     } catch (err) {
       console.error(`sync-bundle failed: ${err.message}`)
       process.exit(1)

package/package.json

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

package/scripts/install.sh

@@ -570,10 +570,14 @@ do_enroll() {
 
   # enroll 内部の syncBundle 失敗は warning だけで握りつぶされる (enroll 自体は
   # success 扱い) ため、ここで bundle 展開状況を verify する。
-  # CLAUDE.md の HUB-AI-RULES マーカーが入っていれば bundle 反映済みと判定。
-  if [[ ! -f "$HOME/.claude/CLAUDE.md" ]] \
-    || ! grep -q "BEGIN HUB-AI-RULES" "$HOME/.claude/CLAUDE.md" 2>/dev/null; then
-    color_warn "Hub AI bundle が ~/.claude に展開されていない可能性。sync-bundle を再試行"
+  # 2 段チェック:
+  #   - CLAUDE.md の HUB-AI-RULES マーカー (運用ルール本体が入っているか)
+  #   - scripts/manifest.json の存在 (バンドル本体のファイル群が落ちているか)
+  # マーカーは通常 syncBundle の最終ステップで書かれるが、scripts/manifest.json
+  # が無いと SessionStart hook / hub-helper.py が動かないため両方必須。どちらかが
+  # 欠けていれば半完了状態とみなして sync-bundle を retry。
+  if _hub_bundle_incomplete; then
+    color_warn "Hub AI bundle が ~/.claude に完全展開されていない可能性。sync-bundle を再試行"
     if ! retry 2 hub-agent sync-bundle; then
       color_warn "hub-agent sync-bundle が継続失敗。あとで手動実行してください:"
       color_warn "  hub-agent sync-bundle"
@@ -581,6 +585,21 @@ do_enroll() {
   fi
 }
 
+# Hub AI bundle の展開状況を判定する。完全に展開されていれば 0、欠けていれば 1。
+# do_enroll の事後 verify と verify_setup の最終チェックで共通利用する。
+_hub_bundle_incomplete() {
+  if [[ ! -f "$HOME/.claude/CLAUDE.md" ]]; then
+    return 0
+  fi
+  if ! grep -q "BEGIN HUB-AI-RULES" "$HOME/.claude/CLAUDE.md" 2>/dev/null; then
+    return 0
+  fi
+  if [[ ! -f "$HOME/.claude/scripts/manifest.json" ]]; then
+    return 0
+  fi
+  return 1
+}
+
 do_install_service() {
   color_step "OS サービスとして自動起動を登録"
   # install-service は launchctl bootout/bootstrap の transient で初回失敗する
@@ -610,12 +629,19 @@ verify_setup() {
     color_warn "Claude Code (claude) コマンドが見つかりません (cockpit から claude を呼ぶ場合は必要)"
   fi
 
-  # 3. Hub AI bundle 展開
-  if [[ -f "$HOME/.claude/CLAUDE.md" ]] \
-    && grep -q "BEGIN HUB-AI-RULES" "$HOME/.claude/CLAUDE.md" 2>/dev/null; then
-    color_ok "Hub AI bundle 展開済み (~/.claude/CLAUDE.md)"
+  # 3. Hub AI bundle 展開 (do_enroll と同じ 2 段チェック: CLAUDE.md マーカー +
+  #    scripts/manifest.json)。CLAUDE.md は書き込まれたが manifest.json が落ちて
+  #    いない半完了状態も検知する。
+  if ! _hub_bundle_incomplete; then
+    local mver
+    mver="$(grep -o '"version"[[:space:]]*:[[:space:]]*"[^"]*"' "$HOME/.claude/scripts/manifest.json" 2>/dev/null | head -1 | sed 's/.*"\([^"]*\)"$/\1/')"
+    color_ok "Hub AI bundle 展開済み (~/.claude/CLAUDE.md + scripts/manifest.json ${mver:+v$mver})"
   else
-    color_warn "Hub AI bundle が ~/.claude に展開されていません"
+    if [[ ! -f "$HOME/.claude/CLAUDE.md" ]] || ! grep -q "BEGIN HUB-AI-RULES" "$HOME/.claude/CLAUDE.md" 2>/dev/null; then
+      color_warn "Hub AI bundle が ~/.claude に展開されていません (CLAUDE.md マーカー欠落)"
+    else
+      color_warn "Hub AI bundle が半完了状態 (~/.claude/scripts/manifest.json 欠落)"
+    fi
     color_warn "  手動再同期: hub-agent sync-bundle"
     errors=$((errors + 1))
   fi

package/src/hub-bundle.mjs

@@ -63,12 +63,41 @@ export async function fetchBundle({ hubUrl, agentId, agentToken, fetchImpl } = {
 }
 
 function isClaudeMdEntry(entry) {
-  return entry.path === "CLAUDE.md" || entry.path === "./CLAUDE.md"
+  return (
+    entry.path === "CLAUDE.md" ||
+    entry.path === "./CLAUDE.md" ||
+    entry.path === "~/.claude/CLAUDE.md"
+  )
 }
 
+/**
+ * backend が返した bundle entry の path 文字列を `~/.claude/` 配下の絶対パスに
+ * 展開する。
+ *
+ * 受け付ける入力形式:
+ *  - `./scripts/hub_helper.py`        — relative (legacy)
+ *  - `scripts/hub_helper.py`          — relative (legacy bare)
+ *  - `~/.claude/scripts/hub_helper.py` — `~/.claude/` プレフィックス付き (現行)
+ *
+ * いずれの形式でも `<HOME>/.claude/scripts/hub_helper.py` に解決する。
+ *
+ * 2026-05-27 までの実装は `~/.claude/` を strip せず `path.join(base, rel)` を
+ * 呼んでいたため、`~` がリテラルディレクトリとして作成され
+ * `<HOME>/.claude/~/.claude/scripts/...` という二重展開された壊れたパスに
+ * ファイルが配置される事故が発生 (新規端末で hub-agent CLI 経由でセットアップ
+ * すると、`readBundleVersion()` が正規パスを読めず bundle_version が null に
+ * なる症状で顕在化)。
+ */
 function intoClaudeAbsolute(rel) {
   const base = resolveClaudeDir()
-  const cleaned = rel.replace(/^\.\//, "")
+  // `./` / `~/.claude/` の二系統を strip して、純粋な `~/.claude/` 配下の
+  // サブパスに正規化してから join する。順序に依存しないよう両方を試す。
+  let cleaned = rel.replace(/^\.\//, "")
+  cleaned = cleaned.replace(/^~\/\.claude\//, "")
+  // 末端で `~/.claude` (末尾スラッシュ無し) を渡された場合のために最終チェック。
+  if (cleaned === "~/.claude" || cleaned === "~") {
+    cleaned = ""
+  }
   return path.join(base, cleaned)
 }
 
@@ -228,7 +257,89 @@ function runOnce(argv, { logger } = {}) {
 }
 
 /**
- * fetch + apply + post_install をまとめて実行。
+ * Bundle response の `mcp_servers` を Claude Code CLI に登録する。
+ *
+ * Hub backend は user の権限 (is_staff 等) でフィルタした後のリストを返す
+ * ため、ここでは届いた全 server を `claude mcp add` する。staff 専用 URL は
+ * そもそも一般ユーザーのレスポンスに含まれないので、ここで二重判定する
+ * 必要は無い (= server URL を agent 側ロジックに埋め込まない設計)。
+ *
+ * `claude mcp add --transport <transport> --scope <scope> <name> <url>` を
+ * 冪等に呼ぶ。既に同名 server が登録されていれば CLI が「Updated server」
+ * メッセージで上書きする (= 何度実行しても安全)。
+ *
+ * `claude` CLI が PATH に居ない / 未インストール環境ではスキップ。
+ * (Cockpit 用途では install.sh が事前に claude をインストールしている
+ * 想定だが、独自運用環境への配慮で fail-soft にする)
+ */
+export async function applyMcpServers(bundle, { logger } = {}) {
+  const servers = Array.isArray(bundle?.mcp_servers) ? bundle.mcp_servers : []
+  if (servers.length === 0) return { added: [], skipped: [] }
+
+  // `claude` CLI の存在確認 (which / where 相当)。なければ no-op。
+  const claudeBin = await resolveClaudeBin()
+  if (!claudeBin) {
+    logger?.info("claude CLI が PATH に無いため MCP server 自動登録をスキップ")
+    return { added: [], skipped: servers.map((s) => s.name) }
+  }
+
+  const added = []
+  const skipped = []
+  for (const s of servers) {
+    if (!s?.name || !s?.url) {
+      skipped.push(s?.name || "<unnamed>")
+      continue
+    }
+    const transport = s.transport || "http"
+    const scope = s.scope || "user"
+    const argv = [
+      claudeBin,
+      "mcp",
+      "add",
+      "--transport",
+      transport,
+      "--scope",
+      scope,
+      s.name,
+      s.url,
+    ]
+    try {
+      await runOnce(argv, { logger })
+      added.push(s.name)
+    } catch (err) {
+      // 既存のため fail することはほぼ無い (CLI は冪等)。ログだけ残して継続。
+      logger?.warn({ name: s.name, err: err.message }, "claude mcp add failed")
+      skipped.push(s.name)
+    }
+  }
+  return { added, skipped }
+}
+
+/**
+ * `claude` バイナリの絶対パスを返す。なければ null。
+ * PATH を辿る (`which` / `where`) ことで fnm / nvm / homebrew / volta 等の
+ * バージョンマネージャ配下にあるものも拾う。
+ */
+async function resolveClaudeBin() {
+  const isWindows = process.platform === "win32"
+  const cmd = isWindows ? "where" : "which"
+  return new Promise((resolve) => {
+    const child = spawn(cmd, ["claude"], { stdio: ["ignore", "pipe", "ignore"] })
+    let out = ""
+    child.stdout?.on("data", (chunk) => {
+      out += chunk.toString("utf-8")
+    })
+    child.on("exit", (code) => {
+      if (code !== 0) return resolve(null)
+      const first = out.split(/\r?\n/).map((s) => s.trim()).filter(Boolean)[0]
+      resolve(first || null)
+    })
+    child.on("error", () => resolve(null))
+  })
+}
+
+/**
+ * fetch + apply + post_install + MCP server 自動登録をまとめて実行。
  */
 export async function syncBundle({ hubUrl, agentId, agentToken, logger, fetchImpl } = {}) {
   const bundle = await fetchBundle({ hubUrl, agentId, agentToken, fetchImpl })
@@ -238,5 +349,11 @@ export async function syncBundle({ hubUrl, agentId, agentToken, logger, fetchImp
   } catch (err) {
     logger?.warn({ err: err.message }, "post_install partially failed (continuing)")
   }
-  return { version: bundle.version, ...result }
+  let mcp = { added: [], skipped: [] }
+  try {
+    mcp = await applyMcpServers(bundle, { logger })
+  } catch (err) {
+    logger?.warn({ err: err.message }, "applyMcpServers failed (continuing)")
+  }
+  return { version: bundle.version, ...result, mcp }
 }