@cocorograph/hub-agent 0.7.24 → 0.7.26

package/package.json

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

package/src/claude-history.mjs

@@ -341,6 +341,87 @@ export async function listSessions({ cwd, projectsRoot, limit = 30, logger }) {
  * @param {{viewingSessionId?: string|null, newestSessionId?: string|null, lastNotifiedNewId?: string|null, lastNotifiedAt?: number|null, now?: number|null, reNotifyMs?: number}} args
  * @returns {{rotated: boolean, newSessionId?: string}}
  */
+/**
+ * 回転候補 jsonl が「bound TUI セッションの本物の /clear 先」か「別アクティビティの jsonl」かを
+ * head 4KB だけ読んで判定する純粋ヘルパ (2a 根治)。
+ *
+ * 背景: 同一 cwd-encoded dir には bound TUI 以外の jsonl が混入し得る:
+ *  - Task tool subagent (`parent_uuid` / `parentUuid` 付き)
+ *  - SDK headless `claude -p` (末尾近傍に `type=result`)
+ *  - 並走 cockpit / 別ターミナル起動の別 TUI (cwd は一致するため弱い signal)
+ *  - 既存 jsonl が touch されて mtime 最新化 (cwd が viewCwd と不一致なケースは cross-cwd 汚染)
+ * これらに対して decideSessionRotation は newest≠viewing で無条件発火するため、ローテに乗ると
+ * viewer が別 jsonl へ rotate-bind されて /clear 後の三点リーダー固着・誤生成中表示を起こす。
+ *
+ * 検出基準 (どれか 1 つでも該当したら reject = 抑止):
+ *  - `parent_uuid` / `parentUuid` / `parent_session_id` / `parentSessionId` (subagent marker)
+ *  - `type === 'result'` (SDK headless の完了 marker。先頭 4KB 内に現れたら頭 0KB の極短セッション)
+ *  - `cwd` フィールドが `viewCwd` と完全不一致 (cross-cwd 汚染)
+ *
+ * 失敗安全: head 読み失敗 / parse 失敗時は accept (旧挙動 = 即時 rotate へ degrade)。
+ * 「/clear が壊れる」リスクは取らないため、不確実時は安全側 (accept) に倒す。
+ *
+ * @param {{candidatePath?: string|null, viewCwd?: string|null, readImpl?: (p:string,n:number)=>Promise<string|null>, logger?: import('pino').Logger}} args
+ * @returns {Promise<{accept: boolean, reason: string}>}
+ */
+export async function validateRotationCandidate({
+  candidatePath,
+  viewCwd,
+  readImpl,
+  logger,
+} = {}) {
+  if (!candidatePath || !viewCwd) {
+    return { accept: true, reason: "missing_args" }
+  }
+  let text
+  try {
+    if (typeof readImpl === "function") {
+      text = await readImpl(candidatePath, 4096)
+    } else {
+      const handle = await open(candidatePath, "r")
+      try {
+        const buf = Buffer.allocUnsafe(4096)
+        const { bytesRead } = await handle.read(buf, 0, buf.length, 0)
+        text = buf.toString("utf-8", 0, bytesRead)
+      } finally {
+        await handle.close().catch(() => {})
+      }
+    }
+  } catch (err) {
+    logger?.warn(
+      { err: err?.message, candidatePath },
+      "rotation candidate head read failed → accept (degrade to legacy behavior)",
+    )
+    return { accept: true, reason: `read_failed:${err?.code || "unknown"}` }
+  }
+  if (!text) return { accept: true, reason: "empty_text" }
+  for (const line of text.split("\n")) {
+    if (!line) continue
+    let obj
+    try {
+      obj = JSON.parse(line)
+    } catch {
+      continue
+    }
+    if (!obj || typeof obj !== "object") continue
+    if (
+      obj.parent_uuid ||
+      obj.parentUuid ||
+      obj.parent_session_id ||
+      obj.parentSessionId
+    ) {
+      return { accept: false, reason: "subagent_marker" }
+    }
+    if (obj.type === "result") {
+      return { accept: false, reason: "headless_result_marker" }
+    }
+    if (typeof obj.cwd === "string" && obj.cwd && obj.cwd !== viewCwd) {
+      return { accept: false, reason: "cwd_mismatch" }
+    }
+  }
+  return { accept: true, reason: "passed" }
+}
+
 export function decideSessionRotation({
   viewingSessionId,
   newestSessionId,

package/src/config.mjs

@@ -10,6 +10,10 @@
 import { promises as fs, constants as fsConsts, readFileSync } from "node:fs"
 import path from "node:path"
 import os from "node:os"
+import { execFile } from "node:child_process"
+import { promisify } from "node:util"
+
+const _execFileP = promisify(execFile)
 
 /**
  * 実行プラットフォームを返す。Hub の UI が OS 別コマンドを出し分けるために使う。
@@ -35,6 +39,34 @@ export function detectPlatform() {
   return process.platform || "unknown"
 }
 
+/** このホストで起動可否を検出する CLI 種別の候補。 */
+export const CLI_CANDIDATES = ["claude", "codex"]
+
+/**
+ * PATH 上に存在する CLI 種別 (capability) を検出する。Cockpit が「この hub-agent で
+ * どの CLI を起動できるか」を知るため、起動時に 1 回呼んで enroll/hello payload で
+ * Hub に広告する (`CockpitAgent.available_clis`)。which/where が成功したものだけ返す。
+ *
+ * @param {{candidates?:string[], exec?:(cmd:string,args:string[])=>Promise<any>}} [opts]
+ *   exec はテスト差し替え用 (既定は which/where を叩く promisified execFile)。
+ * @returns {Promise<string[]>} 例: ["claude", "codex"]
+ */
+export async function detectAvailableClis(opts = {}) {
+  const candidates = opts.candidates || CLI_CANDIDATES
+  const exec = opts.exec || _execFileP
+  const lookup = process.platform === "win32" ? "where" : "which"
+  const found = []
+  for (const cli of candidates) {
+    try {
+      await exec(lookup, [cli])
+      found.push(cli)
+    } catch {
+      /* PATH に無い = 未インストール */
+    }
+  }
+  return found
+}
+
 function resolveConfigDir() {
   if (process.env.HUB_AGENT_CONFIG_DIR) return process.env.HUB_AGENT_CONFIG_DIR
   return path.join(os.homedir(), ".hub")

package/src/enroll.mjs

@@ -8,7 +8,7 @@
  */
 import os from "node:os"
 
-import { detectPlatform, hasConfig, writeConfig } from "./config.mjs"
+import { detectAvailableClis, detectPlatform, hasConfig, writeConfig } from "./config.mjs"
 
 const DEFAULT_HUB_URL = process.env.HUB_URL || "https://hub.cocorograph.com"
 
@@ -34,6 +34,8 @@ export async function enroll(enrollmentToken, opts = {}) {
     throw new Error("global fetch がありません。Node 20+ で実行してください。")
   }
 
+  const availableClis = await detectAvailableClis()
+
   const res = await fetchImpl(`${hubUrl}/api/cockpit/agents/enroll-complete/`, {
     method: "POST",
     headers: { "content-type": "application/json" },
@@ -42,6 +44,7 @@ export async function enroll(enrollmentToken, opts = {}) {
       hostname,
       version,
       platform: detectPlatform(),
+      available_clis: availableClis,
     }),
   })
 

package/src/main.mjs

@@ -19,7 +19,7 @@ import path from "node:path"
 
 import pino from "pino"
 
-import { detectPlatform, readConfig, writeConfig } from "./config.mjs"
+import { detectAvailableClis, detectPlatform, readConfig, writeConfig } from "./config.mjs"
 import { extractLastAssistantText, pickContextText } from "./extract-paragraph.mjs"
 import { loadPlugins, runHookBroadcast, runHookChain } from "./plugin-loader.mjs"
 import { WsClient } from "./ws-client.mjs"
@@ -30,7 +30,9 @@ import { requestSelfUninstall } from "./service-install.mjs"
 import {
   decideSessionRotation,
   fetchSessionHistory,
+  jsonlPath,
   listSessions,
+  validateRotationCandidate,
 } from "./claude-history.mjs"
 import { listAgents } from "./agents.mjs"
 import { listSkills } from "./skills.mjs"
@@ -440,10 +442,15 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     logger.info({ bundleVersion }, "hub bundle version detected")
   }
 
+  // ホストで起動可能な CLI (claude / codex) を 1 回検出して hello で広告する。
+  const availableClis = await detectAvailableClis()
+  logger.info({ availableClis }, "available CLIs detected")
+
   const client = new WsClient(config, {
     logger,
     version,
     platform: detectPlatform(),
+    availableClis,
     bundleVersion,
     bundleVersionProvider: readBundleVersionSync,
     bundleManifestPath: BUNDLE_MANIFEST_PATH,
@@ -2240,6 +2247,39 @@ async function dispatch(msg, ctx) {
                 return
               }
               if (!rotated) return
+
+              // 2a 根治: rotate-bind の前に「候補が本物の /clear 先か」を head 4KB で検証する。
+              // subagent (parent_uuid 等) / SDK headless (type=result) / cwd 不一致のものは
+              // 抑止し、別アクティビティ jsonl への汚染 bind を断つ。検証 NG 時は capped と同様に
+              // tuiRotationNotified を進めて連発を防ぐ。検証失敗 (read error) は accept = 旧挙動。
+              const candidatePath = jsonlPath({
+                cwd: viewCwd,
+                session_id: newSessionId,
+                projectsRoot,
+              })
+              const { accept, reason } = await validateRotationCandidate({
+                candidatePath,
+                viewCwd,
+                logger,
+              })
+              if (!accept) {
+                ctx.tuiRotationNotified.set(key, {
+                  newId: newSessionId,
+                  ts: now,
+                  count: newSessionId === prevNewId ? prevCount + 1 : 1,
+                })
+                logger.warn(
+                  {
+                    session: viewName,
+                    cwd: viewCwd,
+                    view: viewSid,
+                    newest: newestId,
+                    reason,
+                  },
+                  "tui session rotation: suppressed (sibling activity / not a real /clear)",
+                )
+                return
+              }
               // {newId, ts, count} で保存。同一 new への再通知をスロットリング+回数キャップする。
               // newId が変われば count を 1 にリセット、同一なら加算する。
               ctx.tuiRotationNotified.set(key, {
@@ -3189,10 +3229,19 @@ async function dispatch(msg, ctx) {
         }
         try {
           await createTmuxSession(name, cwd, {
+            // cli_kind: 'codex' なら codex を、それ以外は claude を pane 起動する (既定 claude)。
+            cliKind: msg.cli_kind === "codex" ? "codex" : "claude",
             claudeCmd:
               typeof msg.claude_cmd === "string"
                 ? msg.claude_cmd
                 : claudeCmdFromAgentConfig(ctx.config),
+            codexCmd: typeof msg.codex_cmd === "string" ? msg.codex_cmd : undefined,
+            // codex 用設定 (claude 経路では claudeCmd を直接使うため無視される)。
+            model: typeof msg.model === "string" ? msg.model : undefined,
+            sandbox: typeof msg.sandbox === "string" ? msg.sandbox : undefined,
+            approvalPolicy:
+              typeof msg.approval_policy === "string" ? msg.approval_policy : undefined,
+            effort: typeof msg.effort === "string" ? msg.effort : undefined,
             initialPrompt:
               typeof msg.initial_prompt === "string" ? msg.initial_prompt : undefined,
             logger: ctx.logger,

package/src/tmux.mjs

@@ -840,16 +840,29 @@ export async function createSession(name, cwd, opts = {}) {
       // display-message 失敗は致命的でないので飲み込む
     }
   }
-  const claudeCmdBase = opts.claudeCmd ?? DEFAULT_CLAUDE_CMD
-  // アカウント・プロファイル切替 (CLAUDE_CONFIG_DIR) を TUI セッションにも効かせる。
-  // applyActiveProfileEnv が process.env を最新に保つため、既定では env から拾う。
-  const configDir = opts.claudeConfigDir ?? process.env.CLAUDE_CONFIG_DIR ?? null
-  const claudeCmd = injectInitialPrompt(
-    injectConfigDirEnv(claudeCmdBase, configDir),
-    opts.initialPrompt,
-  )
-  if (claudeCmd) {
-    await execFileP(tmuxBin(opts), ["send-keys", "-t", name, claudeCmd, "Enter"])
+  // cliKind に応じて claude / codex いずれの CLI を pane で起動するか分岐する。
+  // 既定は claude（後方互換）。initialPrompt はどちらも位置引数として末尾注入する
+  // (claude "<p>" / codex "<p>" / codex resume <id> "<p>" いずれも [PROMPT] を受ける)。
+  const launchCmd =
+    opts.cliKind === "codex"
+      ? injectInitialPrompt(
+          injectCodexHomeEnv(
+            opts.codexCmd ?? buildCodexCmd(opts),
+            opts.codexHome ?? process.env.CODEX_HOME ?? null,
+          ),
+          opts.initialPrompt,
+        )
+      : injectInitialPrompt(
+          // アカウント・プロファイル切替 (CLAUDE_CONFIG_DIR) を TUI セッションにも効かせる。
+          // applyActiveProfileEnv が process.env を最新に保つため、既定では env から拾う。
+          injectConfigDirEnv(
+            opts.claudeCmd ?? DEFAULT_CLAUDE_CMD,
+            opts.claudeConfigDir ?? process.env.CLAUDE_CONFIG_DIR ?? null,
+          ),
+          opts.initialPrompt,
+        )
+  if (launchCmd) {
+    await execFileP(tmuxBin(opts), ["send-keys", "-t", name, launchCmd, "Enter"])
   }
 }
 
@@ -932,6 +945,99 @@ export function buildFreshCmd(opts = {}) {
   return `claude${flags ? " " + flags : ""}`.trim()
 }
 
+// ---------------------------------------------------------------------------
+// Codex CLI 起動コマンドビルダ (claude 版と同形の純粋関数)
+//
+// 設計メモ:
+//   - codex の対話起動フラグ (v0.142.x 実機確認): `-m/--model` / `-s/--sandbox`
+//     {read-only|workspace-write|danger-full-access} / `-a/--ask-for-approval`
+//     {untrusted|on-request|never} / `-C/--cd` / `-p/--profile`。
+//   - 値は安全なトークン (モデル名・enum) を前提に bare で並べる (claude 版と同方針)。
+//   - codex は claude の `--continue` のような **cwd スコープ継続を持たない**
+//     (rollout は ~/.codex/sessions/Y/M/D に日付シャードで保存され cwd 非依存)。
+//     そのため既定は fresh 起動とし、継続は明示 session_id で resume する。
+//   - アカウント/プロファイル切替は CLAUDE_CONFIG_DIR に対応する `CODEX_HOME` の
+//     ディレクトリ切替で行う (injectCodexHomeEnv)。
+// ---------------------------------------------------------------------------
+
+/**
+ * agent 設定から codex 起動 flags を組み立てる (純粋関数)。
+ * model / sandbox / approvalPolicy / effort のうち指定されたものだけを並べる。
+ * @param {{model?:string,sandbox?:string,approvalPolicy?:string,effort?:string}} [opts]
+ * @returns {string}
+ */
+export function composeCodexFlags(opts = {}) {
+  const model = (opts.model || "").trim()
+  const sandbox = (opts.sandbox || "").trim()
+  const approval = (opts.approvalPolicy || "").trim()
+  const effort = (opts.effort || "").trim()
+  const flags = []
+  if (model) flags.push(`--model ${model}`)
+  if (sandbox) flags.push(`--sandbox ${sandbox}`)
+  if (approval) flags.push(`--ask-for-approval ${approval}`)
+  // effort はフラグが無いため -c 設定上書きで渡す (TOML 解析失敗時は raw 文字列)。
+  if (effort) flags.push(`-c model_reasoning_effort=${effort}`)
+  return flags.join(" ")
+}
+
+/**
+ * codex 既定起動コマンド (claude の buildClaudeCmd 相当)。
+ * hub オプション (model/sandbox/approval/effort) があれば反映、無ければ
+ * HUB_CODEX_CMD 環境変数、どちらも無ければ最小デフォルト `codex`。
+ * @param {{model?:string,sandbox?:string,approvalPolicy?:string,effort?:string}} [opts]
+ * @returns {string}
+ */
+export function buildCodexCmd(opts = {}) {
+  const flags = composeCodexFlags(opts)
+  if (flags) return `codex ${flags}`.trim()
+  if (process.env.HUB_CODEX_CMD) return process.env.HUB_CODEX_CMD
+  return "codex"
+}
+
+/**
+ * まっさらな新規 codex セッション起動用コマンド (純粋関数)。
+ * resume を付けないので codex は新しい rollout で起動する。
+ * @param {{model?:string,sandbox?:string,approvalPolicy?:string,effort?:string}} [opts]
+ * @returns {string}
+ */
+export function buildCodexFreshCmd(opts = {}) {
+  const flags = composeCodexFlags(opts)
+  return `codex${flags ? " " + flags : ""}`.trim()
+}
+
+/**
+ * TUI 載せ替え用の `codex resume <id> [flags]` コマンド (純粋関数)。
+ * session_id を明示 resume することで表示と書込の rollout を一致させる。
+ * @param {string} sessionId codex セッション id (UUID 等)
+ * @param {{model?:string,sandbox?:string,approvalPolicy?:string,effort?:string}} [opts]
+ * @returns {string}
+ * @throws session_id が安全形でなければ throw (コマンド注入防止)
+ */
+export function buildCodexResumeCmd(sessionId, opts = {}) {
+  if (!isSafeSessionId(sessionId)) {
+    throw new Error(`unsafe session_id for codex resume: ${String(sessionId)}`)
+  }
+  const flags = composeCodexFlags(opts)
+  return `codex resume ${sessionId}${flags ? " " + flags : ""}`.trim()
+}
+
+/**
+ * codexCmd の各セグメント先頭に `CODEX_HOME=<dir>` 環境変数代入を注入する
+ * (claude の injectConfigDirEnv 相当)。codexHome が空なら何もしない。
+ * @param {string} codexCmd
+ * @param {string} codexHome
+ * @returns {string}
+ */
+export function injectCodexHomeEnv(codexCmd, codexHome) {
+  if (!codexHome || typeof codexHome !== "string") return codexCmd
+  if (typeof codexCmd !== "string" || codexCmd.length === 0) return codexCmd
+  const assignment = `CODEX_HOME=${JSON.stringify(codexHome)}`
+  return codexCmd
+    .split("||")
+    .map((part) => `${assignment} ${part.trim()}`)
+    .join(" || ")
+}
+
 const _delay = (ms) => new Promise((r) => setTimeout(r, ms))
 
 /**
@@ -1772,7 +1878,12 @@ export async function rebindClaudeSession(name, sessionId, opts = {}) {
   let cmd
   try {
     // fresh=新規起動 (--resume 無し)。それ以外は session_id 検証込みの resume。
-    cmd = opts.fresh ? buildFreshCmd(opts) : buildResumeCmd(sessionId, opts)
+    // cliKind に応じて claude / codex のビルダを使い分ける (既定は claude)。
+    if (opts.cliKind === "codex") {
+      cmd = opts.fresh ? buildCodexFreshCmd(opts) : buildCodexResumeCmd(sessionId, opts)
+    } else {
+      cmd = opts.fresh ? buildFreshCmd(opts) : buildResumeCmd(sessionId, opts)
+    }
   } catch (err) {
     return { ok: false, error: err?.message || String(err) }
   }

package/src/ws-client.mjs

@@ -113,6 +113,8 @@ export class WsClient extends EventEmitter {
     this.bundleWatchDebounceTimer = null
     this.hostname = opts.hostname || os.hostname()
     this.platform = opts.platform || "unknown"
+    // このホストで起動可能な CLI 種別 (capability)。起動時に 1 回検出して hello で広告する。
+    this.availableClis = Array.isArray(opts.availableClis) ? opts.availableClis : []
     this.ws = null
     this.heartbeatTimer = null
     this.reconnectTimer = null
@@ -226,6 +228,7 @@ export class WsClient extends EventEmitter {
       version: this.version,
       bundle_version: this.bundleVersion,
       platform: this.platform,
+      available_clis: this.availableClis,
     })
     // Plan ε: 毎回 reconnect 直後に backend ↔ agent の stream_id を能動同期する
     // (orphan stream の即時 kill 用)。response は main.mjs の dispatch が拾い、