claudeup 4.10.1 → 4.11.0

package/src/ui/App.tsx

@@ -28,6 +28,10 @@ import {
 	migrateMarketplaceRename,
 	recoverMarketplaceSettings,
 } from "../services/claude-settings.js";
+import {
+	checkPluginVersionMismatches,
+	formatMismatchBanner,
+} from "../services/plugin-version-check.js";
 import {
 	checkForUpdates,
 	getCurrentVersion,
@@ -216,7 +220,7 @@ function UpdateBanner({ result }: { result: VersionCheckResult }) {
 				v{result.currentVersion} → v{result.latestVersion}
 			</text>
 			<text fg="gray"> Run: </text>
-			<text fg="cyan">npm i -g claudeup</text>
+			<text fg="cyan">claudeup update</text>
 		</box>
 	);
 }
@@ -250,6 +254,8 @@ interface AppContentInnerProps {
 	onDebugToggle: () => void;
 	updateInfo: VersionCheckResult | null;
 	onExit: () => void;
+	recoveryReport: string | null;
+	mismatchWarning: string | null;
 }
 
 function AppContentInner({
@@ -257,81 +263,24 @@ function AppContentInner({
 	onDebugToggle,
 	updateInfo,
 	onExit,
+	recoveryReport,
+	mismatchWarning,
 }: AppContentInnerProps) {
-	const { state, dispatch } = useApp();
+	const { state } = useApp();
 	const { progress } = state;
 	const dimensions = useDimensions();
-	const [recoveryReport, setRecoveryReport] = useState<string | null>(null);
-
-	// Auto-refresh marketplaces on startup
-	useEffect(() => {
-		const noRefresh = process.argv.includes("--no-refresh");
-		if (noRefresh) return;
-
-		dispatch({
-			type: "SHOW_PROGRESS",
-			state: { message: "Scanning marketplaces..." },
-		});
-
-		// Migrate old marketplace names → magus (idempotent), then repair plugin.json files
-		migrateMarketplaceRename().catch(() => {}); // non-blocking, best-effort
-
-		// Recover stale marketplace registry entries (e.g. "directory" → "github")
-		recoverMarketplaceSettings()
-			.then(async (recovery) => {
-				const msgs: string[] = [];
-				if (recovery.reregistered.length > 0) {
-					msgs.push(
-						`Re-registered as GitHub: ${recovery.reregistered.join(", ")}`,
-					);
-					// Update the marketplace clone now that the source is fixed
-					const { updateMarketplace } = await import(
-						"../services/claude-cli.js"
-					);
-					for (const mp of recovery.reregistered) {
-						try {
-							await updateMarketplace(mp);
-							msgs.push(`Updated marketplace: ${mp}`);
-						} catch {
-							msgs.push(`Failed to update: ${mp}`);
-						}
-					}
-				}
-				if (recovery.enabledAutoUpdate.length > 0) {
-					msgs.push(
-						`Enabled auto-update: ${recovery.enabledAutoUpdate.join(", ")}`,
-					);
-				}
-				if (recovery.removed.length > 0) {
-					msgs.push(
-						`Removed stale: ${recovery.removed.join(", ")}`,
-					);
-				}
-				if (msgs.length > 0) {
-					setRecoveryReport(msgs.join("\n"));
-				}
-			})
-			.catch(() => {}); // non-fatal
-
-		repairAllMarketplaces()
-			.then(async () => {
-				dispatch({ type: "HIDE_PROGRESS" });
-				dispatch({ type: "DATA_REFRESH_COMPLETE" });
-			})
-			.catch(() => {
-				dispatch({ type: "HIDE_PROGRESS" });
-			});
-	}, [dispatch]);
 
 	return (
 		<box flexDirection="column" height={dimensions.terminalHeight}>
 			{updateInfo?.updateAvailable && <UpdateBanner result={updateInfo} />}
 			{recoveryReport && (
 				<box paddingLeft={1} paddingRight={1}>
-					<text bg="green" fg="black">
-						<strong> RECOVERED </strong>
-					</text>
-					<text fg="green"> {recoveryReport.split("\n").join(" | ")}</text>
+					<text fg="green">✓ Fixed: {recoveryReport}</text>
+				</box>
+			)}
+			{mismatchWarning && (
+				<box paddingLeft={1} paddingRight={1}>
+					<text fg="yellow">⚠ {mismatchWarning} (bug #45997)</text>
 				</box>
 			)}
 			{showDebug && (
@@ -367,10 +316,12 @@ interface AppContentProps {
 }
 
 function AppContent({ onExit }: AppContentProps) {
-	const { state } = useApp();
+	const { state, dispatch } = useApp();
 	const { progress } = state;
 	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);
 
 	// Check for updates on startup (non-blocking)
 	useEffect(() => {
@@ -379,17 +330,100 @@ function AppContent({ onExit }: AppContentProps) {
 			.catch(() => {});
 	}, []);
 
+	// Auto-dismiss recovery banner after 5 seconds
+	useEffect(() => {
+		if (!recoveryReport) return;
+		const timer = setTimeout(() => setRecoveryReport(null), 5000);
+		return () => clearTimeout(timer);
+	}, [recoveryReport]);
+
+	// Auto-dismiss mismatch warning after 8 seconds (longer — more important)
+	useEffect(() => {
+		if (!mismatchWarning) return;
+		const timer = setTimeout(() => setMismatchWarning(null), 8000);
+		return () => clearTimeout(timer);
+	}, [mismatchWarning]);
+
+	// Auto-refresh marketplaces on startup
+	useEffect(() => {
+		const noRefresh = process.argv.includes("--no-refresh");
+		if (noRefresh) return;
+
+		dispatch({
+			type: "SHOW_PROGRESS",
+			state: { message: "Scanning marketplaces..." },
+		});
+
+		// Migrate old marketplace names -> magus (idempotent), then repair plugin.json files
+		migrateMarketplaceRename().catch(() => {}); // non-blocking, best-effort
+
+		// Recover stale marketplace registry entries (e.g. "directory" -> "github")
+		recoverMarketplaceSettings()
+			.then(async (recovery) => {
+				const parts: string[] = [];
+				if (recovery.reregistered.length > 0) {
+					// Update the marketplace clone now that the source is fixed
+					const { updateMarketplace } = await import(
+						"../services/claude-cli.js"
+					);
+					for (const mp of recovery.reregistered) {
+						try {
+							await updateMarketplace(mp);
+							parts.push(`${mp} refreshed`);
+						} catch {
+							parts.push(`${mp} (update failed)`);
+						}
+					}
+				}
+				if (recovery.enabledAutoUpdate.length > 0) {
+					parts.push(`auto-update: ${recovery.enabledAutoUpdate.join(", ")}`);
+				}
+				if (recovery.removed.length > 0) {
+					parts.push(`removed: ${recovery.removed.join(", ")}`);
+				}
+				if (parts.length > 0) {
+					setRecoveryReport(parts.join(" | "));
+				}
+			})
+			.catch(() => {}); // non-fatal
+
+		// Check for plugin version mismatches (the [0] index bug)
+		checkPluginVersionMismatches(process.cwd())
+			.then((mismatches) => {
+				if (mismatches.length > 0) {
+					setMismatchWarning(formatMismatchBanner(mismatches));
+				}
+			})
+			.catch(() => {}); // non-fatal
+
+		repairAllMarketplaces()
+			.then(async () => {
+				dispatch({ type: "HIDE_PROGRESS" });
+				dispatch({ type: "DATA_REFRESH_COMPLETE" });
+			})
+			.catch(() => {
+				dispatch({ type: "HIDE_PROGRESS" });
+			});
+	}, [dispatch]);
+
+	// Count transient banners for dimension calculation
+	const transientBannerCount =
+		(recoveryReport ? 1 : 0) + (mismatchWarning ? 1 : 0);
+
 	return (
 		<DimensionsProvider
 			showProgress={!!progress}
 			showDebug={showDebug}
 			showUpdateBanner={!!updateInfo?.updateAvailable}
+			transientBannerCount={transientBannerCount}
 		>
 			<AppContentInner
 				showDebug={showDebug}
 				onDebugToggle={() => setShowDebug((s) => !s)}
 				updateInfo={updateInfo}
 				onExit={onExit}
+				recoveryReport={recoveryReport}
+				mismatchWarning={mismatchWarning}
 			/>
 		</DimensionsProvider>
 	);

package/src/ui/screens/PluginsScreen.js

@@ -14,8 +14,9 @@ import { saveProfile } from "../../services/profiles.js";
 import { installPlugin as cliInstallPlugin, uninstallPlugin as cliUninstallPlugin, updatePlugin as cliUpdatePlugin, } from "../../services/claude-cli.js";
 import { getPluginEnvRequirements, getPluginSourcePath, } from "../../services/plugin-mcp-config.js";
 import { getPluginSetupFromSource, checkMissingDeps, installPluginDeps, } from "../../services/plugin-setup.js";
+import { checkSinglePluginMismatch } from "../../services/plugin-version-check.js";
 import { buildPluginBrowserItems, } from "../adapters/pluginsAdapter.js";
-import { renderPluginRow, renderPluginDetail } from "../renderers/pluginRenderers.js";
+import { renderPluginRow, renderPluginDetail, } from "../renderers/pluginRenderers.js";
 export function PluginsScreen() {
     const { state, dispatch } = useApp();
     const { plugins: pluginsState } = state;
@@ -140,7 +141,10 @@ export function PluginsScreen() {
         if (event.name === "up" || event.name === "k") {
             if (isSearchActive)
                 dispatch({ type: "SET_SEARCHING", isSearching: false });
-            dispatch({ type: "PLUGINS_SELECT", index: Math.max(0, pluginsState.selectedIndex - 1) });
+            dispatch({
+                type: "PLUGINS_SELECT",
+                index: Math.max(0, pluginsState.selectedIndex - 1),
+            });
             return;
         }
         if (event.name === "down" || event.name === "j") {
@@ -167,12 +171,18 @@ export function PluginsScreen() {
             selectableItems[pluginsState.selectedIndex]?.kind === "category") {
             const item = selectableItems[pluginsState.selectedIndex];
             if (item?.kind === "category") {
-                dispatch({ type: "PLUGINS_TOGGLE_MARKETPLACE", name: item.marketplace.name });
+                dispatch({
+                    type: "PLUGINS_TOGGLE_MARKETPLACE",
+                    name: item.marketplace.name,
+                });
             }
             return;
         }
         if (isSearchActive) {
-            if (event.name.length === 1 && !event.ctrl && !event.meta && !/[0-9]/.test(event.name)) {
+            if (event.name.length === 1 &&
+                !event.ctrl &&
+                !event.meta &&
+                !/[0-9]/.test(event.name)) {
                 dispatch({
                     type: "PLUGINS_SET_SEARCH",
                     query: pluginsState.searchQuery + event.name,
@@ -223,6 +233,29 @@ export function PluginsScreen() {
             await saveInstalledPluginVersion(pluginId, version, state.projectPath);
         }
     };
+    /**
+     * Check for version mismatch after a plugin update and warn the user.
+     * The [0] index bug in Claude Code means the update may not actually
+     * take effect if another project's entry is at index 0.
+     */
+    const warnIfVersionMismatch = async (pluginId) => {
+        try {
+            const projectPath = state.projectPath || process.cwd();
+            const mismatch = await checkSinglePluginMismatch(pluginId, projectPath);
+            if (mismatch) {
+                await modal.message("Version Mismatch Warning", `${pluginId} was updated to v${mismatch.currentProjectVersion} for this project, ` +
+                    `but Claude Code will load v${mismatch.firstEntryVersion} due to a known bug.\n\n` +
+                    `The plugin loader always uses the first entry in installed_plugins.json, ` +
+                    `which belongs to another project.\n\n` +
+                    `Bug: https://github.com/anthropics/claude-code/issues/45997\n\n` +
+                    `To fix: open claudeup and use the version alignment tool, ` +
+                    `or update the plugin in all projects to the same version.`, "error");
+            }
+        }
+        catch {
+            // Non-fatal: mismatch check is best-effort
+        }
+    };
     // ── Action handlers ────────────────────────────────────────────────────────
     const handleRefresh = async () => {
         progress.show("Refreshing cache...");
@@ -360,7 +393,9 @@ export function PluginsScreen() {
             const result = await installPluginDeps(missing);
             modal.hideModal();
             if (result.failed.length > 0) {
-                const failMsg = result.failed.map((f) => `${f.pkg}: ${f.error}`).join("\n");
+                const failMsg = result.failed
+                    .map((f) => `${f.pkg}: ${f.error}`)
+                    .join("\n");
                 await modal.message("Partial Install", `Installed: ${result.installed.length}\nFailed:\n${failMsg}`, "error");
             }
             else if (result.installed.length > 0) {
@@ -404,7 +439,10 @@ export function PluginsScreen() {
             const buildScopeLabel = (name, scope, desc) => {
                 const installed = scope?.enabled;
                 const ver = scope?.version;
-                const hasUpdate = ver && latestVersion && ver !== latestVersion && latestVersion !== "0.0.0";
+                const hasUpdate = ver &&
+                    latestVersion &&
+                    ver !== latestVersion &&
+                    latestVersion !== "0.0.0";
                 let label = installed ? `● ${name}` : `○ ${name}`;
                 label += ` (${desc})`;
                 if (ver)
@@ -414,9 +452,18 @@ export function PluginsScreen() {
                 return label;
             };
             const scopeOptions = [
-                { label: buildScopeLabel("User", plugin.userScope, "global"), value: "user" },
-                { label: buildScopeLabel("Project", plugin.projectScope, "team"), value: "project" },
-                { label: buildScopeLabel("Local", plugin.localScope, "private"), value: "local" },
+                {
+                    label: buildScopeLabel("User", plugin.userScope, "global"),
+                    value: "user",
+                },
+                {
+                    label: buildScopeLabel("Project", plugin.projectScope, "team"),
+                    value: "project",
+                },
+                {
+                    label: buildScopeLabel("Local", plugin.localScope, "private"),
+                    value: "local",
+                },
             ];
             const scopeValue = await modal.select(plugin.name, `Select scope to toggle:`, scopeOptions);
             if (scopeValue === null)
@@ -428,7 +475,11 @@ export function PluginsScreen() {
                     : plugin.localScope;
             const isInstalledInScope = selectedScope?.enabled;
             const installedVersion = selectedScope?.version;
-            const scopeLabel = scopeValue === "user" ? "User" : scopeValue === "project" ? "Project" : "Local";
+            const scopeLabel = scopeValue === "user"
+                ? "User"
+                : scopeValue === "project"
+                    ? "Project"
+                    : "Local";
             const hasUpdateInScope = isInstalledInScope &&
                 installedVersion &&
                 latestVersion !== "0.0.0" &&
@@ -465,6 +516,9 @@ export function PluginsScreen() {
                 if (action !== "install") {
                     modal.hideModal();
                 }
+                if (action === "update") {
+                    await warnIfVersionMismatch(plugin.id);
+                }
                 fetchData();
             }
             catch (error) {
@@ -486,6 +540,7 @@ export function PluginsScreen() {
                 await saveVersionForScope(plugin.id, plugin.version, scope);
             }
             modal.hideModal();
+            await warnIfVersionMismatch(plugin.id);
             fetchData();
         }
         catch (error) {
@@ -500,6 +555,7 @@ export function PluginsScreen() {
         if (updatable.length === 0)
             return;
         const scope = pluginsState.scope === "global" ? "user" : "project";
+        const updatedPluginIds = [];
         try {
             for (let i = 0; i < updatable.length; i++) {
                 const plugin = updatable[i];
@@ -508,8 +564,13 @@ export function PluginsScreen() {
                 if (plugin.version) {
                     await saveVersionForScope(plugin.id, plugin.version, scope);
                 }
+                updatedPluginIds.push(plugin.id);
             }
             modal.hideModal();
+            // Check for mismatches on all updated plugins
+            for (const pluginId of updatedPluginIds) {
+                await warnIfVersionMismatch(pluginId);
+            }
             fetchData();
         }
         catch (error) {
@@ -566,6 +627,9 @@ export function PluginsScreen() {
             if (action !== "install") {
                 modal.hideModal();
             }
+            if (action === "update") {
+                await warnIfVersionMismatch(plugin.id);
+            }
             fetchData();
         }
         catch (error) {
@@ -605,8 +669,14 @@ export function PluginsScreen() {
         if (name === null || !name.trim())
             return;
         const scopeChoice = await modal.select("Save to scope", "Where should this profile be saved?", [
-            { label: "User — ~/.claude/profiles.json (available everywhere)", value: "user" },
-            { label: "Project — .claude/profiles.json (shared with team via git)", value: "project" },
+            {
+                label: "User — ~/.claude/profiles.json (available everywhere)",
+                value: "user",
+            },
+            {
+                label: "Project — .claude/profiles.json (shared with team via git)",
+                value: "project",
+            },
         ]);
         if (scopeChoice === null)
             return;
@@ -638,7 +708,10 @@ export function PluginsScreen() {
     const scopeLabel = pluginsState.scope === "global" ? "Global" : "Project";
     const plugins = pluginsState.plugins.status === "success" ? pluginsState.plugins.data : [];
     const installedCount = plugins.filter((p) => p.enabled).length;
-    const updateCount = plugins.filter((p) => p.hasUpdate && (p.userScope?.enabled || p.projectScope?.enabled || p.localScope?.enabled)).length;
+    const updateCount = plugins.filter((p) => p.hasUpdate &&
+        (p.userScope?.enabled ||
+            p.projectScope?.enabled ||
+            p.localScope?.enabled)).length;
     const subtitle = `${scopeLabel} │ ${installedCount} installed${updateCount > 0 ? ` │ ${updateCount} updates` : ""}`;
     const searchPlaceholder = `${scopeLabel} │ ${installedCount} installed${updateCount > 0 ? ` │ ${updateCount} ⬆` : ""} │ / to search`;
     return (_jsx(ScreenLayout, { title: "claudeup Plugins", subtitle: subtitle, currentScreen: "plugins", search: {