From 419c3fb5a44625a943c1d11b9a6f5f458582b2a2 Mon Sep 17 00:00:00 2001
From: Connor Sullivan <dev.connorjs@gmail.com>
Date: Sun, 11 Aug 2024 17:18:53 -0400
Subject: [PATCH] =?UTF-8?q?=F0=9F=9A=9A=20Move=20to=20src=20+=20move=20to?=
 =?UTF-8?q?=20Solution=20target?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Moves to `src` directory structure for JS projects. Moves pnpm install to
Solution target, which will force a single run as desired.
---
 Directory.Solution.targets              | 25 +++++++++++++
 client/Directory.Build.props            |  2 +-
 client/Directory.Build.targets          | 47 +++++++------------------
 client/hello-js/{ => src}/hello.test.ts |  0
 client/hello-js/{ => src}/hello.ts      |  0
 dotnet-with-esproj-example.sln          |  5 +--
 6 files changed, 42 insertions(+), 37 deletions(-)
 create mode 100644 Directory.Solution.targets
 rename client/hello-js/{ => src}/hello.test.ts (100%)
 rename client/hello-js/{ => src}/hello.ts (100%)

diff --git a/Directory.Solution.targets b/Directory.Solution.targets
new file mode 100644
index 0000000..8193a50
--- /dev/null
+++ b/Directory.Solution.targets
@@ -0,0 +1,25 @@
+<Project>
+	<PropertyGroup>
+		<PnpmInstallCheck>$(SolutionDir)node_modules/.dotnet-pnpm-install</PnpmInstallCheck>
+	</PropertyGroup>
+
+	<!-- Install pnpm dependencies at solution/workspace level. -->
+	<Target Name="pnpm install" BeforeTargets="Restore"
+			Inputs="$(SolutionDir)package.json;$(SolutionDir)pnpm-lock.yaml;$(SolutionDir)pnpm-workspace.yaml"
+			Outputs="$(PnpmInstallCheck)"
+	>
+		<Exec Command="pnpm install --workspace-root" StandardOutputImportance="normal" />
+
+		<!-- Print restore message in similar way to csproj (no time though). -->
+		<Message Text="Restored $(SolutionDir)package.json (pnpm install)." Importance="high" />
+
+		<!-- File used for .NET iterative builds (touch updates the timestamp). -->
+		<Touch Files="$(PnpmInstallCheck)" AlwaysCreate="true" />
+	</Target>
+
+	<!-- Clean -->
+	<Target Name="Clean solution" AfterTargets="Clean">
+		<RemoveDir Directories="$(SolutionDir)artifacts" />
+		<RemoveDir Directories="$(PnpmInstallCheck)" />
+	</Target>
+</Project>
diff --git a/client/Directory.Build.props b/client/Directory.Build.props
index 3db7d68..2946d50 100644
--- a/client/Directory.Build.props
+++ b/client/Directory.Build.props
@@ -12,7 +12,7 @@
 
 	<!-- Customize npm build -->
 	<PropertyGroup>
-		<!-- Skip npm install because we are using pnpm; see Directory.Build.targets -->
+		<!-- Skip npm install because we are using workspace; see Directory.Solution.targets -->
 		<ShouldRunNpmInstall>false</ShouldRunNpmInstall>
 	</PropertyGroup>
 
diff --git a/client/Directory.Build.targets b/client/Directory.Build.targets
index 73d5eef..750e33c 100644
--- a/client/Directory.Build.targets
+++ b/client/Directory.Build.targets
@@ -10,30 +10,32 @@
 		<PrettierArgs>--list-different --write</PrettierArgs>
 	</PropertyGroup>
 
-	<!-- Build: eslint -->
-	<Target Name="eslint" AfterTargets="Build">
+	<!-- Build order: tsc, eslint, prettier, and then any project-specific -->
+
+	<!-- Build: tsc -->
+	<Target Name="tsc" AfterTargets="Build">
 		<Exec
-			Command="pnpm exec eslint --format pretty $(EslintArgs) ."
+			Command="pnpm exec tsc --build"
 			StandardOutputImportance="normal"
 		/>
