@os-eco/overstory-cli 0.7.0 → 0.7.3

package/src/commands/doctor.test.ts

@@ -10,10 +10,11 @@
  */
 
 import { afterEach, beforeEach, describe, expect, test } 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 { ValidationError } from "../errors.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import { doctorCommand } from "./doctor.ts";
 
 describe("doctorCommand", () => {
@@ -47,7 +48,7 @@ describe("doctorCommand", () => {
 	afterEach(async () => {
 		process.stdout.write = originalWrite;
 		process.chdir(originalCwd);
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	function output(): string {

package/src/commands/errors.test.ts

@@ -9,11 +9,12 @@
  */
 
 import { afterEach, beforeEach, describe, expect, test } 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 { ValidationError } from "../errors.ts";
 import { createEventStore } from "../events/store.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { InsertEvent } from "../types.ts";
 import { errorsCommand } from "./errors.ts";
 
@@ -64,7 +65,7 @@ describe("errorsCommand", () => {
 	afterEach(async () => {
 		process.stdout.write = originalWrite;
 		process.chdir(originalCwd);
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	function output(): string {

package/src/commands/feed.test.ts

@@ -9,11 +9,12 @@
  */
 
 import { afterEach, beforeEach, describe, expect, test } 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 { ValidationError } from "../errors.ts";
 import { createEventStore } from "../events/store.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { InsertEvent } from "../types.ts";
 import { feedCommand } from "./feed.ts";
 
@@ -64,7 +65,7 @@ describe("feedCommand", () => {
 	afterEach(async () => {
 		process.stdout.write = originalWrite;
 		process.chdir(originalCwd);
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	function output(): string {

package/src/commands/feed.ts

@@ -13,14 +13,7 @@ import { ValidationError } from "../errors.ts";
 import { createEventStore } from "../events/store.ts";
 import { jsonOutput } from "../json.ts";
 import type { ColorFn } from "../logging/color.ts";
-import { color } from "../logging/color.ts";
-import {
-	buildAgentColorMap,
-	buildEventDetail,
-	extendAgentColorMap,
-	formatAbsoluteTime,
-} from "../logging/format.ts";
-import { eventLabel } from "../logging/theme.ts";
+import { buildAgentColorMap, extendAgentColorMap, formatEventLine } from "../logging/format.ts";
 import type { StoredEvent } from "../types.ts";
 
 /**
@@ -28,27 +21,7 @@ import type { StoredEvent } from "../types.ts";
  * HH:MM:SS LABEL agentname    detail
  */
 function printEvent(event: StoredEvent, colorMap: Map<string, ColorFn>): void {
-	const w = process.stdout.write.bind(process.stdout);
-
-	const timeStr = formatAbsoluteTime(event.createdAt);
-
-	const label = eventLabel(event.eventType);
-
-	const levelColorFn =
-		event.level === "error" ? color.red : event.level === "warn" ? color.yellow : null;
-	const applyLevel = (text: string) => (levelColorFn ? levelColorFn(text) : text);
-
-	const detail = buildEventDetail(event, 60);
-	const detailSuffix = detail ? ` ${color.dim(detail)}` : "";
-
-	const agentColorFn = colorMap.get(event.agentName) ?? color.gray;
-	const agentLabel = ` ${agentColorFn(event.agentName.padEnd(15))}`;
-
-	w(
-		`${color.dim(timeStr)} ` +
-			`${applyLevel(label.color(color.bold(label.compact)))}` +
-			`${agentLabel}${detailSuffix}\n`,
-	);
+	process.stdout.write(`${formatEventLine(event, colorMap)}\n`);
 }
 
 interface FeedOpts {

package/src/commands/init.test.ts

@@ -17,7 +17,6 @@ const AGENT_DEF_FILES = [
 	"reviewer.md",
 	"lead.md",
 	"merger.md",
-	"supervisor.md",
 	"coordinator.md",
 	"monitor.md",
 ];
@@ -46,7 +45,7 @@ describe("initCommand: agent-defs deployment", () => {
 		await cleanupTempDir(tempDir);
 	});
 
-	test("creates .overstory/agent-defs/ with all 8 agent definition files", async () => {
+	test("creates .overstory/agent-defs/ with all 7 agent definition files (supervisor deprecated)", async () => {
 		await initCommand({});
 
 		const agentDefsDir = join(tempDir, ".overstory", "agent-defs");

package/src/commands/init.ts

@@ -241,14 +241,6 @@ function buildAgentManifest(): AgentManifest {
 			canSpawn: true,
 			constraints: ["read-only", "no-worktree"],
 		},
-		supervisor: {
-			file: "supervisor.md",
-			model: "opus",
-			tools: ["Read", "Write", "Edit", "Glob", "Grep", "Bash", "Task"],
-			capabilities: ["coordinate", "supervise"],
-			canSpawn: true,
-			constraints: [],
-		},
 		monitor: {
 			file: "monitor.md",
 			model: "sonnet",
@@ -595,6 +587,7 @@ export async function initCommand(opts: InitOptions): Promise<void> {
 	const agentDefFiles = await readdir(overstoryAgentsDir);
 	for (const fileName of agentDefFiles) {
 		if (!fileName.endsWith(".md")) continue;
+		if (fileName === "supervisor.md") continue; // Deprecated: not deployed to new projects
 		const source = Bun.file(join(overstoryAgentsDir, fileName));
 		const content = await source.text();
 		await Bun.write(join(agentDefsTarget, fileName), content);

package/src/commands/inspect.test.ts

@@ -9,13 +9,14 @@
  */
 
 import { afterEach, beforeEach, describe, expect, test } 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 { ValidationError } from "../errors.ts";
 import { createEventStore } from "../events/store.ts";
 import { createMetricsStore } from "../metrics/store.ts";
 import { createSessionStore } from "../sessions/store.ts";
+import { cleanupTempDir } from "../test-helpers.ts";
 import type { InsertEvent, SessionMetrics } from "../types.ts";
 import { gatherInspectData, inspectCommand } from "./inspect.ts";
 
@@ -89,7 +90,7 @@ describe("inspectCommand", () => {
 	afterEach(async () => {
 		process.stdout.write = originalWrite;
 		process.chdir(originalCwd);
-		await rm(tempDir, { recursive: true, force: true });
+		await cleanupTempDir(tempDir);
 	});
 
 	function output(): string {
@@ -153,6 +154,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -185,6 +187,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -221,6 +224,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date(Date.now() - 5_000).toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -257,6 +261,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -296,6 +301,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -331,6 +337,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -366,6 +373,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -410,6 +418,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -453,6 +462,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -502,6 +512,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -539,6 +550,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -577,6 +589,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -614,6 +627,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();
 
@@ -652,6 +666,7 @@ describe("inspectCommand", () => {
 				lastActivity: new Date().toISOString(),
 				escalationLevel: 0,
 				stalledSince: null,
+				transcriptPath: null,
 			});
 			store.close();