Use pre-built js files instead of building on the fly

Author: pranavkm

eng/scripts/CodeCheck.ps1

@@ -130,6 +130,11 @@ try {
         & $dotnet run -p "$repoRoot/eng/tools/BaselineGenerator/"
     }
 
+    Write-Host "Re-generating blazor files"
+    Invoke-Block {
+        & $dotnet build -p "$repoRoot/src/Components/Microsoft.AspNetCore.Components.Browser.JS.npmproj"
+    }
+
     Write-Host "Run git diff to check for pending changes"
 
     # Redirect stderr to stdout because PowerShell does not consistently handle output to stderr

src/Components/Browser.JS/.gitignore

@@ -1,2 +1 @@
 node_modules/
-dist/

src/Components/Browser.JS/Microsoft.AspNetCore.Components.Browser.JS.npmproj

@@ -7,28 +7,20 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <WebpackInputs Include="**\*.ts" Exclude="node_modules\**" />
-    <WebPackOutputs Include="src\dist\blazor.webassembly.js" />
-    <WebPackOutputs Include="src\dist\blazor.server.js" />
+    <WebpackInputs Include="src\**\*.ts" />
+    <WebPackOutputs Include="dist\**\blazor.webassembly.js" />
+    <WebPackOutputs Include="dist\**\blazor.server.js" />
   </ItemGroup>
 
   <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(MSBuildThisFileDirectory), Directory.Build.targets))\Directory.Build.targets" />
 
-  <!--
-    Invoking build[.cmd|.sh] does not restore npm projects by default. This is a performance optimization.
-    However, we want to ensure yarn's restored before a P2P build happens. We'll conservatively yarn install
-    if the node_modules directory is missing.
-  -->
-  <Target Name="FastRestore" DependsOnTargets="Restore" Condition="!Exists('node_modules')" />
-
   <!-- Override the default 'Build' target from eng/targets/Npm.Common.targets. -->
   <Target Name="Build"
-          DependsOnTargets="FastRestore"
           Inputs="@(WebpackInputs)"
           Outputs="@(WebPackOutputs)">
     <RemoveDir Directories="dist" />
-    <Yarn Command="run build:debug" Condition="'$(Configuration)' == 'Debug'" />
-    <Yarn Command="run build:production" Condition="'$(Configuration)' != 'Debug'" />
+    <Yarn Command="run build:debug" />
+    <Yarn Command="run build:production" />
   </Target>
 
 </Project>

src/Components/Browser.JS/dist/Debug/blazor.server.js

@@ -11,5 +11,5 @@ module.exports = (env, args) => ({
         'blazor.webassembly': './Boot.WebAssembly.ts',
         'blazor.server': './Boot.Server.ts',
     },
-    output: { path: path.join(__dirname, '/dist'), filename: '[name].js' }
+    output: { path: path.join(__dirname, '/..', '/dist', args.mode == 'development' ? '/Debug' : '/Release'), filename: '[name].js' }
 });

src/Components/Browser.JS/dist/Debug/blazor.webassembly.js

@@ -48,11 +48,7 @@
 
   <ItemGroup>
     <!-- We need .Browser.JS to build first so we can embed its .js output -->
-    <EmbeddedResource Include="..\..\Browser.JS\src\dist\blazor.server.js" LogicalName="_framework\%(Filename)%(Extension)" />
+    <EmbeddedResource Include="..\..\Browser.JS\dist\$(Configuration)\blazor.server.js" LogicalName="_framework\%(Filename)%(Extension)" />
   </ItemGroup>
 
-  <Target Name="_EnsureBrowserJS" BeforeTargets="CoreCompile">
-    <MSBuild Projects="..\..\Browser.JS\Microsoft.AspNetCore.Components.Browser.JS.npmproj" Targets="Build" />
-  </Target>
-
 </Project>