Add various methods to configure default plugins

index.js

@@ -102,14 +102,110 @@ const publicApi = {
         return this;
     },
 
+    /**
+     * Allows you to configure the paths and the options passed to the clean-webpack-plugin.
+     * A list of available options can be found at https://github.com/johnagan/clean-webpack-plugin
+     *
+     * For example:
+     *
+     *      Encore.configureCleanWebpackPlugin(['*.js'], (options) => {
+     *          options.dry = true;
+     *      })
+     *
+     * @param {Array} paths Paths that should be cleaned, relative to the "root" option
+     * @param {function} cleanWebpackPluginOptionsCallback
+     * @returns {exports}
+     */
+    configureCleanWebpackPlugin(paths = ['**/*'], cleanWebpackPluginOptionsCallback = () => {}) {
+        webpackConfig.configureCleanWebpackPlugin(paths, cleanWebpackPluginOptionsCallback);
+
+        return this;
+    },
+
+    /**
+     * Allows you to configure the options passed to the DefinePlugin.
+     * A list of available options can be found at https://webpack.js.org/plugins/define-plugin/
+     *
+     * For example:
+     *
+     *      Encore.configureDefinePlugin((options) => {
+     *          options.VERSION = JSON.stringify('1.0.0');
+     *      })
+     *
+     * @param {function} definePluginOptionsCallback
+     * @returns {exports}
+     */
+    configureDefinePlugin(definePluginOptionsCallback = () => {}) {
+        webpackConfig.configureDefinePlugin(definePluginOptionsCallback);
+
+        return this;
+    },
+
+    /**
+     * Allows you to configure the options passed to the extract-text-webpack-plugin.
+     * A list of available options can be found at https://github.com/webpack-contrib/extract-text-webpack-plugin
+     *
+     * For example:
+     *
+     *      Encore.configureExtractTextPlugin((options) => {
+     *          options.ignoreOrder = true;
+     *      })
+     *
+     * @param {function} extractTextPluginOptionsCallback
+     * @returns {exports}
+     */
+    configureExtractTextPlugin(extractTextPluginOptionsCallback = () => {}) {
+        webpackConfig.configureExtractTextPlugin(extractTextPluginOptionsCallback);
+
+        return this;
+    },
+
+    /**
+     * Allows you to configure the options passed to the friendly-errors-webpack-plugin.
+     * A list of available options can be found at https://github.com/geowarin/friendly-errors-webpack-plugin
+     *
+     * For example:
+     *
+     *      Encore.configureFriendlyErrorsPlugin((options) => {
+     *          options.clearConsole = true;
+     *      })
+     *
+     * @param {function} friendlyErrorsPluginOptionsCallback
+     * @returns {exports}
+     */
+    configureFriendlyErrorsPlugin(friendlyErrorsPluginOptionsCallback = () => {}) {
+        webpackConfig.configureFriendlyErrorsPlugin(friendlyErrorsPluginOptionsCallback);
+
+        return this;
+    },
+
+    /**
+     * Allows you to configure the options passed to the LoaderOptionsPlugins.
+     * A list of available options can be found at https://webpack.js.org/plugins/loader-options-plugin/
+     *
+     * For example:
+     *
+     *      Encore.configureLoaderOptionsPlugin((options) => {
+     *          options.minimize = true;
+     *      })
+     *
+     * @param {function} loaderOptionsPluginOptionsCallback
+     * @returns {exports}
+     */
+    configureLoaderOptionsPlugin(loaderOptionsPluginOptionsCallback = () => {}) {
+        webpackConfig.configureLoaderOptionsPlugin(loaderOptionsPluginOptionsCallback);
+
+        return this;
+    },
+
     /**
      * Allows you to configure the options passed to webpack-manifest-plugin.
      * A list of available options can be found at https://github.com/danethurber/webpack-manifest-plugin
      *
      * For example:
      *
-     *      Encore.configureManifestPlugin(function(options){
-     *          options.fileName: '../../var/assets/manifest.json'
+     *      Encore.configureManifestPlugin((options) => {
+     *          options.fileName = '../../var/assets/manifest.json';
      *      })
      *
      * @param {function} manifestPluginOptionsCallback
@@ -121,6 +217,26 @@ const publicApi = {
         return this;
     },
 
+    /**
+     * Allows you to configure the options passed to the uglifyjs-webpack-plugin.
+     * A list of available options can be found at https://github.com/webpack-contrib/uglifyjs-webpack-plugin/tree/v0.4.6
+     *
+     * For example:
+     *
+     *      Encore.configureUglifyJsPlugin((options) => {
+     *          options.compress = false;
+     *          options.beautify = true;
+     *      })
+     *
+     * @param {function} uglifyJsPluginOptionsCallback
+     * @returns {exports}
+     */
+    configureUglifyJsPlugin(uglifyJsPluginOptionsCallback = () => {}) {
+        webpackConfig.configureUglifyJsPlugin(uglifyJsPluginOptionsCallback);
+
+        return this;
+    },
+
     /**
      * Adds a JavaScript file that should be webpacked:
      *

lib/WebpackConfig.js

@@ -30,40 +30,60 @@ function validateRuntimeConfig(runtimeConfig) {
 class WebpackConfig {
     constructor(runtimeConfig) {
         validateRuntimeConfig(runtimeConfig);
+
         this.runtimeConfig = runtimeConfig;
-        this.outputPath = null;
-        this.publicPath = null;
-        this.manifestKeyPrefix = null;
         this.entries = new Map();
         this.styleEntries = new Map();
         this.plugins = [];
+        this.loaders = [];
+
+        // Global settings
+        this.outputPath = null;
+        this.publicPath = null;
+        this.manifestKeyPrefix = null;
+        this.sharedCommonsEntryName = null;
+        this.providedVariables = {};
+        this.configuredFilenames = {};
+
+        // Features/Loaders flags
         this.useVersioning = false;
         this.useSourceMaps = false;
+        this.cleanupOutput = false;
+        this.useImagesLoader = true;
+        this.useFontsLoader = true;
         this.usePostCssLoader = false;
-        this.postCssLoaderOptionsCallback = function() {};
-        this.useSassLoader = false;
-        this.sassLoaderOptionsCallback = function() {};
-        this.sassOptions = {
-            resolve_url_loader: true
-        };
         this.useLessLoader = false;
-        this.lessLoaderOptionsCallback = function() {};
-        this.cleanupOutput = false;
-        this.sharedCommonsEntryName = null;
-        this.providedVariables = {};
-        this.babelConfigurationCallback = function() {};
+        this.useSassLoader = false;
         this.useReact = false;
         this.useVueLoader = false;
-        this.vueLoaderOptionsCallback = () => {};
-        this.loaders = [];
         this.useTypeScriptLoader = false;
-        this.tsConfigurationCallback = function() {};
         this.useForkedTypeScriptTypeChecking = false;
+
+        // Features/Loaders options
+        this.sassOptions = {
+            resolve_url_loader: true
+        };
+
+        // Features/Loaders options callbacks
+        this.postCssLoaderOptionsCallback = () => {};
+        this.sassLoaderOptionsCallback = () => {};
+        this.lessLoaderOptionsCallback = () => {};
+        this.babelConfigurationCallback = () => {};
+        this.vueLoaderOptionsCallback = () => {};
+        this.tsConfigurationCallback = () => {};
+
+        // Plugins options
+        this.cleanWebpackPluginPaths = ['**/*'];
+
+        // Plugins callbacks
+        this.cleanWebpackPluginOptionsCallback = () => {};
+        this.definePluginOptionsCallback = () => {};
+        this.extractTextPluginOptionsCallback = () => {};
         this.forkedTypeScriptTypesCheckOptionsCallback = () => {};
