claudeup 3.7.0 → 3.7.2

package/package.json

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

package/src/prerunner/index.js

@@ -4,7 +4,7 @@ import os from "node:os";
 import { UpdateCache } from "../services/update-cache.js";
 import { getAvailablePlugins, clearMarketplaceCache, } from "../services/plugin-manager.js";
 import { runClaude } from "../services/claude-runner.js";
-import { recoverMarketplaceSettings, migrateMarketplaceRename, getGlobalEnabledPlugins, getEnabledPlugins, getLocalEnabledPlugins, } from "../services/claude-settings.js";
+import { recoverMarketplaceSettings, migrateMarketplaceRename, getGlobalEnabledPlugins, getEnabledPlugins, getLocalEnabledPlugins, saveGlobalInstalledPluginVersion, } from "../services/claude-settings.js";
 import { parsePluginId } from "../utils/string-utils.js";
 import { defaultMarketplaces } from "../data/marketplaces.js";
 import { updatePlugin, addMarketplace, isClaudeAvailable, } from "../services/claude-cli.js";
@@ -133,6 +133,7 @@ export async function prerunClaude(claudeArgs, options = {}) {
                         if (cliAvailable) {
                             await updatePlugin(plugin.id, "user");
                         }
+                        await saveGlobalInstalledPluginVersion(plugin.id, plugin.version);
                         autoUpdatedPlugins.push({
                             pluginId: plugin.id,
                             oldVersion: plugin.installedVersion,

package/src/prerunner/index.ts

@@ -13,6 +13,7 @@ import {
 	getGlobalEnabledPlugins,
 	getEnabledPlugins,
 	getLocalEnabledPlugins,
+	saveGlobalInstalledPluginVersion,
 } from "../services/claude-settings.js";
 import { parsePluginId } from "../utils/string-utils.js";
 import { defaultMarketplaces } from "../data/marketplaces.js";
@@ -192,6 +193,7 @@ export async function prerunClaude(
 						if (cliAvailable) {
 							await updatePlugin(plugin.id, "user");
 						}
+						await saveGlobalInstalledPluginVersion(plugin.id, plugin.version);
 
 						autoUpdatedPlugins.push({
 							pluginId: plugin.id,

package/src/services/plugin-setup.js

@@ -160,13 +160,28 @@ async function installBrewPackages(packages, result) {
     }
 }
 /**
- * Install npm global packages
+ * Detect the best available Node.js package installer for globals
+ * Prefers: bun > npm
  */
-async function installNpmPackages(packages, result) {
+async function detectNpmCommand() {
+    const bunPath = await which("bun");
+    if (bunPath) {
+        return { cmd: bunPath, args: ["install", "-g"], label: "bun" };
+    }
     const npmPath = await which("npm");
-    if (!npmPath) {
+    if (npmPath) {
+        return { cmd: npmPath, args: ["install", "-g"], label: "npm" };
+    }
+    return null;
+}
+/**
+ * Install global Node.js packages (prefers bun > npm)
+ */
+async function installNpmPackages(packages, result) {
+    const installer = await detectNpmCommand();
+    if (!installer) {
         for (const pkg of packages) {
-            result.failed.push({ pkg: `npm:${pkg}`, error: "npm not found" });
+            result.failed.push({ pkg: `npm:${pkg}`, error: "No bun or npm found" });
         }
         return;
     }
@@ -175,12 +190,12 @@ async function installNpmPackages(packages, result) {
             result.skipped.push(`npm:${pkg}`);
             continue;
         }
-        const { ok, stderr } = await run(npmPath, ["install", "-g", pkg]);
+        const { ok, stderr } = await run(installer.cmd, [...installer.args, pkg]);
         if (ok) {
-            result.installed.push(`npm:${pkg}`);
+            result.installed.push(`${installer.label}:${pkg}`);
         }
         else {
-            result.failed.push({ pkg: `npm:${pkg}`, error: stderr });
+            result.failed.push({ pkg: `${installer.label}:${pkg}`, error: stderr });
         }
     }
 }

package/src/services/plugin-setup.ts

@@ -196,16 +196,36 @@ async function installBrewPackages(
 }
 
 /**
- * Install npm global packages
+ * Detect the best available Node.js package installer for globals
+ * Prefers: bun > npm
+ */
+async function detectNpmCommand(): Promise<{
+	cmd: string;
+	args: string[];
+	label: string;
+} | null> {
+	const bunPath = await which("bun");
+	if (bunPath) {
+		return { cmd: bunPath, args: ["install", "-g"], label: "bun" };
+	}
+	const npmPath = await which("npm");
+	if (npmPath) {
+		return { cmd: npmPath, args: ["install", "-g"], label: "npm" };
+	}
+	return null;
+}
+
+/**
+ * Install global Node.js packages (prefers bun > npm)
  */
 async function installNpmPackages(
 	packages: string[],
 	result: SetupResult,
 ): Promise<void> {
-	const npmPath = await which("npm");
-	if (!npmPath) {
+	const installer = await detectNpmCommand();
+	if (!installer) {
 		for (const pkg of packages) {
-			result.failed.push({ pkg: `npm:${pkg}`, error: "npm not found" });
+			result.failed.push({ pkg: `npm:${pkg}`, error: "No bun or npm found" });
 		}
 		return;
 	}
@@ -216,11 +236,11 @@ async function installNpmPackages(
 			continue;
 		}
 
-		const { ok, stderr } = await run(npmPath, ["install", "-g", pkg]);
+		const { ok, stderr } = await run(installer.cmd, [...installer.args, pkg]);
 		if (ok) {
-			result.installed.push(`npm:${pkg}`);
+			result.installed.push(`${installer.label}:${pkg}`);
 		} else {
-			result.failed.push({ pkg: `npm:${pkg}`, error: stderr });
+			result.failed.push({ pkg: `${installer.label}:${pkg}`, error: stderr });
 		}
 	}
 }

package/src/ui/screens/PluginsScreen.js

@@ -9,7 +9,8 @@ import { ScrollableList } from "../components/ScrollableList.js";
 import { fuzzyFilter, highlightMatches } from "../../utils/fuzzy-search.js";
 import { getAllMarketplaces } from "../../data/marketplaces.js";
 import { getAvailablePlugins, refreshAllMarketplaces, clearMarketplaceCache, getLocalMarketplacesInfo, } from "../../services/plugin-manager.js";
-import { setMcpEnvVar, getMcpEnvVars, } from "../../services/claude-settings.js";
+import { setMcpEnvVar, getMcpEnvVars, saveGlobalInstalledPluginVersion, saveLocalInstalledPluginVersion, } from "../../services/claude-settings.js";
+import { saveInstalledPluginVersion, } from "../../services/plugin-manager.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";
@@ -399,6 +400,27 @@ export function PluginsScreen() {
             console.error("Error installing plugin deps:", error);
         }
     };
+    /**
+     * Save the installed version to settings after CLI install/update.
+     * Claude CLI doesn't update installedPluginVersions in settings.json,
+     * so we do it ourselves to keep the TUI version display accurate.
+     */
+    const saveVersionAfterInstall = async (pluginId, version, scope) => {
+        try {
+            if (scope === "user") {
+                await saveGlobalInstalledPluginVersion(pluginId, version);
+            }
+            else if (scope === "local") {
+                await saveLocalInstalledPluginVersion(pluginId, version, state.projectPath);
+            }
+            else {
+                await saveInstalledPluginVersion(pluginId, version, state.projectPath);
+            }
+        }
+        catch {
+            // Non-fatal: version display may be stale but plugin still works
+        }
+    };
     const handleSelect = async () => {
         const item = selectableItems[pluginsState.selectedIndex];
         if (!item)
@@ -508,9 +530,11 @@ export function PluginsScreen() {
                 }
                 else if (action === "update") {
                     await cliUpdatePlugin(plugin.id, scope);
+                    await saveVersionAfterInstall(plugin.id, latestVersion, scope);
                 }
                 else {
                     await cliInstallPlugin(plugin.id, scope);
+                    await saveVersionAfterInstall(plugin.id, latestVersion, scope);
                     // On fresh install, configure env vars and install system deps
                     modal.hideModal();
                     await collectPluginEnvVars(plugin.name, plugin.marketplace);
@@ -536,6 +560,7 @@ export function PluginsScreen() {
         modal.loading(`Updating ${plugin.name}...`);
         try {
             await cliUpdatePlugin(plugin.id, scope);
+            await saveVersionAfterInstall(plugin.id, plugin.version || "0.0.0", scope);
             modal.hideModal();
             fetchData();
         }
@@ -555,6 +580,7 @@ export function PluginsScreen() {
         try {
             for (const plugin of updatable) {
                 await cliUpdatePlugin(plugin.id, scope);
+                await saveVersionAfterInstall(plugin.id, plugin.version || "0.0.0", scope);
             }
             modal.hideModal();
             fetchData();
@@ -608,9 +634,11 @@ export function PluginsScreen() {
             }
             else if (action === "update") {
                 await cliUpdatePlugin(plugin.id, scope);
+                await saveVersionAfterInstall(plugin.id, latestVersion, scope);
             }
             else {
                 await cliInstallPlugin(plugin.id, scope);
+                await saveVersionAfterInstall(plugin.id, latestVersion, scope);
                 // On fresh install, configure env vars and install system deps
                 modal.hideModal();
                 await collectPluginEnvVars(plugin.name, plugin.marketplace);

package/src/ui/screens/PluginsScreen.tsx

@@ -17,7 +17,12 @@ import {
 import {
 	setMcpEnvVar,
 	getMcpEnvVars,
+	saveGlobalInstalledPluginVersion,
+	saveLocalInstalledPluginVersion,
 } from "../../services/claude-settings.js";
+import {
+	saveInstalledPluginVersion,
+} from "../../services/plugin-manager.js";
 import {
 	installPlugin as cliInstallPlugin,
 	uninstallPlugin as cliUninstallPlugin,
@@ -531,6 +536,29 @@ export function PluginsScreen() {
 		}
 	};
 
+	/**
+	 * Save the installed version to settings after CLI install/update.
+	 * Claude CLI doesn't update installedPluginVersions in settings.json,
+	 * so we do it ourselves to keep the TUI version display accurate.
+	 */
+	const saveVersionAfterInstall = async (
+		pluginId: string,
+		version: string,
+		scope: PluginScope,
+	): Promise<void> => {
+		try {
+			if (scope === "user") {
+				await saveGlobalInstalledPluginVersion(pluginId, version);
+			} else if (scope === "local") {
+				await saveLocalInstalledPluginVersion(pluginId, version, state.projectPath);
+			} else {
+				await saveInstalledPluginVersion(pluginId, version, state.projectPath);
+			}
+		} catch {
+			// Non-fatal: version display may be stale but plugin still works
+		}
+	};
+
 	const handleSelect = async () => {
 		const item = selectableItems[pluginsState.selectedIndex];
 		if (!item) return;
@@ -663,8 +691,10 @@ export function PluginsScreen() {
 					await cliUninstallPlugin(plugin.id, scope, state.projectPath);
 				} else if (action === "update") {
 					await cliUpdatePlugin(plugin.id, scope);
+					await saveVersionAfterInstall(plugin.id, latestVersion, scope);
 				} else {
 					await cliInstallPlugin(plugin.id, scope);
+					await saveVersionAfterInstall(plugin.id, latestVersion, scope);
 
 					// On fresh install, configure env vars and install system deps
 					modal.hideModal();
@@ -692,6 +722,7 @@ export function PluginsScreen() {
 		modal.loading(`Updating ${plugin.name}...`);
 		try {
 			await cliUpdatePlugin(plugin.id, scope);
+			await saveVersionAfterInstall(plugin.id, plugin.version || "0.0.0", scope);
 			modal.hideModal();
 			fetchData();
 		} catch (error) {
@@ -712,6 +743,7 @@ export function PluginsScreen() {
 		try {
 			for (const plugin of updatable) {
 				await cliUpdatePlugin(plugin.id, scope);
+				await saveVersionAfterInstall(plugin.id, plugin.version || "0.0.0", scope);
 			}
 			modal.hideModal();
 			fetchData();
@@ -771,8 +803,10 @@ export function PluginsScreen() {
 				await cliUninstallPlugin(plugin.id, scope, state.projectPath);
 			} else if (action === "update") {
 				await cliUpdatePlugin(plugin.id, scope);
+				await saveVersionAfterInstall(plugin.id, latestVersion, scope);
 			} else {
 				await cliInstallPlugin(plugin.id, scope);
+				await saveVersionAfterInstall(plugin.id, latestVersion, scope);
 
 				// On fresh install, configure env vars and install system deps
 				modal.hideModal();