@cocorograph/hub-agent 0.8.112 → 0.8.113

package/package.json

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

package/src/codex-profiles.mjs

@@ -29,6 +29,7 @@ import path from "node:path"
 import os from "node:os"
 
 import { paths as hubPaths, ensureConfigDir } from "./config.mjs"
+import { fileSignature, keepRuntimeActive } from "./registry-cache.mjs"
 
 /** 既定 (プライマリ) プロファイルの ID。 */
 export const DEFAULT_CODEX_PROFILE_ID = "default"
@@ -94,6 +95,9 @@ export function registryPath() {
 
 let _cache = null
 
+/** 稼働中の active（env へ反映済みのもの）。keepRuntimeActive の docstring 参照。 */
+let _runtimeActive = null
+
 /** チルダ展開。 */
 function resolveDir(dir) {
   if (!dir) return defaultCodexDir()
@@ -106,13 +110,19 @@ function resolveDir(dir) {
  * レジストリを読み込む。存在しなければ既定プロファイルのみの仮想レジストリを返す。
  * @returns {Promise<{activeId: string, profiles: Array<{id:string,label:string,configDir:string}>}>}
  */
-export async function loadRegistry() {
-  if (_cache) return _cache
+async function readRegistry() {
+  // ⚠️ profiles.mjs と同じく、キャッシュはファイルの状態と組で持つ（手動編集を即時反映する）。
+  const signature = await fileSignature(registryPath())
+  if (_cache && _cache.signature === signature) return _cache.registry
   let parsed = null
   try {
     const raw = await fs.readFile(registryPath(), "utf-8")
     parsed = JSON.parse(raw)
   } catch (err) {
+    // 手編集の途中（保存前の truncate・書きかけ）で壊れた JSON を読んだときは、直前に
+    // 正しく読めた内容を使い続ける（署名は更新しないので、次の呼び出しで読み直す）。
+    // 壊れたまま再起動した場合は、従来どおり起動時に失敗する。
+    if (err instanceof SyntaxError && _cache) return _cache.registry
     if (err.code !== "ENOENT") throw err
   }
 
@@ -143,8 +153,27 @@ export async function loadRegistry() {
     }
   }
 
-  _cache = { activeId, profiles }
-  return _cache
+  const registry = { activeId, profiles }
+  _runtimeActive = keepRuntimeActive(registry, _runtimeActive)
+  _cache = { signature, registry }
+  return registry
+}
+
+/**
+ * レジストリを読み込む（readRegistry の docstring 参照）。
+ *
+ * ⚠️ 読み込みが失敗した時点で稼働中の active がまだ決まっていなければ、既定に固定する。
+ * 起動時に失敗すると main.mjs は既定のディレクトリで env を組むので、あとでファイルが
+ * 直ったときにファイル上の activeId を「稼働中の active」と誤認しないため。stat・パース・
+ * 正規化のどこで失敗しても同じに扱う。固定済みなら（稼働後の失敗では）変えない。
+ */
+export async function loadRegistry() {
+  try {
+    return await readRegistry()
+  } catch (err) {
+    if (!_runtimeActive) _runtimeActive = makeDefaultCodexProfile()
+    throw err
+  }
 }
 
 /** レジストリを永続化する。default は profiles[] に含めず defaultLabel / activeId だけ保存。 */
@@ -237,6 +266,10 @@ export async function setActiveProfile(id) {
   if (!target) throw new Error(`unknown codex profile: ${id}`)
   reg.activeId = id
   await saveRegistry(reg)
+  // 保存と並行して読み直した結果が旧 active のままキャッシュされないよう、ここで捨てる
+  // （同期で続けて書くので、以後に完了する読み直しは新しい active を使う）。
+  _runtimeActive = { ...target }
+  _cache = null
   return target
 }
 
@@ -406,4 +439,5 @@ export async function provisionProfile(configDir) {
 /** テスト用: モジュールキャッシュを破棄する。 */
 export function _clearCache() {
   _cache = null
+  _runtimeActive = null
 }

package/src/main.mjs

@@ -912,27 +912,37 @@ export function wireBridgeEvents(client, { claudeBridge, codexBridge }) {
   }
 }
 
-export async function startDaemon({ version, claudeSdk } = {}) {
-  const logRotationTimer = startLogRotation(logger, { isDeferred: hasInheritedOutputChild })
-  startEventLoopMetrics(logger)
-  const config = await readConfig()
-  if (!config) {
-    throw new Error(
-      "~/.hub/agent.json が見つかりません。`hub-agent enroll <token>` を先に実行してください。",
-    )
-  }
+/**
+ * 起動時の env を組む（シェル補完 → Claude / Codex プロファイルの env 反映）。
+ *
+ * launchd / systemd 起動では .zshrc 等の対話シェル設定が読まれず、~/.claude.json の
+ * MCP 定義が参照する環境変数 (例: Authorization: Bearer ${WRI_MCP_TOKEN}) が解決
+ * できない。ユーザーのログインシェル env + dotenv 層を 1 回解決して process.env を
+ * 補完する。spawn される全 Claude query が継承し、MCP トークンを出どころ
+ * (Keychain / シェル変数 / .env) を問わず引き継げる。WHY: ナレッジ/インフラ/cockpit-mcp-reauth
+ *
+ * ⚠️ プロファイルの env はシェル補完の**後**に反映する。補完は「欠けているキー」を足すので、
+ * 既定プロファイルで CLAUDE_CONFIG_DIR / CODEX_HOME を消したあとに補完すると、シェルに
+ * 書かれた値が戻り、一覧（既定）と実行（シェルの値）が食い違う。
+ * ⚠️ プロファイルを読めなかったときも既定の env を反映する（継承した値を残すと、一覧は
+ * 既定・実行は継承先になる）。profiles.mjs / codex-profiles.mjs の loadRegistry は、この
+ * 失敗時に稼働中の active を既定として固定する。
+ *
+ * @param {object} p
+ * @param {import('pino').Logger} p.logger
+ * @param {object} [p.deps] テスト用の差し替え（hydrate / getClaudeProfile / getCodexProfile）
+ */
+export async function applyStartupEnv({ logger, deps = {} }) {
+  const hydrate = deps.hydrate || hydrateProcessEnvFromShell
+  const getClaudeProfile = deps.getClaudeProfile || getActiveProfile
+  const getCodexProfile = deps.getCodexProfile || codexProfiles.getActiveProfile
 
-  const plugins = await loadPlugins(logger)
-  // ctx と dispatchCtx の shallow copy を跨いで共有する lifecycle 正本。
-  const lifecycle = { restartPending: false, shutdownStarted: false }
-  const ctx = { logger, config, plugins, lifecycle }
+  await hydrate({ logger })
 
-  // 起動時: 保存済み active プロファイルの CLAUDE_CONFIG_DIR を反映してから
-  // Claude SDK を初期化する。profiles.json が無ければ既定 (~/.claude) のまま。
   try {
-    const activeProfile = await getActiveProfile()
-    applyActiveProfileEnv(activeProfile, logger)
+    applyActiveProfileEnv(await getClaudeProfile(), logger)
   } catch (err) {
+    applyActiveProfileEnv(null, logger)
     logger.warn(
       { err: err.message },
       "failed to apply active claude profile; falling back to default ~/.claude",
@@ -941,22 +951,32 @@ export async function startDaemon({ version, claudeSdk } = {}) {
 
   // 同様に active Codex プロファイルの CODEX_HOME を反映する (独立切替)。
   try {
-    const activeCodex = await codexProfiles.getActiveProfile()
-    applyActiveCodexEnv(activeCodex, logger)
+    applyActiveCodexEnv(await getCodexProfile(), logger)
   } catch (err) {
+    applyActiveCodexEnv(null, logger)
     logger.warn(
       { err: err.message },
       "failed to apply active codex profile; falling back to default ~/.codex",
     )
   }
+}
+
+export async function startDaemon({ version, claudeSdk } = {}) {
+  const logRotationTimer = startLogRotation(logger, { isDeferred: hasInheritedOutputChild })
+  startEventLoopMetrics(logger)
+  const config = await readConfig()
+  if (!config) {
+    throw new Error(
+      "~/.hub/agent.json が見つかりません。`hub-agent enroll <token>` を先に実行してください。",
+    )
+  }
+
+  const plugins = await loadPlugins(logger)
+  // ctx と dispatchCtx の shallow copy を跨いで共有する lifecycle 正本。
+  const lifecycle = { restartPending: false, shutdownStarted: false }
+  const ctx = { logger, config, plugins, lifecycle }
 
-  // launchd / systemd 起動では .zshrc 等の対話シェル設定が読まれず、~/.claude.json の
-  // MCP 定義が参照する環境変数 (例: Authorization: Bearer ${WRI_MCP_TOKEN}) が解決
-  // できない。ユーザーのログインシェル env + dotenv 層を 1 回解決して process.env を
-  // 補完する。spawn される全 Claude query が継承し、MCP トークンを出どころ
-  // (Keychain / シェル変数 / .env) を問わず引き継げる。
-  // WHY: ナレッジ/インフラ/cockpit-mcp-reauth
-  await hydrateProcessEnvFromShell({ logger })
+  await applyStartupEnv({ logger })
 
   await runHookBroadcast(plugins, "onAgentStart", ctx)
 

package/src/profiles.mjs

@@ -32,6 +32,7 @@ import path from "node:path"
 import os from "node:os"
 
 import { paths as hubPaths, ensureConfigDir } from "./config.mjs"
+import { fileSignature, keepRuntimeActive } from "./registry-cache.mjs"
 
 /** 既定 (プライマリ) プロファイルの ID。 */
 export const DEFAULT_PROFILE_ID = "default"
@@ -56,9 +57,13 @@ export function registryPath() {
 }
 
 /** モジュール内キャッシュ (頻繁に呼ばれる getActiveConfigDir のため)。
- *  setActiveProfile / 書き込み系で必ず無効化する。 */
+ *  `{ signature, registry }`。signature がファイルの現状と一致するときだけ使う
+ *  （手動編集の即時反映）。setActiveProfile / 書き込み系でも念のため無効化する。 */
 let _cache = null
 
+/** 稼働中の active（env へ反映済みのもの）。keepRuntimeActive の docstring 参照。 */
+let _runtimeActive = null
+
 /** チルダ展開 (`~` / `~/...` を home 起点に解決)。それ以外は path.resolve。 */
 function resolveDir(dir) {
   if (!dir) return defaultConfigDir()
@@ -73,13 +78,22 @@ function resolveDir(dir) {
  *
  * @returns {Promise<{activeId: string, profiles: Array<{id:string,label:string,configDir:string}>}>}
  */
-export async function loadRegistry() {
-  if (_cache) return _cache
+async function readRegistry() {
+  // ⚠️ キャッシュは「読んだ時点のファイルの状態」と組で持ち、毎回 stat して変わっていたら
+  // 読み直す。起動時に読んだ内容を使い続けると、稼働中に ~/.hub/profiles.json を手で
+  // 作成・編集しても agent を再起動するまで反映されなかった（初期登録で再起動が 2 回要った）。
+  // stat は極小ファイル 1 回分なので、頻繁に呼ばれる getActiveConfigDir でも負荷は無視できる。
+  const signature = await fileSignature(registryPath())
+  if (_cache && _cache.signature === signature) return _cache.registry
   let parsed = null
   try {
     const raw = await fs.readFile(registryPath(), "utf-8")
     parsed = JSON.parse(raw)
   } catch (err) {
+    // 手編集の途中（保存前の truncate・書きかけ）で壊れた JSON を読んだときは、直前に
+    // 正しく読めた内容を使い続ける（署名は更新しないので、次の呼び出しで読み直す）。
+    // 壊れたまま再起動した場合は、従来どおり起動時に失敗する。
+    if (err instanceof SyntaxError && _cache) return _cache.registry
     if (err.code !== "ENOENT") throw err
   }
 
@@ -112,8 +126,27 @@ export async function loadRegistry() {
     }
   }
 
-  _cache = { activeId, profiles }
-  return _cache
+  const registry = { activeId, profiles }
+  _runtimeActive = keepRuntimeActive(registry, _runtimeActive)
+  _cache = { signature, registry }
+  return registry
+}
+
+/**
+ * レジストリを読み込む（readRegistry の docstring 参照）。
+ *
+ * ⚠️ 読み込みが失敗した時点で稼働中の active がまだ決まっていなければ、既定に固定する。
+ * 起動時に失敗すると main.mjs は既定のディレクトリで env を組むので、あとでファイルが
+ * 直ったときにファイル上の activeId を「稼働中の active」と誤認しないため。stat・パース・
+ * 正規化のどこで失敗しても同じに扱う。固定済みなら（稼働後の失敗では）変えない。
+ */
+export async function loadRegistry() {
+  try {
+    return await readRegistry()
+  } catch (err) {
+    if (!_runtimeActive) _runtimeActive = makeDefaultProfile()
+    throw err
+  }
 }
 
 /** レジストリを永続化する。default は profiles[] に含めず、追加分と activeId / defaultLabel だけ保存。 */
@@ -215,6 +248,10 @@ export async function setActiveProfile(id) {
   if (!target) throw new Error(`unknown profile: ${id}`)
   reg.activeId = id
   await saveRegistry(reg)
+  // 保存と並行して読み直した結果が旧 active のままキャッシュされないよう、ここで捨てる
+  // （同期で続けて書くので、以後に完了する読み直しは新しい active を使う）。
+  _runtimeActive = { ...target }
+  _cache = null
   return target
 }
 
@@ -537,4 +574,5 @@ async function isBrokenSymlink(p) {
 /** テスト用: モジュールキャッシュを破棄する。 */
 export function _clearCache() {
   _cache = null
+  _runtimeActive = null
 }

package/src/registry-cache.mjs

@@ -0,0 +1,47 @@
+import { promises as fs } from "node:fs"
+
+/**
+ * ファイルの現在の状態を表す文字列を返す（読み込み結果をキャッシュするときの照合キー）。
+ *
+ * パス・inode・サイズ・mtime・ctime の組で、どれかが変われば別物として扱う
+ * （同じミリ秒内の同サイズ書き換えも ctime / inode で拾う。エディタの「別名保存 → rename」も
+ * inode で拾う）。ファイルが無ければ `<path>:absent`。ENOENT 以外の失敗はそのまま投げる。
+ *
+ * @param {string} p 絶対パス
+ * @returns {Promise<string>}
+ */
+export async function fileSignature(p) {
+  try {
+    const st = await fs.stat(p)
+    return `${p}:${st.ino}:${st.size}:${st.mtimeMs}:${st.ctimeMs}`
+  } catch (err) {
+    if (err.code === "ENOENT") return `${p}:absent`
+    throw err
+  }
+}
+
+/**
+ * 稼働中に使っている active プロファイルを、読み直したレジストリへ当て直す。
+ *
+ * ⚠️ active は「env（CLAUDE_CONFIG_DIR / CODEX_HOME）に反映済みのもの」をプロセスが持ち続ける。
+ * ファイルを読み直すたびに activeId や active の configDir を手編集どおりに変えると、一覧は
+ * 新しい active を表示するのに、子プロセスは起動時の env を継承して古いアカウントで動く
+ * （表示と実行が食い違う）。env の反映と既存セッションの停止は UI の切替経路（main.mjs の
+ * profile.switch / codex.profile.switch）だけが行うので、稼働中の active の変更もその経路に限る。
+ * active への手編集（activeId の変更・active の configDir 変更・active の削除）は再起動で反映される。
+ *
+ * @param {{activeId:string, profiles:Array<{id:string,label:string,configDir:string}>}} registry 読み直した結果（破壊的に更新する）
+ * @param {{id:string,label:string,configDir:string}|null} pinned 稼働中の active（初回は null）
+ * @returns {{id:string,label:string,configDir:string}} 以後保持する active
+ */
+export function keepRuntimeActive(registry, pinned) {
+  if (!pinned) {
+    const active = registry.profiles.find((p) => p.id === registry.activeId) || registry.profiles[0]
+    return { ...active }
+  }
+  const hit = registry.profiles.find((p) => p.id === pinned.id)
+  if (!hit) registry.profiles.push({ ...pinned })
+  else if (hit.configDir !== pinned.configDir) hit.configDir = pinned.configDir
+  registry.activeId = pinned.id
+  return pinned
+}