@cocorograph/hub-agent 0.8.21 → 0.8.23

package/src/tmux.mjs

@@ -22,6 +22,7 @@ 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 { detectSessionState, getSessionCwd } from "./state.mjs"
 import { shouldLaunchPaneCli } from "./pane-cli-flag.mjs"
 import { baseSessionName, sessionNameFor } from "./session-names.mjs"
@@ -836,45 +837,65 @@ export async function createSession(name, cwd, opts = {}) {
   //   .bak.conflict に退避）。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 {
-      await ensureClaudeMd({
-        targetDir: instructionDir,
-        dirName,
-        hubUrl: opts.hubUrl,
-        accessToken: opts.hubAccessToken,
-        logger: opts.logger,
-        fetchImpl: opts.fetchImpl,
-        configFilename,
-      })
-    } 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,
-      })
-      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)",
-      )
+      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()
     }
   }
   // 既存チェック