magus-cli 7.7.0 → 7.9.0

package/src/ui/screens/ModelsScreen.tsx

@@ -7,11 +7,13 @@ import {
 	readModelsConfig,
 	readModelsStatus,
 } from "../../services/models-manager.js";
+import { failureReport } from "../adapters/failureReport.js";
 import {
 	type ModelsBrowserItem,
 	buildModelsItems,
 	firstSelectableIndex,
 } from "../adapters/modelsAdapter.js";
+import { stepFailureReport } from "../adapters/startupReports.js";
 import { EmptyFilterState } from "../components/EmptyFilterState.js";
 import { ScrollableList } from "../components/ScrollableList.js";
 import { ScreenLayout } from "../components/layout/index.js";
@@ -23,7 +25,7 @@ import {
 	renderModelRow,
 	renderModelsSummary,
 } from "../renderers/modelRenderers.js";
-import { useApp } from "../state/AppContext.js";
+import { useApp, useFailurePopup } from "../state/AppContext.js";
 import { useDimensions } from "../state/DimensionsContext.js";
 import { asyncValue } from "../state/types.js";
 import { theme } from "../theme.js";
@@ -134,22 +136,15 @@ export function ModelsScreen() {
 		};
 	}, [dispatch]);
 
-	// ── Status line ───────────────────────────────────────────────────────────
+	// ── Feedback ──────────────────────────────────────────────────────────────
 
 	/**
-	 * Set the status line.
-	 *
-	 * No timer, matching the Styles screen: a message that erased itself after a
-	 * few seconds was usually gone by the time anyone looked. It is cleared when
-	 * the NEXT action starts instead.
+	 * No status line and no messages. A success shows as the screen's state —
+	 * the reloaded list, "Model tiers: on", "writing…" while a write runs. What
+	 * did not happen goes into the one failure popup, with why and, when there
+	 * is one, what to do. A key pressed where it does nothing does nothing.
 	 */
-	const showStatus = useCallback(
-		(text: string, tone: "success" | "error" = "success") => {
-			dispatch({ type: "MODELS_STATUS_SET", status: { text, tone } });
-		},
-		[dispatch],
-	);
-	const statusMsg = modelsState.status;
+	const showFailures = useFailurePopup();
 
 	// ── Derived ───────────────────────────────────────────────────────────────
 
