Merge pull request #120 from SpringRoll/feature/184506946-drop_consoles-main

Feature/184506946 drop consoles - Main

Author: 902seanryan

CHANGELOG.md

@@ -5,6 +5,22 @@ 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.0.1] - 2024-06-11
+
+### Changed
+
+- added TerserPlugin options to webpack to drop `console.log` and `console.info` in production builds
+- added TerserPlugin options to webpack to maintain function names in debug builds
+
+- 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

@@ -1,6 +1,6 @@
 {
   "name": "springroll-seed",
-  "version": "2.0.0",
+  "version": "2.0.1",
   "description": "",
   "main": "index.js",
   "scripts": {
@@ -21,12 +21,12 @@
     "eslint-webpack-plugin": "^3.2.0",
     "html-webpack-plugin": "^5.5.0",
     "mini-css-extract-plugin": "^2.6.1",
+    "source-map-loader": "^1.0.0",
+    "springroll": "^2.4.4",
     "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": "^1.0.0",
-    "springroll": "^2.4.4"
-  },
-  "dependencies": {}
+    "webpack-dev-server": "^4.11.1"
+  }
 }

package.json

@@ -5,13 +5,13 @@ const CopyPlugin = require('copy-webpack-plugin');
 const HtmlConfig = require(path.join(__dirname, 'html.config'));
 const CleanPlugin = require('clean-webpack-plugin');
 const ESLintPlugin = require('eslint-webpack-plugin');
+const TerserPlugin = require('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),
@@ -27,8 +27,6 @@ module.exports = (env) => {
   return {
     stats: 'errors-only',
 
-    mode: isProduction ? 'production':'development',
-
     devServer: {
       open: true,
       client: { overlay: true },
@@ -109,6 +107,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,
+                  },
+              },
+          }),
+      ],
+  }
   };
 };