module: fix ERR_REQUIRE_ESM error for null frames

PR-URL: #39593 Reviewed-By: Bradley Farias <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Jul 31, 2021 · b0a6ade · b0a6ade
1 parent 46e6c64
commit b0a6ade
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/internal/errors.js b/lib/internal/errors.js
@@ -808,7 +808,7 @@ function hideInternalStackFrames(error) {
     if (typeof stackFrames === 'object') {
       frames = ArrayPrototypeFilter(
         stackFrames,
-        (frm) => !StringPrototypeStartsWith(frm.getFileName(),
+        (frm) => !StringPrototypeStartsWith(frm.getFileName() || '',
                                             'node:internal')
       );
     }

diff --git a/test/fixtures/es-modules/cjs-esm.js b/test/fixtures/es-modules/cjs-esm.js
@@ -1 +1 @@
-require('./package-type-module/cjs.js');
+eval("require('./package-type-module/cjs.js')");
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		require('./package-type-module/cjs.js');
		eval("require('./package-type-module/cjs.js')");