+		<!-- Print from tsc, similar to C# compile, for CLI consistency -->
+		<Message Text="$(ProjectName) -> $(WorkspaceRoot)artifacts/tsc/$(ProjectName)" Importance="high" />
 	</Target>
 
-	<!-- Build: prettier -->
-	<Target Name="prettier" AfterTargets="Build">
+	<!-- Build: eslint -->
+	<Target Name="eslint" AfterTargets="Build">
 		<Exec
-			Command="pnpm exec prettier --ignore-unknown $(PrettierArgs) ."
+			Command="pnpm exec eslint --format pretty $(EslintArgs) ."
 			StandardOutputImportance="normal"
 		/>
 	</Target>
 
-	<!-- Build: tsc -->
-	<Target Name="tsc" AfterTargets="Build">
+	<!-- Build: prettier -->
+	<Target Name="prettier" AfterTargets="Build">
 		<Exec
-			Command="pnpm exec tsc --build"
+			Command="pnpm exec prettier --ignore-unknown --cache-location $(WorkspaceRoot)artifacts/prettier $(PrettierArgs) ."
 			StandardOutputImportance="normal"
 		/>
-		<!-- Print from tsc, similar to C# compile, for CLI consistency -->
-		<Message Text="$(ProjectName) -> $(WorkspaceRoot)artifacts/tsc/$(ProjectName)" Importance="high" />
 	</Target>
 
 	<!-- Test: vitest -->
@@ -49,27 +51,4 @@
 		/>
 		<Message Text="Tests succeeded: '$(ProjectRoot)' [vitest]" Importance="high" />
 	</Target>
-
-	<!-- Restore: install -->
-	<!-- Hook into the JSPS restore before the standard NPMRestore -->
-	<Target Name="pnpm install" AfterTargets="NPMRestore"
-			Inputs="$(WorkspaceRoot)package.json;$(ProjectRoot)package.json"
-			Outputs="$(WorkspaceRoot)pnpm-lock.yaml"
-	>
-		<!-- Run pnpm install, timing it to match C# output -->
-		<PropertyGroup>
-			<StartTicks>$([System.DateTime]::UtcNow.Ticks)</StartTicks>
-		</PropertyGroup>
-
-		<Exec Command="pnpm -w install" StandardOutputImportance="normal" />
-
-		<PropertyGroup>
-			<FinishTicks>$([System.DateTime]::UtcNow.Ticks)</FinishTicks>
-			<ElapsedTicks>$([MSBuild]::Subtract($(FinishTicks), $(StartTicks)))</ElapsedTicks>
-			<Elapsed>$([System.TimeSpan]::FromTicks($(ElapsedTicks)).Milliseconds)</Elapsed>
-		</PropertyGroup>
-
-		<!-- Print restore message the same way as csproj -->
-		<Message Text="Restored $(MSBuildProjectFullPath) (in $(Elapsed) ms)." Importance="high" />
-	</Target>
 </Project>
diff --git a/client/hello-js/hello.test.ts b/client/hello-js/src/hello.test.ts
similarity index 100%
rename from client/hello-js/hello.test.ts
rename to client/hello-js/src/hello.test.ts
diff --git a/client/hello-js/hello.ts b/client/hello-js/src/hello.ts
similarity index 100%
rename from client/hello-js/hello.ts
rename to client/hello-js/src/hello.ts
diff --git a/dotnet-with-esproj-example.sln b/dotnet-with-esproj-example.sln
index 7d9d584..3164f19 100644
--- a/dotnet-with-esproj-example.sln
+++ b/dotnet-with-esproj-example.sln
@@ -33,10 +33,11 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "shared", "shared", "{161728
 		.editorconfig = .editorconfig
 		.gitattributes = .gitattributes
 		.gitignore = .gitignore
-		Workspace.props = Workspace.props
-		LICENSE = LICENSE
 		ci-build.ps1 = ci-build.ps1
+		Directory.Solution.targets = Directory.Solution.targets
+		LICENSE = LICENSE
 		.github\workflows\pipeline.yaml = .github\workflows\pipeline.yaml
+		Workspace.props = Workspace.props
 	EndProjectSection
 EndProject
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "projects", "projects", "{2E75741F-092F-4C2C-A182-AAF9F38CEC58}"