Fix selector-anb-no-unmatchable performance (#6925)

* Fix `selector-anb-no-unmatchable` performance * Update lib/utils/hasANPlusBNotationPseudoClasses.js Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com> * Create modern-numbers-impress.md --------- Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
stylelint · Jun 17, 2023 · 4e3abf7 · 4e3abf7
1 parent f72a6a9
commit 4e3abf7
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 0 deletions.
diff --git a/.changeset/modern-numbers-impress.md b/.changeset/modern-numbers-impress.md
@@ -0,0 +1,5 @@
+---
+"stylelint": patch
+---
+
+Fixed: `selector-anb-no-unmatchable` performance
diff --git a/lib/rules/selector-anb-no-unmatchable/index.js b/lib/rules/selector-anb-no-unmatchable/index.js
@@ -10,6 +10,7 @@ const isStandardSyntaxRule = require('../../utils/isStandardSyntaxRule');
 const report = require('../../utils/report');
 const ruleMessages = require('../../utils/ruleMessages');
 const validateOptions = require('../../utils/validateOptions');
+const hasANPlusBNotationPseudoClasses = require('../../utils/hasANPlusBNotationPseudoClasses');
 
 const ruleName = 'selector-anb-no-unmatchable';
 
@@ -50,6 +51,8 @@ const rule = (primary) => {
 			}
 
 			ruleNode.selectors.forEach((selector) => {
+				if (!hasANPlusBNotationPseudoClasses(selector)) return;
+
 				let cssTreeSelector;
 
 				try {

diff --git a/lib/utils/hasANPlusBNotationPseudoClasses.js b/lib/utils/hasANPlusBNotationPseudoClasses.js
@@ -0,0 +1,22 @@
+'use strict';
+
+const {
+	aNPlusBNotationPseudoClasses,
+	aNPlusBOfSNotationPseudoClasses,
+} = require('../reference/selectors');
+
+const classes = [
+	...aNPlusBNotationPseudoClasses.values(),
+	...aNPlusBOfSNotationPseudoClasses.values(),
+].join('|');
+const HAS_A_N_PLUS_B_NOTATION_PSEUDO_CLASSES = new RegExp(`\\b:(?:${classes})\\(`, 'i');
+
+/**
+ * Check if a selector contains any pseudo class function that might contain an An+B notation
+ *
+ * @param {string} selector
+ * @returns {boolean}
+ */
+module.exports = function hasANPlusBNotationPseudoClasses(selector) {
+	return HAS_A_N_PLUS_B_NOTATION_PSEUDO_CLASSES.test(selector);
+};