Use ESM

Author: wooorm

Diff for: .gitignore

@@ -1,6 +1,5 @@
 .DS_Store
 *.log
-.nyc_output/
 coverage/
 node_modules/
 yarn.lock

Diff for: .prettierignore

@@ -1,3 +1,2 @@
 coverage/
-*.json
 *.md

Diff for: index.js

@@ -1,14 +1,11 @@
-'use strict'
-
-module.exports = findAndReplace
-
-var visit = require('unist-util-visit-parents')
-var convert = require('unist-util-is/convert')
-var escape = require('escape-string-regexp')
+import escape from 'escape-string-regexp'
+import {visitParents} from 'unist-util-visit-parents'
+import {convert} from 'unist-util-is'
 
+var own = {}.hasOwnProperty
 var splice = [].splice
 
-function findAndReplace(tree, find, replace, options) {
+export function findAndReplace(tree, find, replace, options) {
   var settings
   var schema
 
@@ -47,18 +44,15 @@ function findAndReplace(tree, find, replace, options) {
 
       while (match) {
         position = match.index
-        value = replace.apply(
-          null,
-          [].concat(match, {index: match.index, input: match.input})
-        )
+        value = replace(...match, {index: match.index, input: match.input})
 
         if (value !== false) {
           if (start !== position) {
             nodes.push({type: 'text', value: node.value.slice(start, position)})
           }
 
           if (typeof value === 'string' && value.length > 0) {
-            value = {type: 'text', value: value}
+            value = {type: 'text', value}
           }
 
           if (value) {
@@ -111,7 +105,7 @@ function search(tree, settings, handler) {
   var ignored = convert(settings.ignore || [])
   var result = []
 
-  visit(tree, 'text', visitor)
+  visitParents(tree, 'text', visitor)
 
   return result
 
@@ -146,7 +140,7 @@ function toPairs(schema) {
   var index
 
   if (typeof schema !== 'object') {
-    throw new Error('Expected array or object as schema')
+    throw new TypeError('Expected array or object as schema')
   }
 
   if ('length' in schema) {
@@ -160,7 +154,9 @@ function toPairs(schema) {
     }
   } else {
     for (key in schema) {
-      result.push([toExpression(key), toFunction(schema[key])])
+      if (own.call(schema, key)) {
+        result.push([toExpression(key), toFunction(schema[key])])
+      }
     }
   }
 

Diff for: package.json

@@ -23,27 +23,30 @@
   "contributors": [
     "Titus Wormer <[email protected]> (https://wooorm.com)"
   ],
+  "sideEffects": false,
+  "type": "module",
+  "main": "index.js",
   "files": [
     "index.js"
   ],
   "dependencies": {
-    "escape-string-regexp": "^4.0.0",
-    "unist-util-is": "^4.0.0",
-    "unist-util-visit-parents": "^3.0.0"
+    "escape-string-regexp": "^5.0.0",
+    "unist-util-is": "^5.0.0",
+    "unist-util-visit-parents": "^4.0.0"
   },
   "devDependencies": {
-    "nyc": "^15.0.0",
+    "c8": "^7.0.0",
     "prettier": "^2.0.0",
     "remark-cli": "^9.0.0",
     "remark-preset-wooorm": "^8.0.0",
     "tape": "^5.0.0",
-    "unist-builder": "^2.0.0",
-    "xo": "^0.38.0"
+    "unist-builder": "^3.0.0",
+    "xo": "^0.39.0"
   },
   "scripts": {
     "format": "remark . -qfo && prettier . -w --loglevel warn && xo --fix",
-    "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": "npm run format && npm run test-coverage"
   },
   "prettier": {
@@ -56,18 +59,11 @@
   },
   "xo": {
     "prettier": true,
-    "esnext": false,
     "rules": {
-      "unicorn/prefer-type-error": "off",
-      "guard-for-in": "off"
+      "no-var": "off",
+      "prefer-arrow-callback": "off"
     }
   },
-  "nyc": {
-    "check-coverage": true,
-    "lines": 100,
-    "functions": 100,
-    "branches": 100
-  },
   "remarkConfig": {
     "plugins": [
       "preset-wooorm"

Diff for: 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
@@ -21,9 +24,9 @@ npm install mdast-util-find-and-replace
 ## Use
 
 ```js
-var u = require('unist-builder')
-var inspect = require('unist-util-inspect')
-var findAndReplace = require('mdast-util-find-and-replace')
+import {u} from 'unist-builder'
+import {inspect} from 'unist-util-inspect'
+import {findAndReplace} from 'mdast-util-find-and-replace'
 
 var tree = u('paragraph', [
   u('text', 'Some '),
@@ -65,6 +68,9 @@ paragraph[8]
 
 ## API
 
+This package exports the following identifiers: `findAndReplace`.
+There is no default export.
+
 ### `findAndReplace(tree, find[, replace][, options])`
 
 Find and replace text in [**mdast**][mdast] [*tree*][tree]s.

Diff for: test.js

@@ -1,15 +1,13 @@
-'use strict'
-
-var test = require('tape')
-var u = require('unist-builder')
-var findAndReplace = require('.')
+import test from 'tape'
+import {u} from 'unist-builder'
+import {findAndReplace} from './index.js'
 
 test('findAndReplace', function (t) {
   t.throws(
     function () {
       findAndReplace(create(), true)
     },
-    /^Error: Expected array or object as schema$/,
+    /^TypeError: Expected array or object as schema$/,
     'should throw on invalid search and replaces'
   )
 
@@ -122,7 +120,7 @@ test('findAndReplace', function (t) {
 
   t.deepEqual(
     findAndReplace(create(), {
-      emphasis: function () {
+      emphasis() {
         return u('link', [u('text', 'importance')])
       },
       importance: 'something else'
@@ -166,13 +164,13 @@ test('findAndReplace', function (t) {
     findAndReplace(
       u('paragraph', [u('text', 'Some emphasis, importance, and code.')]),
       {
-        importance: function (value) {
+        importance(value) {
           return u('strong', [u('text', value)])
         },
-        code: function (value) {
+        code(value) {
           return u('inlineCode', value)
         },
-        emphasis: function (value) {
+        emphasis(value) {
           return u('emphasis', [u('text', value)])
         }
       }