Merge pull request #5036 from paulkaplan/webpack4

Update to Webpack 4

package.json

@@ -73,7 +73,7 @@
     "chromedriver": "86.0.0",
     "classnames": "2.2.5",
     "cookie": "0.4.1",
-    "copy-webpack-plugin": "0.2.0",
+    "copy-webpack-plugin": "^4.5.1",
     "create-react-class": "15.6.2",
     "css-loader": "0.23.1",
     "email-validator": "2.0.4",
@@ -93,7 +93,7 @@
     "git-bundle-sha": "0.0.2",
     "glob": "5.0.15",
     "google-libphonenumber": "3.2.6",
-    "html-webpack-plugin": "2.22.0",
+    "html-webpack-plugin": "^3.2.0",
     "iso-3166-2": "0.4.0",
     "jest": "^23.6.0",
     "json-loader": "0.5.2",
@@ -141,7 +141,9 @@
     "tap": "14.10.8",
     "url-loader": "2.3.0",
     "watch": "0.16.0",
-    "webpack": "2.7.0",
+    "webpack": "^4.46.0",
+    "webpack-bundle-analyzer": "^4.4.0",
+    "webpack-cli": "^3.1.0",
     "webpack-dev-middleware": "2.0.4",
     "xhr": "2.2.0"
   },

webpack.config.js

-const autoprefixer = require('autoprefixer');
-const CopyWebpackPlugin = require('copy-webpack-plugin');
 const defaults = require('lodash.defaults');
-const HtmlWebpackPlugin = require('html-webpack-plugin');
 const gitsha = require('git-bundle-sha');
 const path = require('path');
 const webpack = require('webpack');
 
+// Plugins
+const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin;
+const CopyWebpackPlugin = require('copy-webpack-plugin');
+const HtmlWebpackPlugin = require('html-webpack-plugin');
+
+// PostCss
+const autoprefixer = require('autoprefixer');
+
 let routes = require('./src/routes.json');
 const templateConfig = require('./src/template-config.js'); // eslint-disable-line global-require
 
@@ -15,6 +20,9 @@ if (process.env.NODE_ENV !== 'production') {
 
 routes = routes.filter(route => !process.env.VIEW || process.env.VIEW === route.view);
 
+const pageRoutes = routes.filter(function (route) {
+    return !route.redirect;
+});
 let VersionPlugin = function (options) {
     this.options = options || {};
     return this;
@@ -48,27 +56,20 @@ VersionPlugin.prototype.apply = function (compiler) {
 };
 
 // Prepare all entry points
-let entry = {
-    common: [
-        // Vendor
-        'react',
-        'react-dom',
-        'react-intl',
-        'redux',
-        // Init
-        './src/init.js'
-    ]
-};
-routes.forEach(function (route) {
-    if (!route.redirect) {
-        entry[route.name] = `./src/views/${route.view}.jsx`;
-    }
+let entry = {};
+
+pageRoutes.forEach(function (route) {
+    entry[route.name] = [
+        './src/init.js',
+        `./src/views/${route.view}.jsx`
+    ];
 });
 
 // Config
 module.exports = {
     entry: entry,
     devtool: process.env.NODE_ENV === 'production' ? 'none' : 'eval',
+    mode: process.env.NODE_ENV === 'production' ? 'production' : 'development',
     output: {
         path: path.resolve(__dirname, 'build'),
         filename: 'js/[name].bundle.js'
@@ -129,12 +130,20 @@ module.exports = {
     node: {
         fs: 'empty'
     },
+    optimization: {
+        splitChunks: {
+            cacheGroups: {
+                common: {
+                    chunks: 'all',
+                    name: 'common',
+                    minChunks: pageRoutes.length // Extract only chunks common to all html pages
+                }
+            }
+        }
+    },
     plugins: [
         new VersionPlugin({length: 5})
-    ].concat(routes
-        .filter(function (route) {
-            return !route.redirect;
-        })
+    ].concat(pageRoutes
         .map(function (route) {
             return new HtmlWebpackPlugin(defaults({}, {
                 title: route.title,
@@ -165,14 +174,7 @@ module.exports = {
         new CopyWebpackPlugin([{
             from: 'node_modules/scratch-gui/dist/static/assets',
             to: 'static/assets'
-        }])
-    ])
-        .concat(process.env.NODE_ENV === 'production' ? [
-            new webpack.optimize.UglifyJsPlugin({
-                sourceMap: true
-            })
-        ] : [])
-        .concat([
+        }]),
         new webpack.DefinePlugin({
             'process.env.NODE_ENV': '"' + (process.env.NODE_ENV || 'development') + '"',
             'process.env.API_HOST': '"' + (process.env.API_HOST || 'https://api.scratch.mit.edu') + '"',
@@ -185,10 +187,9 @@ module.exports = {
             'process.env.STATIC_HOST': '"' + (process.env.STATIC_HOST || 'https://cdn2.scratch.mit.edu') + '"',
             'process.env.SCRATCH_ENV': '"' + (process.env.SCRATCH_ENV || 'development') + '"',
             'process.env.SENTRY_DSN': '"' + (process.env.SENTRY_DSN || '') + '"'
-            }),
-            new webpack.optimize.CommonsChunkPlugin({
-                name: 'common',
-                filename: 'js/common.bundle.js'
         })
     ])
+        .concat(process.env.ANALYZE_BUNDLE === 'true' ? [
+            new BundleAnalyzerPlugin()
+        ] : [])
 };