@tenphi/eslint-plugin-tasty 1.3.0 → 1.4.0

package/README.md

@@ -185,6 +185,7 @@ for typing shared signatures.
 | `tasty/valid-recipe` | error | Unknown recipe names |
 | `tasty/valid-transition` | warn | Unknown transition property names |
 | `tasty/no-nested-selector` | warn | `&`-prefixed nested selectors (use sub-elements) |
+| `tasty/no-state-in-selector` | warn | Pseudo-class and attribute conditions in sub-element `$` selectors; put conditions in property state maps ([details](docs/rules/no-state-in-selector.md)) |
 | `tasty/static-no-dynamic-values` | error | Dynamic values in `tastyStatic()` |
 | `tasty/static-valid-selector` | error | Invalid selector in `tastyStatic(selector, ...)` |
 | `tasty/require-default-state` | error | Missing default (`''`) or fallback floor (`_`) key in state mappings (skipped for extending calls) |
@@ -212,6 +213,22 @@ for typing shared signatures.
 | `tasty/no-unknown-state-alias` | warn | Unknown `@name` state aliases |
 | `tasty/no-styles-prop` | warn | Direct `styles` prop usage |
 
+### Structural selectors and state maps
+
+Keep a sub-element's `$` selector focused on the element's identity and position.
+`tasty/no-state-in-selector` warns on conditions such as `img:not([width])`,
+`button[disabled]`, and `>@:hover`. Express those conditions in property state
+maps, using `@own(...)` for the sub-element's own state. Root states stay outside
+`@own(...)`.
+
+Tags, combinators, classes, IDs, exact `[data-element="Name"]` identities, and
+pseudo-elements remain valid structural selectors. `:is()` and `:where()` may
+group structural selectors, but conditions inside them still produce a warning:
+`:where(picture)` is accepted, while `:where(img:not([width]))` is reported.
+The rule does not autofix because moving a condition can change its scope and
+the behavior of other properties. See the [rule guide](docs/rules/no-state-in-selector.md)
+for examples and migration advice.
+
 ### Static values and intentional spreads
 
 The recommended rules flag JavaScript-computed style values, including

package/dist/configs.js

