@clinebot/core 0.0.34 → 0.0.35

package/src/{session/default-session-manager.test.ts → transports/local.test.ts}

@@ -5,11 +5,15 @@ import { join } from "node:path";
 import type { AgentResult } from "@clinebot/shared";
 import { setClineDir, setHomeDir } from "@clinebot/shared/storage";
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
-import { TelemetryService } from "../telemetry/TelemetryService";
+import {
+	type StartSessionInput,
+	splitCoreSessionConfig,
+} from "../runtime/runtime-host";
+import { TelemetryService } from "../services/telemetry/TelemetryService";
+import type { SessionManifest } from "../session/session-manifest";
 import { SessionSource } from "../types/common";
 import type { CoreSessionConfig } from "../types/config";
-import { DefaultSessionManager } from "./default-session-manager";
-import type { SessionManifest } from "./session-manifest";
+import { LocalRuntimeHost as RuntimeHostUnderTest } from "./local";
 
 const distinctId = "test-machine-id";
 
@@ -68,7 +72,7 @@ type PluginEventTestHarness = {
 };
 
 function createPluginEventHarness(
-	manager: DefaultSessionManager,
+	manager: RuntimeHostUnderTest,
 ): PluginEventTestHarness {
 	const target = manager as object;
 	return {
@@ -120,6 +124,7 @@ function createConfig(
 		modelId: "mock-model",
 		cwd: "/tmp/project",
 		systemPrompt: "You are a test agent",
+		mode: "act",
 		enableTools: true,
 		enableSpawnAgent: true,
 		enableAgentTeams: true,
@@ -127,6 +132,18 @@ function createConfig(
 	};
 }
 
+function normalizeStartInput(
+	input: Omit<StartSessionInput, "config" | "localRuntime"> & {
+		config: CoreSessionConfig;
+	},
+): StartSessionInput {
+	const split = splitCoreSessionConfig(input.config);
+	return {
+		...input,
+		...split,
+	};
+}
+
 function createGitRepo(cwd: string): void {
 	execFileSync("git", ["-C", cwd, "init"], { stdio: "pipe" });
 	execFileSync("git", ["-C", cwd, "config", "user.name", "Codex Test"], {
@@ -146,7 +163,7 @@ function createGitRepo(cwd: string): void {
 	});
 }
 
-describe("DefaultSessionManager", () => {
+describe("LocalRuntimeHost", () => {
 	const envSnapshot = {
 		HOME: process.env.HOME,
 		CLINE_DIR: process.env.CLINE_DIR,
@@ -191,8 +208,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest.json",
-				transcriptPath: "/tmp/transcript.log",
-				hookPath: "/tmp/hook.log",
 				messagesPath: "/tmp/messages.json",
 				manifest,
 			}),
@@ -225,7 +240,7 @@ describe("DefaultSessionManager", () => {
 			abort: vi.fn(),
 			shutdown: vi.fn().mockResolvedValue(undefined),
 		};
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder: runtimeBuilder as never,
@@ -233,10 +248,12 @@ describe("DefaultSessionManager", () => {
 			telemetry,
 		});
 
-		await manager.start({
-			config: createConfig({ telemetry, sessionId }),
-			prompt: "hello",
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ telemetry, sessionId }),
+				prompt: "hello",
+			}),
+		);
 
 		expect(adapter.emit).toHaveBeenCalledWith(
 			"session.started",
@@ -281,8 +298,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest.json",
-				transcriptPath: "/tmp/transcript.log",
-				hookPath: "/tmp/hook.log",
 				messagesPath: "/tmp/messages.json",
 				manifest,
 			}),
@@ -312,18 +327,20 @@ describe("DefaultSessionManager", () => {
 			abort: vi.fn(),
 			shutdown: vi.fn().mockResolvedValue(undefined),
 		};
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder: runtimeBuilder as never,
 			createAgent: () => agent as never,
 		});
 
-		const started = await manager.start({
-			source: "kanban",
-			config: createConfig({ sessionId }),
-			prompt: "hello",
-		});
+		const started = await manager.start(
+			normalizeStartInput({
+				source: "kanban",
+				config: createConfig({ sessionId }),
+				prompt: "hello",
+			}),
+		);
 
 		expect(sessionService.createRootSessionWithArtifacts).toHaveBeenCalledWith(
 			expect.objectContaining({
@@ -341,8 +358,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest.json",
-				transcriptPath: "/tmp/transcript.log",
-				hookPath: "/tmp/hook.log",
 				messagesPath: "/tmp/messages.json",
 				manifest,
 			}),
@@ -378,8 +393,6 @@ describe("DefaultSessionManager", () => {
 					isSubagent: false,
 					prompt: null,
 					metadata: null,
-					transcriptPath: "/tmp/transcript.log",
-					hookPath: "/tmp/hook.log",
 					messagesPath: "/tmp/messages.json",
 					updatedAt: "2026-01-01T00:00:00.000Z",
 				},
@@ -406,16 +419,18 @@ describe("DefaultSessionManager", () => {
 			abort: vi.fn(),
 			shutdown: vi.fn().mockResolvedValue(undefined),
 		};
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder: runtimeBuilder as never,
 			createAgent: () => agent as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId, teamName: undefined }),
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId, teamName: undefined }),
+			}),
+		);
 
 		expect(runtimeBuilder.build).toHaveBeenCalledWith(
 			expect.objectContaining({
@@ -435,8 +450,6 @@ describe("DefaultSessionManager", () => {
 		const manifest = createManifest(sessionId);
 		const createRootSessionWithArtifacts = vi.fn().mockResolvedValue({
 			manifestPath: "/tmp/manifest.json",
-			transcriptPath: "/tmp/transcript.log",
-			hookPath: "/tmp/hook.log",
 			messagesPath: "/tmp/messages.json",
 			manifest,
 		});
@@ -482,18 +495,20 @@ describe("DefaultSessionManager", () => {
 			messages: [],
 		};
 
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
 			createAgent: () => agent as never,
 		});
 
-		const started = await manager.start({
-			config: createConfig({ sessionId }),
-			prompt: "hello",
-			interactive: false,
-		});
+		const started = await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "hello",
+				interactive: false,
+			}),
+		);
 
 		expect(started.sessionId).toBe(sessionId);
 		expect(started.result?.finishReason).toBe("completed");
@@ -528,8 +543,6 @@ describe("DefaultSessionManager", () => {
 		};
 		const createRootSessionWithArtifacts = vi.fn().mockResolvedValue({
 			manifestPath: "/tmp/manifest-history-meta.json",
-			transcriptPath: "/tmp/transcript-history-meta.log",
-			hookPath: "/tmp/hook-history-meta.log",
 			messagesPath: "/tmp/messages-history-meta.json",
 			manifest: { ...storedManifest },
 		});
@@ -591,18 +604,20 @@ describe("DefaultSessionManager", () => {
 			messages: [],
 		};
 
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
 			createAgent: () => agent as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			prompt: "hello",
-			interactive: false,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "hello",
+				interactive: false,
+			}),
+		);
 
 		expect(updateSession).toHaveBeenCalledWith({
 			sessionId,
@@ -657,8 +672,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-checkpoint-default-off.json",
-				transcriptPath: "/tmp/transcript-checkpoint-default-off.log",
-				hookPath: "/tmp/hook-checkpoint-default-off.log",
 				messagesPath: "/tmp/messages-checkpoint-default-off.json",
 				manifest,
 			}),
