npm - @os-eco/overstory-cli - Versions diffs - 0.9.3 → 0.10.3 - Mend

@os-eco/overstory-cli 0.9.3 → 0.10.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (116) hide show

package/README.md +49 -18
package/agents/builder.md +9 -8
package/agents/coordinator.md +6 -6
package/agents/lead.md +98 -82
package/agents/merger.md +25 -14
package/agents/reviewer.md +22 -16
package/agents/scout.md +17 -12
package/package.json +6 -3
package/src/agents/capabilities.test.ts +85 -0
package/src/agents/capabilities.ts +125 -0
package/src/agents/headless-mail-injector.test.ts +448 -0
package/src/agents/headless-mail-injector.ts +211 -0
package/src/agents/headless-prompt.test.ts +102 -0
package/src/agents/headless-prompt.ts +68 -0
package/src/agents/hooks-deployer.test.ts +514 -14
package/src/agents/hooks-deployer.ts +141 -0
package/src/agents/overlay.test.ts +4 -4
package/src/agents/overlay.ts +30 -8
package/src/agents/turn-lock.test.ts +181 -0
package/src/agents/turn-lock.ts +235 -0
package/src/agents/turn-runner-dispatch.test.ts +182 -0
package/src/agents/turn-runner-dispatch.ts +105 -0
package/src/agents/turn-runner.test.ts +1450 -0
package/src/agents/turn-runner.ts +1166 -0
package/src/commands/clean.ts +56 -1
package/src/commands/completions.test.ts +4 -1
package/src/commands/coordinator.test.ts +127 -0
package/src/commands/coordinator.ts +205 -6
package/src/commands/dashboard.test.ts +188 -0
package/src/commands/dashboard.ts +13 -3
package/src/commands/doctor.ts +94 -77
package/src/commands/group.test.ts +94 -0
package/src/commands/group.ts +49 -20
package/src/commands/init.test.ts +8 -0
package/src/commands/init.ts +8 -1
package/src/commands/log.test.ts +56 -11
package/src/commands/log.ts +134 -69
package/src/commands/mail.test.ts +162 -0
package/src/commands/mail.ts +64 -9
package/src/commands/merge.test.ts +112 -1
package/src/commands/merge.ts +17 -4
package/src/commands/monitor.ts +2 -1
package/src/commands/nudge.test.ts +351 -4
package/src/commands/nudge.ts +356 -34
package/src/commands/run.test.ts +43 -7
package/src/commands/serve/build.test.ts +202 -0
package/src/commands/serve/build.ts +206 -0
package/src/commands/serve/coordinator-actions.test.ts +339 -0
package/src/commands/serve/coordinator-actions.ts +408 -0
package/src/commands/serve/dev.test.ts +168 -0
package/src/commands/serve/dev.ts +117 -0
package/src/commands/serve/mail-actions.test.ts +312 -0
package/src/commands/serve/mail-actions.ts +167 -0
package/src/commands/serve/rest.test.ts +1323 -0
package/src/commands/serve/rest.ts +708 -0
package/src/commands/serve/static.ts +51 -0
package/src/commands/serve/ws.test.ts +361 -0
package/src/commands/serve/ws.ts +332 -0
package/src/commands/serve.test.ts +459 -0
package/src/commands/serve.ts +565 -0
package/src/commands/sling.test.ts +85 -1
package/src/commands/sling.ts +153 -64
package/src/commands/status.test.ts +9 -0
package/src/commands/status.ts +12 -4
package/src/commands/stop.test.ts +174 -1
package/src/commands/stop.ts +107 -8
package/src/commands/supervisor.ts +2 -1
package/src/commands/watch.test.ts +49 -4
package/src/commands/watch.ts +153 -28
package/src/commands/worktree.test.ts +319 -3
package/src/commands/worktree.ts +86 -0
package/src/config.test.ts +78 -0
package/src/config.ts +43 -1
package/src/doctor/consistency.test.ts +106 -0
package/src/doctor/consistency.ts +50 -3
package/src/doctor/serve.test.ts +95 -0
package/src/doctor/serve.ts +86 -0
package/src/doctor/types.ts +2 -1
package/src/doctor/watchdog.ts +57 -1
package/src/events/tailer.test.ts +234 -1
package/src/events/tailer.ts +90 -0
package/src/index.ts +53 -6
package/src/json.ts +29 -0
package/src/mail/client.ts +15 -2
package/src/mail/store.test.ts +82 -0
package/src/mail/store.ts +41 -4
package/src/merge/lock.test.ts +149 -0
package/src/merge/lock.ts +140 -0
package/src/runtimes/__fixtures__/claude-stream-fixture.ts +22 -0
package/src/runtimes/claude.test.ts +791 -1
package/src/runtimes/claude.ts +323 -1
package/src/runtimes/connections.test.ts +141 -1
package/src/runtimes/connections.ts +73 -4
package/src/runtimes/headless-connection.test.ts +264 -0
package/src/runtimes/headless-connection.ts +158 -0
package/src/runtimes/types.ts +10 -0
package/src/schema-consistency.test.ts +1 -0
package/src/sessions/store.test.ts +390 -24
package/src/sessions/store.ts +184 -19
package/src/test-setup.test.ts +31 -0
package/src/test-setup.ts +28 -0
package/src/types.ts +56 -1
package/src/utils/pid.test.ts +85 -1
package/src/utils/pid.ts +86 -1
package/src/utils/process-scan.test.ts +53 -0
package/src/utils/process-scan.ts +76 -0
package/src/watchdog/daemon.test.ts +1520 -411
package/src/watchdog/daemon.ts +442 -83
package/src/watchdog/health.test.ts +157 -0
package/src/watchdog/health.ts +92 -25
package/src/worktree/process.test.ts +71 -0
package/src/worktree/process.ts +25 -5
package/src/worktree/tmux.test.ts +39 -0
package/src/worktree/tmux.ts +23 -3
package/templates/CLAUDE.md.tmpl +19 -8
package/templates/overlay.md.tmpl +3 -2

