@bamboocss/vite 1.30.1 → 1.31.0

package/dist/index.cjs

@@ -1271,7 +1271,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",
@@ -1333,7 +1333,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 +1356,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 +2269,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 +2431,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 +2439,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 +2521,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,7 +2531,7 @@ const bamboocss = (options = {}) => {
 					partial,
 					parseModule: (path) => ctx?.project.parseSourceFile(path),
 					recipeConfigCache,
-					reportSurvivors: strict,
+					reportSurvivors: failOnUnfolded,
 					sourceFile
 				});
 			} catch (error) {
@@ -2541,7 +2542,7 @@ const bamboocss = (options = {}) => {
 			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,7 +2569,7 @@ 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) ?? [];
@@ -2576,7 +2577,7 @@ const bamboocss = (options = {}) => {
 					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.`);
+				throw new Error(`bamboocss: ${survivors.length} call(s) could not be folded, and \`failOnUnfolded\` 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 \`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

@@ -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

@@ -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

@@ -1244,7 +1244,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",
@@ -1306,7 +1306,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 +1329,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 +2242,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 +2404,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 +2412,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 +2494,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,7 +2504,7 @@ const bamboocss = (options = {}) => {
 					partial,
 					parseModule: (path) => ctx?.project.parseSourceFile(path),
 					recipeConfigCache,
-					reportSurvivors: strict,
+					reportSurvivors: failOnUnfolded,
 					sourceFile
 				});
 			} catch (error) {
@@ -2514,7 +2515,7 @@ const bamboocss = (options = {}) => {
 			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,7 +2542,7 @@ 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) ?? [];
@@ -2549,7 +2550,7 @@ const bamboocss = (options = {}) => {
 					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.`);
+				throw new Error(`bamboocss: ${survivors.length} call(s) could not be folded, and \`failOnUnfolded\` 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 \`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.31.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/config": "1.31.0",
+    "@bamboocss/extractor": "1.31.0",
+    "@bamboocss/node": "1.31.0",
+    "@bamboocss/logger": "1.31.0",
+    "@bamboocss/core": "1.31.0",
+    "@bamboocss/shared": "1.31.0",
+    "@bamboocss/types": "1.31.0"
   },
   "devDependencies": {
     "@jridgewell/trace-mapping": "^0.3.31",
     "vite": "7.2.6",
-    "@bamboocss/fixture": "1.30.1"
+    "@bamboocss/fixture": "1.31.0"
   },
   "peerDependencies": {
     "vite": ">=5"