magus-cli 7.11.0 → 7.11.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magus-cli",
-  "version": "7.11.0",
+  "version": "7.11.1",
   "description": "TUI tool for managing Claude Code plugins, MCPs, and configuration",
   "type": "module",
   "main": "src/main.tsx",
@@ -65,8 +65,8 @@
     "typescript": "^5.6.3"
   },
   "optionalDependencies": {
-    "magus-cli-darwin-arm64": "7.11.0",
-    "magus-cli-darwin-x64": "7.11.0",
-    "magus-cli-linux-x64": "7.11.0"
+    "magus-cli-darwin-arm64": "7.11.1",
+    "magus-cli-darwin-x64": "7.11.1",
+    "magus-cli-linux-x64": "7.11.1"
   }
 }

package/src/__tests__/blind/requires-min-version/manifests.test.ts

@@ -39,8 +39,9 @@ describe("shipped plugin manifests under schema 2", () => {
 		});
 	}
 
-	// REQ-26: claudish gets minVersion 10.7.0; no other requirement gets a floor.
-	test("REQ-26 claudish's claudish requirement has minVersion 10.7.0", () => {
+	// REQ-26: claudish gets a minVersion floor; no other requirement gets one. The floor
+	// moved from 10.7.0 to 11.1.0 with claudish plugin 2.5.0, which needs SlotRow.recent.
+	test("REQ-26 claudish's claudish requirement has minVersion 11.1.0", () => {
 		const claudish = declaring.find(({ name }) => name === "claudish");
 		if (!claudish) throw new Error("plugins/claudish declares no requires");
 		const result = parseRequires(claudish.json);
@@ -50,7 +51,7 @@ describe("shipped plugin manifests under schema 2", () => {
 			| (Record<string, unknown> & { name: string })
 			| undefined;
 		expect(req).toBeDefined();
-		expect(req?.minVersion).toBe("10.7.0");
+		expect(req?.minVersion).toBe("11.1.0");
 	});
 
 	test("REQ-26 no other shipped requirement declares a floor", () => {

package/src/__tests__/scope-action-deprecated.test.ts

@@ -0,0 +1,105 @@
+import { describe, expect, test } from "bun:test";
+import {
+	type ScopeStatus,
+	isInstalledInScope,
+	resolveScopeAction,
+} from "../services/plugin-manager.js";
+
+/**
+ * MTL-70: a deprecated plugin (no longer in any marketplace,
+ * `PluginInfo.isOrphaned === true`) can only be removed from a scope key.
+ * Installed in the scope → "uninstall"; not installed → "none". `behind` never
+ * matters for a deprecated plugin. Written blind from the spec.
+ */
+
+const installedScopes: Array<[string, ScopeStatus]> = [
+	["installed with a registry version", { installed: true, version: "3.0.2" }],
+	["installed with no version (files missing)", { installed: true }],
+	['installed at "0.0.0"', { installed: true, version: "0.0.0" }],
+	[
+		"installed with an unorderable sha",
+		{ installed: true, version: "3ea32df27be7" },
+	],
+];
+
+const notInstalledScopes: Array<[string, ScopeStatus | undefined]> = [
+	["an absent scope", undefined],
+	["installed: false", { installed: false }],
+	[
+		"installed: false with a registry version",
+		{ installed: false, version: "3.0.2" },
+	],
+];
+
+describe("resolveScopeAction — deprecated plugin installed in the scope (REQ-1)", () => {
+	for (const [label, scope] of installedScopes) {
+		test(`${label} is installed per isInstalledInScope (precondition)`, () => {
+			expect(isInstalledInScope(scope)).toBe(true);
+		});
+
+		for (const behind of [false, true]) {
+			test(`REQ-1: ${label}, behind=${behind} resolves to uninstall`, () => {
+				expect(resolveScopeAction(scope, behind, true)).toBe("uninstall");
+			});
+		}
+	}
+});
+
+describe("resolveScopeAction — deprecated plugin NOT installed in the scope (REQ-2)", () => {
+	for (const [label, scope] of notInstalledScopes) {
+		test(`${label} is not installed per isInstalledInScope (precondition)`, () => {
+			expect(isInstalledInScope(scope)).toBe(false);
+		});
+
+		for (const behind of [false, true]) {
+			test(`REQ-2: ${label}, behind=${behind} resolves to none — no install, no update`, () => {
+				const action = resolveScopeAction(scope, behind, true);
+				expect(action).toBe("none");
+				expect(action).not.toBe("install");
+				expect(action).not.toBe("update");
+			});
+		}
+	}
+});
+
+describe("resolveScopeAction — orphaned false or omitted is unchanged (REQ-3)", () => {
+	const cases: Array<[string, ScopeStatus | undefined, boolean, string]> = [
+		["files missing", { installed: true }, false, "install"],
+		["files missing, behind", { installed: true }, true, "install"],
+		["never installed (absent)", undefined, false, "install"],
+		["never installed (flag off)", { installed: false }, false, "install"],
+		[
+			"healthy at latest",
+			{ installed: true, version: "3.0.2" },
+			false,
+			"uninstall",
+		],
+		[
+			"healthy and behind",
+			{ installed: true, version: "2.9.0" },
+			true,
+			"update",
+		],
+	];
+
+	for (const [label, scope, behind, expected] of cases) {
+		test(`REQ-3: ${label} with orphaned omitted resolves to ${expected}`, () => {
+			expect(resolveScopeAction(scope, behind)).toBe(expected as never);
+		});
+		test(`REQ-3: ${label} with orphaned=false resolves to ${expected}`, () => {
+			expect(resolveScopeAction(scope, behind, false)).toBe(expected as never);
+		});
+	}
+
+	test("REQ-3: a non-deprecated plugin never resolves to none", () => {
+		const all: Array<ScopeStatus | undefined> = [
+			...installedScopes.map(([, s]) => s),
+			...notInstalledScopes.map(([, s]) => s),
+		];
+		for (const scope of all) {
+			for (const behind of [false, true]) {
+				expect(resolveScopeAction(scope, behind, false)).not.toBe("none");
+			}
+		}
+	});
+});

package/src/cli/router.ts

@@ -103,7 +103,12 @@ const SUBCOMMANDS = new Set([
 	"doctor",
 ]);
 
-/** Subcommands that read or write profile state, and so pass the gate first. */
+/**
+ * Subcommands that read or write profile state, and so pass the gate first.
+ * code-search setup runs `magus profile list` to regenerate settings.json after it
+ * edits profiles.json, so `profile` must keep running the gate
+ * (ai-docs/code-search-setup-profiles.md).
+ */
 const PROFILE_GATED = new Set(["install", "update", "profile", "models"]);
 
 export async function route(

package/src/services/plugin-manager.ts

@@ -371,7 +371,11 @@ export function buildScopeStatus(
 export function resolveScopeAction(
 	scope: ScopeStatus | undefined,
 	behind: boolean,
-): "install" | "update" | "uninstall" {
+	orphaned = false,
+): "install" | "update" | "uninstall" | "none" {
+	// A deprecated plugin is no longer in any marketplace, so install and update
+	// can only fail with a raw CLI error. Removing it is the one action left.
+	if (orphaned) return isInstalledInScope(scope) ? "uninstall" : "none";
 	if (!isInstalledInScope(scope) || scopeFilesMissing(scope)) return "install";
 	return behind ? "update" : "uninstall";
 }

package/src/ui/screens/PluginsScreen.tsx

@@ -1009,7 +1009,9 @@ export function PluginsScreen() {
 			const action = resolveScopeAction(
 				selectedScope,
 				scopeIsBehind(status, scopeValue as PluginScope),
+				plugin.isOrphaned,
 			);
+			if (action === "none") return;
 
 			try {
 				const scope = scopeValue as PluginScope;
@@ -1174,7 +1176,9 @@ export function PluginsScreen() {
 		const action = resolveScopeAction(
 			scopeData,
 			scopeIsBehind(statuses.get(plugin.id), scope),
+			plugin.isOrphaned,
 		);
+		if (action === "none") return;
 
 		try {
 			if (action === "uninstall") {