From 1fedaa3d8fdc40b60d32c43e6ec435a1575cf39c Mon Sep 17 00:00:00 2001
From: Warren Buckley <warren@hackmakedo.com>
Date: Fri, 14 Nov 2025 20:25:19 +0000
Subject: [PATCH] Removes npm commands from MSBuild of the CSPROJ for
 umbraco-extension dotnet new template (#20839)

* Removes npm commands from the MSBuild of the CSPROJ of the umbraco-extension dotnet new template

Was agreed by the community package team to remove this, as this DX can cause more issues than actually help users in our opinion

* Removed the unused value - good catch by Copilot
---
 .../UmbracoExtension/Umbraco.Extension.csproj   | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/templates/UmbracoExtension/Umbraco.Extension.csproj b/templates/UmbracoExtension/Umbraco.Extension.csproj
index 4bc359eaa8..1e515aebbf 100644
--- a/templates/UmbracoExtension/Umbraco.Extension.csproj
+++ b/templates/UmbracoExtension/Umbraco.Extension.csproj
@@ -26,8 +26,6 @@
   </ItemGroup>
 
   <ItemGroup>
-    <ClientAssetsInputs Include="Client\**" Exclude="$(DefaultItemExcludes)" />
-
     <!-- Dont include the client folder as part of packaging nuget build -->
     <Content Remove="Client\**" />
 
@@ -39,19 +37,4 @@
     <Folder Include="wwwroot\" />
   </ItemGroup>
 
-  <!-- Restore and build Client files -->
-  <Target Name="RestoreClient" Inputs="Client\package.json;Client\package-lock.json" Outputs="Client\node_modules\.package-lock.json">
-    <Message Importance="high" Text="Restoring Client NPM packages..." />
-    <Exec Command="npm i" WorkingDirectory="Client" />
-  </Target>
-
-  <Target Name="BuildClient" BeforeTargets="AssignTargetPaths" DependsOnTargets="RestoreClient" Inputs="@(ClientAssetsInputs)" Outputs="$(IntermediateOutputPath)client.complete.txt">
-    <Message Importance="high" Text="Executing Client NPM build script..." />
-    <Exec Command="npm run build" WorkingDirectory="Client" />
-    <ItemGroup>
-      <_ClientAssetsBuildOutput Include="wwwroot\App_Plugins\**" />
-    </ItemGroup>
-    <WriteLinesToFile File="$(IntermediateOutputPath)client.complete.txt" Lines="@(_ClientAssetsBuildOutput)" Overwrite="true" />
-  </Target>
-
 </Project>