claudeup 3.6.5 → 3.6.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claudeup",
-  "version": "3.6.5",
+  "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
@@ -55,16 +57,25 @@ export async function installPlugin(pluginId, scope = "user") {
 }
 /**
  * Uninstall a plugin using claude CLI.
- * Falls back to `disable` if `uninstall` fails (e.g., for plugins
- * installed via old JSON method that the CLI doesn't recognize).
+ * 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") {
+export async function uninstallPlugin(pluginId, scope = "user", projectPath) {
     try {
         await execClaude(["plugin", "uninstall", pluginId, "--scope", scope]);
     }
     catch {
-        // Fallback: disable works for plugins installed via old JSON method
-        await execClaude(["plugin", "disable", pluginId, "--scope", scope]);
+        // 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/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);
 
@@ -76,18 +83,26 @@ export async function installPlugin(
 
 /**
  * Uninstall a plugin using claude CLI.
- * Falls back to `disable` if `uninstall` fails (e.g., for plugins
- * installed via old JSON method that the CLI doesn't recognize).
+ * 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> {
 	try {
 		await execClaude(["plugin", "uninstall", pluginId, "--scope", scope]);
 	} catch {
-		// Fallback: disable works for plugins installed via old JSON method
-		await execClaude(["plugin", "disable", pluginId, "--scope", scope]);
+		// 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) {