v9.0.0 / 2023-09-19

Author: bjornstar

.appveyor.yml

@@ -2,9 +2,8 @@ build: off
 
 environment:
   matrix:
+    - nodejs_version: '20'
     - nodejs_version: '18'
-    - nodejs_version: '16'
-    - nodejs_version: '14'
 
 install:
   - ps: Install-Product node $env:nodejs_version

.github/workflows/nodejs.yml

@@ -8,17 +8,16 @@ on:
 
 jobs:
   build_ubuntu:
-
     runs-on: ubuntu-latest
 
     strategy:
       matrix:
-        node-version: ["14.x", "16.x", "18.x"]
+        node-version: ["18.x", "20.x"]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v3
       with:
         node-version: ${{ matrix.node-version }}
     - run: npm i
@@ -29,13 +28,33 @@ jobs:
 
     strategy:
       matrix:
-        node-version: ["14.x", "16.x", "18.x"]
+        node-version: ["18.x", "20.x"]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
+      uses: actions/setup-node@v3
       with:
         node-version: ${{ matrix.node-version }}
     - run: npm i
     - run: npm test
+
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - uses: actions/setup-node@v3
+      with:
+        node-version: 20
+    - run: npm i
+    - run: npm run lint
+
+  prettier:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - uses: actions/setup-node@v3
+      with:
+        node-version: 20
+    - run: npm i
+    - run: npm run prettier

.travis.yml

@@ -4,7 +4,6 @@ jobs:
       script: npm run lint
 language: node_js
 node_js:
+  - 20
   - 18
-  - 16
-  - 14
 sudo: false

CHANGELOG.md

@@ -1,5 +1,14 @@
 # node-dev
 
+## v9.0.0 / 2023-09-16
+
+- Drop support for node v14 and v16, the new minimum version of node is v18 (@bjornstar)
+- Add support for node v20 (@bjornstar)
+- Removed tests for `experimental-specifier-resolution` as it's no longer supported (@bjornstar)
+- [CI] Test v18 & v20 (@bjornstar)
+- [CI] Perform linting and prettier in github actions (@bjornstar)
+- [`devDependencies`] Update most devDependencies to their latest version (@bjornstar)
+
 ## v8.0.0 / 2022-12-30
 
 - Suppress experimental warnings in node v18 (@tmont)

lib/index.js

@@ -1,8 +1,7 @@
 const { fork } = require('child_process');
 const filewatcher = require('filewatcher');
-const { join } = require('path');
-const semver = require('semver');
-const { pathToFileURL } = require('url');
+const { join } = require('node:path');
+const { pathToFileURL } = require('node:url');
 
 const { clearFactory } = require('./clear');
 const { configureDeps, configureIgnore } = require('./ignore');
