Refactor to remove unneeded utilities

*   Reorder pseudo handlers
*   Add docs to a bunch of functions
*   Refactor to always index queries (will be improved in the future)

Author: wooorm

lib/attribute.js

@@ -48,6 +48,8 @@ export function attribute(query, element, schema) {
 }
 
 /**
+ * Check whether an attribute exists.
+ *
  * `[attr]`
  *
  * @param {RuleAttr} _
@@ -60,6 +62,8 @@ function exists(_, element, info) {
 }
 
 /**
+ * Check whether an attribute has an exact value.
+ *
  * `[attr=value]`
  *
  * @param {RuleAttr} query
@@ -76,6 +80,9 @@ function exact(query, element, info) {
 }
 
 /**
+ * Check whether an attribute, interpreted as a space-separated list, contains
+ * a value.
+ *
  * `[attr~=value]`
  *
  * @param {RuleAttr} query
@@ -102,6 +109,9 @@ function spaceSeparatedList(query, element, info) {
 }
 
 /**
+ * Check whether an attribute has a substring as either the exact value or a
+ * prefix.
+ *
  * `[attr|=value]`
  *
  * @param {RuleAttr} query
@@ -125,6 +135,8 @@ function exactOrPrefix(query, element, info) {
 }
 
 /**
+ * Check whether an attribute has a substring as its start.
+ *
  * `[attr^=value]`
  *
  * @param {RuleAttr} query
@@ -145,6 +157,8 @@ function begins(query, element, info) {
 }
 
 /**
+ * Check whether an attribute has a substring as its end.
+ *
  * `[attr$=value]`
  *
  * @param {RuleAttr} query
@@ -164,6 +178,8 @@ function ends(query, element, info) {
 }
 
 /**
+ * Check whether an attribute contains a substring.
+ *
  * `[attr*=value]`
  *
  * @param {RuleAttr} query

lib/class-name.js

@@ -4,6 +4,8 @@
  */
 
 /**
+ * Check whether an element has all class names.
+ *
  * @param {Rule} query
  * @param {Element} element
  * @returns {boolean}

lib/enter-state.js

@@ -7,11 +7,9 @@
  */
 
 import {direction} from 'direction'
-import {isElement} from 'hast-util-is-element'
 import {toString} from 'hast-util-to-string'
 import {svg} from 'property-information'
 import {visit, EXIT, SKIP} from 'unist-util-visit'
-import {element} from './util.js'
 
 /**
  * Enter a node.
@@ -36,7 +34,7 @@ export function enterState(state, node) {
   /** @type {Direction | undefined} */
   let dirInferred
 
-  if (element(node) && node.properties) {
+  if (node.type === 'element' && node.properties) {
     const lang = node.properties.xmlLang || node.properties.lang
     const type = node.properties.type || 'text'
     const dir = dirProperty(node)
@@ -50,7 +48,7 @@ export function enterState(state, node) {
         state.editableOrEditingHost = true
       }
 
-      if (isElement(node, 'svg')) {
+      if (node.tagName === 'svg') {
         state.schema = svg
       }
 
@@ -62,25 +60,24 @@ export function enterState(state, node) {
         // Explicit `[dir=ltr]`.
         dir === 'ltr' ||
         // HTML with an invalid or no `[dir]`.
-        (dir !== 'auto' && isElement(node, 'html')) ||
+        (dir !== 'auto' && node.tagName === 'html') ||
         // `input[type=tel]` with an invalid or no `[dir]`.
-        (dir !== 'auto' && isElement(node, 'input') && type === 'tel')
+        (dir !== 'auto' && node.tagName === 'input' && type === 'tel')
       ) {
         dirInferred = 'ltr'
         // `[dir=auto]` or `bdi` with an invalid or no `[dir]`.
-      } else if (dir === 'auto' || isElement(node, 'bdi')) {
-        if (isElement(node, 'textarea')) {
+      } else if (dir === 'auto' || node.tagName === 'bdi') {
+        if (node.tagName === 'textarea') {
           // Check contents of `<textarea>`.
           dirInferred = dirBidi(toString(node))
         } else if (
-          isElement(node, 'input') &&
+          node.tagName === 'input' &&
           (type === 'email' ||
             type === 'search' ||
             type === 'tel' ||
             type === 'text')
         ) {
           // Check value of `<input>`.
-          // @ts-expect-error something is `never` in types but this is needed.
           dirInferred = node.properties.value
             ? // @ts-expect-error Assume string
               dirBidi(node.properties.value)
@@ -119,7 +116,11 @@ export function enterState(state, node) {
 
     if (
       child !== node &&
-      (isElement(child, ['bdi', 'script', 'style', 'textare']) ||
+      child.type === 'element' &&
+      (child.tagName === 'bdi' ||
+        child.tagName === 'script' ||
+        child.tagName === 'style' ||
+        child.tagName === 'textare' ||
         dirProperty(child))
     ) {
       return SKIP
@@ -142,7 +143,9 @@ function dirBidi(value) {
  */
 function dirProperty(node) {
   const value =
-    element(node) && node.properties && typeof node.properties.dir === 'string'
+    node.type === 'element' &&
+    node.properties &&
+    typeof node.properties.dir === 'string'
       ? node.properties.dir.toLowerCase()
       : undefined
 

lib/id.js

@@ -4,6 +4,8 @@
  */
 
 /**
+ * Check whether an element has an ID.
+ *
  * @param {Rule} query
  * @param {Element} element
  * @returns {boolean}

lib/name.js

@@ -4,6 +4,8 @@
  */
 
 /**
+ * Check whether an element has a tag name.
+ *
  * @param {Rule} query
  * @param {Element} element
  * @returns {boolean}