fix: update resolver to support protocol imports

NOTE: Augment the original builtin behaviour to use `module.isBuiltin` when available, to support protocol imports (e.g. "node:fs/promises"). The original implementation remains as a fallback
tophat · Nov 13, 2023 · ed9fb6f · ed9fb6f
1 parent d5e34a4
commit ed9fb6f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/eslint-import-resolver-require/package.json b/eslint-import-resolver-require/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@tophat/eslint-import-resolver-require",
-    "version": "1.0.0",
+    "version": "1.0.1",
     "repository": {
         "type": "git",
         "url": "https://github.com/tophat/eslint-config.git",

diff --git a/eslint-import-resolver-require/src/index.ts b/eslint-import-resolver-require/src/index.ts
@@ -1,11 +1,12 @@
-import { builtinModules } from 'module'
+import module from 'module'
 import path from 'path'
 
-const builtins = new Set(builtinModules)
+const builtinModules = new Set(module.builtinModules)
+const isBuiltin = module.isBuiltin || builtinModules.has
 
 const resolve = (source: string, file: string) => {
-    if (builtins.has(source)) {
-        return { found: true }
+    if (isBuiltin(source)) {
+        return { found: true, path: null }
     }
 
     try {