Refactor code-style

Author: wooorm

color-browser.js

@@ -1 +1 @@
-export var color = false
+export const color = false

color.js

@@ -1 +1 @@
-export var color = true
+export const color = true

index.js

@@ -10,18 +10,18 @@
 import {color} from './color.js'
 
 /* c8 ignore next */
-export var inspect = color ? inspectColor : inspectNoColor
+export const inspect = color ? inspectColor : inspectNoColor
 
-var own = {}.hasOwnProperty
+const own = {}.hasOwnProperty
 
-var bold = ansiColor(1, 22)
-var dim = ansiColor(2, 22)
-var yellow = ansiColor(33, 39)
-var green = ansiColor(32, 39)
+const bold = ansiColor(1, 22)
+const dim = ansiColor(2, 22)
+const yellow = ansiColor(33, 39)
+const green = ansiColor(32, 39)
 
 // ANSI color regex.
 /* eslint-disable no-control-regex */
-var colorExpression =
+const colorExpression =
   /(?:(?:\u001B\[)|\u009B)(?:\d{1,3})?(?:(?:;\d{0,3})*)?[A-M|f-m]|\u001B[A-M]/g
 /* eslint-enable no-control-regex */
 
@@ -43,11 +43,9 @@ export function inspectNoColor(node, options) {
  * @param {InspectOptions} [options]
  * @returns {string}
  */
-export function inspectColor(tree, options) {
-  var positions =
-    !options ||
-    options.showPositions === null ||
-    options.showPositions === undefined
+export function inspectColor(tree, options = {}) {
+  const positions =
+    options.showPositions === null || options.showPositions === undefined
       ? true
       : options.showPositions
 
@@ -86,8 +84,8 @@ export function inspectColor(tree, options) {
    */
   function inspectNodes(nodes) {
     /** @type {Array.<string>} */
-    var result = []
-    var index = -1
+    const result = []
+    let index = -1
 
     while (++index < nodes.length) {
       result.push(
@@ -108,21 +106,20 @@ export function inspectColor(tree, options) {
    * @param {Object.<string, unknown>} object
    * @returns {string}
    */
+  // eslint-disable-next-line complexity
   function inspectFields(object) {
     /** @type {Array.<string>} */
-    var result = []
+    const result = []
     /** @type {string} */
-    var key
-    /** @type {unknown} */
-    var value
-    /** @type {string} */
-    var formatted
+    let key
 
     for (key in object) {
       /* c8 ignore next 1 */
       if (!own.call(object, key)) continue
 
-      value = object[key]
+      const value = object[key]
+      /** @type {string} */
+      let formatted
 
       if (
         value === undefined ||
@@ -182,11 +179,11 @@ export function inspectColor(tree, options) {
    * @returns {string}
    */
   function inspectTree(node) {
-    var result = [formatNode(node)]
+    const result = [formatNode(node)]
     // @ts-expect-error: looks like a record.
-    var fields = inspectFields(node)
+    const fields = inspectFields(node)
     // @ts-ignore looks like a parent.
-    var content = inspectNodes(node.children || [])
+    const content = inspectNodes(node.children || [])
     if (fields) result.push(fields)
     if (content) result.push(content)
     return result.join('\n')
@@ -199,11 +196,11 @@ export function inspectColor(tree, options) {
    * @returns {string}
    */
   function formatNode(node) {
-    var result = [bold(node.type)]
-    /** @type {string} */
+    const result = [bold(node.type)]
+    /** @type {string|undefined} */
     // @ts-expect-error: might be available.
-    var kind = node.tagName || node.name
-    var position = positions ? stringifyPosition(node.position) : ''
+    const kind = node.tagName || node.name
+    const position = positions ? stringifyPosition(node.position) : ''
 
     if (typeof kind === 'string') {
       result.push('<', kind, '>')
@@ -234,8 +231,8 @@ export function inspectColor(tree, options) {
  * @returns {string}
  */
 function indent(value, indentation, ignoreFirst) {
-  var lines = value.split('\n')
-  var index = ignoreFirst ? 0 : -1
+  const lines = value.split('\n')
+  let index = ignoreFirst ? 0 : -1
 
   if (!value) return value
 
@@ -253,13 +250,13 @@ function indent(value, indentation, ignoreFirst) {
 function stringifyPosition(value) {
   /** @type {Position} */
   // @ts-ignore
-  var position = value || {}
+  const position = value || {}
   /** @type {Array.<string>} */
-  var result = []
+  const result = []
   /** @type {Array.<string>} */
-  var positions = []
+  const positions = []
   /** @type {Array.<string>} */
-  var offsets = []
+  const offsets = []
 
   point(position.start)
   point(position.end)

package.json

@@ -80,13 +80,7 @@
     "trailingComma": "none"
   },
   "xo": {
-    "prettier": true,
-    "rules": {
-      "complexity": "off",
-      "import/no-mutable-exports": "off",
-      "no-var": "off",
-      "prefer-arrow-callback": "off"
-    }
+    "prettier": true
   },
   "remarkConfig": {
     "plugins": [

readme.md

@@ -24,10 +24,10 @@ npm install unist-util-inspect
 ## Use
 
 ```js
-var u = require('unist-builder')
-var inspect = require('unist-util-inspect')
+import {u} from 'unist-builder'
+import {inspect} from 'unist-util-inspect'
 
-var tree = u('root', [
+const tree = u('root', [
   u('literal', '1'),
   u('parent', [
     u('void', {id: 'a'}),

test.js

@@ -10,17 +10,17 @@ import retext from 'retext'
 import {fromXml} from 'xast-util-from-xml'
 import {inspect, inspectColor, inspectNoColor} from './index.js'
 
-var chalkEnabled = new chalk.Instance({level: 1})
+const chalkEnabled = new chalk.Instance({level: 1})
 
-var paragraph = 'Some simple text. Other “sentence”.'
+const paragraph = 'Some simple text. Other “sentence”.'
 
-test('inspect', function (t) {
+test('inspect', (t) => {
   t.equal(typeof inspect, 'function', 'should be a `function`')
 
   t.end()
 })
 
-test('inspect()', function (t) {
+test('inspect()', (t) => {
   t.equal(
     strip(inspect(retext().parse(paragraph))),
     [
@@ -56,7 +56,7 @@ test('inspect()', function (t) {
     'should work with a list of nodes'
   )
 
-  t.test('should work on non-nodes', function (st) {
+  t.test('should work on non-nodes', (st) => {
     st.equal(strip(inspect('foo')), '"foo"')
     st.equal(strip(inspect(null)), 'null')
     st.equal(strip(inspect(Number.NaN)), 'null')
@@ -330,7 +330,7 @@ test('inspect()', function (t) {
   t.end()
 })
 
-test('inspectNoColor()', function (t) {
+test('inspectNoColor()', (t) => {
   t.equal(
     inspectNoColor(retext().parse(paragraph).children[0].children[0]),
     [
@@ -351,7 +351,7 @@ test('inspectNoColor()', function (t) {
   t.end()
 })
 
-test('inspectColor()', function (t) {
+test('inspectColor()', (t) => {
   t.equal(
     inspectColor(retext().parse(paragraph).children[0].children[0]),
     [