-        this.useImagesLoader = true;
-        this.useFontsLoader = true;
-        this.configuredFilenames = {};
-        this.manifestPluginOptionsCallback = function() {};
+        this.friendlyErrorsPluginOptionsCallback = () => {};
+        this.loaderOptionsPluginOptionsCallback = () => {};
+        this.manifestPluginOptionsCallback = () => {};
+        this.uglifyJsPluginOptionsCallback = () => {};
     }
 
     getContext() {
@@ -118,6 +138,51 @@ class WebpackConfig {
         this.manifestKeyPrefix = manifestKeyPrefix;
     }
 
+    configureCleanWebpackPlugin(paths = ['**/*'], cleanWebpackPluginOptionsCallback = () => {}) {
+        if (!Array.isArray(paths)) {
+            throw new Error('Argument 1 to configureCleanWebpackPlugin() must be an Array');
+        }
+
+        if (typeof cleanWebpackPluginOptionsCallback !== 'function') {
+            throw new Error('Argument 2 to configureCleanWebpackPlugin() must be a callback function');
+        }
+
+        this.cleanWebpackPluginPaths = paths;
+        this.cleanWebpackPluginOptionsCallback = cleanWebpackPluginOptionsCallback;
+    }
+
+    configureDefinePlugin(definePluginOptionsCallback = () => {}) {
+        if (typeof definePluginOptionsCallback !== 'function') {
+            throw new Error('Argument 1 to configureDefinePlugin() must be a callback function');
+        }
+
+        this.definePluginOptionsCallback = definePluginOptionsCallback;
+    }
+
+    configureExtractTextPlugin(extractTextPluginOptionsCallback = () => {}) {
+        if (typeof extractTextPluginOptionsCallback !== 'function') {
+            throw new Error('Argument 1 to configureExtractTextPlugin() must be a callback function');
+        }
+
+        this.extractTextPluginOptionsCallback = extractTextPluginOptionsCallback;
+    }
+
+    configureFriendlyErrorsPlugin(friendlyErrorsPluginOptionsCallback = () => {}) {
+        if (typeof friendlyErrorsPluginOptionsCallback !== 'function') {
+            throw new Error('Argument 1 to configureFriendlyErrorsPlugin() must be a callback function');
+        }
+
+        this.friendlyErrorsPluginOptionsCallback = friendlyErrorsPluginOptionsCallback;
+    }
+
+    configureLoaderOptionsPlugin(loaderOptionsPluginOptionsCallback = () => {}) {
+        if (typeof loaderOptionsPluginOptionsCallback !== 'function') {
+            throw new Error('Argument 1 to configureLoaderOptionsPlugin() must be a callback function');
+        }
+
+        this.loaderOptionsPluginOptionsCallback = loaderOptionsPluginOptionsCallback;
+    }
+
     configureManifestPlugin(manifestPluginOptionsCallback = () => {}) {
         if (typeof manifestPluginOptionsCallback !== 'function') {
             throw new Error('Argument 1 to configureManifestPlugin() must be a callback function');
@@ -126,6 +191,14 @@ class WebpackConfig {
         this.manifestPluginOptionsCallback = manifestPluginOptionsCallback;
     }
 
+    configureUglifyJsPlugin(uglifyJsPluginOptionsCallback = () => {}) {
+        if (typeof uglifyJsPluginOptionsCallback !== 'function') {
+            throw new Error('Argument 1 to configureUglifyJsPlugin() must be a callback function');
+        }
+
+        this.uglifyJsPluginOptionsCallback = uglifyJsPluginOptionsCallback;
+    }
+
     /**
      * Returns the value that should be used as the publicPath,
      * which can be overridden by enabling the webpackDevServer

lib/config-generator.js

@@ -26,7 +26,7 @@ const loaderOptionsPluginUtil = require('./plugins/loader-options');
 const versioningPluginUtil = require('./plugins/versioning');
 const variableProviderPluginUtil = require('./plugins/variable-provider');
 const cleanPluginUtil = require('./plugins/clean');
-const commonChunksPluginUtil = require('./plugins/common-chunks');
+const commonsChunksPluginUtil = require('./plugins/commons-chunks');
 const definePluginUtil = require('./plugins/define');
 const uglifyPluginUtil = require('./plugins/uglify');
 const friendlyErrorPluginUtil = require('./plugins/friendly-errors');
@@ -218,15 +218,15 @@ class ConfigGenerator {
 
         variableProviderPluginUtil(plugins, this.webpackConfig);
 
-        cleanPluginUtil(plugins, this.webpackConfig, ['**/*']);
+        cleanPluginUtil(plugins, this.webpackConfig);
 
-        commonChunksPluginUtil(plugins, this.webpackConfig);
+        commonsChunksPluginUtil(plugins, this.webpackConfig);
 
-        // todo - options here should be configurable
         definePluginUtil(plugins, this.webpackConfig);
+
         uglifyPluginUtil(plugins, this.webpackConfig);
 
-        let friendlyErrorPlugin = friendlyErrorPluginUtil();
+        const friendlyErrorPlugin = friendlyErrorPluginUtil(this.webpackConfig);
         plugins.push(friendlyErrorPlugin);
 
         assetOutputDisplay(plugins, this.webpackConfig, friendlyErrorPlugin);

lib/plugins/clean.js

@@ -16,20 +16,26 @@ const CleanWebpackPlugin = require('clean-webpack-plugin');
  *
  * @param {Array} plugins to push to
  * @param {WebpackConfig} webpackConfig read only variable
- * @param {Array} paths to clean
- * @param {Object} cleanUpOptions
  * @return {void}
  */
-module.exports = function(plugins, webpackConfig, paths, cleanUpOptions = {}) {
+module.exports = function(plugins, webpackConfig) {
 
     if (!webpackConfig.cleanupOutput) {
         return;
     }
 
-    let config = Object.assign({}, cleanUpOptions, {
+    const cleanWebpackPluginOptions = {
         root: webpackConfig.outputPath,
         verbose: false,
-    });
+    };
 
-    plugins.push(new CleanWebpackPlugin(paths, config));
+    webpackConfig.cleanWebpackPluginOptionsCallback.apply(
+        cleanWebpackPluginOptions,
+        [cleanWebpackPluginOptions]
+    );
+
+    plugins.push(new CleanWebpackPlugin(
+        webpackConfig.cleanWebpackPluginPaths,
+        cleanWebpackPluginOptions
+    ));
 };

lib/plugins/define.js

@@ -14,21 +14,24 @@ const webpack = require('webpack');
 /**
  * @param {Array} plugins
  * @param {WebpackConfig} webpackConfig
- * @param {Object} defineOptions
  * @return {void}
  */
-module.exports = function(plugins, webpackConfig, defineOptions = {}) {
+module.exports = function(plugins, webpackConfig) {
 
     if (!webpackConfig.isProduction()) {
         return;
     }
 
-    let defineConfig = Object.assign({}, defineOptions, {
+    const definePluginOptions = {
         'process.env': {
             NODE_ENV: '"production"'
         }
-    });
-    let define = new webpack.DefinePlugin(defineConfig);
+    };
 
-    plugins.push(define);
+    webpackConfig.definePluginOptionsCallback.apply(
+        definePluginOptions,
+        [definePluginOptions]
+    );
+
+    plugins.push(new webpack.DefinePlugin(definePluginOptions));
 };