microsoft / microsoft/TypeScript
MSBuild integration with ASP.NET Core
@joj is already working on this.
Since Nov 20, 2024.
- Dominant language
- Go
- Stars
- 111k
- Forks
- 14.3k
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 132
Description
The ASP.NET SDK has an asset pipeline that processes all the web content for the app to apply optimizations (compression/fingerprinting, etc.).
There are a few things that make integration between the MSBuild SDK and the ASP.NET Core pipeline challenging, and while we've given people guidance, it would be great if we can enable this scenario to work without having to make such changes to their project.
There are two main challenges that we face:
- By default the TypeScript targets run too late in the pipeline for us to see the generated outputs. Our guidance suggests hooking up the relevant targets to the
PrepareForBuildtarget as shown below:
<PrepareForBuildDependsOn>
CompileTypeScript;
CompileTypeScriptWithTSConfig;
GetTypeScriptOutputForPublishing;$(PrepareForBuildDependsOn)
</PrepareForBuildDependsOn>
- I suspect the snippet above is not fully correct, as more targets are involved in the TypeScript setup in
CompileDependsOn. - This makes the TS targets run early enough so that standard targets in the ASP.NET Core pipeline can detect and process them as expected.
- The second challenge that we face is that is typical for people to dump their outputs into the wwwroot folder. When this happens, (after the change above)
GetTypeScriptOutputForPublishingwill add all theGeneratedJavaScriptitems to theContentitem group unconditionally.- This results in the presence of duplicate
Contentitems that interferes with the build. - We've given people the following target to remove the duplicates before the
GetTypeScriptOutputForPublishingtarget adds them.
<Target Name="RemoveDuplicateTypeScriptOutputs" BeforeTargets="GetTypeScriptOutputForPublishing"> <ItemGroup> <Content Remove="@(GeneratedJavaScript)" /> </ItemGroup> </Target> - This results in the presence of duplicate
This gets things into a working state, but it's obviously not trivial for customers to discover/setup in their app. Hopefully we can work together to make this scenario just work by implementing a few simple changes.
- Have a target
ResolveTypeScriptOutputsor similar that produces the compile outputs as items in theContentitem group and that can run early enough in the pipeline (PrepareForBuild is a good candidate, or it can be configurable). - Avoid adding duplicate content items to the
Contentitem group by usingExclude="@(Content)to prevent duplicates. - Checking for
UsingMicrosoftNETSdkStaticWebAssetsto wire upResolveTypeScriptOutputsearly enough in the pipeline so that the outputs can be detected and disable other targets that are not needed (Anything that deals with copying the outputs to the output/publish folder is already handled by the static web assets SDK).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.