magus-cli 7.8.0 → 7.9.0

package/README.md

@@ -155,7 +155,7 @@ check. So does `magus install --check`.
 | MCP | Add MCP servers from a curated catalog |
 | Settings | Claude Code settings, from a catalog of known keys |
 | Profiles | Saved plugin sets (the TUI's own, older profile concept) |
-| CLI Tools | Install claudish, mnemex and friends via the right package manager |
+| CLI Tools | Install and update CLI tools via the right package manager; a tool a plugin installs from a release shows as managed by that plugin |
 | Git State | Gitignore conventions and repo hygiene |
 | Alias | Shell alias + flag management for launching `claude` |
 | Styles | Communication style presets, composed into one native output style |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magus-cli",
-  "version": "7.8.0",
+  "version": "7.9.0",
   "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.8.0",
-    "magus-cli-darwin-x64": "7.8.0",
-    "magus-cli-linux-x64": "7.8.0"
+    "magus-cli-darwin-arm64": "7.9.0",
+    "magus-cli-darwin-x64": "7.9.0",
+    "magus-cli-linux-x64": "7.9.0"
   }
 }

package/src/__tests__/blackbox-parity.test.ts

@@ -17,11 +17,11 @@
  * _synthetic exist only to reach states the real data lacks.
  */
 import { describe, expect, test } from "bun:test";
-import { cliTools } from "../data/cli-tools.js";
 import { selectProfileId } from "../services/active-closure.js";
 import type { PluginInfo } from "../services/plugin-manager.js";
 import { resolveProfile } from "../services/resolver.js";
 import { type ToolFacts, specForCatalogTool } from "../services/tool-probe.js";
+import { managedBins, managedTool } from "../services/toolchain.js";
 import {
 	type BinUpdateItem,
 	type PluginUpdateItem,
@@ -31,7 +31,7 @@ import {
 	planPluginUpdates,
 } from "../services/update-plan.js";
 import type {
-	BinRequirement,
+	ManagedTool,
 	Marketplace,
 	ProfileManifest,
 	ResolvedClosure,
@@ -42,6 +42,7 @@ import {
 	buildPluginBrowserItems,
 } from "../ui/adapters/pluginsAdapter.js";
 import { pluginRowText } from "../ui/renderers/pluginRowText.js";
+import { ownedAsDeclared, selected, testCatalogue } from "./fixtures/bins.js";
 
 // ── fixture ────────────────────────────────────────────────────────────────
 
@@ -199,7 +200,7 @@ function makeManifest(): ProfileManifest {
 	};
 }
 
-const PLUGIN_BINS: Record<string, BinRequirement[]> = {
+const PLUGIN_BINS: Record<string, ManagedTool[]> = {
 	"terminal@magus": [
 		{ name: "tmux", via: "brew", check: "tmux -V" },
 		{
@@ -215,8 +216,8 @@ const PLUGIN_BINS: Record<string, BinRequirement[]> = {
 	],
 };
 
-async function fixtureBins(pluginId: string): Promise<BinRequirement[]> {
-	return (PLUGIN_BINS[pluginId] ?? []).map((b) => ({ ...b }));
+async function fixtureBins(pluginId: string) {
+	return (PLUGIN_BINS[pluginId] ?? []).map((b) => selected(b));
 }
 
 function facts(name: string, over: Partial<ToolFacts>): ToolFacts {
@@ -330,7 +331,9 @@ async function runBothPaths(): Promise<Paths> {
 	if (profileId === null) throw new Error("fixture: no profile selected");
 	const closure = await resolveProfile(manifest, profileId, {
 		binResolver: fixtureBins,
+		ownedResolver: ownedAsDeclared,
 	});
+	const tools = managedBins(closure.bins);
 
 	// CLI path
 	const cliPluginList = planPluginUpdates(
@@ -359,18 +362,18 @@ async function runBothPaths(): Promise<Paths> {
 	// tools
 	const factList = makeFacts();
 	const cliBins = new Map(
-		planBinUpdates(closure.bins, factList).map((b) => [b.name, b]),
+		planBinUpdates(tools, factList).map((b) => [b.name, b]),
 	);
 	const factsByName = new Map(factList.map((f) => [f.name, f]));
-	const declared = new Set(closure.bins.map((b) => b.name));
+	const declared = new Set(tools.map((b) => b.name));
 	const tuiTools = new Map<string, ReturnType<typeof buildToolRow>>();
-	for (const tool of cliTools) {
+	for (const tool of testCatalogue()) {
 		if (!declared.has(tool.name)) continue;
 		tuiTools.set(
 			tool.name,
 			buildToolRow(
 				tool,
-				specForCatalogTool(tool, closure.bins, undefined),
+				specForCatalogTool(tool, tools, undefined),
 				factsByName.get(tool.name),
 			),
 		);
@@ -624,7 +627,8 @@ describe("R7 parity: one fixture through the CLI path and the TUI path", () => {
 				target: cli?.target,
 			});
 			// the TUI judges a declared tool against the closure's own spec
-			const declared = p.closure.bins.find((b) => b.name === name);
+			const bin = p.closure.bins.find((b) => b.name === name);
+			const declared = bin ? managedTool(bin) : null;
 			expect({ name, via: tui?.spec.via, version: tui?.spec.version }).toEqual({
 				name,
 				via: declared?.via,
@@ -697,7 +701,8 @@ describe("fixture discriminates the pre-change logic", () => {
 		const p = await runBothPaths();
 
 		for (const name of ["tmux", "browser-use", "mcp"]) {
-			const spec = p.closure.bins.find((b) => b.name === name);
+			const bin = p.closure.bins.find((b) => b.name === name);
+			const spec = bin ? managedTool(bin) : null;
 			expect(legacyPlanBin(true, Boolean(spec?.version))).toBe("upgrade");
 			expect(p.cliBins.get(name)?.action).toBe("current");
 		}