@@ -689,7 +702,7 @@ describe("DefaultSessionManager", () => {
 				};
 			}),
 		};
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -720,11 +733,13 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			prompt: "hello",
-			interactive: false,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "hello",
+				interactive: false,
+			}),
+		);
 		expect(updateSession).toHaveBeenCalledTimes(1);
 		expect(updateSession).toHaveBeenLastCalledWith({
 			sessionId,
@@ -744,8 +759,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-checkpoint-env-on.json",
-				transcriptPath: "/tmp/transcript-checkpoint-env-on.log",
-				hookPath: "/tmp/hook-checkpoint-env-on.log",
 				messagesPath: "/tmp/messages-checkpoint-env-on.json",
 				manifest,
 			}),
@@ -776,7 +789,7 @@ describe("DefaultSessionManager", () => {
 				};
 			}),
 		};
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -807,14 +820,16 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: {
-				...createConfig({ sessionId, cwd: repoCwd }),
-				checkpoint: { enabled: true },
-			},
-			prompt: "hello",
-			interactive: false,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: {
+					...createConfig({ sessionId, cwd: repoCwd }),
+					checkpoint: { enabled: true },
+				},
+				prompt: "hello",
+				interactive: false,
+			}),
+		);
 		expect(updateSession).toHaveBeenCalledTimes(2);
 		expect(updateSession).toHaveBeenNthCalledWith(1, {
 			sessionId,
@@ -843,8 +858,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-meta.json",
-				transcriptPath: "/tmp/transcript-meta.log",
-				hookPath: "/tmp/hook-meta.log",
 				messagesPath: "/tmp/messages-meta.json",
 				manifest,
 			}),
