@cocorograph/hub-agent 0.5.28 → 0.5.30

package/package.json

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

package/src/main.mjs

@@ -403,6 +403,45 @@ export async function maybeSyncClaudeSettings(msg, ctx) {
   }
 }
 
+/**
+ * Plan ε: `agent.streams.sync.response` を受けて backend ↔ agent の現存
+ * stream_id を能動同期する。
+ *
+ * - local にあって backend にない → ptyBridge.detach で即時 kill (孤児)
+ * - backend にあって local にない → backend に pty.error を通知 (browser に
+ *   "pty 死んでます" を表示するため)
+ *
+ * dispatch から呼ぶ用のヘルパーだが、テストから直接叩けるよう export している。
+ */
+export function handleStreamsSyncResponse(msg, ctx) {
+  const activeIds = new Set(
+    Array.isArray(msg?.active_stream_ids) ? msg.active_stream_ids : [],
+  )
+  const localIds = new Set(ctx.ptyBridge.list())
+  for (const sid of localIds) {
+    if (!activeIds.has(sid)) {
+      ctx.logger?.warn?.(
+        { stream_id: sid, request_id: msg?.request_id },
+        "Plan ε: orphan stream detected, detaching",
+      )
+      ctx.ptyBridge.detach({ stream_id: sid })
+    }
+  }
+  for (const sid of activeIds) {
+    if (!localIds.has(sid)) {
+      ctx.logger?.warn?.(
+        { stream_id: sid, request_id: msg?.request_id },
+        "Plan ε: stream missing on agent, notifying backend (pty.error)",
+      )
+      ctx.client.send({
+        type: "pty.error",
+        stream_id: sid,
+        error: "stream_not_found_on_agent",
+      })
+    }
+  }
+}
+
 async function dispatch(msg, ctx) {
   const t = msg?.type || ""
   try {
@@ -456,6 +495,15 @@ async function dispatch(msg, ctx) {
       case "pty.detach":
         ctx.ptyBridge.detach({ stream_id: msg.stream_id })
         return
+      case "agent.streams.sync.response":
+        // Plan ε: backend ↔ agent の現存 stream_id 能動同期 (WS reconnect 直後に
+        // ws-client が agent.streams.sync.request を送って取得した結果)。
+        // ロジックは handleStreamsSyncResponse へ抽出 (テストから直接呼べる)。
+        // backend が旧 (= Plan ε 未対応) で response が一切来ないケースは何もしない。
+        // 5s タイムアウトのような明示的な timer は持たず、再接続時に再度 request
+        // が飛ぶので無害。
+        handleStreamsSyncResponse(msg, ctx)
+        return
       case "tmux.exec": {
         const args = Array.isArray(msg.args) ? msg.args : []
         const hookResult = await runHookChain(ctx.plugins, "interceptTmuxExec", {

package/src/ws-client.mjs

@@ -11,6 +11,7 @@
  *
  * 仕様書: ナレッジ/インフラ/cockpit-hub-hosted-integration-spec (id=6080)
  */
+import { randomUUID } from "node:crypto"
 import { EventEmitter } from "node:events"
 import fs from "node:fs"
 import os from "node:os"
@@ -68,22 +69,7 @@ export class WsClient extends EventEmitter {
     const ws = new WebSocket(wsUrl, { headers })
     this.ws = ws
 
-    ws.on("open", () => {
-      this.backoff = MIN_BACKOFF_MS
-      this.logger?.info("ws open")
-      // hello 前に最新の bundle version を読み直す (再接続時の鮮度確保)
-      this._refreshBundleVersion()
-      this._sendJson({
-        type: "hello",
-        agent_id: this.config.agent_id,
-        hostname: this.hostname,
-        version: this.version,
-        bundle_version: this.bundleVersion,
-      })
-      this._startHeartbeat()
-      this._startBundleWatcher()
-      this.emit("open")
-    })
+    ws.on("open", () => this._onOpen())
 
     ws.on("message", (data) => {
       let msg
@@ -111,6 +97,43 @@ export class WsClient extends EventEmitter {
     })
   }
 
+  /**
+   * WS open 時の初期送信 + 後続タスク起動。
+   *
+   * - backoff をリセット
+   * - bundle version を最新化 → hello 送信
+   * - Plan ε: agent.streams.sync.request を送信 (毎 reconnect で能動同期)
+   * - heartbeat + bundle watcher を起動
+   *
+   * test からも直接呼べるように method として切り出している (`ws.on("open")`
+   * のクロージャだと spy しづらい)。
+   */
+  _onOpen() {
+    this.backoff = MIN_BACKOFF_MS
+    this.logger?.info("ws open")
+    // hello 前に最新の bundle version を読み直す (再接続時の鮮度確保)
+    this._refreshBundleVersion()
+    this._sendJson({
+      type: "hello",
+      agent_id: this.config.agent_id,
+      hostname: this.hostname,
+      version: this.version,
+      bundle_version: this.bundleVersion,
+    })
+    // Plan ε: 毎回 reconnect 直後に backend ↔ agent の stream_id を能動同期する
+    // (orphan stream の即時 kill 用)。response は main.mjs の dispatch が拾い、
+    // ptyBridge.list() と差分を取って kill する。request_id は uuid で
+    // 重複排除。backend が古い (= Plan ε 未対応) なら "unknown_type" error が
+    // 返るだけで動作には影響しない。
+    this._sendJson({
+      type: "agent.streams.sync.request",
+      request_id: randomUUID(),
+    })
+    this._startHeartbeat()
+    this._startBundleWatcher()
+    this.emit("open")
+  }
+
   /** メッセージを送る。未接続なら no-op (logger.warn)。 */
   send(obj) {
     if (!this.ws || this.ws.readyState !== WebSocket.OPEN) {