Fix MSBuild dependency graph resolution

.github/workflows/build.yml

@@ -0,0 +1,78 @@
+name: Build
+on:
+  push:
+    branches: [main]
+  pull_request:
+    types: [opened, synchronize, closed, reopened]
+    paths:
+      - '**/*.cs'
+      - '**/*.sln'
+      - '**/*.proj'
+      - '**/*.csproj'
+      - '**/*.vcxproj'
+      - '**/*.props'
+      - '**/*.targets'
+      - '**/packages.lock.json'
+  workflow_dispatch:
+env:
+  MSBUILD_CACHES: |
+    **/bin
+    **/obj
+    **/Win32
+    **/x64
+  MSBUILD_DEFAULTS: /m:2 /v:m /clp:Summary /p:Configuration=Release /p:Platform="Any CPU"
+  NUGET_CACHES: |
+    ~/.nuget/packages
+  NUGET_DEFAULTS: -Verbosity quiet -NonInteractive -ConfigFile NuGet.Config -UseLockFile
+jobs:
+  # Runs test suite
+  build:
+    runs-on: windows-latest
+    name: "MSBuild Runner"
+    steps:
+      # Fetches remote repository without --progress option.
+      #
+      # The default behavior of @actions/checkout outputs many noisy lines of
+      # status output in the workflow log, which is problematic for log size.
+      - name: Checkout latest repository commit
+        uses: actions/checkout@v4
+        with:
+          show-progress: false
+
+      # Setup the .NET environment
+      - name: Install .NET Core
+        uses: actions/setup-dotnet@v3
+      # Setup MSBuild environment
+      - name: Setup MSBuild.exe
+        uses: microsoft/setup-msbuild@v1.3.1
+      - name: Restore Build Cache
+        id: msbuild-cache
+        uses: actions/cache@v3
+        with:
+          path: ${{ env.MSBUILD_CACHES }}
+          key: ${{ runner.os }}-msbuild-${{ github.ref_name }}-${{ github.sha }}
+          restore-keys: |
+            ${{ runner.os }}-msbuild-${{ github.ref_name }}
+            ${{ runner.os }}-msbuild-
+
+      # Setup NuGet environment
+      - name: Setup NuGet
+        uses: NuGet/setup-nuget@v1.2.0
+      - name: Restore NuGet Cache
+        id: nuget-cache
+        uses: actions/cache@v3
+        with:
+          path: ${{ env.NUGET_CACHES }}
+          key: ${{ runner.os }}-nuget-${{ hashFiles('**/packages.lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-nuget-
+      - name: Restore NuGet packages
+        if: steps.nuget-cache.outputs.cache-hit != 'true'
+        run: nuget restore SDK.sln ${{ env.NUGET_DEFAULTS }}
+
+      # Build MTGO reference assemblies
+      - name: Build Reference Assemblies
+        run: msbuild Ref.sln /t:Build ${{ env.MSBUILD_DEFAULTS }}
+      # Build the SDK
+      - name: Build SDK Solution
+        run: msbuild SDK.sln /t:Build ${{ env.MSBUILD_DEFAULTS }}

MTGOSDK.MSBuild/build/MTGOSDK.MSBuild.targets

@@ -5,6 +5,9 @@
     at the end of the build. If it's omitted, that can cause build failures
     in subsequent builds because the task assembly can't be overwritten.
   -->
+  <UsingTask AssemblyFile="$(_MTGOSDK_MSBuild)"
+             TaskName="ExtractMTGOInstallation"
+             TaskFactory="TaskHostFactory" />
   <UsingTask AssemblyFile="$(_MTGOSDK_MSBuild)"
              TaskName="GenerateReferenceAssemblies"
              TaskFactory="TaskHostFactory" />
@@ -22,6 +25,23 @@
   <Target Name="GenerateReferenceAssemblies"
           BeforeTargets="CoreCompile"
           DependsOnTargets="ResolveProjectReferences;ResolveAssemblyReferences">
+    <!--
+      Extract the MTGO installation directory paths, or create a temporary
+      extraction directory if no existing MTGO installation could be found.
+    -->
+    <ExtractMTGOInstallation
+        MTGOAppDir="$(MTGOAppDir)"
+        MTGODataDir="$(MTGODataDir)">
+      <!-- Expand the glob to the (local) MTGO install/user directories -->
+      <Output TaskParameter="MTGOAppDir" PropertyName="MTGOAppDir" />
+      <Output TaskParameter="MTGODataDir" PropertyName="MTGODataDir" />
+      <!-- Output the MTGO version for use in the consuming project -->
+      <Output TaskParameter="Version" PropertyName="MTGOVersion" />
+    </ExtractMTGOInstallation>
+
+    <Message Text="MTGOSDK.MSBuild: Extracted MTGO version $(MTGOVersion)"
+             Importance="high" />
+
     <!--
       We build reference assemblies from the latest MTGO installation directory
       to ensure that we're only targeting the latest APIs of the MTGO client.
@@ -31,20 +51,15 @@
     -->
     <GenerateReferenceAssemblies
         MTGOAppDir="$(MTGOAppDir)"
-        MTGODataDir="$(MTGODataDir)"
+        Version="$(MTGOVersion)"
         OutputPath="$(_MTGOSDK_Refs)">
-      <!-- Expand the glob to the (local) MTGO install/user directories -->
-      <Output TaskParameter="MTGOAppDir" PropertyName="MTGOAppDir" />
-      <Output TaskParameter="MTGODataDir" PropertyName="MTGODataDir" />
       <!--
         Updates the internal reference path to reflect the new subpath.
 
         Note that this is only resolved after project dependencies are resolved,
         and cannot be used as part of any `Reference` item groups.
       -->
       <Output TaskParameter="OutputPath" PropertyName="_MTGOSDK_Refs" />
-      <!-- Output the MTGO version for use in the consuming project -->
-      <Output TaskParameter="Version" PropertyName="MTGOVersion" />
     </GenerateReferenceAssemblies>
 
     <!-- TODO: Verify the output reference assemblies w/ ILVerify -->