@@ -890,7 +903,7 @@ describe("DefaultSessionManager", () => {
 				],
 			}),
 		);
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -905,15 +918,17 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({
-				sessionId,
-				providerId: "anthropic",
-				modelId: "claude-sonnet-4-6",
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({
+					sessionId,
+					providerId: "anthropic",
+					modelId: "claude-sonnet-4-6",
+				}),
+				prompt: "hello",
+				interactive: false,
 			}),
-			prompt: "hello",
-			interactive: false,
-		});
+		);
 
 		expect(persistSessionMessages).toHaveBeenCalledTimes(1);
 		const persisted = persistSessionMessages.mock.calls[0]?.[1];
@@ -943,8 +958,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest.json",
-				transcriptPath: "/tmp/transcript.log",
-				hookPath: "/tmp/hook.log",
 				messagesPath: "/tmp/messages.json",
 				manifest,
 			}),
@@ -1001,18 +1014,20 @@ describe("DefaultSessionManager", () => {
 			canStartRun: vi.fn().mockReturnValue(true),
 		};
 
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
 			createAgent: () => agent as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			prompt: "hello",
-			interactive: true,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "hello",
+				interactive: true,
+			}),
+		);
 
 		const harness = createPluginEventHarness(manager);
 		await harness.handlePluginEvent(sessionId, {
@@ -1036,8 +1051,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest.json",
-				transcriptPath: "/tmp/transcript.log",
-				hookPath: "/tmp/hook.log",
 				messagesPath: "/tmp/messages.json",
 				manifest,
 			}),
@@ -1071,18 +1084,20 @@ describe("DefaultSessionManager", () => {
 			canStartRun: vi.fn().mockReturnValue(false),
 		};
 
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
 			createAgent: () => agent as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			prompt: "hello",
-			interactive: true,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "hello",
+				interactive: true,
+			}),
+		);
 
 		const harness = createPluginEventHarness(manager);
 
@@ -1105,6 +1120,75 @@ describe("DefaultSessionManager", () => {
 		]);
 	});
 
