Refactor code-style

Author: wooorm

index.js

@@ -25,11 +25,11 @@ import {visit} from 'unist-util-visit'
  */
 export function zone(node, name, callback) {
   /** @type {number} */
-  var level
+  let level
   /** @type {Node} */
-  var marker
+  let marker
   /** @type {Parent} */
-  var scope
+  let scope
 
   visit(node, gather)
 
@@ -38,14 +38,10 @@ export function zone(node, name, callback) {
    * @type {Visitor}
    */
   function gather(node, index, parent) {
-    var info = commentMarker(node)
-    var match =
+    const info = commentMarker(node)
+    const match =
       info && info.name === name && info.attributes.match(/(start|end)\b/)
-    var type = match && match[0]
-    /** @type {number} */
-    var start
-    /** @type {Array.<Node>} */
-    var result
+    const type = match && match[0]
 
     if (type) {
       if (!scope && type === 'start') {
@@ -62,9 +58,9 @@ export function zone(node, name, callback) {
         }
 
         if (type === 'end' && !level) {
-          start = scope.children.indexOf(marker)
+          const start = scope.children.indexOf(marker)
 
-          result = callback(
+          const result = callback(
             marker,
             scope.children.slice(start + 1, index),
             node,

package.json

@@ -73,9 +73,7 @@
   "xo": {
     "prettier": true,
     "rules": {
-      "capitalized-comments": "off",
-      "no-var": "off",
-      "prefer-arrow-callback": "off"
+      "capitalized-comments": "off"
     }
   },
   "remarkConfig": {

readme.md

@@ -38,14 +38,16 @@ Foo
 And our script, `example.js`, looks as follows:
 
 ```js
-import toVFile from 'to-vfile'
-import remark from 'remark'
+import {readSync} from 'to-vfile'
+import {remark} from 'remark'
 import {zone} from 'mdast-zone'
 
+const file = readSync('example.md')
+
 remark()
   .use(plugin)
-  .process(toVFile.readSync('example.md'), function(err, file) {
-    if (err) throw err
+  .process(file)
+  .then((file) => {
     console.log(String(file))
   })
 

test/fixtures/nodes/index.js

@@ -4,9 +4,9 @@
  * @param {import('unist').Node} tree
  */
 export default function assertion(t, zone, tree) {
-  var count = 0
+  let count = 0
 
-  t.test('nodes', function (st) {
+  t.test('nodes', (st) => {
     st.plan(6)
 
     zone(tree, 'foo', handle)

test/fixtures/range-children/index.js

@@ -4,9 +4,9 @@
  * @param {import('unist').Node} tree
  */
 export default function assertion(t, zone, tree) {
-  var count = 0
+  let count = 0
 
-  t.test('range-children', function (st) {
+  t.test('range-children', (st) => {
     st.plan(3)
 
     zone(tree, 'foo', handle)

test/fixtures/simple/index.js

@@ -4,7 +4,7 @@
  * @param {import('unist').Node} tree
  */
 export default function assertion(t, zone, tree) {
-  t.test('range', function (st) {
+  t.test('range', (st) => {
     st.plan(5)
 
     zone(tree, 'foo', handle)

test/index.js

@@ -11,16 +11,16 @@ 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
+test('mdast-zone', async (t) => {
+  const root = path.join('test', 'fixtures')
+  const fixtures = fs.readdirSync(root)
+  let index = -1
   /** @type {string} */
-  var output
+  let output
   /** @type {string} */
-  var name
+  let name
   /** @type {(t: Test, zone: Zone, node: Node) => void} */
-  var mod
+  let mod
 
   while (++index < fixtures.length) {
     name = fixtures[index]