@theholocron/cli 4.10.2 → 4.10.3

package/README.md

@@ -306,9 +306,11 @@ all three. See the [telemetry guide](https://docs.theholocron.dev/holocron/telem
   capability config packages, builds the capability registry
 - `src/cli.ts` — yargs entry, dispatches subcommands. `holocron run` / `ci` are
   delegated to [`@theholocron/astromech`](../astromech) (`createAstromech`).
-  `holocron sync-github` pushes the reusable `workflow_call` implementations +
-  composite actions from `astromech.reusableTemplates()` to `theholocron/.github`
-  (a pure sync target — never hand-edit its `.github/workflows/*`).
+  `holocron sync` / `holocron setup` write each repo's `.github/workflows/*.yml`
+  thin callers from `astromech.thinCallers()`; `holocron sync-github` pushes the
+  reusable `workflow_call` implementations + composite actions from
+  `astromech.reusableTemplates()` to `theholocron/.github` (a pure sync target —
+  never hand-edit its `.github/workflows/*`).
 - `src/commands/` — `setup`, `sync`, `doctor`, `deploy`, `secret set`,
   `secrets sync`, `npm publish-initial`, `sync-github`, `upgrade node`,
   `plugin create`, `auth`

package/dist/cli.mjs

@@ -3,7 +3,7 @@ import { createRequire } from "node:module";
 import { existsSync, mkdirSync, readFileSync, readdirSync, statSync, writeFileSync } from "node:fs";
 import path, { basename, dirname, join, relative, resolve } from "node:path";
 import { checkbox, input, select } from "@inquirer/prompts";
-import { KNOWN_WORKFLOWS, createAstromech, deriveDeployPaths, extractPreviewConfig, generateCombinedDeployContent, generateThinCallerContent, lintThinCallerWith, normalizeWorkflowWith, reusableTemplates } from "@theholocron/astromech";
+import { KNOWN_WORKFLOWS, createAstromech, extractPreviewConfig, reusableTemplates } from "@theholocron/astromech";
 import { loadTasksConfig } from "@theholocron/astromech/config";
 import yargs from "yargs";
 import { hideBin } from "yargs/helpers";
@@ -4385,31 +4385,16 @@ async function runSetup(input) {
 	if (loader.has("source") && tasks && tasks.length > 0) {
 		const source = loader.get("source");
 		print(style.step("workflows"));
+		const files = createAstromech({
+			cwd: input.context.repoRoot,
+			config: { tasks },
+			orgContext: {
+				org: config.org,
+				domain: config.domain
+			}
+		}).thinCallers();
 		for (const entry of tasks) {
 			const name = typeof entry === "string" ? entry : entry.name;
-			const rawWith = typeof entry === "object" ? entry.with : void 0;
-			const normalized = rawWith ? normalizeWorkflowWith(rawWith) : void 0;
-			let withOverrides = normalized;
-			let comments;
-			if (name === "lint") {
-				let rootFiles = [];
-				try {
-					rootFiles = readdirSync(input.context.repoRoot);
-				} catch {}
-				const lint = lintThinCallerWith({
-					explicit: typeof entry === "object" ? entry.linters : void 0,
-					rootFiles,
-					extra: normalized
-				});
-				withOverrides = lint.withOverrides;
-				comments = lint.comments;
-			}
-			const additionalPaths = (typeof entry === "object" ? entry.paths : void 0) ?? (name === "deploy" && rawWith ? deriveDeployPaths(rawWith) : void 0);
-			if (name === "test" && withOverrides) {
-				const runUnit = withOverrides["run-unit"];
-				const runStorybook = withOverrides["run-storybook"];
-				if (runUnit === false && runStorybook === false) throw new ConfigError("test workflow: at least one of \"run-unit\" or \"run-storybook\" must be true. Library repos use run-unit: true; UI/Storybook repos use run-storybook: true.");
-			}
 			if (!KNOWN_WORKFLOWS.has(name)) {
 				steps.push({
 					capability: "source",
@@ -4420,22 +4405,12 @@ async function runSetup(input) {
 				print(formatStep(steps[steps.length - 1]));
 				continue;
 			}
-			if (name === "deploy" && rawWith) {
-				const previewCfg = extractPreviewConfig(rawWith, {
-					org: config.org,
-					domain: config.domain
-				});
-				if (previewCfg) {
-					const paths = additionalPaths;
-					steps.push(await runStep("source", "write workflow deploy (with preview)", dryRun, async () => {
-						await source.writeWorkflowFile("deploy.yml", `${workflowHeader$1()}${generateCombinedDeployContent(withOverrides, paths, previewCfg)}`);
-					}));
-					print(formatStep(steps[steps.length - 1]));
-					continue;
-				}
-			}
-			steps.push(await runStep("source", `write workflow ${name}`, dryRun, async () => {
-				await source.writeWorkflowFile(`${name}.yml`, `${workflowHeader$1()}${generateThinCallerContent(name, withOverrides, additionalPaths, void 0, comments)}`);
+			const filename = name === "deploy" ? "deploy.yml" : `${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}`;
+			steps.push(await runStep("source", step, dryRun, async () => {
+				await source.writeWorkflowFile(filename, `${workflowHeader$1()}${content}`);
 			}));
 			print(formatStep(steps[steps.length - 1]));
 		}
@@ -5647,51 +5622,36 @@ async function runSync(input) {
 					message: "no tasks configured"
 				});
 				print(formatSyncStep(steps[steps.length - 1]));
-			} else for (const entry of taskEntries) {
-				const name = typeof entry === "string" ? entry : entry.name;
-				const rawWith = typeof entry === "object" ? entry.with : void 0;
-				const normalized = rawWith ? normalizeWorkflowWith(rawWith) : void 0;
-				let withOverrides = normalized;
-				let comments;
-				if (name === "lint") {
-					const lint = lintThinCallerWith({
-						explicit: typeof entry === "object" ? entry.linters : void 0,
-						rootFiles: safeReaddir(input.context.repoRoot),
-						extra: normalized
-					});
-					withOverrides = lint.withOverrides;
-					comments = lint.comments;
-				}
-				const additionalPaths = (typeof entry === "object" ? entry.paths : void 0) ?? (name === "deploy" && rawWith ? deriveDeployPaths(rawWith) : void 0);
-				if (!KNOWN_WORKFLOWS.has(name)) {
-					steps.push({
-						capability: "local",
-						step: `sync workflow ${name}`,
-						status: "skip",
-						message: `unknown workflow "${name}" — no template available`
-					});
-					print(formatSyncStep(steps[steps.length - 1]));
-					continue;
-				}
-				if (name === "deploy" && rawWith) {
-					const previewCfg = extractPreviewConfig(rawWith, {
+			} else {
+				const files = createAstromech({
+					cwd: input.context.repoRoot,
+					config: { tasks: config.tasks },
+					orgContext: {
 						org: config.org,
 						domain: config.domain
-					});
-					if (previewCfg) {
-						steps.push(await runSyncStep("local", "sync workflow deploy (with preview)", dryRun, async () => {
-							const content = `${workflowHeader()}${generateCombinedDeployContent(withOverrides, additionalPaths, previewCfg)}`;
-							await writeWorkflowFile(input.context.repoRoot, "deploy.yml", content);
-						}));
+					}
+				}).thinCallers();
+				for (const entry of taskEntries) {
+					const name = typeof entry === "string" ? entry : entry.name;
+					if (!KNOWN_WORKFLOWS.has(name)) {
+						steps.push({
+							capability: "local",
+							step: `sync workflow ${name}`,
+							status: "skip",
+							message: `unknown workflow "${name}" — no template available`
+						});
 						print(formatSyncStep(steps[steps.length - 1]));
 						continue;
 					}
+					const filename = name === "deploy" ? "deploy.yml" : `${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)" : ""}`;
+					steps.push(await runSyncStep("local", step, dryRun, async () => {
+						await writeWorkflowFile(input.context.repoRoot, filename, `${workflowHeader()}${content}`);
+					}));
+					print(formatSyncStep(steps[steps.length - 1]));
 				}
-				steps.push(await runSyncStep("local", `sync workflow ${name}`, dryRun, async () => {
-					const content = `${workflowHeader()}${generateThinCallerContent(name, withOverrides, additionalPaths, void 0, comments)}`;
-					await writeWorkflowFile(input.context.repoRoot, `${name}.yml`, content);
-				}));
-				print(formatSyncStep(steps[steps.length - 1]));
 			}
 		}
 		if (stepName === "scripts") if (config.syncScripts === false) {
@@ -5781,14 +5741,6 @@ function formatSyncStep(step) {
 	const detail = step.message ? `  (${step.message})` : "";
 	return `    ${icon} ${step.step}${detail}`;
 }
-/** Repo-root filenames for linter auto-detection; `[]` when the dir is unreadable. */
-function safeReaddir(dir) {
-	try {
-		return readdirSync(dir);
-	} catch {
-		return [];
-	}
-}
 async function writePackageJsonField(repoRoot, field, value) {
 	const pkgPath = join(repoRoot, "package.json");
 	let content;