Use ESM

Author: wooorm

.gitignore

@@ -1,8 +1,5 @@
 .DS_Store
 *.log
-.nyc_output/
 coverage/
 node_modules/
-mdast-zone.js
-mdast-zone.min.js
 yarn.lock

.prettierignore

@@ -1,5 +1,2 @@
 coverage/
-mdast-zone.js
-mdast-zone.min.js
-*.json
 *.md

index.js

@@ -1,13 +1,7 @@
-'use strict'
+import {commentMarker} from 'mdast-comment-marker'
+import {visit} from 'unist-util-visit'
 
-var commentMarker = require('mdast-comment-marker')
-var visit = require('unist-util-visit')
-
-module.exports = zone
-
-var splice = [].splice
-
-function zone(node, name, callback) {
+export function zone(node, name, callback) {
   var level
   var marker
   var scope
@@ -44,14 +38,11 @@ function zone(node, name, callback) {
             marker,
             scope.children.slice(start + 1, index),
             node,
-            {start: start, end: index, parent: parent}
+            {start, end: index, parent}
           )
 
           if (result) {
-            splice.apply(
-              scope.children,
-              [start, index - start + 1].concat(result)
-            )
+            scope.children.splice(start, index - start + 1, ...result)
           }
 
           marker = undefined

package.json

@@ -26,39 +26,31 @@
   "contributors": [
     "Titus Wormer <[email protected]> (https://wooorm.com)"
   ],
+  "sideEffects": false,
+  "type": "module",
+  "main": "index.js",
   "files": [
     "index.js"
   ],
   "dependencies": {
-    "mdast-comment-marker": "^1.0.0",
-    "unist-util-visit": "^2.0.0"
+    "mdast-comment-marker": "^2.0.0",
+    "unist-util-visit": "^3.0.0"
   },
   "devDependencies": {
-    "browserify": "^17.0.0",
-    "is-hidden": "^1.0.0",
-    "nyc": "^15.0.0",
+    "c8": "^7.0.0",
+    "is-hidden": "^2.0.0",
     "prettier": "^2.0.0",
     "remark": "^13.0.0",
     "remark-cli": "^9.0.0",
     "remark-preset-wooorm": "^8.0.0",
     "tape": "^5.0.0",
-    "tinyify": "^3.0.0",
-    "xo": "^0.38.0"
+    "xo": "^0.39.0"
   },
   "scripts": {
     "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
-    "build-bundle": "browserify . -s mdastZone -o mdast-zone.js",
-    "build-mangle": "browserify . -s mdastZone -o mdast-zone.min.js -p tinyify",
-    "build": "npm run build-bundle && npm run build-mangle",
-    "test-api": "node test",
-    "test-coverage": "nyc --reporter lcov tape test/index.js",
-    "test": "npm run format && npm run build && npm run test-coverage"
-  },
-  "nyc": {
-    "check-coverage": true,
-    "lines": 100,
-    "functions": 100,
-    "branches": 100
+    "test-api": "node test/index.js",
+    "test-coverage": "c8 --check-coverage --branches 100 --functions 100 --lines 100 --statements 100 --reporter lcov node test/index.js",
+    "test": "npm run format && npm run test-coverage"
   },
   "prettier": {
     "tabWidth": 2,
@@ -70,14 +62,10 @@
   },
   "xo": {
     "prettier": true,
-    "esnext": false,
     "rules": {
-      "unicorn/no-fn-reference-in-iterator": "off",
-      "unicorn/prefer-optional-catch-binding": "off"
-    },
-    "ignores": [
-      "mdast-zone.js"
-    ]
+      "no-var": "off",
+      "prefer-arrow-callback": "off"
+    }
   },
   "remarkConfig": {
     "plugins": [

readme.md

@@ -14,6 +14,9 @@ Useful in [**remark**][remark] plugins.
 
 ## 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
@@ -35,13 +38,13 @@ Foo
 And our script, `example.js`, looks as follows:
 
 ```js
-var vfile = require('to-vfile')
-var remark = require('remark')
-var zone = require('mdast-zone')
+import toVFile from 'to-vfile'
+import remark from 'remark'
+import {zone} from 'mdast-zone'
 
 remark()
   .use(plugin)
-  .process(vfile.readSync('example.md'), function(err, file) {
+  .process(toVFile.readSync('example.md'), function(err, file) {
     if (err) throw err
     console.log(String(file))
   })
@@ -75,6 +78,9 @@ Bar
 
 ## API
 
+This package exports the following identifiers: `zone`.
+There is no default export.
+
 ### `zone(tree, name, handler)`
 
 Search `tree` for comment ranges (“zones”).

test/fixtures/mismatched/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   zone(tree, 'foo', handle)
 
   function handle() {

test/fixtures/nodes/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   var count = 0
 
   t.test('nodes', function (st) {

test/fixtures/non-marker/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   zone(tree, 'foo', handle)
 
   function handle() {

test/fixtures/range-children/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   var count = 0
 
   t.test('range-children', function (st) {

test/fixtures/remove-children/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   zone(tree, 'foo', handle)
 
   function handle(start, nodes, end) {

test/fixtures/remove/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   zone(tree, 'foo', handle)
 
   function handle() {

test/fixtures/replace-children/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   zone(tree, 'foo', handle)
 
   function handle(start, nodes, end) {

test/fixtures/replace/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   zone(tree, 'foo', handle)
 
   function handle() {

test/fixtures/simple/index.js

@@ -1,8 +1,4 @@
-'use strict'
-
-module.exports = assertion
-
-function assertion(t, zone, tree) {
+export default function assertion(t, zone, tree) {
   t.test('range', function (st) {
     st.plan(5)
 

test/index.js

@@ -1,35 +1,36 @@
-'use strict'
-
-var fs = require('fs')
-var path = require('path')
-var test = require('tape')
-var remark = require('remark')
-var hidden = require('is-hidden')
-var zone = require('..')
-
-test('mdast-zone', function (t) {
-  var root = path.join(__dirname, 'fixtures')
+import fs from 'fs'
+import path from 'path'
+import test from 'tape'
+import remark from 'remark'
+import {isHidden} from 'is-hidden'
+import {zone} from '../index.js'
+
+test('mdast-zone', async function (t) {
+  var root = path.join('test', 'fixtures')
   var fixtures = fs.readdirSync(root)
   var index = -1
   var output
   var name
-  var test
+  var mod
 
   while (++index < fixtures.length) {
     name = fixtures[index]
     output = null
 
-    if (hidden(name)) continue
+    if (isHidden(name)) continue
 
     try {
       output = fs.readFileSync(path.join(root, name, 'output.md'))
-    } catch (_) {}
+    } catch {}
 
-    test = require(path.join(root, name))
+    mod =
+      /* eslint-disable-next-line no-await-in-loop */
+      (await import(new URL('fixtures/' + name + '/index.js', import.meta.url)))
+        .default
 
     remark()
       .use(() => (tree) => {
-        test(t, zone, tree)
+        mod(t, zone, tree)
       })
       .process(
         fs.readFileSync(path.join(root, name, 'input.md')),