Merge pull request #54 from CodinGame/fix-node-polyfill

[FIX] Restore node polyfill mocking

Author: CGNonofr

package-lock.json

@@ -68,6 +68,7 @@
     "proxy-polyfill": "^0.3.2",
     "rollup": "2.76.0",
     "rollup-plugin-dts": "^4.2.2",
+    "rollup-plugin-polyfill-node": "^0.10.1",
     "rollup-plugin-postcss": "4.0.2",
     "rollup-plugin-visualizer": "5.6.0",
     "rollup-plugin-web-worker-loader": "1.6.1",

package.json

@@ -6,6 +6,7 @@ import eslint from '@rollup/plugin-eslint'
 import { babel } from '@rollup/plugin-babel'
 import * as rollup from 'rollup'
 import * as recast from 'recast'
+import nodePolyfills from 'rollup-plugin-polyfill-node'
 import path from 'path'
 import pkg from './package.json'
 
@@ -19,6 +20,11 @@ const externals = [
 
 const extensions = ['.js', '.ts']
 
+// Ignore those polyfills by marking them as external, rollup will treeshake-out the imports for us
+// We ignore them because it's imported by a code which isn't included in the final bundle
+// But the polyfill doesn't export all required fields and make the build crash
+const IGNORED_NODE_POLYFILLS = new Set(['os', 'fs'])
+
 export default rollup.defineConfig({
   cache: false,
   input: {
@@ -29,6 +35,9 @@ export default rollup.defineConfig({
     moduleSideEffects: false
   },
   external: function isExternal (source, importer, isResolved) {
+    if (IGNORED_NODE_POLYFILLS.has(source)) {
+      return true
+    }
     if (isResolved) {
       return false
     }
@@ -51,6 +60,7 @@ export default rollup.defineConfig({
     }
   }],
   plugins: [
+    nodePolyfills(),
     eslint({
       throwOnError: true,
       throwOnWarning: true,