claudeup 6.6.0 → 6.8.0

package/package.json

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

package/src/__tests__/conventions-manager.test.ts

@@ -240,11 +240,11 @@ describe("conventions-manager", () => {
 			const content = [
 				"# Project CLAUDE.md",
 				"",
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"## Code Analysis Conventions",
 				"",
 				"> Managed by plugin",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 				"",
 				"Other content",
 			].join("\n");
@@ -253,7 +253,7 @@ describe("conventions-manager", () => {
 
 			expect(result.errors).toEqual([]);
 			expect(result.sections).toHaveLength(1);
-			expect(result.sections[0].section).toBe("code-analysis");
+			expect(result.sections[0].section).toBe("code-search");
 			expect(result.sections[0].version).toBe("1.0.0");
 			expect(result.sections[0].content).toContain("Code Analysis Conventions");
 			expect(result.sections[0].startLine).toBe(3);
@@ -262,9 +262,9 @@ describe("conventions-manager", () => {
 
 		it("should parse multiple sections", () => {
 			const content = [
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Section 1 content",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 				"",
 				"<!-- BEGIN magus:terminal v2.0.0 -->",
 				"Section 2 content",
@@ -275,13 +275,13 @@ describe("conventions-manager", () => {
 
 			expect(result.errors).toEqual([]);
 			expect(result.sections).toHaveLength(2);
-			expect(result.sections[0].section).toBe("code-analysis");
+			expect(result.sections[0].section).toBe("code-search");
 			expect(result.sections[1].section).toBe("terminal");
 		});
 
 		it("should detect missing END marker", () => {
 			const content = [
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Some content without an end marker",
 			].join("\n");
 
@@ -289,18 +289,18 @@ describe("conventions-manager", () => {
 
 			expect(result.errors).toHaveLength(1);
 			expect(result.errors[0].type).toBe("missing_end");
-			expect(result.errors[0]).toHaveProperty("section", "code-analysis");
+			expect(result.errors[0]).toHaveProperty("section", "code-search");
 		});
 
 		it("should detect duplicate sections", () => {
 			const content = [
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"First",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 				"",
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Second",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 			].join("\n");
 
 			const result = parseClaudeMdContent(content);
@@ -309,7 +309,7 @@ describe("conventions-manager", () => {
 				(e) => e.type === "duplicate_section",
 			);
 			expect(dupError).toBeDefined();
-			expect(dupError).toHaveProperty("section", "code-analysis");
+			expect(dupError).toHaveProperty("section", "code-search");
 		});
 
 		it("should ignore markers inside code blocks", () => {
@@ -473,9 +473,9 @@ describe("conventions-manager", () => {
 			const content = [
 				"# Project",
 				"",
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Conventions here",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 			].join("\n");
 			await writeFile(claudeMdPath, content);
 
@@ -505,7 +505,7 @@ describe("conventions-manager", () => {
 
 			const result = await injectClaudeMdSection(
 				claudeMdPath,
-				"code-analysis",
+				"code-search",
 				"1.0.0",
 				"## Code Analysis\n\nPrivate paths here",
 			);
@@ -513,9 +513,9 @@ describe("conventions-manager", () => {
 			expect(result.action).toBe("inserted");
 
 			const content = await readFile(claudeMdPath, "utf-8");
-			expect(content).toContain("<!-- BEGIN magus:code-analysis v1.0.0 -->");
+			expect(content).toContain("<!-- BEGIN magus:code-search v1.0.0 -->");
 			expect(content).toContain("## Code Analysis");
-			expect(content).toContain("<!-- END magus:code-analysis -->");
+			expect(content).toContain("<!-- END magus:code-search -->");
 		});
 
 		it("should insert new section into existing file", async () => {
@@ -524,7 +524,7 @@ describe("conventions-manager", () => {
 
 			const result = await injectClaudeMdSection(
 				claudeMdPath,
-				"code-analysis",
+				"code-search",
 				"1.0.0",
 				"New content",
 			);
@@ -534,7 +534,7 @@ describe("conventions-manager", () => {
 			const content = await readFile(claudeMdPath, "utf-8");
 			expect(content).toContain("# Project");
 			expect(content).toContain("Existing content");
-			expect(content).toContain("<!-- BEGIN magus:code-analysis v1.0.0 -->");
+			expect(content).toContain("<!-- BEGIN magus:code-search v1.0.0 -->");
 		});
 
 		it("should update section when version changes", async () => {
@@ -542,15 +542,15 @@ describe("conventions-manager", () => {
 			const initial = [
 				"# Project",
 				"",
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Old content",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 			].join("\n");
 			await writeFile(claudeMdPath, initial);
 
 			const result = await injectClaudeMdSection(
 				claudeMdPath,
-				"code-analysis",
+				"code-search",
 				"2.0.0",
 				"New content",
 			);
@@ -568,15 +568,15 @@ describe("conventions-manager", () => {
 		it("should skip when version is unchanged", async () => {
 			const claudeMdPath = join(testDir, "CLAUDE.md");
 			const initial = [
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Current content",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 			].join("\n");
 			await writeFile(claudeMdPath, initial);
 
 			const result = await injectClaudeMdSection(
 				claudeMdPath,
-				"code-analysis",
+				"code-search",
 				"1.0.0",
 				"Different content but same version",
 			);
@@ -598,7 +598,7 @@ describe("conventions-manager", () => {
 
 			const result = await injectClaudeMdSection(
 				claudeMdPath,
-				"code-analysis",
+				"code-search",
 				"1.0.0",
 				"New content",
 			);
@@ -610,13 +610,13 @@ describe("conventions-manager", () => {
 		it("should refuse when file has duplicate sections", async () => {
 			const claudeMdPath = join(testDir, "CLAUDE.md");
 			const duplicate = [
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"First",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 				"",
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Second",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 			].join("\n");
 			await writeFile(claudeMdPath, duplicate);
 
@@ -642,21 +642,21 @@ describe("conventions-manager", () => {
 			const content = [
 				"# Project",
 				"",
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"Conventions here",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 				"",
 				"Other content",
 			].join("\n");
 			await writeFile(claudeMdPath, content);
 
-			const result = await removeClaudeMdSection(claudeMdPath, "code-analysis");
+			const result = await removeClaudeMdSection(claudeMdPath, "code-search");
 
 			expect(result.removed).toBe(true);
 
 			const newContent = await readFile(claudeMdPath, "utf-8");
-			expect(newContent).not.toContain("BEGIN magus:code-analysis");
-			expect(newContent).not.toContain("END magus:code-analysis");
+			expect(newContent).not.toContain("BEGIN magus:code-search");
+			expect(newContent).not.toContain("END magus:code-search");
 			expect(newContent).toContain("# Project");
 			expect(newContent).toContain("Other content");
 		});
@@ -665,7 +665,7 @@ describe("conventions-manager", () => {
 			const claudeMdPath = join(testDir, "CLAUDE.md");
 			await writeFile(claudeMdPath, "# Project\n\nSome content\n");
 
-			const result = await removeClaudeMdSection(claudeMdPath, "code-analysis");
+			const result = await removeClaudeMdSection(claudeMdPath, "code-search");
 
 			expect(result.removed).toBe(false);
 		});
@@ -673,7 +673,7 @@ describe("conventions-manager", () => {
 		it("should no-op for non-existent file", async () => {
 			const claudeMdPath = join(testDir, "CLAUDE.md");
 
-			const result = await removeClaudeMdSection(claudeMdPath, "code-analysis");
+			const result = await removeClaudeMdSection(claudeMdPath, "code-search");
 
 			expect(result.removed).toBe(false);
 		});
@@ -724,9 +724,9 @@ describe("conventions-manager", () => {
 		it("should preserve other sections when removing one", async () => {
 			const claudeMdPath = join(testDir, "CLAUDE.md");
 			const content = [
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"First section",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 				"",
 				"<!-- BEGIN magus:terminal v2.0.0 -->",
 				"Second section",
@@ -734,10 +734,10 @@ describe("conventions-manager", () => {
 			].join("\n");
 			await writeFile(claudeMdPath, content);
 
-			await removeClaudeMdSection(claudeMdPath, "code-analysis");
+			await removeClaudeMdSection(claudeMdPath, "code-search");
 
 			const newContent = await readFile(claudeMdPath, "utf-8");
-			expect(newContent).not.toContain("code-analysis");
+			expect(newContent).not.toContain("code-search");
 			expect(newContent).toContain("<!-- BEGIN magus:terminal v2.0.0 -->");
 			expect(newContent).toContain("Second section");
 		});
@@ -751,9 +751,9 @@ describe("conventions-manager", () => {
 		it("should list all managed sections", async () => {
 			const claudeMdPath = join(testDir, "CLAUDE.md");
 			const content = [
-				"<!-- BEGIN magus:code-analysis v1.0.0 -->",
+				"<!-- BEGIN magus:code-search v1.0.0 -->",
 				"First",
-				"<!-- END magus:code-analysis -->",
+				"<!-- END magus:code-search -->",
 				"",
 				"<!-- BEGIN magus:terminal v2.1.0 -->",
 				"Second",
@@ -764,7 +764,7 @@ describe("conventions-manager", () => {
 			const sections = await listClaudeMdSections(claudeMdPath);
 
 			expect(sections).toHaveLength(2);
-			expect(sections[0].section).toBe("code-analysis");
+			expect(sections[0].section).toBe("code-search");
 			expect(sections[0].version).toBe("1.0.0");
 			expect(sections[1].section).toBe("terminal");
 			expect(sections[1].version).toBe("2.1.0");

package/src/__tests__/enabled-not-installed.test.ts

@@ -25,7 +25,7 @@ const plugin = (over: Partial<PluginInfo>): PluginInfo =>
 
 describe("isEnabledButNotInstalled", () => {
 	test("enabled in project scope with no installed version is flagged", () => {
-		// The real case: terminal/code-analysis are in this project's enabledPlugins
+		// The real case: terminal/code-search are in this project's enabledPlugins
 		// but have no registry entry for it and none at user scope.
 		expect(
 			isEnabledButNotInstalled(

package/src/__tests__/mate-availability.test.ts

@@ -0,0 +1,156 @@
+/**
+ * Whether the mate slots are drawn, and what "installed" is allowed to mean.
+ *
+ * The question is asked of the plugin registry through `plugin-manager`, never of
+ * `installed_plugins.json` / `enabledPlugins` / `installedPluginVersions` directly — those
+ * are Claude Code's to own, and the last of them is maintained by nothing but claudeup, so it
+ * goes stale silently. What is tested here is the DECISION on top of that data, which is the
+ * part a real registry cannot exercise on demand.
+ */
+import { describe, expect, test } from "bun:test";
+import {
+	MATE_FORCE_ENV,
+	MATE_PLUGIN_ID,
+	areMatesAvailable,
+	matesForced,
+} from "../services/mate-availability.js";
+import type { PluginInfo } from "../services/plugin-manager.js";
+
+const plugin = (overrides: Partial<PluginInfo> = {}): PluginInfo => ({
+	id: MATE_PLUGIN_ID,
+	name: "multimodel",
+	version: "1.0.0",
+	description: "",
+	marketplace: "magus",
+	marketplaceDisplay: "Magus",
+	enabled: true,
+	...overrides,
+});
+
+const lister = (plugins: PluginInfo[]) => async () => plugins;
+
+describe("areMatesAvailable", () => {
+	test("false when the plugin is nowhere in the list", async () => {
+		expect(await areMatesAvailable(undefined, lister([]))).toBe(false);
+		expect(
+			await areMatesAvailable(
+				undefined,
+				lister([plugin({ id: "dev@magus", name: "dev" })]),
+			),
+		).toBe(false);
+	});
+
+	test("true when it is installed at ANY scope", async () => {
+		// Any scope, because a mate is served by whatever Claude Code has loaded when the
+		// agent spawns — the union of the three. A user-scope install serves a project that
+		// enables nothing of its own.
+		for (const scope of ["userScope", "projectScope", "localScope"] as const) {
+			const one = plugin({ [scope]: { enabled: true, version: "1.0.0" } });
+			expect({
+				scope,
+				available: await areMatesAvailable(undefined, lister([one])),
+			}).toEqual({ scope, available: true });
+		}
+	});
+
+	/**
+	 * The enabled-but-not-installed state is NOT installed.
+	 *
+	 * `enabledPlugins` is a flag only claudeup and the user maintain; the version comes from
+	 * the registry, which is what Claude Code actually loaded. Flag set with no version is the
+	 * broken state the plugin list already renders as "not installed" — and a mate row drawn
+	 * on the strength of it would promise routing through a plugin that never loaded.
+	 */
+	test("false when it is enabled but never actually installed", async () => {
+		const flagged = plugin({ userScope: { enabled: true } });
+		expect(await areMatesAvailable(undefined, lister([flagged]))).toBe(false);
+	});
+
+	test("false when it is installed but disabled", async () => {
+		const off = plugin({
+			userScope: { enabled: false, version: "1.0.0" },
+		});
+		expect(await areMatesAvailable(undefined, lister([off]))).toBe(false);
+	});
+
+	test('a version of "0.0.0" still counts — that is how some plugins record themselves', async () => {
+		const zero = plugin({ userScope: { enabled: true, version: "0.0.0" } });
+		expect(await areMatesAvailable(undefined, lister([zero]))).toBe(true);
+	});
+
+	/**
+	 * FAILS CLOSED, and that is the load-bearing half.
+	 *
+	 * This reaches marketplace resolution, which does network work with its own timeouts. A
+	 * failure means "we do not know", and the honest rendering of not knowing is the screen
+	 * exactly as it was before mates existed — not three rows put on screen by an error.
+	 */
+	test("false when the lookup throws", async () => {
+		const throwing = async () => {
+			throw new Error("marketplace unreachable");
+		};
+		expect(await areMatesAvailable(undefined, throwing)).toBe(false);
+	});
+});
+
+/**
+ * `CLAUDEUP_MATES=1` — draw the slots on a machine that cannot serve them.
+ *
+ * It exists because the three rows ARE the feature: without the plugin installed there is no
+ * way to put them on screen, which leaves `kangaroo`'s eight cells — the widest thing this
+ * layout ever has to fit — as the one part nobody can check by eye.
+ */
+describe("the mate-slot override", () => {
+	test("accepts the affirmative spellings, and nothing else", () => {
+		for (const on of ["1", "true", "on", "yes", "TRUE", " On "]) {
+			expect(matesForced({ [MATE_FORCE_ENV]: on })).toBe(true);
+		}
+		// "0" and "false" are not merely unrecognised, they are the common way to try to turn
+		// a flag OFF. Reading either as true would be the worst possible misparse.
+		for (const off of ["0", "false", "off", "no", "", "maybe"]) {
+			expect(matesForced({ [MATE_FORCE_ENV]: off })).toBe(false);
+		}
+		expect(matesForced({})).toBe(false);
+	});
+
+	/**
+	 * The flag SHORT-CIRCUITS, and a throwing lister is how that is proved.
+	 *
+	 * A lister that throws returns false through every other path in this file, so if the
+	 * answer here is true, the registry was genuinely never consulted — which is the point:
+	 * the network work behind it is the slow part, and its result would be discarded anyway.
+	 */
+	test("returns true without consulting the registry at all", async () => {
+		const throwing = async () => {
+			throw new Error("the registry must not be reached");
+		};
+		const original = process.env[MATE_FORCE_ENV];
+		process.env[MATE_FORCE_ENV] = "1";
+		try {
+			expect(await areMatesAvailable(undefined, throwing)).toBe(true);
+		} finally {
+			if (original === undefined) delete process.env[MATE_FORCE_ENV];
+			else process.env[MATE_FORCE_ENV] = original;
+		}
+	});
+
+	/**
+	 * Negative control for the test above.
+	 *
+	 * Same lister, same call, flag absent — false. Without this pair, a bug that made
+	 * `areMatesAvailable` return true unconditionally would pass the short-circuit test and
+	 * look like proof of the flag.
+	 */
+	test("the same call is false once the flag is gone", async () => {
+		const throwing = async () => {
+			throw new Error("the registry must not be reached");
+		};
+		const original = process.env[MATE_FORCE_ENV];
+		delete process.env[MATE_FORCE_ENV];
+		try {
+			expect(await areMatesAvailable(undefined, throwing)).toBe(false);
+		} finally {
+			if (original !== undefined) process.env[MATE_FORCE_ENV] = original;
+		}
+	});
+});