magus-cli 7.5.2 → 7.5.4

package/src/ui/screens/ProfilesScreen.tsx

@@ -26,7 +26,6 @@ import {
 	renameProfile,
 } from "../../services/profiles.js";
 import { writableMap } from "../../services/settings-file.js";
-import { LiveFileRefusal } from "../../services/tracked-live-files.js";
 import {
 	ClipboardUnavailableError,
 	readClipboard,
@@ -178,9 +177,7 @@ export function ProfilesScreen() {
 			modal.hideModal();
 			await modal.message(
 				"Error",
-				error instanceof LiveFileRefusal
-					? error.message
-					: `Failed to apply profile: ${error}`,
+				`Failed to apply profile: ${error}`,
 				"error",
 			);
 		}
@@ -245,9 +242,7 @@ export function ProfilesScreen() {
 				modal.hideModal();
 				await modal.message(
 					"Error",
-					error instanceof LiveFileRefusal
-						? error.message
-						: `Failed to switch profile: ${error}`,
+					`Failed to switch profile: ${error}`,
 					"error",
 				);
 			}
@@ -280,9 +275,7 @@ export function ProfilesScreen() {
 			modal.hideModal();
 			await modal.message(
 				"Error",
-				error instanceof LiveFileRefusal
-					? error.message
-					: `Failed to apply profile: ${error}`,
+				`Failed to apply profile: ${error}`,
 				"error",
 			);
 		}

package/src/cli/tui-prompt.tsx

@@ -1,143 +0,0 @@
-/**
- * One OpenTUI dialog, for a command that is otherwise not a TUI.
- *
- * `magus update` prints a report and exits; it is not the interactive app. But
- * a question it must ask deserves the same dialog the TUI asks with, so this
- * mounts the REAL `SelectModal` — the component tab 5 and tab 0 use — for the
- * length of one question, then tears the renderer down and lets the command
- * carry on printing. Nothing here re-draws a list, a border or a highlight: the
- * component and the theme own all of that, and a second implementation is how
- * the two surfaces drift apart.
- *
- * Every import of `@opentui/*` and `ui/` is DYNAMIC. `main.tsx` splits the TUI
- * out precisely so `magus hook …`, which Claude Code runs on every matching
- * tool call, never pays to load the renderer — a static import here would undo
- * that for every command that imports this module's file, whether or not it
- * ever asks a question.
- *
- * `screenMode: "main-screen"` keeps the dialog in the scrollback beside the
- * report. The alternate screen would take the terminal over and then hand back
- * a screen with no trace the question was ever asked.
- */
-
-import type { SelectOption } from "../ui/state/types.js";
-
-export interface SelectDialogInput {
-	title: string;
-	message: string;
-	options: SelectOption[];
-	/** Row to start on. Defaults to the first. */
-	defaultIndex?: number;
-}
-
-/**
- * Ask the user to pick one option. Resolves to the chosen value, or null when
- * they cancel with Esc or Ctrl+C — the caller decides whether that is fatal.
- *
- * Falls back to `null` when stdin or stdout is not a TTY. A renderer needs both:
- * `magus update > file` has a TTY stdin and a pipe for stdout, and mounting
- * there writes escape sequences into the file.
- */
-export async function selectDialog(
-	input: SelectDialogInput,
-): Promise<string | null> {
-	if (process.stdin.isTTY !== true || process.stdout.isTTY !== true) {
-		return null;
-	}
-
-	const [{ createCliRenderer }, { createRoot }, React, modals, dimensions] =
-		await Promise.all([
-			import("@opentui/core"),
-			import("@opentui/react"),
-			import("react"),
-			import("../ui/components/modals/index.js"),
-			import("../ui/state/DimensionsContext.js"),
-		]);
-
-	const renderer = await createCliRenderer({
-		screenMode: "main-screen",
-		// BOTH, so the renderer registers no handler of its own and teardown
-		// below is the only path. With either left at its default, a Ctrl+C
-		// destroys the renderer underneath this promise and it never settles.
-		exitSignals: [],
-		exitOnCtrlC: false,
-	});
-
-	const root = createRoot(renderer);
-
-	return await new Promise<string | null>((resolve) => {
-		// Clamped, not trusted: a caller computing an index from a list it built
-		// separately could hand over one that is out of range, and a cursor
-		// pointing past the end makes Enter resolve to null.
-		let index = Math.min(
-			Math.max(0, input.defaultIndex ?? 0),
-			Math.max(0, input.options.length - 1),
-		);
-		let done = false;
-
-		const finish = (value: string | null): void => {
-			if (done) return;
-			done = true;
-			renderer.keyInput.off("keypress", onKey);
-			root.unmount();
-			renderer.destroy();
-			resolve(value);
-		};
-
-		function onKey(key: {
-			name?: string;
-			ctrl?: boolean;
-			sequence?: string;
-		}): void {
-			if ((key.ctrl && key.name === "c") || key.name === "escape") {
-				finish(null);
-				return;
-			}
-			if (key.name === "return" || key.name === "enter") {
-				finish(input.options[index]?.value ?? null);
-				return;
-			}
-			if (key.name === "up" || key.name === "k") {
-				index = (index - 1 + input.options.length) % input.options.length;
-				draw();
-				return;
-			}
-			if (key.name === "down" || key.name === "j") {
-				index = (index + 1) % input.options.length;
-				draw();
-				return;
-			}
-			// The numbers stay live: the list is short, and the plain-text prompt
-			// this replaced numbered it, so the habit already exists.
-			const digit = Number(key.sequence);
-			if (
-				Number.isInteger(digit) &&
-				digit >= 1 &&
-				digit <= input.options.length
-			) {
-				index = digit - 1;
-				finish(input.options[index]?.value ?? null);
-			}
-		}
-
-		const draw = (): void => {
-			root.render(
-				React.createElement(
-					dimensions.DimensionsProvider,
-					null,
-					React.createElement(modals.SelectModal, {
-						title: input.title,
-						message: input.message,
-						options: input.options,
-						defaultIndex: index,
-						onSelect: finish,
-						onCancel: () => finish(null),
-					}),
-				),
-			);
-		};
-
-		renderer.keyInput.on("keypress", onKey);
-		draw();
-	});
-}