@savvy-web/bundler 2.0.9 → 2.0.11

package/index.d.ts

@@ -263,6 +263,11 @@ interface RunOptions {
     target: "dev" | "prod";
     reports: ReadonlyArray<BuildReport>;
     now?: () => Date;
+    buildOk?: boolean | undefined;
+    failure?: {
+      name?: string | undefined;
+      message: string;
+    } | undefined;
   }) => string | undefined;
   /** Injectable ambient-.d.ts copier (defaults to copyAmbientDts). */
   readonly copyAmbientDts?: ((o: CopyAmbientDtsOptions) => void) | undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@savvy-web/bundler",
-	"version": "2.0.9",
+	"version": "2.0.11",
 	"private": false,
 	"description": "Zero-config tsdown-based bundler for Silk Suite TypeScript packages",
 	"homepage": "https://github.com/savvy-web/systems/tree/main/packages/bundler",
@@ -29,7 +29,7 @@
 		"./package.json": "./package.json"
 	},
 	"dependencies": {
-		"@savvy-web/tsdown-plugins": "2.1.8",
+		"@savvy-web/tsdown-plugins": "2.2.0",
 		"@tsdown/exe": "^0.22.12",
 		"effect": "4.0.0-beta.101",
 		"rolldown": "^1.2.0",

package/run.js

@@ -5,6 +5,23 @@ import { dirname, join } from "node:path";
 import { Effect } from "effect";
 
 //#region src/run.ts
+/** Reduce a thrown value to the `{ name?, message }` shape the issues artifact stamps on a failed build. */
+function describeFailure(err) {
+	if (err instanceof Error) return err.name !== "" ? {
+		name: err.name,
+		message: err.message
+	} : { message: err.message };
+	if (typeof err === "object" && err !== null) {
+		const rec = err;
+		const name = typeof rec._tag === "string" ? rec._tag : typeof rec.name === "string" ? rec.name : void 0;
+		const message = typeof rec.message === "string" ? rec.message : String(err);
+		return name !== void 0 ? {
+			name,
+			message
+		} : { message };
+	}
+	return { message: String(err) };
+}
 /** Read and parse package.json at cwd, returning an empty object on any error. */
 function readPackageJson(cwd) {
 	try {
@@ -56,164 +73,167 @@ async function runBuild(config, options) {
 	const version = options.readVersion ? options.readVersion() : pkg.version ?? "0.0.0";
 	const packageName = options.readPackageName ? options.readPackageName() : pkg.name ?? "unknown";
 	const collector = new BuildCollector();
-	const jsx = resolveJsxConfig((options.readTsconfigJsx ?? (() => readTsconfigJsx(cwd)))(), config.jsx);
-	const tsconfigPath = (options.writeTsconfig ?? ((c) => writeResolvedTsconfig({
-		cwd: c,
-		...jsx?.runtime === "automatic" ? {
-			jsx: "react-jsx",
-			jsxImportSource: jsx.importSource
-		} : {},
-		...jsx?.runtime === "classic" ? { jsx: "react" } : {}
-	})))(cwd);
-	const exportsMap = options.readExports ? options.readExports() : pkg.exports;
-	const publishTargets = (options.readPublishTargets ?? (() => {
-		const declared = pkg.publishConfig?.targets;
-		return declared !== void 0 && !Array.isArray(declared) && typeof declared === "object" ? declared : void 0;
-	}))();
-	const writeBinding = options.writeTargetsBinding ?? writeTargetsBinding;
-	const osCpuForValidate = options.readOsCpu ? options.readOsCpu() : {
-		os: pkg.os ?? [],
-		cpu: pkg.cpu ?? []
-	};
-	await Effect.runPromise(Effect.flatMap(ConfigValidator, (v) => v.validate({
-		baseName: packageName,
-		hasExports: exportsMap !== void 0 && Object.keys(exportsMap).length > 0,
-		...publishTargets !== void 0 ? { targets: publishTargets } : {},
-		...config.exe !== void 0 ? { exe: config.exe } : {},
-		osCpu: osCpuForValidate,
-		...config.meta !== void 0 && config.meta !== false ? { meta: config.meta } : {},
-		...config.looseFiles !== void 0 ? { looseFiles: config.looseFiles } : {}
-	})).pipe(Effect.provide(ConfigValidatorLive)));
-	const exeSpecs = config.exe !== void 0 ? normalizeExeOptions(config.exe, osCpuForValidate) : [];
-	if (config.exe !== void 0 && (exeSpecs.length !== 1 || (exeSpecs[0]?.targets.length ?? 0) !== 1)) throw new Error(`exe build requires exactly one binary with one target (got ${exeSpecs.length} spec(s), ${exeSpecs[0]?.targets.length ?? 0} target(s) on the first). A package's exports["."] resolves to a single SEA — cross-platform binaries must each ship as their own per-platform package.`);
-	const exeSpec = exeSpecs[0];
-	const exeTarget = exeSpec?.targets[0];
-	const exeFileName = exeSpec && exeTarget ? computeExeFileName(exeSpec.fileName, exeTarget) : void 0;
-	const exeEntrySource = exeSpec?.entry ?? "./src/bin.ts";
-	const exeRewrite = config.exe !== void 0 && exeFileName !== void 0 ? {
-		source: exeEntrySource,
-		fileName: exeFileName,
-		dir: "bin"
-	} : void 0;
-	const entries = packageJsonEntries({
-		pkg: {
-			exports: exportsMap ?? pkg.exports,
-			bin: pkg.bin
-		},
-		...config.exe !== void 0 ? { excludeSources: [exeEntrySource] } : {}
-	});
-	const hasJsEntries = Object.keys(entries).length > 0;
-	validateSubdirOverrides(config.overrides, entries, packageName);
-	const ambient = extractAmbientDts({
-		exports: exportsMap ?? pkg.exports,
-		bin: pkg.bin
-	}, {});
-	assertNoEntryCollisions(Object.keys(entries), ambient);
-	if (ambient.length > 0 && !hasJsEntries && config.exe === void 0) throw new ConfigValidationError({
-		path: "exports",
-		reason: "a types-only package with only ambient .d.ts exports is not supported — add at least one JS entry (or an exe) alongside the ambient declarations"
-	});
-	if (target === "meta") {
-		(options.writeOutput ?? ((o) => process.stdout.write(`${o.content}\n`)))({
-			target: "stdout",
-			contentType: "text/plain",
-			content: `meta: --target meta is deprecated and now a no-op; meta is emitted by --target prod (${packageName}).`
-		});
-		return;
-	}
-	if (target === "exe") {
-		if (config.exe === void 0) throw new Error("`savvy build --target exe` requires an `exe` option in the build config");
-		await (options.runExeBuild ?? runExeBuild)({
-			cwd,
-			outDir: join(cwd, "dist", "dev", "pkg", "bin"),
-			specs: exeSpecs,
-			collector,
-			groupId: "dev",
-			verbose
-		});
-		(options.writeOutput ?? ((o) => process.stdout.write(`${o.content}\n`)))({
-			target: "stdout",
-			contentType: "text/plain",
-			content: `exe: compiled ${exeSpecs.length} binary/binaries for ${packageName}`
-		});
-		return;
-	}
-	let overridePartitions = [];
-	let baseEntries = entries;
-	let dualExports;
-	const subdirExports = /* @__PURE__ */ new Set();
-	if (config.overrides) {
-		const partitions = [];
-		const overriddenEntryNames = /* @__PURE__ */ new Set();
-		const baseFormatHasCjs = (config.format ?? ["esm"]).includes("cjs");
-		const dualExportKeys = /* @__PURE__ */ new Set();
-		const exportPathByEntry = deriveExportPaths(entries, exportsMap);
-		for (const ov of config.overrides) {
-			if (ov.outSubdir !== void 0 && ov.entries.length !== 1) throw new Error(`overrides: outSubdir "${ov.outSubdir}" must pin exactly one export path (got ${ov.entries.length})`);
-			const partEntry = {};
-			for (const exportPath of ov.entries) {
-				if (exportPath !== "." && !exportPath.startsWith("./")) throw new Error(`overrides: entry "${exportPath}" must be a canonical export path — use "." for the root or a "./"-prefixed subpath (e.g. "./changesets/markdownlint")`);
-				const flatName = createEntryName(exportPath, false);
-				const src = entries[flatName];
-				if (src === void 0) throw new Error(`overrides: export path "${exportPath}" (entry "${flatName}") is not a build entry of ${packageName}`);
-				const entryName = ov.outSubdir !== void 0 ? "index" : flatName;
-				partEntry[entryName] = src;
-				overriddenEntryNames.add(flatName);
-				if (ov.outSubdir !== void 0) subdirExports.add(exportPath);
-				if ((ov.format ?? config.format ?? ["esm"]).includes("cjs")) dualExportKeys.add(exportPath);
-			}
-			partitions.push({
-				entry: partEntry,
-				...ov.format !== void 0 ? { format: ov.format } : {},
-				...ov.externals !== void 0 ? { externals: ov.externals } : {},
-				...ov.bundle !== void 0 ? { bundle: ov.bundle } : {},
-				...ov.bundleNodeModules !== void 0 ? { bundleNodeModules: ov.bundleNodeModules } : {},
-				...ov.bundledPackages !== void 0 ? { bundledPackages: ov.bundledPackages } : {},
-				...ov.dtsExternals !== void 0 ? { dtsExternals: ov.dtsExternals } : {},
-				...ov.platform !== void 0 ? { platform: ov.platform } : {},
-				...ov.css !== void 0 ? { css: ov.css } : {},
-				...ov.outSubdir !== void 0 ? { outSubdir: ov.outSubdir } : {}
-			});
-		}
-		const onlyBase = {};
-		for (const [name, src] of Object.entries(entries)) {
-			if (overriddenEntryNames.has(name)) continue;
-			onlyBase[name] = src;
-			if (baseFormatHasCjs) dualExportKeys.add(exportPathByEntry[name] ?? (name === "index" ? "." : `./${name}`));
-		}
-		overridePartitions = partitions;
-		baseEntries = onlyBase;
-		dualExports = dualExportKeys;
-	}
-	const looseFiles = config.looseFiles !== void 0 ? normalizeLooseFiles(config.looseFiles) : void 0;
-	const { groups, resolution } = target === "dev" ? {
-		groups: [{
-			id: "dev",
-			name: packageName
-		}],
-		resolution: void 0
-	} : deriveProdGroups(publishTargets, packageName);
-	const explicitFormat = config.output?.format;
-	const writeOutput = options.writeOutput ?? ((o) => process.stdout.write(`${o.content}\n`));
-	const renderAndWrite = async () => {
-		const rendered = await Effect.runPromise(renderReport(collector.snapshot(packageName), {
-			...explicitFormat !== void 0 ? { explicitFormat } : {},
-			verbose,
-			noColor: process.env.NO_COLOR !== void 0 || !process.stdout.isTTY
-		}).pipe(Effect.provide(ReportPipelineLive)));
-		for (const output of rendered) writeOutput(output);
-	};
-	const writeIssuesBestEffort = () => {
+	const writeIssuesBestEffort = (err) => {
 		if (target !== "dev" && target !== "prod") return;
 		try {
 			(options.writeIssues ?? writeIssuesArtifact)({
 				cwd,
 				target,
-				reports: collector.snapshot(packageName)
+				reports: collector.snapshot(packageName),
+				buildOk: err === void 0,
+				...err !== void 0 ? { failure: describeFailure(err) } : {}
 			});
 		} catch {}
 	};
+	let renderAndWrite;
 	try {
+		const jsx = resolveJsxConfig((options.readTsconfigJsx ?? (() => readTsconfigJsx(cwd)))(), config.jsx);
+		const tsconfigPath = (options.writeTsconfig ?? ((c) => writeResolvedTsconfig({
+			cwd: c,
+			...jsx?.runtime === "automatic" ? {
+				jsx: "react-jsx",
+				jsxImportSource: jsx.importSource
+			} : {},
+			...jsx?.runtime === "classic" ? { jsx: "react" } : {}
+		})))(cwd);
+		const exportsMap = options.readExports ? options.readExports() : pkg.exports;
+		const publishTargets = (options.readPublishTargets ?? (() => {
+			const declared = pkg.publishConfig?.targets;
+			return declared !== void 0 && !Array.isArray(declared) && typeof declared === "object" ? declared : void 0;
+		}))();
+		const writeBinding = options.writeTargetsBinding ?? writeTargetsBinding;
+		const osCpuForValidate = options.readOsCpu ? options.readOsCpu() : {
+			os: pkg.os ?? [],
+			cpu: pkg.cpu ?? []
+		};
+		await Effect.runPromise(Effect.flatMap(ConfigValidator, (v) => v.validate({
+			baseName: packageName,
+			hasExports: exportsMap !== void 0 && Object.keys(exportsMap).length > 0,
+			...publishTargets !== void 0 ? { targets: publishTargets } : {},
+			...config.exe !== void 0 ? { exe: config.exe } : {},
+			osCpu: osCpuForValidate,
+			...config.meta !== void 0 && config.meta !== false ? { meta: config.meta } : {},
+			...config.looseFiles !== void 0 ? { looseFiles: config.looseFiles } : {}
+		})).pipe(Effect.provide(ConfigValidatorLive)));
+		const exeSpecs = config.exe !== void 0 ? normalizeExeOptions(config.exe, osCpuForValidate) : [];
+		if (config.exe !== void 0 && (exeSpecs.length !== 1 || (exeSpecs[0]?.targets.length ?? 0) !== 1)) throw new Error(`exe build requires exactly one binary with one target (got ${exeSpecs.length} spec(s), ${exeSpecs[0]?.targets.length ?? 0} target(s) on the first). A package's exports["."] resolves to a single SEA — cross-platform binaries must each ship as their own per-platform package.`);
+		const exeSpec = exeSpecs[0];
+		const exeTarget = exeSpec?.targets[0];
+		const exeFileName = exeSpec && exeTarget ? computeExeFileName(exeSpec.fileName, exeTarget) : void 0;
+		const exeEntrySource = exeSpec?.entry ?? "./src/bin.ts";
+		const exeRewrite = config.exe !== void 0 && exeFileName !== void 0 ? {
+			source: exeEntrySource,
+			fileName: exeFileName,
+			dir: "bin"
+		} : void 0;
+		const entries = packageJsonEntries({
+			pkg: {
+				exports: exportsMap ?? pkg.exports,
+				bin: pkg.bin
+			},
+			...config.exe !== void 0 ? { excludeSources: [exeEntrySource] } : {}
+		});
+		const hasJsEntries = Object.keys(entries).length > 0;
+		validateSubdirOverrides(config.overrides, entries, packageName);
+		const ambient = extractAmbientDts({
+			exports: exportsMap ?? pkg.exports,
+			bin: pkg.bin
+		}, {});
+		assertNoEntryCollisions(Object.keys(entries), ambient);
+		if (ambient.length > 0 && !hasJsEntries && config.exe === void 0) throw new ConfigValidationError({
+			path: "exports",
+			reason: "a types-only package with only ambient .d.ts exports is not supported — add at least one JS entry (or an exe) alongside the ambient declarations"
+		});
+		if (target === "meta") {
+			(options.writeOutput ?? ((o) => process.stdout.write(`${o.content}\n`)))({
+				target: "stdout",
+				contentType: "text/plain",
+				content: `meta: --target meta is deprecated and now a no-op; meta is emitted by --target prod (${packageName}).`
+			});
+			return;
+		}
+		if (target === "exe") {
+			if (config.exe === void 0) throw new Error("`savvy build --target exe` requires an `exe` option in the build config");
+			await (options.runExeBuild ?? runExeBuild)({
+				cwd,
+				outDir: join(cwd, "dist", "dev", "pkg", "bin"),
+				specs: exeSpecs,
+				collector,
+				groupId: "dev",
+				verbose
+			});
+			(options.writeOutput ?? ((o) => process.stdout.write(`${o.content}\n`)))({
+				target: "stdout",
+				contentType: "text/plain",
+				content: `exe: compiled ${exeSpecs.length} binary/binaries for ${packageName}`
+			});
+			return;
+		}
+		let overridePartitions = [];
+		let baseEntries = entries;
+		let dualExports;
+		const subdirExports = /* @__PURE__ */ new Set();
+		if (config.overrides) {
+			const partitions = [];
+			const overriddenEntryNames = /* @__PURE__ */ new Set();
+			const baseFormatHasCjs = (config.format ?? ["esm"]).includes("cjs");
+			const dualExportKeys = /* @__PURE__ */ new Set();
+			const exportPathByEntry = deriveExportPaths(entries, exportsMap);
+			for (const ov of config.overrides) {
+				if (ov.outSubdir !== void 0 && ov.entries.length !== 1) throw new Error(`overrides: outSubdir "${ov.outSubdir}" must pin exactly one export path (got ${ov.entries.length})`);
+				const partEntry = {};
+				for (const exportPath of ov.entries) {
+					if (exportPath !== "." && !exportPath.startsWith("./")) throw new Error(`overrides: entry "${exportPath}" must be a canonical export path — use "." for the root or a "./"-prefixed subpath (e.g. "./changesets/markdownlint")`);
+					const flatName = createEntryName(exportPath, false);
+					const src = entries[flatName];
+					if (src === void 0) throw new Error(`overrides: export path "${exportPath}" (entry "${flatName}") is not a build entry of ${packageName}`);
+					const entryName = ov.outSubdir !== void 0 ? "index" : flatName;
+					partEntry[entryName] = src;
+					overriddenEntryNames.add(flatName);
+					if (ov.outSubdir !== void 0) subdirExports.add(exportPath);
+					if ((ov.format ?? config.format ?? ["esm"]).includes("cjs")) dualExportKeys.add(exportPath);
+				}
+				partitions.push({
+					entry: partEntry,
+					...ov.format !== void 0 ? { format: ov.format } : {},
+					...ov.externals !== void 0 ? { externals: ov.externals } : {},
+					...ov.bundle !== void 0 ? { bundle: ov.bundle } : {},
+					...ov.bundleNodeModules !== void 0 ? { bundleNodeModules: ov.bundleNodeModules } : {},
+					...ov.bundledPackages !== void 0 ? { bundledPackages: ov.bundledPackages } : {},
+					...ov.dtsExternals !== void 0 ? { dtsExternals: ov.dtsExternals } : {},
+					...ov.platform !== void 0 ? { platform: ov.platform } : {},
+					...ov.css !== void 0 ? { css: ov.css } : {},
+					...ov.outSubdir !== void 0 ? { outSubdir: ov.outSubdir } : {}
+				});
+			}
+			const onlyBase = {};
+			for (const [name, src] of Object.entries(entries)) {
+				if (overriddenEntryNames.has(name)) continue;
+				onlyBase[name] = src;
+				if (baseFormatHasCjs) dualExportKeys.add(exportPathByEntry[name] ?? (name === "index" ? "." : `./${name}`));
+			}
+			overridePartitions = partitions;
+			baseEntries = onlyBase;
+			dualExports = dualExportKeys;
+		}
+		const looseFiles = config.looseFiles !== void 0 ? normalizeLooseFiles(config.looseFiles) : void 0;
+		const { groups, resolution } = target === "dev" ? {
+			groups: [{
+				id: "dev",
+				name: packageName
+			}],
+			resolution: void 0
+		} : deriveProdGroups(publishTargets, packageName);
+		const explicitFormat = config.output?.format;
+		const writeOutput = options.writeOutput ?? ((o) => process.stdout.write(`${o.content}\n`));
+		renderAndWrite = async () => {
+			const rendered = await Effect.runPromise(renderReport(collector.snapshot(packageName), {
+				...explicitFormat !== void 0 ? { explicitFormat } : {},
+				verbose,
+				noColor: process.env.NO_COLOR !== void 0 || !process.stdout.isTTY
+			}).pipe(Effect.provide(ReportPipelineLive)));
+			for (const output of rendered) writeOutput(output);
+		};
 		if (hasJsEntries || config.exe === void 0) await build({
 			cwd,
 			version,
@@ -306,12 +326,14 @@ async function runBuild(config, options) {
 			}
 		}
 	} catch (err) {
-		await renderAndWrite();
-		writeIssuesBestEffort();
+		writeIssuesBestEffort(err);
+		if (renderAndWrite !== void 0) try {
+			await renderAndWrite();
+		} catch {}
 		throw err;
 	}
-	await renderAndWrite();
 	writeIssuesBestEffort();
+	if (renderAndWrite !== void 0) await renderAndWrite();
 }
 /**
 * Sugar front door: define + run in one call, deriving `cwd`/`argv` from process globals.