@bamboocss/eslint-plugin 1.41.1 → 1.43.0

package/dist/index.cjs

@@ -8,7 +8,7 @@ require("@typescript-eslint/utils/ts-eslint");
 let _bamboocss_shared = require("@bamboocss/shared");
 //#region package.json
 var name = "@bamboocss/eslint-plugin";
-var version = "1.41.1";
+var version = "1.43.0";
 //#endregion
 //#region src/utils/index.ts
 const createRule = _typescript_eslint_utils.ESLintUtils.RuleCreator((name) => `https://github.com/gajus/bamboocss/blob/main/packages/eslint-plugin/docs/rules/${name}.md`);
@@ -252,6 +252,22 @@ const getInvalidTokens = (value, context) => {
 	invalidTokensCache.set(value, invalidTokens);
 	return invalidTokens;
 };
+const unresolvedValueCache = /* @__PURE__ */ new Map();
+/**
+* What the build would say about this property and value, or nothing if it is fine.
+*
+* Asked of the resolver rather than reimplemented, so the squiggle and the build agree by
+* construction. `getInvalidTokens` above answers a narrower question — is this dotted path a
+* token — and cannot see `color: 'mutedd'` or tell `top: 'navH'` from `animationName: 'fadeIn'`,
+* because neither is decidable without the property and its CSS grammar.
+*/
+const getUnresolvedValueMessage = (property, value, context) => {
+	const id = `${property}:${value}`;
+	if (unresolvedValueCache.has(id)) return unresolvedValueCache.get(id);
+	const message = syncAction("explainUnresolvedValue", getSyncOptions(context), property, value);
+	unresolvedValueCache.set(id, message);
+	return message;
+};
 const deprecatedTokensCache = /* @__PURE__ */ new Map();
 const getDeprecatedTokens = (property, value, context) => {
 	const propertyCategory = syncAction("getPropCategory", getSyncOptions(context), property);
@@ -1058,25 +1074,45 @@ const rule$12 = createRule({
 				node
 			});
 		};