+	it("drops and ignores queued prompts once a session is aborting", async () => {
+		const sessionId = "sess-abort-pending";
+		const manifest = createManifest(sessionId);
+		const sessionService = {
+			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
+			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
+				manifestPath: "/tmp/manifest.json",
+				messagesPath: "/tmp/messages.json",
+				manifest,
+			}),
+			persistSessionMessages: vi.fn(),
+			updateSessionStatus: vi.fn().mockResolvedValue({
+				updated: true,
+				endedAt: "2026-01-01T00:00:05.000Z",
+			}),
+			writeSessionManifest: vi.fn(),
+			listSessions: vi.fn().mockResolvedValue([]),
+			deleteSession: vi.fn().mockResolvedValue({ deleted: true }),
+		};
+		const runtimeBuilder = {
+			build: vi.fn().mockReturnValue({
+				tools: [],
+				shutdown: vi.fn(),
+			}),
+		};
+		const agent = {
+			run: vi.fn().mockResolvedValue(createResult()),
+			continue: vi.fn().mockResolvedValue(createResult()),
+			abort: vi.fn(),
+			shutdown: vi.fn().mockResolvedValue(undefined),
+			getMessages: vi.fn().mockReturnValue([]),
+			canStartRun: vi.fn().mockReturnValue(false),
+		};
+
+		const manager = new RuntimeHostUnderTest({
+			distinctId,
+			sessionService: sessionService as never,
+			runtimeBuilder,
+			createAgent: () => agent as never,
+		});
+
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "hello",
+				interactive: true,
+			}),
+		);
+
+		const harness = createPluginEventHarness(manager);
+		await harness.handlePluginEvent(sessionId, {
+			name: "queue_message",
+			payload: { prompt: "queued before abort" },
+		});
+		expect(harness.getPendingPrompts(sessionId)).toEqual([
+			{ prompt: "queued before abort", delivery: "queue" },
+		]);
+
+		await manager.abort(sessionId, new Error("test abort"));
+		expect(agent.abort).toHaveBeenCalledTimes(1);
+		expect(harness.getPendingPrompts(sessionId)).toEqual([]);
+
+		await harness.handlePluginEvent(sessionId, {
+			name: "queue_message",
+			payload: { prompt: "queued after abort" },
+		});
+		expect(harness.getPendingPrompts(sessionId)).toEqual([]);
+	});
+
 	it("preserves per-turn metadata on prior assistant messages across turns", async () => {
 		const sessionId = "sess-meta-multi";
 		const manifest = createManifest(sessionId);
@@ -1183,8 +1267,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-meta-multi.json",
-				transcriptPath: "/tmp/transcript-meta-multi.log",
-				hookPath: "/tmp/hook-meta-multi.log",
 				messagesPath: "/tmp/messages-meta-multi.json",
 				manifest,
 			}),
@@ -1194,21 +1276,23 @@ describe("DefaultSessionManager", () => {
 			listSessions: vi.fn().mockResolvedValue([]),
 			deleteSession: vi.fn().mockResolvedValue({ deleted: true }),
 		};
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
 			createAgent: () => agent as never,
 		});
 
-		await manager.start({
-			config: createConfig({
-				sessionId,
-				providerId: "anthropic",
-				modelId: "claude-sonnet-4-6",
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({
+					sessionId,
+					providerId: "anthropic",
+					modelId: "claude-sonnet-4-6",
+				}),
+				interactive: true,
 			}),
-			interactive: true,
-		});
+		);
 
 		await manager.send({ sessionId, prompt: "hello" });
 		await manager.send({ sessionId, prompt: "again" });
@@ -1244,8 +1328,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-failed-turn.json",
-				transcriptPath: "/tmp/transcript-failed-turn.log",
-				hookPath: "/tmp/hook-failed-turn.log",
 				messagesPath: "/tmp/messages-failed-turn.json",
 				manifest,
 			}),
@@ -1265,7 +1347,7 @@ describe("DefaultSessionManager", () => {
 			{ role: "user", content: [{ type: "text", text: "hello" }] },
 			{ role: "assistant", content: [{ type: "text", text: "partial" }] },
 		];
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -1285,11 +1367,13 @@ describe("DefaultSessionManager", () => {
 		});
 
 		await expect(
-			manager.start({
-				config: createConfig({ sessionId }),
-				prompt: "hello",
-				interactive: false,
-			}),
+			manager.start(
+				normalizeStartInput({
+					config: createConfig({ sessionId }),
+					prompt: "hello",
+					interactive: false,
+				}),
+			),
 		).rejects.toThrow("boom");
 
 		expect(persistSessionMessages).toHaveBeenCalledTimes(1);
@@ -1312,8 +1396,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-2.json",
-				transcriptPath: "/tmp/transcript-2.log",
-				hookPath: "/tmp/hook-2.log",
 				messagesPath: "/tmp/messages-2.json",
 				manifest,
 			}),
@@ -1333,7 +1415,7 @@ describe("DefaultSessionManager", () => {
 		const continueFn = vi
 			.fn()
 			.mockResolvedValue(createResult({ text: "second" }));
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -1348,10 +1430,12 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			interactive: true,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				interactive: true,
+			}),
+		);
 		const first = await manager.send({ sessionId, prompt: "first" });
 		const second = await manager.send({ sessionId, prompt: "second" });
 
@@ -1369,8 +1453,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-usage.json",
-				transcriptPath: "/tmp/transcript-usage.log",
-				hookPath: "/tmp/hook-usage.log",
 				messagesPath: "/tmp/messages-usage.json",
 				manifest,
 			}),
