chore(cli): better group handling (#1204)

* chore: refactored how configuration is created

* chore(cli): refactor how the CLI handles configuration and cli arguments

* chore(cli): added devltool inside the zero configs

* fix: update lib/groups/StatsGroup.js

Co-Authored-By: Rishabh Chawla <rishabh31121999@gmail.com>

* fix: update lib/groups/ZeroConfigGroup.js

Co-Authored-By: Rishabh Chawla <rishabh31121999@gmail.com>

* fix: update lib/groups/ZeroConfigGroup.js

Co-Authored-By: Rishabh Chawla <rishabh31121999@gmail.com>

* chore(cli): applied suggestions

Co-Authored-By: Rishabh Chawla <rishabh31121999@gmail.com>

* chore(cli): post code review

* chore(cli): grammar and code styling

Co-authored-by: Rishabh Chawla <rishabh31121999@gmail.com>

Author: ematipico

.prettierrc

@@ -1,4 +1,5 @@
 {
-	"singleQuote": true,
-	"trailingComma": "all"
+    "singleQuote": true,
+    "trailingComma": "all",
+    "printWidth": 140
 }

bin/cli.js

@@ -36,7 +36,8 @@ const version = packageJson.engines.node;
 
 if (!semver.satisfies(process.version, version)) {
     const rawVersion = version.replace(/[^\d\.]*/, '');
-    logger.error('webpack CLI requires at least Node v' + rawVersion + '. ' + 'You have ' + process.version + '.\n' + 'See https://webpack.js.org/ ' + 'for migration help and similar.');
+    logger.error(`webpack CLI requires at least Node v ${rawVersion}.  You have ${process.version}.`);
+    logger.error('See https://webpack.js.org/ for migration help and similar.');
     process.exit(1);
 }
 

lib/groups/advanced.js lib/groups/AdvancedGroup.jslib/groups/advanced.js renamed to lib/groups/AdvancedGroup.js

@@ -1,4 +1,4 @@
-const GroupHelper = require('../utils/group-helper');
+const GroupHelper = require('../utils/GroupHelper');
 const logger = require('../utils/logger');
 
 class AdvancedGroup extends GroupHelper {

lib/groups/basic.js lib/groups/BasicGroup.jslib/groups/basic.js renamed to lib/groups/BasicGroup.js

@@ -1,9 +1,20 @@
-const GroupHelper = require('../utils/group-helper');
+const GroupHelper = require('../utils/GroupHelper');
+const { core, groups } = require('../utils/cli-flags');
 
 class BasicGroup extends GroupHelper {
     constructor(options) {
         super(options);
-        this.WEBPACK_OPTION_FLAGS = ['prod', 'dev', 'watch', 'w', 'prod', 'p', 'interactive', 'i', 'defaults', 'progress'];
+        this.WEBPACK_OPTION_FLAGS = core
+            .filter(coreFlag => {
+                return coreFlag.group === groups.BASIC_GROUP;
+            })
+            .reduce((result, flagObject) => {
+                result.push(flagObject.name);
+                if (flagObject.alias) {
+                    result.push(flagObject.alias);
+                }
+                return result;
+            }, []);
     }
     resolveFlags() {
         const { args } = this;
@@ -16,11 +27,11 @@ class BasicGroup extends GroupHelper {
             // const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');
             // this.opts.options.plugins = [new BundleAnalyzerPlugin()];
             // }
-            if (arg == 'sourcemap') {
+            if (arg === 'sourcemap') {
                 this.opts.options.devtool = args[arg] || 'eval';
                 this.opts.outputOptions.devtool = args[arg];
             }
-            if (arg == 'entry') {
+            if (arg === 'entry') {
                 this.opts.options[arg] = this.resolveFilePath(args[arg], 'index.js');
             }
         });

lib/groups/config.js lib/groups/ConfigGroup.jslib/groups/config.js renamed to lib/groups/ConfigGroup.js

@@ -2,9 +2,15 @@ const { existsSync } = require('fs');
 const { resolve, sep, dirname, parse } = require('path');
 const { extensions } = require('interpret');
 
-const GroupHelper = require('../utils/group-helper');
-
-const DEFAULT_CONFIG_LOC = ['.webpack/webpack.config', '.webpack/webpack.config.dev', '.webpack/webpack.config.prod', '.webpack/webpackfile', 'webpack.config'];
+const GroupHelper = require('../utils/GroupHelper');
+
+const DEFAULT_CONFIG_LOC = [
+    '.webpack/webpack.config',
+    '.webpack/webpack.config.dev',
+    '.webpack/webpack.config.prod',
+    '.webpack/webpackfile',
+    'webpack.config',
+];
 
 const getDefaultConfigFiles = () => {
     return DEFAULT_CONFIG_LOC.map(filename => {

lib/groups/help.js lib/groups/HelpGroup.jslib/groups/help.js renamed to lib/groups/HelpGroup.js

@@ -1,7 +1,5 @@
 const path = require('path');
-const GroupHelper = require('../utils/group-helper');
-
-const DEFAULT_FILENAME = 'main.js';
+const GroupHelper = require('../utils/GroupHelper');
 
 class OutputGroup extends GroupHelper {
     constructor(options) {
@@ -11,11 +9,16 @@ class OutputGroup extends GroupHelper {
                 output: {},
             },
         };
+
+        this.strategy = {
+            'output.filename': 'prepend',
+            'output.path': 'prepend',
+        };
     }
 
     parseDirectory(metaData) {
         return {
-            filename: DEFAULT_FILENAME,
+            // filename: DEFAULT_FILENAME,
             path: path.resolve(metaData.dir, metaData.name),
         };
     }

lib/groups/output.js lib/groups/OutputGroup.jslib/groups/output.js renamed to lib/groups/OutputGroup.js

@@ -1,5 +1,8 @@
-const GroupHelper = require('../utils/group-helper');
+const GroupHelper = require('../utils/GroupHelper');
 
+/**
+ * StatsGroup gathers information about the stats options
+ */
 class StatsGroup extends GroupHelper {
     constructor(options) {
         super(options);

lib/groups/stats.js lib/groups/StatsGroup.jslib/groups/stats.js renamed to lib/groups/StatsGroup.js

@@ -0,0 +1,45 @@
+const GroupHelper = require('../utils/GroupHelper');
+
+/**
+ * ZeroConfigGroup creates a zero configuration based on the environment
+ */
+class ZeroConfigGroup extends GroupHelper {
+    constructor(options) {
+        super(options);
+    }
+
+    /**
+     * It determines the mode to pass to webpack compiler
+     * @returns {string} The mode
+     */
+    getEnvFromOptionsAndMode() {
+        if (process.env.NODE_ENV && (process.env.NODE_ENV === 'production' || process.env.NODE_ENV === 'development')) {
+            return process.env.NODE_ENV;
+        } else if (this.args.prod) {
+            return 'production';
+        } else if (this.args.dev) {
+            return 'development';
+        }
+        return 'production';
+    }
+
+    resolveZeroConfig() {
+        const defaultConfigType = this.getEnvFromOptionsAndMode();
+        const defaultConfig = require(`./${defaultConfigType}-config`)();
+
+        const isEntryObject = defaultConfig.entry && defaultConfig.entry instanceof Object;
+        const isOutputDefined = defaultConfig.output && defaultConfig.output.filename;
+        const isConflictingOutput = isEntryObject && isOutputDefined && defaultConfig.output.filename === 'bundle.js';
+        if (isConflictingOutput) {
+            defaultConfig.output.filename = '[name].bundle.js';
+        }
+        this.opts.options = defaultConfig;
+    }
+
+    run() {
+        this.resolveZeroConfig();
+        return this.opts;
+    }
+}
+
+module.exports = ZeroConfigGroup;

lib/groups/ZeroConfigGroup.js

@@ -9,6 +9,7 @@ class GroupHelper {
             options: {},
             processingMessageBuffer: [],
         };
+        this.strategy = undefined;
     }
 
     hyphenToUpperCase(name) {
@@ -24,14 +25,11 @@ class GroupHelper {
         if (!arr) {
             return;
         }
-        const result = {};
-        const arrLength = arr.length;
-        for (let i = 0; i < arrLength; i++) {
-            const key = Object.keys(arr[i])[0];
-            const val = arr[i][key];
-            result[this.hyphenToUpperCase(key)] = val;
-        }
-        return result;
+        return arr.reduce((result, currentItem, index) => {
+            const key = Object.keys(currentItem)[0];
+            result[this.hyphenToUpperCase(key)] = currentItem[key];
+            return result;
+        }, {});
     }
 
     // TODO: to re implement
@@ -100,6 +98,10 @@ class GroupHelper {
         }
         return configPathExists ? predefinedConfigPath : configPath;
     }
+
+    run() {
+        throw new Error('You must implement the "run" function');
+    }
 }
 
 module.exports = GroupHelper;