Use ESM

Author: wooorm

.gitignore

@@ -1,8 +1,5 @@
 .DS_Store
 *.log
-.nyc_output/
 coverage/
 node_modules/
-hast-util-find-and-replace.js
-hast-util-find-and-replace.min.js
 yarn.lock

.prettierignore

@@ -1,5 +1,2 @@
 coverage/
-hast-util-find-and-replace.js
-hast-util-find-and-replace.min.js
-*.json
 *.md

index.js

@@ -1,18 +1,13 @@
-'use strict'
-
-var visit = require('unist-util-visit-parents')
-var convert = require('hast-util-is-element/convert')
-var escape = require('escape-string-regexp')
-
-var defaultIgnore = ['title', 'script', 'style', 'svg', 'math']
+import {visitParents} from 'unist-util-visit-parents'
+import {convertElement} from 'hast-util-is-element'
+import escape from 'escape-string-regexp'
 
 var splice = [].splice
+var own = {}.hasOwnProperty
 
-module.exports = findAndReplace
-
-findAndReplace.ignore = defaultIgnore
+export const defaultIgnore = ['title', 'script', 'style', 'svg', 'math']
 
-function findAndReplace(tree, find, replace, options) {
+export function findAndReplace(tree, find, replace, options) {
   var settings
   var schema
 
@@ -51,18 +46,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) {
@@ -112,10 +104,10 @@ function findAndReplace(tree, find, replace, options) {
 }
 
 function search(tree, options, handler) {
-  var ignored = convert(options.ignore || defaultIgnore)
+  var ignored = convertElement(options.ignore || defaultIgnore)
   var result = []
 
-  visit(tree, 'text', visitor)
+  visitParents(tree, 'text', visitor)
 
   return result
 
@@ -150,7 +142,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) {
@@ -164,7 +156,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])])
+      }
     }
   }
 

package.json

@@ -23,33 +23,31 @@
   "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",
-    "hast-util-is-element": "^1.1.0",
-    "unist-util-visit-parents": "^3.0.0"
+    "escape-string-regexp": "^5.0.0",
+    "hast-util-is-element": "^2.0.0",
+    "unist-util-visit-parents": "^4.0.0"
   },
   "devDependencies": {
-    "browserify": "^17.0.0",
-    "hastscript": "^6.0.0",
-    "nyc": "^15.0.0",
+    "c8": "^7.0.0",
+    "hastscript": "^7.0.0",
     "prettier": "^2.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 hastUtilFindAndReplace -o hast-util-find-and-replace.js",
-    "build-mangle": "browserify . -s hastUtilFindAndReplace -o hast-util-find-and-replace.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": "npm run format && npm run build && npm run test-coverage"
+    "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": {
     "tabWidth": 2,
@@ -61,20 +59,10 @@
   },
   "xo": {
     "prettier": true,
-    "esnext": false,
     "rules": {
-      "unicorn/prefer-type-error": "off",
-      "guard-for-in": "off"
-    },
-    "ignores": [
-      "hast-util-find-and-replace.js"
-    ]
-  },
-  "nyc": {
-    "check-coverage": true,
-    "lines": 100,
-    "functions": 100,
-    "branches": 100
+      "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
@@ -21,9 +24,9 @@ npm install hast-util-find-and-replace
 ## Use
 
 ```js
-var h = require('hastscript')
-var inspect = require('unist-util-inspect')
-var findAndReplace = require('hast-util-find-and-replace')
+import {h} from 'hastscript'
+import {inspect} from 'unist-util-inspect'
+import {findAndReplace} from 'hast-util-find-and-replace'
 
 var tree = h('p', [
   'Some ',
@@ -69,6 +72,9 @@ element[9] [tagName="p"]
 
 ## API
 
+This package exports the following identifiers: `findAndReplace`, `defaultIgnore`.
+There is no default export.
+
 ### `findAndReplace(tree, find[, replace][, options])`
 
 Find and replace text in a [**hast**][hast] [*tree*][tree].
@@ -103,7 +109,7 @@ Partial matches are not supported.
 *   `options.ignore` (`Test`, default: `['title', 'script', 'style', 'svg',
     'math']`)
     — Any [`hast-util-is-element`][test] compatible test.
-    The default list can be accessed at `findAndReplace.ignore`
+    The default list is exported as `defaultIgnore`
 
 ###### Returns
 

test.js

@@ -1,15 +1,13 @@
-'use strict'
-
-var test = require('tape')
-var h = require('hastscript')
-var findAndReplace = require('.')
+import test from 'tape'
+import {h} from 'hastscript'
+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 h('a', 'importance')
       },
       importance: 'something else'
@@ -174,13 +172,13 @@ test('findAndReplace', function (t) {
 
   t.deepEqual(
     findAndReplace(h('p', 'Some emphasis, importance, and code.'), {
-      importance: function (match) {
+      importance(match) {
         return h('strong', match)
       },
-      code: function (match) {
+      code(match) {
         return h('code', match)
       },
-      emphasis: function (match) {
+      emphasis(match) {
         return h('em', match)
       }
     }),