claudeup 4.10.0 → 4.10.2

package/package.json

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

package/src/services/claude-cli.js

@@ -53,9 +53,8 @@ async function execClaude(args, timeoutMs = 30000) {
  * Handles enabling + version tracking + cache copy in one shot.
  *
  * If the install fails because the plugin is "not found in marketplace",
- * triggers a marketplace update and retries once. This handles the case
- * where claudeup's remote listing (from GitHub) has newer plugins than
- * the local marketplace clone.
+ * recovers the marketplace registry (fixes stale "directory" sources),
+ * triggers a marketplace update, and retries once.
  */
 export async function installPlugin(pluginId, scope = "user") {
     try {
@@ -64,10 +63,13 @@ export async function installPlugin(pluginId, scope = "user") {
     catch (error) {
         const msg = error instanceof Error ? error.message : String(error);
         if (msg.includes("not found in marketplace")) {
-            // Local marketplace clone is stale — update and retry
             const parts = pluginId.split("@");
             const marketplace = parts[1];
             if (marketplace) {
+                // Fix known_marketplaces.json first (stale "directory" → "github"),
+                // then update, then retry
+                const { recoverMarketplaceSettings } = await import("./claude-settings.js");
+                await recoverMarketplaceSettings();
                 await updateMarketplace(marketplace);
                 await execClaude([
                     "plugin",
@@ -136,6 +138,8 @@ export async function updatePlugin(pluginId, scope = "user") {
             const parts = pluginId.split("@");
             const marketplace = parts[1];
             if (marketplace) {
+                const { recoverMarketplaceSettings } = await import("./claude-settings.js");
+                await recoverMarketplaceSettings();
                 await updateMarketplace(marketplace);
                 await execClaude(["plugin", "install", pluginId, "--scope", scope], 60000);
                 return;

package/src/services/claude-cli.ts

@@ -70,9 +70,8 @@ async function execClaude(args: string[], timeoutMs = 30000): Promise<string> {
  * Handles enabling + version tracking + cache copy in one shot.
  *
  * If the install fails because the plugin is "not found in marketplace",
- * triggers a marketplace update and retries once. This handles the case
- * where claudeup's remote listing (from GitHub) has newer plugins than
- * the local marketplace clone.
+ * recovers the marketplace registry (fixes stale "directory" sources),
+ * triggers a marketplace update, and retries once.
  */
 export async function installPlugin(
 	pluginId: string,
@@ -84,10 +83,15 @@ export async function installPlugin(
 		const msg =
 			error instanceof Error ? error.message : String(error);
 		if (msg.includes("not found in marketplace")) {
-			// Local marketplace clone is stale — update and retry
 			const parts = pluginId.split("@");
 			const marketplace = parts[1];
 			if (marketplace) {
+				// Fix known_marketplaces.json first (stale "directory" → "github"),
+				// then update, then retry
+				const { recoverMarketplaceSettings } = await import(
+					"./claude-settings.js"
+				);
+				await recoverMarketplaceSettings();
 				await updateMarketplace(marketplace);
 				await execClaude([
 					"plugin",
@@ -173,6 +177,10 @@ export async function updatePlugin(
 			const parts = pluginId.split("@");
 			const marketplace = parts[1];
 			if (marketplace) {
+				const { recoverMarketplaceSettings } = await import(
+					"./claude-settings.js"
+				);
+				await recoverMarketplaceSettings();
 				await updateMarketplace(marketplace);
 				await execClaude(
 					["plugin", "install", pluginId, "--scope", scope],

package/src/ui/App.js

@@ -7,7 +7,7 @@ import { DimensionsProvider, useDimensions, } from "./state/DimensionsContext.js
 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 } from "../services/claude-settings.js";
+import { migrateMarketplaceRename, recoverMarketplaceSettings, } from "../services/claude-settings.js";
 import { checkForUpdates, getCurrentVersion, } from "../services/version-check.js";
 import { useKeyboardHandler } from "./hooks/useKeyboardHandler.js";
 import { ProgressBar } from "./components/layout/ProgressBar.js";
@@ -166,7 +166,7 @@ MCP Servers
 function UpdateBanner({ result }) {
     if (!result.updateAvailable)
         return null;
-    return (_jsxs("box", { paddingLeft: 1, paddingRight: 1, children: [_jsx("text", { bg: "yellow", fg: "black", children: _jsx("strong", { children: " UPDATE " }) }), _jsxs("text", { fg: "yellow", children: [" ", "v", result.currentVersion, " \u2192 v", result.latestVersion] }), _jsx("text", { fg: "gray", children: " Run: " }), _jsx("text", { fg: "cyan", children: "npm i -g claudeup" })] }));
+    return (_jsxs("box", { paddingLeft: 1, paddingRight: 1, children: [_jsx("text", { bg: "yellow", fg: "black", children: _jsx("strong", { children: " UPDATE " }) }), _jsxs("text", { fg: "yellow", children: [" ", "v", result.currentVersion, " \u2192 v", result.latestVersion] }), _jsx("text", { fg: "gray", children: " Run: " }), _jsx("text", { fg: "cyan", children: "claudeup update" })] }));
 }
 /**
  * ProgressIndicator Component
@@ -179,6 +179,14 @@ function AppContentInner({ showDebug, onDebugToggle, updateInfo, onExit, }) {
     const { state, dispatch } = useApp();
     const { progress } = state;
     const dimensions = useDimensions();
+    const [recoveryReport, setRecoveryReport] = useState(null);
+    // Auto-dismiss recovery banner after 5 seconds
+    useEffect(() => {
+        if (!recoveryReport)
+            return;
+        const timer = setTimeout(() => setRecoveryReport(null), 5000);
+        return () => clearTimeout(timer);
+    }, [recoveryReport]);
     // Auto-refresh marketplaces on startup
     useEffect(() => {
         const noRefresh = process.argv.includes("--no-refresh");
@@ -190,6 +198,34 @@ function AppContentInner({ showDebug, onDebugToggle, updateInfo, onExit, }) {
         });
         // 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 = [];
+            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
         repairAllMarketplaces()
             .then(async () => {
             dispatch({ type: "HIDE_PROGRESS" });
@@ -199,7 +235,7 @@ function AppContentInner({ showDebug, onDebugToggle, updateInfo, onExit, }) {
             dispatch({ type: "HIDE_PROGRESS" });
         });
     }, [dispatch]);
-    return (_jsxs("box", { flexDirection: "column", height: dimensions.terminalHeight, children: [updateInfo?.updateAvailable && _jsx(UpdateBanner, { result: updateInfo }), 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 } = useApp();

package/src/ui/App.tsx

@@ -24,7 +24,10 @@ import {
 } from "./screens/index.js";
 import type { Screen } from "./state/types.js";
 import { repairAllMarketplaces } from "../services/local-marketplace.js";
-import { migrateMarketplaceRename } from "../services/claude-settings.js";
+import {
+	migrateMarketplaceRename,
+	recoverMarketplaceSettings,
+} from "../services/claude-settings.js";
 import {
 	checkForUpdates,
 	getCurrentVersion,
@@ -213,7 +216,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>
 	);
 }
@@ -258,6 +261,14 @@ function AppContentInner({
 	const { state, dispatch } = useApp();
 	const { progress } = state;
 	const dimensions = useDimensions();
+	const [recoveryReport, setRecoveryReport] = useState<string | null>(null);
+
+	// Auto-dismiss recovery banner after 5 seconds
+	useEffect(() => {
+		if (!recoveryReport) return;
+		const timer = setTimeout(() => setRecoveryReport(null), 5000);
+		return () => clearTimeout(timer);
+	}, [recoveryReport]);
 
 	// Auto-refresh marketplaces on startup
 	useEffect(() => {
@@ -272,6 +283,40 @@ function AppContentInner({
 		// 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
+
 		repairAllMarketplaces()
 			.then(async () => {
 				dispatch({ type: "HIDE_PROGRESS" });
@@ -285,6 +330,11 @@ function AppContentInner({
 	return (
 		<box flexDirection="column" height={dimensions.terminalHeight}>
 			{updateInfo?.updateAvailable && <UpdateBanner result={updateInfo} />}
+			{recoveryReport && (
+				<box paddingLeft={1} paddingRight={1}>
+					<text fg="green">✓ Fixed: {recoveryReport}</text>
+				</box>
+			)}
 			{showDebug && (
 				<box paddingLeft={1} paddingRight={1}>
 					<text fg="#888888">