magus-cli 7.6.0 → 7.7.0

package/src/ui/hooks/useProfileStartupModal.ts

@@ -17,10 +17,9 @@
 
 import { useCallback, useRef } from "react";
 import { getManifestPath } from "../../services/manifest.js";
-import {
-	type ActivationResult,
-	type ProfileChoice,
-	activationNotes,
+import type {
+	ActivationResult,
+	ProfileChoice,
 } from "../../services/profile-activate.js";
 import type { DisplacedArtifact } from "../../services/profile-live.js";
 import {
@@ -37,8 +36,8 @@ import {
 } from "../../services/profile-startup.js";
 import { applyProfileChoice } from "../../services/profile-state.js";
 import { SettingsParseError } from "../../services/settings-file.js";
-import { useApp, useNavigation } from "../state/AppContext.js";
-import type { ModalState } from "../state/types.js";
+import { useApp, useNotify } from "../state/AppContext.js";
+import type { ModalState, NoticeLine } from "../state/types.js";
 
 export interface BuildProfileModalArgs {
 	need: ProfileStartupNeed;
@@ -85,38 +84,79 @@ export function buildProfileStartupModal(
 	};
 }
 
-/** What changed, phrased for a message modal rather than a terminal. */
-export function describeOutcome(
-	profileId: string,
-	displaced: DisplacedArtifact[],
-	projectPath: string,
-	/** True when this happened without asking, following the committed default. */
-	automatic = false,
-	/** Anything else activation reported (see `activationNotes`). */
-	notes: string[] = [],
-): string {
-	const lines = [
-		automatic
-			? `This project declares "${profileId}" as its profile, so magus activated it.`
-			: `"${profileId}" is now the active profile.`,
+/** `typescript-lsp@claude-plugins-official` → `typescript-lsp`. */
+function pluginName(pluginId: string): string {
+	const at = pluginId.lastIndexOf("@");
+	return at > 0 ? pluginId.slice(0, at) : pluginId;
+}
+
+/** "a", "a and b", "a, b and c". */
+function listed(names: readonly string[]): string {
+	if (names.length <= 1) return names.join("");
+	return `${names.slice(0, -1).join(", ")} and ${names[names.length - 1]}`;
+}
+
+/**
+ * What an activation did, as notices — for the startup question and the
+ * Profiles tab alike: one line saying which profile is in use, then
+ * {@link activationWarnings}.
+ */
+export function activationNotices(result: ActivationResult): NoticeLine[] {
+	return [
+		{ text: `Using profile "${result.profileId}"`, tone: "success" },
+		...activationWarnings(result),
 	];
-	if (displaced.length > 0) {
-		lines.push(
-			"",
-			"These were moved aside to make room for the profile's files, and are kept:",
-			...displaced.map((d) => {
-				const from = d.link.startsWith(projectPath)
-					? d.link.slice(projectPath.length + 1)
-					: d.link;
-				const to = d.savedTo.startsWith(projectPath)
-					? d.savedTo.slice(projectPath.length + 1)
-					: d.savedTo;
-				return `  ${from} → ${to}`;
-			}),
-		);
+}
+
+/**
+ * What an activation left for the user to act on, one warning per thing, in
+ * user terms. Alone, without {@link activationNotices}' "Using profile" line,
+ * for an activation the user did not ask for — a re-render at startup, a
+ * profile format update — where which profile is in use is not news.
+ *
+ * Deliberately NOT {@link activationNotes}, which the CLI prints. Several of
+ * its lines are for a terminal log and would only be noise on this banner:
+ * where displaced files were backed up (the backup still happens; nobody needs
+ * to be told where), edits recorded into the profile being left (the switch
+ * confirmation already said so), skills handed back to the team. Tracked
+ * generated files are left out too: every activation signals
+ * PROFILE_ACTIVATED, and the app re-runs its gitignore check on it — the
+ * startup question and a Profiles-tab switch alike — which asks about exactly
+ * those files, with a way to fix them.
+ */
+export function activationWarnings(result: ActivationResult): NoticeLine[] {
+	const lines: NoticeLine[] = [];
+	const report = result.dependencies;
+	for (const { plugin, dependency } of report?.blocked ?? []) {
+		lines.push({
+			text: `${pluginName(plugin)} won't load: profile "${result.profileId}" turns off ${pluginName(dependency)}, which it needs`,
+			tone: "warning",
+		});
 	}
-	if (notes.length > 0) lines.push("", ...notes);
-	return lines.join("\n");
+	// The plugin's details are not on this machine yet, so what it depends on
+	// could not be enabled. One line for all of them: the fix is the same.
+	const missing = (report?.unreadable ?? []).map(pluginName);
+	if (missing.length > 0) {
+		lines.push({
+			text: `${listed(missing)} ${missing.length === 1 ? "isn't" : "aren't"} downloaded yet — run magus install`,
+			tone: "warning",
+		});
+	}
+	for (const f of result.skills.failed) {
+		lines.push({
+			text: `Skill "${f.skill}" could not be installed: ${f.reason}`,
+			tone: "warning",
+		});
+	}
+	// The one case that names a path: two copies of a skill, and which to keep
+	// is the user's call — they need to know where the other one is.
+	for (const c of result.conversion.conflicts) {
+		lines.push({
+			text: `Skill "${c.folder}" exists in two versions; kept ${c.kept}, the other is at ${c.other}`,
+			tone: "warning",
+		});
+	}
+	return lines;
 }
 
 /** Project-relative, so a path does not wrap over two lines. */
@@ -170,7 +210,10 @@ export function describeFailure(
 export interface ProfileStartupModalCallbacks {
 	/** Leave magus: the modal's only non-action. */
 	onQuit: () => void;
-	/** Called once a choice has activated a profile — see App's gitignore re-check. */
+	/**
+	 * Called once a choice has activated a profile, or a format update has
+	 * rewritten it — the app dispatches PROFILE_ACTIVATED on it.
+	 */
 	onActivated?: () => void;
 	/**
 	 * The question is ANSWERED: a profile is active, or the user chose to quit.
@@ -185,7 +228,8 @@ export interface ProfileStartupModalCallbacks {
  * question on screen NOW, which is not always the one magus opened with: "Update
  * it now" asks a new one, and a manifest that stopped parsing turns the question
  * into the unparseable one. `working` is true while an activation or a format
- * update is running — its result modal takes the slot when it settles.
+ * update is running — when it settles it empties the slot (success, reported as
+ * a notice) or puts its error modal there.
  */
 interface ProfileFlow {
 	need: ProfileStartupNeed;
@@ -198,7 +242,7 @@ export function useProfileStartupModal({
 	onAnswered,
 }: ProfileStartupModalCallbacks) {
 	const { state, dispatch } = useApp();
-	const { navigateToScreen } = useNavigation();
+	const notify = useNotify();
 	const projectPath = state.projectPath;
 	// A ref, not state: `show` is recreated on every render, and the flow must
 	// survive that — a re-show has to know what was last asked, and that work is
@@ -235,7 +279,6 @@ export function useProfileStartupModal({
 			 */
 			const apply = (
 				choice: ProfileChoice,
-				automatic: boolean,
 				current: ProfileStartupNeed = need,
 			): void => {
 				flowRef.current = { need: current, working: true };
@@ -269,28 +312,14 @@ export function useProfileStartupModal({
 						// question never came back.
 						answered();
 						onActivated?.();
-						const { profileId, displaced } = result;
-						dispatch({
-							type: "SHOW_MODAL",
-							modal: {
-								type: "message",
-								title: "Profile active",
-								message: describeOutcome(
-									profileId,
-									displaced,
-									projectPath,
-									automatic,
-									activationNotes(result),
-								),
-								variant: "success",
-								onDismiss: () => {
-									dispatch({ type: "HIDE_MODAL" });
-									// The Profiles tab is where the result is visible, and
-									// where the user will want to look next.
-									navigateToScreen("profiles");
-								},
-							},
-						});
+						// No result modal and no jump to the Profiles tab: the user
+						// stays where they are, told in the notice banner. An automatic
+						// activation is reported the same way, so it is never
+						// invisible.
+						dispatch({ type: "HIDE_MODAL" });
+						for (const line of activationNotices(result)) {
+							notify(line.text, line.tone);
+						}
 					})
 					.catch((error: unknown) => {
 						// The manifest stopped parsing between launch and this answer —
@@ -344,11 +373,24 @@ export function useProfileStartupModal({
 				void prepareProfileStartup(projectPath, { convert: true })
 					.then((result) => {
 						const outcome = formatUpdateOutcome(result);
+						// What the conversion's activation warns about — a plugin that
+						// will not load, one not downloaded yet, a skill that failed —
+						// in user terms. Posted whichever way it went, except a refusal,
+						// which converted nothing. Posting under the next question is
+						// safe: notices do not run down while a modal is open.
+						const warnings = (): void => {
+							for (const activation of result.activations) {
+								for (const line of activationWarnings(activation)) {
+									notify(line.text, line.tone);
+								}
+							}
+						};
 						if (outcome.kind === "ask") {
 							// Converted, but no profile is active yet: a new question,
 							// still unanswered.
 							onActivated?.();
 							ask(outcome.need);
+							warnings();
 							return;
 						}
 						if (outcome.kind === "failed") {
@@ -370,19 +412,9 @@ export function useProfileStartupModal({
 						// Nothing left to ask: the converted profile is active.
 						answered();
 						onActivated?.();
-						dispatch({
-							type: "SHOW_MODAL",
-							modal: {
-								type: "message",
-								title: outcome.title,
-								message: outcome.message,
-								variant: "success",
-								onDismiss: () => {
-									dispatch({ type: "HIDE_MODAL" });
-									navigateToScreen("profiles");
-								},
-							},
-						});
+						dispatch({ type: "HIDE_MODAL" });
+						notify(outcome.notice);
+						warnings();
 					})
 					.catch((error: unknown) => {
 						flowRef.current = { need, working: false };
@@ -427,11 +459,7 @@ export function useProfileStartupModal({
 								askName(suggestion, current, value, check.reason);
 								return;
 							}
-							apply(
-								{ kind: "adopt", id: check.id, name: check.name },
-								false,
-								current,
-							);
+							apply({ kind: "adopt", id: check.id, name: check.name }, current);
 						},
 						onCancel: () => ask(current),
 					},
@@ -447,7 +475,7 @@ export function useProfileStartupModal({
 						onChoose: (choice) =>
 							choice.kind === "adopt"
 								? askName(choice.id, current)
-								: apply(choice, false, current),
+								: apply(choice, current),
 						onUpdateFormat: updateFormat,
 						onQuit: quit,
 					}),
@@ -466,15 +494,15 @@ export function useProfileStartupModal({
 
 			// The repo already answered and there is nothing of the user's to
 			// replace: asking here would ask someone to re-decide what their team
-			// decided, with nothing at stake. Reported in the outcome modal, so it
+			// decided, with nothing at stake. Reported in the notice banner, so it
 			// is never invisible.
 			if (need.autoActivate && need.declaredDefault !== null) {
-				apply({ kind: "activate", id: need.declaredDefault }, true);
+				apply({ kind: "activate", id: need.declaredDefault });
 				return;
 			}
 			ask();
 		},
-		[dispatch, navigateToScreen, projectPath, onQuit, onActivated, onAnswered],
+		[dispatch, notify, projectPath, onQuit, onActivated, onAnswered],
 	);
 
 	return { show };

package/src/ui/hooks/useStaleLoadModal.ts

@@ -1,6 +1,6 @@
 import { useCallback } from "react";
-import { useApp, useNavigation } from "../state/AppContext.js";
-import type { ModalState } from "../state/types.js";
+import { useApp, useNavigation, useNotify } from "../state/AppContext.js";
+import type { ModalState, NoticeLine } from "../state/types.js";
 
 /**
  * Startup broken-plugin modal: magus launches, finds that Claude Code will load
@@ -71,12 +71,13 @@ export function buildStaleLoadModal(args: BuildStaleLoadModalArgs): ModalState {
 export function useStaleLoadModal() {
 	const { dispatch } = useApp();
 	const { navigateToScreen } = useNavigation();
+	const notify = useNotify();
 
 	const show = useCallback(
 		(args: {
 			brokenCount: number;
 			/** Runs the repair. Resolves to what to tell the user afterwards. */
-			onFix: () => Promise<string> | string;
+			onFix: () => Promise<readonly NoticeLine[]> | readonly NoticeLine[];
 		}) => {
 			if (args.brokenCount === 0) return;
 			const dismiss = () => dispatch({ type: "HIDE_MODAL" });
@@ -97,16 +98,11 @@ export function useStaleLoadModal() {
 					void Promise.resolve()
 						.then(() => args.onFix())
 						.then((summary) => {
-							dispatch({
-								type: "SHOW_MODAL",
-								modal: {
-									type: "message",
-									title: "Plugin repair",
-									message: summary,
-									variant: "info",
-									onDismiss: dismiss,
-								},
-							});
+							// Hidden, not replaced: the loading modal swallows every key,
+							// and the result belongs in the notice banner, outside the
+							// modal slot the next startup question is waiting for.
+							dispatch({ type: "HIDE_MODAL" });
+							for (const line of summary) notify(line.text, line.tone);
 						})
 						.catch((err) => {
 							dispatch({
@@ -129,33 +125,47 @@ export function useStaleLoadModal() {
 			});
 			dispatch({ type: "SHOW_MODAL", modal });
 		},
-		[dispatch, navigateToScreen],
+		[dispatch, navigateToScreen, notify],
 	);
 
 	return { show };
 }
 
 /**
- * One line per repair attempt, for the modal that follows the fix.
+ * What the repair did, as notices: one line for every plugin that now loads
+ * the version installed, and a warning for each that still does not.
  *
- * Exported and pure so the wording is tested rather than inspected by eye. Each
- * line says what the loader resolves NOW, because that — not the row count — is
- * the thing the user came here to change.
+ * Exported and pure so the wording is tested rather than inspected by eye. The
+ * lines say what loads NOW, because that — not the row count — is the thing
+ * the user came here to change. A failure is never folded into the success
+ * line: `ok` is the loader re-asked after the write, and a repair that did not
+ * take must read as one.
  */
-export function staleLoadRepairSummary(
+export function staleLoadRepairNotices(
 	results: ReadonlyArray<{
 		pluginId: string;
 		target: string;
 		ok: boolean;
 		error?: string;
 	}>,
-): string {
-	if (results.length === 0) return "Nothing to repair.";
-	return results
-		.map((r) =>
-			r.ok
-				? `✓ ${r.pluginId} now loads v${r.target}`
-				: `✗ ${r.pluginId} — ${r.error ?? "repair did not take effect"}`,
-		)
-		.join("\n");
+): NoticeLine[] {
+	if (results.length === 0) {
+		return [{ text: "Nothing needed repairing", tone: "success" }];
+	}
+	const lines: NoticeLine[] = [];
+	const fixed = results.filter((r) => r.ok);
+	if (fixed.length > 0) {
+		lines.push({
+			text: `Now loading ${fixed.map((r) => `${r.pluginId} v${r.target}`).join(", ")}`,
+			tone: "success",
+		});
+	}
+	for (const r of results) {
+		if (r.ok) continue;
+		lines.push({
+			text: `${r.pluginId} not repaired: ${r.error ?? "repair did not take effect"}`,
+			tone: "warning",
+		});
+	}
+	return lines;
 }

package/src/ui/screens/AliasScreen.tsx

@@ -39,7 +39,7 @@ import {
 import { ScreenLayout } from "../components/layout/index.js";
 import { useScreenKeys } from "../input/KeyRouter.js";
 import { moveDirection } from "../keymap.js";
-import { useApp, useModal } from "../state/AppContext.js";
+import { useApp, useModal, useNotify } from "../state/AppContext.js";
 import { displayWidth, truncate } from "../text-width.js";
 import { theme } from "../theme.js";
 
@@ -114,6 +114,7 @@ type ViewMode = { kind: "main" } | { kind: "flag-detail"; flagId: string };
 export function AliasScreen() {
 	const { state } = useApp();
 	const modal = useModal();
+	const notify = useNotify();
 
 	const [config, setConfig] = useState<AliasConfig | null>(null);
 	const [shells, setShells] = useState<ShellTarget[]>([]);
@@ -565,10 +566,10 @@ export function AliasScreen() {
 			shells.find((s) => s.exists);
 
 		if (!target) {
-			await modal.message(
-				"No shells detected",
-				"No supported rc files found. Create ~/.zshrc, ~/.bashrc, or ~/.config/fish/config.fish first.",
-				"info",
+			// Nothing was written. Named files, because creating one is the fix.
+			notify(
+				"No shell config to write to — create ~/.zshrc, ~/.bashrc or ~/.config/fish/config.fish first",
+				"warning",
 			);
 			return;
 		}
@@ -602,7 +603,7 @@ export function AliasScreen() {
 				"error",
 			);
 		}
-	}, [config, shells, modal, pendingAdoptRange]);
+	}, [config, shells, modal, notify, pendingAdoptRange]);
 
 	// ── Keyboard ──────────────────────────────────────────────────────────
 	// j/k navigate the selectable subset, jumping over headers/legend lines.

package/src/ui/screens/CliToolsScreen.tsx

@@ -4,7 +4,7 @@ import React, { useEffect, useCallback, useState, useRef } from "react";
 import { ScreenLayout } from "../components/layout/index.js";
 import { useScreenKeys } from "../input/KeyRouter.js";
 import { moveDirection } from "../keymap.js";
-import { useApp, useModal } from "../state/AppContext.js";
+import { useApp, useModal, useNotify } from "../state/AppContext.js";
 import { useDimensions } from "../state/DimensionsContext.js";
 
 import { cliTools } from "../../data/cli-tools.js";
@@ -91,6 +91,7 @@ export function CliToolsScreen() {
 	const { state, dispatch } = useApp();
 	const { cliTools: cliToolsState } = state;
 	const modal = useModal();
+	const notify = useNotify();
 	const dimensions = useDimensions();
 
 	const [rows, setRows] = useState<CliToolRow[]>(
@@ -333,21 +334,27 @@ export function CliToolsScreen() {
 
 		modal.hideModal();
 
-		const report = [
-			errors.length > 0
-				? `These removals failed. Run them by hand:\n${errors.join("\n")}`
-				: "",
+		const stays =
 			leftBehind.length > 0
 				? `${stuckList(leftBehind)}\nRemove it by hand if you want it gone.`
-				: "",
-		].filter((part) => part !== "");
+				: "";
 
-		if (report.length > 0) {
+		// A failed removal is an error, and stays a dialog with the commands to
+		// run. An install magus could never remove is only a warning: nothing
+		// failed, and the row still shows the conflict until it is dealt with.
+		if (errors.length > 0) {
 			await modal.message(
-				errors.length > 0 ? "Partly removed" : "One install stays",
-				report.join("\n\n"),
-				errors.length > 0 ? "error" : "info",
+				"Partly removed",
+				[
+					`These removals failed. Run them by hand:\n${errors.join("\n")}`,
+					stays,
+				]
+					.filter((part) => part !== "")
+					.join("\n\n"),
+				"error",
 			);
+		} else if (stays !== "") {
+			notify(stays.replace(/\n/g, " "), "warning");
 		}
 
 		handleRefresh();
@@ -359,11 +366,7 @@ export function CliToolsScreen() {
 		// it anyway" the CLI no longer does either.
 		const updatable = rows.filter((r) => r.status?.state === "upgrade");
 		if (updatable.length === 0) {
-			await modal.message(
-				"Up to Date",
-				"No tool is known to be behind.",
-				"info",
-			);
+			notify("No tool is known to be behind");
 			return;
 		}
 

package/src/ui/screens/EnvVarsScreen.tsx

@@ -54,9 +54,12 @@ export function SettingsScreen() {
 		}
 	}, [dispatch, state.projectPath]);
 
+	// Load data on mount or when dataRefreshVersion changes — a profile
+	// activating changes what this screen shows.
+	// biome-ignore lint/correctness/useExhaustiveDependencies: dataRefreshVersion is the external refresh trigger; bumping it must re-run the fetch even though the body never reads it
 	useEffect(() => {
 		fetchData();
-	}, [fetchData]);
+	}, [fetchData, state.dataRefreshVersion]);
 
 	const listItems = useMemo((): SettingsBrowserItem[] => {
 		if (settings.values.status !== "success") return [];

package/src/ui/screens/GitignoreScreen.tsx

@@ -19,7 +19,7 @@ import {
 	type RuleRow,
 	renderRuleRow,
 } from "../renderers/gitignoreRenderers.js";
-import { useApp, useModal } from "../state/AppContext.js";
+import { useApp, useModal, useNotify } from "../state/AppContext.js";
 import { theme } from "../theme.js";
 
 // Clean rows are grouped by the manifest that defined them. Order = display
@@ -36,6 +36,7 @@ const SOURCE_LABELS: Record<Tier, string> = {
 export function GitignoreScreen() {
 	const { state } = useApp();
 	const modal = useModal();
+	const notify = useNotify();
 	const renderer = useRenderer();
 	const cwd = state.projectPath;
 
@@ -57,9 +58,12 @@ export function GitignoreScreen() {
 		}
 	}, [cwd]);
 
+	// Load on mount or when dataRefreshVersion changes — a profile activating
+	// can turn committed files into generated ones this screen lists.
+	// biome-ignore lint/correctness/useExhaustiveDependencies: dataRefreshVersion is the external refresh trigger; bumping it must re-run the load even though the body never reads it
 	useEffect(() => {
 		void reload();
-	}, [reload]);
+	}, [reload, state.dataRefreshVersion]);
 
 	// Build the managed list: every desired gitignore rule, with all matching
 	// violations grouped under the rule that owns them.
@@ -163,19 +167,11 @@ export function GitignoreScreen() {
 	// Fix just the highlighted row.
 	const handleFixOne = useCallback(async () => {
 		if (!selectedRow) {
-			await modal.message(
-				"No row selected",
-				"Use ↑↓ to highlight a row that needs fixing, then press f.",
-				"info",
-			);
+			notify("Highlight a row that needs fixing, then press f", "warning");
 			return;
 		}
 		if (selectedRow.violations.length === 0) {
-			await modal.message(
-				"Nothing to fix",
-				`${selectedRow.rule.pattern} already matches. f only applies to rows that need fixing.`,
-				"info",
-			);
+			notify(`${selectedRow.rule.pattern} already matches — nothing to fix`);
 			return;
 		}
 
@@ -204,16 +200,12 @@ export function GitignoreScreen() {
 				"error",
 			);
 		}
-	}, [selectedRow, modal, cwd, reload]);
+	}, [selectedRow, modal, notify, cwd, reload]);
 
 	const handleApplyTemplate = useCallback(async () => {
 		const names = Object.keys(BUILTIN_TEMPLATES);
 		if (names.length === 0) {
-			await modal.message(
-				"No templates",
-				"No built-in templates available.",
-				"info",
-			);
+			notify("No built-in templates available", "warning");
 			return;
 		}
 		const selected = await modal.select(
@@ -224,15 +216,16 @@ export function GitignoreScreen() {
 		if (!selected) return;
 		try {
 			const r = await applyTemplate(cwd, selected);
-			const conflictNote =
-				r.conflicts.length > 0
-					? `\n\nSkipped due to conflicts: ${r.conflicts.join(", ")}`
-					: "";
-			await modal.message(
-				"Template applied",
-				`Wrote ${r.appliedTo}\n\nAdded ${r.added.ignore.length} ignore + ${r.added.track.length} track entries.${conflictNote}`,
-				"success",
+			notify(
+				`Applied "${selected}": ${r.added.ignore.length} ignore + ${r.added.track.length} track entries`,
 			);
+			// Skipped entries are not visible in the list, so they get their own line.
+			if (r.conflicts.length > 0) {
+				notify(
+					`Skipped, conflicting with existing rules: ${r.conflicts.join(", ")}`,
+					"warning",
+				);
+			}
 		} catch (err) {
 			await modal.message(
 				"Template failed",
@@ -241,7 +234,7 @@ export function GitignoreScreen() {
 			);
 		}
 		await reload();
-	}, [cwd, modal, reload]);
+	}, [cwd, modal, notify, reload]);
 
 	/**
 	 * Open the project manifest in $EDITOR, creating it first if needed.

package/src/ui/screens/McpRegistryScreen.tsx

@@ -2,7 +2,12 @@ import React, { useEffect, useCallback, useState, useRef } from "react";
 import { ScreenLayout } from "../components/layout/index.js";
 import { useScreenKeys, useSearchInput } from "../input/KeyRouter.js";
 import { moveDirection } from "../keymap.js";
-import { useApp, useModal, useNavigation } from "../state/AppContext.js";
+import {
+	useApp,
+	useModal,
+	useNavigation,
+	useNotify,
+} from "../state/AppContext.js";
 import { useDimensions } from "../state/DimensionsContext.js";
 
 import { addMcpServer, setAllowMcp } from "../../services/claude-settings.js";
@@ -75,6 +80,7 @@ export function McpRegistryScreen() {
 	const { state, dispatch } = useApp();
 	const { mcpRegistry } = state;
 	const modal = useModal();
+	const notify = useNotify();
 	const { navigateToScreen } = useNavigation();
 	const dimensions = useDimensions();
 
@@ -190,11 +196,8 @@ export function McpRegistryScreen() {
 			await setAllowMcp(true, state.projectPath);
 			await addMcpServer(server.name, config, state.projectPath);
 			modal.hideModal();
-			await modal.message(
-				"Installed",
-				`${server.name} has been configured.\n\nRestart Claude Code to activate.`,
-				"success",
-			);
+			// A warning: the server does nothing until the user restarts.
+			notify(`Added ${server.name} — restart Claude Code to use it`, "warning");
 		} catch (error) {
 			modal.hideModal();
 			await modal.message("Error", `Failed to install: ${error}`, "error");