claudeup 4.11.0 → 4.11.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claudeup",
-  "version": "4.11.0",
+  "version": "4.11.1",
   "description": "TUI tool for managing Claude Code plugins, MCPs, and configuration",
   "type": "module",
   "main": "src/main.tsx",

package/src/__tests__/plugin-version-check.test.ts

@@ -115,7 +115,7 @@ const {
 	fixPluginVersionMismatch,
 	fixAllPluginVersionMismatches,
 	formatMismatchWarning,
-	formatMismatchBanner,
+	formatMismatchModal,
 } = await import("../services/plugin-version-check.js");
 
 // ── Test suite ───────────────────────────────────────────────────────────────
@@ -739,8 +739,8 @@ describe("formatMismatchWarning", () => {
 	});
 });
 
-describe("formatMismatchBanner", () => {
-	it("formats concise banner text", () => {
+describe("formatMismatchModal", () => {
+	it("formats modal text with plugin details and bug link", () => {
 		const mismatches = [
 			{
 				pluginId: "terminal@magus",
@@ -751,10 +751,12 @@ describe("formatMismatchBanner", () => {
 			},
 		];
 
-		const banner = formatMismatchBanner(mismatches);
+		const modal = formatMismatchModal(mismatches);
 
-		expect(banner).toContain("terminal@magus");
-		expect(banner).toContain("v2.0.0");
-		expect(banner).toContain("v4.0.2");
+		expect(modal).toContain("terminal");
+		expect(modal).toContain("v2.0.0");
+		expect(modal).toContain("v4.0.2");
+		expect(modal).toContain("45997");
+		expect(modal).toContain("wrong plugin version");
 	});
 });

package/src/services/plugin-version-check.js

@@ -184,11 +184,20 @@ export function formatMismatchWarning(mismatches) {
     return lines.join("\n");
 }
 /**
- * Format mismatch info for TUI display.
+ * Format mismatch info for TUI modal display.
+ * Returns a multi-line string suitable for a modal message.
  */
-export function formatMismatchBanner(mismatches) {
-    const parts = mismatches.map((m) => `${m.pluginId}: loading v${m.firstEntryVersion} instead of v${m.currentProjectVersion}`);
-    return `Version mismatch: ${parts.join(", ")}`;
+export function formatMismatchModal(mismatches) {
+    const lines = [];
+    lines.push("Claude Code has a bug where it loads the wrong plugin version.");
+    lines.push("It always uses the first entry in the registry, ignoring which");
+    lines.push("project you're in. These plugins are affected:\n");
+    for (const m of mismatches) {
+        const name = m.pluginId.split("@")[0];
+        lines.push(`  ${name}: will load v${m.firstEntryVersion} instead of v${m.currentProjectVersion}`);
+    }
+    lines.push(`\nBug: github.com/anthropics/claude-code/issues/45997`);
+    return lines.join("\n");
 }
 // ── Helpers ──────────────────────────────────────────────────────────────────
 /**

package/src/services/plugin-version-check.ts

@@ -268,16 +268,31 @@ export function formatMismatchWarning(
 }
 
 /**
- * Format mismatch info for TUI display.
+ * Format mismatch info for TUI modal display.
+ * Returns a multi-line string suitable for a modal message.
  */
-export function formatMismatchBanner(
+export function formatMismatchModal(
 	mismatches: VersionMismatchInfo[],
 ): string {
-	const parts = mismatches.map(
-		(m) =>
-			`${m.pluginId}: loading v${m.firstEntryVersion} instead of v${m.currentProjectVersion}`,
+	const lines: string[] = [];
+	lines.push(
+		"Claude Code has a bug where it loads the wrong plugin version.",
+	);
+	lines.push(
+		"It always uses the first entry in the registry, ignoring which",
 	);
-	return `Version mismatch: ${parts.join(", ")}`;
+	lines.push("project you're in. These plugins are affected:\n");
+
+	for (const m of mismatches) {
+		const name = m.pluginId.split("@")[0];
+		lines.push(
+			`  ${name}: will load v${m.firstEntryVersion} instead of v${m.currentProjectVersion}`,
+		);
+	}
+
+	lines.push(`\nBug: github.com/anthropics/claude-code/issues/45997`);
+
+	return lines.join("\n");
 }
 
 // ── Helpers ──────────────────────────────────────────────────────────────────

package/src/ui/App.js

@@ -8,7 +8,7 @@ import { ModalContainer } from "./components/modals/index.js";
 import { PluginsScreen, McpScreen, McpRegistryScreen, SettingsScreen, CliToolsScreen, ModelSelectorScreen, ProfilesScreen, SkillsScreen, } from "./screens/index.js";
 import { repairAllMarketplaces } from "../services/local-marketplace.js";
 import { migrateMarketplaceRename, recoverMarketplaceSettings, } from "../services/claude-settings.js";
-import { checkPluginVersionMismatches, formatMismatchBanner, } from "../services/plugin-version-check.js";
+import { checkPluginVersionMismatches, fixAllPluginVersionMismatches, formatMismatchModal, } from "../services/plugin-version-check.js";
 import { checkForUpdates, getCurrentVersion, } from "../services/version-check.js";
 import { useKeyboardHandler } from "./hooks/useKeyboardHandler.js";
 import { ProgressBar } from "./components/layout/ProgressBar.js";
@@ -176,11 +176,11 @@ function UpdateBanner({ result }) {
 function ProgressIndicator({ message, current, total, }) {
     return (_jsx("box", { paddingLeft: 1, paddingRight: 1, children: _jsx(ProgressBar, { message: message, current: current, total: total }) }));
 }
-function AppContentInner({ showDebug, onDebugToggle, updateInfo, onExit, recoveryReport, mismatchWarning, }) {
+function AppContentInner({ showDebug, onDebugToggle, updateInfo, onExit, recoveryReport, }) {
     const { state } = useApp();
     const { progress } = state;
     const dimensions = useDimensions();
-    return (_jsxs("box", { flexDirection: "column", height: dimensions.terminalHeight, children: [updateInfo?.updateAvailable && _jsx(UpdateBanner, { result: updateInfo }), recoveryReport && (_jsx("box", { paddingLeft: 1, paddingRight: 1, children: _jsxs("text", { fg: "green", children: ["\u2713 Fixed: ", recoveryReport] }) })), mismatchWarning && (_jsx("box", { paddingLeft: 1, paddingRight: 1, children: _jsxs("text", { fg: "yellow", children: ["\u26A0 ", mismatchWarning, " (bug #45997)"] }) })), showDebug && (_jsx("box", { paddingLeft: 1, paddingRight: 1, children: _jsxs("text", { fg: "#888888", children: ["DEBUG: ", dimensions.terminalWidth, "x", dimensions.terminalHeight, " | content=", dimensions.contentHeight, " | screen=", state.currentRoute.screen] }) })), progress && _jsx(ProgressIndicator, { ...progress }), _jsx("box", { flexDirection: "column", height: dimensions.contentHeight, paddingLeft: 1, paddingRight: 1, children: _jsx(Router, {}) }), _jsx(GlobalKeyHandler, { onDebugToggle: onDebugToggle, onExit: onExit }), _jsx(ModalContainer, {})] }));
+    return (_jsxs("box", { flexDirection: "column", height: dimensions.terminalHeight, children: [updateInfo?.updateAvailable && _jsx(UpdateBanner, { result: updateInfo }), recoveryReport && (_jsx("box", { paddingLeft: 1, paddingRight: 1, children: _jsxs("text", { fg: "green", children: ["\u2713 Fixed: ", recoveryReport] }) })), showDebug && (_jsx("box", { paddingLeft: 1, paddingRight: 1, children: _jsxs("text", { fg: "#888888", children: ["DEBUG: ", dimensions.terminalWidth, "x", dimensions.terminalHeight, " | content=", dimensions.contentHeight, " | screen=", state.currentRoute.screen] }) })), progress && _jsx(ProgressIndicator, { ...progress }), _jsx("box", { flexDirection: "column", height: dimensions.contentHeight, paddingLeft: 1, paddingRight: 1, children: _jsx(Router, {}) }), _jsx(GlobalKeyHandler, { onDebugToggle: onDebugToggle, onExit: onExit }), _jsx(ModalContainer, {})] }));
 }
 function AppContent({ onExit }) {
     const { state, dispatch } = useApp();
@@ -188,7 +188,7 @@ function AppContent({ onExit }) {
     const [showDebug, setShowDebug] = useState(false);
     const [updateInfo, setUpdateInfo] = useState(null);
     const [recoveryReport, setRecoveryReport] = useState(null);
-    const [mismatchWarning, setMismatchWarning] = useState(null);
+    const [mismatchData, setMismatchData] = useState(null);
     // Check for updates on startup (non-blocking)
     useEffect(() => {
         checkForUpdates()
@@ -202,13 +202,72 @@ function AppContent({ onExit }) {
         const timer = setTimeout(() => setRecoveryReport(null), 5000);
         return () => clearTimeout(timer);
     }, [recoveryReport]);
-    // Auto-dismiss mismatch warning after 8 seconds (longer — more important)
+    // Show mismatch modal when data arrives
     useEffect(() => {
-        if (!mismatchWarning)
+        if (!mismatchData || mismatchData.length === 0)
             return;
-        const timer = setTimeout(() => setMismatchWarning(null), 8000);
-        return () => clearTimeout(timer);
-    }, [mismatchWarning]);
+        dispatch({
+            type: "SHOW_MODAL",
+            modal: {
+                type: "select",
+                title: "⚠ Plugin Version Mismatch",
+                message: formatMismatchModal(mismatchData),
+                options: [
+                    {
+                        label: "Fix all projects",
+                        value: "fix",
+                        description: "Align all projects to this project's versions",
+                    },
+                    {
+                        label: "Dismiss",
+                        value: "dismiss",
+                        description: "Ignore for now",
+                    },
+                ],
+                onSelect: async (value) => {
+                    dispatch({ type: "HIDE_MODAL" });
+                    if (value === "fix") {
+                        dispatch({
+                            type: "SHOW_PROGRESS",
+                            state: { message: "Fixing plugin versions..." },
+                        });
+                        try {
+                            await fixAllPluginVersionMismatches(mismatchData);
+                            dispatch({ type: "HIDE_PROGRESS" });
+                            dispatch({
+                                type: "SHOW_MODAL",
+                                modal: {
+                                    type: "message",
+                                    title: "Fixed",
+                                    message: "All plugin versions aligned. Restart Claude Code for changes to take effect.",
+                                    variant: "success",
+                                    onDismiss: () => dispatch({ type: "HIDE_MODAL" }),
+                                },
+                            });
+                        }
+                        catch {
+                            dispatch({ type: "HIDE_PROGRESS" });
+                            dispatch({
+                                type: "SHOW_MODAL",
+                                modal: {
+                                    type: "message",
+                                    title: "Error",
+                                    message: "Failed to fix plugin versions. Try running claudeup again.",
+                                    variant: "error",
+                                    onDismiss: () => dispatch({ type: "HIDE_MODAL" }),
+                                },
+                            });
+                        }
+                    }
+                    setMismatchData(null);
+                },
+                onCancel: () => {
+                    dispatch({ type: "HIDE_MODAL" });
+                    setMismatchData(null);
+                },
+            },
+        });
+    }, [mismatchData, dispatch]);
     // Auto-refresh marketplaces on startup
     useEffect(() => {
         const noRefresh = process.argv.includes("--no-refresh");
@@ -252,7 +311,7 @@ function AppContent({ onExit }) {
         checkPluginVersionMismatches(process.cwd())
             .then((mismatches) => {
             if (mismatches.length > 0) {
-                setMismatchWarning(formatMismatchBanner(mismatches));
+                setMismatchData(mismatches);
             }
         })
             .catch(() => { }); // non-fatal
@@ -266,8 +325,8 @@ function AppContent({ onExit }) {
         });
     }, [dispatch]);
     // Count transient banners for dimension calculation
-    const transientBannerCount = (recoveryReport ? 1 : 0) + (mismatchWarning ? 1 : 0);
-    return (_jsx(DimensionsProvider, { showProgress: !!progress, showDebug: showDebug, showUpdateBanner: !!updateInfo?.updateAvailable, transientBannerCount: transientBannerCount, children: _jsx(AppContentInner, { showDebug: showDebug, onDebugToggle: () => setShowDebug((s) => !s), updateInfo: updateInfo, onExit: onExit, recoveryReport: recoveryReport, mismatchWarning: mismatchWarning }) }));
+    const transientBannerCount = recoveryReport ? 1 : 0;
+    return (_jsx(DimensionsProvider, { showProgress: !!progress, showDebug: showDebug, showUpdateBanner: !!updateInfo?.updateAvailable, transientBannerCount: transientBannerCount, children: _jsx(AppContentInner, { showDebug: showDebug, onDebugToggle: () => setShowDebug((s) => !s), updateInfo: updateInfo, onExit: onExit, recoveryReport: recoveryReport }) }));
 }
 export function App({ onExit }) {
     return (_jsx(AppProvider, { children: _jsx(AppContent, { onExit: onExit }) }));

package/src/ui/App.tsx

@@ -30,7 +30,9 @@ import {
 } from "../services/claude-settings.js";
 import {
 	checkPluginVersionMismatches,
-	formatMismatchBanner,
+	fixAllPluginVersionMismatches,
+	formatMismatchModal,
+	type VersionMismatchInfo,
 } from "../services/plugin-version-check.js";
 import {
 	checkForUpdates,
@@ -255,7 +257,6 @@ interface AppContentInnerProps {
 	updateInfo: VersionCheckResult | null;
 	onExit: () => void;
 	recoveryReport: string | null;
-	mismatchWarning: string | null;
 }
 
 function AppContentInner({
@@ -264,7 +265,6 @@ function AppContentInner({
 	updateInfo,
 	onExit,
 	recoveryReport,
-	mismatchWarning,
 }: AppContentInnerProps) {
 	const { state } = useApp();
 	const { progress } = state;
@@ -278,11 +278,6 @@ function AppContentInner({
 					<text fg="green">✓ Fixed: {recoveryReport}</text>
 				</box>
 			)}
-			{mismatchWarning && (
-				<box paddingLeft={1} paddingRight={1}>
-					<text fg="yellow">⚠ {mismatchWarning} (bug #45997)</text>
-				</box>
-			)}
 			{showDebug && (
 				<box paddingLeft={1} paddingRight={1}>
 					<text fg="#888888">
@@ -321,7 +316,7 @@ function AppContent({ onExit }: AppContentProps) {
 	const [showDebug, setShowDebug] = useState(false);
 	const [updateInfo, setUpdateInfo] = useState<VersionCheckResult | null>(null);
 	const [recoveryReport, setRecoveryReport] = useState<string | null>(null);
-	const [mismatchWarning, setMismatchWarning] = useState<string | null>(null);
+	const [mismatchData, setMismatchData] = useState<VersionMismatchInfo[] | null>(null);
 
 	// Check for updates on startup (non-blocking)
 	useEffect(() => {
@@ -337,12 +332,71 @@ function AppContent({ onExit }: AppContentProps) {
 		return () => clearTimeout(timer);
 	}, [recoveryReport]);
 
-	// Auto-dismiss mismatch warning after 8 seconds (longer — more important)
+	// Show mismatch modal when data arrives
 	useEffect(() => {
-		if (!mismatchWarning) return;
-		const timer = setTimeout(() => setMismatchWarning(null), 8000);
-		return () => clearTimeout(timer);
-	}, [mismatchWarning]);
+		if (!mismatchData || mismatchData.length === 0) return;
+		dispatch({
+			type: "SHOW_MODAL",
+			modal: {
+				type: "select",
+				title: "⚠ Plugin Version Mismatch",
+				message: formatMismatchModal(mismatchData),
+				options: [
+					{
+						label: "Fix all projects",
+						value: "fix",
+						description: "Align all projects to this project's versions",
+					},
+					{
+						label: "Dismiss",
+						value: "dismiss",
+						description: "Ignore for now",
+					},
+				],
+				onSelect: async (value: string) => {
+					dispatch({ type: "HIDE_MODAL" });
+					if (value === "fix") {
+						dispatch({
+							type: "SHOW_PROGRESS",
+							state: { message: "Fixing plugin versions..." },
+						});
+						try {
+							await fixAllPluginVersionMismatches(mismatchData);
+							dispatch({ type: "HIDE_PROGRESS" });
+							dispatch({
+								type: "SHOW_MODAL",
+								modal: {
+									type: "message",
+									title: "Fixed",
+									message:
+										"All plugin versions aligned. Restart Claude Code for changes to take effect.",
+									variant: "success",
+									onDismiss: () => dispatch({ type: "HIDE_MODAL" }),
+								},
+							});
+						} catch {
+							dispatch({ type: "HIDE_PROGRESS" });
+							dispatch({
+								type: "SHOW_MODAL",
+								modal: {
+									type: "message",
+									title: "Error",
+									message: "Failed to fix plugin versions. Try running claudeup again.",
+									variant: "error",
+									onDismiss: () => dispatch({ type: "HIDE_MODAL" }),
+								},
+							});
+						}
+					}
+					setMismatchData(null);
+				},
+				onCancel: () => {
+					dispatch({ type: "HIDE_MODAL" });
+					setMismatchData(null);
+				},
+			},
+		});
+	}, [mismatchData, dispatch]);
 
 	// Auto-refresh marketplaces on startup
 	useEffect(() => {
@@ -391,7 +445,7 @@ function AppContent({ onExit }: AppContentProps) {
 		checkPluginVersionMismatches(process.cwd())
 			.then((mismatches) => {
 				if (mismatches.length > 0) {
-					setMismatchWarning(formatMismatchBanner(mismatches));
+					setMismatchData(mismatches);
 				}
 			})
 			.catch(() => {}); // non-fatal
@@ -408,7 +462,7 @@ function AppContent({ onExit }: AppContentProps) {
 
 	// Count transient banners for dimension calculation
 	const transientBannerCount =
-		(recoveryReport ? 1 : 0) + (mismatchWarning ? 1 : 0);
+		recoveryReport ? 1 : 0;
 
 	return (
 		<DimensionsProvider
@@ -423,8 +477,7 @@ function AppContent({ onExit }: AppContentProps) {
 				updateInfo={updateInfo}
 				onExit={onExit}
 				recoveryReport={recoveryReport}
-				mismatchWarning={mismatchWarning}
-			/>
+				/>
 		</DimensionsProvider>
 	);
 }