Refactor code-style

Author: wooorm

index.js

@@ -10,30 +10,28 @@
  * @returns {string}
  */
 export function toString(node, separator = '') {
-  var index = -1
-  /** @type {Array.<Node>} */
-  var children
-  /** @type {Array.<string>} */
-  var values
+  let index = -1
 
-  // @ts-ignore Looks like an object.
+  // @ts-expect-error Looks like an object.
   if (!node || (!Array.isArray(node) && !node.type)) {
     throw new Error('Expected node, not `' + node + '`')
   }
 
-  // @ts-ignore Looks like a literal.
+  // @ts-expect-error Looks like a literal.
   if (typeof node.value === 'string') return node.value
 
-  // @ts-ignore Looks like a list of nodes or parent.
-  children = ('length' in node ? node : node.children) || []
+  /** @type {Array.<Node>} */
+  // @ts-expect-error Looks like a list of nodes or parent.
+  const children = (Array.isArray(node) ? node : node.children) || []
 
   // Shortcut: This is pretty common, and a small performance win.
   if (children.length === 1 && 'value' in children[0]) {
-    // @ts-ignore Looks like a literal.
+    // @ts-expect-error Looks like a literal.
     return children[0].value
   }
 
-  values = []
+  /** @type {Array.<string>} */
+  const values = []
 
   while (++index < children.length) {
     values[index] = toString(children[index], separator)

package.json

@@ -65,11 +65,7 @@
     "trailingComma": "none"
   },
   "xo": {
-    "prettier": true,
-    "rules": {
-      "no-var": "off",
-      "prefer-arrow-callback": "off"
-    }
+    "prettier": true
   },
   "remarkConfig": {
     "plugins": [

test.js

@@ -2,9 +2,9 @@ import test from 'tape'
 import {u} from 'unist-builder'
 import {toString} from './index.js'
 
-test('toString()', function (t) {
+test('toString()', (t) => {
   t.throws(
-    function () {
+    () => {
       // @ts-expect-error: runtime.
       toString()
     },
@@ -13,7 +13,7 @@ test('toString()', function (t) {
   )
 
   t.throws(
-    function () {
+    () => {
       toString({value: 'foo'})
     },
     /\[object Object]/,