Add webpack-bundle-analyzer plugin.

iclanton · iclanton · commit 7c733dc643c1 · 2020-01-27T17:54:03.000-08:00
diff --git a/build-tests/localization-plugin-test/package.json b/build-tests/localization-plugin-test/package.json
@@ -13,6 +13,7 @@
     "@microsoft/rush-stack-compiler-3.5": "0.4.2",
     "ts-loader": "6.0.0",
     "webpack": "~4.31.0",
+    "webpack-bundle-analyzer": "~3.6.0",
     "webpack-cli": "~3.3.2",
     "webpack-dev-server": "~3.9.0"
   }
diff --git a/build-tests/localization-plugin-test/webpack.config.js b/build-tests/localization-plugin-test/webpack.config.js
@@ -4,6 +4,7 @@ const path = require('path');
 const webpack = require('webpack');
 
 const { LocalizationPlugin } = require('@rushstack/localization-plugin');
+const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');
 
 module.exports = function(env) {
   const configuration = {
@@ -15,7 +16,7 @@ module.exports = function(env) {
           loader: require.resolve('ts-loader'),
           exclude: /(node_modules)/,
           options: {
-            compiler: require.resolve('@microsoft/rush-stack-compiler-3.4/node_modules/typescript'),
+            compiler: require.resolve('@microsoft/rush-stack-compiler-3.5/node_modules/typescript'),
             logLevel: 'ERROR',
             configFile: path.resolve(__dirname, 'tsconfig.json')
           }
@@ -73,6 +74,14 @@ module.exports = function(env) {
         serveLocale: {
           locale: 'en-us'
         }
+      }),
+      new BundleAnalyzerPlugin({
+        openAnalyzer: false,
+        analyzerMode: 'static',
+        reportFilename: path.join(__dirname, 'temp', 'stats.html'),
+        generateStatsFile: true,
+        statsFilename: path.join(__dirname, 'temp', 'stats.json'),
+        logLevel: 'error'
       })
     ]
   };
diff --git a/common/config/rush/nonbrowser-approved-packages.json b/common/config/rush/nonbrowser-approved-packages.json
@@ -598,6 +598,10 @@
       "name": "webpack-cli",
       "allowedCategories": [ "tests" ]
     },
+    {
+      "name": "webpack-bundle-analyzer",
+      "allowedCategories": [ "tests" ]
+    },
     {
       "name": "wordwrap",
       "allowedCategories": [ "libraries" ]
diff --git a/common/config/rush/pnpm-lock.yaml b/common/config/rush/pnpm-lock.yaml
diff --git a/webpack/localization-plugin/package.json b/webpack/localization-plugin/package.json
@@ -30,11 +30,11 @@
   "devDependencies": {
     "@types/webpack": "4.39.8",
     "@microsoft/rush-stack-compiler-3.5": "0.4.2",
-    "gulp": "~4.0.2",
     "@microsoft/node-library-build": "6.4.3",
     "@types/glob": "7.1.1",
     "@types/loader-utils": "1.1.3",
     "@types/jju": "1.4.1",
-    "@types/lodash": "4.14.116"
+    "@types/lodash": "4.14.116",
+    "gulp": "~4.0.2"
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,7 @@`
`13`	`13`	`"@microsoft/rush-stack-compiler-3.5": "0.4.2",`
`14`	`14`	`"ts-loader": "6.0.0",`
`15`	`15`	`"webpack": "~4.31.0",`
	`16`	`+ "webpack-bundle-analyzer": "~3.6.0",`
`16`	`17`	`"webpack-cli": "~3.3.2",`
`17`	`18`	`"webpack-dev-server": "~3.9.0"`
`18`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,11 +30,11 @@`
`30`	`30`	`"devDependencies": {`
`31`	`31`	`"@types/webpack": "4.39.8",`
`32`	`32`	`"@microsoft/rush-stack-compiler-3.5": "0.4.2",`
`33`		`- "gulp": "~4.0.2",`
`34`	`33`	`"@microsoft/node-library-build": "6.4.3",`
`35`	`34`	`"@types/glob": "7.1.1",`
`36`	`35`	`"@types/loader-utils": "1.1.3",`
`37`	`36`	`"@types/jju": "1.4.1",`
`38`		`- "@types/lodash": "4.14.116"`
	`37`	`+ "@types/lodash": "4.14.116",`
	`38`	`+ "gulp": "~4.0.2"`
`39`	`39`	`}`
`40`	`40`	`}`