@cocorograph/hub-agent 0.8.55 → 0.8.56

package/package.json

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

package/src/claude-credentials.mjs

@@ -131,6 +131,45 @@ export async function existsGenericPassword(service, opts = {}) {
   }
 }
 
+/**
+ * 非既定プロファイルのハッシュ付き Keychain 認証を削除する。
+ *
+ * プロファイル削除 (profiles.mjs deleteProfile) の beforeRemove で呼ぶ。候補は literal / realpath の
+ * `credentialService()` だけで、素の `PLAIN_CREDENTIAL_SERVICE` (既定プロファイル用) は
+ * **絶対に対象へ含めない**。未存在は無視し、それ以外の失敗は削除全体を止める。
+ *
+ * @param {string} configDir
+ * @param {{execFileP?: Function, platform?: string, realpath?: string|null}} [opts]
+ *   realpath は呼び出し側が rm 前に取った値 (削除後は fs.realpath が失敗するため)
+ * @returns {Promise<number>} 削除できた件数 (非 darwin は 0)
+ */
+export async function deleteKeychainCredentials(configDir, opts = {}) {
+  if (!isKeychainPlatform(opts.platform)) return 0
+  const exec = opts.execFileP || execFileP
+  const dirs = new Set([configDir])
+  if (opts.realpath) dirs.add(opts.realpath)
+  else {
+    try { dirs.add(await fs.realpath(configDir)) } catch { /* 削除済み等 */ }
+  }
+  const services = new Set([...dirs].map(credentialService))
+  services.delete(PLAIN_CREDENTIAL_SERVICE)
+  let deleted = 0
+  for (const service of services) {
+    try {
+      await exec("security", ["delete-generic-password", "-s", service])
+      deleted += 1
+    } catch (err) {
+      const msg = `${err?.message || ""}${err?.stderr || ""}`
+      if (msg.includes("could not be found") || msg.includes("SecKeychainSearchCopyNext")) continue
+      const detail = String(err?.stderr || err?.message || "unknown error").trim().replace(/\s+/g, " ")
+      const failure = new Error(`Keychain service ${service} の削除に失敗しました: ${detail}`)
+      failure.code = "io"
+      throw failure
+    }
+  }
+  return deleted
+}
+
 /**
  * その config ディレクトリで Claude Code がログイン済みか (UI の「要ログイン」表示用)。
  *

package/src/codex-profiles.mjs

@@ -149,19 +149,22 @@ export async function loadRegistry() {
 
 /** レジストリを永続化する。default は profiles[] に含めず defaultLabel / activeId だけ保存。 */
 async function saveRegistry(reg) {
-  await ensureConfigDir()
-  const def = reg.profiles.find((p) => p.id === DEFAULT_CODEX_PROFILE_ID)
-  const payload = {
-    activeId: reg.activeId,
-    defaultLabel: def ? def.label : undefined,
-    profiles: reg.profiles
-      .filter((p) => p.id !== DEFAULT_CODEX_PROFILE_ID)
-      .map((p) => ({ id: p.id, label: p.label, configDir: p.configDir })),
+  try {
+    await ensureConfigDir()
+    const def = reg.profiles.find((p) => p.id === DEFAULT_CODEX_PROFILE_ID)
+    const payload = {
+      activeId: reg.activeId,
+      defaultLabel: def ? def.label : undefined,
+      profiles: reg.profiles
+        .filter((p) => p.id !== DEFAULT_CODEX_PROFILE_ID)
+        .map((p) => ({ id: p.id, label: p.label, configDir: p.configDir })),
+    }
+    await fs.writeFile(registryPath(), JSON.stringify(payload, null, 2), {
+      mode: 0o600,
+    })
+  } finally {
+    _cache = null
   }
-  await fs.writeFile(registryPath(), JSON.stringify(payload, null, 2), {
-    mode: 0o600,
-  })
-  _cache = null
 }
 
 /** プロファイル一覧と active を返す (UI 表示用)。 */
@@ -252,8 +255,53 @@ export async function renameProfile(id, label) {
   return { id: target.id, label: target.label, configDir: target.configDir }
 }
 
+function deletionError(code, message, cause) {
+  const err = new Error(message, cause ? { cause } : undefined)
+  err.code = code
+  return err
+}
+
+export async function deleteProfile(id) {
+  if (id === DEFAULT_CODEX_PROFILE_ID) throw deletionError("default", "既定プロファイルは削除できません")
+  const reg = await loadRegistry()
+  const target = reg.profiles.find((p) => p.id === id)
+  if (!target) throw deletionError("unknown", `未知のプロファイルです: ${id}`)
+  if (reg.activeId === id) throw deletionError("active", "アクティブなプロファイルです。先に別のアカウントへ切り替えてください")
+  const configDir = target.configDir
+  const expected = `.codex-${id}`
+  if (!path.isAbsolute(configDir) || path.dirname(configDir) !== os.homedir()
+    || path.basename(configDir) !== expected || !/^\.codex-\d+$/.test(expected)) {
+    throw deletionError("unsafe_path", `安全でない configDir です: ${configDir}`)
+  }
+  const defaults = new Set([defaultCodexDir()])
+  try { defaults.add(await fs.realpath(defaultCodexDir())) } catch { /* 未作成 */ }
+  let st = null
+  let realpath = null
+  try {
+    st = await fs.lstat(configDir)
+    realpath = await fs.realpath(configDir)
+  } catch (err) {
+    if (err?.code !== "ENOENT") throw deletionError("io", `configDir の確認に失敗しました: ${err.message}`, err)
+  }
+  if ((st && (!st.isDirectory() || st.isSymbolicLink())) || defaults.has(configDir)
+    || (realpath !== null && defaults.has(realpath))) {
+    throw deletionError("unsafe_path", `安全でない configDir です: ${configDir}`)
+  }
+  let configDirRemoved = false
+  if (st) {
+    try { await fs.rm(configDir, { recursive: true, force: true }); configDirRemoved = true }
+    catch (err) { throw deletionError("io", `configDir の削除に失敗しました: ${err.message}`, err) }
+  }
+  const updated = { ...reg, profiles: reg.profiles.filter((p) => p.id !== id) }
+  try { await saveRegistry(updated) } catch (err) { throw deletionError("io", `レジストリの保存に失敗しました: ${err.message}`, err) }
+  return { id: target.id, label: target.label, configDir, configDirRemoved, realpath }
+}
+
 /** 次の連番 ID (既存の数値 id の最大 +1、最低 1)。 */
 function nextNumericId(profiles) {
+  // 削除済み id の再利用は安全: Codex の認証は configDir 内の auth.json だけで、
+  // deleteProfile はディレクトリ削除が成功した後にしかレジストリから id を外さない
+  // (不変条件: 認証を残したまま id が解放されることはない)。
   let max = 0
   for (const p of profiles) {
     const n = Number.parseInt(p.id, 10)

package/src/dispatch-scheduler.mjs

@@ -69,13 +69,14 @@ function chatMode(type) {
 }
 
 function globalMutationMode(type) {
-  if (type === "profile.switch" || type === "profile.add" || type === "profile.rename") {
+  if (type === "profile.switch" || type === "profile.add" || type === "profile.rename" || type === "profile.delete") {
     return "claude"
   }
   if (
     type === "codex.profile.switch" ||
     type === "codex.profile.add" ||
-    type === "codex.profile.rename"
+    type === "codex.profile.rename" ||
+    type === "codex.profile.delete"
   ) {
     return "codex"
   }

package/src/main.mjs

@@ -82,6 +82,7 @@ import {
   setActiveProfile,
   addProfile,
   renameProfile,
+  deleteProfile,
 } from "./profiles.mjs"
 // Codex アカウント・プロファイル (独立切替)。profiles.mjs と同名 export が多いため
 // 名前衝突を避けて namespace import する。
@@ -91,6 +92,7 @@ import {
   hasFileCredentialsCached,
   isKeychainPlatform,
   profileHasKeychainCredentials,
+  deleteKeychainCredentials,
 } from "./claude-credentials.mjs"
 import {
   buildClaudeCmd,
@@ -5046,6 +5048,30 @@ export async function dispatch(msg, ctx) {
         }
         return
       }
+      case "profile.delete": {
+        const id = msg.profile_id || ""
+        try {
+          const profileApi = ctx.profiles || { deleteProfile }
+          let keychainDeleted = false
+          const deleted = await profileApi.deleteProfile(id, {
+            beforeRemove: async (target, realpath) => {
+              keychainDeleted = (ctx.isKeychainPlatform || isKeychainPlatform)()
+                ? (await (ctx.deleteKeychainCredentials || deleteKeychainCredentials)(target.configDir, { realpath })) > 0
+                : false
+            },
+          })
+          ;(ctx.clearCredentialCheckCache || clearCredentialCheckCache)(deleted.configDir)
+          ctx.logger.info({ profile_id: id, config_dir_removed: deleted.configDirRemoved, keychain_deleted: keychainDeleted }, "claude profile deleted")
+          ctx.client.send({
+            type: "profile.delete.result", request_id: msg.request_id, ok: true,
+            profile_id: id, config_dir_removed: deleted.configDirRemoved, keychain_deleted: keychainDeleted,
+          })
+        } catch (err) {
+          ctx.client.send({ type: "profile.delete.result", request_id: msg.request_id, ok: false,
+            profile_id: id, reason: err?.code || "io", error: err?.message || String(err) })
+        }
+        return
+      }
       case "codex.profile.list": {
         // Codex アカウント・プロファイル一覧と active を返す (Cockpit の Codex 切替UI用)。
         // 認証はファイル (<CODEX_HOME>/auth.json) の有無で判定 (Keychain 不要)。
@@ -5183,6 +5209,20 @@ export async function dispatch(msg, ctx) {
         }
         return
       }
+      case "codex.profile.delete": {
+        const id = msg.profile_id || ""
+        try {
+          const profileApi = ctx.codexProfiles || codexProfiles
+          const deleted = await profileApi.deleteProfile(id)
+          ctx.logger.info({ profile_id: id, config_dir_removed: deleted.configDirRemoved }, "codex profile deleted")
+          ctx.client.send({ type: "codex.profile.delete.result", request_id: msg.request_id, ok: true,
+            profile_id: id, config_dir_removed: deleted.configDirRemoved, keychain_deleted: false })
+        } catch (err) {
+          ctx.client.send({ type: "codex.profile.delete.result", request_id: msg.request_id, ok: false,
+            profile_id: id, reason: err?.code || "io", error: err?.message || String(err) })
+        }
+        return
+      }
       case "tmux.exec": {
         // ロジックは handleTmuxExec へ抽出 (テストから直接呼べる)。
         await handleTmuxExec(msg, ctx)

package/src/profiles.mjs

@@ -114,19 +114,22 @@ export async function loadRegistry() {
 
 /** レジストリを永続化する。default は profiles[] に含めず、追加分と activeId / defaultLabel だけ保存。 */
 async function saveRegistry(reg) {
-  await ensureConfigDir()
-  const def = reg.profiles.find((p) => p.id === DEFAULT_PROFILE_ID)
-  const payload = {
-    activeId: reg.activeId,
-    defaultLabel: def ? def.label : undefined,
-    profiles: reg.profiles
-      .filter((p) => p.id !== DEFAULT_PROFILE_ID)
-      .map((p) => ({ id: p.id, label: p.label, configDir: p.configDir })),
+  try {
+    await ensureConfigDir()
+    const def = reg.profiles.find((p) => p.id === DEFAULT_PROFILE_ID)
+    const payload = {
+      activeId: reg.activeId,
+      defaultLabel: def ? def.label : undefined,
+      profiles: reg.profiles
+        .filter((p) => p.id !== DEFAULT_PROFILE_ID)
+        .map((p) => ({ id: p.id, label: p.label, configDir: p.configDir })),
+    }
+    await fs.writeFile(registryPath(), JSON.stringify(payload, null, 2), {
+      mode: 0o600,
+    })
+  } finally {
+    _cache = null
   }
-  await fs.writeFile(registryPath(), JSON.stringify(payload, null, 2), {
-    mode: 0o600,
-  })
-  _cache = null
 }
 
 /** プロファイル一覧と active を返す (UI 表示用)。 */
@@ -231,8 +234,74 @@ export async function renameProfile(id, label) {
   return { id: target.id, label: target.label, configDir: target.configDir }
 }
 
+function deletionError(code, message, cause) {
+  const err = new Error(message, cause ? { cause } : undefined)
+  err.code = code
+  return err
+}
+
+/**
+ * 非既定・非 active プロファイルの実体と登録を安全に削除する。
+ * @param {string} id
+ * @param {{beforeRemove?: (target: {id:string,label:string,configDir:string}, realpath: string|null) => Promise<void>}} [opts]
+ */
+export async function deleteProfile(id, opts = {}) {
+  if (id === DEFAULT_PROFILE_ID) throw deletionError("default", "既定プロファイルは削除できません")
+  const reg = await loadRegistry()
+  const target = reg.profiles.find((p) => p.id === id)
+  if (!target) throw deletionError("unknown", `未知のプロファイルです: ${id}`)
+  if (reg.activeId === id) {
+    throw deletionError("active", "アクティブなプロファイルです。先に別のアカウントへ切り替えてください")
+  }
+
+  const configDir = target.configDir
+  const expected = `.claude-${id}`
+  if (!path.isAbsolute(configDir) || path.dirname(configDir) !== os.homedir()
+    || path.basename(configDir) !== expected || !/^\.claude-\d+$/.test(expected)) {
+    throw deletionError("unsafe_path", `安全でない configDir です: ${configDir}`)
+  }
+  const defaults = new Set([defaultConfigDir()])
+  try { defaults.add(await fs.realpath(defaultConfigDir())) } catch { /* 未作成 */ }
+  let realpath = null
+  let st = null
+  try {
+    st = await fs.lstat(configDir)
+    realpath = await fs.realpath(configDir)
+  } catch (err) {
+    if (err?.code !== "ENOENT") throw deletionError("io", `configDir の確認に失敗しました: ${err.message}`, err)
+  }
+  if ((st && (!st.isDirectory() || st.isSymbolicLink())) || defaults.has(configDir)
+    || (realpath !== null && defaults.has(realpath))) {
+    throw deletionError("unsafe_path", `安全でない configDir です: ${configDir}`)
+  }
+  if (opts.beforeRemove) {
+    try {
+      await opts.beforeRemove({ id: target.id, label: target.label, configDir }, realpath)
+    } catch (err) {
+      if (!err.code) err.code = "io"
+      throw err
+    }
+  }
+  let configDirRemoved = false
+  if (st) {
+    try {
+      await fs.rm(configDir, { recursive: true, force: true })
+      configDirRemoved = true
+    } catch (err) {
+      throw deletionError("io", `configDir の削除に失敗しました: ${err.message}`, err)
+    }
+  }
+  const updated = { ...reg, profiles: reg.profiles.filter((p) => p.id !== id) }
+  try { await saveRegistry(updated) } catch (err) {
+    throw deletionError("io", `レジストリの保存に失敗しました: ${err.message}`, err)
+  }
+  return { id: target.id, label: target.label, configDir, configDirRemoved, realpath }
+}
+
 /** 次の連番 ID (既存の数値 id の最大 +1、最低 1)。 */
 function nextNumericId(profiles) {
+  // 削除は Keychain → ディレクトリ → レジストリの順で、認証が残ったまま id が
+  // 解放されることはない (deleteProfile の beforeRemove)。
   let max = 0
   for (const p of profiles) {
     const n = Number.parseInt(p.id, 10)