feat: Remove universal selector when mixed with a tag name selector

src/replaceRuleSelector.js

@@ -4,16 +4,56 @@ import balancedMatch from "balanced-match"
 
 const pseudoClass = ":matches"
 const selectorElementRE = /^[a-zA-Z]/
+const selectorUniversalRE = /^\*(?!\|)/
+const selectorNamespacedUniversalRE = /\|\*$/
 
 function isElementSelector(selector) {
   const matches = selectorElementRE.exec(selector)
   // console.log({selector, matches})
   return matches
 }
 
+function isUniversalSelector(selector) {
+  const matches = selectorUniversalRE.exec(selector)
+  // console.log({selector, matches})
+  return matches
+}
+
+function isUniversalSelectorWithNS(selector) {
+  const matches = selectorNamespacedUniversalRE.exec(selector)
+  // console.log({selector, matches})
+  return matches
+}
+
 function normalizeSelector(selector, preWhitespace, pre) {
-  if (isElementSelector(selector) && !isElementSelector(pre)) {
-    return `${ preWhitespace}${ selector }${ pre }`
+  if (isUniversalSelector(selector) && isUniversalSelector(pre)) {
+    return `${ preWhitespace }${ pre }${ selector.substring(1) }`
+  }
+
+  if (isElementSelector(pre)) {
+    if (isUniversalSelector(selector)) {
+      return `${ preWhitespace }${ pre }${ selector.substring(1) }`
+    }
+    else if (isUniversalSelectorWithNS(selector)) {
+      return `${ preWhitespace }${
+        selector.substring(0, selector.length - 1)
+      }${ pre }`
+    }
+  }
+
+  if (isElementSelector(selector)) {
+    if (isUniversalSelector(pre)) {
+      return `${ preWhitespace }${ selector }${ pre.substring(1) }`
+    }
+    else if (isUniversalSelectorWithNS(pre)) {
+      return `${ preWhitespace }${
+        pre.substring(0, pre.length - 1)
+      }${ selector }`
+    }
+
+    else if (!isElementSelector(pre)) {
+      return `${ preWhitespace}${ selector }${ pre }`
+    }
   }
 
   return `${ preWhitespace }${ pre }${ selector }`

test/index.js

@@ -160,5 +160,30 @@ article h3 + p {}`,
     "regression https://github.com/postcss/postcss-selector-matches/issues/10"
   )
 
+  t.equal(
+    transform(
+      "*:matches(a), " +
+        "c:matches(*), " +
+        "ns|*:matches(b), " +
+        "*|*:matches(d), " +
+        "*:matches(ns|e) {}"
+    ),
+    "a, c, ns|b, *|d, ns|e {}",
+    "universal selector is removed when mixed with a tag name selector"
+  )
+
+  t.equal(
+    transform("*.foo:matches(bar) {}"),
+    "bar.foo {}",
+    "universal selector is removed from compound selector when mixed with " +
+      "a tag name selector"
+  )
+
+  t.equal(
+    transform("*.foo:matches(*.bar, tag:baz), *:matches(*) {}"),
+    "*.foo.bar, tag:baz.foo, * {}",
+    "only one universal selector is kept"
+  )
+
   t.end()
 })