@bamboocss/vite 1.35.5 → 1.36.0

package/dist/index.cjs

@@ -635,6 +635,54 @@ const collectModuleScopeNames = (sourceFile) => {
 	}
 	return names;
 };
+/**
+* Every identifier in this module that reads the binding `name` declared at `declaration`.
+*
+* Replaces `nameNode.findReferencesAsNodes()`, which is a TypeScript *language-service*
+* query. The first such query forces `synchronizeHostData` -> `createProgram`, binding the
+* whole transitive `.d.ts` closure of the project — the exact cost `createTsProject`'s
+* `skipAddingFilesFromTsConfig`, `skipFileDependencyResolution` and `skipLoadingLibFiles`
+* exist to avoid, and which none of them govern. On a 2,278-file app it loaded 24,081 source
+* files and 4.4 GB of AST and symbols, 80% of the heap, and OOMed the build. The retained
+* strings were `googleapis` and `typescript`; none of it can reference a recipe binding.
+*
+* A recipe binding is module-scoped or imported, so every read of it is in this file. A
+* syntactic walk answers the same question over one AST the parser has already built.
+*
+* ## Where this deliberately over-reports
+*
+* Shadowing is not resolved. If any nested scope declares the same name, every matching
+* identifier is returned rather than only the ones that bind to `declaration`. That direction
+* is chosen on purpose: over-reporting fails the build with a diagnostic naming a real line,
+* 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 = [];
+	for (const identifier of sourceFile.getDescendantsOfKind(ts_morph.SyntaxKind.Identifier)) {
+		if (identifier.getText() !== name) continue;
+		if (identifier === declaration) continue;
+		const parent = identifier.getParent();
+		if (!parent) continue;
+		if (ts_morph.Node.isPropertyAccessExpression(parent) && parent.getNameNode() === identifier) continue;
+		if (ts_morph.Node.isPropertyAssignment(parent) && parent.getNameNode() === identifier) continue;
+		if (ts_morph.Node.isBindingElement(parent) && parent.getPropertyNameNode() === identifier) continue;
+		if ((ts_morph.Node.isJsxOpeningElement(parent) || ts_morph.Node.isJsxSelfClosingElement(parent) || ts_morph.Node.isJsxClosingElement(parent)) && parent.getTagNameNode() === identifier) continue;
+		if (ts_morph.Node.isJsxAttribute(parent) && parent.getNameNode() === identifier) continue;
+		if ((ts_morph.Node.isMethodDeclaration(parent) || ts_morph.Node.isPropertyDeclaration(parent) || ts_morph.Node.isGetAccessorDeclaration(parent) || ts_morph.Node.isSetAccessorDeclaration(parent) || ts_morph.Node.isMethodSignature(parent) || ts_morph.Node.isPropertySignature(parent) || ts_morph.Node.isEnumMember(parent)) && parent.getNameNode() === identifier) continue;
+		if ((ts_morph.Node.isVariableDeclaration(parent) || ts_morph.Node.isParameterDeclaration(parent) || ts_morph.Node.isFunctionDeclaration(parent) || ts_morph.Node.isClassDeclaration(parent) || ts_morph.Node.isBindingElement(parent)) && parent.getNameNode() === identifier) continue;
+		if (ts_morph.Node.isImportSpecifier(parent) || ts_morph.Node.isExportSpecifier(parent)) {
+			if (parent.getNameNode() !== identifier) continue;
+			if (ts_morph.Node.isExportSpecifier(parent)) {
+				references.push(identifier);
+				continue;
+			}
+			continue;
+		}
+		references.push(identifier);
+	}
+	return references;
+};
 //#endregion
 //#region src/fold-recipe.ts
 /**
@@ -2383,30 +2431,36 @@ const foldSource = (options) => {
 	* also allowed to remain when it joins an arbitrary external class; only fully analyzable
 	* arguments receive Bamboo's semantic composition guarantee.
 	*/
