magus-cli 7.9.0 → 7.10.0

package/src/cli/update-view.ts

@@ -34,6 +34,10 @@ import {
 	type UpdatePlan,
 	isKeptDeprecated,
 } from "../services/update-plan.js";
+import type {
+	UnsetWorktree,
+	WorktreeSetupResult,
+} from "../services/worktree-profiles.js";
 import { type ReportBadgeKind, reportBadge } from "../ui/theme-mode.js";
 import { brand } from "../ui/theme.js";
 import { formatCommand } from "../utils/command-utils.js";
@@ -903,3 +907,72 @@ export function pendingHeadline(plan: UpdatePlan): string {
 		!plan.skills.some((s) => s.action === "install");
 	return onlyRemovals ? "Changes pending" : "Updates available";
 }
+
+// -- other worktrees -----------------------------------------------------------
+//
+// The other working trees of this repository that have no profile set up
+// (`findUnsetWorktrees`). `magus update` sets up the ones the project already
+// answered for, exactly as it does its own folder, and says so in one line.
+// No section and no question: there is nothing to decide. Read-only runs
+// (`magus doctor`, `--check`, `--dry-run`) get the same one line as a warning.
+
+/** `Worktree a` or `Worktrees a, b, c + 2 more`, so the row stays one line. */
+function worktreeLabel(names: readonly string[]): string {
+	const shown = names.slice(0, 3).join(", ");
+	const list = names.length > 3 ? `${shown} + ${names.length - 3} more` : shown;
+	return `${names.length === 1 ? "Worktree" : "Worktrees"} ${bold(list)}`;
+}
+
+/**
+ * The worktree rows: after a setup (`results`), one `✓` line for every
+ * worktree set up and a `✗` line for each failure; on a read-only run
+ * (`results` null), one `⚠` line for the ones `magus update` would set up.
+ * A worktree that needs a choice gets its own `⚠` line either way. Empty for
+ * an empty list: a repository with nothing to say prints nothing.
+ */
+export function worktreeLines(
+	worktrees: readonly UnsetWorktree[],
+	results: readonly WorktreeSetupResult[] | null,
+): string[] {
+	if (worktrees.length === 0) return [];
+	const lines = [""];
+	if (results === null) {
+		const auto = worktrees.filter((w) => w.autoActivate).map((w) => w.name);
+		if (auto.length > 0) {
+			const one = auto.length === 1;
+			lines.push(
+				warn(
+					`${worktreeLabel(auto)} ${one ? "is" : "are"} missing this project's plugins. magus update sets ${one ? "it" : "them"} up.`,
+				),
+			);
+		}
+	} else {
+		const done = results.filter((r) => r.ok).map((r) => r.name);
+		if (done.length > 0) {
+			lines.push(
+				ok(
+					`${worktreeLabel(done)} now ${done.length === 1 ? "has" : "have"} this project's plugins.`,
+				),
+			);
+		}
+		for (const r of results) {
+			if (!r.ok) {
+				lines.push(
+					fail(
+						`Worktree ${bold(r.name)} was not set up: ${r.reason.replace(/\.+$/, "")}.`,
+					),
+				);
+			}
+		}
+	}
+	for (const w of worktrees) {
+		if (!w.autoActivate) {
+			lines.push(
+				warn(
+					`Worktree ${bold(w.name)} is missing this project's plugins. Run magus update there to choose a profile.`,
+				),
+			);
+		}
+	}
+	return lines;
+}

package/src/cli/update.ts

@@ -84,6 +84,7 @@ import {
 	planSkillUpdates,
 	summarizePlan,
 } from "../services/update-plan.js";
+import type { ProfileManifest } from "../types/index.js";
 import { brand } from "../ui/theme.js";
 import { bold, dim, fg, width } from "./ansi.js";
 import { ensureManifest, requireActiveProfile } from "./bootstrap.js";
@@ -108,6 +109,7 @@ import {
 	unknownToolsNotice,
 	warn as warnLine,
 } from "./update-view.js";
+import { setUpOtherWorktrees } from "./worktrees.js";
 
 interface UpdateFlags {
 	/** Report drift and exit 1 if anything is behind. The CI gate. */
@@ -250,6 +252,25 @@ export async function runUpdateCommand(
 	// what it promised. A refused `--check` exits 1, because that is a gate.
 	if (!current) return flags.dryRun ? 0 : 1;
 
+	const code = await updateProfile(flags, projectPath, current, startedAt);
+
+	// This project is on a profile; now the other worktrees of the repository,
+	// set up the same way without asking. After the update, so their line is
+	// not lost above the plan. Read-only under `--check` and `--dry-run`, and
+	// never a change to the exit code: those are other projects.
+	await setUpOtherWorktrees(projectPath, {
+		mayWrite: !flags.check && !flags.dryRun,
+	});
+	return code;
+}
+
+/** The update itself, once the project's own profile gate has passed. */
+async function updateProfile(
+	flags: UpdateFlags,
+	projectPath: string,
+	current: ProfileManifest,
+	startedAt: number,
+): Promise<number> {
 	const profileIds = Object.keys(current.profiles);
 	if (flags.profile && !current.profiles[flags.profile]) {
 		console.error(

package/src/cli/worktrees.ts

@@ -0,0 +1,32 @@
+/**
+ * The other worktrees of this repository at the end of `magus update`: the ones
+ * with no profile set up. Each one whose project already declares a profile,
+ * with nothing of the user's to replace, is set up without asking, exactly as
+ * `magus update` sets up its own folder. Then one line says so.
+ *
+ * Detection and apply are `services/worktree-profiles.ts`, shared with
+ * `magus doctor` and the TUI; the words are `update-view.ts`.
+ */
+
+import {
+	findUnsetWorktrees,
+	setUpWorktrees,
+} from "../services/worktree-profiles.js";
+import { worktreeLines } from "./update-view.js";
+
+/**
+ * Set up what can be set up when `mayWrite`, and print the rows. Under
+ * `--check` and `--dry-run` it only reports. Never changes the command's exit
+ * code: these are other projects, and a failure in one is reported on its row.
+ */
+export async function setUpOtherWorktrees(
+	projectPath: string,
+	opts: { mayWrite: boolean },
+): Promise<void> {
+	const worktrees = await findUnsetWorktrees(projectPath);
+	if (worktrees.length === 0) return;
+	const results = opts.mayWrite
+		? await setUpWorktrees(worktrees.filter((w) => w.autoActivate))
+		: null;
+	for (const line of worktreeLines(worktrees, results)) console.log(line);
+}