gentle-pi 4.4.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-profile-freeze.test.ts

@@ -282,14 +282,14 @@ test("drift notice: shown once per distinct change of the defaults", () => {
 	f.pinLocal("local");
 	assert.equal(
 		inheritedProfileDriftNotice(opts),
-		'el Gentleman: the default profile changed to "local" (local), this session keeps "frontier" (local). Press Enter on a profile in /gentle:profiles to adopt it.',
+		'el Gentleman: the default profile changed to "local" (local), this session keeps "frontier" (local). Open /gentle:profiles and choose Use in this session to adopt one.',
 	);
 	assert.equal(inheritedProfileDriftNotice(opts), undefined, "the same change is not repeated");
 	f.pinLocal(undefined);
 	f.setActive("team");
 	assert.equal(
 		inheritedProfileDriftNotice(opts),
-		'el Gentleman: the default profile changed to "team" (global), this session keeps "frontier" (local). Press Enter on a profile in /gentle:profiles to adopt it.',
+		'el Gentleman: the default profile changed to "team" (global), this session keeps "frontier" (local). Open /gentle:profiles and choose Use in this session to adopt one.',
 	);
 	assert.equal(inheritedProfileDriftNotice(opts), undefined);
 	assert.equal(readFrozenInheritedProfile("session-a")?.profile?.name, "frontier", "the notice never adopts the new default");
@@ -305,6 +305,8 @@ test("drift notice: an explicit binding learns its profile definition changed in
 	const notice = inheritedProfileDriftNotice(opts);
 	assert.match(notice ?? "", /the profile definition changed to "team"/);
 	assert.match(notice ?? "", /this session keeps the binding from selection/);
+	assert.match(notice ?? "", /Open \/gentle:profiles and choose Use in this session to adopt one\.$/, "the notice names the current adoption action");
+	assert.doesNotMatch(notice ?? "", /Press Enter/, "the retired Enter instruction never comes back");
 	assert.equal(inheritedProfileDriftNotice(opts), undefined, "the same change is not repeated");
 
 	f.editProfile("team", { worker: { model: "openai/older" } });
@@ -344,14 +346,14 @@ test("drift notice covers a session frozen without a profile and a default that
 	f.pinLocal("frontier");
 	assert.equal(
 		inheritedProfileDriftNotice(none),
-		'el Gentleman: the default profile changed to "frontier" (local), this session keeps no profile. Press Enter on a profile in /gentle:profiles to adopt it.',
+		'el Gentleman: the default profile changed to "frontier" (local), this session keeps no profile. Open /gentle:profiles and choose Use in this session to adopt one.',
 	);
 	const kept = options(f, "session-kept");
 	freezeSessionProfileAtStartup(kept);
 	f.pinLocal(undefined);
 	assert.equal(
 		inheritedProfileDriftNotice(kept),
-		'el Gentleman: the default profile changed to no profile, this session keeps "frontier" (local). Press Enter on a profile in /gentle:profiles to adopt it.',
+		'el Gentleman: the default profile changed to no profile, this session keeps "frontier" (local). Open /gentle:profiles and choose Use in this session to adopt one.',
 	);
 });
 

package/tests/shell-bar.test.ts

@@ -107,7 +107,7 @@ test("visual visibility hides only selected optional status segments", () => {
 	assert.doesNotMatch(renderShellHeaderBar(buildShellHeaderModel(data), plainTheme, 160, undefined, settings).text, /gpt-5\.5|ctx|\$9\.49|usage/);
 });
 
-test("Status groups running job descriptions beside RDD without a compact count", () => {
+test("Status keeps RDD beside structured groups while running jobs moved to their own card", () => {
 	const data = model({ jobs: [
 		{ id: "job-2", label: "CI for PR 1860", startedAt: 2 },
 		{ id: "job-1", label: "Build runtime", startedAt: 1 },
@@ -115,12 +115,12 @@ test("Status groups running job descriptions beside RDD without a compact count"
 	for (const width of [30, 60]) {
 		const lines = renderShellSidebarBar(data, plainTheme, width);
 		const text = lines.join("\n");
-		assert.match(text, /Jobs[\s\S]*CI for PR 1860[\s\S]*Build runtime/);
+		// Running jobs live in their own sidebar card; the Status card never
+		// prints them, so the two surfaces cannot duplicate one job.
+		assert.doesNotMatch(text, /Jobs|CI for PR 1860|Build runtime/);
 		assert.match(text, /🌹 RDD/);
-		assert.doesNotMatch(text, /2 jobs/);
 		assert.ok(lines.every((line) => visibleWidth(line) <= width));
 	}
-	assert.doesNotMatch(renderShellSidebarBar(model({ jobs: [] }), plainTheme, 60).join("\n"), /Jobs/);
 });
 
 test("header shows only the running job count when requested and preserves it at mobile widths", () => {

package/tests/shell-sidebar-layout.test.ts

@@ -589,6 +589,23 @@ test("a rail digest refreshes live state that no invalidation announces", (t) =>
 	assert.equal(todoRenders, 2, "explicit invalidation still reaches a rail without a digest");
 });
 
+test("the jobs rail section paints between Status and Agents", (t) => {
+	const f = fixture();
+	sidebarPart(f.tui, "jobs", { render: () => ["jobs"], invalidate() {} }, { digest: () => "jobs", render: () => ["Jobs card"], invalidate() {} });
+	sidebarPart(f.tui, "agents", { render: () => ["agents"], invalidate() {} }, { digest: () => "agents", render: () => ["Agents card"], invalidate() {} });
+	sidebarPart(f.tui, "todo", { render: () => ["todo"], invalidate() {} }, { digest: () => "todo", render: () => ["Todo card"], invalidate() {} });
+	t.after(installSidebar(f.tui, theme));
+	const text = renderLayoutFrame(f.root, 140, 20, () => {}).lines.join("\n");
+	const footer = text.indexOf("Status");
+	const jobs = text.indexOf("Jobs card");
+	const agents = text.indexOf("Agents card");
+	const todo = text.indexOf("Todo card");
+	for (const [name, index] of [["Status", footer], ["Jobs card", jobs], ["Agents card", agents], ["Todo card", todo]] as const) {
+		assert.ok(index >= 0, `${name} paints in the rail`);
+	}
+	assert.ok(footer < jobs && jobs < agents && agents < todo, `sections paint in order, got: ${text}`);
+});
+
 test("a theme switch repaints rail sections whose digest did not change", (t) => {
 	const f = fixture();
 	// Pi hands extensions a live theme proxy and announces a switch only through

package/tests/vim-editor-adapter.test.ts

@@ -277,12 +277,12 @@ test("resolveVimRuntime resolves local bundle cli entrypoint when provided", ()
 });
 
 test("audited Pi editor releases are one frozen exact allowlist that includes the installed TUI", () => {
-  assert.deepEqual([...AUDITED_PI_EDITOR_VERSIONS], ["0.99.1", "0.99.2", "1.0.0", "1.1.0", "1.2.0"]);
+  assert.deepEqual([...AUDITED_PI_EDITOR_VERSIONS], ["0.99.1", "0.99.2", "1.0.0", "1.1.0", "1.2.0", "1.3.0", "1.4.0"]);
   assert.equal(bundledAgent.VERSION, INSTALLED_PI, "bundled and local metadata must describe the same actual installed release");
   assert.ok(Object.isFrozen(AUDITED_PI_EDITOR_VERSIONS));
   assert.throws(() => (AUDITED_PI_EDITOR_VERSIONS as unknown as string[]).push("0.99.3"), TypeError);
   assert.ok(isAuditedPiEditorVersion(INSTALLED_PI), `installed pi-tui ${INSTALLED_PI} must be audited`);
-  for (const unknown of ["0.99.3", "0.99.10", "0.99", "v0.99.2", " 0.99.2", "0.99.2-rc.1", ">=0.99.2", "1.0.1", ">=1.0.0", "v1.0.0", " 1.0.0", "1.0.0-rc.1", "1.2.1", "1.2.0-rc.1", "v1.2.0", ">=1.2.0", undefined, 99.2]) {
+  for (const unknown of ["0.99.3", "0.99.10", "0.99", "v0.99.2", " 0.99.2", "0.99.2-rc.1", ">=0.99.2", "1.0.1", ">=1.0.0", "v1.0.0", " 1.0.0", "1.0.0-rc.1", "1.2.1", "1.2.0-rc.1", "v1.2.0", ">=1.2.0", "1.4.1", "1.5.0", "1.4.0-rc.1", undefined, 99.2]) {
     assert.equal(isAuditedPiEditorVersion(unknown), false, String(unknown));
   }
 });