Refactor code-style

Author: wooorm

lib/index.js

@@ -1,10 +1,12 @@
+/* eslint-disable max-depth */
+
 /**
- * @typedef {import('xast').Root} Root
  * @typedef {import('xast').Element} Element
+ * @typedef {import('xast').Root} Root
  */
 
 /**
- * @typedef {string | Omit<Entry, 'lang' | 'alternate'>} Alternate
+ * @typedef {Omit<Entry, 'alternate' | 'lang'> | string} Alternate
  *   Alternative content, typically a translation.
  *
  *   To define different fields, either use a full entry object:
@@ -45,13 +47,14 @@
  *   Entries represent a single URL and describe them with metadata.
  * @property {string} url
  *   Full URL (`<loc>`, example: `'https://example.org/'`).
- * @property {number | string | Date | null | undefined} [modified]
- *   Value indicating when the page last changed (`<lastmod>`).
+ * @property {Date | number | string | null | undefined} [modified]
+ *   Value indicating when the page last changed (`<lastmod>`) (optional).
  * @property {string | null | undefined} [lang]
- *   BCP 47 tag indicating the language of the page (example: `'en-GB'`).
+ *   BCP 47 tag indicating the language of the page (example: `'en-GB'`,
+ *   optional).
  * @property {Record<string, Alternate> | null | undefined} [alternate]
  *   Translations of the page, where each key is a BCP 47 tag and each value an
- *   entry (example: `{nl: 'https://example.nl/'}`).
+ *   entry (example: `{nl: 'https://example.nl/'}`, optional).
  *
  *   Alternate resources inherit fields from the entry they are described in.
  */
@@ -66,11 +69,12 @@ const own = {}.hasOwnProperty
 /**
  * Build a sitemap.
  *
- * @param {Array<string | Entry> | null | undefined} [data]
- *   Entries to build a sitemap for.
+ * @param {Array<Entry | string> | null | undefined} [data]
+ *   Entries to build a sitemap for (optional).
  * @returns {Root}
  *   Sitemap.
  */
+// eslint-disable-next-line complexity
 export function sitemap(data) {
   /** @type {Array<Element>} */
   const nodes = []
@@ -166,7 +170,7 @@ export function sitemap(data) {
 
       nodes.push(node)
 
-      if (entry.modified !== undefined && entry.modified !== null) {
+      if (entry.modified !== null && entry.modified !== undefined) {
         const modified = toDate(entry.modified)
 
         if (Number.isNaN(modified.valueOf())) {
@@ -209,7 +213,7 @@ export function sitemap(data) {
 }
 
 /**
- * @param {string | Entry} d
+ * @param {Entry | string} d
  */
 function toEntry(d) {
   /** @type {Entry} */
@@ -221,10 +225,15 @@ function toEntry(d) {
     url = d
   } else {
     url = d.url
-    if (d.lang !== undefined && d.lang !== null) entry.lang = normalize(d.lang)
-    if (d.modified !== undefined && d.modified !== null)
+
+    if (d.lang !== null && d.lang !== undefined) {
+      entry.lang = normalize(d.lang)
+    }
+
+    if (d.modified !== null && d.modified !== undefined)
       entry.modified = d.modified
-    if (d.alternate !== undefined && d.alternate !== null)
+
+    if (d.alternate !== null && d.alternate !== undefined)
       entry.alternate = d.alternate
   }
 
@@ -233,7 +242,7 @@ function toEntry(d) {
 }
 
 /**
- * @param {Date | string | number} value
+ * @param {Date | number | string} value
  * @returns {Date}
  */
 export function toDate(value) {