buffer: fix deprecation warning emit

Due to npm using workers on Windows which inititate processes
for code within node_modules, the current way of testing is a
little too strict to catch all occurrences.

PR-URL: https://github.com/nodejs/node/pull/20163
Fixes: https://github.com/nodejs/node/issues/20160
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Minwoo Jung <minwoo@nodesource.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

1 files changed, 1 insertions, 15 deletions

diff --git a/lib/internal/util.js b/lib/internal/util.js
index 3cd370258b..ce25317b77 100644
--- a/lib/internal/util.js
+++ b/lib/internal/util.js
@@ -332,21 +332,8 @@ function spliceOne(list, index) {
 const kNodeModulesRE = /^(.*)[\\/]node_modules[\\/]/;
 
 let getStructuredStack;
-let mainPrefix;
 
 function isInsideNodeModules() {
-  // Match the prefix of the main file, if it is inside a `node_modules` folder,
-  // up to and including the innermost `/node_modules/` bit, so that
-  // we can later ignore files which are at the same node_modules level.
-  // For example, having the main script at `/c/node_modules/d/d.js`
-  // would match all code coming from `/c/node_modules/`.
-  if (mainPrefix === undefined) {
-    const match = process.mainModule &&
-                  process.mainModule.filename &&
-                  process.mainModule.filename.match(kNodeModulesRE);
-    mainPrefix = match ? match[0] : '';
-  }
-
   if (getStructuredStack === undefined) {
     // Lazy-load to avoid a circular dependency.
     const { runInNewContext } = require('vm');
@@ -375,8 +362,7 @@ function isInsideNodeModules() {
     // it's likely from Node.js core.
     if (!/^\/|\\/.test(filename))
       continue;
-    const match = filename.match(kNodeModulesRE);
-    return !!match && match[0] !== mainPrefix;
+    return kNodeModulesRE.test(filename);
   }
 
   return false;  // This should be unreachable.