@@ -189,18 +184,17 @@ export function ModelsScreen() {
 	 *
 	 * No `ensureManifest` is passed. That helper prompts on the terminal, and
 	 * this process's terminal belongs to OpenTUI — a project with no profiles
-	 * gets a plain `.claude/models.json`, and the status line says so.
+	 * gets a plain `.claude/models.json`.
+	 *
+	 * A press while a write is in flight does nothing: the header already says
+	 * "writing…". Neither does one with no preset under the cursor — the only
+	 * other row is the status header, which the cursor never lands on, so that
+	 * is an empty filtered list.
 	 */
 	const handleApply = useCallback(async () => {
-		if (modelsState.isApplying) {
-			showStatus("Already writing the routing — one at a time", "error");
-			return;
-		}
-		if (!selectedItem || selectedItem.kind !== "preset") {
-			showStatus("Move to a preset row and press a", "error");
-			return;
-		}
-		const { config, custom } = selectedItem;
+		if (modelsState.isApplying) return;
+		if (!selectedItem || selectedItem.kind !== "preset") return;
+		const { config, custom, label } = selectedItem;
 
 		dispatch({ type: "MODELS_APPLY_START" });
 		try {
@@ -213,24 +207,24 @@ export function ModelsScreen() {
 				projectPath: state.projectPath,
 				preset: custom ? config : config.preset,
 			});
+			// Applied: the reloaded list and the header show it. Said only when
+			// the profile did NOT record it — the next magus install renders the
+			// live settings from the profile and drops the routing.
 			await fetchData();
-			const where = result.profile
-				? `profile "${result.profile}"`
-				: "this project";
-			// Say plainly when the manifest was NOT updated: without that record a
-			// `magus install` rewrites the profile and the routing disappears.
-			const durability =
-				result.profile && !result.recordedInManifest
-					? " — not in profiles.json, so `magus install` will drop it"
-					: "";
-			showStatus(
-				`Applied ${selectedItem.label} in ${where}, hook ${result.hook}${durability}`,
-				result.profile && !result.recordedInManifest ? "error" : "success",
-			);
+			if (result.profile && !result.recordedInManifest) {
+				await showFailures(
+					failureReport(`${label} is active, but not saved in the profile`, [
+						{
+							subject: `profile ${result.profile}`,
+							reason:
+								"the routing is not recorded in it, so the next magus install drops it",
+						},
+					]),
+				);
+			}
 		} catch (error) {
-			showStatus(
-				error instanceof Error ? error.message : String(error),
-				"error",
+			await showFailures(
+				stepFailureReport("No model routing was applied", label, error),
 			);
 		} finally {
 			dispatch({ type: "MODELS_APPLY_END" });
@@ -240,26 +234,24 @@ export function ModelsScreen() {
 		selectedItem,
 		state.projectPath,
 		fetchData,
-		showStatus,
+		showFailures,
 		dispatch,
 	]);
 
+	// Off: the reloaded header reads "Model tiers: off".
 	const handleClear = useCallback(async () => {
-		if (modelsState.isApplying) {
-			showStatus("Already writing the routing — one at a time", "error");
-			return;
-		}
+		if (modelsState.isApplying) return;
 		dispatch({ type: "MODELS_APPLY_START" });
 		try {
 			await clearModels(state.projectPath);
 			await fetchData();
-			showStatus(
-				"Model tiers off — subagents inherit the session model again. The hook stays registered and is a no-op without a config.",
-			);
 		} catch (error) {
-			showStatus(
-				error instanceof Error ? error.message : String(error),
-				"error",
+			await showFailures(
+				stepFailureReport(
+					"Model tiers were not turned off",
+					"model routing",
+					error,
+				),
 			);
 		} finally {
 			dispatch({ type: "MODELS_APPLY_END" });
@@ -268,7 +260,7 @@ export function ModelsScreen() {
 		modelsState.isApplying,
 		state.projectPath,
 		fetchData,
-		showStatus,
+		showFailures,
 		dispatch,
 	]);
 
@@ -301,67 +293,36 @@ export function ModelsScreen() {
 		blur: () => dispatch({ type: "SET_SEARCHING", isSearching: false }),
 	});
 
-	/**
-	 * The previous message belongs to the previous ACTION, so every key this
-	 * screen acts on clears it first. Keys another layer takes — a digit or Tab
-	 * leaving the tab, text typed into the search box — leave it alone, so a
-	 * round trip through another tab keeps it.
-	 */
-	const clearStatus = () => {
-		if (modelsState.status) dispatch({ type: "MODELS_STATUS_CLEAR" });
-	};
-
 	useScreenKeys("models", {
 		// Enter and `a` are the same action here. There is nothing to "open" on
 		// this screen — a preset row has one thing it can do — so binding Enter
 		// to anything else would leave the most obvious key doing nothing.
 		apply: () => {
-			clearStatus();
 			void handleApply();
 		},
 		off: () => {
-			clearStatus();
 			void handleClear();
 		},
 		reload: () => {
-			clearStatus();
 			void fetchData();
 		},
 		search: () => {
-			clearStatus();
 			dispatch({ type: "SET_SEARCHING", isSearching: true });
 		},
 		// Only while a committed filter is set; otherwise Esc is the global back.
 		"clear-filter": () => {
 			if (!modelsState.searchQuery) return false;
-			clearStatus();
 			dispatch({ type: "MODELS_SET_SEARCH", query: "" });
 			dispatch({ type: "MODELS_SELECT", index: 0 });
 			return true;
 		},
 		move: (key) => {
-			clearStatus();
 			move(moveDirection(key));
 		},
 	});
 
 	// ── Chrome ────────────────────────────────────────────────────────────────
 
-	// Transient feedback takes the full-width notice row under the header, never
-	// the header itself: a long message beside the title shrinks the title away
-	// and the two read as one garbled sentence.
-	const noticeContent = statusMsg ? (
-		<text
-			fg={
-				statusMsg.tone === "success"
-					? theme.colors.success
-					: theme.colors.danger
-			}
-		>
-			{statusMsg.text}
-		</text>
-	) : undefined;
-
 	// The file's `preset` string is deliberately not shown beside the state: it records what
 	// was last APPLIED, so a config edited afterwards would wear a built-in's name here while
 	// the list correctly calls it `Custom`.
@@ -398,7 +359,6 @@ export function ModelsScreen() {
 			title="magus Models"
 			currentScreen="models"
 			statusLine={statusContent}
-			notice={noticeContent}
 			search={
 				modelsState.searchQuery || isSearchActive
 					? {
@@ -469,8 +429,8 @@ export function ModelsScreen() {
 								configPath: snapshot?.path ?? null,
 								width: listWidth,
 								// Rows the list did not use. One held back because the
-								// search bar and the notice each take a row from the panel
-								// without taking one from `listPanelHeight` — and an
+								// search bar takes a row from the panel without taking
+								// one from `listPanelHeight` — and an
 								// over-tall panel overprints rather than clipping.
 								available:
 									dimensions.listPanelHeight -