@theholocron/cli 4.19.0 → 5.0.0-alpha.4

package/dist/cli.mjs

@@ -2154,12 +2154,11 @@ function render$10(inputs) {
 		main: "./src/index.ts",
 		exports: { ".": "./src/index.ts" },
 		scripts: {
-			build: "tsdown",
-			lint: "eslint .",
-			typecheck: "tsc --noEmit",
-			test: "vitest run",
+			"delivery.build": "tsdown",
+			"sourceQuality.staticAnalysis": "holocron run sourceQuality.staticAnalysis --",
+			"verification.typeSafety": "holocron run verification.typeSafety --",
+			"verification.unitTests": "holocron run verification.unitTests --",
 			"test:watch": "vitest",
-			"test:coverage": "vitest run --coverage",
 			validate: "tsx scripts/validate.mjs"
 		},
 		peerDependencies: { "@theholocron/cli": "workspace:*" },
@@ -2879,7 +2878,7 @@ function runPluginCreate(input) {
 			execFn("pnpm", [
 				"--filter",
 				pkg,
-				"typecheck"
+				"verification.typeSafety"
 			], {
 				cwd,
 				stdio: "inherit"
@@ -2887,7 +2886,7 @@ function runPluginCreate(input) {
 			execFn("pnpm", [
 				"--filter",
 				pkg,
-				"lint"
+				"sourceQuality.staticAnalysis"
 			], {
 				cwd,
 				stdio: "inherit"
@@ -2895,7 +2894,7 @@ function runPluginCreate(input) {
 			execFn("pnpm", [
 				"--filter",
 				pkg,
-				"test"
+				"verification.unitTests"
 			], {
 				cwd,
 				stdio: "inherit"
@@ -3018,6 +3017,193 @@ function printNextSteps$1(print, inputs) {
 * "revoke this token at <url>" reminder — same pattern as `rando vc
 * setup` for the ephemeral GH admin PAT.
 */
+/**
+* `holocron publish --sync-trust <old-file> <new-file>` — bulk-migrates npm
+* Trusted Publisher (OIDC) config to a new workflow filename across every
+* public package. See issue #689 / `scripts/npm-trust-migrate.sh` (the
+* shell-script prototype this supersedes) for the full "why": npm ties
+* Trusted Publisher to a workflow FILENAME per package, so renaming the
+* release workflow breaks OIDC publish (`ENEEDAUTH`) until every package's
+* config is updated to match.
+*
+* Requires npm@11.15.0+ (`npm trust`) — always shells out through
+* `npx -y npm@latest` regardless of the locally-installed npm version.
+*
+* npm's `trust` mutations — and reads, once a short-lived OTP grant lapses —
+* require a fresh interactive browser 2FA approval (an `EOTP` error with an
+* `authUrl`). No token bypasses this (npm's own docs: Granular Access Tokens
+* with the bypass-2FA option are explicitly unsupported for trust commands).
+* This command can't drive that browser step itself, so on the first `EOTP`
+* it stops immediately, prints the `authUrl`, and returns — the operator
+* approves it and re-runs; already-migrated packages are skipped on the next
+* pass, so it's safe to resume from wherever it stopped.
+*/
+async function runSyncTrust(input) {
+	const print = input.print ?? ((line) => console.log(line));
+	const logger = input.logger ?? getLogger();
+	const cwd = input.cwd ?? process.cwd();
+	const repo = input.repo ?? "theholocron/holocron";
+	const dryRun = input.dryRun ?? false;
+	const exec = input.exec ?? defaultExec$1;
+	const packageNames = input.packages ?? discoverPublicPackages(cwd);
+	const npm = "npx";
+	const npmArgs = ["-y", "npm@latest"];
+	print(`Holocron publish --sync-trust${dryRun ? " (dry-run)" : ""}`);
+	print(`  ${input.oldFile} -> ${input.newFile}  (repo: ${repo})`);
+	print(`  ${packageNames.length} public packages`);
+	print("");
+	logger.info({
+		oldFile: input.oldFile,
+		newFile: input.newFile,
+		packages: packageNames.length
+	}, "sync-trust: start");
+	if (packageNames.length === 0) {
+		const message = "nothing to sync (no packages/* and root package.json is private or unnamed)";
+		print(`  ✗ ${message}`);
+		return {
+			status: "fail",
+			message,
+			packages: []
+		};
+	}
+	if (dryRun) {
+		for (const name of packageNames) print(`  … would sync ${name}`);
+		return {
+			status: "dry-run",
+			message: "dry-run — no npm calls made",
+			packages: []
+		};
+	}
+	const results = [];
+	for (const name of packageNames) {
+		print(`=== ${name} ===`);
+		const parsed = parseJson((await exec(npm, [
+			...npmArgs,
+			"trust",
+			"list",
+			name,
+			"--json"
+		], { cwd })).stdout);
+		if (isEotp(parsed)) {
+			const authUrl = parsed.error?.authUrl ?? "";
+			print(`  needs fresh 2FA — open and approve: ${authUrl}`);
+			print("  then re-run; already-migrated packages are skipped.");
+			results.push({
+				name,
+				status: "needs-auth",
+				message: authUrl
+			});
+			logger.warn({
+				package: name,
+				authUrl
+			}, "sync-trust: needs auth, stopping");
+			return {
+				status: "fail",
+				message: "npm 2FA grant needed — see printed authUrl",
+				packages: results
+			};
+		}
+		const entry = parsed;
+		if (entry.file === input.newFile) {
+			print(`  already on ${input.newFile}, skipping`);
+			results.push({
+				name,
+				status: "skipped"
+			});
+			continue;
+		}
+		if (entry.id) {
+			print(`  revoking existing trust (file=${entry.file ?? "?"}, id=${entry.id})`);
+			const revoke = await exec(npm, [
+				...npmArgs,
+				"trust",
+				"revoke",
+				name,
+				"--id",
+				entry.id,
+				"-y"
+			], { cwd });
+			if (revoke.exitCode !== 0 && isEotp(parseJson(revoke.stdout))) {
+				const authUrl = parseJson(revoke.stdout).error?.authUrl ?? "";
+				print(`  needs fresh 2FA — open and approve: ${authUrl}`);
+				results.push({
+					name,
+					status: "needs-auth",
+					message: authUrl
+				});
+				return {
+					status: "fail",
+					message: "npm 2FA grant needed — see printed authUrl",
+					packages: results
+				};
+			}
+		} else print("  no existing trust config found");
+		print(`  creating trust: ${input.newFile}`);
+		const create = await exec(npm, [
+			...npmArgs,
+			"trust",
+			"github",
+			name,
+			"--repo",
+			repo,
+			"--file",
+			input.newFile,
+			"--allow-publish",
+			"--allow-stage-publish",
+			"-y"
+		], { cwd });
+		if (create.exitCode === 0) {
+			print("  ✓ ok");
+			results.push({
+				name,
+				status: "ok"
+			});
+		} else {
+			const createParsed = parseJson(create.stdout);
+			if (isEotp(createParsed)) {
+				const authUrl = createParsed.error?.authUrl ?? "";
+				print(`  needs fresh 2FA — open and approve: ${authUrl}`);
+				results.push({
+					name,
+					status: "needs-auth",
+					message: authUrl
+				});
+				return {
+					status: "fail",
+					message: "npm 2FA grant needed — see printed authUrl",
+					packages: results
+				};
+			}
+			const message = create.stderr.trim() || create.stdout.trim() || `exit ${create.exitCode}`;
+			print(`  ✗ FAILED: ${message}`);
+			results.push({
+				name,
+				status: "fail",
+				message
+			});
+		}
+	}
+	const failed = results.filter((r) => r.status === "fail");
+	logger.info({
+		ok: results.filter((r) => r.status === "ok").length,
+		failed: failed.length
+	}, "sync-trust: done");
+	return {
+		status: failed.length > 0 ? "fail" : "ok",
+		...failed.length > 0 ? { message: `${failed.length} package(s) failed` } : {},
+		packages: results
+	};
+}
+function parseJson(raw) {
+	try {
+		return JSON.parse(raw);
+	} catch {
+		return;
+	}
+}
+function isEotp(parsed) {
+	return typeof parsed === "object" && parsed !== null && "error" in parsed && parsed.error?.code === "EOTP";
+}
 async function runPublish(input = {}) {
 	const print = input.print ?? ((line) => console.log(line));
 	const logger = input.logger ?? getLogger();
@@ -3180,7 +3366,7 @@ function printNextSteps(print, env, packageNames, repoName) {
 	print("");
 	print("  → next: configure Trusted Publisher for each package on npm:");
 	for (const name of packageNames) print(`    https://www.npmjs.com/package/${name}/access`);
-	print(`    Publisher: GitHub Actions   Org: theholocron   Repo: ${repoName}   Workflow: release.yml`);
+	print(`    Publisher: GitHub Actions   Org: theholocron   Repo: ${repoName}   Workflow: delivery.publish.yml`);
 	if (env.get("NPM_TOKEN")) {
 		print("");
 		print("  → cleanup: $NPM_TOKEN was used. Revoke it now (no API for self-revoke; UI-only):");
@@ -4518,6 +4704,10 @@ const { workflowHeader: workflowHeader$1 } = createHeader({
 	source: "packages/cli/src/commands/setup/run-setup.ts",
 	tool: "holocron setup"
 });
+const { workflowHeader: thinCallerHeader } = createHeader({
+	source: "packages/astromech/src/thin-callers.ts",
+	tool: "holocron setup"
+});
 async function runSetup(input) {
 	const print = input.print ?? ((line) => console.log(line));
 	const logger = getLogger();
@@ -4552,7 +4742,7 @@ async function runSetup(input) {
 			}, { skipCodes: SECURITY_SKIP_CODES[method] }));
 			print(formatStep(steps[steps.length - 1]));
 		}
-		const usesAdvancedCodeQL = (config.tasks ?? []).map((e) => typeof e === "string" ? e : e.name).includes("codeql");
+		const usesAdvancedCodeQL = (config.tasks ?? []).map((e) => typeof e === "string" ? e : e.name).includes("security.codeScanning");
 		steps.push(await runStep("source", usesAdvancedCodeQL ? "disableDefaultCodeScanning" : "enableCodeScanning", dryRun, async () => {
 			if (usesAdvancedCodeQL) await source.disableDefaultCodeScanning();
 			else return await source.enableCodeScanning();
@@ -4589,7 +4779,8 @@ async function runSetup(input) {
 		}).thinCallers();
 		for (const entry of tasks) {
 			const name = typeof entry === "string" ? entry : entry.name;
-			if (!KNOWN_WORKFLOWS.has(name)) {
+			const isCombinedContentOnly = name === "knowledge.docs" || name === "knowledge.components";
+			if (!KNOWN_WORKFLOWS.has(name) && !isCombinedContentOnly) {
 				steps.push({
 					capability: "source",
 					step: `write workflow ${name}`,
@@ -4599,12 +4790,12 @@ async function runSetup(input) {
 				print(formatStep(steps[steps.length - 1]));
 				continue;
 			}
-			const filename = name === "deploy" ? "deploy.yml" : `${name}.yml`;
+			const filename = `${name}.yml`;
 			const content = files.get(filename);
 			if (content === void 0) continue;
-			const step = name === "deploy" && content.includes("workflows/preview.yml@main") ? "write workflow deploy (with preview)" : `write workflow ${name}`;
+			const step = content.includes("workflows/preview.yml@main") ? `write workflow ${name} (with preview)` : `write workflow ${name}`;
 			steps.push(await runStep("source", step, dryRun, async () => {
-				await source.writeWorkflowFile(filename, `${workflowHeader$1()}${content}`);
+				await source.writeWorkflowFile(filename, `${thinCallerHeader()}${content}`);
 			}));
 			print(formatStep(steps[steps.length - 1]));
 		}
@@ -4699,7 +4890,7 @@ async function runSetup(input) {
 			print(formatStep(steps[steps.length - 1]));
 		}
 		{
-			const hasTestWorkflow = (config.tasks ?? []).map((e) => typeof e === "string" ? e : e.name).includes("test");
+			const hasTestWorkflow = (config.tasks ?? []).map((e) => typeof e === "string" ? e : e.name).includes("verification.unitTests");
 			const existing = await readFile(join(input.context.repoRoot, "codecov.yml"), "utf8").catch(() => null);
 			if (!hasTestWorkflow && existing == null) steps.push({
 				capability: "source",
@@ -4718,6 +4909,23 @@ async function runSetup(input) {
 			}));
 			print(formatStep(steps[steps.length - 1]));
 		}
+		{
+			const content = createAstromech({
+				cwd: input.context.repoRoot,
+				config: { tasks: config.tasks },
+				logger
+			}).turboConfig();
+			if (content === null) steps.push({
+				capability: "source",
+				step: "write turbo.json",
+				status: "skip",
+				message: "no fan-out-eligible tasks configured"
+			});
+			else steps.push(await runStep("source", "write turbo.json", dryRun, async () => {
+				await source.writeRepoFile("turbo.json", content);
+			}));
+			print(formatStep(steps[steps.length - 1]));
+		}
 		if (source.syncLabels) {
 			steps.push(await runStep("source", "sync labels", dryRun, async () => {
 				return source.syncLabels(CANONICAL_LABELS, STALE_LABELS);
@@ -4856,7 +5064,7 @@ async function runSetup(input) {
 	if (loader.has("deployment")) {
 		const deploy = loader.get("deployment");
 		print(style.step("deployment"));
-		const deployEntry = (config.tasks ?? []).map((e) => typeof e === "string" ? { name: e } : e).find((e) => e.name === "deploy");
+		const deployEntry = (config.tasks ?? []).map((e) => typeof e === "string" ? { name: e } : e).find((e) => e.name === "delivery.deploy" || e.name === "knowledge.docs" || e.name === "knowledge.components");
 		const previewCfg = deployEntry?.with ? extractPreviewConfig(deployEntry.with, {
 			org: config.org,
 			domain: config.domain
@@ -5580,7 +5788,7 @@ async function runSyncWiki(input) {
 //#endregion
 //#region src/commands/sync.ts
 const { workflowHeader } = createHeader({
-	source: "packages/cli/src/commands/sync.ts",
+	source: "packages/astromech/src/thin-callers.ts",
 	tool: "holocron sync"
 });
 const SYNC_STEPS = [
@@ -5594,6 +5802,7 @@ const SYNC_STEPS = [
 	"readme",
 	"workflows",
 	"scripts",
+	"turbo",
 	"wiki"
 ];
 const LOCAL_STEPS = /* @__PURE__ */ new Set([
@@ -5603,6 +5812,7 @@ const LOCAL_STEPS = /* @__PURE__ */ new Set([
 	"readme",
 	"workflows",
 	"scripts",
+	"turbo",
 	"wiki"
 ]);
 async function runSync(input) {
@@ -5751,6 +5961,7 @@ async function runSync(input) {
 		"readme",
 		"workflows",
 		"scripts",
+		"turbo",
 		"wiki"
 	]) {
 		if (requestedSteps !== void 0 && !requestedSteps.includes(stepName)) continue;
@@ -5855,7 +6066,8 @@ async function runSync(input) {
 				}).thinCallers();
 				for (const entry of taskEntries) {
 					const name = typeof entry === "string" ? entry : entry.name;
-					if (!KNOWN_WORKFLOWS.has(name)) {
+					const isCombinedContentOnly = name === "knowledge.docs" || name === "knowledge.components";
+					if (!KNOWN_WORKFLOWS.has(name) && !isCombinedContentOnly) {
 						steps.push({
 							capability: "local",
 							step: `sync workflow ${name}`,
@@ -5865,10 +6077,10 @@ async function runSync(input) {
 						print(formatSyncStep(steps[steps.length - 1]));
 						continue;
 					}
-					const filename = name === "deploy" ? "deploy.yml" : `${name}.yml`;
+					const filename = `${name}.yml`;
 					const content = files.get(filename);
 					if (content === void 0) continue;
-					const step = `sync workflow ${name}${name === "deploy" && content.includes("workflows/preview.yml@main") ? " (with preview)" : ""}`;
+					const step = `sync workflow ${name}${content.includes("workflows/preview.yml@main") ? " (with preview)" : ""}`;
 					steps.push(await runSyncStep("local", step, dryRun, async () => {
 						await writeWorkflowFile(input.context.repoRoot, filename, `${workflowHeader()}${content}`);
 					}));
@@ -5900,6 +6112,27 @@ async function runSync(input) {
 			}));
 			print(formatSyncStep(steps[steps.length - 1]));
 		}
+		if (stepName === "turbo") {
+			const content = createAstromech({
+				cwd: input.context.repoRoot,
+				config: { tasks: config.tasks },
+				logger
+			}).turboConfig();
+			if (content === null) {
+				steps.push({
+					capability: "local",
+					step: "sync turbo.json",
+					status: "skip",
+					message: "no fan-out-eligible tasks configured"
+				});
+				print(formatSyncStep(steps[steps.length - 1]));
+			} else {
+				steps.push(await runSyncStep("local", "sync turbo.json", dryRun, async () => {
+					await writeFile(join(input.context.repoRoot, "turbo.json"), content, "utf8");
+				}));
+				print(formatSyncStep(steps[steps.length - 1]));
+			}
+		}
 		if (stepName === "wiki") {
 			const result = await runSyncWiki({
 				loaded: input.loaded,
@@ -7751,9 +7984,31 @@ try {
 	}).option("otp", {
 		type: "string",
 		describe: "One-time password from your authenticator (required if npm needs 2FA for writes)"
+	}).option("sync-trust", {
+		type: "array",
+		describe: "Migrate npm Trusted Publisher config to a new workflow filename across every public package: --sync-trust <old-file> <new-file>. See issue #689 — npm requires fresh interactive 2FA per package; re-run after approving the printed authUrl if it stops early."
+	}).option("repo", {
+		type: "string",
+		describe: "owner/repo for Trusted Publisher config (--sync-trust). Defaults to theholocron/holocron."
 	}), async (argv) => {
+		if (argv.syncTrust) {
+			const [oldFile, newFile] = argv.syncTrust;
+			if (!oldFile || !newFile) {
+				getLogger().error("publish --sync-trust requires exactly two values: <old-file> <new-file>");
+				process.exitCode = 1;
+				return;
+			}
+			if ((await runSyncTrust({
+				cwd: argv.cwd,
+				oldFile,
+				newFile,
+				dryRun: argv.dryRun,
+				...argv.repo ? { repo: argv.repo } : {}
+			})).status === "fail") process.exitCode = 1;
+			return;
+		}
 		if (!argv.initial) {
-			getLogger().error("publish: only `--initial` is supported today. Run `holocron publish --initial`.");
+			getLogger().error("publish: only `--initial` and `--sync-trust` are supported today. Run `holocron publish --initial`.");
 			process.exitCode = 1;
 			return;
 		}