-		const handleLiteralOrTemplate = (node) => {
+		/**
+		* With the property in hand, ask the resolver the whole question.
+		*
+		* `getInvalidTokens` can only answer "is this dotted path a token", because that is all a
+		* value tells you on its own. The property is what makes `color: 'mutedd'` decidable at all,
+		* and what separates `top: 'navH'` from `animationName: 'fadeIn'` — one is a typo and the
+		* other is a `<custom-ident>` the grammar asks for.
+		*/
+		const reportUnresolved = (node, property, value) => {
+			if (!value) return;
+			const message = getUnresolvedValueMessage(property, value, context);
+			if (message) context.report({
+				data: { message },
+				messageId: "unresolvedValue",
+				node
+			});
+		};
+		const handleValue = (node, property) => {
 			if (!node) return;
 			if (isLiteral(node)) {
 				const value = node.value?.toString();
 				sendReport(node, value);
+				if (property) reportUnresolved(node, property, value);
 			} else if (isTemplateLiteral(node) && node.expressions.length === 0) {
 				const value = node.quasis[0].value.raw;
 				sendReport(node.quasis[0], value);
+				if (property) reportUnresolved(node.quasis[0], property, value);
 			}
 		};
 		return {
 			JSXAttribute(node) {
 				if (!node.value || !isBambooProp(node, context)) return;
-				if (isLiteral(node.value)) handleLiteralOrTemplate(node.value);
-				else if (isJSXExpressionContainer(node.value)) handleLiteralOrTemplate(node.value.expression);
+				const property = isJSXIdentifier(node.name) ? node.name.name : void 0;
+				if (isLiteral(node.value)) handleValue(node.value, property);
+				else if (isJSXExpressionContainer(node.value)) handleValue(node.value.expression, property);
 			},
 			Property(node) {
 				if (!isIdentifier(node.key) || !(0, _typescript_eslint_utils_ast_utils.isNodeOfTypes)([_typescript_eslint_utils.AST_NODE_TYPES.Literal, _typescript_eslint_utils.AST_NODE_TYPES.TemplateLiteral])(node.value) || !isBambooAttribute(node, context) || isRecipeVariant(node, context)) return;
-				handleLiteralOrTemplate(node.value);
+				handleValue(node.value, node.key.name);
 			},
 			TaggedTemplateExpression(node) {
 				if (!getTaggedTemplateCaller(node)) return;
@@ -1114,7 +1150,10 @@ const rule$12 = createRule({
 	defaultOptions: [],
 	meta: {
 		docs: { description: "Disallow the use of invalid token paths within token function syntax." },
-		messages: { noInvalidTokenPaths: "`{{token}}` is an invalid token path." },
+		messages: {
+			noInvalidTokenPaths: "`{{token}}` is an invalid token path.",
+			unresolvedValue: "{{message}}"
+		},
 		schema: [],
 		type: "problem"
 	},

package/dist/index.mjs

@@ -8,7 +8,7 @@ import "@typescript-eslint/utils/ts-eslint";
 import { getArbitraryValue, parseFallbackValue } from "@bamboocss/shared";
 //#region package.json
 var name = "@bamboocss/eslint-plugin";
-var version = "1.41.1";
+var version = "1.43.0";
 //#endregion
 //#region src/utils/index.ts
 const createRule = ESLintUtils.RuleCreator((name) => `https://github.com/gajus/bamboocss/blob/main/packages/eslint-plugin/docs/rules/${name}.md`);
@@ -252,6 +252,22 @@ const getInvalidTokens = (value, context) => {
 	invalidTokensCache.set(value, invalidTokens);
 	return invalidTokens;
 };
+const unresolvedValueCache = /* @__PURE__ */ new Map();
+/**
+* What the build would say about this property and value, or nothing if it is fine.
+*
+* Asked of the resolver rather than reimplemented, so the squiggle and the build agree by
+* construction. `getInvalidTokens` above answers a narrower question — is this dotted path a
+* token — and cannot see `color: 'mutedd'` or tell `top: 'navH'` from `animationName: 'fadeIn'`,
+* because neither is decidable without the property and its CSS grammar.
+*/
+const getUnresolvedValueMessage = (property, value, context) => {
+	const id = `${property}:${value}`;
+	if (unresolvedValueCache.has(id)) return unresolvedValueCache.get(id);
+	const message = syncAction("explainUnresolvedValue", getSyncOptions(context), property, value);
+	unresolvedValueCache.set(id, message);
+	return message;
+};
 const deprecatedTokensCache = /* @__PURE__ */ new Map();
 const getDeprecatedTokens = (property, value, context) => {
 	const propertyCategory = syncAction("getPropCategory", getSyncOptions(context), property);
@@ -1058,25 +1074,45 @@ const rule$12 = createRule({
 				node
 			});
 		};
-		const handleLiteralOrTemplate = (node) => {
+		/**
+		* With the property in hand, ask the resolver the whole question.
+		*
+		* `getInvalidTokens` can only answer "is this dotted path a token", because that is all a
+		* value tells you on its own. The property is what makes `color: 'mutedd'` decidable at all,
+		* and what separates `top: 'navH'` from `animationName: 'fadeIn'` — one is a typo and the
+		* other is a `<custom-ident>` the grammar asks for.
+		*/
+		const reportUnresolved = (node, property, value) => {
+			if (!value) return;
+			const message = getUnresolvedValueMessage(property, value, context);
+			if (message) context.report({
+				data: { message },
+				messageId: "unresolvedValue",
+				node
+			});
+		};
+		const handleValue = (node, property) => {
 			if (!node) return;
 			if (isLiteral(node)) {
 				const value = node.value?.toString();
 				sendReport(node, value);
+				if (property) reportUnresolved(node, property, value);
 			} else if (isTemplateLiteral(node) && node.expressions.length === 0) {
 				const value = node.quasis[0].value.raw;
 				sendReport(node.quasis[0], value);
+				if (property) reportUnresolved(node.quasis[0], property, value);
 			}
 		};
 		return {
 			JSXAttribute(node) {
 				if (!node.value || !isBambooProp(node, context)) return;
-				if (isLiteral(node.value)) handleLiteralOrTemplate(node.value);
-				else if (isJSXExpressionContainer(node.value)) handleLiteralOrTemplate(node.value.expression);
+				const property = isJSXIdentifier(node.name) ? node.name.name : void 0;
+				if (isLiteral(node.value)) handleValue(node.value, property);
+				else if (isJSXExpressionContainer(node.value)) handleValue(node.value.expression, property);
 			},
 			Property(node) {
 				if (!isIdentifier(node.key) || !isNodeOfTypes([AST_NODE_TYPES.Literal, AST_NODE_TYPES.TemplateLiteral])(node.value) || !isBambooAttribute(node, context) || isRecipeVariant(node, context)) return;
-				handleLiteralOrTemplate(node.value);
+				handleValue(node.value, node.key.name);
 			},
 			TaggedTemplateExpression(node) {
 				if (!getTaggedTemplateCaller(node)) return;
@@ -1114,7 +1150,10 @@ const rule$12 = createRule({
 	defaultOptions: [],
 	meta: {
 		docs: { description: "Disallow the use of invalid token paths within token function syntax." },
-		messages: { noInvalidTokenPaths: "`{{token}}` is an invalid token path." },
+		messages: {
+			noInvalidTokenPaths: "`{{token}}` is an invalid token path.",
+			unresolvedValue: "{{message}}"
+		},
 		schema: [],
 		type: "problem"
 	},

package/dist/utils/worker.cjs

@@ -63,6 +63,21 @@ const filterDeprecatedTokens = (context, tokens) => {
 const filterInvalidTokens = (context, paths) => {
 	return paths.filter((path$2) => !context.utility.tokens.view.get(path$2));
 };
+/**
+* The build's own verdict on one property/value pair, for the editor.
+*
+* The same predicate and the same sentence the build prints — not a second implementation of
+* either. The type layer used to give this feedback by narrowing every generated prop type, at
+* a measured 19% of the types and 28% of the instantiations in a real project, and it could
+* only ever say "not assignable"; the resolver knows the value is a `sizes` token used on a
+* property that reads `spacing`, and knowing that is the whole point.
+*/
+const explainUnresolvedValue = (context, property, value) => {
+	const { utility } = context;
+	if (!utility.isUnresolvedTokenValue(property, value)) return void 0;
+	const key = utility.resolveShorthand(property);
+	return utility.explainUnresolvedToken(utility.unresolvedTokenRef(key, utility.bareTokenPath(key, value)));
+};
 const getPropertyCategory = (context, _attribute) => {
 	const attribute = resolveLongHand(context, _attribute) || _attribute;
 	const attributeConfig = context.utility.config[attribute];
@@ -82,6 +97,7 @@ async function runAsync(action, options, ...args) {
 	switch (action) {
 		case "filterDeprecatedTokens": return filterDeprecatedTokens(context, ...args);
 		case "filterImports": return filterImports(context, ...args);
+		case "explainUnresolvedValue": return explainUnresolvedValue(context, ...args);
 		case "filterInvalidTokens": return filterInvalidTokens(context, ...args);
 		case "getPropCategory": return getPropertyCategory(context, ...args);
 		case "isColorAttribute": return isColorAttribute(context, ...args);

package/dist/utils/worker.d.cts

@@ -22,6 +22,7 @@ type MatchImportResult = {
   name: string;
 };
 declare function run(action: 'filterInvalidTokens', options: Options, paths: string[]): string[];
+declare function run(action: 'explainUnresolvedValue', options: Options, property: string, value: string): string | undefined;
 declare function run(action: 'isColorToken', options: Options, value: string): boolean;
 declare function run(action: 'isColorAttribute', options: Options, attribute: string): boolean;
 declare function run(action: 'isValidFile', options: Options): boolean;
@@ -34,6 +35,7 @@ declare function run(action: 'filterImports', options: Options, imports: MatchIm
 declare function run(action: 'getPropCategory', options: Options, property: string): string;
 declare function run(action: 'filterDeprecatedTokens', options: Options, tokens: DeprecatedToken[]): DeprecatedToken[];
 declare function runAsync(action: 'filterInvalidTokens', options: Options, paths: string[]): Promise<string[]>;
+declare function runAsync(action: 'explainUnresolvedValue', options: Options, property: string, value: string): Promise<string | undefined>;
 declare function runAsync(action: 'isColorToken', options: Options, value: string): Promise<boolean>;
 declare function runAsync(action: 'isColorAttribute', options: Options, attribute: string): Promise<boolean>;
 declare function runAsync(action: 'isValidFile', options: Options, fileName: string): Promise<string>;

package/dist/utils/worker.d.mts

@@ -22,6 +22,7 @@ type MatchImportResult = {
   name: string;
 };
 declare function run(action: 'filterInvalidTokens', options: Options, paths: string[]): string[];
+declare function run(action: 'explainUnresolvedValue', options: Options, property: string, value: string): string | undefined;
 declare function run(action: 'isColorToken', options: Options, value: string): boolean;
 declare function run(action: 'isColorAttribute', options: Options, attribute: string): boolean;
 declare function run(action: 'isValidFile', options: Options): boolean;
@@ -34,6 +35,7 @@ declare function run(action: 'filterImports', options: Options, imports: MatchIm
 declare function run(action: 'getPropCategory', options: Options, property: string): string;
 declare function run(action: 'filterDeprecatedTokens', options: Options, tokens: DeprecatedToken[]): DeprecatedToken[];
 declare function runAsync(action: 'filterInvalidTokens', options: Options, paths: string[]): Promise<string[]>;
+declare function runAsync(action: 'explainUnresolvedValue', options: Options, property: string, value: string): Promise<string | undefined>;
 declare function runAsync(action: 'isColorToken', options: Options, value: string): Promise<boolean>;
 declare function runAsync(action: 'isColorAttribute', options: Options, attribute: string): Promise<boolean>;
 declare function runAsync(action: 'isValidFile', options: Options, fileName: string): Promise<string>;

package/dist/utils/worker.mjs

@@ -38,6 +38,21 @@ const filterDeprecatedTokens = (context, tokens) => {
 const filterInvalidTokens = (context, paths) => {
 	return paths.filter((path) => !context.utility.tokens.view.get(path));
 };
+/**
+* The build's own verdict on one property/value pair, for the editor.
+*
+* The same predicate and the same sentence the build prints — not a second implementation of
+* either. The type layer used to give this feedback by narrowing every generated prop type, at
+* a measured 19% of the types and 28% of the instantiations in a real project, and it could
+* only ever say "not assignable"; the resolver knows the value is a `sizes` token used on a
+* property that reads `spacing`, and knowing that is the whole point.
+*/
+const explainUnresolvedValue = (context, property, value) => {
+	const { utility } = context;
+	if (!utility.isUnresolvedTokenValue(property, value)) return void 0;
+	const key = utility.resolveShorthand(property);
+	return utility.explainUnresolvedToken(utility.unresolvedTokenRef(key, utility.bareTokenPath(key, value)));
+};
 const getPropertyCategory = (context, _attribute) => {
 	const attribute = resolveLongHand(context, _attribute) || _attribute;
 	const attributeConfig = context.utility.config[attribute];
@@ -57,6 +72,7 @@ async function runAsync(action, options, ...args) {
 	switch (action) {
 		case "filterDeprecatedTokens": return filterDeprecatedTokens(context, ...args);
 		case "filterImports": return filterImports(context, ...args);
+		case "explainUnresolvedValue": return explainUnresolvedValue(context, ...args);
 		case "filterInvalidTokens": return filterInvalidTokens(context, ...args);
 		case "getPropCategory": return getPropertyCategory(context, ...args);
 		case "isColorAttribute": return isColorAttribute(context, ...args);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bamboocss/eslint-plugin",
-  "version": "1.41.1",
+  "version": "1.43.0",
   "description": "ESLint plugin for Bamboo CSS",
   "homepage": "https://bamboocss.com",
   "license": "MIT",
@@ -36,9 +36,9 @@
     "@typescript-eslint/utils": "^8.21.0",
     "micromatch": "^4.0.8",
     "synckit": "^0.9.0",
-    "@bamboocss/config": "1.41.1",
-    "@bamboocss/generator": "1.41.1",
-    "@bamboocss/shared": "1.41.1"
+    "@bamboocss/config": "1.43.0",
+    "@bamboocss/generator": "1.43.0",
+    "@bamboocss/shared": "1.43.0"
   },
   "devDependencies": {
     "@types/micromatch": "^4.0.10",
@@ -46,10 +46,10 @@
     "eslint": "^9.35.0",
     "multiline-ts": "^4.0.1",
     "typescript": "^5.7.2",
-    "@bamboocss/dev": "1.41.1",
-    "@bamboocss/preset-bamboo": "1.41.1",
-    "@bamboocss/preset-base": "1.41.1",
-    "@bamboocss/types": "1.41.1"
+    "@bamboocss/dev": "1.43.0",
+    "@bamboocss/preset-bamboo": "1.43.0",
+    "@bamboocss/preset-base": "1.43.0",
+    "@bamboocss/types": "1.43.0"
   },
   "peerDependencies": {
     "eslint": "*"