@@ -92,20 +91,16 @@ module.exports = function (
   function start() {
     isPaused = false;
 
-    const args = nodeArgs.slice();
-
-    args.push(`--require=${resolveMain(localPath('wrap'))}`);
-
-    const loaderName = semver.satisfies(process.version, '>=16.12.0') ? 'load' : 'get-format';
-
-    const loaderURL = pathToFileURL(resolveMain(localPath(join('loaders', `${loaderName}.mjs`))));
-
-    args.push(`--experimental-loader=${loaderURL.href}`);
+    const loaderURL = pathToFileURL(resolveMain(localPath(join('loaders', 'load.mjs'))));
 
     child = fork(script, scriptArgs, {
       cwd: process.cwd(),
       env: process.env,
-      execArgv: args
+      execArgv: [
+        ...nodeArgs.slice(),
+        `--experimental-loader=${loaderURL.href}`,
+        `--require=${resolveMain(localPath('wrap'))}`
+      ]
     });
 
     if (respawn) {

lib/loaders/load.mjs

@@ -1,16 +1,23 @@
-import { createRequire } from 'module';
-import { fileURLToPath } from 'url';
+import { createRequire } from 'node:module';
+import { fileURLToPath } from 'node:url';
 import { send } from './ipc.mjs';
 
 const require = createRequire(import.meta.url);
 
-export async function load(url, context, defaultLoad) {
+let connectedPort;
+
+export async function initialize({ port } = {}) {
+  connectedPort = port;
+}
+
+export async function load(url, context, nextLoad) {
   const required = url.startsWith('file://') ? fileURLToPath(url) : url;
 
   send({ required });
+  if (connectedPort) connectedPort.postMessage({ required });
 
   try {
-    return await defaultLoad(url, context, defaultLoad);
+    return await nextLoad(url, context);
   } catch (error) {
     if (error.code !== 'ERR_UNKNOWN_FILE_EXTENSION') throw error;
     return require('get-package-type')(required).then(format => {

lib/register-loader.js

@@ -0,0 +1,27 @@
+const { register } = require('node:module');
+const { join } = require('node:path');
+const { pathToFileURL } = require('node:url');
+const semver = require('semver');
+
+const { send } = require('./ipc');
+const localPath = require('./local-path');
+const resolveMain = require('./resolve-main');
+
+exports.registerLoader = () => {
+  if (!semver.satisfies(process.version, '>=20.6.0')) return;
+
+  const loaderURL = pathToFileURL(resolveMain(localPath(join('loaders', 'load.mjs'))));
+
+  const { port1, port2 } = new MessageChannel();
+  port1.on('message', ({ required } = {}) => {
+    send({ required });
+  });
+
+  register(loaderURL.href, {
+    parentURL: loaderURL.href,
+    data: { port: port2 },
+    transferList: [port2]
+  });
+
+  return port1;
+};

lib/wrap.js

@@ -1,11 +1,12 @@
-const { dirname, extname } = require('path');
-const childProcess = require('child_process');
+const childProcess = require('node:child_process');
+const { dirname, extname } = require('node:path');
+const { isMainThread } = require('node:worker_threads');
 const { sync: resolve } = require('resolve');
-const { isMainThread } = require('worker_threads');
 
 const { getConfig } = require('./cfg');
 const hook = require('./hook');
 const { relay, send } = require('./ipc');
+const { registerLoader } = require('./register-loader');
 const resolveMain = require('./resolve-main');
 const suppressExperimentalWarnings = require('./suppress-experimental-warnings');
 
@@ -25,8 +26,14 @@ if (process.env.NODE_DEV_PRELOAD) {
   require(process.env.NODE_DEV_PRELOAD);
 }
 
-// We want to exit on SIGTERM, but defer to existing SIGTERM handlers.
-process.once('SIGTERM', () => process.listenerCount('SIGTERM') || process.exit(0));
+const port = registerLoader();
+
+process.once('SIGTERM', () => {
+  port?.close();
+  // We want to exit on SIGTERM, but defer to existing SIGTERM handlers.
+  if (process.listenerCount('SIGTERM')) return;
+  process.exit(0);
+});
 
 if (fork) {
   // Overwrite child_process.fork() so that we can hook into forked processes

package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-dev",
-  "version": "8.0.0",
+  "version": "9.0.0",
   "description": "Restarts your app when files are modified",
   "keywords": [
     "restart",
@@ -24,35 +24,36 @@
   },
   "main": "./lib",
   "engines": {
-    "node": ">=14"
+    "node": ">=18"
   },
   "scripts": {
     "lint": "eslint lib test bin/node-dev",
-    "test": "node test",
-    "prepare": "husky install"
+    "prepare": "husky install",
+    "prettier": "prettier lib test bin/node-dev",
+    "test": "node test"
   },
   "dependencies": {
     "dateformat": "^3.0.3",
     "dynamic-dedupe": "^0.3.0",
-    "filewatcher": "~3.0.0",
+    "filewatcher": "^3.0.1",
     "get-package-type": "^0.1.0",
     "minimist": "^1.2.6",
     "node-notifier": "^8.0.1",
-    "resolve": "^1.22.0",
-    "semver": "^7.3.7"
+    "resolve": "^1.22.6",
+    "semver": "^7.5.4"
   },
   "devDependencies": {
-    "@types/node": "^18.11.18",
-    "eslint": "^8.30.0",
-    "eslint-plugin-import": "^2.26.0",
-    "husky": "^8.0.2",
-    "lint-staged": "^13.1.0",
-    "prettier": "^2.6.2",
-    "tap": "^16.3.2",
+    "@types/node": "^20.6.2",
+    "eslint": "^8.49.0",
+    "eslint-plugin-import": "^2.28.1",
+    "husky": "^8.0.3",
+    "lint-staged": "^14.0.1",
+    "prettier": "^3.0.3",
+    "tap": "^18.0.4",
     "tap-xunit": "^2.4.1",
     "touch": "^3.1.0",
-    "ts-node": "^10.7.0",
-    "typescript": "^4.6.3"
+    "ts-node": "^10.9.1",
+    "typescript": "^5.2.2"
   },
   "lint-staged": {
     "*.{js,mjs}": "eslint --cache --fix",

test/fixture/cluster.js

@@ -1,4 +1,4 @@
-const { disconnect, fork, isMaster, isWorker } = require('cluster');
+const { fork, isMaster, isWorker } = require('node:cluster');
 
 const createWorker = i => {
   const worker = fork();
@@ -26,15 +26,16 @@ if (isWorker) {
 }
 
 if (isMaster) {
+  const workers = [];
   for (let i = 0; i < 2; i += 1) {
     console.log('Forking worker', i);
-    createWorker(i);
+    workers.push(createWorker(i));
   }
 
   process.once('SIGTERM', () => {
     console.log('Master received SIGTERM');
-    disconnect(() => {
-      console.log('All workers disconnected.');
-    });
+    workers.forEach(worker => worker.kill());
+    console.log('All workers disconnected.');
+    process.exit(0);
   });
 }