chore(npm): updated webpack version and config cleanup

package.json

@@ -109,7 +109,7 @@
     "typedoc": "^0.4.2",
     "typescript": "2.0.2",
     "url-loader": "^0.5.7",
-    "webpack": "2.1.0-beta.22",
+    "webpack": "^2.1.0-beta.23",
     "webpack-dev-server": "2.1.0-beta.3",
     "webpack-md5-hash": "0.0.5",
     "webpack-merge": "^0.14.0",
@@ -157,6 +157,7 @@
     "tree-kill": "^1.0.0",
     "ts-node": "^1.3.0",
     "tslint": "^3.11.0",
-    "walk-sync": "^0.2.6"
+    "walk-sync": "^0.2.6",
+    "webpack": "^2.1.0-beta.23"
   }
 }

packages/angular-cli/models/webpack-build-common.ts

@@ -36,8 +36,7 @@ export function getWebpackCommonConfig(
   return {
     devtool: 'source-map',
     resolve: {
-      extensions: ['', '.ts', '.js'],
-      root: appRoot
+      extensions: ['.ts', '.js']
     },
     context: path.resolve(__dirname, './'),
     entry: entry,
@@ -46,16 +45,15 @@ export function getWebpackCommonConfig(
       filename: '[name].bundle.js'
     },
     module: {
-      preLoaders: [
+      loaders: [
         {
+          enforce: "left",
           test: /\.js$/,
           loader: 'source-map-loader',
           exclude: [
             /node_modules/
           ]
-        }
-      ],
-      loaders: [
+        },
         {
           test: /\.ts$/,
           loaders: [
@@ -159,7 +157,7 @@ export function getWebpackCommonConfig(
     ],
     node: {
       fs: 'empty',
-      global: 'window',
+      global: true,
       crypto: 'empty',
       module: false,
       clearImmediate: false,

packages/angular-cli/models/webpack-build-development.ts

@@ -1,4 +1,5 @@
 const path = require('path');
+import * as webpack from 'webpack';
 
 export const getWebpackDevConfigPartial = function(projectRoot: string, appConfig: any) {
   return {
@@ -9,14 +10,20 @@ export const getWebpackDevConfigPartial = function(projectRoot: string, appConfi
       sourceMapFilename: '[name].map',
       chunkFilename: '[id].chunk.js'
     },
-    tslint: {
-      emitErrors: false,
-      failOnHint: false,
-      resourcePath: path.resolve(projectRoot, appConfig.root)
-    },
+    plugins: [
+      new webpack.LoaderOptionsPlugin({
+        options: {
+          tslint: {
+            emitErrors: false,
+            failOnHint: false,
+            resourcePath: path.resolve(projectRoot, appConfig.root)
+          },
+        }
+      })
+    ],
     node: {
       fs: 'empty',
-      global: 'window',
+      global: true,
       crypto: 'empty',
       process: true,
       module: false,

packages/angular-cli/models/webpack-build-production.ts

@@ -5,7 +5,6 @@ import * as webpack from 'webpack';
 
 export const getWebpackProdConfigPartial = function(projectRoot: string, appConfig: any) {
   return {
-    debug: false,
     devtool: 'source-map',
     output: {
       path: path.resolve(projectRoot, appConfig.outDir),
@@ -25,27 +24,31 @@ export const getWebpackProdConfigPartial = function(projectRoot: string, appConf
           test: /\.js$|\.html$/,
           threshold: 10240,
           minRatio: 0.8
+      }),
+      new webpack.LoaderOptionsPlugin({
+        options: {
+          tslint: {
+            emitErrors: true,
+            failOnHint: true,
+            resourcePath: path.resolve(projectRoot, appConfig.root)
+          },
+          htmlLoader: {
+            minimize: true,
+            removeAttributeQuotes: false,
+            caseSensitive: true,
+            customAttrSurround: [
+              [/#/, /(?:)/],
+              [/\*/, /(?:)/],
+              [/\[?\(?/, /(?:)/]
+            ],
+            customAttrAssign: [/\)?\]?=/]
+          }
+        }
       })
     ],
-    tslint: {
-      emitErrors: true,
-      failOnHint: true,
-      resourcePath: path.resolve(projectRoot, appConfig.root)
-    },
-    htmlLoader: {
-      minimize: true,
-      removeAttributeQuotes: false,
-      caseSensitive: true,
-      customAttrSurround: [
-        [/#/, /(?:)/],
-        [/\*/, /(?:)/],
-        [/\[?\(?/, /(?:)/]
-      ],
-      customAttrAssign: [/\)?\]?=/]
-    },
     node: {
       fs: 'empty',
-      global: 'window',
+      global: true,
       crypto: 'empty',
       process: true,
       module: false,

packages/angular-cli/models/webpack-build-test.js

@@ -11,8 +11,7 @@ const getWebpackTestConfig = function (projectRoot, environment, appConfig) {
     devtool: 'inline-source-map',
     context: path.resolve(__dirname, './'),
     resolve: {
-      extensions: ['', '.ts', '.js'],
-      root: appRoot
+      extensions: ['.ts', '.js']
     },
     entry: {
       test: path.resolve(appRoot, appConfig.test)
@@ -22,24 +21,24 @@ const getWebpackTestConfig = function (projectRoot, environment, appConfig) {
       filename: '[name].bundle.js'
     },
     module: {
-      preLoaders: [
+      loaders: [
         {
           test: /\.ts$/,
+          enforce: 'left',
           loader: 'tslint-loader',
           exclude: [
             path.resolve(projectRoot, 'node_modules')
           ]
         },
         {
           test: /\.js$/,
+          enforce: 'left',
           loader: 'source-map-loader',
           exclude: [
             path.resolve(projectRoot, 'node_modules/rxjs'),
             path.resolve(projectRoot, 'node_modules/@angular')
           ]
-        }
-      ],
-      loaders: [
+        },
         {
           test: /\.ts$/,
           loaders: [
@@ -58,23 +57,22 @@ const getWebpackTestConfig = function (projectRoot, environment, appConfig) {
           ],
           exclude: [/\.e2e\.ts$/]
         },
-        { test: /\.json$/, loader: 'json-loader' },
-        { test: /\.css$/,  loaders: ['raw-loader', 'postcss-loader'] },
-        { test: /\.styl$/, loaders: ['raw-loader', 'postcss-loader', 'stylus-loader'] },
-        { test: /\.less$/, loaders: ['raw-loader', 'postcss-loader', 'less-loader'] },
-        { test: /\.scss$|\.sass$/, loaders: ['raw-loader', 'postcss-loader', 'sass-loader'] },
-        { test: /\.(jpg|png)$/, loader: 'url-loader?limit=128000' },
-        { test: /\.html$/, loader: 'raw-loader', exclude: [path.resolve(appRoot, appConfig.index)] }
-      ],
-      postLoaders: [
         {
           test: /\.(js|ts)$/, loader: 'sourcemap-istanbul-instrumenter-loader',
+          enforce: 'right',
           exclude: [
             /\.(e2e|spec)\.ts$/,
             /node_modules/
           ],
           query: { 'force-sourcemap': true }
-        }
+        },
+        { test: /\.json$/, loader: 'json-loader' },
+        { test: /\.css$/,  loaders: ['raw-loader', 'postcss-loader'] },
+        { test: /\.styl$/, loaders: ['raw-loader', 'postcss-loader', 'stylus-loader'] },
+        { test: /\.less$/, loaders: ['raw-loader', 'postcss-loader', 'less-loader'] },
+        { test: /\.scss$|\.sass$/, loaders: ['raw-loader', 'postcss-loader', 'sass-loader'] },
+        { test: /\.(jpg|png)$/, loader: 'url-loader?limit=128000' },
+        { test: /\.html$/, loader: 'raw-loader', exclude: [path.resolve(appRoot, appConfig.index)] }
       ]
     },
     plugins: [
@@ -89,16 +87,20 @@ const getWebpackTestConfig = function (projectRoot, environment, appConfig) {
         new RegExp(path.resolve(appRoot, appConfig.environments['source'])
           .replace(/[\-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/g, '\\$&')),
         path.resolve(appRoot, appConfig.environments[environment])
-      )
+      ),
+      new webpack.LoaderOptionsPlugin({
+        options: {
+          tslint: {
+            emitErrors: false,
+            failOnHint: false,
+            resourcePath: `./${appConfig.root}`
+          }
+        }
+      })
     ],
-    tslint: {
-      emitErrors: false,
-      failOnHint: false,
-      resourcePath: `./${appConfig.root}`
-    },
     node: {
       fs: 'empty',
-      global: 'window',
+      global: true,
       process: false,
       crypto: 'empty',
       module: false,