Support Node.js v21 (#40)

* Renaming a context property
* Importing extensionless now works

Author: bengl

.github/workflows/ci.yml

@@ -10,7 +10,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x, 14.x, 16.10.0, 16.16.0, 16.17.0, 16.x, 17.x, 18.5.0, 18.x, 20.x]
+        node-version: [12.x, 14.x, 16.10.0, 16.16.0, 16.17.0, 16.x, 17.x, 18.5.0, 18.x, 20.9.0, 20.x, 21.x]
         os: [ubuntu-latest, macos-latest, windows-latest]
 
     steps:

hook.js

@@ -100,7 +100,11 @@ function createHook (meta) {
       return url
     }
 
-    if (context.importAssertions && context.importAssertions.type === 'json') {
+    // Node.js v21 renames importAssertions to importAttributes
+    if (
+      (context.importAssertions && context.importAssertions.type === 'json') ||
+      (context.importAttributes && context.importAttributes.type === 'json')
+    ) {
       return url
     }
 

test/other/import-executable.mjs

@@ -4,6 +4,13 @@
 
 import { rejects } from 'assert'
 (async () => {
+  const [processMajor, processMinor] = process.versions.node.split('.').map(Number)
+  const extensionlessSupported = processMajor >= 21 ||
+    (processMajor === 20 && processMinor >= 10)
+  if (extensionlessSupported) {
+    // Files without extension are supported in Node.js >= 20.10.0
+    return
+  }
   await rejects(() => import('./executable'), {
     name: 'TypeError',
     code: 'ERR_UNKNOWN_FILE_EXTENSION'