package/src/commands/serve/build.test.ts ADDED Viewed

@@ -0,0 +1,202 @@
+/**
+ * Tests for ensureUiBuild auto-build helper.
+ *
+ * Real filesystem (temp dirs + utimesSync) drives the freshness comparison;
+ * the runner is always stubbed so we never invoke a real bun install/build.
+ */
+import { afterEach, beforeEach, describe, expect, test } from "bun:test";
+import { mkdirSync, mkdtempSync, rmSync, utimesSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { ensureUiBuild, type RunnerResult } from "./build.ts";
+interface RunnerCall {
+	cmd: string[];
+	cwd: string;
+}
+function makeRunner(results: RunnerResult[]): {
+	runner: (cmd: string[], cwd: string) => Promise<RunnerResult>;
+	calls: RunnerCall[];
+} {
+	const calls: RunnerCall[] = [];
+	let i = 0;
+	return {
+		calls,
+		runner: async (cmd, cwd) => {
+			calls.push({ cmd, cwd });
+			const r = results[i] ?? { exitCode: 0, stderr: "" };
+			i += 1;
+			return r;
+		},
+	};
+}
+function setupUiDir(root: string): { uiDir: string } {
+	const uiDir = join(root, "ui");
+	mkdirSync(join(uiDir, "src"), { recursive: true });
+	writeFileSync(join(uiDir, "src", "main.ts"), 'console.log("hi")');
+	writeFileSync(join(uiDir, "index.html"), "<html></html>");
+	writeFileSync(join(uiDir, "package.json"), '{"name":"ui"}');
+	return { uiDir };
+}
+describe("ensureUiBuild", () => {
+	let tempDir: string;
+	beforeEach(() => {
+		tempDir = mkdtempSync(join(tmpdir(), "overstory-ui-build-"));
+	});
+	afterEach(() => {
+		rmSync(tempDir, { recursive: true, force: true });
+	});
+	test("triggers build when dist/index.html is missing", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		// Pretend node_modules already exists so we skip install.
+		mkdirSync(join(uiDir, "node_modules"));
+		const { runner, calls } = makeRunner([{ exitCode: 0, stderr: "" }]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: () => {} });
+		expect(calls.length).toBe(1);
+		expect(calls[0]?.cmd).toEqual(["bun", "run", "build"]);
+		expect(calls[0]?.cwd).toBe(uiDir);
+	});
+	test("triggers build when source mtime is newer than dist mtime", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		mkdirSync(join(uiDir, "node_modules"));
+		mkdirSync(join(uiDir, "dist"));
+		writeFileSync(join(uiDir, "dist", "index.html"), "<html>old</html>");
+		// Make dist older, src newer (definite mtime separation).
+		const past = new Date(Date.now() - 60_000);
+		const future = new Date(Date.now() + 60_000);
+		utimesSync(join(uiDir, "dist", "index.html"), past, past);
+		utimesSync(join(uiDir, "src", "main.ts"), future, future);
+		const { runner, calls } = makeRunner([{ exitCode: 0, stderr: "" }]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: () => {} });
+		expect(calls.length).toBe(1);
+		expect(calls[0]?.cmd).toEqual(["bun", "run", "build"]);
+	});
+	test("skips build when dist is newer than every source file", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		mkdirSync(join(uiDir, "node_modules"));
+		mkdirSync(join(uiDir, "dist"));
+		writeFileSync(join(uiDir, "dist", "index.html"), "<html>fresh</html>");
+		// All sources older than dist.
+		const past = new Date(Date.now() - 60_000);
+		utimesSync(join(uiDir, "src", "main.ts"), past, past);
+		utimesSync(join(uiDir, "index.html"), past, past);
+		utimesSync(join(uiDir, "package.json"), past, past);
+		// Dist is "now" — already the newest.
+		const { runner, calls } = makeRunner([]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: () => {} });
+		expect(calls.length).toBe(0);
+	});
+	test("runs install only when node_modules is missing", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		// node_modules deliberately missing.
+		const { runner, calls } = makeRunner([
+			{ exitCode: 0, stderr: "" }, // install
+			{ exitCode: 0, stderr: "" }, // build
+		]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: () => {} });
+		expect(calls.length).toBe(2);
+		expect(calls[0]?.cmd).toEqual(["bun", "install"]);
+		expect(calls[0]?.cwd).toBe(uiDir);
+		expect(calls[1]?.cmd).toEqual(["bun", "run", "build"]);
+	});
+	test("does NOT run install when node_modules exists", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		mkdirSync(join(uiDir, "node_modules"));
+		const { runner, calls } = makeRunner([{ exitCode: 0, stderr: "" }]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: () => {} });
+		expect(calls.length).toBe(1);
+		expect(calls[0]?.cmd).toEqual(["bun", "run", "build"]);
+	});
+	test("throws on non-zero install exit; error includes stderr", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		const { runner } = makeRunner([{ exitCode: 1, stderr: "lockfile out of date" }]);
+		await expect(ensureUiBuild({ uiDir, _runner: runner, log: () => {} })).rejects.toThrow(
+			/lockfile out of date/,
+		);
+	});
+	test("throws on non-zero build exit; error includes stderr", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		mkdirSync(join(uiDir, "node_modules"));
+		const { runner } = makeRunner([{ exitCode: 2, stderr: "type error in main.ts" }]);
+		await expect(ensureUiBuild({ uiDir, _runner: runner, log: () => {} })).rejects.toThrow(
+			/type error in main\.ts/,
+		);
+	});
+	test("logs progress messages on build path", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		// node_modules missing → install + build → 3 log lines (install, build, built).
+		const messages: string[] = [];
+		const { runner } = makeRunner([
+			{ exitCode: 0, stderr: "" },
+			{ exitCode: 0, stderr: "" },
+		]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: (m) => messages.push(m) });
+		expect(messages).toEqual(["Installing UI dependencies…", "Building UI…", "UI built"]);
+	});
+	test("no-ops when ui/src is absent (production-install case)", async () => {
+		// No setupUiDir() — tempDir has no ui/ at all, mirroring a fresh `ov init`
+		// in a project that doesn't carry a UI workspace (overstory-916d).
+		const uiDir = join(tempDir, "ui");
+		const messages: string[] = [];
+		const { runner, calls } = makeRunner([]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: (m) => messages.push(m) });
+		// Neither install nor build runs, and no progress messages are emitted.
+		expect(calls.length).toBe(0);
+		expect(messages.length).toBe(0);
+	});
+	test("walks ui/src/ recursively and triggers on nested-file mtime", async () => {
+		const { uiDir } = setupUiDir(tempDir);
+		mkdirSync(join(uiDir, "node_modules"));
+		mkdirSync(join(uiDir, "src", "components"), { recursive: true });
+		writeFileSync(join(uiDir, "src", "components", "App.tsx"), "export default null;");
+		mkdirSync(join(uiDir, "dist"));
+		writeFileSync(join(uiDir, "dist", "index.html"), "<html>old</html>");
+		// dist older than the nested file.
+		const past = new Date(Date.now() - 60_000);
+		const future = new Date(Date.now() + 60_000);
+		utimesSync(join(uiDir, "dist", "index.html"), past, past);
+		utimesSync(join(uiDir, "src", "main.ts"), past, past);
+		utimesSync(join(uiDir, "src", "components", "App.tsx"), future, future);
+		const { runner, calls } = makeRunner([{ exitCode: 0, stderr: "" }]);
+		await ensureUiBuild({ uiDir, _runner: runner, log: () => {} });
+		expect(calls.length).toBe(1);
+	});
+});

