build: upgrade to Node.js 20 (#29)

* ci: modernize pipeline

* deps: bump dev dependencies

* docs: typo

* build: force exit jest tests to avoid getting stack on unresolved handles

* build: upgrade Node.js 20

* chore: add engine requirement

* ci: restore build job

* ci: use correct output folder

.github/workflows/test.yml

@@ -6,32 +6,44 @@ on:
       - main
 
 jobs:
-  sanity:
+  build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Setup Node
+        uses: actions/setup-node@v3
         with:
-          node-version: 16
-      - name: install devDependencies
+          node-version-file: '.nvmrc'
+          cache: 'npm'
+
+      - name: Install Dependencies
         run: npm ci
-      - name: run the tests
-        run: npm test
-      - name: build new dist
+
+      - name: Build
         run: npx ncc build src/index.js -o dist/reference
-      - name: compare actual build with expected
+
+      - name: Compare Distribution
         run: diff -U 5 dist/reference/index.js dist/main/index.js
-  build:
+
+  test:
     strategy:
       matrix:
         platform: [ubuntu-latest, macos-latest, windows-latest]
     runs-on: ${{ matrix.platform }}
     steps:
-      - uses: actions/checkout@v3
-      - uses: actions/setup-node@v3
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Setup Node
+        uses: actions/setup-node@v3
         with:
-          node-version: 16
-      - name: install devDependencies
+          node-version-file: '.nvmrc'
+          cache: 'npm'
+
+      - name: Install Dependencies
         run: npm ci
-      - name: run the tests
+
+      - name: Test
         run: npm test

.nvmrc

@@ -0,0 +1 @@
+v20

dist/main/index.js

@@ -16314,7 +16314,7 @@ async function awaitExecution(child) {
 
         // Enhanced timeout security: Wait for the process to exit, or no output for 2 minutes.
         // Saucectl is expected to output one . per second, so this is quite conservative to keep
-        // this thereshold at 2 minutes.
+        // this threshold at 2 minutes.
         while (!exited) {
             const currentTime = new Date().getTime();
             const twoMinutes = 2 * 60 * 1000;
@@ -16329,6 +16329,7 @@ async function awaitExecution(child) {
 
 module.exports = { awaitExecution };
 
+
 /***/ }),
 
 /***/ 1430: