@cocorograph/hub-agent 0.8.76 → 0.8.78

package/src/sibling-inject.mjs

@@ -80,6 +80,7 @@ import path from "node:path"
 import { verifySiblingTokenAny, siblingTokenCwd } from "./sibling-token.mjs"
 import { describeLiveSessionsUnion } from "./live-sessions.mjs"
 import { ReviewEventStore, validReviewEvent } from "./review-event-store.mjs"
+import { listChatTargets } from "./chat-targets.mjs"
 
 export const SIBLING_REQUESTS_DIR =
   process.env.COCKPIT_SIBLING_REQUESTS_DIR || "/tmp/cockpit_sibling_requests"
@@ -431,6 +432,7 @@ export class SiblingInjectBridge {
     pidStartedAt = defaultPidStartedAt,
     onRestartRequested,
     isInputBlocked,
+    starter,
     watchFn = watch,
   } = {}) {
     this.dir = dir
@@ -447,6 +449,7 @@ export class SiblingInjectBridge {
     this._pidStartedAt = pidStartedAt
     this._onRestartRequested = onRestartRequested
     this._isInputBlocked = isInputBlocked
+    this._starter = starter
     this._watchFn = watchFn
     this._newTurnInFlight = 0
     /** @type {Map<string, Map<string, {event: object, recordedAt: number}>>} cwd+session ごとの進行中委譲。 */
@@ -932,7 +935,10 @@ export class SiblingInjectBridge {
         result = { ok: false, error: "invalid_token" }
       } else {
         // 判定は live-sessions.mjs に一本化する (tmux.list_sessions の warm と同じ関数)。
-        result = { ok: true, live_sessions: describeLiveSessionsUnion(this._bridges()) }
+        const live_sessions = describeLiveSessionsUnion(this._bridges())
+        const { targets, errors } = await listChatTargets({ logger: this.logger })
+        const registered_targets = targets.map(({ cwd, kind, name }) => ({ cwd, kind, name }))
+        result = { ok: true, live_sessions, registered_targets, errors }
       }
     } else if (!cwd || (!isDelegateEvent && badKind) || badBody) {
       result = { ok: false, error: "invalid_request" }
@@ -1113,7 +1119,29 @@ export class SiblingInjectBridge {
         //    **実配送先 (r.session)** を返す。
         const r = bridge?.sendToCwd
           ? await this._sendNewTurn(
-            () => bridge.sendToCwd(cwd, text, injectSource, request_id),
+            async () => {
+              const first = await bridge.sendToCwd(cwd, text, injectSource, request_id)
+              if (first.status !== "not_found" || target_session_id || !this._starter) return first
+              const started = await this._starter({ cwd, cliKind: target_cli_kind })
+              // ⚠️ 起動側の失敗理由をここで捨てない。捨てると送信元には最後の
+              // `no_live_session_for_cwd` しか届かず、「準備中でもう一度送れば届く」と
+              // 「その cwd には恒久的に会話が無い」を区別できない。配送結果そのものは
+              // not_found のままなので、診断情報として添えて下流の応答へ持ち上げる。
+              if (!started?.ok) {
+                return started?.error
+                  ? { ...first, startup_error: started.error, startup_retryable: started.retryable === true }
+                  : first
+              }
+              // ⚠️ 起動側は cwd を展開・解決してから会話を作る (chat-session-starter の
+              // expandTilde と prepareChatWorkspace)。ここで展開前の cwd のまま引き直すと、
+              // 実際に作られた会話 (解決後 cwd) と一致せず、起こした直後に not_found へ落ちる。
+              // 解決後の cwd は起動側だけが知っているので、その戻り値を使う。
+              const startedCwd = typeof started.cwd === "string" && started.cwd ? started.cwd : cwd
+              const retry = await bridge.sendToCwd(startedCwd, text, injectSource, request_id)
+              // 起動後の曖昧さは候補案内へ渡し、不在のときだけ最初の診断へ戻す。
+              if (retry.status === "not_found") return first
+              return retry
+            },
             { request_id, cwd, target_cli_kind, op: "normal" },
           )
           : { status: "not_found" }
@@ -1159,7 +1187,16 @@ export class SiblingInjectBridge {
             ...ambiguousTargetOutcome(target_cli_kind, undefined, r.candidates),
           }
         } else {
-          result = { ok: false, error: "no_live_session_for_cwd", ...this._diagnose(cwd, target_cli_kind) }
+          result = {
+            ok: false,
+            error: "no_live_session_for_cwd",
+            ...this._diagnose(cwd, target_cli_kind),
+            // ⚠️ `retryable` を載せるのは自動起動が「配送前に」失敗した場合だけ。配送を試みた
+            // 結果が不明な要求に再試行印を付けると、送信元の自動再送で二重配送になる。
+            ...(r.startup_error
+              ? { startup_error: r.startup_error, retryable: r.startup_retryable === true }
+              : {}),
+          }
         }
       }
     }