package/src/commands/serve/build.ts ADDED Viewed

@@ -0,0 +1,206 @@
+/**
+ * Auto-build helper for `ov serve` (production mode).
+ *
+ * Detects whether ui/dist/index.html is missing or older than any tracked
+ * source/config file in the ui/ workspace, runs `bun install` (only if
+ * node_modules is missing) followed by `bun run build` when so. Silent when
+ * the existing build is up to date.
+ *
+ * No-ops when the project has no `ui/src` directory — the production-install
+ * case where the user's project doesn't carry a UI workspace and is meant to
+ * serve the prebuilt assets shipped inside the @os-eco/overstory-cli package
+ * (see overstory-916d).
+ *
+ * The runner and filesystem helpers are injectable so tests can drive the
+ * decision logic without invoking real subprocess builds.
+ */
+import {
+	existsSync as defaultExistsSync,
+	readdirSync as defaultReaddirSync,
+	statSync as defaultStatSync,
+} from "node:fs";
+import { join, sep } from "node:path";
+export interface RunnerResult {
+	exitCode: number;
+	stderr: string;
+}
+export interface EnsureUiBuildOptions {
+	uiDir: string;
+	log?: (msg: string) => void;
+	_statSync?: typeof defaultStatSync;
+	_existsSync?: typeof defaultExistsSync;
+	_readdirSync?: typeof defaultReaddirSync;
+	_runner?: (cmd: string[], cwd: string) => Promise<RunnerResult>;
+}
+/** Files at the workspace root that affect the build output. */
+const ROOT_TRACKED_FILES = [
+	"index.html",
+	"build.ts",
+	"package.json",
+	"tsconfig.app.json",
+	"tsconfig.json",
+	"components.json",
+];
+/**
+ * Recursively walk a directory, calling visit() on each regular file path.
+ * Skips entries that throw on stat (broken symlinks, races during npm
+ * install). Uses the injected readdir/stat for testability.
+ */
+function walkDir(
+	dir: string,
+	visit: (path: string) => void,
+	exists: typeof defaultExistsSync,
+	readdir: typeof defaultReaddirSync,
+	stat: typeof defaultStatSync,
+): void {
+	if (!exists(dir)) return;
+	let entries: string[];
+	try {
+		entries = readdir(dir);
+	} catch {
+		return;
+	}
+	for (const name of entries) {
+		const full = dir + sep + name;
+		let st: ReturnType<typeof defaultStatSync>;
+		try {
+			st = stat(full);
+		} catch {
+			continue;
+		}
+		if (st.isDirectory()) {
+			walkDir(full, visit, exists, readdir, stat);
+		} else if (st.isFile()) {
+			visit(full);
+		}
+	}
+}
+async function defaultRunner(cmd: string[], cwd: string): Promise<RunnerResult> {
+	if (cmd.length === 0) {
+		throw new Error("ensureUiBuild runner: empty command");
+	}
+	const proc = Bun.spawn(cmd as string[], {
+		cwd,
+		stdout: "pipe",
+		stderr: "pipe",
+	});
+	const stderrChunks: string[] = [];
+	const pipe = async (
+		stream: ReadableStream<Uint8Array> | null,
+		sink: (line: string) => void,
+		capture?: string[],
+	): Promise<void> => {
+		if (stream === null) return;
+		const decoder = new TextDecoder();
+		let buf = "";
+		const reader = stream.getReader();
+		try {
+			while (true) {
+				const { done, value } = await reader.read();
+				if (done) break;
+				const chunk = decoder.decode(value, { stream: true });
+				if (capture) capture.push(chunk);
+				buf += chunk;
+				let idx = buf.indexOf("\n");
+				while (idx !== -1) {
+					sink(buf.slice(0, idx));
+					buf = buf.slice(idx + 1);
+					idx = buf.indexOf("\n");
+				}
+			}
+			if (buf.length > 0) sink(buf);
+		} finally {
+			reader.releaseLock();
+		}
+	};
+	const writeOut = (line: string): void => {
+		process.stderr.write(`[ui-build] ${line}\n`);
+	};
+	await Promise.all([pipe(proc.stdout, writeOut), pipe(proc.stderr, writeOut, stderrChunks)]);
+	const exitCode = await proc.exited;
+	return { exitCode, stderr: stderrChunks.join("") };
+}
+/**
+ * Ensure that ui/dist/ has a build that is newer than every tracked source
+ * file. No-op when the build is current. Throws when a required subprocess
+ * (install / build) fails; the thrown Error includes the captured stderr.
+ */
+export async function ensureUiBuild(opts: EnsureUiBuildOptions): Promise<void> {
+	const exists = opts._existsSync ?? defaultExistsSync;
+	const stat = opts._statSync ?? defaultStatSync;
+	const readdir = opts._readdirSync ?? defaultReaddirSync;
+	const runner = opts._runner ?? defaultRunner;
+	const log =
+		opts.log ??
+		((msg: string): void => {
+			process.stderr.write(`[ui-build] ${msg}\n`);
+		});
+	// Production-install case: when the project has no ui/src, there are no
+	// sources to build from. ov serve falls back to the prebuilt assets shipped
+	// inside the npm package — see resolveUiDistPath() in serve.ts.
+	if (!exists(join(opts.uiDir, "src"))) return;
+	const distIndex = join(opts.uiDir, "dist", "index.html");
+	let needBuild = false;
+	if (!exists(distIndex)) {
+		needBuild = true;
+	} else {
+		let distMtime = 0;
+		try {
+			distMtime = stat(distIndex).mtimeMs;
+		} catch {
+			needBuild = true;
+		}
+		if (!needBuild) {
+			let newest = 0;
+			const visit = (path: string): void => {
+				try {
+					const m = stat(path).mtimeMs;
+					if (m > newest) newest = m;
+				} catch {
+					// File vanished between readdir and stat — ignore.
+				}
+			};
+			walkDir(join(opts.uiDir, "src"), visit, exists, readdir, stat);
+			for (const name of ROOT_TRACKED_FILES) {
+				const full = join(opts.uiDir, name);
+				if (exists(full)) visit(full);
+			}
+			if (newest > distMtime) needBuild = true;
+		}
+	}
+	if (!needBuild) return;
+	if (!exists(join(opts.uiDir, "node_modules"))) {
+		log("Installing UI dependencies…");
+		const install = await runner(["bun", "install"], opts.uiDir);
+		if (install.exitCode !== 0) {
+			throw new Error(
+				`UI dependency install failed (exit ${install.exitCode}): ${install.stderr.trim()}`,
+			);
+		}
+	}
+	log("Building UI…");
+	const build = await runner(["bun", "run", "build"], opts.uiDir);
+	if (build.exitCode !== 0) {
+		throw new Error(`UI build failed (exit ${build.exitCode}): ${build.stderr.trim()}`);
+	}
+	log("UI built");
+}