@bamboocss/vite 1.37.10 → 1.37.11

package/dist/index.cjs

@@ -751,16 +751,21 @@ 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.
+* 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) => byText(identifiersByName, sourceFile, () => {
+const identifierIndex = (sourceFile) => {
 	const index = /* @__PURE__ */ new Map();
 	for (const identifier of sourceFile.getDescendantsOfKind(ts_morph.SyntaxKind.Identifier)) {
 		const text = identifier.getText();
@@ -769,10 +774,10 @@ const identifierIndex = (sourceFile) => byText(identifiersByName, sourceFile, ()
 		else index.set(text, [identifier]);
 	}
 	return index;
-});
-const localReferencesTo = (sourceFile, name, declaration) => {
+};
+const localReferencesTo = (index, name, declaration) => {
 	const references = [];
-	for (const identifier of identifierIndex(sourceFile).get(name) ?? []) {
+	for (const identifier of index.get(name) ?? []) {
 		if (identifier === declaration) continue;
 		const parent = identifier.getParent();
 		if (!parent) continue;
@@ -2557,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;
@@ -2564,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;

package/dist/index.mjs

@@ -721,16 +721,21 @@ 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.
+* 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) => byText(identifiersByName, sourceFile, () => {
+const identifierIndex = (sourceFile) => {
 	const index = /* @__PURE__ */ new Map();
 	for (const identifier of sourceFile.getDescendantsOfKind(SyntaxKind.Identifier)) {
 		const text = identifier.getText();
@@ -739,10 +744,10 @@ const identifierIndex = (sourceFile) => byText(identifiersByName, sourceFile, ()
 		else index.set(text, [identifier]);
 	}
 	return index;
-});
-const localReferencesTo = (sourceFile, name, declaration) => {
+};
+const localReferencesTo = (index, name, declaration) => {
 	const references = [];
-	for (const identifier of identifierIndex(sourceFile).get(name) ?? []) {
+	for (const identifier of index.get(name) ?? []) {
 		if (identifier === declaration) continue;
 		const parent = identifier.getParent();
 		if (!parent) continue;
@@ -2527,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;
@@ -2534,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;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bamboocss/vite",
-  "version": "1.37.10",
+  "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.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"
+    "@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.10"
+    "@bamboocss/fixture": "1.37.11"
   },
   "peerDependencies": {
     "vite": ">=5"