reffy 18.8.0 → 18.8.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reffy",
-  "version": "18.8.0",
+  "version": "18.8.2",
   "description": "W3C/WHATWG spec dependencies exploration companion. Features a short set of tools to study spec references as well as WebIDL term definitions and references found in W3C specifications.",
   "repository": {
     "type": "git",
@@ -45,7 +45,7 @@
   "devDependencies": {
     "respec": "35.4.0",
     "respec-hljs": "2.1.1",
-    "rollup": "4.41.1",
+    "rollup": "4.42.0",
     "undici": "^7.0.0"
   },
   "overrides": {

package/src/browserlib/extract-cssdfn.mjs

@@ -308,16 +308,37 @@ export default function () {
 
   // Specs typically do not make the syntax of selectors such as `:visited`
   // explicit because it essentially goes without saying: the syntax is the
-  // selector's name itself. Note that the syntax of selectors that are
-  // function-like such as `:nth-child()` cannot be inferred in the same way.
-  for (const selector of res.selectors) {
-    if (!selector.value && !selector.name.match(/\(/)) {
+  // selector's name itself. One nuance is that, for combinators such as `||`,
+  // tokens needs to be enclosed in single-quotes for the syntax to be valid.
+  // Note the syntax of selectors that are function-like such as `:nth-child()`
+  // cannot be inferred in the same way.
+  function setValueFromName(selector) {
+    if (selector.value) {
+      return;
+    }
+    if (selector.name.match(/\(/)) {
+      // Function-like selector
+      return;
+    }
+    if (selector.name.match(/^[:a-z]/i)) {
+      // Keyword-like selector that is not a combinator
       selector.value = selector.name;
     }
-    for (const subSelector of selector.values ?? []) {
-      if (!subSelector.value && !subSelector.name.match(/\(/)) {
-        subSelector.value = subSelector.name;
+    else {
+      // Combinator, let's enclose tokens in single-quotes
+      const tokens = selector.name.split('');
+      if (tokens.length === 1) {
+        selector.value = `'${tokens[0]}'`;
       }
+      else {
+        selector.value = tokens.map(token => `'${token}'`).join(' ');
+      }
+    }
+  }
+  for (const selector of res.selectors) {
+    setValueFromName(selector);
+    for (const subSelector of selector.values ?? []) {
+      setValueFromName(subSelector);
     }
   }