claudeup 6.2.0 → 6.3.0

package/src/cli/doctor.ts

@@ -8,16 +8,13 @@
  *   3. Convention compliance — .gitignore + CLAUDE.md blocks (ported engine).
  */
 
-import fs from "fs-extra";
 import path from "node:path";
-import { readManifest } from "../services/manifest.js";
-import { resolveAllProfiles } from "../services/resolver.js";
+import fs from "fs-extra";
 import { checkBinaries, missingBinaries } from "../services/doctor-bins.js";
-import {
-	activeProfile,
-	profileDir,
-} from "../services/symlink-manager.js";
 import { checkProject, fixDoctorIssues } from "../services/doctor.js";
+import { readManifest } from "../services/manifest.js";
+import { resolveAllProfiles } from "../services/resolver.js";
+import { activeProfile, profileDir } from "../services/symlink-manager.js";
 
 export async function runDoctorCommand(args: string[]): Promise<number> {
 	const fix = args.includes("--fix");
@@ -42,7 +39,9 @@ export async function runDoctorCommand(args: string[]): Promise<number> {
 					problems++;
 				}
 			} else {
-				console.log(`✓ All ${closure.bins.length} binary dependencies present.`);
+				console.log(
+					`✓ All ${closure.bins.length} binary dependencies present.`,
+				);
 			}
 		}
 	}
