Detect node: as builtin

Fixes Resolve `node:` dependencies as built-in #193
dependency-check-team · Nov 22, 2021 · 1f0eafa · 1f0eafa
1 parent 558f047
commit 1f0eafa
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lib/resolve-dependency.js b/lib/resolve-dependency.js
@@ -43,7 +43,8 @@ const getDeps = async function (file, extensions, { deps, seen, core }) {
   const relatives = []
 
   for (let req of requires) {
-    const isCore = builtins.includes(req)
+    const isCore = builtins.includes(req.startsWith('node:') ? req.slice(5) : req)
+
     if (isNotRelative(req) && !isCore) {
       // require('foo/bar') -> require('foo')
       if (req[0] !== '@' && req.includes('/')) req = req.split('/')[0] || ''

diff --git a/test/mock-negative/index.mjs b/test/mock-negative/index.mjs
@@ -1,2 +1,4 @@
+import http from 'http'
+import path from 'node:path'
 import example from 'example'
 import foo from './foo.js'