Refactor code-style

Author: wooorm

Diff for: index.js

@@ -36,7 +36,7 @@ import escape from 'escape-string-regexp'
 import {visitParents} from 'unist-util-visit-parents'
 import {convert} from 'unist-util-is'
 
-var own = {}.hasOwnProperty
+const own = {}.hasOwnProperty
 
 /**
  * @param tree mdast tree
@@ -64,9 +64,9 @@ export const findAndReplace =
      */
     function (tree, find, replace, options) {
       /** @type {Options} */
-      var settings
+      let settings
       /** @type {FindAndReplaceSchema|FindAndReplaceList} */
-      var schema
+      let schema
 
       if (typeof find === 'string' || find instanceof RegExp) {
         // @ts-expect-error don’t expect options twice.
@@ -82,9 +82,9 @@ export const findAndReplace =
         settings = {}
       }
 
-      var ignored = convert(settings.ignore || [])
-      var pairs = toPairs(schema)
-      var pairIndex = -1
+      const ignored = convert(settings.ignore || [])
+      const pairs = toPairs(schema)
+      let pairIndex = -1
 
       while (++pairIndex < pairs.length) {
         visitParents(tree, 'text', visitor)
@@ -94,11 +94,11 @@ export const findAndReplace =
 
       /** @type {import('unist-util-visit-parents').Visitor<Text>} */
       function visitor(node, parents) {
-        var index = -1
+        let index = -1
         /** @type {Parent} */
-        var parent
+        let parent
         /** @type {Parent} */
-        var grandparent
+        let grandparent
 
         while (++index < parents.length) {
           // @ts-expect-error mdast vs. unist parent.
@@ -127,18 +127,18 @@ export const findAndReplace =
        * @returns {VisitorResult}
        */
       function handler(node, parent) {
-        var find = pairs[pairIndex][0]
-        var replace = pairs[pairIndex][1]
+        const find = pairs[pairIndex][0]
+        const replace = pairs[pairIndex][1]
         /** @type {Array.<PhrasingContent>} */
-        var nodes = []
-        var start = 0
-        var index = parent.children.indexOf(node)
+        let nodes = []
+        let start = 0
+        let index = parent.children.indexOf(node)
         /** @type {number} */
-        var position
+        let position
         /** @type {RegExpMatchArray} */
-        var match
+        let match
         /** @type {Array.<PhrasingContent>|PhrasingContent|string|false|undefined|null} */
-        var value
+        let value
 
         find.lastIndex = 0
 
@@ -196,11 +196,11 @@ export const findAndReplace =
  * @returns {Pairs}
  */
 function toPairs(schema) {
-  var index = -1
+  let index = -1
   /** @type {Pairs} */
-  var result = []
+  const result = []
   /** @type {string} */
-  var key
+  let key
 
   if (typeof schema !== 'object') {
     throw new TypeError('Expected array or object as schema')

Diff for: package.json

@@ -66,11 +66,7 @@
     "trailingComma": "none"
   },
   "xo": {
-    "prettier": true,
-    "rules": {
-      "no-var": "off",
-      "prefer-arrow-callback": "off"
-    }
+    "prettier": true
   },
   "remarkConfig": {
     "plugins": [

Diff for: readme.md

@@ -28,7 +28,7 @@ import {u} from 'unist-builder'
 import {inspect} from 'unist-util-inspect'
 import {findAndReplace} from 'mdast-util-find-and-replace'
 
-var tree = u('paragraph', [
+const tree = u('paragraph', [
   u('text', 'Some '),
   u('emphasis', [u('text', 'emphasis')]),
   u('text', ' and '),

Diff for: test.js

@@ -2,9 +2,9 @@ import test from 'tape'
 import {u} from 'unist-builder'
 import {findAndReplace} from './index.js'
 
-test('findAndReplace', function (t) {
+test('findAndReplace', (t) => {
   t.throws(
-    function () {
+    () => {
       // @ts-expect-error runtime.
       findAndReplace(create(), true)
     },
@@ -44,7 +44,7 @@ test('findAndReplace', function (t) {
     findAndReplace(
       create(),
       /em(\w+)is/,
-      function (/** @type {string} */ _, /** @type {string} */ $1) {
+      (/** @type {string} */ _, /** @type {string} */ $1) => {
         return '[' + $1 + ']'
       }
     ),
@@ -61,7 +61,7 @@ test('findAndReplace', function (t) {
   )
 
   t.deepEqual(
-    findAndReplace(create(), 'emphasis', function () {
+    findAndReplace(create(), 'emphasis', () => {
       return u('delete', [u('break')])
     }),
     u('paragraph', [