fix: lock file maintenance (#107)

BREAKING CHANGE: drop node 12, move to esm

Author: renovate[bot]

.baserc.json

@@ -1,4 +1,5 @@
 {
+  "name": "@dword-design/node",
   "seeAlso": [
     { "repository": "output-files", "description": "Output a tree of files and directories by providing an object. Especially useful for testing with real files." },
     { "repository": "expect-mocha-image-snapshot", "description": "A wrapper around jest-image-snapshot that makes it compatible to Mocha." },

.github/workflows/build.yml

@@ -17,7 +17,7 @@ jobs:
             github.event.pull_request.head.ref || '' }}
       - uses: actions/setup-node@v3
         with:
-          node-version: 14
+          node-version: 16
       - run: git config --global user.email "[email protected]"
       - run: git config --global user.name "GitHub Actions"
       - run: yarn --frozen-lockfile
@@ -46,31 +46,31 @@ jobs:
         with:
           node-version: ${{ matrix.node }}
       - run: yarn --frozen-lockfile
-      - run: yarn test
+      - env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: yarn test
       - if: failure()
         uses: actions/upload-artifact@v3
         with:
           name: Image Snapshot Diffs
           path: "**/__image_snapshots__/__diff_output__"
-      - if: matrix.os == 'ubuntu-latest' && matrix.node == 14
-        uses: codecov/codecov-action@v2
+      - if: matrix.os == 'ubuntu-latest' && matrix.node == 16
+        uses: codecov/codecov-action@v3
         with:
-          fail_ci_if_error: true
           token: ${{ secrets.CODECOV_TOKEN }}
     strategy:
       matrix:
-        exclude:
-          - node: 12
+        include:
+          - node: 14
+            os: ubuntu-latest
+          - node: 16
+            os: ubuntu-latest
+          - node: 18
+            os: ubuntu-latest
+          - node: 16
             os: macos-latest
-          - node: 12
+          - node: 16
             os: windows-latest
-        node:
-          - 12
-          - 14
-        os:
-          - macos-latest
-          - windows-latest
-          - ubuntu-latest
 name: build
 on:
   pull_request: {}

.github/workflows/deprecated-dependencies.yml

@@ -24,6 +24,7 @@ jobs:
         with:
           comment: Auto-closing the issue
           issue-number: ${{ steps.create-deprecation-issue.outputs.number }}
+      - uses: gautamkrishnar/keepalive-workflow@v1
 name: deprecated-dependencies
 on:
   schedule:

.github/workflows/sync-metadata.yml

@@ -8,6 +8,7 @@ jobs:
           approve: false
           commitMessage: "fix: write GitHub metadata to package.json [{changes}]"
           githubToken: ${{ secrets.GITHUB_TOKEN }}
+      - uses: gautamkrishnar/keepalive-workflow@v1
 name: sync-metadata
 on:
   schedule:

.gitpod.Dockerfile

@@ -4,9 +4,17 @@ FROM gitpod/workspace-full:latest
 RUN curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | sudo bash
 RUN sudo apt-get install git-lfs
 RUN git lfs install
+
+# https://www.gitpod.io/docs/languages/javascript
+RUN bash -c 'VERSION="16"       && source $HOME/.nvm/nvm.sh && nvm install $VERSION       && nvm use $VERSION && nvm alias default $VERSION'
+RUN echo "nvm use default &>/dev/null" >> ~/.bashrc.d/51-nvm-fix
+
 RUN echo "\nexport PATH=$(yarn global bin):\$PATH" >> /home/gitpod/.bashrc
+
 RUN yarn global add gitpod-env-per-project @babel/node @babel/core
+
 RUN sudo apt-get install -y graphviz
+
 RUN brew install gh
 
 # Puppeteer dependencies

package.json

@@ -22,13 +22,16 @@
   "funding": "https://github.com/sponsors/dword-design",
   "license": "MIT",
   "author": "Sebastian Landwehr <[email protected]>",
+  "type": "module",
+  "exports": "./dist/index.js",
   "main": "dist/index.js",
   "files": [
     "dist"
   ],
   "scripts": {
     "checkUnknownFiles": "base checkUnknownFiles",
     "commit": "base commit",
+    "depcheck": "base depcheck",
     "dev": "base dev",
     "lint": "base lint",
     "prepare": "base prepare",
@@ -40,12 +43,12 @@
     "tmp-promise": "^3.0.0"
   },
   "devDependencies": {
-    "@dword-design/base": "^8.0.0",
+    "@dword-design/base": "^9.2.0",
     "@dword-design/functions": "^4.0.0",
     "fs-extra": "^10.0.0"
   },
   "engines": {
-    "node": ">=12"
+    "node": ">=14"
   },
   "publishConfig": {
     "access": "public"

src/index.spec.js

@@ -1,8 +1,8 @@
 import { delay } from '@dword-design/functions'
-import { ensureDir, exists, outputFile, remove } from 'fs-extra'
+import fs from 'fs-extra'
 import P from 'path'
 
-import self from '.'
+import self from './index.js'
 
 export default {
   'async function': async () => {
@@ -19,14 +19,14 @@ export default {
   },
   dir: async () => {
     const cwd = process.cwd()
-    await ensureDir('foo')
+    await fs.ensureDir('foo')
     let innerCwd
     await self(() => (innerCwd = process.cwd()), {
       dir: 'foo',
     })
     expect(P.basename(innerCwd).startsWith('tmp-')).toBeTruthy()
     expect(P.dirname(innerCwd)).toEqual(P.join(cwd, 'foo'))
-    await remove('foo')
+    await fs.remove('foo')
   },
   'error: async': async () => {
     const cwd = process.cwd()
@@ -39,7 +39,7 @@ export default {
       })
     ).rejects.toThrow()
     expect(path).toBeDefined()
-    expect(await exists(path)).toBeFalsy()
+    expect(await fs.exists(path)).toBeFalsy()
     expect(process.cwd()).toEqual(cwd)
   },
   'error: sync': async () => {
@@ -52,19 +52,19 @@ export default {
       })
     ).rejects.toThrow()
     expect(path).toBeDefined()
-    expect(await exists(path)).toBeFalsy()
+    expect(await fs.exists(path)).toBeFalsy()
     expect(process.cwd()).toEqual(cwd)
   },
   'non-empty': async () => {
     let path
     let innerFileExists = false
     await self(async () => {
       path = process.cwd()
-      await outputFile('foo.txt', 'foo')
-      innerFileExists = await exists('foo.txt')
+      await fs.outputFile('foo.txt', 'foo')
+      innerFileExists = await fs.exists('foo.txt')
     })
     expect(innerFileExists).toBeTruthy()
-    expect(await exists(path)).toBeFalsy()
+    expect(await fs.exists(path)).toBeFalsy()
   },
   prefix: async () => {
     let path
@@ -82,7 +82,7 @@ export default {
     expect(P.dirname(path)).toEqual(cwd)
     await reset()
     expect(process.cwd()).toEqual(cwd)
-    expect(await exists(path)).toBeFalsy()
+    expect(await fs.exists(path)).toBeFalsy()
   },
   simple: async () => {
     const cwd = process.cwd()
@@ -91,37 +91,37 @@ export default {
     expect(process.cwd()).toEqual(cwd)
     expect(P.basename(path).startsWith('tmp-')).toBeTruthy()
     expect(P.dirname(path)).toEqual(cwd)
-    expect(await exists(path)).toBeFalsy()
+    expect(await fs.exists(path)).toBeFalsy()
   },
   'unsafe cleanup and dir': async () => {
     let path
-    await ensureDir('foo')
+    await fs.ensureDir('foo')
     await expect(
       self(
         async () => {
-          await outputFile('test.txt', '')
+          await fs.outputFile('test.txt', '')
           path = process.cwd()
         },
         { dir: 'foo', unsafeCleanup: false }
       )
     ).rejects.toThrow()
     expect(P.dirname(path)).toEqual(P.resolve('foo'))
-    expect(await exists(path)).toBeTruthy()
-    await remove(path)
-    await remove('foo')
+    expect(await fs.exists(path)).toBeTruthy()
+    await fs.remove(path)
+    await fs.remove('foo')
   },
   'unsafe cleanup cwd': async () => {
     let path
     await expect(
       self(
         async () => {
-          await outputFile('test.txt', '')
+          await fs.outputFile('test.txt', '')
           path = process.cwd()
         },
         { unsafeCleanup: false }
       )
     ).rejects.toThrow()
-    expect(await exists(path)).toBeTruthy()
-    await remove(path)
+    expect(await fs.exists(path)).toBeTruthy()
+    await fs.remove(path)
   },
 }