@@ -13,6 +13,7 @@ const recommended = {
 	"tasty/valid-directional-modifier": "error",
 	"tasty/valid-radius-shape": "error",
 	"tasty/no-nested-selector": "error",
+	"tasty/no-state-in-selector": "warn",
 	"tasty/static-no-dynamic-values": "error",
 	"tasty/static-valid-selector": "error",
 	"tasty/valid-preset": "error",

package/dist/configs.js.map

@@ -1 +1 @@
-{"version":3,"file":"configs.js","names":[],"sources":["../src/configs.ts"],"sourcesContent":["import type { TSESLint } from '@typescript-eslint/utils';\n\nexport const recommended: TSESLint.SharedConfig.RulesRecord = {\n  'tasty/known-property': 'warn',\n  'tasty/valid-value': 'error',\n  'tasty/valid-color-token': 'error',\n  'tasty/valid-custom-unit': 'error',\n  'tasty/valid-boolean-property': 'error',\n  'tasty/valid-state-key': 'error',\n  'tasty/valid-styles-structure': 'error',\n  'tasty/no-nested-state-map': 'error',\n  'tasty/no-important': 'error',\n  'tasty/valid-sub-element': 'error',\n  'tasty/valid-directional-modifier': 'error',\n  'tasty/valid-radius-shape': 'error',\n  'tasty/no-nested-selector': 'error',\n  'tasty/static-no-dynamic-values': 'error',\n  'tasty/static-valid-selector': 'error',\n  'tasty/valid-preset': 'error',\n  'tasty/valid-recipe': 'error',\n  'tasty/valid-transition': 'warn',\n  'tasty/require-default-state': 'error',\n  'tasty/no-own-at-root': 'warn',\n  'tasty/valid-default-state-order': 'warn',\n  'tasty/prefer-shorthand-property': 'warn',\n  'tasty/no-raw-color-values': 'warn',\n  'tasty/no-raw-transition-duration': 'warn',\n  'tasty/no-runtime-styles-mutation': 'warn',\n  'tasty/no-style-spread': 'warn',\n  'tasty/consistent-token-usage': 'warn',\n  'tasty/prefer-auto-calc': 'warn',\n  'tasty/prefer-state-negation': 'warn',\n  'tasty/prefer-custom-property-syntax': 'warn',\n  'tasty/prefer-hide': 'warn',\n  'tasty/prefer-directional-shorthand': 'warn',\n  'tasty/prefer-longhand-property': 'error',\n};\n\nexport const strict: TSESLint.SharedConfig.RulesRecord = {\n  ...recommended,\n  'tasty/valid-custom-property': 'warn',\n  'tasty/no-unknown-state-alias': 'warn',\n  'tasty/no-styles-prop': 'warn',\n  'tasty/valid-state-definition': 'warn',\n};\n"],"mappings":";AAEA,MAAa,cAAiD;CAC5D,wBAAwB;CACxB,qBAAqB;CACrB,2BAA2B;CAC3B,2BAA2B;CAC3B,gCAAgC;CAChC,yBAAyB;CACzB,gCAAgC;CAChC,6BAA6B;CAC7B,sBAAsB;CACtB,2BAA2B;CAC3B,oCAAoC;CACpC,4BAA4B;CAC5B,4BAA4B;CAC5B,kCAAkC;CAClC,+BAA+B;CAC/B,sBAAsB;CACtB,sBAAsB;CACtB,0BAA0B;CAC1B,+BAA+B;CAC/B,wBAAwB;CACxB,mCAAmC;CACnC,mCAAmC;CACnC,6BAA6B;CAC7B,oCAAoC;CACpC,oCAAoC;CACpC,yBAAyB;CACzB,gCAAgC;CAChC,0BAA0B;CAC1B,+BAA+B;CAC/B,uCAAuC;CACvC,qBAAqB;CACrB,sCAAsC;CACtC,kCAAkC;CACnC;AAED,MAAa,SAA4C;CACvD,GAAG;CACH,+BAA+B;CAC/B,gCAAgC;CAChC,wBAAwB;CACxB,gCAAgC;CACjC"}
+{"version":3,"file":"configs.js","names":[],"sources":["../src/configs.ts"],"sourcesContent":["import type { TSESLint } from '@typescript-eslint/utils';\n\nexport const recommended: TSESLint.SharedConfig.RulesRecord = {\n  'tasty/known-property': 'warn',\n  'tasty/valid-value': 'error',\n  'tasty/valid-color-token': 'error',\n  'tasty/valid-custom-unit': 'error',\n  'tasty/valid-boolean-property': 'error',\n  'tasty/valid-state-key': 'error',\n  'tasty/valid-styles-structure': 'error',\n  'tasty/no-nested-state-map': 'error',\n  'tasty/no-important': 'error',\n  'tasty/valid-sub-element': 'error',\n  'tasty/valid-directional-modifier': 'error',\n  'tasty/valid-radius-shape': 'error',\n  'tasty/no-nested-selector': 'error',\n  'tasty/no-state-in-selector': 'warn',\n  'tasty/static-no-dynamic-values': 'error',\n  'tasty/static-valid-selector': 'error',\n  'tasty/valid-preset': 'error',\n  'tasty/valid-recipe': 'error',\n  'tasty/valid-transition': 'warn',\n  'tasty/require-default-state': 'error',\n  'tasty/no-own-at-root': 'warn',\n  'tasty/valid-default-state-order': 'warn',\n  'tasty/prefer-shorthand-property': 'warn',\n  'tasty/no-raw-color-values': 'warn',\n  'tasty/no-raw-transition-duration': 'warn',\n  'tasty/no-runtime-styles-mutation': 'warn',\n  'tasty/no-style-spread': 'warn',\n  'tasty/consistent-token-usage': 'warn',\n  'tasty/prefer-auto-calc': 'warn',\n  'tasty/prefer-state-negation': 'warn',\n  'tasty/prefer-custom-property-syntax': 'warn',\n  'tasty/prefer-hide': 'warn',\n  'tasty/prefer-directional-shorthand': 'warn',\n  'tasty/prefer-longhand-property': 'error',\n};\n\nexport const strict: TSESLint.SharedConfig.RulesRecord = {\n  ...recommended,\n  'tasty/valid-custom-property': 'warn',\n  'tasty/no-unknown-state-alias': 'warn',\n  'tasty/no-styles-prop': 'warn',\n  'tasty/valid-state-definition': 'warn',\n};\n"],"mappings":";AAEA,MAAa,cAAiD;CAC5D,wBAAwB;CACxB,qBAAqB;CACrB,2BAA2B;CAC3B,2BAA2B;CAC3B,gCAAgC;CAChC,yBAAyB;CACzB,gCAAgC;CAChC,6BAA6B;CAC7B,sBAAsB;CACtB,2BAA2B;CAC3B,oCAAoC;CACpC,4BAA4B;CAC5B,4BAA4B;CAC5B,8BAA8B;CAC9B,kCAAkC;CAClC,+BAA+B;CAC/B,sBAAsB;CACtB,sBAAsB;CACtB,0BAA0B;CAC1B,+BAA+B;CAC/B,wBAAwB;CACxB,mCAAmC;CACnC,mCAAmC;CACnC,6BAA6B;CAC7B,oCAAoC;CACpC,oCAAoC;CACpC,yBAAyB;CACzB,gCAAgC;CAChC,0BAA0B;CAC1B,+BAA+B;CAC/B,uCAAuC;CACvC,qBAAqB;CACrB,sCAAsC;CACtC,kCAAkC;CACnC;AAED,MAAa,SAA4C;CACvD,GAAG;CACH,+BAA+B;CAC/B,gCAAgC;CAChC,wBAAwB;CACxB,gCAAgC;CACjC"}

package/dist/index.js

@@ -5,6 +5,7 @@ import valid_custom_unit_default from "./rules/valid-custom-unit.js";
 import valid_state_key_default from "./rules/valid-state-key.js";
 import valid_styles_structure_default from "./rules/valid-styles-structure.js";
 import no_nested_selector_default from "./rules/no-nested-selector.js";
+import no_state_in_selector_default from "./rules/no-state-in-selector.js";
 import valid_custom_property_default from "./rules/valid-custom-property.js";
 import valid_preset_default from "./rules/valid-preset.js";
 import valid_recipe_default from "./rules/valid-recipe.js";
@@ -48,6 +49,7 @@ const ruleMap = {
 	"valid-state-key": valid_state_key_default,
 	"valid-styles-structure": valid_styles_structure_default,
 	"no-nested-selector": no_nested_selector_default,
+	"no-state-in-selector": no_state_in_selector_default,
 	"valid-custom-property": valid_custom_property_default,
 	"valid-preset": valid_preset_default,
 	"valid-recipe": valid_recipe_default,

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","names":["rules.knownProperty","rules.validValue","rules.validColorToken","rules.validCustomUnit","rules.validStateKey","rules.validStylesStructure","rules.noNestedSelector","rules.validCustomProperty","rules.validPreset","rules.validRecipe","rules.validBooleanProperty","rules.validDirectionalModifier","rules.validRadiusShape","rules.noImportant","rules.validSubElement","rules.noNestedStateMap","rules.staticNoDynamicValues","rules.staticValidSelector","rules.preferShorthandProperty","rules.validTransition","rules.requireDefaultState","rules.noUnknownStateAlias","rules.noRawColorValues","rules.noRawTransitionDuration","rules.noRawMotionDuration","rules.noStylesProp","rules.consistentTokenUsage","rules.noRuntimeStylesMutation","rules.noStyleSpread","rules.validStateDefinition","rules.noOwnAtRoot","rules.validDefaultStateOrder","rules.preferAutoCalc","rules.preferStateNegation","rules.preferCustomPropertySyntax","rules.preferHide","rules.preferDirectionalShorthand","rules.preferLonghandProperty"],"sources":["../src/index.ts"],"sourcesContent":["import { createRequire } from 'node:module';\nimport type { TSESLint } from '@typescript-eslint/utils';\nimport * as rules from './rules/index.js';\nimport { recommended, strict } from './configs.js';\n\nconst ruleMap: Record<string, TSESLint.RuleModule<string, unknown[]>> = {\n  'known-property': rules.knownProperty,\n  'valid-value': rules.validValue,\n  'valid-color-token': rules.validColorToken,\n  'valid-custom-unit': rules.validCustomUnit,\n  'valid-state-key': rules.validStateKey,\n  'valid-styles-structure': rules.validStylesStructure,\n  'no-nested-selector': rules.noNestedSelector,\n  'valid-custom-property': rules.validCustomProperty,\n  'valid-preset': rules.validPreset,\n  'valid-recipe': rules.validRecipe,\n  'valid-boolean-property': rules.validBooleanProperty,\n  'valid-directional-modifier': rules.validDirectionalModifier,\n  'valid-radius-shape': rules.validRadiusShape,\n  'no-important': rules.noImportant,\n  'valid-sub-element': rules.validSubElement,\n  'no-nested-state-map': rules.noNestedStateMap,\n  'static-no-dynamic-values': rules.staticNoDynamicValues,\n  'static-valid-selector': rules.staticValidSelector,\n  'prefer-shorthand-property': rules.preferShorthandProperty,\n  'valid-transition': rules.validTransition,\n  'require-default-state': rules.requireDefaultState,\n  'no-unknown-state-alias': rules.noUnknownStateAlias,\n  'no-raw-color-values': rules.noRawColorValues,\n  'no-raw-transition-duration': rules.noRawTransitionDuration,\n  'no-raw-motion-duration': rules.noRawMotionDuration,\n  'no-styles-prop': rules.noStylesProp,\n  'consistent-token-usage': rules.consistentTokenUsage,\n  'no-runtime-styles-mutation': rules.noRuntimeStylesMutation,\n  'no-style-spread': rules.noStyleSpread,\n  'valid-state-definition': rules.validStateDefinition,\n  'no-own-at-root': rules.noOwnAtRoot,\n  'valid-default-state-order': rules.validDefaultStateOrder,\n  'prefer-auto-calc': rules.preferAutoCalc,\n  'prefer-state-negation': rules.preferStateNegation,\n  'prefer-custom-property-syntax': rules.preferCustomPropertySyntax,\n  'prefer-hide': rules.preferHide,\n  'prefer-directional-shorthand': rules.preferDirectionalShorthand,\n  'prefer-longhand-property': rules.preferLonghandProperty,\n};\n\n// Read from package.json so `meta.version` cannot drift from the released\n// version, as the previous hardcoded literal had. `../package.json` resolves to\n// the package root from both `src/` and the unbundled `dist/`.\nconst { version } = createRequire(import.meta.url)('../package.json') as {\n  version: string;\n};\n\nconst plugin = {\n  meta: {\n    name: '@tenphi/eslint-plugin-tasty',\n    version,\n  },\n  rules: ruleMap,\n  configs: {\n    recommended: {\n      plugins: {\n        get tasty() {\n          return plugin;\n        },\n      },\n      rules: recommended,\n    },\n    strict: {\n      plugins: {\n        get tasty() {\n          return plugin;\n        },\n      },\n      rules: strict,\n    },\n  },\n} satisfies TSESLint.FlatConfig.Plugin & {\n  configs: Record<string, TSESLint.FlatConfig.Config>;\n};\n\nexport default plugin;\n\nexport { recommended, strict } from './configs.js';\nexport type {\n  TastyValidationConfig,\n  ResolvedConfig,\n  StyleFunctionConfig,\n} from './types.js';\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAKA,MAAM,UAAkE;CACtE,kBAAkBA;CAClB,eAAeC;CACf,qBAAqBC;CACrB,qBAAqBC;CACrB,mBAAmBC;CACnB,0BAA0BC;CAC1B,sBAAsBC;CACtB,yBAAyBC;CACzB,gBAAgBC;CAChB,gBAAgBC;CAChB,0BAA0BC;CAC1B,8BAA8BC;CAC9B,sBAAsBC;CACtB,gBAAgBC;CAChB,qBAAqBC;CACrB,uBAAuBC;CACvB,4BAA4BC;CAC5B,yBAAyBC;CACzB,6BAA6BC;CAC7B,oBAAoBC;CACpB,yBAAyBC;CACzB,0BAA0BC;CAC1B,uBAAuBC;CACvB,8BAA8BC;CAC9B,0BAA0BC;CAC1B,kBAAkBC;CAClB,0BAA0BC;CAC1B,8BAA8BC;CAC9B,mBAAmBC;CACnB,0BAA0BC;CAC1B,kBAAkBC;CAClB,6BAA6BC;CAC7B,oBAAoBC;CACpB,yBAAyBC;CACzB,iCAAiCC;CACjC,eAAeC;CACf,gCAAgCC;CAChC,4BAA4BC;CAC7B;AAKD,MAAM,EAAE,YAAY,cAAc,OAAO,KAAK,IAAI,CAAC,kBAAkB;AAIrE,MAAM,SAAS;CACb,MAAM;EACJ,MAAM;EACN;EACD;CACD,OAAO;CACP,SAAS;EACP,aAAa;GACX,SAAS,EACP,IAAI,QAAQ;AACV,WAAO;MAEV;GACD,OAAO;GACR;EACD,QAAQ;GACN,SAAS,EACP,IAAI,QAAQ;AACV,WAAO;MAEV;GACD,OAAO;GACR;EACF;CACF"}
+{"version":3,"file":"index.js","names":["rules.knownProperty","rules.validValue","rules.validColorToken","rules.validCustomUnit","rules.validStateKey","rules.validStylesStructure","rules.noNestedSelector","rules.noStateInSelector","rules.validCustomProperty","rules.validPreset","rules.validRecipe","rules.validBooleanProperty","rules.validDirectionalModifier","rules.validRadiusShape","rules.noImportant","rules.validSubElement","rules.noNestedStateMap","rules.staticNoDynamicValues","rules.staticValidSelector","rules.preferShorthandProperty","rules.validTransition","rules.requireDefaultState","rules.noUnknownStateAlias","rules.noRawColorValues","rules.noRawTransitionDuration","rules.noRawMotionDuration","rules.noStylesProp","rules.consistentTokenUsage","rules.noRuntimeStylesMutation","rules.noStyleSpread","rules.validStateDefinition","rules.noOwnAtRoot","rules.validDefaultStateOrder","rules.preferAutoCalc","rules.preferStateNegation","rules.preferCustomPropertySyntax","rules.preferHide","rules.preferDirectionalShorthand","rules.preferLonghandProperty"],"sources":["../src/index.ts"],"sourcesContent":["import { createRequire } from 'node:module';\nimport type { TSESLint } from '@typescript-eslint/utils';\nimport * as rules from './rules/index.js';\nimport { recommended, strict } from './configs.js';\n\nconst ruleMap: Record<string, TSESLint.RuleModule<string, unknown[]>> = {\n  'known-property': rules.knownProperty,\n  'valid-value': rules.validValue,\n  'valid-color-token': rules.validColorToken,\n  'valid-custom-unit': rules.validCustomUnit,\n  'valid-state-key': rules.validStateKey,\n  'valid-styles-structure': rules.validStylesStructure,\n  'no-nested-selector': rules.noNestedSelector,\n  'no-state-in-selector': rules.noStateInSelector,\n  'valid-custom-property': rules.validCustomProperty,\n  'valid-preset': rules.validPreset,\n  'valid-recipe': rules.validRecipe,\n  'valid-boolean-property': rules.validBooleanProperty,\n  'valid-directional-modifier': rules.validDirectionalModifier,\n  'valid-radius-shape': rules.validRadiusShape,\n  'no-important': rules.noImportant,\n  'valid-sub-element': rules.validSubElement,\n  'no-nested-state-map': rules.noNestedStateMap,\n  'static-no-dynamic-values': rules.staticNoDynamicValues,\n  'static-valid-selector': rules.staticValidSelector,\n  'prefer-shorthand-property': rules.preferShorthandProperty,\n  'valid-transition': rules.validTransition,\n  'require-default-state': rules.requireDefaultState,\n  'no-unknown-state-alias': rules.noUnknownStateAlias,\n  'no-raw-color-values': rules.noRawColorValues,\n  'no-raw-transition-duration': rules.noRawTransitionDuration,\n  'no-raw-motion-duration': rules.noRawMotionDuration,\n  'no-styles-prop': rules.noStylesProp,\n  'consistent-token-usage': rules.consistentTokenUsage,\n  'no-runtime-styles-mutation': rules.noRuntimeStylesMutation,\n  'no-style-spread': rules.noStyleSpread,\n  'valid-state-definition': rules.validStateDefinition,\n  'no-own-at-root': rules.noOwnAtRoot,\n  'valid-default-state-order': rules.validDefaultStateOrder,\n  'prefer-auto-calc': rules.preferAutoCalc,\n  'prefer-state-negation': rules.preferStateNegation,\n  'prefer-custom-property-syntax': rules.preferCustomPropertySyntax,\n  'prefer-hide': rules.preferHide,\n  'prefer-directional-shorthand': rules.preferDirectionalShorthand,\n  'prefer-longhand-property': rules.preferLonghandProperty,\n};\n\n// Read from package.json so `meta.version` cannot drift from the released\n// version, as the previous hardcoded literal had. `../package.json` resolves to\n// the package root from both `src/` and the unbundled `dist/`.\nconst { version } = createRequire(import.meta.url)('../package.json') as {\n  version: string;\n};\n\nconst plugin = {\n  meta: {\n    name: '@tenphi/eslint-plugin-tasty',\n    version,\n  },\n  rules: ruleMap,\n  configs: {\n    recommended: {\n      plugins: {\n        get tasty() {\n          return plugin;\n        },\n      },\n      rules: recommended,\n    },\n    strict: {\n      plugins: {\n        get tasty() {\n          return plugin;\n        },\n      },\n      rules: strict,\n    },\n  },\n} satisfies TSESLint.FlatConfig.Plugin & {\n  configs: Record<string, TSESLint.FlatConfig.Config>;\n};\n\nexport default plugin;\n\nexport { recommended, strict } from './configs.js';\nexport type {\n  TastyValidationConfig,\n  ResolvedConfig,\n  StyleFunctionConfig,\n} from './types.js';\n"],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAKA,MAAM,UAAkE;CACtE,kBAAkBA;CAClB,eAAeC;CACf,qBAAqBC;CACrB,qBAAqBC;CACrB,mBAAmBC;CACnB,0BAA0BC;CAC1B,sBAAsBC;CACtB,wBAAwBC;CACxB,yBAAyBC;CACzB,gBAAgBC;CAChB,gBAAgBC;CAChB,0BAA0BC;CAC1B,8BAA8BC;CAC9B,sBAAsBC;CACtB,gBAAgBC;CAChB,qBAAqBC;CACrB,uBAAuBC;CACvB,4BAA4BC;CAC5B,yBAAyBC;CACzB,6BAA6BC;CAC7B,oBAAoBC;CACpB,yBAAyBC;CACzB,0BAA0BC;CAC1B,uBAAuBC;CACvB,8BAA8BC;CAC9B,0BAA0BC;CAC1B,kBAAkBC;CAClB,0BAA0BC;CAC1B,8BAA8BC;CAC9B,mBAAmBC;CACnB,0BAA0BC;CAC1B,kBAAkBC;CAClB,6BAA6BC;CAC7B,oBAAoBC;CACpB,yBAAyBC;CACzB,iCAAiCC;CACjC,eAAeC;CACf,gCAAgCC;CAChC,4BAA4BC;CAC7B;AAKD,MAAM,EAAE,YAAY,cAAc,OAAO,KAAK,IAAI,CAAC,kBAAkB;AAIrE,MAAM,SAAS;CACb,MAAM;EACJ,MAAM;EACN;EACD;CACD,OAAO;CACP,SAAS;EACP,aAAa;GACX,SAAS,EACP,IAAI,QAAQ;AACV,WAAO;MAEV;GACD,OAAO;GACR;EACD,QAAQ;GACN,SAAS,EACP,IAAI,QAAQ;AACV,WAAO;MAEV;GACD,OAAO;GACR;EACF;CACF"}

package/dist/rules/no-state-in-selector.js

@@ -0,0 +1,85 @@
+import { createRule } from "../create-rule.js";
+import { getKeyName, getStringValue, unwrapExpression } from "../utils.js";
+import { TastyContext, styleObjectListeners } from "../context.js";
+import selectorParser from "postcss-selector-parser";
+
+//#region src/rules/no-state-in-selector.ts
+const STRUCTURAL_PSEUDOS = new Set([":is", ":where"]);
+const LEGACY_PSEUDO_ELEMENTS = new Set([
+	":before",
+	":after",
+	":first-line",
+	":first-letter"
+]);
+function firstStateCondition(selector) {
+	let root;
+	try {
+		root = selectorParser().astSync(selector);
+	} catch {
+		return null;
+	}
+	let condition = null;
+	root.walk((node) => {
+		if (node.type === "attribute") {
+			if (node.attribute === "data-element" && !node.namespace && node.operator === "=" && node.value) return;
+		} else if (node.type === "pseudo") {
+			let name = node.value.toLowerCase();
+			if (name.includes("\\")) {
+				const colons = name.startsWith("::") ? "::" : ":";
+				const identifier = selectorParser().astSync(name.slice(colons.length)).first?.first;
+				if (identifier?.type === "tag") name = colons + identifier.value.toLowerCase();
+			}
+			if (name.startsWith("::") || LEGACY_PSEUDO_ELEMENTS.has(name) || STRUCTURAL_PSEUDOS.has(name)) return;
+		} else return;
+		condition = node.toString().trim();
+		return false;
+	});
+	return condition;
+}
+var no_state_in_selector_default = createRule({
+	name: "no-state-in-selector",
+	meta: {
+		type: "suggestion",
+		docs: { description: "Keep sub-element selectors structural and express conditions in property state maps" },
+		messages: { stateInSelector: "Move state condition '{{condition}}' out of the sub-element '$' selector and into property state maps. Use @own(...) for the sub-element's own state." },
+		schema: []
+	},
+	defaultOptions: [],
+	create(context) {
+		const ctx = new TastyContext(context);
+		const visited = /* @__PURE__ */ new WeakSet();
+		function handleStyleObject(node) {
+			if (visited.has(node) || !ctx.isStyleObject(node)) return;
+			visited.add(node);
+			const isSubElement = ctx.isInsideSubElement(node);
+			for (const prop of node.properties) {
+				if (prop.type !== "Property") continue;
+				const key = prop.computed ? getStringValue(unwrapExpression(prop.key)) : getKeyName(prop.key);
+				if (!prop.computed && key && ctx.isSubElementKey(key)) {
+					const child = unwrapExpression(prop.value);
+					if (child.type === "ObjectExpression") handleStyleObject(child);
+				}
+				if (!isSubElement || key !== "$") continue;
+				const selector = getStringValue(unwrapExpression(prop.value));
+				if (selector === null) continue;
+				const condition = firstStateCondition(selector);
+				if (condition === null) continue;
+				context.report({
+					node: prop.value,
+					messageId: "stateInSelector",
+					data: { condition }
+				});
+			}
+		}
+		return {
+			ImportDeclaration(node) {
+				ctx.trackImport(node);
+			},
+			...styleObjectListeners(handleStyleObject)
+		};
+	}
+});
+
+//#endregion
+export { no_state_in_selector_default as default };
+//# sourceMappingURL=no-state-in-selector.js.map

package/dist/rules/no-state-in-selector.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-state-in-selector.js","names":[],"sources":["../../src/rules/no-state-in-selector.ts"],"sourcesContent":["import type { TSESTree } from '@typescript-eslint/utils';\nimport selectorParser from 'postcss-selector-parser';\nimport { createRule } from '../create-rule.js';\nimport { TastyContext, styleObjectListeners } from '../context.js';\nimport { getKeyName, getStringValue, unwrapExpression } from '../utils.js';\n\nconst STRUCTURAL_PSEUDOS = new Set([':is', ':where']);\nconst LEGACY_PSEUDO_ELEMENTS = new Set([\n  ':before',\n  ':after',\n  ':first-line',\n  ':first-letter',\n]);\n\nfunction firstStateCondition(selector: string): string | null {\n  let root: selectorParser.Root;\n  try {\n    root = selectorParser().astSync(selector);\n  } catch {\n    // This recommendation is not a CSS syntax validator. Incomplete selectors\n    // must not crash lint or produce a misleading state-map recommendation.\n    return null;\n  }\n\n  let condition: string | null = null;\n  root.walk((node) => {\n    if (node.type === 'attribute') {\n      // An exact data-element identity is structural, like Tasty's uppercase\n      // element names. Other attributes express conditions on that element.\n      if (\n        node.attribute === 'data-element' &&\n        !node.namespace &&\n        node.operator === '=' &&\n        node.value\n      ) {\n        return;\n      }\n    } else if (node.type === 'pseudo') {\n      let name = node.value.toLowerCase();\n      if (name.includes('\\\\')) {\n        // The parser decodes tag identifiers but preserves pseudo spelling.\n        const colons = name.startsWith('::') ? '::' : ':';\n        const identifier = selectorParser().astSync(name.slice(colons.length))\n          .first?.first;\n        if (identifier?.type === 'tag') {\n          name = colons + identifier.value.toLowerCase();\n        }\n      }\n      if (\n        name.startsWith('::') ||\n        LEGACY_PSEUDO_ELEMENTS.has(name) ||\n        STRUCTURAL_PSEUDOS.has(name)\n      ) {\n        // Continue walking: :where(img:hover) and ::slotted(img[width])\n        // still contain conditions, although the outer wrapper is structural.\n        return;\n      }\n    } else {\n      return;\n    }\n\n    condition = node.toString().trim();\n    return false;\n  });\n  return condition;\n}\n\nexport default createRule<[], 'stateInSelector'>({\n  name: 'no-state-in-selector',\n  meta: {\n    type: 'suggestion',\n    docs: {\n      description:\n        'Keep sub-element selectors structural and express conditions in property state maps',\n    },\n    messages: {\n      stateInSelector:\n        \"Move state condition '{{condition}}' out of the sub-element '$' selector and into property state maps. Use @own(...) for the sub-element's own state.\",\n    },\n    schema: [],\n  },\n  defaultOptions: [],\n  create(context) {\n    const ctx = new TastyContext(context);\n    const visited = new WeakSet<TSESTree.ObjectExpression>();\n\n    function handleStyleObject(node: TSESTree.ObjectExpression) {\n      if (visited.has(node) || !ctx.isStyleObject(node)) return;\n      visited.add(node);\n      const isSubElement = ctx.isInsideSubElement(node);\n\n      for (const prop of node.properties) {\n        if (prop.type !== 'Property') continue;\n        const key = prop.computed\n          ? getStringValue(unwrapExpression(prop.key))\n          : getKeyName(prop.key);\n        // Some listeners match only the shared/JSX root. Follow its owned\n        // sub-elements explicitly, without entering state maps or metadata.\n        if (!prop.computed && key && ctx.isSubElementKey(key)) {\n          const child = unwrapExpression(prop.value);\n          if (child.type === 'ObjectExpression') handleStyleObject(child);\n        }\n        if (!isSubElement || key !== '$') continue;\n\n        const selector = getStringValue(unwrapExpression(prop.value));\n        if (selector === null) continue;\n        const condition = firstStateCondition(selector);\n        if (condition === null) continue;\n\n        context.report({\n          node: prop.value,\n          messageId: 'stateInSelector',\n          data: { condition },\n        });\n      }\n    }\n\n    return {\n      ImportDeclaration(node) {\n        ctx.trackImport(node);\n      },\n      ...styleObjectListeners(handleStyleObject),\n    };\n  },\n});\n"],"mappings":";;;;;;AAMA,MAAM,qBAAqB,IAAI,IAAI,CAAC,OAAO,SAAS,CAAC;AACrD,MAAM,yBAAyB,IAAI,IAAI;CACrC;CACA;CACA;CACA;CACD,CAAC;AAEF,SAAS,oBAAoB,UAAiC;CAC5D,IAAI;AACJ,KAAI;AACF,SAAO,gBAAgB,CAAC,QAAQ,SAAS;SACnC;AAGN,SAAO;;CAGT,IAAI,YAA2B;AAC/B,MAAK,MAAM,SAAS;AAClB,MAAI,KAAK,SAAS,aAGhB;OACE,KAAK,cAAc,kBACnB,CAAC,KAAK,aACN,KAAK,aAAa,OAClB,KAAK,MAEL;aAEO,KAAK,SAAS,UAAU;GACjC,IAAI,OAAO,KAAK,MAAM,aAAa;AACnC,OAAI,KAAK,SAAS,KAAK,EAAE;IAEvB,MAAM,SAAS,KAAK,WAAW,KAAK,GAAG,OAAO;IAC9C,MAAM,aAAa,gBAAgB,CAAC,QAAQ,KAAK,MAAM,OAAO,OAAO,CAAC,CACnE,OAAO;AACV,QAAI,YAAY,SAAS,MACvB,QAAO,SAAS,WAAW,MAAM,aAAa;;AAGlD,OACE,KAAK,WAAW,KAAK,IACrB,uBAAuB,IAAI,KAAK,IAChC,mBAAmB,IAAI,KAAK,CAI5B;QAGF;AAGF,cAAY,KAAK,UAAU,CAAC,MAAM;AAClC,SAAO;GACP;AACF,QAAO;;AAGT,mCAAe,WAAkC;CAC/C,MAAM;CACN,MAAM;EACJ,MAAM;EACN,MAAM,EACJ,aACE,uFACH;EACD,UAAU,EACR,iBACE,yJACH;EACD,QAAQ,EAAE;EACX;CACD,gBAAgB,EAAE;CAClB,OAAO,SAAS;EACd,MAAM,MAAM,IAAI,aAAa,QAAQ;EACrC,MAAM,0BAAU,IAAI,SAAoC;EAExD,SAAS,kBAAkB,MAAiC;AAC1D,OAAI,QAAQ,IAAI,KAAK,IAAI,CAAC,IAAI,cAAc,KAAK,CAAE;AACnD,WAAQ,IAAI,KAAK;GACjB,MAAM,eAAe,IAAI,mBAAmB,KAAK;AAEjD,QAAK,MAAM,QAAQ,KAAK,YAAY;AAClC,QAAI,KAAK,SAAS,WAAY;IAC9B,MAAM,MAAM,KAAK,WACb,eAAe,iBAAiB,KAAK,IAAI,CAAC,GAC1C,WAAW,KAAK,IAAI;AAGxB,QAAI,CAAC,KAAK,YAAY,OAAO,IAAI,gBAAgB,IAAI,EAAE;KACrD,MAAM,QAAQ,iBAAiB,KAAK,MAAM;AAC1C,SAAI,MAAM,SAAS,mBAAoB,mBAAkB,MAAM;;AAEjE,QAAI,CAAC,gBAAgB,QAAQ,IAAK;IAElC,MAAM,WAAW,eAAe,iBAAiB,KAAK,MAAM,CAAC;AAC7D,QAAI,aAAa,KAAM;IACvB,MAAM,YAAY,oBAAoB,SAAS;AAC/C,QAAI,cAAc,KAAM;AAExB,YAAQ,OAAO;KACb,MAAM,KAAK;KACX,WAAW;KACX,MAAM,EAAE,WAAW;KACpB,CAAC;;;AAIN,SAAO;GACL,kBAAkB,MAAM;AACtB,QAAI,YAAY,KAAK;;GAEvB,GAAG,qBAAqB,kBAAkB;GAC3C;;CAEJ,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tenphi/eslint-plugin-tasty",
-  "version": "1.3.0",
+  "version": "1.4.0",
   "description": "ESLint plugin for validating tasty() and tastyStatic() style objects",
   "type": "module",
   "exports": {
@@ -44,7 +44,8 @@
   },
   "dependencies": {
     "@typescript-eslint/utils": "^8.56.0",
-    "jiti": "^2.6.1"
+    "jiti": "^2.6.1",
+    "postcss-selector-parser": "^7.1.6"
   },
   "devDependencies": {
     "@changesets/changelog-github": "^0.5.2",