module: use kNodeModulesRE to detect node_modules

This is faster and more consistent with other places using the regular expression to detect node_modules. PR-URL: nodejs#55243 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
joyeecheung · Feb 5, 2025 · 534c432 · 534c432
1 parent 78a4e05
commit 534c432
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/lib/internal/modules/esm/load.js b/lib/internal/modules/esm/load.js
@@ -5,7 +5,9 @@ const {
   RegExpPrototypeExec,
   decodeURIComponent,
 } = primordials;
-const { kEmptyObject } = require('internal/util');
+const {
+  kEmptyObject,
+} = require('internal/util');
 
 const { defaultGetFormat } = require('internal/modules/esm/get_format');
 const { validateAttributes, emitImportAssertionWarning } = require('internal/modules/esm/assert');

diff --git a/lib/internal/util.js b/lib/internal/util.js
@@ -477,6 +477,10 @@ function spliceOne(list, index) {
 
 const kNodeModulesRE = /^(?:.*)[\\/]node_modules[\\/]/;
 
+function isUnderNodeModules(filename) {
+  return filename && (RegExpPrototypeExec(kNodeModulesRE, filename) !== null);
+}
+
 let getStructuredStackImpl;
 
 function lazyGetStructuredStack() {
@@ -524,7 +528,7 @@ function isInsideNodeModules() {
       ) {
         continue;
       }
-      return RegExpPrototypeExec(kNodeModulesRE, filename) !== null;
+      return isUnderNodeModules(filename);
     }
   }
   return false;
@@ -913,6 +917,7 @@ module.exports = {
   isArrayBufferDetached,
   isError,
   isInsideNodeModules,
+  isUnderNodeModules,
   join,
   lazyDOMException,
   lazyDOMExceptionClass,