claudeup 6.6.0 → 6.7.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claudeup",
-  "version": "6.6.0",
+  "version": "6.7.1",
   "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.7.1",
+    "claudeup-darwin-x64": "6.7.1",
+    "claudeup-linux-x64": "6.7.1"
   }
 }

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__/models-presets-marketplace.test.ts

@@ -0,0 +1,141 @@
+/**
+ * The agent tier table is keyed by agent id, and nothing else checks those ids.
+ *
+ * `DEFAULT_AGENT_GRADES` in `data/models-presets.ts` names
+ * agents as `plugin:name`. A rename in the marketplace (`code-search:analyze` →
+ * something else, `dev:synthesizer` → `dev:aggregator`) breaks the key with no signal: the
+ * renamed agent silently falls to the preset's fallback, and the stale key routes nothing.
+ * This test reads the marketplace source tree beside this repo and fails on either drift.
+ *
+ * Two directions:
+ *   (a) every preset key names an agent that exists (or is a Claude Code built-in);
+ *   (b) every marketplace agent is classified — in the tier table, in the frontmatter map,
+ *       a vision agent, or explicitly left to the fallback in FALLBACK_BY_DESIGN below.
+ *       Absence is a legitimate routing choice (fallback = `normal`), but it must be a
+ *       choice: a new agent fails here until someone decides.
+ */
+import { describe, expect, test } from "bun:test";
+import { existsSync, readFileSync, readdirSync } from "node:fs";
+import { join, resolve } from "node:path";
+import { DEFAULT_AGENT_GRADES, VISION_AGENTS } from "../data/models-presets.js";
+import { PREDEFINED_PROFILES } from "../data/predefined-profiles.js";
+
+const REPO = resolve(import.meta.dir, "..", "..", "..", "..");
+const MARKETPLACE = join(REPO, ".claude-plugin", "marketplace.json");
+const inMagusSrc = existsSync(MARKETPLACE) && existsSync(join(REPO, "plugins"));
+
+/** Claude Code's own agents: not in any marketplace, addressed by bare name. */
+const BUILT_IN = new Set([
+	"Explore",
+	"Plan",
+	"general-purpose",
+	"statusline-setup",
+	"claude-code-guide",
+]);
+
+/**
+ * Marketplace agents deliberately left to the preset's `fallback` grade. Each entry is a
+ * decision, not an omission: remove the line when the agent gets a tier. A NEW agent is not
+ * on this list, so adding one to the marketplace fails (b) until it is placed somewhere.
+ */
+const FALLBACK_BY_DESIGN = new Set([
+	"dev:developer",
+	"dev:devops",
+	"dev:frontend-developer",
+	"dev:researcher",
+	"dev:qa-engineer",
+	"madbench:bench",
+	"video-editing:timeline-builder",
+	"video-editing:transcriber",
+	"video-editing:video-processor",
+]);
+
+interface MarketplaceAgent {
+	id: string;
+	file: string;
+	frontmatterModel: string | undefined;
+}
+
+/** Every `plugins/<p>/agents/*.md` of every plugin the marketplace lists, by frontmatter name. */
+function marketplaceAgents(): MarketplaceAgent[] {
+	const manifest = JSON.parse(readFileSync(MARKETPLACE, "utf8")) as {
+		plugins: { name: string; source: string }[];
+	};
+	const out: MarketplaceAgent[] = [];
+	for (const plugin of manifest.plugins) {
+		const agentsDir = join(REPO, plugin.source, "agents");
+		if (!existsSync(agentsDir)) continue;
+		for (const entry of readdirSync(agentsDir)) {
+			if (!entry.endsWith(".md")) continue;
+			const file = join(agentsDir, entry);
+			const text = readFileSync(file, "utf8");
+			const fm = text.match(/^---\n([\s\S]*?)\n---/);
+			if (!fm) continue;
+			const name = fm[1].match(/^name:\s*(.+?)\s*$/m)?.[1];
+			if (!name) continue;
+			const model = fm[1].match(/^model:\s*(.+?)\s*$/m)?.[1];
+			out.push({
+				id: `${plugin.name}:${name}`,
+				file: join(plugin.source, "agents", entry),
+				frontmatterModel: model,
+			});
+		}
+	}
+	return out;
+}
+
+describe.skipIf(!inMagusSrc)(
+	"models-presets agent ids track the marketplace",
+	() => {
+		const agents = marketplaceAgents();
+		const ids = new Set(agents.map((a) => a.id));
+		const presetKeys = [...Object.keys(DEFAULT_AGENT_GRADES)];
+
+		test("reads a non-trivial marketplace", () => {
+			expect(agents.length).toBeGreaterThan(10);
+		});
+
+		test("(a) every preset key names an existing agent or a built-in", () => {
+			const dead = presetKeys.filter((k) => !BUILT_IN.has(k) && !ids.has(k));
+			expect(dead).toEqual([]);
+		});
+
+		test("(b) every marketplace agent is classified somewhere", () => {
+			const classified = new Set<string>([
+				...presetKeys,
+				...VISION_AGENTS,
+				...FALLBACK_BY_DESIGN,
+			]);
+			const unplaced = agents
+				.filter((a) => !classified.has(a.id))
+				.map((a) => a.id);
+			expect(unplaced).toEqual([]);
+		});
+
+		test("FALLBACK_BY_DESIGN and VISION_AGENTS name agents that still exist", () => {
+			const stale = [...FALLBACK_BY_DESIGN, ...VISION_AGENTS].filter(
+				(k) => !ids.has(k),
+			);
+			expect(stale).toEqual([]);
+		});
+
+		// A profile's magusPlugins is an install list, not documentation: resolver.ts expands
+		// each entry to `<name>@<marketplace>: latest`. A plugin rename moves under it silently,
+		// which is how 13.0.0's code-analysis -> code-search rename left six profiles installing
+		// a plugin the marketplace no longer lists. The agent checks above never read this array.
+		test("(d) every predefined profile names plugins the marketplace still lists", () => {
+			const marketplace = JSON.parse(readFileSync(MARKETPLACE, "utf8")) as {
+				plugins: { name: string }[];
+			};
+			const known = new Set(marketplace.plugins.map((p) => p.name));
+			const dead: string[] = [];
+			for (const profile of PREDEFINED_PROFILES) {
+				for (const entry of profile.magusPlugins ?? []) {
+					const name = entry.split("@")[0];
+					if (!known.has(name)) dead.push(`${profile.id}: ${entry}`);
+				}
+			}
+			expect(dead).toEqual([]);
+		});
+	},
+);