@@ -1410,7 +1492,7 @@ describe("DefaultSessionManager", () => {
 				},
 			}),
 		);
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -1425,10 +1507,12 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			interactive: true,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				interactive: true,
+			}),
+		);
 
 		await manager.send({ sessionId, prompt: "first" });
 		expect(await manager.getAccumulatedUsage(sessionId)).toEqual({
@@ -1456,8 +1540,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-queue.json",
-				transcriptPath: "/tmp/transcript-queue.log",
-				hookPath: "/tmp/hook-queue.log",
 				messagesPath: "/tmp/messages-queue.json",
 				manifest,
 			}),
@@ -1478,7 +1560,7 @@ describe("DefaultSessionManager", () => {
 		const continueFn = vi
 			.fn()
 			.mockResolvedValue(createResult({ text: "next" }));
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -1498,10 +1580,12 @@ describe("DefaultSessionManager", () => {
 			events.push(event);
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			interactive: true,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				interactive: true,
+			}),
+		);
 
 		await expect(
 			manager.send({ sessionId, prompt: "queued first", delivery: "queue" }),
@@ -1556,7 +1640,7 @@ describe("DefaultSessionManager", () => {
 	});
 
 	it("returns undefined accumulated usage for unknown sessions", async () => {
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: {
 				ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
@@ -1592,8 +1676,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-fail.json",
-				transcriptPath: "/tmp/transcript-fail.log",
-				hookPath: "/tmp/hook-fail.log",
 				messagesPath: "/tmp/messages-fail.json",
 				manifest,
 			}),
@@ -1612,7 +1694,7 @@ describe("DefaultSessionManager", () => {
 		};
 		const run = vi.fn().mockRejectedValue(new Error("run failed"));
 		const agentShutdown = vi.fn().mockResolvedValue(undefined);
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -1628,11 +1710,13 @@ describe("DefaultSessionManager", () => {
 		});
 
 		await expect(
-			manager.start({
-				config: createConfig({ sessionId }),
-				prompt: "hello",
-				interactive: false,
-			}),
+			manager.start(
+				normalizeStartInput({
+					config: createConfig({ sessionId }),
+					prompt: "hello",
+					interactive: false,
+				}),
+			),
 		).rejects.toThrow("run failed");
 		expect(sessionService.updateSessionStatus).toHaveBeenCalledWith(
 			sessionId,
@@ -1661,7 +1745,7 @@ describe("DefaultSessionManager", () => {
 			}),
 		};
 		const agentShutdown = vi.fn().mockResolvedValue(undefined);
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -1676,10 +1760,12 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		const started = await manager.start({
-			config: createConfig({ sessionId: "sess-no-prompt" }),
-			interactive: true,
-		});
+		const started = await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId: "sess-no-prompt" }),
+				interactive: true,
+			}),
+		);
 		await manager.stop(started.sessionId);
 
 		expect(
@@ -1697,8 +1783,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-oauth.json",
-				transcriptPath: "/tmp/transcript-oauth.log",
-				hookPath: "/tmp/hook-oauth.log",
 				messagesPath: "/tmp/messages-oauth.json",
 				manifest,
 			}),
@@ -1722,7 +1806,7 @@ describe("DefaultSessionManager", () => {
 		};
 		const run = vi.fn().mockResolvedValue(createResult({ text: "ok" }));
 		const updateConnection = vi.fn();
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -1746,14 +1830,16 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({
-				sessionId,
-				providerId: "openai-codex",
-				apiKey: "oauth-access-old",
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({
+					sessionId,
+					providerId: "openai-codex",
+					apiKey: "oauth-access-old",
+				}),
+				interactive: true,
 			}),
-			interactive: true,
-		});
+		);
 		await manager.send({ sessionId, prompt: "hello" });
 
 		expect(updateConnectionDefaults).toHaveBeenCalledWith({
@@ -1772,8 +1858,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-provider-config.json",
-				transcriptPath: "/tmp/transcript-provider-config.log",
-				hookPath: "/tmp/hook-provider-config.log",
 				messagesPath: "/tmp/messages-provider-config.json",
 				manifest,
 			}),
