Add strict to tsconfig.json

Author: wooorm

index.js

@@ -1,4 +1,6 @@
 /**
+ * @typedef {import('mdast').Root|import('mdast').Content} Node
+ *
  * @typedef Options
  * @property {boolean} [includeImageAlt=true]
  */
@@ -8,31 +10,27 @@
  * Prefer the node’s plain-text fields, otherwise serialize its children,
  * and if the given value is an array, serialize the nodes in it.
  *
- * @param {unknown} node
+ * @param {unknown} value
  * @param {Options} [options]
  * @returns {string}
  */
-export function toString(node, options = {}) {
+export function toString(value, options = {}) {
   const {includeImageAlt = true} = options
-  return one(node, includeImageAlt)
+  return one(value, includeImageAlt)
 }
 
 /**
- * @param {unknown} node
+ * @param {unknown} value
  * @param {boolean} includeImageAlt
  * @returns {string}
  */
-function one(node, includeImageAlt) {
+function one(value, includeImageAlt) {
   return (
-    (node &&
-      typeof node === 'object' &&
-      // @ts-ignore looks like a literal.
-      (node.value ||
-        // @ts-ignore looks like an image.
-        (includeImageAlt ? node.alt : '') ||
-        // @ts-ignore looks like a parent.
-        ('children' in node && all(node.children, includeImageAlt)) ||
-        (Array.isArray(node) && all(node, includeImageAlt)))) ||
+    (node(value) &&
+      (('value' in value && value.value) ||
+        (includeImageAlt && 'alt' in value && value.alt) ||
+        ('children' in value && all(value.children, includeImageAlt)))) ||
+    (Array.isArray(value) && all(value, includeImageAlt)) ||
     ''
   )
 }
@@ -53,3 +51,11 @@ function all(values, includeImageAlt) {
 
   return result.join('')
 }
+
+/**
+ * @param {unknown} value
+ * @returns {value is Node}
+ */
+function node(value) {
+  return Boolean(value && typeof value === 'object')
+}

package.json

@@ -32,6 +32,9 @@
     "index.d.ts",
     "index.js"
   ],
+  "dependencies": {
+    "@types/mdast": "^3.0.0"
+  },
   "devDependencies": {
     "@types/tape": "^4.0.0",
     "c8": "^7.0.0",

test.js

@@ -2,6 +2,7 @@ import test from 'tape'
 import {toString} from './index.js'
 
 test('toString', (t) => {
+  // @ts-expect-error: runtime.
   t.equal(toString(), '', 'should not fail on a missing node')
   t.equal(toString(null), '', 'should not fail on `null` missing node')
 

tsconfig.json

@@ -10,6 +10,7 @@
     "declaration": true,
     "emitDeclarationOnly": true,
     "allowSyntheticDefaultImports": true,
-    "skipLibCheck": true
+    "skipLibCheck": true,
+    "strict": true
   }
 }