claudeup 3.6.4 → 3.6.6

package/package.json

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

package/src/services/claude-cli.js

@@ -11,6 +11,8 @@
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 import { which } from "../utils/command-utils.js";
+import { removeGlobalInstalledPluginVersion, removeLocalInstalledPluginVersion, } from "./claude-settings.js";
+import { removeInstalledPluginVersion, } from "./plugin-manager.js";
 const execFileAsync = promisify(execFile);
 /**
  * Get the path to the claude CLI binary
@@ -54,11 +56,27 @@ export async function installPlugin(pluginId, scope = "user") {
     await execClaude(["plugin", "install", pluginId, "--scope", scope]);
 }
 /**
- * Uninstall a plugin using claude CLI
- * Handles disabling + version removal in one shot
+ * Uninstall a plugin using claude CLI.
+ * Falls back to direct settings removal if CLI uninstall fails
+ * (e.g., for plugins installed via old JSON method).
+ * @param projectPath - Required for project/local scope fallback
  */
-export async function uninstallPlugin(pluginId, scope = "user") {
-    await execClaude(["plugin", "uninstall", pluginId, "--scope", scope]);
+export async function uninstallPlugin(pluginId, scope = "user", projectPath) {
+    try {
+        await execClaude(["plugin", "uninstall", pluginId, "--scope", scope]);
+    }
+    catch {
+        // Fallback: directly remove from settings JSON
+        if (scope === "user") {
+            await removeGlobalInstalledPluginVersion(pluginId);
+        }
+        else if (scope === "local" && projectPath) {
+            await removeLocalInstalledPluginVersion(pluginId, projectPath);
+        }
+        else if (projectPath) {
+            await removeInstalledPluginVersion(pluginId, projectPath);
+        }
+    }
 }
 /**
  * Enable a previously disabled plugin

package/src/services/claude-cli.ts

@@ -12,6 +12,13 @@
 import { execFile } from "node:child_process";
 import { promisify } from "node:util";
 import { which } from "../utils/command-utils.js";
+import {
+	removeGlobalInstalledPluginVersion,
+	removeLocalInstalledPluginVersion,
+} from "./claude-settings.js";
+import {
+	removeInstalledPluginVersion,
+} from "./plugin-manager.js";
 
 const execFileAsync = promisify(execFile);
 
@@ -75,14 +82,28 @@ export async function installPlugin(
 }
 
 /**
- * Uninstall a plugin using claude CLI
- * Handles disabling + version removal in one shot
+ * Uninstall a plugin using claude CLI.
+ * Falls back to direct settings removal if CLI uninstall fails
+ * (e.g., for plugins installed via old JSON method).
+ * @param projectPath - Required for project/local scope fallback
  */
 export async function uninstallPlugin(
 	pluginId: string,
 	scope: PluginScope = "user",
+	projectPath?: string,
 ): Promise<void> {
-	await execClaude(["plugin", "uninstall", pluginId, "--scope", scope]);
+	try {
+		await execClaude(["plugin", "uninstall", pluginId, "--scope", scope]);
+	} catch {
+		// Fallback: directly remove from settings JSON
+		if (scope === "user") {
+			await removeGlobalInstalledPluginVersion(pluginId);
+		} else if (scope === "local" && projectPath) {
+			await removeLocalInstalledPluginVersion(pluginId, projectPath);
+		} else if (projectPath) {
+			await removeInstalledPluginVersion(pluginId, projectPath);
+		}
+	}
 }
 
 /**

package/src/ui/screens/PluginsScreen.js

@@ -457,7 +457,7 @@ export function PluginsScreen() {
             try {
                 const scope = scopeValue;
                 if (action === "uninstall") {
-                    await cliUninstallPlugin(plugin.id, scope);
+                    await cliUninstallPlugin(plugin.id, scope, state.projectPath);
                 }
                 else if (action === "update") {
                     await cliUpdatePlugin(plugin.id, scope);
@@ -556,7 +556,7 @@ export function PluginsScreen() {
         modal.loading(`${actionLabel}...`);
         try {
             if (action === "uninstall") {
-                await cliUninstallPlugin(plugin.id, scope);
+                await cliUninstallPlugin(plugin.id, scope, state.projectPath);
             }
             else if (action === "update") {
                 await cliUpdatePlugin(plugin.id, scope);
@@ -611,7 +611,7 @@ export function PluginsScreen() {
             return; // Cancelled
         modal.loading(`Uninstalling ${plugin.name}...`);
         try {
-            await cliUninstallPlugin(plugin.id, scopeValue);
+            await cliUninstallPlugin(plugin.id, scopeValue, state.projectPath);
             modal.hideModal();
             fetchData();
         }

package/src/ui/screens/PluginsScreen.tsx

@@ -595,7 +595,7 @@ export function PluginsScreen() {
 			try {
 				const scope = scopeValue as PluginScope;
 				if (action === "uninstall") {
-					await cliUninstallPlugin(plugin.id, scope);
+					await cliUninstallPlugin(plugin.id, scope, state.projectPath);
 				} else if (action === "update") {
 					await cliUpdatePlugin(plugin.id, scope);
 				} else {
@@ -702,7 +702,7 @@ export function PluginsScreen() {
 
 		try {
 			if (action === "uninstall") {
-				await cliUninstallPlugin(plugin.id, scope);
+				await cliUninstallPlugin(plugin.id, scope, state.projectPath);
 			} else if (action === "update") {
 				await cliUpdatePlugin(plugin.id, scope);
 			} else {
@@ -769,7 +769,7 @@ export function PluginsScreen() {
 		modal.loading(`Uninstalling ${plugin.name}...`);
 
 		try {
-			await cliUninstallPlugin(plugin.id, scopeValue as PluginScope);
+			await cliUninstallPlugin(plugin.id, scopeValue as PluginScope, state.projectPath);
 			modal.hideModal();
 			fetchData();
 		} catch (error) {