Simply replaced to node20

kachick/wait-other-jobs#564

.github/workflows/ci.yml

@@ -25,7 +25,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version: 20
           cache: npm
       - name: install dependencies
         run: npm ci --ignore-scripts
@@ -55,7 +55,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version: 20
           cache: npm
       - name: install dependencies
         run: npm ci --ignore-scripts
@@ -68,7 +68,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v3
         with:
-          node-version: 16
+          node-version: 20
           cache: npm
       - name: install dependencies
         run: npm ci --ignore-scripts

CONTRIBUTING.md

@@ -61,12 +61,3 @@ esbuild app.jsx --bundle --platform=node --packages=external
 
 If you do this, your dependencies must still be present on the file system at run-time since they are no longer included in the bundle.
 ```
-
-## Why using nodejs16 instead of deno/bun/nodejs18?
-
-They are not yet supported in JavaScript action engine.
-
-- <https://github.com/actions/runner/blob/5421fe3f7107f770c904ed4c7e506ae7a5cde2c2/src/Runner.Worker/ActionManifestManager.cs#L492>
-- <https://github.com/kachick/wait-other-jobs/pull/273#issuecomment-1306058624>
-
-After bumped to nodejs18, I'd like to replace jest with [built-in test runner](https://github.com/nodejs/node/pull/42325).

action.yml

@@ -18,5 +18,5 @@ inputs:
     required: false
     default: 'false'
 runs:
-  using: 'node16'
+  using: 'node20'
   main: 'dist/index.js'

flake.nix

@@ -13,15 +13,7 @@
     flake-utils.lib.eachDefaultSystem (system:
       let
         # https://discourse.nixos.org/t/mark-a-devshell-dependency-as-insecure/24354/3
-        pkgs = import nixpkgs
-          {
-            inherit system;
-            config = {
-              permittedInsecurePackages = [
-                "nodejs-16.20.2"
-              ];
-            };
-          };
+        pkgs = nixpkgs.legacyPackages.${system};
       in
       {
         devShells.default = with pkgs;
@@ -31,7 +23,7 @@
               # https://github.com/kachick/dotfiles/pull/228
               bashInteractive
 
-              nodejs_16
+              nodejs_20
               dprint
               cargo-make
               nil

jest.config.ts

@@ -8,7 +8,7 @@ const config: Config = {
     '^.+\\.(t|j)sx?$': [
       'esbuild-jest-transform',
       {
-        'target': 'node16',
+        'target': 'node20',
         'packages': 'external',
       },
     ],

package.json

@@ -4,7 +4,7 @@
   "description": "",
   "main": "lib/main.js",
   "engines": {
-    "node": "16.x"
+    "node": "20.x"
   },
   "scripts": {
     "prepackage": "tsx scripts/rmrf.ts lib dist",

tsconfig.base.json

@@ -1,14 +1,16 @@
 {
-  // Import from https://github.com/tsconfig/bases/commit/0ae4791797d8ba3dd2bca9aca880eb2d3e30e6fb
-  // To avoid issues around tsconfig.extends array
+  // Import from tsconfig/bases
+  //   - https://github.com/tsconfig/bases/blob/5ee68ffc8325546335649525a3553067d2d27830/bases/node20.json
+  //   - https://github.com/tsconfig/bases/blob/5ee68ffc8325546335649525a3553067d2d27830/bases/strictest.json
+  // Inlined to avoid problems around tsconfig.extends array
   "$schema": "https://json.schemastore.org/tsconfig",
-  "display": "Node 16 + Strictest",
+  "display": "Node 20 + Strictest",
   "compilerOptions": {
     "lib": [
-      "es2021"
+      "es2023"
     ],
-    "module": "commonjs",
-    "target": "es2021",
+    "module": "node16",
+    "target": "es2022",
     "strict": true,
     "esModuleInterop": true,
     "skipLibCheck": true,

tsconfig.json

@@ -2,8 +2,6 @@
   // TODO: Use tsconfig/base after all tools support tsconfig.extends array
   "extends": "./tsconfig.base.json",
   "compilerOptions": {
-    // https://www.typescriptlang.org/docs/handbook/esm-node.html
-    "module": "node16",
     "noEmit": true,
     "noUnusedLocals": false
   },