package/src/tmux.mjs

@@ -19,9 +19,8 @@ import os from "node:os"
 import path from "node:path"
 import { promisify } from "node:util"
 
-import { ensureClaudeMd } from "./claude-md.mjs"
-import { syncClaudeMdWithHub } from "./claude-md-sync.mjs"
-import { createSessionBudget } from "./hub-request-deadline.mjs"
+import { expandTilde, hubProjectsBase, prepareChatWorkspace } from "./chat-workspace.mjs"
+export { expandTilde, isUnderHubProjectsBase, projectInstructionDir } from "./chat-workspace.mjs"
 import { detectSessionState, getSessionCwd } from "./state.mjs"
 import { shouldLaunchPaneCli } from "./pane-cli-flag.mjs"
 import { baseSessionName, sessionNameFor } from "./session-names.mjs"
@@ -32,61 +31,10 @@ const execFileP = promisify(execFile)
 
 const DEFAULT_TMUX_BIN = "tmux"
 
-// HUB_PROJECTS_BASE は実行時に解決する。テストや起動環境で
-// `process.env.HUB_PROJECTS_BASE` を上書きできるよう、import 時に
-// const として固定しない (tests/tmux.test.mjs が動的に差し替えるため)。
-function hubProjectsBase() {
-  return process.env.HUB_PROJECTS_BASE || path.join(os.homedir(), "hub", "projects")
-}
-
-function sanitizeTmuxName(s) {
+export function sanitizeTmuxName(s) {
   return s.replace(/[.:]/g, "-")
 }
 
-/**
- * 先頭の `~` / `~/` を `os.homedir()` に展開する。
- *
- * tmux は `-c` に渡された cwd をシェル展開しないため、フロントから
- * 受け取った `~/hub/projects/...` を絶対パスに変換する必要がある。
- * 既に絶対パス / 相対パスなら noop。
- */
-export function expandTilde(p) {
-  if (typeof p !== "string" || p.length === 0) return p
-  if (p === "~") return os.homedir()
-  if (p.startsWith("~/")) return path.join(os.homedir(), p.slice(2))
-  return p
-}
-
-/**
- * 絶対パス `p` が HUB_PROJECTS_BASE 配下かを判定する。
- *
- * `createSession` の cwd 自動作成判定で使う。base そのもの (`~/hub/projects`)
- * は対象外とし、配下のサブディレクトリのみ true を返す。`path.relative` で
- * `..` を含まないかチェックすることで `/Users/.../hub/projects-backup` のような
- * 接頭辞一致のみの誤検知を排除する。
- */
-export function isUnderHubProjectsBase(p) {
-  if (typeof p !== "string" || p.length === 0) return false
-  const abs = path.resolve(p)
-  const base = path.resolve(hubProjectsBase())
-  if (abs === base) return false
-  const rel = path.relative(base, abs)
-  if (!rel || rel.startsWith("..") || path.isAbsolute(rel)) return false
-  return true
-}
-
-/**
- * Hub workspace またはその worktree から、案件指示ファイルを置く親 workspace を返す。
- * worktree 内の AGENTS.md / CLAUDE.md を別管理すると Hub 正本が分岐するため、
- * `<workspace>/.claude/worktrees/<name>` で起動しても常に `<workspace>` を対象にする。
- */
-export function projectInstructionDir(p) {
-  if (!isUnderHubProjectsBase(p)) return path.resolve(p)
-  const base = path.resolve(hubProjectsBase())
-  const [workspace] = path.relative(base, path.resolve(p)).split(path.sep)
-  return path.join(base, workspace)
-}
-
 /**
  * git branch 名から worktree dir 名 (= tmux session 名) を導出する。
  * 例: "feature/multi-area" -> "feature-multi-area"
@@ -159,26 +107,31 @@ export async function createWorktreeDir(parentDir, branch) {
 
 /**
  * `~/hub/projects/<project>/.claude/worktrees/<wt>` を走査し、
- * worktree session 名 → 詳細情報の Map を返す。
+ * worktree の配列と走査エラーを返す（同名でも別 cwd なら保持する）。
  *
  * - 各 worktree dir は `.git` ファイル/ディレクトリの存在で git worktree と判定
  * - session 名はディレクトリ名を sanitize したもの (tmux session 名命名規則)
  *
- * 戻り値は `{ parent_session_name, cwd, branch }` の dict。cockpit (cockpit
+ * entries は `{ name, parent_session_name, cwd, branch }`。cockpit (cockpit
  * worktree-discovery PR 1719) が tmux.list_sessions 応答に worktree_stubs として
  * 同梱するために branch / cwd まで返す。`branch` は `git -C <wtPath> branch
  * --show-current` を best-effort で取る。
  *
  * 移植元: D00000_cockpit/webapp/lib/workspaces.ts (findWorktrees)
  */
-export async function buildWorktreeIndex() {
-  const out = new Map()
+export async function listWorktreeEntries() {
+  const entries = []
+  const errors = []
+  const recordError = (scope, path, err) => {
+    if (err?.code !== "ENOENT") errors.push({ scope, path, code: err?.code ?? null, message: err?.message || String(err) })
+  }
   const projectsBase = hubProjectsBase()
   let projects
   try {
     projects = await fs.readdir(projectsBase, { withFileTypes: true })
-  } catch {
-    return out
+  } catch (err) {
+    recordError("projects-base", projectsBase, err)
+    return { entries, errors }
   }
   for (const p of projects) {
     if (!p.isDirectory() || p.name.startsWith(".")) continue
@@ -186,7 +139,8 @@ export async function buildWorktreeIndex() {
     let wts
     try {
       wts = await fs.readdir(wtBase, { withFileTypes: true })
-    } catch {
+    } catch (err) {
+      recordError("workspace", wtBase, err)
       continue
     }
     for (const wt of wts) {
@@ -195,7 +149,8 @@ export async function buildWorktreeIndex() {
       try {
         // .git が存在する = 正規 git worktree
         await fs.stat(path.join(wtPath, ".git"))
-      } catch {
+      } catch (err) {
+        recordError("workspace", path.join(wtPath, ".git"), err)
         continue
       }
       let branch = null
@@ -206,7 +161,7 @@ export async function buildWorktreeIndex() {
         // best-effort: detached / 取得失敗時は null のままにする
       }
       const sessionName = sanitizeTmuxName(wt.name)
-      out.set(sessionName, {
+      entries.push({
         name: sessionName,
         parent_session_name: sanitizeTmuxName(p.name),
         cwd: wtPath,
@@ -214,7 +169,13 @@ export async function buildWorktreeIndex() {
       })
     }
   }
-  return out
+  return { entries, errors }
+}
+
+/** 既存 tmux 向け: 同名は後勝ち、走査エラーは従来どおり握りつぶす。 */
+export async function buildWorktreeIndex() {
+  const { entries } = await listWorktreeEntries()
+  return new Map(entries.map((entry) => [entry.name, entry]))
 }
 
 /**
@@ -310,8 +271,11 @@ async function inspectWorktreeDirty(wtPath) {
  * - 未コミットの変更等で git remove 自体が失敗した場合はそのエラーを上位に投げる
  *
  * @param {string} name
- * @param {{ force?: boolean, tmuxBin?: string }} [opts]
- * @returns {Promise<{ name: string, wt_path: string, dirty?: boolean, changes?: string[], unpushed_commits?: number }>}
+ * @param {{ force?: boolean, tmuxBin?: string, stopSessions?: (wtPath: string) => Promise<number> }} [opts]
+ *   `stopSessions` は **dirty 判定を通過したあと・dir を消す前**に 1 回だけ呼ばれる
+ *   (**停止 API を呼んだ**本数を返す。止まったこと・終了したことの確認は行わない——本文の
+ *   コメント参照)。投げた場合は削除へ進まない。
+ * @returns {Promise<{ name: string, wt_path: string, dirty?: boolean, changes?: string[], unpushed_commits?: number, stop_requested_sessions?: number }>}
  */
 export async function removeWorktree(name, opts = {}) {
   // T05945: frontend は常にベース名を送るが、万一 `<wt>__codex` が来ても worktree index を
@@ -342,6 +306,27 @@ export async function removeWorktree(name, opts = {}) {
     }
   }
 
+  // ⚠️ dir を消す前に、この worktree を cwd とする **会話**へ停止要求を出す (ユーザー決定
+  // 「会話を止めてから削除」)。会話の実体が tmux から離れたので、止める対象もセッションでは
+  // なく会話になる (下の tmux kill が同じ理由で存在する)。stopSessions が投げたらここで
+  // 中断し、dir は消さない。
+  //
+  // ⚠️ ここが保証するのは「**対象の停止 API を呼んだこと**」だけで、「止まったこと」でも
+  // 「プロセスが終了したこと」でもない (2026-09-16 判断)。止まったかどうかを確かめられないのは
+  // bridge が内部の停止失敗を握り潰すため (呼び出し側へ届かない)。終了を待たないのは、待てる
+  // 作りになっていないため——
+  // codex 側は 1 セッションに履歴取得用の一時 client が併存し (codex-stream-bridge.mjs の
+  // historySnapshotFreshOperations)、それらは停止要求後に終了を確認せず集合から外れる。
+  // 全部を終了待ちの対象に通すには bridge の client 管理を作り替えることになる。一方で
+  // 取り逃す害は小さい: stop() は SIGTERM を送ったうえで 3 秒後に SIGKILL を必ず飛ばすので、
+  // 孤児が生きるのは長くても数秒で、その間に消えた dir へ書き込むこともできない。
+  // 「削除の直後に、消えた cwd を指す会話が**新しく**生まれる」方は実害があるので、そちらは
+  // 停止ではなく削除中の attach 拒否 (chat-removal-guard.mjs) で塞いでいる。
+  // ⚠️ この停止を tmux.mjs の中で実装しない。bridge を import すると循環参照になるうえ、
+  // 「どれが生きているか」の正本は live-sessions.mjs 側にある。呼び出し元から注入させる。
+  const stopRequestedSessions =
+    typeof opts.stopSessions === "function" ? (await opts.stopSessions(resolved)) || 0 : 0
+
   // 走行中の session があれば kill。T05945: 同一 worktree に claude (`<wt>`) と codex
   // (`<wt>__codex`) の 2 セッションが並びうるため、両方を kill してから dir を消す
   // (片方が残ったまま dir を消すと、生存セッションの cwd が消滅した状態になる)。
@@ -373,7 +358,7 @@ export async function removeWorktree(name, opts = {}) {
     ? resolved.slice(0, -wtSegment.length - 1)
     : path.resolve(resolved, "..", "..", "..")
   await execFileP("git", ["-C", parentRepo, "worktree", "remove", "--force", resolved])
-  return { name: sanitized, wt_path: resolved, dirty: false }
+  return { name: sanitized, wt_path: resolved, dirty: false, stop_requested_sessions: stopRequestedSessions }
 }
 
 /**
@@ -791,113 +776,7 @@ export async function createSession(name, cwd, opts = {}) {
     launchPaneCli && opts.cliKind === "codex"
       ? opts.codexCmd ?? buildCodexCmd(opts)
       : null
-  // cwd 存在チェック (旧 Cockpit 単体実装と同等の防御)
-  // 無いまま tmux に渡すと pane の current_path が HOME など別ディレクトリに
-  // fallback して、SessionStart hook が project を解決できなくなる。
-  //
-  // 例外: 解決後のパスが HUB_PROJECTS_BASE 配下 (= 案件 workspace ディレクトリ)
-  // なら、未 clone のメンバー Mac でも cockpit から起動できるよう mkdir -p で
-  // 自動生成する。任意の絶対パス (leader cwd 等) や HUB_PROJECTS_BASE 外は
-  // 引き続き `cwd not found` を返し、誤った workspace 起動を防ぐ。
-  try {
-    const st = await fs.stat(resolvedCwd)
-    if (!st.isDirectory()) throw new Error("cwd not a directory")
-  } catch (err) {
-    if (err?.code === "ENOENT") {
-      if (isUnderHubProjectsBase(resolvedCwd)) {
-        try {
-          await fs.mkdir(resolvedCwd, { recursive: true })
-          opts.logger?.info?.(
-            { session: name, cwd: resolvedCwd },
-            "auto-created workspace dir before tmux new-session",
-          )
-        } catch (mkdirErr) {
-          throw new Error(
-            `cwd mkdir failed: ${mkdirErr?.message || String(mkdirErr)}`,
-          )
-        }
-      } else {
-        throw new Error(`cwd not found: ${resolvedCwd}`)
-      }
-    } else if (err?.message === "cwd not a directory") {
-      throw err
-    } else {
-      throw new Error(`cwd stat failed: ${err?.message || String(err)}`)
-    }
-  }
-  // HUB_PROJECTS_BASE 配下の workspace の場合は CLAUDE.md を必要なら生成し、
-  // Hub の正本（Project.claude_md）から受け取る。
-  // - ensureClaudeMd: 既存 CLAUDE.md は触らない。無ければ director frontmatter
-  //   または placeholder を生成する。
-  // - syncClaudeMdWithHub: **受け取り専用**。Hub 側更新はローカルへ pull（.bak 退避
-  //   してから上書き）、端末側で編集されていても Hub へは送り返さない（正本は
-  //   プレースホルダ入りで、端末にあるのはそのレンダリング結果のため）。乖離は
-  //   warn ＋ `config-md-receive-only-drift` で見える形に残す。唯一の例外が
-  //   **初回登録**（Hub の正本が空 + そのファイルが元からあった場合に 1 度だけ push）。
-  //   Hub が一度も保存されていないプロジェクトでは何もしない（テナント標準
-  //   CLAUDE.md の運用を尊重する）。
-  // どちらも失敗しても session 起動自体は止めない (warn のみ)。
-  // ⚠️ 2 つの呼び出しで **1 本の budget を共有**する。1 往復ごとの期限だけでは
-  //   往復の本数ぶん足し算されてセッション起動が長時間止まりうるため、Hub 処理に
-  //   割り当てた wall-clock 予算（予算の生成時からの経過時間で、通信の待ち時間の
-  //   累積ではない）で全体を打ち切る。予算切れなら「何も書かずに次回へ持ち越す」。
-  if (isUnderHubProjectsBase(resolvedCwd)) {
-    const instructionDir = projectInstructionDir(resolvedCwd)
-    const dirName = path.basename(instructionDir)
-    const configFilename = opts.cliKind === "codex" ? "AGENTS.md" : "CLAUDE.md"
-    const budget = createSessionBudget()
-    // ⚠️ `ensureClaudeMd` の戻り値を捨てない。`source === 'existing'` は
-    // 「このセッションより前からローカルにファイルがあった」＝ **初回登録**
-    // （Hub に無くローカルにあるものを 1 度だけ Hub へ取り込む）の唯一の根拠で、
-    // ディスクに残さずここから同期へ手渡す。例外時は **undefined のまま渡さない**
-    // （＝登録しない。何が起きたか分からない回に Hub を触らせない安全側）。
-    let localFilePreexisted
-    try {
-      try {
-        const ensure = await ensureClaudeMd({
-          targetDir: instructionDir,
-          dirName,
-          hubUrl: opts.hubUrl,
-          accessToken: opts.hubAccessToken,
-          logger: opts.logger,
-          fetchImpl: opts.fetchImpl,
-          configFilename,
-          budget,
-        })
-        localFilePreexisted = ensure?.source === "existing"
-      } catch (err) {
-        opts.logger?.warn?.(
-          { session: name, cwd: resolvedCwd, err: err?.message || String(err) },
-          "ensureClaudeMd failed (workspace dir is usable, but no CLAUDE.md)",
-        )
-      }
-      try {
-        const syncResult = await syncClaudeMdWithHub({
-          targetDir: instructionDir,
-          dirName,
-          hubUrl: opts.hubUrl,
-          accessToken: opts.hubAccessToken,
-          logger: opts.logger,
-          fetchImpl: opts.fetchImpl,
-          configFilename,
-          budget,
-          // undefined のときはキーごと渡さない（呼び出し側の意図を明示する）。
-          ...(localFilePreexisted === undefined ? {} : { localFilePreexisted }),
-        })
-        opts.logger?.debug?.(
-          { session: name, dirName, sync: syncResult },
-          "syncClaudeMdWithHub completed",
-        )
-      } catch (err) {
-        opts.logger?.warn?.(
-          { session: name, cwd: resolvedCwd, err: err?.message || String(err) },
-          "syncClaudeMdWithHub failed (session starts anyway)",
-        )
-      }
-    } finally {
-      budget.clear()
-    }
-  }
+  await prepareChatWorkspace({ ...opts, cwd: resolvedCwd, session: name })
   // 既存チェック
   // ⚠️ `-t =<name>` で完全一致を強制する。tmux の `-t <name>` は「完全一致 →
   // 先頭一致(prefix) → fnmatch」の順で解決するため、素の `-t <name>` だと