Update tsconfig.json

Author: wooorm

Diff for: lib/index.js

@@ -15,6 +15,7 @@ import {one} from './tree.js'
  * @param {Options} [options]
  * @returns {string}
  */
+// eslint-disable-next-line complexity
 export function toHtml(node, options = {}) {
   const quote = options.quote || '"'
   /** @type {Quote} */
@@ -32,20 +33,20 @@ export function toHtml(node, options = {}) {
     omit: options.omitOptionalTags ? omission : undefined,
     quote,
     alternative,
-    smart: options.quoteSmart,
-    unquoted: options.preferUnquoted,
-    tight: options.tightAttributes,
-    upperDoctype: options.upperDoctype,
-    tightDoctype: options.tightDoctype,
-    bogusComments: options.bogusComments,
-    tightLists: options.tightCommaSeparatedLists,
-    tightClose: options.tightSelfClosing,
-    collapseEmpty: options.collapseEmptyAttributes,
-    dangerous: options.allowDangerousHtml,
+    smart: options.quoteSmart || false,
+    unquoted: options.preferUnquoted || false,
+    tight: options.tightAttributes || false,
+    upperDoctype: options.upperDoctype || false,
+    tightDoctype: options.tightDoctype || false,
+    bogusComments: options.bogusComments || false,
+    tightLists: options.tightCommaSeparatedLists || false,
+    tightClose: options.tightSelfClosing || false,
+    collapseEmpty: options.collapseEmptyAttributes || false,
+    dangerous: options.allowDangerousHtml || false,
     voids: options.voids || htmlVoidElements.concat(),
     entities: options.entities || {},
-    close: options.closeSelfClosing,
-    closeEmpty: options.closeEmptyElements
+    close: options.closeSelfClosing || false,
+    closeEmpty: options.closeEmptyElements || false
   }
 
   return one(

Diff for: lib/omission/opening.js

@@ -82,6 +82,7 @@ function colgroup(node, index, parent) {
 
   // Previous colgroup was already omitted.
   if (
+    parent &&
     isElement(previous, 'colgroup') &&
     closing(previous, parent.children.indexOf(previous), parent)
   ) {
@@ -102,6 +103,7 @@ function tbody(node, index, parent) {
 
   // Previous table section was already omitted.
   if (
+    parent &&
     isElement(previous, ['thead', 'tbody']) &&
     closing(previous, parent.children.indexOf(previous), parent)
   ) {

Diff for: lib/omission/util/siblings.js

@@ -19,14 +19,14 @@ function siblings(increment) {
   /**
    * Find applicable siblings in a direction.
    *
-   * @param {Parent} parent
-   * @param {number} index
-   * @param {boolean} [includeWhitespace=false]
+   * @param {Parent | null | undefined} parent
+   * @param {number | null | undefined} index
+   * @param {boolean | null | undefined} [includeWhitespace=false]
    * @returns {Child}
    */
   function sibling(parent, index, includeWhitespace) {
-    const siblings = parent && parent.children
-    let offset = index + increment
+    const siblings = parent ? parent.children : []
+    let offset = (index || 0) + increment
     let next = siblings && siblings[offset]
 
     if (!includeWhitespace) {

Diff for: lib/tree.js

@@ -51,7 +51,7 @@ export function one(ctx, node, index, parent) {
  * Serialize all children of `parent`.
  *
  * @param {Context} ctx
- * @param {Parent} parent
+ * @param {Parent | null | undefined} parent
  * @returns {string}
  */
 export function all(ctx, parent) {
@@ -133,7 +133,7 @@ export function element(ctx, node, index, parent) {
 
 /**
  * @param {Context} ctx
- * @param {Properties} props
+ * @param {Properties | undefined} props
  * @returns {string}
  */
 function serializeAttributes(ctx, props) {
@@ -142,20 +142,20 @@ function serializeAttributes(ctx, props) {
   let index = -1
   /** @type {string} */
   let key
-  /** @type {string} */
-  let value
-  /** @type {string} */
-  let last
 
-  for (key in props) {
-    if (props[key] !== undefined && props[key] !== null) {
-      value = serializeAttribute(ctx, key, props[key])
-      if (value) values.push(value)
+  if (props) {
+    for (key in props) {
+      if (props[key] !== undefined && props[key] !== null) {
+        const value = serializeAttribute(ctx, key, props[key])
+        if (value) values.push(value)
+      }
     }
   }
 
   while (++index < values.length) {
-    last = ctx.tight ? values[index].charAt(values[index].length - 1) : null
+    const last = ctx.tight
+      ? values[index].charAt(values[index].length - 1)
+      : null
 
     // In tight mode, don’t add a space after quoted attributes.
     if (index !== values.length - 1 && last !== '"' && last !== "'") {
@@ -176,7 +176,7 @@ function serializeAttributes(ctx, props) {
 function serializeAttribute(ctx, key, value) {
   const info = find(ctx.schema, key)
   let quote = ctx.quote
-  /** @type {string} */
+  /** @type {string | undefined} */
   let result
 
   if (info.overloadedBoolean && (value === info.attribute || value === '')) {

Diff for: lib/types.js

@@ -17,17 +17,17 @@
  * @callback Handle
  * @param {Context} context
  * @param {Node} node
- * @param {number|null} index
- * @param {Parent|null} parent
+ * @param {number | null} index
+ * @param {Parent | undefined | null} parent
  * @returns {string}
  *
  * @callback OmitHandle
  * @param {Element} node
- * @param {number|null} index
- * @param {Parent|null} parent
+ * @param {number | null | undefined} index
+ * @param {Parent | null | undefined} parent
  * @returns {boolean}
  *
- * @typedef {{opening?: OmitHandle, closing?: OmitHandle}} Omission
+ * @typedef {{opening: OmitHandle, closing: OmitHandle}} Omission
  *
  * @typedef {'html'|'svg'} Space
  * @typedef {'"'|"'"} Quote
@@ -57,7 +57,7 @@
  * @property {number} valid
  * @property {number} safe
  * @property {Schema} schema
- * @property {Omission} omit
+ * @property {Omission | undefined} omit
  * @property {Quote} quote
  * @property {Quote} alternative
  * @property {boolean} smart

Diff for: package.json

@@ -52,7 +52,6 @@
     "prettier": "^2.0.0",
     "remark-cli": "^11.0.0",
     "remark-preset-wooorm": "^9.0.0",
-    "rimraf": "^3.0.0",
     "tape": "^5.0.0",
     "type-coverage": "^2.0.0",
     "typescript": "^4.0.0",
@@ -61,9 +60,9 @@
   },
   "scripts": {
     "prepack": "npm run build && npm run format",
-    "build": "rimraf \"{lib,test}/**/*.d.ts\" \"*.d.ts\" && tsc && type-coverage",
+    "build": "tsc --build --clean && tsc --build && type-coverage",
     "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
-    "test-api": "node test/index.js",
+    "test-api": "node --conditions development test/index.js",
     "test-coverage": "c8 --check-coverage --100 --reporter lcov npm run test-api",
     "test": "npm run build && npm run format && npm run test-coverage"
   },

Diff for: tsconfig.json

@@ -1,15 +1,17 @@
 {
-  "include": ["*.js", "lib/**/*.js", "test/**/*.js"],
+  "include": ["**/**.js"],
+  "exclude": ["coverage/", "node_modules/"],
   "compilerOptions": {
-    "target": "ES2020",
-    "lib": ["ES2020"],
-    "module": "ES2020",
-    "moduleResolution": "node",
-    "allowJs": true,
     "checkJs": true,
     "declaration": true,
     "emitDeclarationOnly": true,
-    "allowSyntheticDefaultImports": true,
-    "skipLibCheck": true
+    "exactOptionalPropertyTypes": true,
+    "forceConsistentCasingInFileNames": true,
+    "lib": ["es2020"],
+    "module": "node16",
+    "newLine": "lf",
+    "skipLibCheck": true,
+    "strict": true,
+    "target": "es2020"
   }
 }