Update @types/hast, utilities

Author: wooorm

lib/index.js

@@ -1,25 +1,6 @@
-/**
- * @typedef {import('hast').Element} Element
- */
-
 import {convertElement} from 'hast-util-is-element'
 
-/**
- * Check if a node is an element with a [*transparent content model*].
- *
- * @type {import('hast-util-is-element').AssertPredicate<Element & {tagName: 'a' | 'audio' | 'canvas' | 'datalist' | 'dd' | 'del' | 'ins' | 'map' | 'menu' | 'noscript' | 'object' | 'ol' | 'picture' | 'select' | 'table' | 'tbody' | 'tfoot' | 'thead' | 'tr' | 'ul' | 'video'}>}
- * @param value
- *   Thing to check (typically `Node`).
- * @returns
- *   Whether `value` is an element with a transparent content model.
- *
- *   The elements `a`, `audio`, `canvas`, `datalist`, `dd`, `del`, `ins`,
- *   `map`, `menu`, `noscript`, `object`, `ol`, `picture`, `select`, `table`,
- *   `tbody`, `tfoot`, `thead`, `tr`, `ul`, and `video` have transparent
- *   content models.
- */
-// @ts-expect-error Sure, the assertion matches.
-export const transparent = convertElement([
+const names = new Set([
   'a',
   'audio',
   'canvas',
@@ -42,3 +23,26 @@ export const transparent = convertElement([
   'ul',
   'video'
 ])
+
+/**
+ * Check if a node is an element with a [*transparent content model*].
+ *
+ * @param value
+ *   Thing to check (typically `Node`).
+ * @returns
+ *   Whether `value` is an element with a transparent content model.
+ *
+ *   The elements `a`, `audio`, `canvas`, `datalist`, `dd`, `del`, `ins`,
+ *   `map`, `menu`, `noscript`, `object`, `ol`, `picture`, `select`, `table`,
+ *   `tbody`, `tfoot`, `thead`, `tr`, `ul`, and `video` have transparent
+ *   content models.
+ */
+export const transparent = convertElement(
+  /**
+   * @param element
+   * @returns {element is {tagName: 'a' | 'audio' | 'canvas' | 'datalist' | 'dd' | 'del' | 'ins' | 'map' | 'menu' | 'noscript' | 'object' | 'ol' | 'picture' | 'select' | 'table' | 'tbody' | 'tfoot' | 'thead' | 'tr' | 'ul' | 'video'}}
+   */
+  function (element) {
+    return names.has(element.tagName)
+  }
+)

package.json

@@ -33,8 +33,7 @@
     "index.js"
   ],
   "dependencies": {
-    "@types/hast": "^2.0.0",
-    "hast-util-is-element": "^2.0.0"
+    "hast-util-is-element": "^3.0.0"
   },
   "devDependencies": {
     "@types/node": "^20.0.0",

test.js

@@ -10,6 +10,7 @@ test('transparent', () => {
     'should expose the public api'
   )
 
+  // @ts-expect-error: check how a missing `node` is handled.
   assert.equal(transparent(), false, 'should return `false` without node')
 
   assert.equal(transparent(null), false, 'should return `false` with `null`')