package/src/__tests__/plugin-manager-fallback.test.ts

@@ -83,7 +83,7 @@ describe("resolveMarketplacePlugins — offline fallback", () => {
 				"magus",
 				buildLocalMarketplace("magus", [
 					{
-						name: "code-analysis",
+						name: "code-search",
 						version: "5.3.0",
 						description: "Codebase investigation tools",
 						category: "development",
@@ -108,12 +108,10 @@ describe("resolveMarketplacePlugins — offline fallback", () => {
 
 		expect(resolved.plugins).toHaveLength(2);
 		expect(resolved.plugins.map((p) => p.name).sort()).toEqual([
-			"code-analysis",
+			"code-search",
 			"dev",
 		]);
-		const codeAnalysis = resolved.plugins.find(
-			(p) => p.name === "code-analysis",
-		);
+		const codeAnalysis = resolved.plugins.find((p) => p.name === "code-search");
 		expect(codeAnalysis?.version).toBe("5.3.0");
 		expect(codeAnalysis?.description).toBe("Codebase investigation tools");
 		expect(codeAnalysis?.category).toBe("development");

package/src/__tests__/resolver.test.ts

@@ -91,9 +91,9 @@ describe("resolveExtends", () => {
 			extends: "growth-marketer",
 		};
 		const merged = resolveExtends(entry);
-		// image-generate/video-editing ship on the magus-marketing channel and
+		// image/video-editing ship on the magus-marketing channel and
 		// pin it explicitly — they must NOT be rewritten to @magus.
-		expect(merged.plugins!["image-generate@magus-marketing"]).toBe("latest");
+		expect(merged.plugins!["image@magus-marketing"]).toBe("latest");
 		expect(merged.plugins!["video-editing@magus-marketing"]).toBe("latest");
 		expect(merged.plugins!["image-generate@magus"]).toBeUndefined();
 		// Bare names still default to @magus.
@@ -302,7 +302,7 @@ describe("marketplaces derived from plugin ids", () => {
 				name: "Mixed",
 				plugins: {
 					"dev@magus": "latest",
-					"image-generate@magus-marketing": "latest",
+					"image@magus-marketing": "latest",
 					"feature-dev@claude-plugins-official": "latest",
 				},
 			},

package/src/data/models-presets.ts

@@ -40,19 +40,18 @@ export const DEFAULT_AGENT_GRADES: Record<string, Grade> = {
 	// what is true; both were on `smart` because "writes an important document" got confused
 	// with "makes an expensive judgement".
 	"dev:spec-writer": "normal",
-	"dev:synthesizer": "normal",
+	"dev:aggregator": "normal",
 
 	// Reading a lot is not analysing.
 	//
-	// `detective` is read-only: it locates implementations, traces a path and reports
+	// `analyze` is read-only: it locates implementations, traces a path and reports
 	// file:line. That is high-volume retrieval with light inference — the same shape as
 	// `Explore`, with more structure in the output. It sat on `smart` because "investigates"
 	// sounds like thinking; what it actually does is look.
-	"code-analysis:detective": "cheap",
+	"code-search:analyze": "cheap",
 
-	// Search, transcription and bookkeeping: bounded, mechanical, high-volume.
+	// Search and bookkeeping: bounded, mechanical, high-volume.
 	Explore: "cheap",
-	"dev:scribe": "cheap",
 	"dev:stack-detector": "cheap",
 	"dev:docs": "cheap",
 	"terminal:tui-navigator": "cheap",
@@ -69,7 +68,7 @@ export const DEFAULT_AGENT_GRADES: Record<string, Grade> = {
  * answers to "what will this preset cost me", and the lump sum shows neither.
  *
  * Read out of `plugins/dev/commands/{dev,debug,investigate}.md` — agents only. Those files
- * also name skills (`dev:context-detection`, `code-analysis:investigate`) which are not
+ * also name skills (`dev:context-detection`, `code-search:investigate`) which are not
  * dispatched through the Agent tool and so are not routed at all.
  */
 export const WORKFLOWS: { name: string; agents: string[] }[] = [
@@ -79,7 +78,7 @@ export const WORKFLOWS: { name: string; agents: string[] }[] = [
 			"dev:stack-detector",
 			"dev:architect",
 			"dev:developer",
-			"dev:test-architect",
+			"dev:qa-engineer",
 			"dev:reviewer",
 		],
 	},
@@ -87,14 +86,14 @@ export const WORKFLOWS: { name: string; agents: string[] }[] = [
 		name: "debug",
 		agents: [
 			"dev:stack-detector",
-			"code-analysis:detective",
+			"code-search:analyze",
 			"dev:debugger",
 			"dev:developer",
 		],
 	},
 	{
 		name: "investigate",
-		agents: ["code-analysis:detective", "dev:researcher"],
+		agents: ["code-search:analyze", "dev:researcher"],
 	},
 ];
 
@@ -102,7 +101,7 @@ export const WORKFLOWS: { name: string; agents: string[] }[] = [
  * Agents whose model is a CAPABILITY choice, not a spend choice.
  *
  * The three tiers answer "how much model", and that is the wrong question for an agent that
- * reads images. `designer:design-review` compares a rendered screen against a reference and
+ * reads images. `designer:review` compares a rendered screen against a reference and
  * `designer:ui` audits a screenshot for usability: what they need is the model that SEES
  * best, which is not a point on a cheap-to-smart line and does not move when the preset
  * changes.
@@ -112,7 +111,7 @@ export const WORKFLOWS: { name: string; agents: string[] }[] = [
  * preset's `fallback`, which is a defensible default and an honest one; pinning them to a
  * model id guessed here is neither.
  */
-export const VISION_AGENTS = ["designer:design-review", "designer:ui"] as const;
+export const VISION_AGENTS = ["designer:review", "designer:ui"] as const;
 
 function preset(
 	name: string,

package/src/data/predefined-profiles.ts

@@ -52,7 +52,7 @@ export const PREDEFINED_PROFILES: PredefinedProfile[] = [
 		magusPlugins: [
 			"setup",
 			"multimodel",
-			"code-analysis",
+			"code-search",
 			"dev",
 			"browser-use",
 			"designer",
@@ -90,7 +90,7 @@ export const PREDEFINED_PROFILES: PredefinedProfile[] = [
 		icon: "★",
 		magusPlugins: [
 			"dev",
-			"code-analysis",
+			"code-search",
 			"terminal",
 			"setup",
 			"designer",
@@ -129,7 +129,7 @@ export const PREDEFINED_PROFILES: PredefinedProfile[] = [
 		name: "Backend Forge",
 		description: "API development, debugging, code quality, data workflows",
 		icon: "★",
-		magusPlugins: ["dev", "code-analysis", "terminal", "setup", "multimodel"],
+		magusPlugins: ["dev", "code-search", "terminal", "setup", "multimodel"],
 		anthropicPlugins: [
 			"feature-dev",
 			"code-review",
@@ -159,7 +159,7 @@ export const PREDEFINED_PROFILES: PredefinedProfile[] = [
 		icon: "★",
 		magusPlugins: [
 			"dev",
-			"code-analysis",
+			"code-search",
 			"terminal",
 			"setup",
 			"multimodel",
@@ -194,11 +194,11 @@ export const PREDEFINED_PROFILES: PredefinedProfile[] = [
 		icon: "★",
 		magusPlugins: [
 			"dev",
-			"code-analysis",
+			"code-search",
 			"terminal",
 			"setup",
 			"browser-use",
-			"image-generate@magus-marketing",
+			"image@magus-marketing",
 			"video-editing@magus-marketing",
 		],
 		anthropicPlugins: [
@@ -226,7 +226,7 @@ export const PREDEFINED_PROFILES: PredefinedProfile[] = [
 		description:
 			"Planning, code review, coordination, and broad repo visibility",
 		icon: "★",
-		magusPlugins: ["dev", "code-analysis", "terminal", "setup", "multimodel"],
+		magusPlugins: ["dev", "code-search", "terminal", "setup", "multimodel"],
 		anthropicPlugins: [
 			"code-review",
 			"pr-review-toolkit",

package/src/services/claude-settings.ts

@@ -347,7 +347,7 @@ async function overlayRegistryVersions(
 		// binary, and `claude plugin install` writes it at no scope — so only
 		// claudeup's own past writes can be in there, and they go stale silently.
 		// Measured: user settings claimed browser-use 1.1.2 / terminal 4.0.2 /
-		// code-analysis 5.1.0 while the registry said 1.4.0 / 4.1.4 / 5.3.1, and
+		// code-search 5.1.0 while the registry said 1.4.0 / 4.1.4 / 5.3.1, and
 		// none of the claimed versions had a cache directory. Keeping the stale
 		// value made every one of them read as a permanent pending update.
 		if (entries && entries.length > 0) delete merged[pluginId];

package/src/services/community-styles.ts

@@ -727,7 +727,7 @@ export function describeCommunityFailure(
 // ─── Staleness: advisory, two-phase, never automatic ─────────────────────────
 
 /**
- * 24 hours, matching image-generate's staleness precedent in the root CLAUDE.md.
+ * 24 hours, matching the image plugin's staleness precedent in the root CLAUDE.md.
  *
  * The TTL is not only a "when to re-check" rule — it is also a "how long may we
  * keep SAYING up-to-date" rule. A check from three days ago is evidence about