@bamboocss/vite 1.37.8 → 1.37.10

package/dist/index.cjs

@@ -80,20 +80,31 @@ const pruneStaticCss = (css, session, { prune = true } = {}) => {
 	};
 	if (prune) root.walkRules((rule) => {
 		if (!isUtilityRule(rule)) return;
-		const classes = /* @__PURE__ */ new Set();
+		let removedAny = false;
+		let selector;
 		try {
-			(0, postcss_selector_parser.default)((selectors) => {
-				selectors.walkClasses((classNode) => {
-					classes.add(bare(classNode.toString().slice(1)));
+			selector = (0, postcss_selector_parser.default)((selectors) => {
+				selectors.each((candidate) => {
+					const classes = /* @__PURE__ */ new Set();
+					candidate.walkClasses((classNode) => {
+						classes.add(bare(classNode.toString().slice(1)));
+					});
+					if (classes.size !== 1) return;
+					const [className] = classes;
+					if (!className || !prunable.has(className) || used.has(className)) return;
+					candidate.remove();
+					removedAny = true;
 				});
 			}).processSync(rule.selector);
 		} catch {
 			return;
 		}
-		if (classes.size !== 1) return;
-		const [className] = classes;
-		if (!className || !prunable.has(className) || used.has(className)) return;
-		rule.remove();
+		if (!removedAny) return;
+		if (!selector.trim()) {
+			rule.remove();
+			return;
+		}
+		rule.selector = selector;
 	});
 	let removed = true;
 	while (removed) {
@@ -740,10 +751,28 @@ const collectModuleScopeNames = (sourceFile) => {
 * while under-reporting ships an element whose class has no rule behind it and says nothing.
 * Shadowing a recipe binding is rare; silently shipping unstyled markup is not recoverable.
 */
+const identifiersByName = /* @__PURE__ */ new WeakMap();
+/**
+* Every identifier in the module, grouped by the name it spells.
+*
+* Walking the whole tree per binding made this O(bindings x identifiers): a module declaring
+* ten recipes walked its identifiers ten times. One walk answers for all of them, and the
+* result is cached against the source text like the module-scope names beside it, so a
+* re-transform of unchanged text reuses it.
+*/
+const identifierIndex = (sourceFile) => byText(identifiersByName, sourceFile, () => {
+	const index = /* @__PURE__ */ new Map();
+	for (const identifier of sourceFile.getDescendantsOfKind(ts_morph.SyntaxKind.Identifier)) {
+		const text = identifier.getText();
+		const known = index.get(text);
+		if (known) known.push(identifier);
+		else index.set(text, [identifier]);
+	}
+	return index;
+});
 const localReferencesTo = (sourceFile, name, declaration) => {
 	const references = [];
-	for (const identifier of sourceFile.getDescendantsOfKind(ts_morph.SyntaxKind.Identifier)) {
-		if (identifier.getText() !== name) continue;
+	for (const identifier of identifierIndex(sourceFile).get(name) ?? []) {
 		if (identifier === declaration) continue;
 		const parent = identifier.getParent();
 		if (!parent) continue;
@@ -2828,18 +2857,24 @@ const bamboocss = (options = {}) => {
 		skipped: /* @__PURE__ */ new Map()
 	};
 	const staticSession = createStaticCompilationSession();
-	const survivors = [];
-	const survivorKeys = /* @__PURE__ */ new Set();
+	/**
+	* Indexed by file, because the only bulk operation on it is "forget this one's".
+	*
+	* A flat array meant every transform scanned every survivor and then rebuilt the dedupe key
+	* set from scratch — O(modules x survivors) across a build, and worst exactly when a build is
+	* already failing and the user is iterating on it. One project had 736 of them across 9,461
+	* modules, which is seven million string builds to discard.
+	*/
+	const survivorsByFile = /* @__PURE__ */ new Map();
+	const allSurvivors = () => [...survivorsByFile.values()].flat();
 	const addSurvivor = (entry) => {
-		const key = `${entry.file}:${entry.line}:${entry.name}:${entry.reason}`;
-		if (survivorKeys.has(key)) return;
-		survivorKeys.add(key);
-		survivors.push(entry);
+		const forFile = survivorsByFile.get(entry.file) ?? [];
+		if (forFile.some((seen) => seen.line === entry.line && seen.name === entry.name && seen.reason === entry.reason)) return;
+		forFile.push(entry);
+		survivorsByFile.set(entry.file, forFile);
 	};
 	const clearSurvivorsFor = (file) => {
-		for (let index = survivors.length - 1; index >= 0; index--) if (survivors[index]?.file === file) survivors.splice(index, 1);
-		survivorKeys.clear();
-		for (const entry of survivors) survivorKeys.add(`${entry.file}:${entry.line}:${entry.name}:${entry.reason}`);
+		survivorsByFile.delete(file);
 	};
 	const createSurvivorError = (entries) => {
 		const byFile = /* @__PURE__ */ new Map();
@@ -2898,8 +2933,7 @@ const bamboocss = (options = {}) => {
 				totals.files = 0;
 				totals.filesWithFolds = 0;
 				totals.skipped.clear();
-				survivors.length = 0;
-				survivorKeys.clear();
+				survivorsByFile.clear();
 				recipeConfigCache.clear();
 				resetStaticCompilationSession(staticSession);
 			}
@@ -3002,7 +3036,8 @@ const bamboocss = (options = {}) => {
 			}
 			if (reportSkipped && result.skipped.length) _bamboocss_logger.logger.info("vite:transform", formatSkipped(filePath, result.skipped));
 			for (const dependency of result.dependencies) this.addWatchFile?.(dependency);
-			if (command === "serve" && survivors.some((entry) => entry.file === filePath)) throw createSurvivorError(survivors.filter((entry) => entry.file === filePath));
+			const forFile = survivorsByFile.get(filePath);
+			if (command === "serve" && forFile?.length) throw createSurvivorError(forFile);
 			if (!result.folded.length) return null;
 			_bamboocss_logger.logger.debug("vite:transform", `Compiled ${result.folded.length} call(s) in ${filePath}`);
 			return {
@@ -3011,6 +3046,7 @@ const bamboocss = (options = {}) => {
 			};
 		},
 		buildEnd() {
+			const survivors = allSurvivors();
 			if (survivors.length) throw createSurvivorError(survivors);
 			if (typeof this.getModuleInfo === "function") {
 				if (!staticSession.cssLoaded) throw new Error(`bamboocss: compiled class values were produced, but ${JSON.stringify(VIRTUAL_CSS_ID)} was not imported. Add \`import ${JSON.stringify(VIRTUAL_CSS_ID)}\` once, from a JavaScript or TypeScript module in the application entry graph.\n\nIt has to be a JS import. \`@import\` from a stylesheet does not reach it: the id names a virtual module resolved by this plugin, and Vite resolves CSS \`@import\` before plugin resolution, so it fails as an unresolvable path. A project that ships one preloaded stylesheet imports this from its entry module instead, and lets Vite emit the CSS asset.`);

package/dist/index.mjs

@@ -50,20 +50,31 @@ const pruneStaticCss = (css, session, { prune = true } = {}) => {
 	};
 	if (prune) root.walkRules((rule) => {
 		if (!isUtilityRule(rule)) return;
-		const classes = /* @__PURE__ */ new Set();
+		let removedAny = false;
+		let selector;
 		try {
-			selectorParser((selectors) => {
-				selectors.walkClasses((classNode) => {
-					classes.add(bare(classNode.toString().slice(1)));
+			selector = selectorParser((selectors) => {
+				selectors.each((candidate) => {
+					const classes = /* @__PURE__ */ new Set();
+					candidate.walkClasses((classNode) => {
+						classes.add(bare(classNode.toString().slice(1)));
+					});
+					if (classes.size !== 1) return;
+					const [className] = classes;
+					if (!className || !prunable.has(className) || used.has(className)) return;
+					candidate.remove();
+					removedAny = true;
 				});
 			}).processSync(rule.selector);
 		} catch {
 			return;
 		}
-		if (classes.size !== 1) return;
-		const [className] = classes;
-		if (!className || !prunable.has(className) || used.has(className)) return;
-		rule.remove();
+		if (!removedAny) return;
+		if (!selector.trim()) {
+			rule.remove();
+			return;
+		}
+		rule.selector = selector;
 	});
 	let removed = true;
 	while (removed) {
@@ -710,10 +721,28 @@ const collectModuleScopeNames = (sourceFile) => {
 * while under-reporting ships an element whose class has no rule behind it and says nothing.
 * Shadowing a recipe binding is rare; silently shipping unstyled markup is not recoverable.
 */
+const identifiersByName = /* @__PURE__ */ new WeakMap();
+/**
+* Every identifier in the module, grouped by the name it spells.
+*
+* Walking the whole tree per binding made this O(bindings x identifiers): a module declaring
+* ten recipes walked its identifiers ten times. One walk answers for all of them, and the
+* result is cached against the source text like the module-scope names beside it, so a
+* re-transform of unchanged text reuses it.
+*/
+const identifierIndex = (sourceFile) => byText(identifiersByName, sourceFile, () => {
+	const index = /* @__PURE__ */ new Map();
+	for (const identifier of sourceFile.getDescendantsOfKind(SyntaxKind.Identifier)) {
+		const text = identifier.getText();
+		const known = index.get(text);
+		if (known) known.push(identifier);
+		else index.set(text, [identifier]);
+	}
+	return index;
+});
 const localReferencesTo = (sourceFile, name, declaration) => {
 	const references = [];
-	for (const identifier of sourceFile.getDescendantsOfKind(SyntaxKind.Identifier)) {
-		if (identifier.getText() !== name) continue;
+	for (const identifier of identifierIndex(sourceFile).get(name) ?? []) {
 		if (identifier === declaration) continue;
 		const parent = identifier.getParent();
 		if (!parent) continue;
@@ -2798,18 +2827,24 @@ const bamboocss = (options = {}) => {
 		skipped: /* @__PURE__ */ new Map()
 	};
 	const staticSession = createStaticCompilationSession();
-	const survivors = [];
-	const survivorKeys = /* @__PURE__ */ new Set();
+	/**
+	* Indexed by file, because the only bulk operation on it is "forget this one's".
+	*
+	* A flat array meant every transform scanned every survivor and then rebuilt the dedupe key
+	* set from scratch — O(modules x survivors) across a build, and worst exactly when a build is
+	* already failing and the user is iterating on it. One project had 736 of them across 9,461
+	* modules, which is seven million string builds to discard.
+	*/
+	const survivorsByFile = /* @__PURE__ */ new Map();
+	const allSurvivors = () => [...survivorsByFile.values()].flat();
 	const addSurvivor = (entry) => {
-		const key = `${entry.file}:${entry.line}:${entry.name}:${entry.reason}`;
-		if (survivorKeys.has(key)) return;
-		survivorKeys.add(key);
-		survivors.push(entry);
+		const forFile = survivorsByFile.get(entry.file) ?? [];
+		if (forFile.some((seen) => seen.line === entry.line && seen.name === entry.name && seen.reason === entry.reason)) return;
+		forFile.push(entry);
+		survivorsByFile.set(entry.file, forFile);
 	};
 	const clearSurvivorsFor = (file) => {
-		for (let index = survivors.length - 1; index >= 0; index--) if (survivors[index]?.file === file) survivors.splice(index, 1);
-		survivorKeys.clear();
-		for (const entry of survivors) survivorKeys.add(`${entry.file}:${entry.line}:${entry.name}:${entry.reason}`);
+		survivorsByFile.delete(file);
 	};
 	const createSurvivorError = (entries) => {
 		const byFile = /* @__PURE__ */ new Map();
@@ -2868,8 +2903,7 @@ const bamboocss = (options = {}) => {
 				totals.files = 0;
 				totals.filesWithFolds = 0;
 				totals.skipped.clear();
-				survivors.length = 0;
-				survivorKeys.clear();
+				survivorsByFile.clear();
 				recipeConfigCache.clear();
 				resetStaticCompilationSession(staticSession);
 			}
@@ -2972,7 +3006,8 @@ const bamboocss = (options = {}) => {
 			}
 			if (reportSkipped && result.skipped.length) logger.info("vite:transform", formatSkipped(filePath, result.skipped));
 			for (const dependency of result.dependencies) this.addWatchFile?.(dependency);
-			if (command === "serve" && survivors.some((entry) => entry.file === filePath)) throw createSurvivorError(survivors.filter((entry) => entry.file === filePath));
+			const forFile = survivorsByFile.get(filePath);
+			if (command === "serve" && forFile?.length) throw createSurvivorError(forFile);
 			if (!result.folded.length) return null;
 			logger.debug("vite:transform", `Compiled ${result.folded.length} call(s) in ${filePath}`);
 			return {
@@ -2981,6 +3016,7 @@ const bamboocss = (options = {}) => {
 			};
 		},
 		buildEnd() {
+			const survivors = allSurvivors();
 			if (survivors.length) throw createSurvivorError(survivors);
 			if (typeof this.getModuleInfo === "function") {
 				if (!staticSession.cssLoaded) throw new Error(`bamboocss: compiled class values were produced, but ${JSON.stringify(VIRTUAL_CSS_ID)} was not imported. Add \`import ${JSON.stringify(VIRTUAL_CSS_ID)}\` once, from a JavaScript or TypeScript module in the application entry graph.\n\nIt has to be a JS import. \`@import\` from a stylesheet does not reach it: the id names a virtual module resolved by this plugin, and Vite resolves CSS \`@import\` before plugin resolution, so it fails as an unresolvable path. A project that ships one preloaded stylesheet imports this from its entry module instead, and lets Vite emit the CSS asset.`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bamboocss/vite",
-  "version": "1.37.8",
+  "version": "1.37.10",
   "description": "Vite integration for Bamboo CSS",
   "homepage": "https://bamboocss.com",
   "license": "MIT",
@@ -40,18 +40,18 @@
     "postcss": "8.5.26",
     "postcss-selector-parser": "7.1.5",
     "ts-morph": "28.0.0",
-    "@bamboocss/config": "1.37.8",
-    "@bamboocss/core": "1.37.8",
-    "@bamboocss/extractor": "1.37.8",
-    "@bamboocss/logger": "1.37.8",
-    "@bamboocss/node": "1.37.8",
-    "@bamboocss/shared": "1.37.8",
-    "@bamboocss/types": "1.37.8"
+    "@bamboocss/config": "1.37.10",
+    "@bamboocss/core": "1.37.10",
+    "@bamboocss/extractor": "1.37.10",
+    "@bamboocss/node": "1.37.10",
+    "@bamboocss/logger": "1.37.10",
+    "@bamboocss/types": "1.37.10",
+    "@bamboocss/shared": "1.37.10"
   },
   "devDependencies": {
     "@jridgewell/trace-mapping": "^0.3.31",
     "vite": "7.2.6",
-    "@bamboocss/fixture": "1.37.8"
+    "@bamboocss/fixture": "1.37.10"
   },
   "peerDependencies": {
     "vite": ">=5"