@cocorograph/hub-agent 0.6.57 → 0.6.59

package/package.json

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

package/src/main.mjs

@@ -10,7 +10,8 @@
  *
  * 仕様書: ナレッジ/インフラ/cockpit-hub-hosted-integration-spec (id=6080)
  */
-import { readFileSync, watch as fsWatch } from "node:fs"
+import { spawn as _childSpawn } from "node:child_process"
+import { existsSync, readFileSync, watch as fsWatch } from "node:fs"
 import { mkdir, readFile, readdir, rename, unlink, writeFile } from "node:fs/promises"
 import { randomUUID } from "node:crypto"
 import os from "node:os"
@@ -293,6 +294,57 @@ export async function syncTmuxProfileEnv(profile, log) {
   )
 }
 
+/**
+ * `setup_hub_ai.py --silent` を best-effort でキックし、全プロファイルの settings.json へ
+ * bundle hooks (cockpit_permission_bridge 等) を反映させる。
+ *
+ * 新規プロファイル追加 (profile.add) 直後に呼ぶ。provisionProfile は
+ * `~/.claude/settings.json` を新プロファイルへコピーするが、コピー元が古い / setup 未走の
+ * タイミングだと新プロファイルにフックが欠けうる。setup_hub_ai は全プロファイル対応 (案1)
+ * なので 1 回叩けば新プロファイルの初回セッションでもフックが揃う。
+ *
+ * 失敗・未 setup 環境 (スクリプト不在 / python3 無し) でも **例外を投げない**。detached +
+ * stdio ignore で daemon をブロックしない。session_start.py の auto-update と同じ
+ * `--silent` で headless 実行する。
+ *
+ * @param {{logger?: object, spawnImpl?: Function, existsImpl?: Function, homedir?: string}} [opts]
+ *   spawnImpl / existsImpl / homedir はテスト用の注入口 (既定は node 標準)。
+ * @returns {boolean} spawn を試みたら true、スクリプト不在等でスキップしたら false。
+ */
+export function kickSetupHubAi({
+  logger,
+  spawnImpl,
+  existsImpl,
+  homedir,
+} = {}) {
+  try {
+    const home = homedir || os.homedir()
+    const setupScript = path.join(home, ".claude", "scripts", "setup_hub_ai.py")
+    const exists = existsImpl || existsSync
+    if (!exists(setupScript)) {
+      logger?.info?.(
+        { setupScript },
+        "kickSetupHubAi: setup_hub_ai.py 不在 - skip",
+      )
+      return false
+    }
+    const doSpawn = spawnImpl || _childSpawn
+    const child = doSpawn("python3", [setupScript, "--silent"], {
+      detached: true,
+      stdio: "ignore",
+    })
+    child?.on?.("error", (e) =>
+      logger?.warn?.({ err: e?.message }, "kickSetupHubAi: spawn error"),
+    )
+    child?.unref?.()
+    logger?.info?.({}, "kickSetupHubAi: setup_hub_ai --silent を起動 (hooks 同期)")
+    return true
+  } catch (e) {
+    logger?.warn?.({ err: e?.message }, "kickSetupHubAi: 失敗 (無視)")
+    return false
+  }
+}
+
 export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
   const config = await readConfig()
   if (!config) {
@@ -1795,6 +1847,10 @@ async function dispatch(msg, ctx) {
             needs_login: created.needsLogin,
             login_command: `CLAUDE_CONFIG_DIR=${created.configDir} claude`,
           })
+          // 新規プロファイルへ bundle hooks (cockpit_permission_bridge 等) を即時反映する
+          // best-effort キック (詳細は kickSetupHubAi の docstring 参照)。失敗・未 setup
+          // 環境でも profile.add は成功扱い (kickSetupHubAi は例外を投げない)。
+          kickSetupHubAi({ logger: ctx.logger })
         } catch (err) {
           ctx.client.send({
             type: "profile.add.result",

package/src/tui-viewer-registry.mjs

@@ -66,6 +66,8 @@ export class TuiViewerRegistry {
     this.ttlSec = ttlSec
     this.logger = logger
     this._sweepTimer = null
+    /** atomic write の tmp 名を一意化するための単調増加カウンタ (race 回避)。 */
+    this._tmpSeq = 0
   }
 
   /** ディレクトリを作成し、起動時の残骸を掃除して周期 sweep を張る。 */
@@ -131,15 +133,29 @@ export class TuiViewerRegistry {
 
   /**
    * atomic write (tmp+rename)。書けなくても agent は落とさない。
+   *
+   * tmp 名は ``<fp>.<pid>.<seq>.tmp`` で **書込ごとに一意化**する。固定 ``<fp>.tmp`` を
+   * 使うと、同一マーカー (同 session_id / 同 cwd) へ複数タブ・高頻度ハートビートが並行
+   * note() したとき、先勝ちの rename が tmp を移動した後に後発の rename が ENOENT で
+   * 失敗する (rename '<fp>.tmp' → '<fp>': no such file)。マーカー自体は勝者が書くので
+   * 致命的ではないが、ログを汚し鮮度更新を取りこぼす余地があった。一意 tmp なら各書込が
+   * 独立した tmp を rename するため衝突しない。
+   *
    * @param {string} fp
    * @param {string} body
    */
   async _atomicWrite(fp, body) {
-    const tmp = `${fp}.tmp`
+    const tmp = `${fp}.${process.pid}.${++this._tmpSeq}.tmp`
     try {
       await writeFile(tmp, body)
       await rename(tmp, fp)
     } catch (err) {
+      // rename 前に書けた tmp が残りうるので掃除する (sweep は *.json のみ対象)。
+      try {
+        await unlink(tmp)
+      } catch {
+        /* 無ければ no-op */
+      }
       this.logger?.warn({ err: err?.message, fp }, "viewer marker write failed")
     }
   }