@bamboocss/vite 1.37.9 → 1.37.11

package/dist/index.cjs

@@ -751,10 +751,33 @@ 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 localReferencesTo = (sourceFile, name, declaration) => {
-	const references = [];
+/**
+* Every identifier in the module, grouped by the name it spells.
+*
+* Built once per pass and handed to each lookup, because walking the whole tree per binding
+* made that O(bindings x identifiers): a module declaring ten recipes walked its identifiers
+* ten times.
+*
+* Deliberately *not* memoized across passes, unlike the module-scope names beside it. That
+* cache holds strings, which outlive anything; this one holds nodes, and a node does not
+* survive its source file being replaced — `addSourceFile` overwrites, which forgets every
+* node previously taken from it. Keying on the source text does not help, because identical
+* text re-parsed is a fresh tree: the cache hits and returns nodes that throw
+* `Attempted to get information from a node that was removed or forgotten` on the next read.
+*/
+const identifierIndex = (sourceFile) => {
+	const index = /* @__PURE__ */ new Map();
 	for (const identifier of sourceFile.getDescendantsOfKind(ts_morph.SyntaxKind.Identifier)) {
-		if (identifier.getText() !== name) continue;
+		const text = identifier.getText();
+		const known = index.get(text);
+		if (known) known.push(identifier);
+		else index.set(text, [identifier]);
+	}
+	return index;
+};
+const localReferencesTo = (index, name, declaration) => {
+	const references = [];
+	for (const identifier of index.get(name) ?? []) {
 		if (identifier === declaration) continue;
 		const parent = identifier.getParent();
 		if (!parent) continue;
@@ -2539,6 +2562,7 @@ const foldSource = (options) => {
 		const sourceFile = ownSourceFile ?? candidates[0]?.node.getSourceFile() ?? recipeDefinitions[0]?.call.getSourceFile();
 		if (!sourceFile) return;
 		const importedRecipeBindings = new Set(parserResult.importedRecipes?.keys() ?? []);
+		const identifiers = identifierIndex(sourceFile);
 		for (const binding of new Set([...recipeConfigs.keys(), ...importedRecipeBindings])) {
 			const entry = recipeConfigs.get(binding);
 			if (entry === AMBIGUOUS) continue;
@@ -2546,7 +2570,7 @@ const foldSource = (options) => {
 			const definition = entry?.box?.getNode?.();
 			const nameNode = definition?.getSourceFile() === sourceFile ? definition?.getFirstAncestorByKind(ts_morph.SyntaxKind.VariableDeclaration)?.getNameNode() : importSpecifierFor(sourceFile, binding);
 			if (!nameNode || !ts_morph.Node.isIdentifier(nameNode)) continue;
-			const references = localReferencesTo(sourceFile, binding, nameNode);
+			const references = localReferencesTo(identifiers, binding, nameNode);
 			if (skipped.filter((item) => SURVIVES_TO_RUNTIME.has(item.reason) && item.end > item.start).some((item) => references.some((ref) => ref.getStart() >= item.start && ref.getStart() < item.end))) continue;
 			const survivor = references.find((ref) => !applied.some(([from, to]) => ref.getStart() >= from && ref.getStart() < to));
 			if (!survivor) continue;
@@ -2839,18 +2863,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();
@@ -2909,8 +2939,7 @@ const bamboocss = (options = {}) => {
 				totals.files = 0;
 				totals.filesWithFolds = 0;
 				totals.skipped.clear();
-				survivors.length = 0;
-				survivorKeys.clear();
+				survivorsByFile.clear();
 				recipeConfigCache.clear();
 				resetStaticCompilationSession(staticSession);
 			}
@@ -3013,7 +3042,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 {
@@ -3022,6 +3052,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

@@ -721,10 +721,33 @@ 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 localReferencesTo = (sourceFile, name, declaration) => {
-	const references = [];
+/**
+* Every identifier in the module, grouped by the name it spells.
+*
+* Built once per pass and handed to each lookup, because walking the whole tree per binding
+* made that O(bindings x identifiers): a module declaring ten recipes walked its identifiers
+* ten times.
+*
+* Deliberately *not* memoized across passes, unlike the module-scope names beside it. That
+* cache holds strings, which outlive anything; this one holds nodes, and a node does not
+* survive its source file being replaced — `addSourceFile` overwrites, which forgets every
+* node previously taken from it. Keying on the source text does not help, because identical
+* text re-parsed is a fresh tree: the cache hits and returns nodes that throw
+* `Attempted to get information from a node that was removed or forgotten` on the next read.
+*/
+const identifierIndex = (sourceFile) => {
+	const index = /* @__PURE__ */ new Map();
 	for (const identifier of sourceFile.getDescendantsOfKind(SyntaxKind.Identifier)) {
-		if (identifier.getText() !== name) continue;
+		const text = identifier.getText();
+		const known = index.get(text);
+		if (known) known.push(identifier);
+		else index.set(text, [identifier]);
+	}
+	return index;
+};
+const localReferencesTo = (index, name, declaration) => {
+	const references = [];
+	for (const identifier of index.get(name) ?? []) {
 		if (identifier === declaration) continue;
 		const parent = identifier.getParent();
 		if (!parent) continue;
@@ -2509,6 +2532,7 @@ const foldSource = (options) => {
 		const sourceFile = ownSourceFile ?? candidates[0]?.node.getSourceFile() ?? recipeDefinitions[0]?.call.getSourceFile();
 		if (!sourceFile) return;
 		const importedRecipeBindings = new Set(parserResult.importedRecipes?.keys() ?? []);
+		const identifiers = identifierIndex(sourceFile);
 		for (const binding of new Set([...recipeConfigs.keys(), ...importedRecipeBindings])) {
 			const entry = recipeConfigs.get(binding);
 			if (entry === AMBIGUOUS) continue;
@@ -2516,7 +2540,7 @@ const foldSource = (options) => {
 			const definition = entry?.box?.getNode?.();
 			const nameNode = definition?.getSourceFile() === sourceFile ? definition?.getFirstAncestorByKind(SyntaxKind.VariableDeclaration)?.getNameNode() : importSpecifierFor(sourceFile, binding);
 			if (!nameNode || !Node.isIdentifier(nameNode)) continue;
-			const references = localReferencesTo(sourceFile, binding, nameNode);
+			const references = localReferencesTo(identifiers, binding, nameNode);
 			if (skipped.filter((item) => SURVIVES_TO_RUNTIME.has(item.reason) && item.end > item.start).some((item) => references.some((ref) => ref.getStart() >= item.start && ref.getStart() < item.end))) continue;
 			const survivor = references.find((ref) => !applied.some(([from, to]) => ref.getStart() >= from && ref.getStart() < to));
 			if (!survivor) continue;
@@ -2809,18 +2833,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();
@@ -2879,8 +2909,7 @@ const bamboocss = (options = {}) => {
 				totals.files = 0;
 				totals.filesWithFolds = 0;
 				totals.skipped.clear();
-				survivors.length = 0;
-				survivorKeys.clear();
+				survivorsByFile.clear();
 				recipeConfigCache.clear();
 				resetStaticCompilationSession(staticSession);
 			}
@@ -2983,7 +3012,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 {
@@ -2992,6 +3022,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.9",
+  "version": "1.37.11",
   "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.9",
-    "@bamboocss/core": "1.37.9",
-    "@bamboocss/extractor": "1.37.9",
-    "@bamboocss/logger": "1.37.9",
-    "@bamboocss/node": "1.37.9",
-    "@bamboocss/types": "1.37.9",
-    "@bamboocss/shared": "1.37.9"
+    "@bamboocss/logger": "1.37.11",
+    "@bamboocss/node": "1.37.11",
+    "@bamboocss/extractor": "1.37.11",
+    "@bamboocss/core": "1.37.11",
+    "@bamboocss/config": "1.37.11",
+    "@bamboocss/shared": "1.37.11",
+    "@bamboocss/types": "1.37.11"
   },
   "devDependencies": {
     "@jridgewell/trace-mapping": "^0.3.31",
     "vite": "7.2.6",
-    "@bamboocss/fixture": "1.37.9"
+    "@bamboocss/fixture": "1.37.11"
   },
   "peerDependencies": {
     "vite": ">=5"