Use ESM

Author: wooorm

.gitignore

@@ -1,8 +1,5 @@
 .DS_Store
 *.log
-.nyc_output/
 coverage/
 node_modules/
-unist-util-inspect.js
-unist-util-inspect.min.js
 yarn.lock

.prettierignore

@@ -1,5 +1,2 @@
 coverage/
-unist-util-inspect.js
-unist-util-inspect.min.js
-*.json
 *.md

color-browser.js

@@ -1 +1 @@
-module.exports = false
+export var color = false

color.js

@@ -1 +1 @@
-module.exports = true
+export var color = true

index.js

@@ -1,29 +1,33 @@
-'use strict'
+import {color} from './color.js'
 
-var color = require('./color')
+export var inspect = color
+  ? inspectColor
+  : /* istanbul ignore next */ inspectNoColor
 
-module.exports = color ? inspect : /* istanbul ignore next */ noColor
-
-inspect.color = noColor.color = inspect
-inspect.noColor = noColor.noColor = noColor
+var own = {}.hasOwnProperty
 
 var bold = ansiColor(1, 22)
 var dim = ansiColor(2, 22)
 var yellow = ansiColor(33, 39)
 var green = ansiColor(32, 39)
 
 // ANSI color regex.
+/* eslint-disable-next-line no-control-regex */
 var colorExpression = /(?:(?:\u001B\[)|\u009B)(?:\d{1,3})?(?:(?:;\d{0,3})*)?[A-M|f-m]|\u001B[A-M]/g
 
 // Inspects a node, without using color.
-function noColor(node) {
-  return inspect(node).replace(colorExpression, '')
+export function inspectNoColor(node) {
+  return inspectColor(node).replace(colorExpression, '')
 }
 
 // Inspects a node.
-function inspect(tree, options) {
+export function inspectColor(tree, options) {
   var positions =
-    !options || options.showPositions == null ? true : options.showPositions
+    !options ||
+    options.showPositions === null ||
+    options.showPositions === undefined
+      ? true
+      : options.showPositions
 
   return inspectValue(tree)
 
@@ -69,6 +73,8 @@ function inspect(tree, options) {
     var formatted
 
     for (key in object) {
+      if (!own.call(object, key)) continue
+
       value = object[key]
 
       if (
@@ -116,7 +122,7 @@ function inspect(tree, options) {
 
     return indent(
       result.join('\n'),
-      (object.children && object.children.length ? dim('│') : ' ') + ' '
+      (object.children && object.children.length > 0 ? dim('│') : ' ') + ' '
     )
   }
 
@@ -176,8 +182,8 @@ function stringifyPosition(value) {
   point(position.start)
   point(position.end)
 
-  if (positions.length) result.push(positions.join('-'))
-  if (offsets.length) result.push(offsets.join('-'))
+  if (positions.length > 0) result.push(positions.join('-'))
+  if (offsets.length > 0) result.push(offsets.join('-'))
 
   return result.join(', ')
 

package.json

@@ -23,47 +23,45 @@
   "contributors": [
     "Titus Wormer <[email protected]> (https://wooorm.com)"
   ],
-  "files": [
-    "index.js",
-    "color.js",
-    "color-browser.js",
-    "types/index.d.ts"
-  ],
+  "sideEffects": false,
+  "type": "module",
+  "main": "index.js",
+  "types": "types/index.d.ts",
   "browser": {
     "./color.js": "./color-browser.js"
   },
   "react-native": {
     "./color.js": "./color-browser.js"
   },
-  "types": "types/index.d.ts",
+  "files": [
+    "types/index.d.ts",
+    "color.js",
+    "color-browser.js",
+    "index.js"
+  ],
   "devDependencies": {
     "@types/unist": "^2.0.0",
-    "browserify": "^17.0.0",
+    "c8": "^7.0.0",
     "chalk": "^4.0.0",
     "dtslint": "^4.0.0",
     "hastscript": "^6.0.0",
-    "nyc": "^15.0.0",
     "prettier": "^2.0.0",
     "remark-cli": "^9.0.0",
     "remark-preset-wooorm": "^8.0.0",
     "retext": "^7.0.0",
     "strip-ansi": "^6.0.0",
     "tape": "^5.0.0",
-    "tinyify": "^3.0.0",
     "unist-builder": "^2.0.0",
     "xast-util-from-xml": "^1.0.0",
     "xastscript": "^2.0.0",
     "xo": "^0.38.0"
   },
   "scripts": {
     "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
-    "build-bundle": "browserify . -s unistUtilInspect -o unist-util-inspect.js",
-    "build-mangle": "browserify . -s unistUtilInspect -o unist-util-inspect.min.js -p tinyify",
-    "build": "npm run build-bundle && npm run build-mangle",
-    "test-api": "node test",
-    "test-coverage": "nyc --reporter lcov tape test.js",
+    "test-api": "node test.js",
+    "test-coverage": "c8 --check-coverage --branches 100 --functions 100 --lines 100 --statements 100 --reporter lcov node test.js",
     "test-types": "dtslint types",
-    "test": "npm run format && npm run build && npm run test-coverage && npm run test-types"
+    "test": "npm run format && npm run test-coverage && npm run test-types"
   },
   "prettier": {
     "tabWidth": 2,
@@ -75,35 +73,12 @@
   },
   "xo": {
     "prettier": true,
-    "esnext": false,
     "rules": {
       "complexity": "off",
-      "eqeqeq": [
-        "error",
-        "always",
-        {
-          "null": "ignore"
-        }
-      ],
-      "guard-for-in": "off",
-      "no-control-regex": "off",
-      "no-eq-null": "off",
-      "no-multi-assign": "off",
-      "import/no-extraneous-dependencies": "off",
-      "unicorn/explicit-length-check": "off",
-      "unicorn/prefer-number-properties": "off",
-      "unicorn/prefer-includes": "off"
-    },
-    "ignore": [
-      "types",
-      "unist-util-inspect.js"
-    ]
-  },
-  "nyc": {
-    "check-coverage": true,
-    "lines": 100,
-    "functions": 100,
-    "branches": 100
+      "import/no-mutable-exports": "off",
+      "no-var": "off",
+      "prefer-arrow-callback": "off"
+    }
   },
   "remarkConfig": {
     "plugins": [

readme.md

@@ -12,6 +12,9 @@
 
 ## Install
 
+This package is [ESM only](https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c):
+Node 12+ is needed to use it and it must be `import`ed instead of `require`d.
+
 [npm][]:
 
 ```sh
@@ -51,6 +54,10 @@ root[2]
 
 ## API
 
+This package exports the following identifiers: `inspect`, `inspectColor`, and
+`inspectNoColor`.
+There is no default export.
+
 ### `inspect(node[, options])`
 
 Inspect the given [`node`][node].
@@ -65,12 +72,13 @@ Whether to include positional information (`boolean`, default: `true`).
 
 `string` — String representing `node`.
 
-### `inspect.<style>[.<style>…](node[, options])`
+### `inspectColor(node[, options])`
+
+Inspect, with ANSI color sequences (default in Node).
 
-Where `<style>` is either `color` or `noColor`.
+### `inspectNoColor(node[, options])`
 
-To explicitly add or remove ANSI sequences, use `inspect.color(node)` or
-`inspect.noColor(node)`.
+Inspect, but without ANSI color sequences (default in browser).
 
 ## Contribute
 

test.js

@@ -1,14 +1,12 @@
-'use strict'
-
-var test = require('tape')
-var chalk = require('chalk')
-var strip = require('strip-ansi')
-var u = require('unist-builder')
-var h = require('hastscript')
-var x = require('xastscript')
-var retext = require('retext')
-var fromXml = require('xast-util-from-xml')
-var inspect = require('.')
+import test from 'tape'
+import chalk from 'chalk'
+import strip from 'strip-ansi'
+import u from 'unist-builder'
+import h from 'hastscript'
+import x from 'xastscript'
+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})
 
@@ -17,16 +15,6 @@ var paragraph = 'Some simple text. Other “sentence”.'
 test('inspect', function (t) {
   t.equal(typeof inspect, 'function', 'should be a `function`')
 
-  t.test('should have `color` and `noColor` properties', function (st) {
-    st.equal(typeof inspect.color, 'function')
-    st.equal(typeof inspect.noColor, 'function')
-
-    st.equal(typeof inspect.color.noColor, 'function')
-    st.equal(typeof inspect.noColor.color, 'function')
-
-    st.end()
-  })
-
   t.end()
 })
 
@@ -69,7 +57,7 @@ test('inspect()', function (t) {
   t.test('should work on non-nodes', function (st) {
     st.equal(strip(inspect('foo')), '"foo"')
     st.equal(strip(inspect(null)), 'null')
-    st.equal(strip(inspect(NaN)), 'null')
+    st.equal(strip(inspect(Number.NaN)), 'null')
     st.equal(strip(inspect(3)), '3')
 
     st.end()
@@ -340,9 +328,9 @@ test('inspect()', function (t) {
   t.end()
 })
 
-test('inspect.noColor()', function (t) {
+test('inspectNoColor()', function (t) {
   t.equal(
-    inspect.noColor(retext().parse(paragraph).children[0].children[0]),
+    inspectNoColor(retext().parse(paragraph).children[0].children[0]),
     [
       'SentenceNode[6] (1:1-1:18, 0-17)',
       '├─0 WordNode[1] (1:1-1:5, 0-4)',
@@ -361,9 +349,9 @@ test('inspect.noColor()', function (t) {
   t.end()
 })
 
-test('inspect.color()', function (t) {
+test('inspectColor()', function (t) {
   t.equal(
-    inspect.color(retext().parse(paragraph).children[0].children[0]),
+    inspectColor(retext().parse(paragraph).children[0].children[0]),
     [
       chalkEnabled.bold('SentenceNode') +
         chalkEnabled.dim('[') +