Adjust webpack config to webpack 5 node polyfill removal

packages/react-error-overlay/package.json

@@ -83,5 +83,8 @@
       "/fixtures/",
       "setupJest.js"
     ]
+  },
+  "dependencies": {
+    "path-browserify": "^1.0.0"
   }
 }

packages/react-error-overlay/src/utils/unmapper.js

@@ -9,7 +9,7 @@
 import StackFrame from './stack-frame';
 import { getSourceMap } from './getSourceMap';
 import { getLinesAround } from './getLinesAround';
-import path from 'path';
+import path from 'path-browserify';
 
 function count(search: string, string: string): number {
   // Count starts at -1 because a do-while loop always runs at least once

packages/react-error-overlay/webpack.config.js

@@ -38,10 +38,11 @@ module.exports = {
   optimization: {
     nodeEnv: false,
   },
-  node: {
-    fs: 'empty',
-    process: false,
-  },
+  // todo: Update before final webpack 5 release
+  // node: {
+  //   fs: 'empty',
+  //   process: false,
+  // },
   performance: {
     hints: false,
   },

packages/react-scripts/config/webpack.config.js

@@ -726,16 +726,17 @@ module.exports = function(webpackEnv) {
     ].filter(Boolean),
     // Some libraries import Node modules but don't use them in the browser.
     // Tell Webpack to provide empty mocks for them so importing them works.
-    node: {
-      module: 'empty',
-      dgram: 'empty',
-      dns: 'mock',
-      fs: 'empty',
-      http2: 'empty',
-      net: 'empty',
-      tls: 'empty',
-      child_process: 'empty',
-    },
+    // todo: Update before final webpack 5 release
+    // node: {
+    //   module: 'empty',
+    //   dgram: 'empty',
+    //   dns: 'mock',
+    //   fs: 'empty',
+    //   http2: 'empty',
+    //   net: 'empty',
+    //   tls: 'empty',
+    //   child_process: 'empty',
+    // },
     // Turn off performance processing because we utilize
     // our own hints via the FileSizeReporter
     performance: false,