Update unist-util-visit

Author: wooorm

lib/handlers/media.js

@@ -32,7 +32,6 @@ export function media(h, node) {
   /** @type {ElementChild} */
   let child
 
-  // @ts-expect-error: assume valid content.
   visit({type: 'root', children: nodes}, 'link', findLink)
 
   // If the content links to something, or if it’s not phrasing…

lib/handlers/table.js

@@ -41,16 +41,10 @@ function inspect(node) {
   /** @type {Array.<string|null>} */
   const align = [null]
 
-  visit(node, 'element', visitor)
-
-  return {align, headless}
-
-  /** @type {import('unist-util-visit').Visitor<Element>} */
-  function visitor(child) {
+  visit(node, 'element', (child) => {
     // If there is a `thead`, assume there is a header row.
-    if (cell(child)) {
+    if (cell(child) && child.properties) {
       if (!align[cellIndex]) {
-        // @ts-expect-error: `props` are defined.
         align[cellIndex] = String(child.properties.align || '') || null
       }
 
@@ -66,7 +60,9 @@ function inspect(node) {
       rowIndex++
       cellIndex = 0
     }
-  }
+  })
+
+  return {align, headless}
 }
 
 /**

lib/index.js

@@ -12,7 +12,6 @@
  * @typedef {import('./types.js').MdastRoot} MdastRoot
  */
 
-import {hasProperty} from 'hast-util-has-property'
 // @ts-expect-error: next.
 import minifyWhitespace from 'rehype-minify-whitespace'
 import {convert} from 'unist-util-is'
@@ -97,7 +96,16 @@ export function toMdast(tree, options = {}) {
     }
   )
 
-  visit(tree, 'element', onelement)
+  visit(tree, 'element', (node) => {
+    const id =
+      node.properties &&
+      'id' in node.properties &&
+      String(node.properties.id).toUpperCase()
+
+    if (id && !own.call(byId, id)) {
+      byId[id] = node
+    }
+  })
 
   minifyWhitespace({newlines: options.newlines === true})(tree)
 
@@ -115,17 +123,6 @@ export function toMdast(tree, options = {}) {
 
   return mdast
 
-  /** @type {import('unist-util-visit').Visitor<Element>} */
-  function onelement(node) {
-    const id =
-      // @ts-expect-error: `hasProperty` just checked it.
-      hasProperty(node, 'id') && String(node.properties.id).toUpperCase()
-
-    if (id && !own.call(byId, id)) {
-      byId[id] = node
-    }
-  }
-
   /**
    * Collapse text nodes, and fix whitespace.
    * Most of this is taken care of by `rehype-minify-whitespace`, but

package.json

@@ -50,7 +50,7 @@
     "repeat-string": "^1.0.0",
     "trim-trailing-lines": "^2.0.0",
     "unist-util-is": "^5.0.0",
-    "unist-util-visit": "^3.0.0"
+    "unist-util-visit": "^4.0.0"
   },
   "devDependencies": {
     "@types/tape": "^4.0.0",