magus-cli 7.7.0 → 7.9.0

package/src/ui/renderers/cliToolRenderers.tsx

@@ -5,14 +5,18 @@ import {
 	cliToolUninstall,
 	uninstallBlocker,
 } from "../../services/cli-tool-commands.js";
+import { doctorFixCommand } from "../../shared/plugin/hooks/lib/requires-probe.js";
 import { formatCommand } from "../../utils/command-utils.js";
 import {
 	type CliToolEnterAction,
 	type CliToolRow,
+	type CliToolsRow,
+	type PluginManagedRow,
 	cliToolEnterAction,
 	cliToolStatusText,
 	displayVersion,
 	isInstalled,
+	isPluginManagedRow,
 } from "../adapters/cliToolsAdapter.js";
 import { rowLead } from "../components/primitives/index.js";
 import { type UiColor, theme } from "../theme.js";
@@ -94,13 +98,139 @@ function toneColor(tone: StatusTone): UiColor {
 	}
 }
 
+// ─── Plugin-managed rows ───────────────────────────────────────────────────────
+
+/** The mark and words for a plugin-managed row, from the dependency probe. */
+function managedStatus(presence: PluginManagedRow["presence"]): {
+	icon: string;
+	color: UiColor;
+	text: string;
+} {
+	switch (presence?.status) {
+		case undefined:
+			return { icon: "○", color: theme.colors.muted, text: "● Checking..." };
+		case "present":
+			return { icon: "●", color: theme.colors.success, text: "● Installed" };
+		case "off-path":
+			return {
+				icon: "!",
+				color: theme.colors.warning,
+				text: `! Installed in ${presence.prefix}, which is not on the PATH Claude Code starts with`,
+			};
+		case "missing":
+			return { icon: "○", color: theme.colors.muted, text: "○ Not installed" };
+		case "unknown":
+			return {
+				icon: "?",
+				color: theme.colors.warning,
+				text: `? Could not tell: ${presence.reason}`,
+			};
+	}
+}
+
+function renderPluginManagedRow(
+	row: PluginManagedRow,
+	isSelected: boolean,
+): React.ReactNode {
+	const { icon, color } = managedStatus(row.presence);
+	const suffix =
+		row.presence === null ? " ..." : ` · managed by ${row.managedBy}`;
+	if (isSelected) {
+		return (
+			<text bg={theme.selection.bg} fg={theme.selection.fg}>
+				{rowLead(true)}
+				{icon} {row.entry.displayName}
+				{suffix}{" "}
+			</text>
+		);
+	}
+	return (
+		<text fg={theme.colors.text}>
+			<span fg={color}>
+				{rowLead(false)}
+				{icon}
+			</span>
+			<span fg={theme.colors.text}> {row.entry.displayName}</span>
+			<span fg={theme.colors.dim}>{suffix}</span>
+		</text>
+	);
+}
+
+function renderPluginManagedDetail(
+	row: PluginManagedRow,
+	width: number,
+): React.ReactNode {
+	const { entry, managedBy, presence } = row;
+	const status = managedStatus(presence);
+	const here =
+		presence?.status === "present" || presence?.status === "off-path";
+	return (
+		<box flexDirection="column" width={width}>
+			<box flexDirection="row" marginBottom={1}>
+				<text fg={theme.colors.info}>
+					<strong>{entry.displayName}</strong>
+				</text>
+			</box>
+
+			<text fg={theme.colors.muted}>{entry.description}</text>
+
+			<box marginTop={1} flexDirection="column">
+				<box flexDirection="row">
+					<text fg={theme.colors.muted} flexShrink={0}>
+						{"Status   "}
+					</text>
+					<text fg={status.color} minWidth={0}>
+						{wrapValue(status.text, width)}
+					</text>
+				</box>
+				<box flexDirection="row">
+					<text fg={theme.colors.muted} flexShrink={0}>
+						{"Managed  "}
+					</text>
+					<text fg={theme.colors.text}>by {managedBy}</text>
+				</box>
+				{!here && presence !== null && row.installText ? (
+					<box flexDirection="row">
+						<text fg={theme.colors.muted} flexShrink={0}>
+							{"Install  "}
+						</text>
+						<text fg={theme.colors.info} minWidth={0}>
+							{wrapValue(row.installText, width)}
+						</text>
+					</box>
+				) : null}
+				<box flexDirection="row">
+					<text fg={theme.colors.muted} flexShrink={0}>
+						{"Website  "}
+					</text>
+					<box flexGrow={1} minWidth={0}>
+						<text fg={theme.colors.info} wrapMode="char">
+							{entry.website}
+						</text>
+					</box>
+				</box>
+			</box>
+
+			<box marginTop={2}>
+				<text fg={theme.colors.muted}>
+					{wrapText(
+						`${managedBy} installs ${entry.name} and decides its version; it cannot be installed, updated or removed from here. ${doctorFixCommand(null)} installs it when it is missing.`,
+						width,
+					).join("\n")}
+				</text>
+			</box>
+		</box>
+	);
+}
+
 // ─── Row renderer ──────────────────────────────────────────────────────────────
 
 export function renderCliToolRow(
-	row: CliToolRow,
+	row: CliToolsRow,
 	_index: number,
 	isSelected: boolean,
 ): React.ReactNode {
+	if (isPluginManagedRow(row)) return renderPluginManagedRow(row, isSelected);
 	const { tool, status, install } = row;
 	const installed = isInstalled(status);
 	const allMethods = install?.all ?? [];
@@ -167,10 +297,13 @@ export function renderCliToolRow(
 // ─── Detail renderer ───────────────────────────────────────────────────────────
 
 export function renderCliToolDetail(
-	row: CliToolRow | undefined,
+	row: CliToolsRow | undefined,
 	/** Usable detail-panel columns (`detailPanelWidth`); the scrollbox does not constrain width. */
 	width: number,
 ): React.ReactNode {
+	if (row && isPluginManagedRow(row)) {
+		return renderPluginManagedDetail(row, width);
+	}
 	if (!row) {
 		return (
 			<box

package/src/ui/renderers/mcpRenderers.tsx

@@ -1,5 +1,5 @@
 import type React from "react";
-import type { McpServer } from "../../types/index.js";
+import type { McpRegistryServer, McpServer } from "../../types/index.js";
 import { rowLead } from "../components/primitives/index.js";
 import { theme } from "../theme.js";
 
@@ -10,11 +10,27 @@ import { theme } from "../theme.js";
  */
 const SERVER_ICON = "⬢";
 
+/**
+ * A server added or removed this magus session, on its row: a running Claude
+ * Code picks the change up only after it restarts. One string for the curated
+ * and the registry rows alike.
+ */
+const RESTART_MARK = " ↻ restart Claude Code";
+
 // ─── List item types ───────────────────────────────────────────────────────────
 
 export type McpListItem =
 	| { kind: "category"; label: string }
-	| { kind: "server"; server: McpServer; isInstalled: boolean };
+	| {
+			kind: "server";
+			server: McpServer;
+			isInstalled: boolean;
+			/**
+			 * Added or removed in this magus session: a running Claude Code picks
+			 * it up only after it restarts. Absent means false.
+			 */
+			needsRestart?: boolean;
+	  };
 
 // ─── Row renderers ─────────────────────────────────────────────────────────────
 
@@ -39,13 +55,15 @@ export function renderMcpRow(
 	const { server, isInstalled } = item;
 	const icon = isInstalled ? "●" : "○";
 	const iconColor = isInstalled ? theme.colors.success : theme.colors.muted;
+	const restart = item.needsRestart ? RESTART_MARK : "";
 
 	if (isSelected) {
 		return (
 			<text bg={theme.selection.bg} fg={theme.selection.fg}>
 				{rowLead(true)}
 				{icon} {server.name}
-				{server.requiresConfig ? " ⚙" : ""}{" "}
+				{server.requiresConfig ? " ⚙" : ""}
+				{restart}{" "}
 			</text>
 		);
 	}
@@ -58,10 +76,91 @@ export function renderMcpRow(
 			</span>
 			<span fg={theme.colors.text}> {server.name}</span>
 			{server.requiresConfig ? <span fg={theme.colors.warning}> ⚙</span> : null}
+			{restart ? <span fg={theme.colors.warning}>{restart}</span> : null}
 		</text>
 	);
 }
 
+// ─── Registry rows ─────────────────────────────────────────────────────────────
+
+/**
+ * One server on the MCP Registry screen, with the same two states a curated
+ * row on the MCP screen carries: installed in this project, and — when it was
+ * added this magus session — that a running Claude Code picks it up only after
+ * it restarts.
+ *
+ * Adding from the registry used to show nothing at all: the restart state was
+ * drawn only on curated rows, and a registry server is not one, so the promise
+ * "shown on the MCP screen" had no row to be shown on.
+ */
+export interface McpRegistryListItem {
+	server: McpRegistryServer;
+	isInstalled: boolean;
+	needsRestart: boolean;
+}
+
+export function renderMcpRegistryRow(
+	item: McpRegistryListItem,
+	_index: number,
+	isSelected: boolean,
+): React.ReactNode {
+	const { server, isInstalled, needsRestart } = item;
+	const version = server.version ? ` v${server.version}` : "";
+	const icon = isInstalled ? "● " : "";
+	const restart = needsRestart ? RESTART_MARK : "";
+	if (isSelected) {
+		return (
+			<text bg={theme.selection.bg} fg={theme.selection.fg}>
+				{rowLead(true)}
+				{icon}
+				{server.name}
+				{version}
+				{restart}{" "}
+			</text>
+		);
+	}
+	return (
+		<text fg={theme.colors.text}>
+			<span>{rowLead(false)}</span>
+			{icon ? <span fg={theme.colors.success}>{icon}</span> : null}
+			<span>
+				<strong>{server.name}</strong>
+			</span>
+			<span fg={theme.colors.info}>{version}</span>
+			{restart ? <span fg={theme.colors.warning}>{restart}</span> : null}
+		</text>
+	);
+}
+
+/**
+ * The registry detail panel's state lines — installed or not, and the
+ * restart — in the same words and colours as a curated server's detail.
+ */
+export function renderMcpRegistryStatus(
+	item: McpRegistryListItem,
+): React.ReactNode {
+	return (
+		<box flexDirection="column">
+			<box flexDirection="row">
+				<text fg={theme.colors.muted}>{"Status "}</text>
+				{item.isInstalled ? (
+					<text fg={theme.colors.success}>{"● Installed"}</text>
+				) : (
+					<text fg={theme.colors.muted}>{"○ Not installed"}</text>
+				)}
+			</box>
+			{item.needsRestart ? (
+				<box flexDirection="row">
+					<text fg={theme.colors.muted}>{"Takes effect "}</text>
+					<text fg={theme.colors.warning} wrapMode="word">
+						{"↻ after restarting Claude Code"}
+					</text>
+				</box>
+			) : null}
+		</box>
+	);
+}
+
 // ─── Detail renderer ───────────────────────────────────────────────────────────
 
 export function renderMcpDetail(
@@ -112,6 +211,14 @@ export function renderMcpDetail(
 						<text fg={theme.colors.muted}>{"○ Not installed"}</text>
 					)}
 				</box>
+				{item.needsRestart ? (
+					<box flexDirection="row">
+						<text fg={theme.colors.muted}>{"Takes effect "}</text>
+						<text fg={theme.colors.warning} wrapMode="word">
+							{"↻ after restarting Claude Code"}
+						</text>
+					</box>
+				) : null}
 				<box flexDirection="row">
 					<text fg={theme.colors.muted}>{"Type "}</text>
 					<text fg={theme.colors.text}>

package/src/ui/renderers/pluginRenderers.tsx

@@ -1,5 +1,6 @@
 import type React from "react";
 import { getMarketplaceVersion } from "../../services/marketplace-fetcher.js";
+import { mcpSettingsNeededText } from "../../services/plugin-env-needs.js";
 import {
 	compareVersions,
 	hasMissingFiles,
@@ -10,11 +11,11 @@ import {
 } from "../../services/plugin-manager.js";
 import type { ScopeStatus } from "../../services/plugin-manager.js";
 import type { PluginComponentKind } from "../../services/skills-manager.js";
-import type {
-	PluginUpdateAction,
-	PluginUpdateItem,
+import {
+	type PluginUpdateAction,
+	type PluginUpdateItem,
+	isKeptDeprecated,
 } from "../../services/update-plan.js";
-import { isKnownVersion } from "../../services/version-snapshot.js";
 import type { PluginRelease } from "../../types/index.js";
 import { highlightMatches } from "../../utils/fuzzy-search.js";
 import type {
@@ -43,7 +44,11 @@ import type {
 import { componentChip } from "../theme-mode.js";
 import { type UiColor, theme } from "../theme.js";
 import { renderMarkdown } from "./markdownRenderer.js";
-import { type PluginRowSegment, pluginRowText } from "./pluginRowText.js";
+import {
+	type PluginRowSegment,
+	isKnownVersion,
+	pluginRowText,
+} from "./pluginRowText.js";
 import { type StatusTone, pluginStatusText } from "./pluginStatusText.js";
 
 // ─── Category renderers ───────────────────────────────────────────────────────
@@ -285,6 +290,7 @@ const ACTION_LABEL: Record<PluginUpdateAction, string> = {
 	// The DETAIL panel's label, not the row's — this is where the mechanism is
 	// allowed to be named, and the note beside it spells out both versions.
 	"stale-load": "BROKEN — OLDER VERSION LOADS",
+	remove: "DEPRECATED",
 	unknown: "UNKNOWN",
 };
 
@@ -321,12 +327,14 @@ function StatusLine({
 		: status.action === "update" && status.target
 			? `→ v${status.target}`
 			: "";
+	// Moved or still required: kept on purpose, which is known — never UNKNOWN.
+	const label = isKeptDeprecated(status) ? "KEPT" : ACTION_LABEL[status.action];
 	return (
 		<KeyValueLine
 			label="Status"
 			value={
 				<span>
-					<span fg={toneColor(tone)}>{ACTION_LABEL[status.action]}</span>
+					<span fg={toneColor(tone)}>{label}</span>
 					{detail ? <span fg={theme.colors.text}> {detail}</span> : null}
 				</span>
 			}
@@ -349,6 +357,57 @@ function pluginDetail(
 
 	// Published by no marketplace we read. The status line says which of the
 	// three things that means — moved, withdrawn, or its marketplace unread.
+	//
+	// Deprecated — a marketplace we read no longer lists it — is the one of the
+	// three that is a fact, and gets one plain line in the user's terms. The row
+	// already says `a` removes it; `d` removes just this one.
+	if (plugin.isOrphaned && status?.action === "remove") {
+		return (
+			<box flexDirection="column">
+				<box justifyContent="center">
+					<text bg={theme.colors.danger} fg={theme.hints.fg}>
+						<strong> {plugin.name} — DEPRECATED </strong>
+					</text>
+				</box>
+				{/* No Status line: the heading says DEPRECATED and the line below
+				    says why, so the note would be the same fact a third time. */}
+				<box marginTop={1}>
+					<text fg={theme.colors.text}>
+						No longer published by{" "}
+						{plugin.marketplaceDisplay || plugin.marketplace}. Claude Code does
+						not load it.
+					</text>
+				</box>
+				<ActionHints hints={[{ key: "d", label: "Remove", tone: "danger" }]} />
+			</box>
+		);
+	}
+	// Kept on purpose: it moved to another marketplace, or a plugin that stays
+	// still requires it. The status line's note says which; nothing is wrong
+	// enough for amber, and `d` would only undo a deliberate keep.
+	if (plugin.isOrphaned && status && isKeptDeprecated(status)) {
+		return (
+			<box flexDirection="column">
+				<box justifyContent="center">
+					<text bg={theme.colors.muted} fg={theme.hints.fg}>
+						<strong> {plugin.name} — KEPT </strong>
+					</text>
+				</box>
+				<box marginTop={1}>
+					<StatusLine status={status} />
+				</box>
+				<box marginTop={1}>
+					<text fg={theme.colors.muted}>
+						No longer published by{" "}
+						{plugin.marketplaceDisplay || plugin.marketplace}.{" "}
+						{status.reason === "moved"
+							? "magus leaves this copy installed; the plugin is still published elsewhere."
+							: "magus keeps it installed while a plugin that needs it stays."}
+					</text>
+				</box>
+			</box>
+		);
+	}
 	if (plugin.isOrphaned) {
 		return (
 			<box flexDirection="column">
@@ -466,6 +525,18 @@ function pluginDetail(
 					</text>
 				</box>
 			) : null}
+			{/*
+			 * A lasting state, not a message: derived from the current config each
+			 * load, so it is here for as long as the variables are unset — however
+			 * the install question was answered — and gone once they are set.
+			 */}
+			{item.mcpSettingsNeeded ? (
+				<box>
+					<text fg={theme.colors.warning}>
+						⚙ {mcpSettingsNeededText(item.mcpSettingsNeeded)}
+					</text>
+				</box>
+			) : null}
 			{/*
 			 * THE panel where the mechanism is spelled out in full. The row above
 			 * says only "BROKEN — press a to fix", on purpose; a reader who wants to

package/src/ui/renderers/pluginRowText.ts

@@ -12,27 +12,43 @@
  *  - an orphan's text depended on the cursor — the selected row said
  *    "moved → X", the unselected one always said "deprecated".
  *
+ * A row shows the current version and the state, nothing about history. It
+ * used to badge what changed since magus last looked ("v1.8.0 → v1.7.4
+ * updated"), from a snapshot of versions seen, and rendered a downgrade as
+ * "updated" beside an offer to update again.
+ *
  * Pure and `.ts`, so a test can assert on exactly what the row prints.
  */
 
-import { isKnownVersion } from "../../services/version-snapshot.js";
 import type { PluginPluginItem } from "../adapters/pluginsAdapter.js";
 import { type StatusTone, pluginStatusText } from "./pluginStatusText.js";
 
+/**
+ * Sentinel recorded by Claude Code for a plugin that is installed but whose
+ * version is unknown — every plugin in the official Anthropic marketplace uses
+ * it. It is not a real version and must never be rendered as one.
+ */
+export const UNKNOWN_VERSION = "0.0.0";
+
+/** True when a version string carries information worth printing. */
+export function isKnownVersion(v: string | undefined): v is string {
+	return !!v && v !== UNKNOWN_VERSION;
+}
+
 export interface PluginRowSegment {
 	text: string;
 	tone: StatusTone | "text";
 }
 
 export interface PluginRowText {
-	/** After the name: the version, a history badge, then the status suffix. */
+	/** After the name: the version, then the status suffix. */
 	segments: PluginRowSegment[];
 	/** The segments joined — exactly what the row renders after the name. */
 	text: string;
 }
 
 export function pluginRowText(item: PluginPluginItem): PluginRowText {
-	const { plugin, status } = item;
+	const { status } = item;
 	const segments: PluginRowSegment[] = [];
 
 	// A version is shown only when it carries information. "0.0.0" means
@@ -43,32 +59,16 @@ export function pluginRowText(item: PluginPluginItem): PluginRowText {
 	// `plugin.installedVersion` printed a version where the classifier had none —
 	// "v0.8.1 files missing", with 0.8.1 read off a scope not installed here.
 	const installed = status?.installed ?? undefined;
-	const shown = isKnownVersion(installed) ? installed : undefined;
-
-	// "What just changed" and "what state it is in now" are independent facts
-	// and both can be true at once: an external update can move a plugin
-	// 2.9.0 → 3.0.1 while the marketplace already offers 3.3.1. So the badge
-	// replaces only the version, and the status suffix is always appended.
-	if (plugin.recentlyUpdatedFrom && shown) {
-		// Show what actually happened, not just where it landed — the update may
-		// have come from Claude Code or a direct CLI call without telling us.
-		segments.push({
-			text: ` v${plugin.recentlyUpdatedFrom} → v${shown} updated`,
-			tone: "success",
-		});
-	} else if (plugin.recentlyInstalled) {
-		// Newly seen. A plugin can be genuinely installed with no version we can
-		// name, so the version is optional here.
-		segments.push({
-			text: shown ? ` v${shown} new` : " new",
-			tone: "success",
-		});
-	} else if (shown) {
-		segments.push({ text: ` v${shown}`, tone: "muted" });
+	if (isKnownVersion(installed)) {
+		segments.push({ text: ` v${installed}`, tone: "muted" });
 	}
 
 	const suffix = pluginStatusText(status);
 	if (suffix.suffix) segments.push({ text: suffix.suffix, tone: suffix.tone });
 
+	// Its MCP server lacks variables. The same ⚙ the MCP screen puts on a
+	// server that needs configuring; the detail panel says how many, and the key.
+	if (item.mcpSettingsNeeded) segments.push({ text: " ⚙", tone: "warning" });
+
 	return { segments, text: segments.map((s) => s.text).join("") };
 }

package/src/ui/renderers/pluginStatusText.ts

@@ -11,7 +11,10 @@
  * which is what makes the two front ends say the same thing.
  */
 
-import type { PluginUpdateItem } from "../../services/update-plan.js";
+import {
+	type PluginUpdateItem,
+	isKeptDeprecated,
+} from "../../services/update-plan.js";
 
 export type StatusTone = "muted" | "warning" | "danger" | "success" | "info";
 
@@ -27,6 +30,16 @@ export function pluginStatusText(
 	if (!status) return { suffix: "", tone: "muted" };
 	const withNote = status.note ? ` (${status.note})` : "";
 
+	// No longer published here, and kept on purpose — it moved, or a plugin
+	// that stays still requires it. A fact, not "cannot tell": muted, with the
+	// note ("moved to Magus Marketing"), never the amber "?".
+	if (isKeptDeprecated(status)) {
+		return {
+			suffix: ` ${status.note ?? "no longer published"}`,
+			tone: "muted",
+		};
+	}
+
 	switch (status.action) {
 		case "unknown":
 			// Amber, not red: "cannot tell" is not a failure.
@@ -55,6 +68,10 @@ export function pluginStatusText(
 			// whose session runs a version nobody installed is broken, and there is
 			// now a key that fixes it.
 			return { suffix: " BROKEN — press a to fix", tone: "danger" };
+		case "remove":
+			// No longer published. Red, like BROKEN: Claude Code refuses to load it,
+			// and `a` removes it. Which marketplace dropped it is the detail panel's.
+			return { suffix: " deprecated — press a to remove", tone: "danger" };
 		case "install":
 			// `absent` only ever comes from a profile pin (see `pinForRow`), so it
 			// may say so. An undeclared row whose files are missing must not.