@theholocron/cli 5.0.0-alpha.122 → 5.0.0-alpha.123

package/dist/cli.mjs

@@ -309,6 +309,8 @@ const style = {
 	warn: (msg) => `${chalk.yellow("⚠")} ${msg}`,
 	fail: (msg) => `${chalk.red("✗")} ${msg}`,
 	step: (msg) => `${chalk.cyan("→")} ${msg}`,
+	/** A dim `·` marker for a row that's neither a pass nor a failure — loaded but inactive, or skipped. */
+	skip: (msg) => chalk.dim(`· ${msg}`),
 	hint: (msg) => chalk.dim(msg),
 	dim: (msg) => chalk.dim(msg),
 	header: (msg) => chalk.bold(msg)
@@ -1117,6 +1119,27 @@ async function runCleanupPreview(input) {
 	}
 }
 //#endregion
+//#region src/ui/status-row.ts
+/** Right-pads `s` to `width` with spaces — never truncates. */
+function pad(s, width) {
+	return s.length >= width ? s : s + " ".repeat(width - s.length);
+}
+/**
+* One row of a status report: a 2-space indent, the status icon (✓ / ✗ / ·),
+* then `columns` joined with a two-space gap (blank columns dropped, so a
+* row that has nothing for its last column doesn't trail whitespace).
+* Column padding (`pad`) is the caller's job — `doctor` pads capability and
+* provider to a fixed width; `clone`'s repo names and paths don't, since
+* they vary too widely for fixed-width alignment to read cleanly.
+*
+* Shared by `doctor` (capability / provider / message) and `clone` (repo /
+* destination / reason) so every status table in the CLI reads the same way.
+*/
+function statusRow(status, columns) {
+	const joined = columns.filter((c) => c.length > 0).join("  ");
+	return `  ${status === "ok" ? style.success(joined) : status === "fail" ? style.fail(joined) : style.skip(joined)}`;
+}
+//#endregion
 //#region src/commands/clone.ts
 /**
 * Accepts a bare org (`"theholocron"`) — clones every repo in it — or an
@@ -1190,6 +1213,7 @@ async function runClone(input) {
 			stdio: "inherit"
 		}).status };
 	});
+	print("");
 	if (!existsSync(targetDir)) {
 		if (dryRun) print(style.dim(`  would create ${targetDir}`));
 		else mkdirSync(targetDir, { recursive: true });
@@ -1240,7 +1264,11 @@ async function runClone(input) {
 	for (const repo of repos) {
 		const dest = join(targetDir, repoDirName(repo));
 		if (existsSync(dest)) {
-			print(style.dim(`  skip   ${repo.full_name} → ${dest} (already exists)`));
+			print(statusRow("skip", [
+				repo.full_name,
+				dest,
+				"already exists"
+			]));
 			skipped++;
 			continue;
 		}
@@ -1252,7 +1280,11 @@ async function runClone(input) {
 		print(style.step(`  clone  ${repo.full_name} → ${dest}`));
 		const { clone_url } = repo;
 		if (!clone_url.startsWith("https://github.com/")) {
-			print(style.fail(`  failed ${repo.full_name} → unexpected clone URL: ${clone_url}`));
+			print(statusRow("fail", [
+				repo.full_name,
+				dest,
+				`unexpected clone URL: ${clone_url}`
+			]));
 			failed++;
 			continue;
 		}
@@ -1260,7 +1292,11 @@ async function runClone(input) {
 		try {
 			encodedToken = encodeTokenForGitHttpAuth(input.token);
 		} catch (err) {
-			print(style.fail(`  failed ${repo.full_name} → invalid token format: ${errorMessage(err)}`));
+			print(statusRow("fail", [
+				repo.full_name,
+				dest,
+				`invalid token format: ${errorMessage(err)}`
+			]));
 			failed++;
 			continue;
 		}
@@ -1270,14 +1306,14 @@ async function runClone(input) {
 			`https://x-access-token:${encodedToken}@github.com/${clone_url.slice(19)}`,
 			dest
 		], { cwd: targetDir }).status !== 0) {
-			print(style.fail(`  failed ${repo.full_name}`));
+			print(statusRow("fail", [repo.full_name, dest]));
 			failed++;
 		} else {
-			print(style.success(`  cloned ${repo.name} → ${dest}`));
+			print(statusRow("ok", [repo.full_name, dest]));
 			cloned++;
 		}
 	}
-	const summary = `${cloned} cloned, ${skipped} skipped, ${failed} failed`;
+	const summary = `${cloned} cloned, ${failed} failed, ${skipped} skipped`;
 	print("");
 	print(dryRun ? style.dim(`  dry-run: ${summary}`) : failed > 0 ? style.fail(`  ${summary}`) : style.success(`  ${summary}`));
 	const status = dryRun ? "dry-run" : failed > 0 ? "fail" : "ok";
@@ -1623,7 +1659,7 @@ async function runDoctor(input) {
 	assertPluginsResolvable(loader, "doctor");
 	const rows = [];
 	const config = input.loaded.resolved;
-	print(style.header(`Holocron doctor — ${config.name}`));
+	print("");
 	print(style.dim(`  config: ${input.loaded.filepath}`));
 	print("");
 	for (const failure of loader.loadFailures()) rows.push({
@@ -1658,10 +1694,7 @@ async function runDoctor(input) {
 			status: row.status,
 			detail: row.message
 		}, `doctor: ${row.capability}`);
-		const label = `${pad(row.capability, 14)} via ${pad(row.provider, 14)}  ${row.message}`;
-		if (row.status === "ok") print(`  ${style.success(label)}`);
-		else if (row.status === "fail") print(`  ${style.fail(label)}`);
-		else print(`  ${style.dim(`· ${label}`)}`);
+		print(statusRow(row.status, [`${pad(row.capability, 14)} via ${pad(row.provider, 14)}`, row.message]));
 	}
 	const summary = rows.reduce((acc, r) => {
 		acc[r.status] += 1;
@@ -1673,7 +1706,7 @@ async function runDoctor(input) {
 	});
 	const summaryLine = `${summary.ok} ok, ${summary.fail} fail, ${summary.skip} skipped`;
 	print("");
-	print(summary.fail > 0 ? style.fail(summaryLine) : style.success(summaryLine));
+	print(style.header(summaryLine));
 	return {
 		rows,
 		summary
@@ -1727,9 +1760,6 @@ function mk(capability, provider, status, message) {
 		message
 	};
 }
-function pad(s, width) {
-	return s.length >= width ? s : s + " ".repeat(width - s.length);
-}
 //#endregion
 //#region src/commands/new.ts
 /**