claudeup 4.10.0 → 4.10.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claudeup",
-  "version": "4.10.0",
+  "version": "4.10.1",
   "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";
@@ -179,6 +179,7 @@ function AppContentInner({ showDebug, onDebugToggle, updateInfo, onExit, }) {
     const { state, dispatch } = useApp();
     const { progress } = state;
     const dimensions = useDimensions();
+    const [recoveryReport, setRecoveryReport] = useState(null);
     // Auto-refresh marketplaces on startup
     useEffect(() => {
         const noRefresh = process.argv.includes("--no-refresh");
@@ -190,6 +191,35 @@ 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 msgs = [];
+            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" });
@@ -199,7 +229,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 && (_jsxs("box", { paddingLeft: 1, paddingRight: 1, children: [_jsx("text", { bg: "green", fg: "black", children: _jsx("strong", { children: " RECOVERED " }) }), _jsxs("text", { fg: "green", children: [" ", recoveryReport.split("\n").join(" | ")] })] })), 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,
@@ -258,6 +261,7 @@ function AppContentInner({
 	const { state, dispatch } = useApp();
 	const { progress } = state;
 	const dimensions = useDimensions();
+	const [recoveryReport, setRecoveryReport] = useState<string | null>(null);
 
 	// Auto-refresh marketplaces on startup
 	useEffect(() => {
@@ -272,6 +276,43 @@ 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 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" });
@@ -285,6 +326,14 @@ function AppContentInner({
 	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>
+				</box>
+			)}
 			{showDebug && (
 				<box paddingLeft={1} paddingRight={1}>
 					<text fg="#888888">