feat(webpack): default config

Author: PatrickJS

helpers.js

@@ -1,28 +1,32 @@
 var path = require('path');
 var zlib = require('zlib');
-var validateWebpackConfig = require('webpack-validator');
+var webpackMerge = require('webpack-merge');
+var webpackDefaults = require('./webpack.default.conf.js');
+
 
 // Helper functions
 
-exports.validateWebpackConfig = validateWebpackConfig;
-exports.validate = validateWebpackConfig;
+function defaults(config) {
+  return webpackMerge(webpackDefaults, config);
+}
+
+function hasProcessFlag(flag) {
+  return process.argv.join('').indexOf(flag) > -1;
+}
 
 function gzipMaxLevel(buffer, callback) {
   return zlib['gzip'](buffer, {level: 9}, callback)
 }
-exports.gzipMaxLevel = gzipMaxLevel;
 
 function root(args) {
   args = Array.prototype.slice.call(arguments, 0);
   return path.join.apply(path, [__dirname].concat(args));
 }
-exports.root = root;
 
 function rootNode(args) {
   args = Array.prototype.slice.call(arguments, 0);
   return root.apply(path, ['node_modules'].concat(args));
 }
-exports.rootNode = rootNode;
 
 function prependExt(extensions, args) {
   args = args || [];
@@ -33,5 +37,11 @@ function prependExt(extensions, args) {
     }));
   }, ['']);
 }
+
+exports.defaults = defaults;
+exports.hasProcessFlag = hasProcessFlag;
+exports.gzipMaxLevel = gzipMaxLevel;
+exports.root = root;
+exports.rootNode = rootNode;
 exports.prependExt = prependExt;
 exports.prepend = prependExt;

package.json

@@ -91,9 +91,8 @@
     "url-loader": "^0.5.7",
     "webpack": "^1.12.14",
     "webpack-dev-server": "^1.14.1",
-    "webpack-load-plugins": "^0.1.2",
     "webpack-md5-hash": "^0.0.5",
-    "webpack-validator": "^1.0.0-beta.4"
+    "webpack-merge": "^0.8.3"
   },
   "repository": {
     "type": "git",

webpack.config.js

@@ -7,7 +7,7 @@ var CopyWebpackPlugin = require('copy-webpack-plugin');
 var HtmlWebpackPlugin = require('html-webpack-plugin');
 
 var ENV = process.env.ENV = process.env.NODE_ENV = 'development';
-var HMR = process.argv.join('').indexOf('hot') > -1;
+var HMR = helpers.hasProcessFlag('hot');
 
 var metadata = {
   title: 'Angular2 Webpack Starter by @gdi2990 from @AngularClass',
@@ -19,28 +19,19 @@ var metadata = {
 };
 /*
  * Config
+ * with default values at webpack.default.conf
  */
-module.exports = helpers.validate({
+module.exports = helpers.defaults({
   // static data for index.html
   metadata: metadata,
-  // for faster builds use 'eval'
-  devtool: 'source-map',
-  debug: true,
-  // cache: false,
+  // devtool: 'eval' // for faster builds use 'eval'
 
   // our angular app
   entry: { 'polyfills': './src/polyfills.ts', 'main': './src/main.ts' },
 
   // Config for our build files
   output: {
-    path: helpers.root('dist'),
-    filename: '[name].bundle.js',
-    sourceMapFilename: '[name].map',
-    chunkFilename: '[id].chunk.js'
-  },
-
-  resolve: {
-    extensions: ['', '.ts', '.async.ts', '.js']
+    path: helpers.root('dist')
   },
 
   module: {
@@ -83,20 +74,10 @@ module.exports = helpers.validate({
   ],
 
   // Other module loader config
-  tslint: {
-    emitErrors: false,
-    failOnHint: false,
-    resourcePath: 'src',
-  },
 
   // our Webpack Development Server config
   devServer: {
     port: metadata.port,
-    host: metadata.host,
-    // contentBase: 'src/',
-    historyApiFallback: true,
-    watchOptions: { aggregateTimeout: 300, poll: 1000 }
-  },
-  // we need this due to problems with es6-shim
-  node: {global: 'window', progress: false, crypto: 'empty', module: false, clearImmediate: false, setImmediate: false}
+    host: metadata.host
+  }
 });

webpack.default.conf.js

@@ -0,0 +1,41 @@
+var path = require('path');
+
+module.exports = {
+  devtool: 'source-map',
+  stats: { colors: true, reasons: true },
+  resolve: {
+    extensions: ['', '.ts', '.js']
+  },
+  debug: true,
+  output: {
+    filename: '[name].bundle.js',
+    sourceMapFilename: '[name].map',
+    chunkFilename: '[id].chunk.js'
+  },
+  module: {
+    noParse: [
+      path.join(__dirname, 'zone.js', 'dist'),
+      path.join(__dirname, 'angular2', 'bundles')
+    ]
+  },
+  node: {
+    global: 'window',
+    progress: false,
+    crypto: 'empty',
+    module: false,
+    clearImmediate: false,
+    setImmediate: false
+  },
+  tslint: {
+    emitErrors: false,
+    failOnHint: false,
+    resourcePath: 'src',
+  },
+  devServer: {
+    historyApiFallback: true,
+    watchOptions: {
+      aggregateTimeout: 300,
+      poll: 1000
+    }
+  }
+};

webpack.prod.config.js

@@ -31,7 +31,7 @@ var metadata = {
 /*
  * Config
  */
-module.exports = helpers.validate({
+module.exports = helpers.defaults({
   // static data for index.html
   metadata: metadata,
 
@@ -54,8 +54,6 @@ module.exports = helpers.validate({
 
   resolve: {
     cache: false,
-    // ensure loader extensions match
-    extensions: ['', '.ts','.js']
   },
 
   module: {
@@ -193,13 +191,4 @@ module.exports = helpers.validate({
   },
   // don't use devServer for production
 
-  // we need this due to problems with es6-shim
-  node: {
-    global: 'window',
-    progress: false,
-    crypto: 'empty',
-    module: false,
-    clearImmediate: false,
-    setImmediate: false
-  }
 });

webpack.test.config.js

@@ -9,10 +9,7 @@ var ENV = process.env.ENV = process.env.NODE_ENV = 'test';
 /*
  * Config
  */
-module.exports = helpers.validate({
-  resolve: {
-    extensions: ['', '.ts','.js']
-  },
+module.exports = helpers.defaults({
   devtool: 'inline-source-map',
   module: {
     preLoaders: [
@@ -58,14 +55,8 @@ module.exports = helpers.validate({
           /node_modules/
         ]
       }
-    ],
-    noParse: [
-      helpers.root('zone.js/dist'),
-      helpers.root('angular2/bundles')
     ]
   },
-  stats: { colors: true, reasons: true },
-  debug: false,
   plugins: [
     new DefinePlugin({
       // Environment helpers
@@ -83,14 +74,5 @@ module.exports = helpers.validate({
       '__param': 'ts-helper/param',
     })
   ],
-    // we need this due to problems with es6-shim
-  node: {
-    global: 'window',
-    progress: false,
-    crypto: 'empty',
-    module: false,
-    clearImmediate: false,
-    setImmediate: false
-  }
 });