gentle-pi 4.3.0 → 4.5.0

package/tests/runtime-harness.mjs

@@ -33,7 +33,7 @@ const EXPECTED_BANNER_COMMANDS = [
 const EXPECTED_COMMANDS = [
 	"gentle:install-delegation",
 	"gentle:install-review",
-	"gentle:models",
+	"gentle:profiles",
 	"gentle:persona",
 	"gentle:status",
 	"gentle:doctor",
@@ -43,6 +43,8 @@ const EXPECTED_COMMANDS = [
 
 const FORBIDDEN_COMPAT_COMMANDS = [
 	"gentle:install-assets",
+	// Retired: /gentle:profiles opens the same models screen.
+	"gentle:models",
 	"gentle-ai:models",
 	"gentleman:models",
 	"gentle-ai:persona",
@@ -1301,16 +1303,16 @@ async function run() {
 		assert.doesNotMatch(invalidGlobalSkippedAgent, /model: legacy\/provider-model/);
 		assert.equal(legacyCtx.ui.notifications.at(-1).level, "warning");
 		assert.match(legacyCtx.ui.notifications.at(-1).message, /skipped model config/);
+		// An unreadable models.json does not block the models screen (Make default
+		// discloses it), and closing the screen leaves the file untouched.
 		let modelPanelOpened = false;
 		legacyCtx.ui.custom = () => {
 			modelPanelOpened = true;
-			return Promise.resolve({ type: "save", config: {} });
+			return Promise.resolve({ type: "close" });
 		};
-		await commands.get("gentle:models").handler("", legacyCtx);
-		assert.equal(modelPanelOpened, false);
+		await commands.get("gentle:profiles").handler("", legacyCtx);
+		assert.equal(modelPanelOpened, true);
 		assert.equal(await readFile(globalModelsPath, "utf8"), "{ invalid json");
-		assert.equal(legacyCtx.ui.notifications.at(-1).level, "warning");
-		assert.match(legacyCtx.ui.notifications.at(-1).message, /cannot open model config/);
 		await writeFile(globalModelsPath, JSON.stringify({}, null, 2));
 		await hooks.get("session_start")[0]({ reason: "startup" }, legacyCtx);
 		const emptyGlobalPreservesAgent = await readFile(
@@ -1629,61 +1631,50 @@ async function run() {
 			{ provider: "safe", id: "model" },
 			{ provider: "evil\u001b]52;c;Zm9v\u0007", id: "model" },
 		];
-		ctx.ui.custom = (factory) => {
-			const panel = factory({ terminal: { rows: 24 } }, null, null, () => undefined);
-			const initialLines = panel.render(120);
-			const plainInitialLines = initialLines.map(stripAnsi);
-			assert.ok(
-				plainInitialLines[0].startsWith("╭") && plainInitialLines.at(-1).startsWith("╰"),
-				"model panel should render inside a bordered card",
-			);
-			assert.ok(
-				initialLines.length === 24,
-				"model panel should fill the 24-row terminal like /gentle:profiles",
-			);
-			assert.ok(
-				plainInitialLines.some((line) => /↓ \d+ more agent\(s\)/.test(line)),
-				"long model agent list should render a down-scroll indicator",
-			);
-			assert.ok(
-				plainInitialLines.some((line) => line.includes("Continue")),
-				"long model agent list should keep Continue visible",
-			);
-			assert.doesNotMatch(
-				initialLines.join("\n"),
-				/\u001b\]|\u0007/,
-				"model panel must strip unsafe terminal control sequences from agent labels",
-			);
-			assert.doesNotMatch(
-				plainInitialLines.join("\n"),
-				/\]52|\[31m/,
-				"model panel must strip user-provided terminal escapes from labels",
-			);
-			for (let i = 0; i < 20; i++) panel.handleInput("j");
-			const scrolledLines = panel.render(120);
-			const plainScrolledLines = scrolledLines.map(stripAnsi);
-			assert.ok(
-				scrolledLines.length === 24,
-				"scrolled model agent list should keep filling the terminal",
-			);
-			assert.ok(
-				plainScrolledLines.some((line) => /↑ \d+ more agent\(s\)/.test(line)),
-				"long model agent list should render an up-scroll indicator after navigation",
-			);
-			panel.handleInput("G");
-			const bottomLines = panel.render(120);
-			const plainBottomLines = bottomLines.map(stripAnsi);
-			assert.ok(
-				bottomLines.length === 24,
-				"bottom model agent list should keep filling the terminal",
-			);
-			assert.ok(
-				plainBottomLines.some((line) => line.includes("▸ ← Back")),
-				"G should jump to the Back action",
-			);
-			return Promise.resolve({ type: "cancel" });
-		};
-		await commands.get("gentle:models").handler("", ctx);
+		// /gentle:profiles opens the models screen on the profile this repository
+		// resolves: one table, profiles as tabs. Its render contract: a bordered
+		// card filling the terminal, scroll indicators for long agent lists, and
+		// sanitized labels everywhere, including the model card.
+		let panelVisits = 0;
+		ctx.ui.custom = (factory) =>
+			new Promise((resolve) => {
+				panelVisits += 1;
+				const panel = factory({ terminal: { rows: 24 }, requestRender() {} }, null, null, resolve);
+				const initialLines = panel.render(120);
+				const plainInitialLines = initialLines.map(stripAnsi);
+				assert.ok(
+					plainInitialLines[0].startsWith("╭") && plainInitialLines.at(-1).startsWith("╰"),
+					"the models screen should render inside a bordered card",
+				);
+				assert.equal(initialLines.length, 24, "the models screen should fill the 24-row terminal");
+				assert.match(plainInitialLines[1], /current/, "the tab line names the profile on screen");
+				assert.ok(
+					plainInitialLines.some((line) => /↓ \d+ more/.test(line)),
+					"a long agent list should render a down-scroll indicator",
+				);
+				assert.doesNotMatch(initialLines.join("\n"), /\u001b\]|\u0007/, "labels must not carry unsafe control sequences");
+				assert.doesNotMatch(plainInitialLines.join("\n"), /\]52|\[31m/, "labels must not carry user-provided escapes");
+				for (let i = 0; i < 20; i++) panel.handleInput("j");
+				const plainScrolledLines = panel.render(120).map(stripAnsi);
+				assert.equal(plainScrolledLines.length, 24, "the scrolled list keeps filling the terminal");
+				assert.ok(
+					plainScrolledLines.some((line) => /↑ \d+ more/.test(line)),
+					"navigation down should render an up-scroll indicator",
+				);
+				for (let i = 0; i < 40; i++) panel.handleInput("j");
+				assert.ok(
+					panel.render(120).map(stripAnsi).some((line) => /▸\s+worker\b/.test(line)),
+					"moving past the end lands on the last agent row",
+				);
+				panel.handleInput("\r"); // the model card lists the registry, sanitized
+				const card = panel.render(120);
+				assert.match(card.map(stripAnsi).join("\n"), /Model for worker/);
+				assert.doesNotMatch(card.join("\n"), /\u001b\]|\u0007/, "registry labels are sanitized in the model card");
+				panel.handleInput("\u001b"); // close the card
+				panel.handleInput("\u001b"); // close the screen: nothing changed, one esc is enough
+			});
+		await commands.get("gentle:profiles").handler("", ctx);
+		assert.equal(panelVisits, 1, "closing an unchanged screen ends the command");
 
 		await hooks.get("session_start")[0]({ reason: "startup" }, ctx);
 		const legacyAppliedAgent = await readFile(
@@ -1693,154 +1684,140 @@ async function run() {
 		assert.match(legacyAppliedAgent, /model: openai\/gpt-5/);
 		assert.doesNotMatch(legacyAppliedAgent, /thinking:/);
 
-		ctx.ui.custom = () =>
-			Promise.resolve({
-				type: "save",
-				config: {
-					"sdd-apply": { model: "openai/gpt-5", thinking: "high" },
-					worker: { model: "openai/gpt-5-mini", thinking: "low" },
-					researcher: { model: "openai/gpt-5-mini", thinking: "low" },
-					"project-special": { model: "openai/gpt-5-mini", thinking: "low" },
-					"global-special": { model: "openai/gpt-5-mini", thinking: "low" },
-				},
-			});
-		await commands.get("gentle:models").handler("", ctx);
-		assert.doesNotMatch(
-			ctx.ui.notifications.at(-1).message,
-			/[\u001b\u0007]/,
-			"model save notification must strip terminal control sequences from discovered agent names",
-		);
-
-		const savedConfig = JSON.parse(
-			await readFile(globalModelsPath, "utf8"),
-		);
-		assert.deepEqual(savedConfig["sdd-apply"], {
-			model: "openai/gpt-5",
-			thinking: "high",
-		});
-		assert.equal(
-			existsSync(join(modelsCwd, ".pi", "gentle-ai", "models.json")),
-			false,
-			"/gentle:models must save model routing globally, not per project",
-		);
-
-		const applyAgent = await readFile(
-			join(modelsCwd, ".pi", "agents", "sdd-apply.md"),
-			"utf8",
-		);
-		assert.match(applyAgent, /model: openai\/gpt-5/);
-		assert.match(applyAgent, /thinking: high/);
-
-		const projectSubagents = JSON.parse(
-			await readFile(join(modelsCwd, ".pi", "subagents.json"), "utf8"),
-		);
-		assert.equal(
-			projectSubagents.model_profiles["sdd-apply"].model,
-			"openai/gpt-5",
-		);
-		assert.equal(projectSubagents.model_profiles["sdd-apply"].effort, "high");
-		assert.equal(
-			projectSubagents.model_profiles.worker.model,
-			"openai/gpt-5-mini",
-		);
-		assert.equal(projectSubagents.model_profiles.worker.effort, "low");
-		const routedProjectSubagentWorker = await readFile(
-			join(modelsCwd, ".pi", "subagents", "worker.md"),
+		// Save writes the table into the profile store only (kind
+		// gentle-pi.agent_model_profiles). Global models.json, subagents.json, and
+		// agent frontmatter stay untouched — Make default is the guarded global apply.
+		const projectSubagentsBefore = await readFile(
+			join(modelsCwd, ".pi", "subagents.json"),
 			"utf8",
 		);
-		assert.match(routedProjectSubagentWorker, /model: openai\/gpt-5-mini/);
-		assert.match(routedProjectSubagentWorker, /thinking: low/);
-		const shadowedProjectAgentWorker = await readFile(
-			join(modelsCwd, ".pi", "agents", "worker.md"),
+		const agentFrontmatterBefore = await readFile(
+			join(modelsCwd, ".pi", "agents", "sdd-apply.md"),
 			"utf8",
 		);
-		assert.match(shadowedProjectAgentWorker, /model: existing\/project-worker/);
-		assert.match(shadowedProjectAgentWorker, /thinking: high/);
-		assert.equal(
-			projectSubagents.model_profiles["project-special"].model,
-			"openai/gpt-5-mini",
-		);
-		assert.equal(projectSubagents.model_profiles["project-special"].effort, "low");
-		const globalSubagents = JSON.parse(await readFile(globalSubagentsPath, "utf8"));
+		const pickEffort = (panel, steps) => {
+			panel.handleInput("e");
+			for (let i = 0; i < 10; i++) panel.handleInput("\u001b[D"); // back to inherit
+			for (let i = 0; i < steps; i++) panel.handleInput("\u001b[C");
+			panel.handleInput("\r");
+		};
+		let saveVisits = 0;
+		ctx.ui.custom = (factory) =>
+			new Promise((resolve) => {
+				saveVisits += 1;
+				const panel = factory({ terminal: { rows: 24 }, requestRender() {} }, null, null, resolve);
+				if (saveVisits === 1) {
+					panel.handleInput("j"); // the Model ▾ / Effort ▾ header row: every agent
+					panel.handleInput("\r");
+					for (const character of "safe") panel.handleInput(character); // type-to-search
+					panel.handleInput("\r"); // safe/model for every agent
+					pickEffort(panel, 5); // inherit off minimal low medium [high]
+					panel.handleInput("\u0013"); // ctrl+s
+					return;
+				}
+				panel.handleInput("\u001b");
+			});
+		await commands.get("gentle:profiles").handler("", ctx);
+		assert.equal(saveVisits, 2, "Save reopens the screen once before it closes");
+
+		const profilesPath = join(globalConfigHome, "profiles.json");
+		const savedStore = JSON.parse(await readFile(profilesPath, "utf8"));
+		assert.equal(savedStore.kind, "gentle-pi.agent_model_profiles");
+		assert.equal(savedStore.version, 1);
+		assert.deepEqual(savedStore.profiles.current["sdd-apply"], { model: "safe/model", thinking: "high" });
+		assert.deepEqual(savedStore.profiles.current["project-special"], { model: "safe/model", thinking: "high" });
 		assert.equal(
-			globalSubagents.model_profiles.researcher.model,
-			"openai/gpt-5-mini",
+			await readFile(globalModelsPath, "utf8"),
+			JSON.stringify({ "sdd-apply": "openai/gpt-5" }, null, 2),
+			"Save must keep routing edits in the profile store, not write global models.json",
 		);
-		assert.equal(globalSubagents.model_profiles.researcher.effort, "low");
 		assert.equal(
-			globalSubagents.model_profiles["global-special"].model,
-			"openai/gpt-5-mini",
+			existsSync(join(modelsCwd, ".pi", "gentle-ai", "models.json")),
+			false,
+			"Save must keep routing edits in the profile store, not per project",
 		);
-		assert.equal(globalSubagents.model_profiles["global-special"].effort, "low");
+		assert.equal(await readFile(join(modelsCwd, ".pi", "subagents.json"), "utf8"), projectSubagentsBefore);
+		const untouchedAgent = await readFile(join(modelsCwd, ".pi", "agents", "sdd-apply.md"), "utf8");
+		assert.equal(untouchedAgent, agentFrontmatterBefore);
+		assert.doesNotMatch(untouchedAgent, /thinking:/);
 		assert.equal(existsSync(join(modelsCwd, ".pi", "settings.json")), false);
 
 		const kittyE = "\x1b[101u";
 		assert.notEqual(kittyE, "e");
 		assert.equal(matchesKey(kittyE, "e"), true);
 
-		let customPanelCalls = 0;
+		// Custom model ids: Custom model id… in the model card hands the draft back,
+		// ctx.ui.input asks for the id, the screen reopens with it, and Save stores it.
+		const chooseCustom = (panel) => {
+			panel.handleInput("\r");
+			for (const character of "zzzz") panel.handleInput(character); // nothing matches: only Custom model id… is left
+			panel.handleInput("\r");
+		};
+		let customVisits = 0;
 		ctx.ui.input = async () => "custom/provider-model";
 		ctx.ui.custom = (factory) =>
 			new Promise((resolve) => {
-				customPanelCalls += 1;
-				const panel = factory(null, null, null, resolve);
-				if (customPanelCalls === 1) {
-					panel.handleInput(kittyE); // effort picker for all agents
-					for (let i = 0; i < 4; i++) panel.handleInput("j"); // medium
+				customVisits += 1;
+				const panel = factory({ terminal: { rows: 24 }, requestRender() {} }, null, null, resolve);
+				if (customVisits === 1) {
+					panel.handleInput("j"); // every agent
+					panel.handleInput(kittyE); // the effort row (a kitty "e" still matches)
+					for (let i = 0; i < 10; i++) panel.handleInput("\u001b[D");
+					for (let i = 0; i < 4; i++) panel.handleInput("\u001b[C"); // medium
 					panel.handleInput("\r");
-					panel.handleInput("c"); // custom model from the same unsaved draft
+					chooseCustom(panel); // custom model for every agent, same unsaved draft
+					return;
+				}
+				if (customVisits === 2) {
+					panel.handleInput("\u0013"); // ctrl+s saves the draft reopened after the custom id
 					return;
 				}
-				panel.handleInput("\u0013"); // ctrl+s saves the draft reopened after custom model input
+				panel.handleInput("\u001b");
 			});
-		await commands.get("gentle:models").handler("", ctx);
-
-		const customSavedConfig = JSON.parse(
-			await readFile(globalModelsPath, "utf8"),
-		);
-		assert.deepEqual(customSavedConfig["sdd-apply"], {
+		await commands.get("gentle:profiles").handler("", ctx);
+		assert.equal(customVisits, 3);
+		const customSavedStore = JSON.parse(await readFile(profilesPath, "utf8"));
+		assert.deepEqual(customSavedStore.profiles.current["sdd-apply"], {
 			model: "custom/provider-model",
 			thinking: "medium",
 		});
 
-		let invalidCustomCalls = 0;
+		// An invalid custom id is refused on the screen and changes nothing.
+		let invalidCustomVisits = 0;
 		ctx.ui.input = async () => "bad\nmodel: injected";
 		ctx.ui.custom = (factory) =>
 			new Promise((resolve) => {
-				invalidCustomCalls += 1;
-				const panel = factory(null, null, null, resolve);
-				if (invalidCustomCalls === 1) {
-					panel.handleInput("c");
+				invalidCustomVisits += 1;
+				const panel = factory({ terminal: { rows: 24 }, requestRender() {} }, null, null, resolve);
+				if (invalidCustomVisits === 1) {
+					panel.handleInput("j");
+					chooseCustom(panel);
 					return;
 				}
-				panel.handleInput("\u001b");
+				if (invalidCustomVisits === 2) {
+					assert.match(panel.render(120).map(stripAnsi).join(" "), /Custom model id must be one provider\/model line/);
+				}
+				panel.handleInput("\u001b"); // nothing changed: one esc closes
 			});
-		await commands.get("gentle:models").handler("", ctx);
-		assert.match(
-			ctx.ui.notifications.at(-1).message,
-			/Custom model id must be a single-line/,
-		);
-		const rejectedCustomConfig = JSON.parse(
-			await readFile(globalModelsPath, "utf8"),
-		);
-		assert.deepEqual(rejectedCustomConfig["sdd-apply"], {
+		await commands.get("gentle:profiles").handler("", ctx);
+		assert.equal(invalidCustomVisits, 2);
+		const rejectedCustomStore = JSON.parse(await readFile(profilesPath, "utf8"));
+		assert.deepEqual(rejectedCustomStore.profiles.current["sdd-apply"], {
 			model: "custom/provider-model",
 			thinking: "medium",
 		});
 
-		let exportPanelCalls = 0;
+		// models.json export and restore are subcommands; neither opens the screen.
+		let screenOpened = false;
 		ctx.ui.custom = () => {
-			exportPanelCalls += 1;
-			return Promise.resolve(exportPanelCalls === 1 ? { type: "export", config: {} } : { type: "cancel" });
+			screenOpened = true;
+			return Promise.resolve({ type: "close" });
 		};
-		await commands.get("gentle:models").handler("", ctx);
+		await commands.get("gentle:profiles").handler("export-routing", ctx);
 		const exported = JSON.parse(await readFile(join(globalConfigHome, "models.export.json"), "utf8"));
 		assert.equal(exported.kind, "gentle-pi.agent_model_routing");
 		assert.equal(exported.version, 1);
-		assert.deepEqual(exported.agents["sdd-apply"], {
-			model: "custom/provider-model",
-			thinking: "medium",
-		});
+		assert.deepEqual(exported.agents["sdd-apply"], { model: "openai/gpt-5" }, "export reads the saved global routing");
 
 		await writeFile(
 			join(globalConfigHome, "models.export.json"),
@@ -1850,59 +1827,47 @@ async function run() {
 				agents: { "sdd-apply": { model: "restore/provider", thinking: "high" } },
 			}, null, 2),
 		);
-		let restorePanelCalls = 0;
 		ctx.ui.confirm = async () => true;
-		ctx.ui.custom = () => {
-			restorePanelCalls += 1;
-			return Promise.resolve(restorePanelCalls === 1 ? { type: "restore", config: {} } : { type: "cancel" });
-		};
-		await commands.get("gentle:models").handler("", ctx);
+		await commands.get("gentle:profiles").handler("restore-routing", ctx);
+		assert.equal(screenOpened, false, "export and restore never open the screen");
 		const restoredConfig = JSON.parse(await readFile(globalModelsPath, "utf8"));
-		assert.deepEqual(restoredConfig["sdd-apply"], {
-			model: "restore/provider",
-			thinking: "high",
-		});
+		assert.deepEqual(restoredConfig["sdd-apply"], { model: "restore/provider", thinking: "high" });
 		const restoredAgent = await readFile(join(modelsCwd, ".pi", "agents", "sdd-apply.md"), "utf8");
 		assert.match(restoredAgent, /model: restore\/provider/);
 		assert.match(restoredAgent, /thinking: high/);
+		// Restore is global-only: the profile store keeps its own routing.
+		const afterRestoreStore = JSON.parse(await readFile(profilesPath, "utf8"));
+		assert.equal(afterRestoreStore.profiles.current["sdd-apply"].model, "custom/provider-model");
 
-		// issue #286: `thinking: "max"` must survive save normalization and
-		// reach both subagents.json (effort) and agent frontmatter (thinking).
-		ctx.ui.custom = () =>
-			Promise.resolve({
-				type: "save",
-				config: { "sdd-apply": { model: "openai/gpt-5", thinking: "max" } },
-			});
-		await commands.get("gentle:models").handler("", ctx);
-		const maxSavedConfig = JSON.parse(await readFile(globalModelsPath, "utf8"));
-		assert.equal(maxSavedConfig["sdd-apply"].thinking, "max");
-		const maxSubagents = JSON.parse(
-			await readFile(join(modelsCwd, ".pi", "subagents.json"), "utf8"),
-		);
-		assert.equal(maxSubagents.model_profiles["sdd-apply"].effort, "max");
-		const maxApplyAgent = await readFile(
-			join(modelsCwd, ".pi", "agents", "sdd-apply.md"),
-			"utf8",
-		);
-		assert.match(maxApplyAgent, /thinking: max/);
-
-		// issue #286: effort picker must offer `max` after `xhigh` and save it.
-		let maxPickerCalls = 0;
+		// issue #286: `thinking: "max"` must survive save normalization into the
+		// profile store, and the effort row must offer `max` after `xhigh`.
+		let maxVisits = 0;
 		ctx.ui.custom = (factory) =>
 			new Promise((resolve) => {
-				maxPickerCalls += 1;
-				const panel = factory(null, null, null, resolve);
-				if (maxPickerCalls === 1) {
-					panel.handleInput(kittyE); // open effort picker (set-all row)
-					for (let i = 0; i < 7; i++) panel.handleInput("j"); // max
+				maxVisits += 1;
+				const panel = factory({ terminal: { rows: 24 }, requestRender() {} }, null, null, resolve);
+				if (maxVisits === 1) {
+					panel.handleInput("j"); // every agent
+					panel.handleInput("e");
+					const strip = panel.render(120).map(stripAnsi).join("\n");
+					const xhighIndex = strip.indexOf("xhigh");
+					assert.ok(xhighIndex >= 0 && strip.indexOf("max", xhighIndex + 5) > xhighIndex, "the effort row must offer max after xhigh");
+					for (let i = 0; i < 10; i++) panel.handleInput("\u001b[C"); // to the last level: max
 					panel.handleInput("\r");
-					panel.handleInput("\u0013"); // ctrl+s saves the draft
+					panel.handleInput("\u0013");
 					return;
 				}
+				panel.handleInput("\u001b");
 			});
-		await commands.get("gentle:models").handler("", ctx);
-		const pickerMaxConfig = JSON.parse(await readFile(globalModelsPath, "utf8"));
-		assert.equal(pickerMaxConfig["sdd-apply"].thinking, "max");
+		await commands.get("gentle:profiles").handler("", ctx);
+		assert.equal(maxVisits, 2);
+		const maxStore = JSON.parse(await readFile(profilesPath, "utf8"));
+		assert.equal(maxStore.profiles.current["sdd-apply"].thinking, "max");
+		assert.equal(
+			maxStore.profiles.current["sdd-apply"].model,
+			"custom/provider-model",
+			"an effort-only save keeps the profile's saved model",
+		);
 	} finally {
 		await rm(modelsCwd, { recursive: true, force: true });
 		await rm(globalModelsPath, { force: true });
@@ -1919,7 +1884,19 @@ async function run() {
 	}
 }
 
-run().catch((error) => {
+// A run that awaits a promise nobody settles (a mocked panel that never
+// finishes) lets the event loop drain, and Node would exit 0 with every later
+// section skipped. Treat an unfinished run as a failure instead.
+let runCompleted = false;
+process.on("exit", () => {
+	if (runCompleted || process.exitCode) return;
+	console.error("runtime harness exited before run() completed: a section awaited a promise that never settled");
+	process.exitCode = 1;
+});
+
+run().then(() => {
+	runCompleted = true;
+}).catch((error) => {
 	console.error(error);
 	process.exitCode = 1;
 }).finally(async () => {

package/tests/session-changes.test.ts

@@ -4,6 +4,7 @@ import { mkdtemp, writeFile, rm } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { SessionChanges, readChangeSnapshot, sessionChangeRoots, SESSION_CHANGE_ENTRY, MAX_CHANGE_BYTES } from "../lib/session-changes.ts";
+import { emptyChanges } from "../lib/shell-changes.ts";
 import { SESSION_WORKTREE_ENTRY } from "../lib/session-worktree-registry.ts";
 
 const text = (text: string) => ({ kind: "text" as const, text });
@@ -12,7 +13,7 @@ const evidence = (id = "a", root = "/repo", path = "file.ts", before = text("old
 
 test("session changes starts empty without reading the worktree or its untracked contents", () => {
 	const changes = new SessionChanges("session");
-	assert.deepEqual(changes.model, { files: [], added: 0, deleted: 0 });
+	assert.deepEqual(changes.model, { ...emptyChanges(), digest: emptyChanges().digest });
 	assert.deepEqual(changes.worktrees, []);
 });