Merge pull request #121 from SpringRoll/feature/184506946-drop_consoles-pixi

Feature/184506946 drop consoles - Pixi

Author: 902seanryan

CHANGELOG.md

@@ -5,6 +5,20 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+
+## [2.1.0] - unreleased
+
+### Changed
+
+- Pixi: added TerserPlugin options to webpack to drop `console.log` and `console.info` in production builds 
+- Pixi: added TerserPlugin options to webpack to maintain function names in debug builds
+
+- Phaser: added TerserPlugin options to webpack to drop `console.log` and `console.info` in production builds 
+- Phaser: added TerserPlugin options to webpack to maintain function names in debug builds
+
+- CreateJS: added TerserPlugin options to webpack to drop `console.log` and `console.info` in production builds 
+- CreateJS: added TerserPlugin options to webpack to maintain function names in debug builds
+
 ## [2.0.0] - 2024-05-09
 
 ### Changed

package-lock.json

@@ -21,12 +21,13 @@
     "eslint-webpack-plugin": "^3.2.0",
     "html-webpack-plugin": "^5.5.0",
     "mini-css-extract-plugin": "^2.6.1",
+    "source-map-loader": "^4.0.1",
+    "springroll": "^2.6.0",
     "style-loader": "^3.3.1",
+    "terser-webpack-plugin": "^5.3.10",
     "webpack": "^5.75.0",
     "webpack-cli": "^4.10.0",
-    "webpack-dev-server": "^4.11.1",
-    "source-map-loader": "^4.0.1",
-    "springroll": "^2.6.0"
+    "webpack-dev-server": "^4.11.1"
   },
   "dependencies": {
     "@pixi/sound": "^6.0.0",

package.json

@@ -6,13 +6,13 @@ const HtmlConfig = require(path.join(__dirname, 'html.config'));
 const CleanPlugin = require('clean-webpack-plugin');
 const os = require('os');
 const ESLintPlugin = require('eslint-webpack-plugin');
+import TerserPlugin from 'terser-webpack-plugin';
 
 const deploy = path.join(__dirname, 'deploy');
-const isProduction = process.env.NODE_ENV == "production";
 
-// keep the env param to be explicit, eslint disable should be removed when template is in use
-// eslint-disable-next-line no-unused-vars
 module.exports = (env) => {
+  const isProduction = !!env.production;
+
   const plugins = [
     new CleanPlugin.CleanWebpackPlugin(),
     new HtmlWebpackPlugin(HtmlConfig),
@@ -133,6 +133,21 @@ module.exports = (env) => {
           use: ['source-map-loader'],
         },
       ]
-    }
+    },
+    optimization: {
+      minimize: true,
+      minimizer: [
+          new TerserPlugin({
+              terserOptions: {
+                  mangle: {
+                      keep_fnames: isProduction ? false : true,
+                  },
+                  compress: {
+                      drop_console: isProduction ? ['log', 'info']: false,
+                  },
+              },
+          }),
+      ],
+  }
   };
 };