magus-cli 7.13.0 → 7.14.0

package/src/cli/market-view.ts

@@ -8,6 +8,7 @@
  */
 
 import type {
+	LocalCopy,
 	MarketPlan,
 	MarketResult,
 	SkillStatusRow,
@@ -89,23 +90,24 @@ export function stepLine(step: StepRecord, label: string): string {
 	return fail(`${label}: ${step.detail ?? "failed"}`);
 }
 
-function localCopyLine(result: MarketResult): string[] {
-	const copy = result.localCopy;
-	if (!copy) return [];
+function copyLine(copy: LocalCopy): string {
 	const whose = copy.scope === "user" ? "your USER" : "this project's";
 	if (copy.action === "removed") {
-		return [
-			ok(
-				`removed ${whose} copy of ${bold(copy.name)}${copy.scope === "project" ? ` — commit the deletion (.claude/skills/${copy.name})` : ""}`,
-			),
-		];
+		return ok(
+			`removed ${whose} copy of ${bold(copy.name)}${copy.scope === "project" ? ` — commit the deletion (.claude/skills/${copy.name})` : ""}`,
+		);
 	}
 	if (copy.action === "created") {
-		return [ok(`${whose} copy of ${bold(copy.name)} is ready to edit`)];
+		return ok(`${whose} copy of ${bold(copy.name)} is ready to edit`);
 	}
+	return info(`kept ${whose} copy of ${bold(copy.name)}: ${copy.reason ?? ""}`);
+}
+
+function localCopyLine(result: MarketResult): string[] {
 	return [
-		info(`kept ${whose} copy of ${bold(copy.name)}: ${copy.reason ?? ""}`),
-	];
+		...(result.localCopy ? [result.localCopy] : []),
+		...(result.localCopies ?? []),
+	].map(copyLine);
 }
 
 function installLines(result: MarketResult): string[] {
@@ -213,6 +215,22 @@ export function statusLines(result: MarketResult): string[] {
 export function listLines(result: MarketResult): string[] {
 	const lines = headerFor("market list");
 	const rows = result.marketplaces ?? [];
+	const local = result.local ?? [];
+	const localLines =
+		local.length === 0
+			? []
+			: [
+					`  ${bold("Here, in no marketplace")}`,
+					...local.map(
+						(l) => `  ${scopeBadge(l.scope)} ${bold(l.name)}  ${dim(l.kind)}`,
+					),
+					note(
+						rows.length === 0
+							? "a company marketplace is needed before these can be added to one"
+							: "add one with `magus market skill add <skill> --plugin <p@m> --create-plugin` or `magus market plugin add <path>`",
+					),
+					"",
+				];
 	if (rows.length === 0) {
 		lines.push(info(result.message));
 		lines.push(
@@ -220,10 +238,13 @@ export function listLines(result: MarketResult): string[] {
 				"create one with `magus market create`, or `magus market connect` an existing one",
 			),
 		);
-		return [...lines, ""];
+		return [...lines, "", ...localLines];
 	}
 	for (const m of rows) {
-		lines.push(`  ${bold(m.name)}  ${dim(m.repo)}  ${scopeBadge(m.scope)}`);
+		const by = m.company ? `  by ${m.company}` : "";
+		lines.push(
+			`  ${bold(m.name)}${by}  ${dim(m.repo)}  ${scopeBadge(m.scope)}`,
+		);
 		if (m.plugins.length === 0) lines.push(listItem(dim("no plugins yet")));
 		for (const p of m.plugins) {
 			lines.push(
@@ -235,7 +256,7 @@ export function listLines(result: MarketResult): string[] {
 		}
 		lines.push("");
 	}
-	return lines;
+	return [...lines, ...localLines];
 }
 
 /** `magus market --help`. */
@@ -244,23 +265,38 @@ export function marketHelp(): string {
 
 Usage: magus market <command> [args]
 
+The screen for all of this is \`magus edit\`; these commands run without one.
+
   list                      Company marketplaces on this machine, with their
                             plugins and skills. Changes nothing.
   fetch [<m>]               Bring magus's copies of company marketplaces up to
                             date with GitHub. Changes nothing else.
-  create <owner>/<repo> --private|--public [--name <m>] [--description <t>]
+  create <owner>/<repo> --private|--public [--name <m>] [--company <name>]
+         [--description <t>]
                             Create a GitHub repository holding a new company
                             marketplace and add it to this machine.
-  connect <owner/repo|git-url|path> [--name <m>]
+                            --company is the name magus shows it by ("by
+                            <name>"); the GitHub owner when left out. One line,
+                            at most 64 characters.
+  connect <owner/repo|git-url|path> [--name <m>] [--company <name>]
                             Use an existing repository as a company marketplace.
-                            --name only when the repository holds no marketplace.
+                            --name and --company only when the repository holds
+                            no marketplace; otherwise its manifest names both.
   remove <m>                Remove a company marketplace from this machine,
                             uninstalling its plugins. The repository stays.
   plugin create <p@m> [--description <t>]
   plugin delete <p@m>       Add or delete a plugin in the marketplace.
-  skill add <skill> --plugin <p@m> [--from project|global] [--keep-local]
+  plugin add <path> [--to <m>] [--name <p>]
+                            Copy a plugin folder of your own (one holding
+                            .claude-plugin/plugin.json) into the marketplace
+                            as a new plugin, at its own version (0.1.0 when it
+                            has none). The folder stays where it is.
+  skill add <skill> --plugin <p@m> [--create-plugin [--description <t>]]
+            [--from project|global] [--keep-local]
                             Add a skill from this project (or your USER skills)
                             to a plugin, then use the plugin's version.
+                            --create-plugin creates the plugin in the same
+                            commit when the marketplace has none of that name.
   skill edit <skill> [--plugin <p@m>] [--to project|global]
                             Copy a plugin's skill here to edit it.
   skill remove <skill> --plugin <p@m>
@@ -274,6 +310,11 @@ Usage: magus market <command> [args]
                             then remove the local copy. --force replaces the
                             plugin's version when the copy's origin is unknown
                             or both sides changed. --dry-run shows the plan.
+  publish --plugin <p@m> [--keep-local] [--dry-run]
+                            Send every skill of the plugin edited here as one
+                            new patch version. A copy whose origin is unknown,
+                            or that changed on both sides, is left out: publish
+                            it on its own with --force.
 
   @m may be left out when this machine has one company marketplace.
 

package/src/cli/market.ts

@@ -7,17 +7,18 @@
  * only that plan. Under `--json` one MarketResult is printed, on the last
  * line of stdout, and nothing is asked.
  *
- * Bare `magus market` is the interactive screen; this module answers "tui"
- * for it and the router decides.
+ * Bare `magus market` prints the help; the interactive screen is `magus
+ * edit`.
  */
 
 import {
+	planPluginAdd,
 	planPluginCreate,
 	planPluginDelete,
 } from "../services/market-ops-plugin.js";
 import {
-	type PublishArgs,
 	marketStatus,
+	planPluginPublish,
 	planPublish,
 	provisionalPublish,
 } from "../services/market-ops-publish.js";
@@ -61,6 +62,7 @@ import { askYesNo, promptValue } from "./prompt.js";
 /** Flags that take a value, and flags that stand alone. */
 const VALUE_FLAGS = new Set([
 	"--name",
+	"--company",
 	"--description",
 	"--plugin",
 	"--from",
@@ -70,6 +72,7 @@ const BOOL_FLAGS = new Set([
 	"--private",
 	"--public",
 	"--keep-local",
+	"--create-plugin",
 	"--force",
 	"--dry-run",
 	"--json",
@@ -276,15 +279,15 @@ function warnBlock(text: string): string {
 
 /**
  * `publish --dry-run`: the plan, or the answer a publish would give, and
- * nothing written. On state older than GitHub's, a provisional plan with no
- * compare, still exit 0.
+ * nothing written. On state older than GitHub's, a provisional answer with
+ * no compare (`provisional`), still exit 0; without one, that error.
  */
 async function dryRunPublish(
-	args: PublishArgs,
+	plan: () => Promise<PlanAnswer>,
 	io: Io,
-	cwd: string,
+	provisional?: (error: MarketError) => MarketResult,
 ): Promise<number> {
-	const answer = await planPublish(args, cwd);
+	const answer = await plan();
 	if (answer.kind === "plan") {
 		const p = answer.plan;
 		const result: MarketResult = {
@@ -307,8 +310,11 @@ async function dryRunPublish(
 		emit(answer.result, io);
 		return exitCodeFor(answer.result);
 	}
-	if (answer.error.code === "stale" || answer.error.code === "no-network") {
-		emit(provisionalPublish(args, cwd, answer.error), io);
+	if (
+		provisional &&
+		(answer.error.code === "stale" || answer.error.code === "no-network")
+	) {
+		emit(provisional(answer.error), io);
 		return 0;
 	}
 	const result = errorResult(answer.command, answer.error);
@@ -316,15 +322,12 @@ async function dryRunPublish(
 	return answer.error.exit ?? exitCodeFor(result);
 }
 
-/**
- * Run `magus market <argv>`. Returns the exit code, or "tui" for a bare
- * `magus market`.
- */
+/** Run `magus market <argv>`. Returns the exit code. */
 export async function runMarketCommand(
 	argv: string[],
 	version: string,
 	cwd: string = process.cwd(),
-): Promise<number | "tui"> {
+): Promise<number> {
 	if (argv.includes("--help") || argv.includes("-h")) {
 		console.log(marketHelp());
 		return 0;
@@ -333,7 +336,11 @@ export async function runMarketCommand(
 		console.log(`magus v${version}`);
 		return 0;
 	}
-	if (argv.length === 0) return "tui";
+	if (argv.length === 0) {
+		console.log(marketHelp());
+		console.log("Open the screen with `magus edit`.");
+		return 0;
+	}
 
 	const json = argv.includes("--json");
 	const io: Io = {
@@ -348,7 +355,7 @@ export async function runMarketCommand(
 		switch (verb) {
 			case "list": {
 				expect(p, [], { min: 0, max: 0, usage: "magus market list [--json]" });
-				const result = await marketList();
+				const result = await marketList(cwd);
 				emit(result, io);
 				return exitCodeFor(result);
 			}
@@ -367,12 +374,16 @@ export async function runMarketCommand(
 			}
 			case "create": {
 				const usage =
-					"magus market create <owner>/<repo> --private|--public [--name <m>] [--description <t>]";
-				expect(p, ["--private", "--public", "--name", "--description"], {
-					min: 1,
-					max: 1,
-					usage,
-				});
+					"magus market create <owner>/<repo> --private|--public [--name <m>] [--company <name>] [--description <t>]";
+				expect(
+					p,
+					["--private", "--public", "--name", "--company", "--description"],
+					{
+						min: 1,
+						max: 1,
+						usage,
+					},
+				);
 				const priv = p.flags.has("--private");
 				const pub = p.flags.has("--public");
 				if (priv === pub) {
@@ -386,6 +397,7 @@ export async function runMarketCommand(
 							repo: p.positional[0] as string,
 							visibility: priv ? "private" : "public",
 							name: str(p, "--name"),
+							company: str(p, "--company"),
 							description: str(p, "--description"),
 						}),
 					io,
@@ -394,15 +406,20 @@ export async function runMarketCommand(
 				);
 			}
 			case "connect": {
-				expect(p, ["--name"], {
+				expect(p, ["--name", "--company"], {
 					min: 1,
 					max: 1,
-					usage: "magus market connect <owner/repo|git-url|path> [--name <m>]",
+					usage:
+						"magus market connect <owner/repo|git-url|path> [--name <m>] [--company <name>]",
 				});
 				return await runWrite(
 					() =>
 						planConnect(
-							{ source: p.positional[0] as string, name: str(p, "--name") },
+							{
+								source: p.positional[0] as string,
+								name: str(p, "--name"),
+								company: str(p, "--company"),
+							},
 							cwd,
 						),
 					io,
@@ -435,6 +452,27 @@ export async function runMarketCommand(
 						cwd,
 					);
 				}
+				if (sub === "add") {
+					expect(sp, ["--to", "--name"], {
+						min: 1,
+						max: 1,
+						usage: "magus market plugin add <path> [--to <m>] [--name <p>]",
+					});
+					return await runWrite(
+						() =>
+							planPluginAdd(
+								{
+									path: ref as string,
+									to: str(p, "--to"),
+									name: str(p, "--name"),
+								},
+								cwd,
+							),
+						io,
+						yes,
+						cwd,
+					);
+				}
 				if (sub === "delete") {
 					expect(sp, [], {
 						min: 1,
@@ -448,22 +486,40 @@ export async function runMarketCommand(
 						cwd,
 					);
 				}
-				throw new UsageError("usage: magus market plugin create|delete <p@m>");
+				throw new UsageError(
+					"usage: magus market plugin create|delete <p@m>, or magus market plugin add <path>",
+				);
 			}
 			case "skill": {
 				const [sub, skill, ...extra] = p.positional;
 				const sp = { ...p, positional: skill ? [skill, ...extra] : [] };
 				if (sub === "add") {
 					const usage =
-						"magus market skill add <skill> --plugin <p@m> [--from project|global] [--keep-local]";
-					expect(sp, ["--plugin", "--from", "--keep-local"], {
-						min: 1,
-						max: 1,
-						usage,
-					});
+						"magus market skill add <skill> --plugin <p@m> [--create-plugin [--description <t>]] [--from project|global] [--keep-local]";
+					expect(
+						sp,
+						[
+							"--plugin",
+							"--from",
+							"--keep-local",
+							"--create-plugin",
+							"--description",
+						],
+						{
+							min: 1,
+							max: 1,
+							usage,
+						},
+					);
 					const plugin = str(p, "--plugin");
 					if (!plugin)
 						throw new UsageError(`--plugin is required — usage: ${usage}`);
+					const createPlugin = p.flags.has("--create-plugin");
+					if (p.flags.has("--description") && !createPlugin) {
+						throw new UsageError(
+							`--description describes a plugin --create-plugin creates — usage: ${usage}`,
+						);
+					}
 					const from = choice(
 						p,
 						"--from",
@@ -478,6 +534,12 @@ export async function runMarketCommand(
 									plugin,
 									from,
 									keepLocal: p.flags.has("--keep-local"),
+									...(createPlugin
+										? {
+												createPlugin: true,
+												description: str(p, "--description"),
+											}
+										: {}),
 								},
 								cwd,
 							),
@@ -538,25 +600,47 @@ export async function runMarketCommand(
 				return exitCodeFor(result);
 			}
 			case "publish": {
+				const usage =
+					"magus market publish <skill> [--plugin <p@m>] [--from project|global] [--keep-local] [--force] [--dry-run], or magus market publish --plugin <p@m> [--keep-local] [--dry-run]";
 				expect(
 					p,
 					["--plugin", "--from", "--keep-local", "--force", "--dry-run"],
-					{
-						min: 1,
-						max: 1,
-						usage:
-							"magus market publish <skill> [--plugin <p@m>] [--from project|global] [--keep-local] [--force] [--dry-run]",
-					},
+					{ min: 0, max: 1, usage },
 				);
-				const args: PublishArgs = {
-					skill: p.positional[0] as string,
+				const skill = p.positional[0];
+				const keepLocal = p.flags.has("--keep-local");
+				if (skill === undefined) {
+					const plugin = str(p, "--plugin");
+					if (!plugin) {
+						throw new UsageError(
+							`name a skill, or a plugin with --plugin — usage: ${usage}`,
+						);
+					}
+					for (const flag of ["--from", "--force"]) {
+						if (p.flags.has(flag)) {
+							throw new UsageError(
+								`${flag} applies to one named skill: \`magus market publish <skill> ${flag} …\``,
+							);
+						}
+					}
+					const plan = () => planPluginPublish({ plugin, keepLocal }, cwd);
+					if (p.flags.has("--dry-run")) return await dryRunPublish(plan, io);
+					return await runWrite(plan, io, yes, cwd);
+				}
+				const args = {
+					skill,
 					plugin: str(p, "--plugin"),
 					from: choice(p, "--from", ["project", "global"] as const, "project"),
-					keepLocal: p.flags.has("--keep-local"),
+					keepLocal,
 					force: p.flags.has("--force"),
 				};
-				if (p.flags.has("--dry-run")) return await dryRunPublish(args, io, cwd);
-				return await runWrite(() => planPublish(args, cwd), io, yes, cwd);
+				const plan = () => planPublish(args, cwd);
+				if (p.flags.has("--dry-run")) {
+					return await dryRunPublish(plan, io, (error) =>
+						provisionalPublish(args, cwd, error),
+					);
+				}
+				return await runWrite(plan, io, yes, cwd);
 			}
 			default:
 				throw new UsageError(

package/src/cli/router.ts

@@ -30,8 +30,8 @@ export interface RouteOutcome {
 	theme?: ThemeMode;
 	/**
 	 * Which interactive app to launch when `launchTui` is true: the dependency
-	 * TUI (bare `magus`, the default) or the company marketplace screens (bare
-	 * `magus market`).
+	 * TUI (bare `magus`, the default) or the company marketplace screens
+	 * (`magus edit`).
 	 */
 	app?: "main" | "market";
 }
@@ -93,6 +93,21 @@ const HANDLED = (exitCode: number): RouteOutcome => ({
 });
 const LAUNCH_TUI: RouteOutcome = { handled: false, launchTui: true };
 
+/** `magus edit --help`. */
+function editHelp(): string {
+	return `magus edit — open the company marketplace screens
+
+Usage: magus edit
+
+Lists the company marketplaces on this machine, their plugins and skills, and
+creates, connects and changes them: publishes edited skills (one skill, or a
+whole plugin as one new version), and adds this project's and your USER skills
+and plugin folders to a marketplace. Needs a terminal.
+
+\`magus market --help\` lists the commands that do the same without one.
+`;
+}
+
 /**
  * Every name the switch below handles.
  *
@@ -108,6 +123,7 @@ const SUBCOMMANDS = new Set([
 	"models",
 	"doctor",
 	"market",
+	"edit",
 ]);
 
 /**
@@ -130,10 +146,10 @@ export async function route(
 	themeEnv?: ThemeEnv,
 ): Promise<RouteOutcome> {
 	// Top-level flags take precedence over any command token — for every
-	// command but `market`, whose own verbs answer --help and --version (and
-	// write nothing doing so). `magus upgrade --help` must still print help
-	// rather than upgrade, so this exception is exactly argv[0] === "market".
-	const marketFirst = argv[0] === "market";
+	// command but `market` and `edit`, which answer --help and --version
+	// themselves (and write nothing doing so). `magus upgrade --help` must still
+	// print help rather than upgrade, so this exception is exactly argv[0].
+	const marketFirst = argv[0] === "market" || argv[0] === "edit";
 	if (!marketFirst && (argv.includes("--version") || argv.includes("-v"))) {
 		console.log(`magus v${version}`);
 		const result = await checkForUpdates();
@@ -229,8 +245,23 @@ export async function route(
 				return HANDLED(await runDoctorCommand(rest));
 			case "market": {
 				const { runMarketCommand } = await import("./market.js");
-				const outcome = await runMarketCommand(rest, version);
-				if (outcome !== "tui") return HANDLED(outcome);
+				return HANDLED(await runMarketCommand(rest, version));
+			}
+			case "edit": {
+				if (rest.includes("--help") || rest.includes("-h")) {
+					console.log(editHelp());
+					return HANDLED(0);
+				}
+				if (rest.includes("--version") || rest.includes("-v")) {
+					console.log(`magus v${version}`);
+					return HANDLED(0);
+				}
+				if (rest.length > 0) {
+					console.error(
+						"magus edit: takes no arguments — the commands are `magus market <verb>` (`magus market --help` lists them)",
+					);
+					return HANDLED(2);
+				}
 				return {
 					handled: false,
 					launchTui: true,
@@ -327,14 +358,15 @@ applied by a PreToolUse hook magus registers at user scope.
 
 Company marketplaces — your team's own Claude Code plugin marketplace on
 GitHub, listed first in magus. Every change prints its plan and asks first.
-  market             Open the company marketplace screens
+  edit               Open the company marketplace screens
   market list        Company marketplaces on this machine, with their plugins
   market create      Create one on GitHub;  market connect  use an existing one
   market remove <m>  Remove one from this machine (the repository stays)
-  market plugin create|delete <p@m>
+  market plugin create|delete <p@m>;  market plugin add <path>
   market skill add|edit|remove <skill>
   market status      How your local copies of company skills stand
   market publish <skill>  Send a local skill's edits back to its plugin
+  market publish --plugin <p@m>  Every edited skill of a plugin, one version
   market --help      Every market command, its flags and exit codes
 
 Other commands:

package/src/data/marketplaces.ts

@@ -250,6 +250,7 @@ export function getAllMarketplaces(
 				deprecated: defaultMp?.deprecated,
 				owned: defaultMp?.owned,
 				...(local.company ? { company: true } : {}),
+				...(local.ownerName ? { ownerName: local.ownerName } : {}),
 			});
 		}
 	}

package/src/main.tsx

@@ -97,7 +97,7 @@ async function main(): Promise<void> {
 	if (outcome.app === "market") {
 		if (!isInteractive) {
 			console.error(
-				"magus market opens an interactive screen, which needs a terminal. `magus market --help` lists the commands that run without one.",
+				"magus edit opens an interactive screen, which needs a terminal. `magus market --help` lists the commands that run without one.",
 			);
 			process.exit(2);
 		}

package/src/market-tui.tsx

@@ -1,5 +1,5 @@
 /**
- * The interactive screens behind bare `magus market`: the company
+ * The interactive screens behind `magus edit`: the company
  * marketplaces on this machine, and the plugins and skills of one of them.
  *
  * Imported dynamically from `main.tsx`, like `tui.tsx`, so no other command

package/src/services/local-marketplace.ts

@@ -55,6 +55,8 @@ export interface LocalMarketplace {
 	 * and the name and source are on this machine's trust list.
 	 */
 	company?: boolean;
+	/** `owner.name` from the manifest, trimmed; absent when empty or missing. */
+	ownerName?: string;
 }
 
 /** Claude Code's marketplace clones, resolved per call. */
@@ -322,6 +324,11 @@ async function scanSingleMarketplace(
 				? manifest.metadata.pluginRoot
 				: undefined;
 
+		const ownerName: string | undefined =
+			typeof manifest.owner?.name === "string" && manifest.owner.name.trim()
+				? manifest.owner.name.trim()
+				: undefined;
+
 		const plugins: LocalMarketplacePlugin[] = [];
 
 		if (manifest.plugins && Array.isArray(manifest.plugins)) {
@@ -398,6 +405,7 @@ async function scanSingleMarketplace(
 			company:
 				readMarker(manifest) &&
 				isTrustedIn(trusted, marketplaceName, remoteSource),
+			...(ownerName ? { ownerName } : {}),
 		};
 	} catch {
 		return null;

package/src/services/market-change.ts

@@ -140,8 +140,8 @@ export interface ChangeSpec {
 	/** Names a pull-request branch. */
 	slug: string;
 	prBody: string;
-	/** A skill directory whose committed files must equal these. */
-	verify?: { dir: string; files: readonly SkillFile[] };
+	/** Directories whose committed files must equal these, each. */
+	verify?: ReadonlyArray<{ dir: string; files: readonly SkillFile[] }>;
 	/** Set when the commit is made, for later steps. */
 	state: { commit?: string; pushed?: boolean };
 }
@@ -186,15 +186,15 @@ export function changeSteps(spec: ChangeSpec): MarketStep[] {
 				}
 				const made = await gitCommit(dir, spec.message);
 				if (!made.ok) throw gitError(made, spec.label);
-				if (spec.verify) {
-					const tree = await lsTree(dir, made.value, spec.verify.dir);
+				for (const check of spec.verify ?? []) {
+					const tree = await lsTree(dir, made.value, check.dir);
 					if (!tree.ok) throw gitError(tree, spec.label);
-					const prefix = `${spec.verify.dir}/`;
+					const prefix = `${check.dir}/`;
 					const committed = tree.value.map((e) => ({
 						path: e.path.slice(prefix.length),
 						oid: e.oid,
 					}));
-					if (!sameFiles(committed, spec.verify.files)) {
+					if (!sameFiles(committed, check.files)) {
 						throw new MarketError(
 							"other",
 							"the commit did not hold exactly the planned files — nothing was pushed",