+	/** The specifier this module imported `binding` through, if it did. */
+	function importSpecifierFor(sourceFile, binding) {
+		for (const declaration of sourceFile.getImportDeclarations()) {
+			if (declaration.isTypeOnly()) continue;
+			for (const named of declaration.getNamedImports()) {
+				if (named.isTypeOnly()) continue;
+				if ((named.getAliasNode() ?? named.getNameNode()).getText() === binding) return named.getAliasNode() ?? named.getNameNode();
+			}
+		}
+	}
 	function reportRuntimeBindings() {
 		const sourceFile = ownSourceFile ?? candidates[0]?.node.getSourceFile() ?? recipeDefinitions[0]?.call.getSourceFile();
 		if (!sourceFile) return;
-		for (const [binding, entry] of recipeConfigs) {
+		const importedRecipeBindings = new Set(parserResult.importedRecipes?.keys() ?? []);
+		for (const binding of new Set([...recipeConfigs.keys(), ...importedRecipeBindings])) {
+			const entry = recipeConfigs.get(binding);
 			if (entry === AMBIGUOUS) continue;
-			const definition = entry.box?.getNode?.();
-			if (!definition || definition.getSourceFile() !== sourceFile) continue;
-			const nameNode = definition.getFirstAncestorByKind(ts_morph.SyntaxKind.VariableDeclaration)?.getNameNode();
+			if (!entry && !importedRecipeBindings.has(binding)) continue;
+			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 = nameNode.findReferencesAsNodes();
+			const references = localReferencesTo(sourceFile, 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;
-			const survivorFile = survivor.getSourceFile();
-			const foreign = survivorFile !== sourceFile;
 			skipped.push({
 				name: binding,
 				reason: "runtime-binding",
-				start: foreign ? nameNode.getStart() : survivor.getStart(),
-				end: foreign ? nameNode.getEnd() : survivor.getEnd(),
-				...foreign ? { origin: {
-					filePath: survivorFile.getFilePath(),
-					line: survivor.getStartLineNumber()
-				} } : {}
+				start: survivor.getStart(),
+				end: survivor.getEnd()
 			});
 		}
 		const bambooModules = [
@@ -2870,12 +2924,7 @@ const bamboocss = (options = {}) => {
 			for (const entry of result.skipped) {
 				if (entry.reason === "not-imported" || entry.reason === "overlapping") continue;
 				if (entry.name === "cx" && entry.reason === "dynamic") continue;
-				addSurvivor(entry.origin ? {
-					file: entry.origin.filePath,
-					line: entry.origin.line,
-					name: entry.name,
-					reason: entry.reason
-				} : {
+				addSurvivor({
 					file: filePath,
 					line: lineAt(code, entry.start),
 					name: entry.name,

package/dist/index.mjs

@@ -605,6 +605,54 @@ const collectModuleScopeNames = (sourceFile) => {
 	}
 	return names;
 };
+/**
+* Every identifier in this module that reads the binding `name` declared at `declaration`.
+*
+* Replaces `nameNode.findReferencesAsNodes()`, which is a TypeScript *language-service*
+* query. The first such query forces `synchronizeHostData` -> `createProgram`, binding the
+* whole transitive `.d.ts` closure of the project — the exact cost `createTsProject`'s
+* `skipAddingFilesFromTsConfig`, `skipFileDependencyResolution` and `skipLoadingLibFiles`
+* exist to avoid, and which none of them govern. On a 2,278-file app it loaded 24,081 source
+* files and 4.4 GB of AST and symbols, 80% of the heap, and OOMed the build. The retained
+* strings were `googleapis` and `typescript`; none of it can reference a recipe binding.
+*
+* A recipe binding is module-scoped or imported, so every read of it is in this file. A
+* syntactic walk answers the same question over one AST the parser has already built.
+*
+* ## Where this deliberately over-reports
+*
+* Shadowing is not resolved. If any nested scope declares the same name, every matching
+* identifier is returned rather than only the ones that bind to `declaration`. That direction
+* is chosen on purpose: over-reporting fails the build with a diagnostic naming a real line,
+* 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 = [];
+	for (const identifier of sourceFile.getDescendantsOfKind(SyntaxKind.Identifier)) {
+		if (identifier.getText() !== name) continue;
+		if (identifier === declaration) continue;
+		const parent = identifier.getParent();
+		if (!parent) continue;
+		if (Node.isPropertyAccessExpression(parent) && parent.getNameNode() === identifier) continue;
+		if (Node.isPropertyAssignment(parent) && parent.getNameNode() === identifier) continue;
+		if (Node.isBindingElement(parent) && parent.getPropertyNameNode() === identifier) continue;
+		if ((Node.isJsxOpeningElement(parent) || Node.isJsxSelfClosingElement(parent) || Node.isJsxClosingElement(parent)) && parent.getTagNameNode() === identifier) continue;
+		if (Node.isJsxAttribute(parent) && parent.getNameNode() === identifier) continue;
+		if ((Node.isMethodDeclaration(parent) || Node.isPropertyDeclaration(parent) || Node.isGetAccessorDeclaration(parent) || Node.isSetAccessorDeclaration(parent) || Node.isMethodSignature(parent) || Node.isPropertySignature(parent) || Node.isEnumMember(parent)) && parent.getNameNode() === identifier) continue;
+		if ((Node.isVariableDeclaration(parent) || Node.isParameterDeclaration(parent) || Node.isFunctionDeclaration(parent) || Node.isClassDeclaration(parent) || Node.isBindingElement(parent)) && parent.getNameNode() === identifier) continue;
+		if (Node.isImportSpecifier(parent) || Node.isExportSpecifier(parent)) {
+			if (parent.getNameNode() !== identifier) continue;
+			if (Node.isExportSpecifier(parent)) {
+				references.push(identifier);
+				continue;
+			}
+			continue;
+		}
+		references.push(identifier);
+	}
+	return references;
+};
 //#endregion
 //#region src/fold-recipe.ts
 /**
@@ -2353,30 +2401,36 @@ const foldSource = (options) => {
 	* also allowed to remain when it joins an arbitrary external class; only fully analyzable
 	* arguments receive Bamboo's semantic composition guarantee.
 	*/
+	/** The specifier this module imported `binding` through, if it did. */
+	function importSpecifierFor(sourceFile, binding) {
+		for (const declaration of sourceFile.getImportDeclarations()) {
+			if (declaration.isTypeOnly()) continue;
+			for (const named of declaration.getNamedImports()) {
+				if (named.isTypeOnly()) continue;
+				if ((named.getAliasNode() ?? named.getNameNode()).getText() === binding) return named.getAliasNode() ?? named.getNameNode();
+			}
+		}
+	}
 	function reportRuntimeBindings() {
 		const sourceFile = ownSourceFile ?? candidates[0]?.node.getSourceFile() ?? recipeDefinitions[0]?.call.getSourceFile();
 		if (!sourceFile) return;
-		for (const [binding, entry] of recipeConfigs) {
+		const importedRecipeBindings = new Set(parserResult.importedRecipes?.keys() ?? []);
+		for (const binding of new Set([...recipeConfigs.keys(), ...importedRecipeBindings])) {
+			const entry = recipeConfigs.get(binding);
 			if (entry === AMBIGUOUS) continue;
-			const definition = entry.box?.getNode?.();
-			if (!definition || definition.getSourceFile() !== sourceFile) continue;
-			const nameNode = definition.getFirstAncestorByKind(SyntaxKind.VariableDeclaration)?.getNameNode();
+			if (!entry && !importedRecipeBindings.has(binding)) continue;
+			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 = nameNode.findReferencesAsNodes();
+			const references = localReferencesTo(sourceFile, 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;
-			const survivorFile = survivor.getSourceFile();
-			const foreign = survivorFile !== sourceFile;
 			skipped.push({
 				name: binding,
 				reason: "runtime-binding",
-				start: foreign ? nameNode.getStart() : survivor.getStart(),
-				end: foreign ? nameNode.getEnd() : survivor.getEnd(),
-				...foreign ? { origin: {
-					filePath: survivorFile.getFilePath(),
-					line: survivor.getStartLineNumber()
-				} } : {}
+				start: survivor.getStart(),
+				end: survivor.getEnd()
 			});
 		}
 		const bambooModules = [
@@ -2840,12 +2894,7 @@ const bamboocss = (options = {}) => {
 			for (const entry of result.skipped) {
 				if (entry.reason === "not-imported" || entry.reason === "overlapping") continue;
 				if (entry.name === "cx" && entry.reason === "dynamic") continue;
-				addSurvivor(entry.origin ? {
-					file: entry.origin.filePath,
-					line: entry.origin.line,
-					name: entry.name,
-					reason: entry.reason
-				} : {
+				addSurvivor({
 					file: filePath,
 					line: lineAt(code, entry.start),
 					name: entry.name,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bamboocss/vite",
-  "version": "1.35.5",
+  "version": "1.36.0",
   "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.35.5",
-    "@bamboocss/core": "1.35.5",
-    "@bamboocss/extractor": "1.35.5",
-    "@bamboocss/shared": "1.35.5",
-    "@bamboocss/logger": "1.35.5",
-    "@bamboocss/node": "1.35.5",
-    "@bamboocss/types": "1.35.5"
+    "@bamboocss/config": "1.36.0",
+    "@bamboocss/core": "1.36.0",
+    "@bamboocss/extractor": "1.36.0",
+    "@bamboocss/logger": "1.36.0",
+    "@bamboocss/node": "1.36.0",
+    "@bamboocss/shared": "1.36.0",
+    "@bamboocss/types": "1.36.0"
   },
   "devDependencies": {
     "@jridgewell/trace-mapping": "^0.3.31",
     "vite": "7.2.6",
-    "@bamboocss/fixture": "1.35.5"
+    "@bamboocss/fixture": "1.36.0"
   },
   "peerDependencies": {
     "vite": ">=5"