Feature/esm plus

.eslintrc.json

@@ -1,5 +1,5 @@
 {
-  "extends": "xo/esnext",
+  "extends": "xo",
   "rules": {
     "semi": [
       "error",

.github/workflows/node.js.yml

@@ -16,12 +16,12 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x, 14.x]
+        node-version: [14.x, 16.x]
 
     steps:
     - uses: actions/checkout@v2
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v2
       with:
         node-version: ${{ matrix.node-version }}
     - run: npm ci

.github/workflows/npm-publish.yml

@@ -12,9 +12,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
+      - uses: actions/setup-node@v2
         with:
-          node-version: 12
+          node-version: '16.x'
       - run: npm ci
       - run: npm test
 
@@ -23,25 +23,28 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
+      - uses: actions/setup-node@v2
         with:
-          node-version: 12
-          registry-url: https://registry.npmjs.org/
+          node-version: '16.x'
+          registry-url: 'https://registry.npmjs.org'
       - run: npm ci
       - run: npm publish
         env:
-          NODE_AUTH_TOKEN: ${{secrets.npm_token}}
+          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
 
   publish-gpr:
     needs: build
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      packages: write
     steps:
       - uses: actions/checkout@v2
-      - uses: actions/setup-node@v1
+      - uses: actions/setup-node@v2
         with:
-          node-version: 12
-          registry-url: https://npm.pkg.github.com/
+          node-version: '16.x'
+          registry-url: 'https://npm.pkg.github.com'
       - run: npm ci
       - run: npm publish
         env:
-          NODE_AUTH_TOKEN: ${{secrets.GITHUB_TOKEN}}
+          NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

cli.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-const diff = require('./lib/diff')
+import diff from './lib/diff.js'
 
 const output = ({error, result}) => {
   if (error) {

cli.test.js

@@ -1,5 +1,5 @@
-const {spawn} = require('child_process')
-const test = require('tape')
+import {spawn} from 'node:child_process'
+import test from 'tape'
 
 const run = async args => {
   const stderr = []
@@ -57,10 +57,10 @@ test('cli, when using tree with missing files', async assert => {
         '06.txt',
         '07.txt',
         '08.txt',
-        '09.txt'
-      ].join('\n')
+        '09.txt',
+      ].join('\n'),
     ],
-    code: 0
+    code: 0,
   }
 
   assert.deepEqual(actual, expected, message)
@@ -75,7 +75,7 @@ test('cli, when using invalid source directory', async assert => {
   const expected = {
     stderr: ['invalid path provided "./fixtures/invalid" (ENOENT)'],
     stdout: [],
-    code: 1
+    code: 1,
   }
 
   assert.deepEqual(actual, expected, message)
@@ -90,7 +90,7 @@ test('cli, when using invalid target directory', async assert => {
   const expected = {
     stderr: ['invalid path provided "./fixtures/invalid" (ENOENT)'],
     stdout: [],
-    code: 1
+    code: 1,
   }
 
   assert.deepEqual(actual, expected, message)
@@ -105,7 +105,7 @@ test('cli, when using file as source directory', async assert => {
   const expected = {
     stderr: ['invalid path provided "./fixtures/file.txt"; given file, expected directory (ENOTDIR)'],
     stdout: [],
-    code: 1
+    code: 1,
   }
 
   assert.deepEqual(actual, expected, message)
@@ -120,7 +120,7 @@ test('cli, when using file as target directory', async assert => {
   const expected = {
     stderr: ['invalid path provided "./fixtures/file.txt"; given file, expected directory (ENOTDIR)'],
     stdout: [],
-    code: 1
+    code: 1,
   }
 
   assert.deepEqual(actual, expected, message)

lib/diff.js

@@ -1,4 +1,4 @@
-const list = require('./list')
+import list from './list.js'
 
 const diff = async ({source, target}) => {
   const left = await list(source)
@@ -11,4 +11,4 @@ const diff = async ({source, target}) => {
     .map(file => file.name)
 }
 
-module.exports = diff
+export default diff

lib/diff.test.js

@@ -1,12 +1,12 @@
-const test = require('tape')
-const diff = require('./diff')
+import test from 'tape'
+import diff from './diff.js'
 
 test('diff files, when using same tree', async assert => {
   const message = 'diff result must be empty'
 
   const actual = await diff({
     source: './fixtures/source',
-    target: './fixtures/source'
+    target: './fixtures/source',
   })
   const expected = []
 
@@ -20,7 +20,7 @@ test('diff files, when using distinct tree', async assert => {
 
   const actual = await diff({
     source: './fixtures/source',
-    target: './fixtures/target'
+    target: './fixtures/target',
   })
   const expected = []
 
@@ -34,7 +34,7 @@ test('diff files, when using tree with missing files', async assert => {
 
   const actual = await diff({
     source: './fixtures/source',
-    target: './fixtures/target/a'
+    target: './fixtures/target/a',
   })
   const expected = [
     '00.txt',
@@ -44,7 +44,7 @@ test('diff files, when using tree with missing files', async assert => {
     '06.txt',
     '07.txt',
     '08.txt',
-    '09.txt'
+    '09.txt',
   ]
 
   assert.deepEqual(actual, expected, message)
@@ -58,7 +58,7 @@ test('diff files, when using invalid source directory', async assert => {
   try {
     await diff({
       source: './fixtures/invalid',
-      target: './fixtures/target'
+      target: './fixtures/target',
     })
 
     assert.fail(message)
@@ -78,7 +78,7 @@ test('diff files, when using invalid target directory', async assert => {
   try {
     await diff({
       source: './fixtures/source',
-      target: './fixtures/invalid'
+      target: './fixtures/invalid',
     })
 
     assert.fail(message)
@@ -98,7 +98,7 @@ test('diff files, when using file as source directory', async assert => {
   try {
     await diff({
       source: './fixtures/file.txt',
-      target: './fixtures/target'
+      target: './fixtures/target',
     })
 
     assert.fail(message)
@@ -118,7 +118,7 @@ test('diff files, when using file as target directory', async assert => {
   try {
     await diff({
       source: './fixtures/source',
-      target: './fixtures/file.txt'
+      target: './fixtures/file.txt',
     })
 
     assert.fail(message)

lib/list.js

@@ -1,5 +1,5 @@
-const path = require('path')
-const fs = require('fs').promises
+import path from 'node:path'
+import fs from 'node:fs/promises'
 
 const list = async pathdir => {
   try {
@@ -27,4 +27,4 @@ const list = async pathdir => {
   }
 }
 
-module.exports = list
+export default list

lib/list.test.js

@@ -1,5 +1,5 @@
-const test = require('tape')
-const list = require('./list')
+import test from 'tape'
+import list from './list.js'
 
 test('list files, when using flat tree', async assert => {
   const message = 'should list 10 files'
@@ -15,7 +15,7 @@ test('list files, when using flat tree', async assert => {
     {path: 'fixtures/source', name: '06.txt'},
     {path: 'fixtures/source', name: '07.txt'},
     {path: 'fixtures/source', name: '08.txt'},
-    {path: 'fixtures/source', name: '09.txt'}
+    {path: 'fixtures/source', name: '09.txt'},
   ]
 
   assert.deepEqual(actual, expected, message)
@@ -37,7 +37,7 @@ test('list files, when using tree with branches', async assert => {
     {path: 'fixtures/target/b', name: '03.txt'},
     {path: 'fixtures/target/b/b', name: '04.txt'},
     {path: 'fixtures/target/c', name: '05.txt'},
-    {path: 'fixtures/target/c/c', name: '06.txt'}
+    {path: 'fixtures/target/c/c', name: '06.txt'},
   ]
 
   assert.deepEqual(actual, expected, message)