@@ -70,7 +69,9 @@ export async function runDoctorCommand(args: string[]): Promise<number> {
 			if (actionable.length > 0) {
 				console.log("\nConventions:");
 				for (const check of actionable) {
-					console.log(`  ${check.status === "fail" ? "✗" : "•"} ${check.message}`);
+					console.log(
+						`  ${check.status === "fail" ? "✗" : "•"} ${check.message}`,
+					);
 				}
 				const fixable = actionable.filter((c) => c.fixable).length;
 				const failures = actionable.filter((c) => c.status === "fail").length;

package/src/cli/install.ts

@@ -48,9 +48,9 @@ import type {
 	SkillInfo,
 } from "../types/index.js";
 import { formatCommand, resolveExecutable } from "../utils/command-utils.js";
+import { runCommand, runShellScript } from "../utils/run.js";
 import { ensureManifest } from "./bootstrap.js";
 import { confirm, promptValue } from "./prompt.js";
-import { runCommand, runShellScript } from "../utils/run.js";
 
 interface InstallFlags {
 	check: boolean;
@@ -292,8 +292,8 @@ export async function runInstallCommand(
 				);
 				for (const c of drift) console.error(`  ${c.kind}: ${c.detail}`);
 				console.error(
-					`\nRun \`claudeup profile sync\` to promote them into .claude/profiles.json,\n` +
-						`or re-run with --force to discard them.`,
+					"\nRun `claudeup profile sync` to promote them into .claude/profiles.json,\n" +
+						"or re-run with --force to discard them.",
 				);
 				return 1;
 			}

package/src/cli/profile.ts

@@ -66,7 +66,7 @@ export async function initProfile(
 			`${getManifestPath(projectPath)} already defines: ${Object.keys(existing.profiles).join(", ")}`,
 		);
 		console.error(
-			`Use \`claudeup profile sync\` to promote local edits into it.`,
+			"Use `claudeup profile sync` to promote local edits into it.",
 		);
 		return 1;
 	}
@@ -85,7 +85,7 @@ export async function listProfiles(projectPath: string): Promise<number> {
 	const active = await activeProfile(projectPath);
 	console.log("Profiles:");
 	for (const id of ids) {
-		const entry = manifest.profiles[id]!;
+		const entry = manifest.profiles[id];
 		const marker = id === active ? "●" : " ";
 		const materialized = (await fs.pathExists(profileDir(id, projectPath)))
 			? ""
@@ -105,14 +105,15 @@ export async function showProfile(
 		return 1;
 	}
 	const manifest = await readManifest(projectPath);
-	if (!manifest.profiles[name]) {
+	const entry = manifest.profiles[name];
+	if (!entry) {
 		console.error(
 			`Profile "${name}" not found. Available: ${Object.keys(manifest.profiles).join(", ")}`,
 		);
 		return 1;
 	}
 	const closure = await resolveProfile(manifest, name);
-	console.log(`Profile: ${name} (${manifest.profiles[name]!.name})`);
+	console.log(`Profile: ${name} (${entry.name})`);
 	for (const line of summarizeClosure(closure)) console.log(`  ${line}`);
 	return 0;
 }

package/src/cli/prompt.ts

@@ -30,4 +30,3 @@ export async function promptValue(question: string): Promise<string> {
 		rl.close();
 	}
 }
-

package/src/cli/router.ts

@@ -6,7 +6,9 @@
  * to the interactive TUI, preserving claudeup's historical behavior.
  */
 
+import type { ThemeMode } from "@opentui/core";
 import { checkForUpdates } from "../services/version-check.js";
+import { parseThemeWord } from "../ui/theme-resolve.js";
 import { runClaudeCommand } from "./claude.js";
 import { runDoctorCommand } from "./doctor.js";
 import { runInstallCommand } from "./install.js";
@@ -21,6 +23,58 @@ export interface RouteOutcome {
 	exitCode?: number;
 	/** No subcommand matched — the caller should launch the TUI. */
 	launchTui: boolean;
+	/** From --theme, validated. Only meaningful when launchTui is true. */
+	theme?: ThemeMode;
+}
+
+export type ThemeFlagParse =
+	| { ok: true; theme?: ThemeMode; rest: string[] }
+	| { ok: false; message: string };
+
+/**
+ * Pull `--theme light|dark` (or `--theme=…`) out of argv.
+ *
+ * Exact words only, the same rule every other theme source follows — so
+ * `--theme Dark` errors instead of silently differing from `TERM_THEME=Dark`.
+ * A flag is typed on purpose, so a bad value is an error rather than the "no
+ * opinion" an inherited variable gets. `--theme` as the last token, or followed
+ * by another `--flag`, is the same error.
+ */
+export function parseThemeFlag(argv: string[]): ThemeFlagParse {
+	const rest: string[] = [];
+	let theme: ThemeMode | undefined;
+
+	const invalid = (got: string | undefined): ThemeFlagParse => ({
+		ok: false,
+		message:
+			got === undefined
+				? 'claudeup: --theme must be "light" or "dark" (no value given)'
+				: `claudeup: --theme must be "light" or "dark" (got "${got}")`,
+	});
+
+	for (let i = 0; i < argv.length; i += 1) {
+		const token = argv[i] as string;
+		let value: string | undefined;
+
+		if (token === "--theme") {
+			const next = argv[i + 1];
+			if (next === undefined || next.startsWith("--"))
+				return invalid(undefined);
+			value = next;
+			i += 1;
+		} else if (token.startsWith("--theme=")) {
+			value = token.slice("--theme=".length);
+		} else {
+			rest.push(token);
+			continue;
+		}
+
+		const parsed = parseThemeWord(value);
+		if (parsed === null) return invalid(value);
+		theme = parsed;
+	}
+
+	return theme === undefined ? { ok: true, rest } : { ok: true, theme, rest };
 }
 
 const HANDLED = (exitCode: number): RouteOutcome => ({
@@ -50,7 +104,15 @@ export async function route(
 		return HANDLED(0);
 	}
 
-	const [command, ...rest] = argv;
+	const themeFlag = parseThemeFlag(argv);
+	if (!themeFlag.ok) {
+		console.error(themeFlag.message);
+		return HANDLED(1);
+	}
+
+	// Dispatch on argv with the theme tokens removed, so `--theme dark update`
+	// still runs `update`. Subcommands ignore the flag; only the TUI carries it.
+	const [command, ...rest] = themeFlag.rest;
 
 	switch (command) {
 		case "claude":
@@ -67,7 +129,9 @@ export async function route(
 			return HANDLED(await runDoctorCommand(rest));
 		default:
 			// Bare `claudeup` or an unrecognized token → interactive TUI.
-			return LAUNCH_TUI;
+			return themeFlag.theme === undefined
+				? LAUNCH_TUI
+				: { ...LAUNCH_TUI, theme: themeFlag.theme };
 	}
 }
 
@@ -83,6 +147,9 @@ Options:
   -v, --version  Show version and check for updates
   -h, --help     Show this help message
   --no-refresh   Skip auto-refresh of marketplaces on startup
+  --theme light|dark   Force the palette. Otherwise: CLAUDEUP_THEME, then
+                       TERM_THEME, then a 250 ms terminal query, then COLORFGBG,
+                       then dark. Only the exact words "light"/"dark" count.
 
 Dependency management — keep this project's plugins, CLI tools and skills
 installed and current. The manifest is .claude/profiles.json, committed; a repo

package/src/cli/update-view.ts

@@ -24,13 +24,13 @@
  * own word, so the report survives NO_COLOR, a pipe, and colour blindness.
  */
 
+import { binInstallCommand, binUpgradeCommand } from "../services/toolchain.js";
 import type {
 	BinUpdateItem,
 	PluginUpdateItem,
 	SkillUpdateItem,
 	UpdatePlan,
 } from "../services/update-plan.js";
-import { binInstallCommand, binUpgradeCommand } from "../services/toolchain.js";
 import { brand } from "../ui/theme.js";
 import { formatCommand } from "../utils/command-utils.js";
 import {
@@ -197,7 +197,9 @@ export function stepFrame(steps: Step[], tick: number, now: number): string[] {
 				? `${fg(brand.accent, `${step.progress.done}/${step.progress.total}`)} `
 				: "";
 		const detail =
-			step.state === "running" ? fg(brand.accent, step.detail) : dim(step.detail);
+			step.state === "running"
+				? fg(brand.accent, step.detail)
+				: dim(step.detail);
 		return `${PAD}${stepMark(step, tick)}  ${label} ${bar} ${time}  ${count}${detail}`;
 	});
 }
@@ -214,10 +216,10 @@ export function stepRecord(steps: Step[], now: number): string[] {
 	const times = steps.map((s) => stepDuration(s, now) ?? 0);
 	const slowest = Math.max(1, ...times);
 	return steps.map((step, i) => {
-		const share = (times[i]! / slowest) * 100;
+		const share = (times[i] / slowest) * 100;
 		const bar = meter(share, BAR, ramps.severity);
 		const label = padEnd(step.label, STEP_LABEL_WIDTH);
-		const time = dim(padStart(elapsed(times[i]!), 6));
+		const time = dim(padStart(elapsed(times[i]), 6));
 		return `${PAD}${stepMark(step, 0)}  ${label} ${bar} ${time}  ${dim(
 			step.detail,
 		)}`;
@@ -275,7 +277,8 @@ function section(
 ): string[] {
 	if (rows.length === 0) return [];
 	const counts = new Map<ActionKey, number>();
-	for (const row of rows) counts.set(row.action, (counts.get(row.action) ?? 0) + 1);
+	for (const row of rows)
+		counts.set(row.action, (counts.get(row.action) ?? 0) + 1);
 	const bar = stackedBar(
 		BAR,
 		[...counts].map(([action, value]) => ({
@@ -284,7 +287,9 @@ function section(
 		})),
 	);
 	const legend = [...counts]
-		.map(([action, n]) => fg(ACTIONS[action].color, `${n} ${ACTIONS[action].label.toLowerCase()}`))
+		.map(([action, n]) =>
+			fg(ACTIONS[action].color, `${n} ${ACTIONS[action].label.toLowerCase()}`),
+		)
 		.join(dim(" · "));
 
 	return [

package/src/cli/update.ts

@@ -108,7 +108,6 @@ export function parseArgs(args: string[]): UpdateFlags {
 	};
 }
 
-
 // -- phase runner ------------------------------------------------------------
 
 /**
@@ -254,7 +253,9 @@ export async function runUpdateCommand(
 				// Only now is there a denominator; until `onStart` fires the step
 				// shows a sweep rather than a meter that would have to guess one.
 				steps.marketplaces.progress = { done: 0, total: names.length };
-				say(names.length === 0 ? "none eligible" : `pulling ${names.join(", ")}`);
+				say(
+					names.length === 0 ? "none eligible" : `pulling ${names.join(", ")}`,
+				);
 			},
 			onSettled: (name, status) => {
 				settled++;
@@ -285,7 +286,9 @@ export async function runUpdateCommand(
 			),
 		);
 		notices.push(
-			noteLine(`claude plugin marketplace update ${name}  # to refresh it once`),
+			noteLine(
+				`claude plugin marketplace update ${name}  # to refresh it once`,
+			),
 		);
 	}
 	for (const name of refresh.failed) {
@@ -439,7 +442,9 @@ export async function runUpdateCommand(
 
 	if (!(await isClaudeAvailable())) {
 		console.error(
-			failLine("claude CLI not found on PATH — cannot install or update plugins."),
+			failLine(
+				"claude CLI not found on PATH — cannot install or update plugins.",
+			),
 		);
 		return 1;
 	}

package/src/cli/upgrade.ts

@@ -26,7 +26,8 @@ export async function runUpgradeCommand(): Promise<number> {
 			timeout: 5000,
 		}).trim();
 		const bunPath = `${bunGlobalBin}/claudeup`;
-		if (existsSync(bunPath)) installations.push({ manager: "bun", path: bunPath });
+		if (existsSync(bunPath))
+			installations.push({ manager: "bun", path: bunPath });
 	} catch {
 		// bun not installed or no global claudeup
 	}
@@ -38,7 +39,8 @@ export async function runUpgradeCommand(): Promise<number> {
 			timeout: 5000,
 		}).trim();
 		const npmPath = `${npmPrefix}/bin/claudeup`;
-		if (existsSync(npmPath)) installations.push({ manager: "npm", path: npmPath });
+		if (existsSync(npmPath))
+			installations.push({ manager: "npm", path: npmPath });
 	} catch {
 		// npm not installed or no global claudeup
 	}
@@ -50,12 +52,12 @@ export async function runUpgradeCommand(): Promise<number> {
 			timeout: 5000,
 		}).trim();
 
-		console.log(`⚠ claudeup is installed via multiple package managers:\n`);
+		console.log("⚠ claudeup is installed via multiple package managers:\n");
 		for (const inst of installations) {
 			const tag = inst.path === activePath ? " (active)" : " (shadowed)";
 			console.log(`  ${inst.manager}: ${inst.path}${tag}`);
 		}
-		console.log(`\nTo fix, keep one and remove the other:`);
+		console.log("\nTo fix, keep one and remove the other:");
 		for (const inst of installations) {
 			console.log(`  ${inst.manager} uninstall -g claudeup`);
 		}