@bamboocss/vite 1.30.1 → 1.32.0

package/dist/index.cjs

@@ -1108,8 +1108,7 @@ const createCssContext = (ctx) => ({
 	hash: Boolean(ctx.hash.className),
 	conditions: {
 		shift: ctx.conditions.shift,
-		finalize: ctx.conditions.finalize,
-		breakpoints: { keys: ctx.conditions.breakpoints.keys }
+		finalize: ctx.conditions.finalize
 	},
 	utility: {
 		prefix: ctx.utility.prefix,
@@ -1196,8 +1195,7 @@ const createRuntimeRecipe = (ctx) => {
 			hash: Boolean(ctx.hash.className),
 			conditions: {
 				shift: ctx.conditions.shift,
-				finalize: ctx.conditions.finalize,
-				breakpoints: { keys: ctx.conditions.breakpoints.keys }
+				finalize: ctx.conditions.finalize
 			},
 			utility: {
 				prefix: ctx.utility.prefix,
@@ -1271,7 +1269,7 @@ const UNFOLDABLE_TYPES = new Set(["cva", "sva"]);
 *
 * `overlapping` is handled by the enclosing fold, and `not-imported` is somebody else's
 * function of the same name — neither leaves a call of ours. `not-foldable` is a `cva`/`sva`
-* definition, which keeps the recipe runtime rather than the css engine; see `strict`.
+* definition, which keeps the recipe runtime rather than the css engine; see `failOnUnfolded`.
 */
 const SURVIVES_TO_RUNTIME = new Set([
 	"dynamic",
@@ -1280,7 +1278,8 @@ const SURVIVES_TO_RUNTIME = new Set([
 	"unsupported-kind",
 	"no-call-expression",
 	"empty",
-	"unresolved-token"
+	"unresolved-token",
+	"fold-failed"
 ]);
 /**
 * A call of a recipe the file bound itself: `const badge = cva(...)`, then `badge({ tone })`.
@@ -1333,7 +1332,7 @@ const isValueReference = (identifier) => {
 *
 * `cva` and `sva` are there for the reason `SURVIVES_TO_RUNTIME` omits `not-foldable`: a
 * recipe *definition* cannot fold to a class string and never could, and what it keeps is the
-* recipe runtime rather than the css engine — which `strict` accepts. Their unfoldable
+* recipe runtime rather than the css engine — which `failOnUnfolded` accepts. Their unfoldable
 * invocations are reported separately, as `recipe-call`.
 */
 const PERMITTED_BINDINGS = new Set([
@@ -1356,10 +1355,11 @@ const TRAILING_INDEX = /\/index$/;
 /**
 * An argument that cannot run anything when it is evaluated.
 *
-* `token(path, fallback)` evaluates both arguments before the call, so a fold that drops
-* the fallback also drops whatever evaluating it would have done. `token('x', compute())`
-* is pathological, but the fold's contract is behaviour preservation and a literal is the
-* cheap way to prove it: no call, no property read, no getter.
+* `token()` takes one argument, but javascript evaluates every argument a call site passes
+* before the call — so a fold that drops an extra one also drops whatever evaluating it would
+* have done. `token('x', compute())` is pathological and no longer type-checks, and the fold's
+* contract is behaviour preservation regardless: a literal is the cheap way to prove it, since
+* it means no call, no property read, no getter.
 */
 const isInertArgument = (node) => ts_morph.Node.isStringLiteral(node) || ts_morph.Node.isNumericLiteral(node) || ts_morph.Node.isNoSubstitutionTemplateLiteral(node) || node.getKind() === ts_morph.SyntaxKind.TrueKeyword || node.getKind() === ts_morph.SyntaxKind.FalseKeyword || node.getKind() === ts_morph.SyntaxKind.NullKeyword || ts_morph.Node.isIdentifier(node) && node.getText() === "undefined";
 /**
@@ -2268,7 +2268,7 @@ const foldSource = (options) => {
 	* Deliberately not driven by `parserResult`: that is the recogniser, and the point here is
 	* to catch what it did not see. A namespace import called as `s.cva(...)`, a default
 	* import, a specifier that resolved to nothing — each leaves a live reference and no ledger
-	* entry at all, which is how `strict` came to pass a build that still shipped the engine.
+	* entry at all, which is how `failOnUnfolded` came to pass a build that still shipped the engine.
 	*
 	* The helpers the fold itself writes are excluded: `cx`, `cvaPick`, `splitProps` and the
 	* leaf helper live in `cx` and pull no engine, so a reference to one is the fold working
@@ -2430,7 +2430,7 @@ const formatSkipped = (id, skipped) => {
 * with no matching rule.
 */
 const bamboocss = (options = {}) => {
-	const { transform = true, partial, configPath, cwd, reportSkipped = false, reportSummary = true, strict = false } = options;
+	const { transform = true, partial, configPath, cwd, reportSkipped = false, reportSummary = true, failOnUnfolded = false } = options;
 	/** Totals across the build, for the summary. */
 	const totals = {
 		folded: 0,
@@ -2438,7 +2438,7 @@ const bamboocss = (options = {}) => {
 		filesWithFolds: 0,
 		skipped: /* @__PURE__ */ new Map()
 	};
-	/** Under `strict`, every call that would still reach the runtime. */
+	/** Under `failOnUnfolded`, every call that would still reach the runtime. */
 	const survivors = [];
 	/**
 	* Recipe configs read out of modules other than the one being transformed.
@@ -2520,7 +2520,7 @@ const bamboocss = (options = {}) => {
 			try {
 				const sourceFile = ctx.project.addSourceFile(filePath, code);
 				const parserResult = ctx.project.parseSourceFile(filePath);
-				if (!parserResult || parserResult.isEmpty() && !strict) return null;
+				if (!parserResult || parserResult.isEmpty() && !failOnUnfolded) return null;
 				result = foldSource({
 					ctx,
 					code,
@@ -2530,18 +2530,26 @@ const bamboocss = (options = {}) => {
 					partial,
 					parseModule: (path) => ctx?.project.parseSourceFile(path),
 					recipeConfigCache,
-					reportSurvivors: strict,
+					reportSurvivors: failOnUnfolded,
 					sourceFile
 				});
 			} catch (error) {
 				_bamboocss_logger.logger.caughtError("vite:transform", `Failed to fold ${filePath}`, error);
+				totals.files++;
+				totals.skipped.set("fold-failed", (totals.skipped.get("fold-failed") ?? 0) + 1);
+				if (failOnUnfolded) survivors.push({
+					file: filePath,
+					line: 1,
+					name: "fold",
+					reason: "fold-failed"
+				});
 				return null;
 			}
 			totals.files++;
 			totals.folded += result.folded.length;
 			if (result.folded.length) totals.filesWithFolds++;
 			for (const entry of result.skipped) totals.skipped.set(entry.reason, (totals.skipped.get(entry.reason) ?? 0) + 1);
-			if (strict) {
+			if (failOnUnfolded) {
 				for (const entry of result.skipped) {
 					if (!SURVIVES_TO_RUNTIME.has(entry.reason)) continue;
 					survivors.push({
@@ -2568,15 +2576,17 @@ const bamboocss = (options = {}) => {
 			};
 		},
 		buildEnd() {
-			if (strict && survivors.length) {
+			if (failOnUnfolded && survivors.length) {
 				const byFile = /* @__PURE__ */ new Map();
 				for (const entry of survivors) {
 					const list = byFile.get(entry.file) ?? [];
 					list.push(entry);
 					byFile.set(entry.file, list);
 				}
-				const detail = Array.from(byFile.entries()).map(([file, entries]) => [`  ${file}`, ...entries.map((e) => `    ${e.line}: ${e.name}${e.reason === "runtime-binding" ? "" : "()"} — ${e.reason}`)].join("\n")).join("\n");
-				throw new Error(`bamboocss: ${survivors.length} call(s) could not be folded, and \`strict\` is on.\n\n${detail}\n\nEach one keeps \`styled-system/css\` in the bundle, so the engine cannot be dropped however many other calls folded. Make the values static, move the variation into a \`cva\` variant, or generate them with \`staticCss\` — or set \`strict: false\` to accept the runtime.`);
+				const named = (e) => e.reason === "runtime-binding" || e.reason === "fold-failed" ? e.name : `${e.name}()`;
+				const detail = Array.from(byFile.entries()).map(([file, entries]) => [`  ${file}`, ...entries.map((e) => `    ${e.line}: ${named(e)} — ${e.reason}`)].join("\n")).join("\n");
+				const threw = survivors.some((entry) => entry.reason === "fold-failed");
+				throw new Error(`bamboocss: ${survivors.length} call(s) could not be folded, and \`failOnUnfolded\` is on.\n\n${detail}\n\n` + (threw ? "`fold-failed` is a module the fold threw on — see the error logged for it above. Nothing was established about its calls either way, so it cannot support the guarantee this option makes.\n\n" : "") + "Each one keeps `styled-system/css` in the bundle, so the engine cannot be dropped however many other calls folded. Make the values static, move the variation into a `cva` variant, or generate them with `staticCss` — or set `failOnUnfolded: false` to accept the runtime.");
 			}
 			if (!transform || !reportSummary) return;
 			const declined = Array.from(totals.skipped.values()).reduce((sum, count) => sum + count, 0);

package/dist/index.d.cts

@@ -105,7 +105,7 @@ declare const createRuntimeCss: (ctx: Context) => RuntimeCss;
  * Why a call site was left alone. Surfaced through `panda`-style diagnostics so a
  * user can tell the difference between "this folded" and "this silently didn't".
  */
-type SkipReason = 'dynamic' | 'raw-call' | 'not-foldable' | 'recipe-call' | 'unsupported-kind' | 'not-imported' | 'no-call-expression' | 'overlapping' | 'empty' | 'unresolved-token' | 'runtime-binding';
+type SkipReason = 'dynamic' | 'raw-call' | 'not-foldable' | 'recipe-call' | 'unsupported-kind' | 'not-imported' | 'no-call-expression' | 'overlapping' | 'empty' | 'unresolved-token' | 'runtime-binding' | 'fold-failed';
 interface FoldedCall {
   name: string;
   /**
@@ -198,7 +198,7 @@ interface FoldOptions {
    *
    * This asks what the guarantee actually claims: after the rewrite, is anything from a
    * bamboo module still referenced? Off by default because it costs an identifier walk, and
-   * only `strict` needs an answer it can fail a build on.
+   * only `failOnUnfolded` needs an answer it can fail a build on.
    */
   reportSurvivors?: boolean;
   /**
@@ -284,9 +284,13 @@ interface BambooVitePluginOptions {
    * writing recipes — and recipes keep their own much smaller runtime by design. What this
    * guarantees is narrower and checkable: nothing still calls `css()`.
    *
+   * Named for what it checks. It was `strict`, which meant nothing in common with
+   * `strictTokens` or a pattern's old `strict` — three unrelated options sharing a word, to
+   * the point that comments in this repo used a bare "strict" for all three.
+   *
    * @default false
    */
-  strict?: boolean;
+  failOnUnfolded?: boolean;
 }
 /**
  * Vite integration for Bamboo CSS.

package/dist/index.d.mts

@@ -105,7 +105,7 @@ declare const createRuntimeCss: (ctx: Context) => RuntimeCss;
  * Why a call site was left alone. Surfaced through `panda`-style diagnostics so a
  * user can tell the difference between "this folded" and "this silently didn't".
  */
-type SkipReason = 'dynamic' | 'raw-call' | 'not-foldable' | 'recipe-call' | 'unsupported-kind' | 'not-imported' | 'no-call-expression' | 'overlapping' | 'empty' | 'unresolved-token' | 'runtime-binding';
+type SkipReason = 'dynamic' | 'raw-call' | 'not-foldable' | 'recipe-call' | 'unsupported-kind' | 'not-imported' | 'no-call-expression' | 'overlapping' | 'empty' | 'unresolved-token' | 'runtime-binding' | 'fold-failed';
 interface FoldedCall {
   name: string;
   /**
@@ -198,7 +198,7 @@ interface FoldOptions {
    *
    * This asks what the guarantee actually claims: after the rewrite, is anything from a
    * bamboo module still referenced? Off by default because it costs an identifier walk, and
-   * only `strict` needs an answer it can fail a build on.
+   * only `failOnUnfolded` needs an answer it can fail a build on.
    */
   reportSurvivors?: boolean;
   /**
@@ -284,9 +284,13 @@ interface BambooVitePluginOptions {
    * writing recipes — and recipes keep their own much smaller runtime by design. What this
    * guarantees is narrower and checkable: nothing still calls `css()`.
    *
+   * Named for what it checks. It was `strict`, which meant nothing in common with
+   * `strictTokens` or a pattern's old `strict` — three unrelated options sharing a word, to
+   * the point that comments in this repo used a bare "strict" for all three.
+   *
    * @default false
    */
-  strict?: boolean;
+  failOnUnfolded?: boolean;
 }
 /**
  * Vite integration for Bamboo CSS.

package/dist/index.mjs

@@ -1081,8 +1081,7 @@ const createCssContext = (ctx) => ({
 	hash: Boolean(ctx.hash.className),
 	conditions: {
 		shift: ctx.conditions.shift,
-		finalize: ctx.conditions.finalize,
-		breakpoints: { keys: ctx.conditions.breakpoints.keys }
+		finalize: ctx.conditions.finalize
 	},
 	utility: {
 		prefix: ctx.utility.prefix,
@@ -1169,8 +1168,7 @@ const createRuntimeRecipe = (ctx) => {
 			hash: Boolean(ctx.hash.className),
 			conditions: {
 				shift: ctx.conditions.shift,
-				finalize: ctx.conditions.finalize,
-				breakpoints: { keys: ctx.conditions.breakpoints.keys }
+				finalize: ctx.conditions.finalize
 			},
 			utility: {
 				prefix: ctx.utility.prefix,
@@ -1244,7 +1242,7 @@ const UNFOLDABLE_TYPES = new Set(["cva", "sva"]);
 *
 * `overlapping` is handled by the enclosing fold, and `not-imported` is somebody else's
 * function of the same name — neither leaves a call of ours. `not-foldable` is a `cva`/`sva`
-* definition, which keeps the recipe runtime rather than the css engine; see `strict`.
+* definition, which keeps the recipe runtime rather than the css engine; see `failOnUnfolded`.
 */
 const SURVIVES_TO_RUNTIME = new Set([
 	"dynamic",
@@ -1253,7 +1251,8 @@ const SURVIVES_TO_RUNTIME = new Set([
 	"unsupported-kind",
 	"no-call-expression",
 	"empty",
-	"unresolved-token"
+	"unresolved-token",
+	"fold-failed"
 ]);
 /**
 * A call of a recipe the file bound itself: `const badge = cva(...)`, then `badge({ tone })`.
@@ -1306,7 +1305,7 @@ const isValueReference = (identifier) => {
 *
 * `cva` and `sva` are there for the reason `SURVIVES_TO_RUNTIME` omits `not-foldable`: a
 * recipe *definition* cannot fold to a class string and never could, and what it keeps is the
-* recipe runtime rather than the css engine — which `strict` accepts. Their unfoldable
+* recipe runtime rather than the css engine — which `failOnUnfolded` accepts. Their unfoldable
 * invocations are reported separately, as `recipe-call`.
 */
 const PERMITTED_BINDINGS = new Set([
@@ -1329,10 +1328,11 @@ const TRAILING_INDEX = /\/index$/;
 /**
 * An argument that cannot run anything when it is evaluated.
 *
-* `token(path, fallback)` evaluates both arguments before the call, so a fold that drops
-* the fallback also drops whatever evaluating it would have done. `token('x', compute())`
-* is pathological, but the fold's contract is behaviour preservation and a literal is the
-* cheap way to prove it: no call, no property read, no getter.
+* `token()` takes one argument, but javascript evaluates every argument a call site passes
+* before the call — so a fold that drops an extra one also drops whatever evaluating it would
+* have done. `token('x', compute())` is pathological and no longer type-checks, and the fold's
+* contract is behaviour preservation regardless: a literal is the cheap way to prove it, since
+* it means no call, no property read, no getter.
 */
 const isInertArgument = (node) => Node.isStringLiteral(node) || Node.isNumericLiteral(node) || Node.isNoSubstitutionTemplateLiteral(node) || node.getKind() === SyntaxKind.TrueKeyword || node.getKind() === SyntaxKind.FalseKeyword || node.getKind() === SyntaxKind.NullKeyword || Node.isIdentifier(node) && node.getText() === "undefined";
 /**
@@ -2241,7 +2241,7 @@ const foldSource = (options) => {
 	* Deliberately not driven by `parserResult`: that is the recogniser, and the point here is
 	* to catch what it did not see. A namespace import called as `s.cva(...)`, a default
 	* import, a specifier that resolved to nothing — each leaves a live reference and no ledger
-	* entry at all, which is how `strict` came to pass a build that still shipped the engine.
+	* entry at all, which is how `failOnUnfolded` came to pass a build that still shipped the engine.
 	*
 	* The helpers the fold itself writes are excluded: `cx`, `cvaPick`, `splitProps` and the
 	* leaf helper live in `cx` and pull no engine, so a reference to one is the fold working
@@ -2403,7 +2403,7 @@ const formatSkipped = (id, skipped) => {
 * with no matching rule.
 */
 const bamboocss = (options = {}) => {
-	const { transform = true, partial, configPath, cwd, reportSkipped = false, reportSummary = true, strict = false } = options;
+	const { transform = true, partial, configPath, cwd, reportSkipped = false, reportSummary = true, failOnUnfolded = false } = options;
 	/** Totals across the build, for the summary. */
 	const totals = {
 		folded: 0,
@@ -2411,7 +2411,7 @@ const bamboocss = (options = {}) => {
 		filesWithFolds: 0,
 		skipped: /* @__PURE__ */ new Map()
 	};
-	/** Under `strict`, every call that would still reach the runtime. */
+	/** Under `failOnUnfolded`, every call that would still reach the runtime. */
 	const survivors = [];
 	/**
 	* Recipe configs read out of modules other than the one being transformed.
@@ -2493,7 +2493,7 @@ const bamboocss = (options = {}) => {
 			try {
 				const sourceFile = ctx.project.addSourceFile(filePath, code);
 				const parserResult = ctx.project.parseSourceFile(filePath);
-				if (!parserResult || parserResult.isEmpty() && !strict) return null;
+				if (!parserResult || parserResult.isEmpty() && !failOnUnfolded) return null;
 				result = foldSource({
 					ctx,
 					code,
@@ -2503,18 +2503,26 @@ const bamboocss = (options = {}) => {
 					partial,
 					parseModule: (path) => ctx?.project.parseSourceFile(path),
 					recipeConfigCache,
-					reportSurvivors: strict,
+					reportSurvivors: failOnUnfolded,
 					sourceFile
 				});
 			} catch (error) {
 				logger.caughtError("vite:transform", `Failed to fold ${filePath}`, error);
+				totals.files++;
+				totals.skipped.set("fold-failed", (totals.skipped.get("fold-failed") ?? 0) + 1);
+				if (failOnUnfolded) survivors.push({
+					file: filePath,
+					line: 1,
+					name: "fold",
+					reason: "fold-failed"
+				});
 				return null;
 			}
 			totals.files++;
 			totals.folded += result.folded.length;
 			if (result.folded.length) totals.filesWithFolds++;
 			for (const entry of result.skipped) totals.skipped.set(entry.reason, (totals.skipped.get(entry.reason) ?? 0) + 1);
-			if (strict) {
+			if (failOnUnfolded) {
 				for (const entry of result.skipped) {
 					if (!SURVIVES_TO_RUNTIME.has(entry.reason)) continue;
 					survivors.push({
@@ -2541,15 +2549,17 @@ const bamboocss = (options = {}) => {
 			};
 		},
 		buildEnd() {
-			if (strict && survivors.length) {
+			if (failOnUnfolded && survivors.length) {
 				const byFile = /* @__PURE__ */ new Map();
 				for (const entry of survivors) {
 					const list = byFile.get(entry.file) ?? [];
 					list.push(entry);
 					byFile.set(entry.file, list);
 				}
-				const detail = Array.from(byFile.entries()).map(([file, entries]) => [`  ${file}`, ...entries.map((e) => `    ${e.line}: ${e.name}${e.reason === "runtime-binding" ? "" : "()"} — ${e.reason}`)].join("\n")).join("\n");
-				throw new Error(`bamboocss: ${survivors.length} call(s) could not be folded, and \`strict\` is on.\n\n${detail}\n\nEach one keeps \`styled-system/css\` in the bundle, so the engine cannot be dropped however many other calls folded. Make the values static, move the variation into a \`cva\` variant, or generate them with \`staticCss\` — or set \`strict: false\` to accept the runtime.`);
+				const named = (e) => e.reason === "runtime-binding" || e.reason === "fold-failed" ? e.name : `${e.name}()`;
+				const detail = Array.from(byFile.entries()).map(([file, entries]) => [`  ${file}`, ...entries.map((e) => `    ${e.line}: ${named(e)} — ${e.reason}`)].join("\n")).join("\n");
+				const threw = survivors.some((entry) => entry.reason === "fold-failed");
+				throw new Error(`bamboocss: ${survivors.length} call(s) could not be folded, and \`failOnUnfolded\` is on.\n\n${detail}\n\n` + (threw ? "`fold-failed` is a module the fold threw on — see the error logged for it above. Nothing was established about its calls either way, so it cannot support the guarantee this option makes.\n\n" : "") + "Each one keeps `styled-system/css` in the bundle, so the engine cannot be dropped however many other calls folded. Make the values static, move the variation into a `cva` variant, or generate them with `staticCss` — or set `failOnUnfolded: false` to accept the runtime.");
 			}
 			if (!transform || !reportSummary) return;
 			const declined = Array.from(totals.skipped.values()).reduce((sum, count) => sum + count, 0);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bamboocss/vite",
-  "version": "1.30.1",
+  "version": "1.32.0",
   "description": "Vite integration for Bamboo CSS",
   "homepage": "https://bamboocss.com",
   "license": "MIT",
@@ -37,18 +37,18 @@
   "dependencies": {
     "magic-string": "0.30.21",
     "ts-morph": "28.0.0",
-    "@bamboocss/config": "1.30.1",
-    "@bamboocss/core": "1.30.1",
-    "@bamboocss/node": "1.30.1",
-    "@bamboocss/logger": "1.30.1",
-    "@bamboocss/extractor": "1.30.1",
-    "@bamboocss/shared": "1.30.1",
-    "@bamboocss/types": "1.30.1"
+    "@bamboocss/core": "1.32.0",
+    "@bamboocss/config": "1.32.0",
+    "@bamboocss/extractor": "1.32.0",
+    "@bamboocss/node": "1.32.0",
+    "@bamboocss/logger": "1.32.0",
+    "@bamboocss/shared": "1.32.0",
+    "@bamboocss/types": "1.32.0"
   },
   "devDependencies": {
     "@jridgewell/trace-mapping": "^0.3.31",
     "vite": "7.2.6",
-    "@bamboocss/fixture": "1.30.1"
+    "@bamboocss/fixture": "1.32.0"
   },
   "peerDependencies": {
     "vite": ">=5"