@@ -1800,7 +1884,7 @@ describe("DefaultSessionManager", () => {
 			getMessages: vi.fn().mockReturnValue([]),
 			messages: [],
 		});
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder: {
@@ -1821,14 +1905,16 @@ describe("DefaultSessionManager", () => {
 			} as never,
 		});
 
-		await manager.start({
-			config: createConfig({
-				sessionId,
-				providerId: "vertex",
-				modelId: "claude-sonnet-4@20250514",
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({
+					sessionId,
+					providerId: "vertex",
+					modelId: "claude-sonnet-4@20250514",
+				}),
+				interactive: true,
 			}),
-			interactive: true,
-		});
+		);
 		await manager.send({ sessionId, prompt: "hello" });
 
 		expect(createAgent).toHaveBeenCalledWith(
@@ -1854,8 +1940,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-loop.json",
-				transcriptPath: "/tmp/transcript-loop.log",
-				hookPath: "/tmp/hook-loop.log",
 				messagesPath: "/tmp/messages-loop.json",
 				manifest,
 			}),
@@ -1875,7 +1959,7 @@ describe("DefaultSessionManager", () => {
 			getMessages: vi.fn().mockReturnValue([]),
 			messages: [],
 		});
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder: {
@@ -1887,15 +1971,17 @@ describe("DefaultSessionManager", () => {
 			createAgent: createAgent as never,
 		});
 
-		await manager.start({
-			config: createConfig({
-				sessionId,
-				execution: {
-					loopDetection: { softThreshold: 4, hardThreshold: 8 },
-				},
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({
+					sessionId,
+					execution: {
+						loopDetection: { softThreshold: 4, hardThreshold: 8 },
+					},
+				}),
+				interactive: true,
 			}),
-			interactive: true,
-		});
+		);
 		await manager.send({ sessionId, prompt: "test" });
 
 		expect(createAgent).toHaveBeenCalledWith(
@@ -1914,8 +2000,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-compaction.json",
-				transcriptPath: "/tmp/transcript-compaction.log",
-				hookPath: "/tmp/hook-compaction.log",
 				messagesPath: "/tmp/messages-compaction.json",
 				manifest,
 			}),
