fix: Restore dynamicRequire but for perf_hooks only

Author: kamilogorek

packages/browser/test/package/npm-build.js

@@ -2,17 +2,14 @@ const fs = require('fs');
 const path = require('path');
 const webpack = require('webpack');
 const { JSDOM } = require('jsdom');
-// runTests();
+
 webpack(
   {
     entry: path.join(__dirname, 'test-code.js'),
     output: {
       path: __dirname,
       filename: 'tmp.js',
     },
-    // resolve: {
-    //   mainFields: ['main'],
-    // },
     mode: 'development',
   },
   (err, stats) => {
@@ -28,10 +25,12 @@ webpack(
 
     if (stats.hasErrors()) {
       console.error(info.errors);
+      process.exit(1);
     }
 
     if (stats.hasWarnings()) {
       console.warn(info.warnings);
+      process.exit(1);
     }
 
     runTests();

packages/utils/src/misc.ts

@@ -20,6 +20,16 @@ interface SentryGlobal {
   };
 }
 
+/**
+ * Requires a module which is protected against bundler minification.
+ *
+ * @param request The module path to resolve
+ */
+export function dynamicRequire(mod: any, request: string): any {
+  // tslint:disable-next-line: no-unsafe-any
+  return mod.require(request);
+}
+
 /**
  * Checks whether we're in the Node.js or Browser environment
  *
@@ -365,8 +375,7 @@ const performanceFallback: CrossPlatformPerformance = {
 export const crossPlatformPerformance: CrossPlatformPerformance = (() => {
   if (isNodeEnv()) {
     try {
-      const req = require;
-      const perfHooks = req('perf_hooks') as { performance: CrossPlatformPerformance };
+      const perfHooks = dynamicRequire(module, 'perf_hooks') as { performance: CrossPlatformPerformance };
       return perfHooks.performance;
     } catch (_) {
       return performanceFallback;