From 7bc67e1481c47cb7ffff8f30b1fac7a74ab75ada Mon Sep 17 00:00:00 2001
From: Steve Sanderson <SteveSandersonMS@users.noreply.github.com>
Date: Fri, 6 Jul 2018 16:23:44 +0100
Subject: [PATCH] Simple approach for minifying blazor.*.js on release builds.
 Fixes #1003

Later we might want to emit both min and non-min variants of the file, when there is some way for developers to control which one they use
---
 .../Microsoft.AspNetCore.Blazor.Browser.JS.csproj           | 3 ++-
 src/Microsoft.AspNetCore.Blazor.Browser.JS/package.json     | 3 ++-
 .../webpack.config.js                                       | 6 +++---
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/Microsoft.AspNetCore.Blazor.Browser.JS/Microsoft.AspNetCore.Blazor.Browser.JS.csproj b/src/Microsoft.AspNetCore.Blazor.Browser.JS/Microsoft.AspNetCore.Blazor.Browser.JS.csproj
index 9b25255af7..7c93ad7769 100644
--- a/src/Microsoft.AspNetCore.Blazor.Browser.JS/Microsoft.AspNetCore.Blazor.Browser.JS.csproj
+++ b/src/Microsoft.AspNetCore.Blazor.Browser.JS/Microsoft.AspNetCore.Blazor.Browser.JS.csproj
@@ -26,7 +26,8 @@
 
   <Target Name="RunWebpack" AfterTargets="ResolveReferences" Inputs="@(WebpackInputs)" Outputs="dist\blazor.webassembly.js" DependsOnTargets="EnsureNpmRestored">
     <RemoveDir Directories="dist" />
-    <Exec Command="npm run build" />
+    <Exec Command="npm run build:debug" Condition="'$(Configuration)' == 'Debug'" />
+    <Exec Command="npm run build:production" Condition="'$(Configuration)' != 'Debug'" />
     <ItemGroup>
       <EmbeddedResource Include="dist\blazor.webassembly.js" LogicalName="blazor./blazor.webassembly.js" />
     </ItemGroup>
diff --git a/src/Microsoft.AspNetCore.Blazor.Browser.JS/package.json b/src/Microsoft.AspNetCore.Blazor.Browser.JS/package.json
index fe404a842e..d6a5245a76 100644
--- a/src/Microsoft.AspNetCore.Blazor.Browser.JS/package.json
+++ b/src/Microsoft.AspNetCore.Blazor.Browser.JS/package.json
@@ -4,7 +4,8 @@
   "description": "",
   "main": "index.js",
   "scripts": {
-    "build": "webpack --mode development",
+    "build:debug": "webpack --mode development",
+    "build:production": "webpack --mode production",
     "test": "echo \"Error: no test specified\" && exit 1"
   },
   "devDependencies": {
diff --git a/src/Microsoft.AspNetCore.Blazor.Browser.JS/webpack.config.js b/src/Microsoft.AspNetCore.Blazor.Browser.JS/webpack.config.js
index 349dd45a41..a15019a8bf 100644
--- a/src/Microsoft.AspNetCore.Blazor.Browser.JS/webpack.config.js
+++ b/src/Microsoft.AspNetCore.Blazor.Browser.JS/webpack.config.js
@@ -1,9 +1,9 @@
 const path = require('path');
 const webpack = require('webpack');
 
-module.exports = {
+module.exports = (env, args) => ({
     resolve: { extensions: ['.ts', '.js'] },
-    devtool: 'inline-source-map',
+    devtool: args.mode === 'development' ? 'inline-source-map' : 'none',
     module: {
         rules: [{ test: /\.ts?$/, loader: 'ts-loader' }]
     },
@@ -11,4 +11,4 @@ module.exports = {
         'blazor.webassembly': './src/Boot.WebAssembly.ts',
     },
     output: { path: path.join(__dirname, '/dist'), filename: '[name].js' }
-};
+});