@os-eco/overstory-cli 0.7.4 → 0.7.5

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@os-eco/overstory-cli",
-	"version": "0.7.4",
+	"version": "0.7.5",
 	"description": "Multi-agent orchestration for AI coding agents — spawn workers in git worktrees via tmux, coordinate through SQLite mail, merge with tiered conflict resolution. Pluggable runtime adapters for Claude Code, Pi, and more.",
 	"author": "Jaymin West",
 	"license": "MIT",

package/src/commands/coordinator.test.ts

@@ -540,7 +540,9 @@ describe("startCoordinator", () => {
 		expect(calls.createSession).toHaveLength(1);
 		const cmd = calls.createSession[0]?.command ?? "";
 		expect(cmd).toContain("--append-system-prompt");
-		expect(cmd).toContain("# Coordinator Agent");
+		// File path is passed via $(cat ...) instead of inlining content (overstory#45)
+		expect(cmd).toContain("$(cat '");
+		expect(cmd).toContain("agent-defs/coordinator.md");
 	});
 
 	test("reads model from manifest instead of hardcoding", async () => {

package/src/commands/coordinator.ts

@@ -363,17 +363,20 @@ async function startCoordinator(
 		// Inject the coordinator base definition via --append-system-prompt so the
 		// coordinator knows its role, hierarchy rules, and delegation patterns
 		// (overstory-gaio, overstory-0kwf).
+		// Pass the file path (not content) so the shell inside the tmux pane reads
+		// it via $(cat ...) — avoids tmux IPC "command too long" errors with large
+		// agent definitions (overstory#45).
 		const agentDefPath = join(projectRoot, ".overstory", "agent-defs", "coordinator.md");
 		const agentDefFile = Bun.file(agentDefPath);
-		let appendSystemPrompt: string | undefined;
+		let appendSystemPromptFile: string | undefined;
 		if (await agentDefFile.exists()) {
-			appendSystemPrompt = await agentDefFile.text();
+			appendSystemPromptFile = agentDefPath;
 		}
 		const spawnCmd = runtime.buildSpawnCommand({
 			model: resolvedModel.model,
 			permissionMode: "bypass",
 			cwd: projectRoot,
-			appendSystemPrompt,
+			appendSystemPromptFile,
 			env: {
 				...runtime.buildEnv(resolvedModel),
 				OVERSTORY_AGENT_NAME: COORDINATOR_NAME,

package/src/commands/monitor.ts

@@ -142,17 +142,18 @@ async function startMonitor(opts: { json: boolean; attach: boolean }): Promise<v
 		}
 
 		// Spawn tmux session at project root with Claude Code (interactive mode).
+		// Pass file path (not content) to avoid tmux "command too long" (overstory#45).
 		const agentDefPath = join(projectRoot, ".overstory", "agent-defs", "monitor.md");
 		const agentDefFile = Bun.file(agentDefPath);
-		let appendSystemPrompt: string | undefined;
+		let appendSystemPromptFile: string | undefined;
 		if (await agentDefFile.exists()) {
-			appendSystemPrompt = await agentDefFile.text();
+			appendSystemPromptFile = agentDefPath;
 		}
 		const spawnCmd = runtime.buildSpawnCommand({
 			model: resolvedModel.model,
 			permissionMode: "bypass",
 			cwd: projectRoot,
-			appendSystemPrompt,
+			appendSystemPromptFile,
 			env: {
 				...runtime.buildEnv(resolvedModel),
 				OVERSTORY_AGENT_NAME: MONITOR_NAME,

package/src/commands/supervisor.ts

@@ -169,18 +169,19 @@ async function startSupervisor(opts: {
 
 		// Spawn tmux session at project root with Claude Code (interactive mode).
 		// Inject the supervisor base definition via --append-system-prompt.
+		// Pass file path (not content) to avoid tmux "command too long" (overstory#45).
 		const tmuxSession = `overstory-${config.project.name}-supervisor-${opts.name}`;
 		const agentDefPath = join(projectRoot, ".overstory", "agent-defs", "supervisor.md");
 		const agentDefFile = Bun.file(agentDefPath);
-		let appendSystemPrompt: string | undefined;
+		let appendSystemPromptFile: string | undefined;
 		if (await agentDefFile.exists()) {
-			appendSystemPrompt = await agentDefFile.text();
+			appendSystemPromptFile = agentDefPath;
 		}
 		const spawnCmd = runtime.buildSpawnCommand({
 			model: resolvedModel.model,
 			permissionMode: "bypass",
 			cwd: projectRoot,
-			appendSystemPrompt,
+			appendSystemPromptFile,
 			env: {
 				...runtime.buildEnv(resolvedModel),
 				OVERSTORY_AGENT_NAME: opts.name,

package/src/index.ts

@@ -45,7 +45,7 @@ import { OverstoryError, WorktreeError } from "./errors.ts";
 import { jsonError } from "./json.ts";
 import { brand, chalk, muted, setQuiet } from "./logging/color.ts";
 
-export const VERSION = "0.7.4";
+export const VERSION = "0.7.5";
 
 const rawArgs = process.argv.slice(2);
 

package/src/runtimes/claude.test.ts

@@ -73,6 +73,46 @@ describe("ClaudeRuntime", () => {
 			);
 		});
 
+		test("with appendSystemPromptFile uses $(cat ...) expansion", () => {
+			const opts: SpawnOpts = {
+				model: "opus",
+				permissionMode: "bypass",
+				cwd: "/project",
+				env: {},
+				appendSystemPromptFile: "/project/.overstory/agent-defs/coordinator.md",
+			};
+			const cmd = runtime.buildSpawnCommand(opts);
+			expect(cmd).toBe(
+				`claude --model opus --permission-mode bypassPermissions --append-system-prompt "$(cat '/project/.overstory/agent-defs/coordinator.md')"`,
+			);
+		});
+
+		test("appendSystemPromptFile with single quotes in path", () => {
+			const opts: SpawnOpts = {
+				model: "opus",
+				permissionMode: "bypass",
+				cwd: "/project",
+				env: {},
+				appendSystemPromptFile: "/project/it's a path/agent.md",
+			};
+			const cmd = runtime.buildSpawnCommand(opts);
+			expect(cmd).toContain("$(cat '/project/it'\\''s a path/agent.md')");
+		});
+
+		test("appendSystemPromptFile takes precedence over appendSystemPrompt", () => {
+			const opts: SpawnOpts = {
+				model: "opus",
+				permissionMode: "bypass",
+				cwd: "/project",
+				env: {},
+				appendSystemPromptFile: "/project/.overstory/agent-defs/coordinator.md",
+				appendSystemPrompt: "This inline content should be ignored",
+			};
+			const cmd = runtime.buildSpawnCommand(opts);
+			expect(cmd).toContain("$(cat ");
+			expect(cmd).not.toContain("This inline content should be ignored");
+		});
+
 		test("without appendSystemPrompt omits the flag", () => {
 			const opts: SpawnOpts = {
 				model: "haiku",

package/src/runtimes/claude.ts

@@ -54,7 +54,14 @@ export class ClaudeRuntime implements AgentRuntime {
 		const permMode = opts.permissionMode === "bypass" ? "bypassPermissions" : "default";
 		let cmd = `claude --model ${opts.model} --permission-mode ${permMode}`;
 
-		if (opts.appendSystemPrompt) {
+		if (opts.appendSystemPromptFile) {
+			// Read from file at shell expansion time — avoids tmux IPC message size
+			// limits (~8-16KB) that cause "command too long" errors when large agent
+			// definitions are inlined. The $(cat ...) expands inside the tmux pane's
+			// shell, so the tmux IPC message only carries the short command string.
+			const escaped = opts.appendSystemPromptFile.replace(/'/g, "'\\''");
+			cmd += ` --append-system-prompt "$(cat '${escaped}')"`;
+		} else if (opts.appendSystemPrompt) {
 			// Single-quote the content for safe shell expansion.
 			// POSIX single-quoted strings cannot contain single quotes, so escape
 			// them using the standard technique: end quote, escaped quote, start quote.

package/src/runtimes/pi.test.ts

@@ -126,6 +126,34 @@ describe("PiRuntime", () => {
 			);
 		});
 
+		test("with appendSystemPromptFile uses $(cat ...) expansion", () => {
+			const opts: SpawnOpts = {
+				model: "opus",
+				permissionMode: "bypass",
+				cwd: "/project",
+				env: {},
+				appendSystemPromptFile: "/project/.overstory/agent-defs/coordinator.md",
+			};
+			const cmd = runtime.buildSpawnCommand(opts);
+			expect(cmd).toBe(
+				`pi --model anthropic/claude-opus-4-6 --append-system-prompt "$(cat '/project/.overstory/agent-defs/coordinator.md')"`,
+			);
+		});
+
+		test("appendSystemPromptFile takes precedence over appendSystemPrompt", () => {
+			const opts: SpawnOpts = {
+				model: "opus",
+				permissionMode: "bypass",
+				cwd: "/project",
+				env: {},
+				appendSystemPromptFile: "/project/.overstory/agent-defs/coordinator.md",
+				appendSystemPrompt: "This inline content should be ignored",
+			};
+			const cmd = runtime.buildSpawnCommand(opts);
+			expect(cmd).toContain("$(cat ");
+			expect(cmd).not.toContain("This inline content should be ignored");
+		});
+
 		test("without appendSystemPrompt omits the flag", () => {
 			const opts: SpawnOpts = {
 				model: "haiku",

package/src/runtimes/pi.ts

@@ -75,7 +75,11 @@ export class PiRuntime implements AgentRuntime {
 	buildSpawnCommand(opts: SpawnOpts): string {
 		let cmd = `pi --model ${this.expandModel(opts.model)}`;
 
-		if (opts.appendSystemPrompt) {
+		if (opts.appendSystemPromptFile) {
+			// Read from file at shell expansion time — avoids tmux command length limits.
+			const escaped = opts.appendSystemPromptFile.replace(/'/g, "'\\''");
+			cmd += ` --append-system-prompt "$(cat '${escaped}')"`;
+		} else if (opts.appendSystemPrompt) {
 			// POSIX single-quote escape: end quote, backslash-quote, start quote.
 			const escaped = opts.appendSystemPrompt.replace(/'/g, "'\\''");
 			cmd += ` --append-system-prompt '${escaped}'`;

package/src/runtimes/types.ts

@@ -15,6 +15,8 @@ export interface SpawnOpts {
 	systemPrompt?: string;
 	/** Optional system prompt suffix appended after the base instructions. */
 	appendSystemPrompt?: string;
+	/** Path to a file whose contents are appended as system prompt (avoids tmux command length limits). */
+	appendSystemPromptFile?: string;
 	/** Working directory for the spawned process. */
 	cwd: string;
 	/** Additional environment variables to pass to the spawned process. */