feat!: ESM only, Node 20+ (#311)

* feat!: ESM only, Node 20+\n\n- Migrate package to ESM-only (type: module, exports map)\n- Update engines to >=20.19.0 and .nvmrc to 20.19.0\n- TypeScript: NodeNext module with ES2022 target, verbatimModuleSyntax\n- Update internal imports to .js extensions for ESM\n- Switch tests from Ava to Vitest (TypeScript) with ava-compat shim\n- Add vitest.config.ts with .snapshots dir; port tests and skip flaky watch-mode\n- Remove Ava/NYC/Codecov and coverage tooling\n- Update CI workflows to Node 20.19.0 and use vitest run\n- Update README for new requirements and ESM usage\n- Rename config files to .cjs for ESM-compat (eslint/prettier/commitlint)\n

* chore(ci): add eslint devDependency so `pnpm lint:js` works in CI

* chore(eslint): configure NodeNext TS resolver; drop inline suppressions

test: remove unused 'compiler' assignments in skipped watch-mode tests

- Add eslint-import-resolver-typescript and resolver settings
- Remove per-line import/no-unresolved disables in src/{index,hooks}.ts
- Update watch-mode and import-update tests to not assign to undeclared 'compiler'

* ci(workflows): avoid duplicate runs on PR pushes; bump actions; move Windows to windows-2022\n\n- Limit push triggers to master; run PR workflows on pull_request only\n- Add concurrency with cancel-in-progress to dedupe per PR/branch\n- Checkout@v4 + fetch-depth: 0 for reliable origin/master refs\n- Setup-node@v4; Windows runner from 2019 -> 2022

* chore(eslint): configure NodeNext resolver; test(ts): allow TS extension imports via tsconfig.eslint

- ESLint: add typescript + node resolvers; remove per-line suppressions in src
- Style: use node: protocol for built-in modules in src
- Tests: declare watch-mode `compiler` handles and add teardown; keep tests skipped
- TS: move "allowImportingTsExtensions" to tsconfig.eslint.json (avoid build error)
- deps(dev): add eslint-import-resolver-typescript so import/no-unresolved resolves .js<->.ts under NodeNext

* Resolve rebase conflicts for ESLint resolver settings and lockfile (eslint-import-resolver-typescript ^4.4.4)

* chore(eslint): configure TypeScript resolver for NodeNext; remove inline import/no-unresolved disables\n\n- Add eslint-import-resolver-typescript and resolver settings to .eslintrc.cjs\n- Drop per-line import/no-unresolved suppressions in src/index.ts and src/hooks.ts\n\ntest: use .js import specifiers throughout and fix skipped watch tests\n\n- Switch tests to import emitted .js (NodeNext-friendly) instead of .ts\n- Remove assignments to undeclared 'compiler' in skipped watch tests\n\nBuild/lint/test all green locally

* chore(eslint): broaden node resolver extensions for ESM/CJS/TS variants\n\nstyle(node): use -prefixed core module specifiers in src/\n\nVerification: lint OK (0 errors, 1 warning); build OK; tests: 44 passed, 5 skipped

* fix: resolve rebase conflicts\n\n- Finalize ESLint resolver settings (NodeNext + TS, broad extensions)\n- Bump eslint-import-resolver-typescript to ^4.4.4; fix duplicate key\n- Normalize pnpm-lock.yaml to HEAD variants

* chore(eslint): include root JS configs in tsconfig.eslint.json (./*.{js,cjs,mjs})

* chore(lint): expand lint-staged to include TS; dedupe eslint --cache flag

- lint-staged: run eslint on *.{js,ts,tsx,cjs,mjs}
- scripts: remove duplicate --cache in lint:js
- lockfile: sync after install

* test(integration): save watcher handle to close in teardown; fix lints config

- import-update.ts/watch-mode.ts: assign result of watch(...) to compiler so test.after can close it
- package.json: de-dupe --cache; expand lint-staged to include TS/JS variants
- tsconfig.eslint.json: use explicit include globs for js/cjs/mjs (no brace expansion)

Verification: lint OK (0 errors, 1 warning); build OK; vitest: 44 passed, 5 skipped

# Conflicts:
#	tsconfig.eslint.json

* docs(readme): remove CJS dynamic import note per Node 20.19.0 review

---------

Co-authored-by: CharlieHelps <[email protected]>

Author: charliecreates[bot]

.eslintrc.cjs

@@ -0,0 +1,21 @@
+module.exports = {
+  extends: ["shellscape/typescript"],
+  parserOptions: {
+    project: ["./tsconfig.eslint.json"],
+    tsconfigRootDir: __dirname,
+  },
+  settings: {
+    // Ensure import/no-unresolved can resolve NodeNext + TS paths where source files
+    // use .js specifiers that map to .ts during authoring.
+    'import/resolver': {
+      typescript: {
+        // Use the ESLint tsconfig which includes tests
+        project: './tsconfig.eslint.json',
+        alwaysTryTypes: true
+      },
+      node: {
+        extensions: ['.js', '.mjs', '.cjs', '.ts', '.mts', '.cts', '.d.ts']
+      }
+    }
+  }
+};

.eslintrc.js

@@ -1,36 +1,46 @@
 name: Windows
 
 on:
+  # Run on all PRs (regardless of base branch)
   pull_request:
     types:
-      - edited
       - opened
       - synchronize
+      - reopened
+      - ready_for_review
+  # Run on direct pushes to the default branch only (avoid duplicate runs for PR branches)
   push:
     branches:
-      - '*'
+      - master
 
 jobs:
   build:
-    runs-on: windows-2019
+    runs-on: windows-2022
+
+    # Ensure only one run per PR/branch is active; cancel superseded runs on new pushes
+    concurrency:
+      group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.head_ref || github.ref_name }}
+      cancel-in-progress: true
 
     strategy:
       matrix:
-        node: ['18']
+        node: ['20.19.0']
 
     name: Node v${{ matrix.node }}
     steps:
       - name: Configure git line-breaks
         run: git config --global core.autocrlf false
 
       - name: Checkout Commit
-        uses: actions/checkout@v1
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
 
       - name: Checkout Master
         run: git branch -f master origin/master
 
       - name: Setup Node
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node }}
 

.github/workflows/node-windows.yml

@@ -1,31 +1,41 @@
 name: Validate
 
 on:
+  # Run on all PRs (regardless of base branch)
   pull_request:
     types:
-      - edited
       - opened
       - synchronize
+      - reopened
+      - ready_for_review
+  # Run on direct pushes to the default branch only (avoid duplicate runs for PR branches)
   push:
     branches:
-      - '*'
+      - master
 
 jobs:
   build:
     runs-on: ubuntu-latest
 
+    # Ensure only one run per PR/branch is active; cancel superseded runs on new pushes
+    concurrency:
+      group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.head_ref || github.ref_name }}
+      cancel-in-progress: true
+
     strategy:
       matrix:
-        node: ['20', '18', '16']
+        node: ['20.19.0']
 
     name: Node v${{ matrix.node }}
 
     steps:
       - name: Checkout Commit
-        uses: actions/checkout@v1
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
 
       - name: Setup Node
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node }}
 
@@ -54,4 +64,4 @@ jobs:
         run: pnpm lint:js
 
       - name: Run Tests
-        run: pnpm ci:coverage
+        run: pnpm ci:test

.github/workflows/validate.yml

@@ -1 +1 @@
-18
+20.19.0

.nvmrc

@@ -0,0 +1 @@
+module.exports = require('eslint-config-shellscape/prettier');

.prettierrc.cjs

@@ -1,16 +1,10 @@
-[tests]: https://img.shields.io/circleci/project/github/shellscape/webpack-manifest-plugin.svg
-[tests-url]: https://circleci.com/gh/shellscape/webpack-manifest-plugin
-[cover]: https://codecov.io/gh/shellscape/webpack-manifest-plugin/branch/master/graph/badge.svg
-[cover-url]: https://codecov.io/gh/shellscape/webpack-manifest-plugin
 [size]: https://packagephobia.now.sh/badge?p=webpack-manifest-plugin
 [size-url]: https://packagephobia.now.sh/result?p=webpack-manifest-plugin
 
 <div align="center">
 	<img width="256" src="https://raw.githubusercontent.com/shellscape/webpack-manifest-plugin/master/assets/manifest.svg?sanitize=true" alt="webpack-manfiest-plugin"><br/><br/>
 </div>
 
-[![tests][tests]][tests-url]
-[![cover][cover]][cover-url]
 [![size][size]][size-url]
 [![libera manifesto](https://img.shields.io/badge/libera-manifesto-lightgrey.svg)](https://liberamanifesto.com)
 
@@ -24,7 +18,7 @@ A Webpack plugin for generating an asset manifest.
 
 `webpack-manifest-plugin` is an [evergreen 🌲](./.github/FAQ.md#what-does-evergreen-mean) module.
 
-This module requires an [Active LTS](https://github.com/nodejs/Release) Node version (v12.0.0+) and Webpack v5.0.0.
+This module now requires Node.js v20.19.0 or newer and Webpack v5.0.0.
 
 ## Contributing
 
@@ -48,19 +42,16 @@ _Note: We recommend using [webpack-nano](https://github.com/shellscape/webpack-n
 
 ## Usage
 
-Create a `webpack.config.js` file:
+Create a `webpack.config.js` or `webpack.config.mjs` file (ESM):
 
 ```js
-const { WebpackManifestPlugin } = require('webpack-manifest-plugin');
-const options = { ... };
-
-module.exports = {
-	// an example entry definition
-	entry: [ 'app.js'	],
-  ...
-  plugins: [
-    new WebpackManifestPlugin(options)
-  ]
+import { WebpackManifestPlugin } from 'webpack-manifest-plugin';
+
+export default {
+  // an example entry definition
+  entry: ['app.js'],
+  // ...
+  plugins: [new WebpackManifestPlugin(/* options */)]
 };
 ```
 

.prettierrc.js

@@ -0,0 +1 @@
+module.exports = { extends: ['@commitlint/config-conventional'] };