@os-eco/overstory-cli 0.7.0 → 0.7.3

package/src/agents/hooks-deployer.ts

@@ -3,6 +3,13 @@ import { dirname, join } from "node:path";
 import { DEFAULT_QUALITY_GATES } from "../config.ts";
 import { AgentError } from "../errors.ts";
 import type { QualityGate } from "../types.ts";
+import {
+	DANGEROUS_BASH_PATTERNS,
+	INTERACTIVE_TOOLS,
+	NATIVE_TEAM_TOOLS,
+	SAFE_BASH_PREFIXES,
+	WRITE_TOOLS,
+} from "./guard-rules.ts";
 
 /**
  * Capabilities that must never modify project files.
@@ -31,96 +38,6 @@ const COORDINATION_CAPABILITIES = new Set(["coordinator", "supervisor", "monitor
  */
 const COORDINATION_SAFE_PREFIXES = ["git add", "git commit"];
 
-/**
- * Claude Code native team/task tools that bypass overstory orchestration.
- * All overstory agents must use `overstory sling` for delegation, not these.
- */
-const NATIVE_TEAM_TOOLS = [
-	"Task",
-	"TeamCreate",
-	"TeamDelete",
-	"SendMessage",
-	"TaskCreate",
-	"TaskUpdate",
-	"TaskList",
-	"TaskGet",
-	"TaskOutput",
-	"TaskStop",
-];
-
-/**
- * Tools that require human interaction and block indefinitely in non-interactive
- * tmux sessions. Agents run non-interactively and must never call these tools.
- * Use overstory mail (--type question) to escalate to the orchestrator instead.
- */
-const INTERACTIVE_TOOLS = ["AskUserQuestion", "EnterPlanMode", "EnterWorktree"];
-
-/** Tools that non-implementation agents must not use. */
-const WRITE_TOOLS = ["Write", "Edit", "NotebookEdit"];
-
-/**
- * Bash commands that modify files and must be blocked for non-implementation agents.
- * Each pattern is a regex fragment used inside a grep -qE check.
- */
-const DANGEROUS_BASH_PATTERNS = [
-	"sed\\s+-i",
-	"sed\\s+--in-place",
-	"echo\\s+.*>",
-	"printf\\s+.*>",
-	"cat\\s+.*>",
-	"tee\\s",
-	"\\bvim\\b",
-	"\\bnano\\b",
-	"\\bvi\\b",
-	"\\bmv\\s",
-	"\\bcp\\s",
-	"\\brm\\s",
-	"\\bmkdir\\s",
-	"\\btouch\\s",
-	"\\bchmod\\s",
-	"\\bchown\\s",
-	">>",
-	"\\bgit\\s+add\\b",
-	"\\bgit\\s+commit\\b",
-	"\\bgit\\s+merge\\b",
-	"\\bgit\\s+push\\b",
-	"\\bgit\\s+reset\\b",
-	"\\bgit\\s+checkout\\b",
-	"\\bgit\\s+rebase\\b",
-	"\\bgit\\s+stash\\b",
-	"\\bnpm\\s+install\\b",
-	"\\bbun\\s+install\\b",
-	"\\bbun\\s+add\\b",
-	// Runtime eval flags — bypass shell pattern guards by executing JS/Python directly
-	"\\bbun\\s+-e\\b",
-	"\\bbun\\s+--eval\\b",
-	"\\bnode\\s+-e\\b",
-	"\\bnode\\s+--eval\\b",
-	"\\bdeno\\s+eval\\b",
-	"\\bpython3?\\s+-c\\b",
-	"\\bperl\\s+-e\\b",
-	"\\bruby\\s+-e\\b",
-];
-
-/**
- * Bash commands that are always safe for non-implementation agents.
- * If a command starts with any of these prefixes, it bypasses the dangerous command check.
- * This whitelist is checked BEFORE the blocklist.
- */
-const SAFE_BASH_PREFIXES = [
-	"ov ",
-	"overstory ",
-	"bd ",
-	"sd ",
-	"git status",
-	"git log",
-	"git diff",
-	"git show",
-	"git blame",
-	"git branch",
-	"mulch ",
-];
-
 /**
  * Extract command prefixes from quality gate configurations.
  *

package/src/agents/identity.test.ts

@@ -1,8 +1,9 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdir, mkdtemp, rm } from "node:fs/promises";
+import { mkdir, mkdtemp } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { AgentError } from "../errors.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { AgentIdentity } from "../types.ts";
 import { createIdentity, loadIdentity, updateIdentity } from "./identity.ts";
 
@@ -14,7 +15,7 @@ describe("identity", () => {
 	});
 
 	afterEach(async () => {
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	describe("createIdentity", () => {

package/src/agents/manifest.test.ts

@@ -1,8 +1,9 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdir, mkdtemp, rm } from "node:fs/promises";
+import { mkdir, mkdtemp } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { AgentError } from "../errors.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { AgentManifest, OverstoryConfig } from "../types.ts";
 import { createManifestLoader, resolveModel, resolveProviderEnv } from "./manifest.ts";
 
@@ -41,7 +42,7 @@ describe("createManifestLoader", () => {
 	});
 
 	afterEach(async () => {
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	/** Write the manifest JSON and create matching .md files. */
@@ -806,7 +807,7 @@ describe("manifest validation accepts arbitrary model strings", () => {
 	});
 
 	afterEach(async () => {
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	test("accepts provider-prefixed model string", async () => {

package/src/agents/overlay.test.ts

@@ -1,8 +1,9 @@
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdir, mkdtemp, rm } from "node:fs/promises";
+import { mkdir, mkdtemp } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { AgentError } from "../errors.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { OverlayConfig, QualityGate } from "../types.ts";
 import {
 	formatQualityGatesBash,
@@ -265,7 +266,7 @@ describe("generateOverlay", () => {
 		expect(output).not.toContain("bun run lint");
 	});
 
-	test("scout completion section includes bd close and mail send", async () => {
+	test("scout completion section includes sd close and mail send", async () => {
 		const config = makeConfig({
 			capability: "scout",
 			agentName: "recon-1",
@@ -274,7 +275,7 @@ describe("generateOverlay", () => {
 		});
 		const output = await generateOverlay(config);
 
-		expect(output).toContain("bd close overstory-task1");
+		expect(output).toContain("sd close overstory-task1");
 		expect(output).toContain("ov mail send --to lead-alpha");
 	});
 
@@ -426,14 +427,14 @@ describe("generateOverlay", () => {
 		expect(output).not.toContain("Quality Gates");
 	});
 
-	test("default trackerCli renders as bd in quality gates", async () => {
+	test("default trackerCli renders as sd in quality gates", async () => {
 		const config = makeConfig({ capability: "builder", taskId: "overstory-task1" });
 		const output = await generateOverlay(config);
 
-		expect(output).toContain("bd close overstory-task1");
+		expect(output).toContain("sd close overstory-task1");
 	});
 
-	test("custom trackerCli replaces bd in quality gates", async () => {
+	test("custom trackerCli replaces sd in quality gates", async () => {
 		const config = makeConfig({
 			capability: "builder",
 			trackerCli: "sd",
@@ -489,11 +490,11 @@ describe("generateOverlay", () => {
 		expect(output).not.toContain("{{TRACKER_NAME}}");
 	});
 
-	test("defaults backward-compatible: no trackerCli/trackerName produces bd/beads", async () => {
+	test("defaults: no trackerCli/trackerName produces sd/seeds", async () => {
 		const config = makeConfig({ capability: "builder", taskId: "overstory-back" });
 		const output = await generateOverlay(config);
 
-		expect(output).toContain("bd close overstory-back");
+		expect(output).toContain("sd close overstory-back");
 	});
 
 	test("dispatch overrides: skipReview injects SKIP REVIEW directive for leads", async () => {
@@ -584,7 +585,7 @@ describe("writeOverlay", () => {
 	});
 
 	afterEach(async () => {
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	test("creates .claude/CLAUDE.md in worktree directory", async () => {

package/src/agents/overlay.ts

@@ -182,7 +182,7 @@ function formatQualityGates(config: OverlayConfig): string {
 			"Before reporting completion:",
 			"",
 			`1. **Record mulch learnings:** \`ml record <domain> --type <convention|pattern|reference> --description "..."\` — capture reusable knowledge from your work`,
-			`2. **Close issue:** \`${config.trackerCli ?? "bd"} close ${config.taskId} --reason "summary of findings"\``,
+			`2. **Close issue:** \`${config.trackerCli ?? "sd"} close ${config.taskId} --reason "summary of findings"\``,
 			`3. **Send results:** \`ov mail send --to ${config.parentAgent ?? "coordinator"} --subject "done" --body "Summary" --type result --agent ${config.agentName}\``,
 			"",
 			"You are a read-only agent. Do NOT commit, modify files, or run quality gates.",
@@ -207,7 +207,7 @@ function formatQualityGates(config: OverlayConfig): string {
 		`${gateLines.length + 1}. **Commit:** all changes committed to your branch (${config.branchName})`,
 		`${gateLines.length + 2}. **Record mulch learnings:** \`ml record <domain> --type <convention|pattern|failure|decision> --description "..." --outcome-status success --outcome-agent ${config.agentName}\` — capture insights from your work`,
 		`${gateLines.length + 3}. **Signal completion:** send \`worker_done\` mail to ${config.parentAgent ?? "coordinator"}: \`ov mail send --to ${config.parentAgent ?? "coordinator"} --subject "Worker done: ${config.taskId}" --body "Quality gates passed." --type worker_done --agent ${config.agentName}\``,
-		`${gateLines.length + 4}. **Close issue:** \`${config.trackerCli ?? "bd"} close ${config.taskId} --reason "summary of changes"\``,
+		`${gateLines.length + 4}. **Close issue:** \`${config.trackerCli ?? "sd"} close ${config.taskId} --reason "summary of changes"\``,
 		"",
 		"Do NOT push to the canonical branch. Your work will be merged by the",
 		"coordinator via `ov merge`.",
@@ -225,7 +225,7 @@ function formatConstraints(config: OverlayConfig): string {
 			"",
 			"- You are **read-only**: do NOT modify, create, or delete any files",
 			"- Do NOT commit, push, or make any git state changes",
-			`- Report completion via \`${config.trackerCli ?? "bd"} close\` AND \`ov mail send --type result\``,
+			`- Report completion via \`${config.trackerCli ?? "sd"} close\` AND \`ov mail send --type result\``,
 			"- If you encounter a blocking issue, send mail with `--priority urgent --type error`",
 		].join("\n");
 	}
@@ -238,7 +238,7 @@ function formatConstraints(config: OverlayConfig): string {
 		"- Only modify files in your File Scope",
 		`- Commit only to your branch: ${config.branchName}`,
 		"- Never push to the canonical branch",
-		`- Report completion via \`${config.trackerCli ?? "bd"} close\` AND \`ov mail send --type result\``,
+		`- Report completion via \`${config.trackerCli ?? "sd"} close\` AND \`ov mail send --type result\``,
 		"- If you encounter a blocking issue, send mail with `--priority urgent --type error`",
 	].join("\n");
 }
@@ -318,7 +318,7 @@ export async function generateOverlay(config: OverlayConfig): Promise<string> {
 		"{{QUALITY_GATE_STEPS}}": formatQualityGatesSteps(config.qualityGates),
 		"{{QUALITY_GATE_BASH}}": formatQualityGatesBash(config.qualityGates),
 		"{{QUALITY_GATE_CAPABILITIES}}": formatQualityGatesCapabilities(config.qualityGates),
-		"{{TRACKER_CLI}}": config.trackerCli ?? "bd",
+		"{{TRACKER_CLI}}": config.trackerCli ?? "sd",
 		"{{TRACKER_NAME}}": config.trackerName ?? "seeds",
 	};
 

package/src/commands/agents.test.ts

@@ -3,10 +3,11 @@
  */
 
 import { afterEach, beforeEach, describe, expect, it, mock } from "bun:test";
-import { mkdtemp, rm } from "node:fs/promises";
+import { mkdtemp } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { createSessionStore } from "../sessions/store.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { AgentSession } from "../types.ts";
 import { agentsCommand, discoverAgents, extractFileScope } from "./agents.ts";
 
@@ -65,7 +66,7 @@ Some expertise here.
 	});
 
 	afterEach(async () => {
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 });
 
@@ -108,6 +109,7 @@ describe("discoverAgents", () => {
 			lastActivity: "2024-01-01T00:01:00Z",
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		};
 
 		store.upsert(session);
@@ -140,6 +142,7 @@ describe("discoverAgents", () => {
 			lastActivity: "2024-01-01T00:01:00Z",
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		};
 
 		const scout: AgentSession = {
@@ -159,6 +162,7 @@ describe("discoverAgents", () => {
 			lastActivity: "2024-01-01T00:01:00Z",
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		};
 
 		store.upsert(builder);
@@ -191,6 +195,7 @@ describe("discoverAgents", () => {
 			lastActivity: "2024-01-01T00:01:00Z",
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		};
 
 		const completed: AgentSession = {
@@ -210,6 +215,7 @@ describe("discoverAgents", () => {
 			lastActivity: "2024-01-01T00:02:00Z",
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		};
 
 		store.upsert(working);
@@ -230,7 +236,7 @@ describe("discoverAgents", () => {
 	});
 
 	afterEach(async () => {
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 });
 
@@ -317,6 +323,6 @@ logging:
 	afterEach(async () => {
 		process.stdout.write = originalStdoutWrite;
 		process.chdir(originalCwd);
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 });

package/src/commands/clean.test.ts

@@ -159,6 +159,7 @@ describe("--all", () => {
 			lastActivity: new Date().toISOString(),
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		});
 		store.close();
 
@@ -303,6 +304,7 @@ describe("individual flags", () => {
 			lastActivity: new Date().toISOString(),
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		});
 		store.close();
 
@@ -424,6 +426,7 @@ describe("synthetic session-end events", () => {
 			lastActivity: new Date().toISOString(),
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 			...overrides,
 		};
 	}

package/src/commands/completions.test.ts

@@ -12,8 +12,8 @@ import {
 } from "./completions.ts";
 
 describe("COMMANDS array", () => {
-	it("should have exactly 30 commands", () => {
-		expect(COMMANDS).toHaveLength(30);
+	it("should have exactly 33 commands", () => {
+		expect(COMMANDS).toHaveLength(33);
 	});
 
 	it("should include all expected command names", () => {
@@ -48,6 +48,9 @@ describe("COMMANDS array", () => {
 		expect(names).toContain("feed");
 		expect(names).toContain("logs");
 		expect(names).toContain("stop");
+		expect(names).toContain("ecosystem");
+		expect(names).toContain("upgrade");
+		expect(names).toContain("completions");
 	});
 });
 
@@ -59,7 +62,7 @@ describe("generateBash", () => {
 		expect(script).toContain("_init_completion");
 	});
 
-	it("should include all 27 command names", () => {
+	it("should include all 33 command names", () => {
 		const script = generateBash();
 		for (const cmd of COMMANDS) {
 			expect(script).toContain(cmd.name);
@@ -93,7 +96,7 @@ describe("generateZsh", () => {
 		expect(script).toContain("_arguments");
 	});
 
-	it("should include all 27 command names", () => {
+	it("should include all 33 command names", () => {
 		const script = generateZsh();
 		for (const cmd of COMMANDS) {
 			expect(script).toContain(cmd.name);
@@ -123,7 +126,7 @@ describe("generateFish", () => {
 		expect(script).toContain("__fish_use_subcommand");
 	});
 
-	it("should include all 27 command names", () => {
+	it("should include all 33 command names", () => {
 		const script = generateFish();
 		for (const cmd of COMMANDS) {
 			expect(script).toContain(cmd.name);

package/src/commands/completions.ts

@@ -58,6 +58,9 @@ export const COMMANDS: readonly CommandDef[] = [
 		desc: "Initialize .overstory/ in current project",
 		flags: [
 			{ name: "--force", desc: "Overwrite existing configuration" },
+			{ name: "--yes", desc: "Accept all defaults without prompting" },
+			{ name: "-y", desc: "Alias for --yes" },
+			{ name: "--name", desc: "Project name", takesValue: true },
 			{ name: "--help", desc: "Show help" },
 		],
 	},
@@ -76,7 +79,13 @@ export const COMMANDS: readonly CommandDef[] = [
 			{ name: "--files", desc: "Exclusive file scope (comma-separated)", takesValue: true },
 			{ name: "--parent", desc: "Parent agent name", takesValue: true },
 			{ name: "--depth", desc: "Current hierarchy depth", takesValue: true },
+			{ name: "--skip-scout", desc: "Skip scout phase for lead agents" },
+			{ name: "--skip-review", desc: "Skip review phase for lead agents" },
+			{ name: "--skip-task-check", desc: "Skip task existence validation" },
 			{ name: "--force-hierarchy", desc: "Bypass hierarchy validation" },
+			{ name: "--max-agents", desc: "Max children per lead", takesValue: true },
+			{ name: "--dispatch-max-agents", desc: "Per-lead max agents ceiling", takesValue: true },
+			{ name: "--runtime", desc: "Runtime adapter", takesValue: true },
 			{ name: "--json", desc: "JSON output" },
 			{ name: "--help", desc: "Show help" },
 		],
@@ -107,6 +116,7 @@ export const COMMANDS: readonly CommandDef[] = [
 			{ name: "--json", desc: "JSON output" },
 			{ name: "--verbose", desc: "Extra per-agent detail" },
 			{ name: "--agent", desc: "Filter by agent", takesValue: true },
+			{ name: "--all", desc: "Show sessions from all runs" },
 			{ name: "--watch", desc: "Watch mode" },
 			{ name: "--interval", desc: "Poll interval in ms", takesValue: true },
 			{ name: "--help", desc: "Show help" },
@@ -138,6 +148,7 @@ export const COMMANDS: readonly CommandDef[] = [
 		flags: [
 			{ name: "--branch", desc: "Specific branch to merge", takesValue: true },
 			{ name: "--all", desc: "All completed branches" },
+			{ name: "--into", desc: "Target branch to merge into", takesValue: true },
 			{ name: "--dry-run", desc: "Check for conflicts only" },
 			{ name: "--json", desc: "JSON output" },
 			{ name: "--help", desc: "Show help" },
@@ -190,8 +201,10 @@ export const COMMANDS: readonly CommandDef[] = [
 					"merge",
 					"logs",
 					"version",
+					"ecosystem",
 				],
 			},
+			{ name: "--fix", desc: "Attempt to auto-fix issues" },
 			{ name: "--help", desc: "Show help" },
 		],
 	},
@@ -356,7 +369,7 @@ export const COMMANDS: readonly CommandDef[] = [
 	},
 	{
 		name: "supervisor",
-		desc: "Per-project supervisor agent",
+		desc: "[DEPRECATED] Per-project supervisor agent",
 		flags: [
 			{ name: "--json", desc: "JSON output" },
 			{ name: "--help", desc: "Show help" },
@@ -606,6 +619,29 @@ export const COMMANDS: readonly CommandDef[] = [
 			},
 		],
 	},
+	{
+		name: "ecosystem",
+		desc: "Show a summary dashboard of all installed os-eco tools",
+		flags: [
+			{ name: "--json", desc: "JSON output" },
+			{ name: "--help", desc: "Show help" },
+		],
+	},
+	{
+		name: "upgrade",
+		desc: "Upgrade overstory to the latest version",
+		flags: [
+			{ name: "--check", desc: "Compare current vs latest without installing" },
+			{ name: "--all", desc: "Upgrade all os-eco tools" },
+			{ name: "--json", desc: "JSON output" },
+			{ name: "--help", desc: "Show help" },
+		],
+	},
+	{
+		name: "completions",
+		desc: "Generate shell completions",
+		flags: [{ name: "--help", desc: "Show help" }],
+	},
 ] as const;
 
 export function generateBash(): string {
@@ -618,7 +654,7 @@ export function generateBash(): string {
 		"  local cur prev words cword",
 		"  _init_completion || return",
 		"",
-		"  local commands='init sling prime stop status dashboard inspect merge nudge clean doctor log logs watch trace errors feed replay costs metrics spec coordinator supervisor hooks monitor mail group worktree run'",
+		"  local commands='agents init sling prime stop status dashboard inspect merge nudge clean doctor log logs watch trace errors feed replay costs metrics spec coordinator supervisor hooks monitor mail group worktree run ecosystem upgrade completions'",
 		"",
 		"  # Top-level completion",
 		"  if [[ $cword -eq 1 ]]; then",

package/src/commands/coordinator.test.ts

@@ -297,6 +297,7 @@ function makeCoordinatorSession(overrides: Partial<AgentSession> = {}): AgentSes
 		lastActivity: new Date().toISOString(),
 		escalationLevel: 0,
 		stalledSince: null,
+		transcriptPath: null,
 		...overrides,
 	};
 }

package/src/commands/coordinator.ts

@@ -15,7 +15,6 @@
 import { mkdir, unlink } from "node:fs/promises";
 import { join } from "node:path";
 import { Command } from "commander";
-import { deployHooks } from "../agents/hooks-deployer.ts";
 import { createIdentity, loadIdentity } from "../agents/identity.ts";
 import { createManifestLoader, resolveModel } from "../agents/manifest.ts";
 import { loadConfig } from "../config.ts";
@@ -320,13 +319,26 @@ async function startCoordinator(
 			store.updateState(COORDINATOR_NAME, "completed");
 		}
 
+		// Resolve model and runtime early (needed for deployConfig and spawn)
+		const manifestLoader = createManifestLoader(
+			join(projectRoot, config.agents.manifestPath),
+			join(projectRoot, config.agents.baseDir),
+		);
+		const manifest = await manifestLoader.load();
+		const resolvedModel = resolveModel(config, manifest, "coordinator", "opus");
+		const runtime = getRuntime(undefined, config);
+
 		// Deploy hooks to the project root so the coordinator gets event logging,
 		// mail check --inject, and activity tracking via the standard hook pipeline.
 		// The ENV_GUARD prefix on all hooks (both template and generated guards)
 		// ensures they only activate when OVERSTORY_AGENT_NAME is set (i.e. for
 		// the coordinator's tmux session), so the user's own Claude Code session
 		// at the project root is unaffected.
-		await deployHooks(projectRoot, COORDINATOR_NAME, "coordinator");
+		await runtime.deployConfig(projectRoot, undefined, {
+			agentName: COORDINATOR_NAME,
+			capability: "coordinator",
+			worktreePath: projectRoot,
+		});
 
 		// Create coordinator identity if first run
 		const identityBaseDir = join(projectRoot, ".overstory", "agents");
@@ -343,15 +355,6 @@ async function startCoordinator(
 			});
 		}
 
-		// Resolve model from config > manifest > fallback
-		const manifestLoader = createManifestLoader(
-			join(projectRoot, config.agents.manifestPath),
-			join(projectRoot, config.agents.baseDir),
-		);
-		const manifest = await manifestLoader.load();
-		const resolvedModel = resolveModel(config, manifest, "coordinator", "opus");
-		const runtime = getRuntime(undefined, config);
-
 		// Preflight: verify tmux is installed before attempting to spawn.
 		// Without this check, a missing tmux leads to cryptic errors later.
 		await tmux.ensureTmuxAvailable();
@@ -402,6 +405,7 @@ async function startCoordinator(
 			lastActivity: new Date().toISOString(),
 			escalationLevel: 0,
 			stalledSince: null,
+			transcriptPath: null,
 		};
 
 		store.upsert(session);

package/src/commands/costs.test.ts

@@ -9,12 +9,13 @@
  */
 
 import { afterEach, beforeEach, describe, expect, test } from "bun:test";
-import { mkdir, mkdtemp, rm } from "node:fs/promises";
+import { mkdir, mkdtemp } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { ValidationError } from "../errors.ts";
 import { createMetricsStore } from "../metrics/store.ts";
 import { createSessionStore } from "../sessions/store.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { SessionMetrics } from "../types.ts";
 import { costsCommand } from "./costs.ts";
 
@@ -72,7 +73,7 @@ describe("costsCommand", () => {
 	afterEach(async () => {
 		process.stdout.write = originalWrite;
 		process.chdir(originalCwd);
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	function output(): string {
@@ -780,6 +781,7 @@ describe("costsCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			sessionStore.close();
 
@@ -836,6 +838,7 @@ describe("costsCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			sessionStore.close();
 
@@ -900,6 +903,7 @@ describe("costsCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			sessionStore.upsert({
 				id: "sess-002",
@@ -918,6 +922,7 @@ describe("costsCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			sessionStore.close();
 
@@ -977,6 +982,7 @@ describe("costsCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			sessionStore.close();
 
@@ -1047,7 +1053,7 @@ describe("costsCommand", () => {
 
 		afterEach(async () => {
 			process.env.HOME = originalHome;
-			await rm(tempHome, { recursive: true, force: true });
+			await cleanupTempDir(tempHome);
 		});
 
 		test("--self shows orchestrator cost when transcript exists", async () => {