Stop using internal webpack importer for eventsource

stellar · Nov 15, 2023 · 2913ccd · 2913ccd
1 parent 29aaf8a
commit 2913ccd
Show file tree

Hide file tree

Showing 3 changed files with 509 additions and 495 deletions.
diff --git a/config/webpack.config.browser.js b/config/webpack.config.browser.js
@@ -63,19 +63,18 @@ const config = {
     new ESLintPlugin({
       overrideConfigFile: path.resolve(__dirname, '../.eslintrc.js')
     }),
-    // Ignore native modules (sodium-native)
-    new webpack.IgnorePlugin({ resourceRegExp: /sodium-native/ }),
-    // Should be polyfilled
+    // Ignore native modules (sodium-native) and opt-in polyfills
+    new webpack.IgnorePlugin({ resourceRegExp: /(sodium-native|eventsource)/ }),
     new webpack.IgnorePlugin({ resourceRegExp: /eventsource/ }),
     new NodePolyfillPlugin({
       includeAliases: ['http', 'https'] // others aren't needed
     }),
     new webpack.ProvidePlugin({
       Buffer: ['buffer', 'Buffer']
-    })
+    }),
   ],
-  watchOptions: {
-    ignored: /(node_modules|coverage|lib|dist)/
+  externals: {
+    "eventsource": "require('eventsource')"
   }
 };
 

diff --git a/package.json b/package.json
@@ -112,7 +112,6 @@
     "eslint-plugin-prefer-import": "^0.0.1",
     "eslint-plugin-prettier": "^5.0.0",
     "eslint-webpack-plugin": "^4.0.1",
-    "eventsource": "^2.0.2",
     "ghooks": "^2.0.4",
     "husky": "^8.0.3",
     "jsdoc": "^4.0.2",