Skip experimental in recommending workloads (#45767)

We've had issues around (heuristically) finding the appropriate workloads to recommend that the user install if they're missing packs. This should help in that it will avoid recommending experimental workloads if there are any other options.

I don't feel too attached to this PR.

Author: Forgind

src/Resolvers/Microsoft.NET.Sdk.WorkloadManifestReader/WorkloadSuggestionFinder.cs

@@ -187,6 +187,7 @@ private static T FindBest<T>(IEnumerable<T> values, params Comparison<T>[] compa
 
         internal static WorkloadSuggestion GetBestSuggestion(ICollection<WorkloadSuggestion> suggestions) => FindBest(
                 suggestions,
+                (x, y) => ContainsExperimental(y.Workloads) - ContainsExperimental(x.Workloads),
                 (x, y) => y.ExtraPacks - x.ExtraPacks,
                 (x, y) => y.Workloads.Count - x.Workloads.Count);
 
@@ -195,6 +196,8 @@ internal static WorkloadSuggestion GetBestSuggestion(ICollection<WorkloadSuggest
         /// </summary>
         public WorkloadSuggestion GetBestSuggestion() => GetBestSuggestion(UnsortedSuggestions);
 
+        private static int ContainsExperimental(HashSet<WorkloadId> set) => set.Any(w => w.ToString().Contains("experimental")) ? 1 : 0;
+
         /// <summary>
         /// A partial or complete suggestion for workloads to install, annotated with which requested packs it does not satisfy
         /// </summary>

test/Microsoft.NET.Build.Tests/WorkloadTests.cs

@@ -315,7 +315,7 @@ public void Given_multi_target_It_should_get_suggested_workload_by_GetRequiredWo
                 // Conditionally check the OS and modify the expected workloads on Linux
                 if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux))
                 {
-                    expected = "android;wasi-experimental";
+                    expected = "android;wasm-tools";
                 }
 
                 getValuesCommand.GetValues()