@@ -1936,7 +2020,7 @@ describe("DefaultSessionManager", () => {
 			messages: [],
 		});
 		const compact = vi.fn();
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder: {
@@ -1948,17 +2032,19 @@ describe("DefaultSessionManager", () => {
 			createAgent: createAgent as never,
 		});
 
-		await manager.start({
-			config: createConfig({
-				sessionId,
-				compaction: {
-					enabled: true,
-					strategy: "basic",
-					compact,
-				},
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({
+					sessionId,
+					compaction: {
+						enabled: true,
+						strategy: "basic",
+						compact,
+					},
+				}),
+				interactive: true,
 			}),
-			interactive: true,
-		});
+		);
 		await manager.send({ sessionId, prompt: "test" });
 
 		expect(createAgent).toHaveBeenCalledWith(
@@ -1986,8 +2072,6 @@ describe("DefaultSessionManager", () => {
 				ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 				createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 					manifestPath: "/tmp/manifest-format.json",
-					transcriptPath: "/tmp/transcript-format.log",
-					hookPath: "/tmp/hook-format.log",
 					messagesPath: "/tmp/messages-format.json",
 					manifest,
 				}),
@@ -1998,7 +2082,7 @@ describe("DefaultSessionManager", () => {
 				deleteSession: vi.fn().mockResolvedValue({ deleted: true }),
 			};
 			const run = vi.fn().mockResolvedValue(createResult({ text: "ok" }));
-			const manager = new DefaultSessionManager({
+			const manager = new RuntimeHostUnderTest({
 				distinctId,
 				sessionService: sessionService as never,
 				runtimeBuilder: {
@@ -2018,14 +2102,16 @@ describe("DefaultSessionManager", () => {
 					}) as never,
 			});
 
-			await manager.start({
-				config: createConfig({
-					sessionId,
-					cwd: join(tempCwd, "docs"),
-					workspaceRoot: tempCwd,
+			await manager.start(
+				normalizeStartInput({
+					config: createConfig({
+						sessionId,
+						cwd: join(tempCwd, "docs"),
+						workspaceRoot: tempCwd,
+					}),
+					interactive: true,
 				}),
-				interactive: true,
-			});
+			);
 			await manager.send({
 				sessionId,
 				prompt: '<user_input mode="act">explain @src/app.ts</user_input>',
@@ -2049,8 +2135,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-oauth-retry.json",
-				transcriptPath: "/tmp/transcript-oauth-retry.log",
-				hookPath: "/tmp/hook-oauth-retry.log",
 				messagesPath: "/tmp/messages-oauth-retry.json",
 				manifest,
 			}),
@@ -2086,7 +2170,7 @@ describe("DefaultSessionManager", () => {
 				apiKey: "oauth-access-new",
 				refreshed: true,
 			});
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -2106,14 +2190,16 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({
-				sessionId,
-				providerId: "openai-codex",
-				apiKey: "oauth-access-old",
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({
+					sessionId,
+					providerId: "openai-codex",
+					apiKey: "oauth-access-old",
+				}),
+				interactive: true,
 			}),
-			interactive: true,
-		});
+		);
 		const result = await manager.send({ sessionId, prompt: "hello" });
 
 		expect(result?.text).toBe("retried");
@@ -2142,8 +2228,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-team-auto-continue.json",
-				transcriptPath: "/tmp/transcript-team-auto-continue.log",
-				hookPath: "/tmp/hook-team-auto-continue.log",
 				messagesPath: "/tmp/messages-team-auto-continue.json",
 				manifest,
 			}),
@@ -2223,7 +2307,7 @@ describe("DefaultSessionManager", () => {
 				],
 			}),
 		);
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -2238,10 +2322,12 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			interactive: false,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				interactive: false,
+			}),
+		);
 		const result = await manager.send({
 			sessionId,
 			prompt: "run teammate work",
@@ -2282,8 +2368,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-team-task-failure-messages.json",
-				transcriptPath: "/tmp/transcript-team-task-failure-messages.log",
-				hookPath: "/tmp/hook-team-task-failure-messages.log",
 				messagesPath: "/tmp/messages-team-task-failure-messages.json",
 				manifest,
 			}),
@@ -2315,7 +2399,7 @@ describe("DefaultSessionManager", () => {
 			{ role: "user", content: [{ type: "text", text: "delegated prompt" }] },
 			{ role: "assistant", content: [{ type: "text", text: "partial work" }] },
 		];
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -2343,11 +2427,13 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			prompt: "run teammate work",
-			interactive: false,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "run teammate work",
+				interactive: false,
+			}),
+		);
 
 		expect(sessionService.onTeamTaskStart).toHaveBeenCalledTimes(1);
 		expect(sessionService.onTeamTaskEnd).toHaveBeenCalledWith(
@@ -2367,8 +2453,6 @@ describe("DefaultSessionManager", () => {
 			ensureSessionsDir: vi.fn().mockReturnValue("/tmp/sessions"),
 			createRootSessionWithArtifacts: vi.fn().mockResolvedValue({
 				manifestPath: "/tmp/manifest-team-task-progress.json",
-				transcriptPath: "/tmp/transcript-team-task-progress.log",
-				hookPath: "/tmp/hook-team-task-progress.log",
 				messagesPath: "/tmp/messages-team-task-progress.json",
 				manifest,
 			}),
@@ -2397,7 +2481,7 @@ describe("DefaultSessionManager", () => {
 				),
 		};
 
-		const manager = new DefaultSessionManager({
+		const manager = new RuntimeHostUnderTest({
 			distinctId,
 			sessionService: sessionService as never,
 			runtimeBuilder,
@@ -2451,11 +2535,13 @@ describe("DefaultSessionManager", () => {
 				}) as never,
 		});
 
-		await manager.start({
-			config: createConfig({ sessionId }),
-			prompt: "run teammate work",
-			interactive: false,
-		});
+		await manager.start(
+			normalizeStartInput({
+				config: createConfig({ sessionId }),
+				prompt: "run teammate work",
+				interactive: false,
+			}),
+		);
 
 		expect(sessionService.onTeamTaskProgress).toHaveBeenCalledWith(
 			sessionId,