Use SDK style projects for Dev18

jbevain · jbevain · commit 229f1d4896c9 · 2025-06-23T09:33:33.000-07:00
diff --git a/.gitignore b/.gitignore
@@ -8,12 +8,16 @@ ValidationSuiteResults
 Assets/Plugins**
 obj/
 /UserSettings/
+/ProjectSettings/*.asset
 /*.csproj
 /*.sln
+/*.csproj.user
+/*.slnx
 /Temp
 *.bak
 *.bak.meta
 .DS_Store
 
+Packages/packages-lock.json
 Packages/com.unity.ide.visualstudio/Editor/COMIntegration/Release/build/
 Packages/com.unity.ide.visualstudio/Editor/COMIntegration/Release/COMIntegration.exe
diff --git a/Packages/com.unity.ide.visualstudio/Editor/ProjectGeneration/GeneratorFactory.cs b/Packages/com.unity.ide.visualstudio/Editor/ProjectGeneration/GeneratorFactory.cs
@@ -5,7 +5,6 @@
  *--------------------------------------------------------------------------------------------*/
 
 using System;
-using System.Collections.Generic;
 
 namespace Microsoft.Unity.VisualStudio.Editor
 {
@@ -17,22 +16,19 @@ internal enum GeneratorStyle
 
 	internal static class GeneratorFactory
 	{
-		private static readonly Dictionary<GeneratorStyle, IGenerator> _generators = new Dictionary<GeneratorStyle, IGenerator>();
-
-		static GeneratorFactory()
-		{
-			_generators.Add(GeneratorStyle.SDK, new SdkStyleProjectGeneration());
-			_generators.Add(GeneratorStyle.Legacy, new LegacyStyleProjectGeneration());
-		}
+		private static readonly SdkStyleProjectGeneration _sdkStyleProjectGeneration = new SdkStyleProjectGeneration();
+		private static readonly LegacyStyleProjectGeneration _legacyStyleProjectGeneration = new LegacyStyleProjectGeneration();
 
 		public static IGenerator GetInstance(GeneratorStyle style)
 		{
 			var forceStyleString = OnSelectingCSProjectStyle();
 			if (forceStyleString != null && Enum.TryParse<GeneratorStyle>(forceStyleString, out var forceStyle))
 				style = forceStyle;
 
-			if (_generators.TryGetValue(style, out var result))
-				return result;
+			if (style == GeneratorStyle.SDK)
+				return _sdkStyleProjectGeneration;
+			if (style == GeneratorStyle.Legacy)
+				return _legacyStyleProjectGeneration;
 
 			throw new ArgumentException("Unknown generator style");
 		}
diff --git a/Packages/com.unity.ide.visualstudio/Editor/ProjectGeneration/ProjectGeneration.cs b/Packages/com.unity.ide.visualstudio/Editor/ProjectGeneration/ProjectGeneration.cs
@@ -586,6 +586,11 @@ public string ProjectFile(Assembly assembly)
 #endif
 
 		public string SolutionFile()
+		{
+			return SolutionFileImpl();
+		}
+
+		internal virtual string SolutionFileImpl()
 		{
 			return Path.Combine(ProjectDirectory.NormalizePathSeparators(), $"{InvalidCharactersRegexPattern.Replace(m_ProjectName, "_")}.sln");
 		}
@@ -821,6 +826,26 @@ internal virtual void GetProjectFooter(StringBuilder footerBuilder)
 		{
 		}
 
+		private static string GetSolutionText()
+		{
+			return string.Join(k_WindowsNewline,
+			@"",
+			@"Microsoft Visual Studio Solution File, Format Version {0}",
+			@"# Visual Studio {1}",
+			@"{2}",
+			@"Global",
+			@"    GlobalSection(SolutionConfigurationPlatforms) = preSolution",
+			@"        Debug|Any CPU = Debug|Any CPU",
+			@"        Release|Any CPU = Release|Any CPU",
+			@"    EndGlobalSection",
+			@"    GlobalSection(ProjectConfigurationPlatforms) = postSolution",
+			@"{3}",
+			@"    EndGlobalSection",
+			@"{4}",
+			@"EndGlobal",
+			@"").Replace("    ", "\t");
+		}
+
 		private void SyncSolution(IEnumerable<Assembly> assemblies)
 		{
 			if (InvalidCharactersRegexPattern.IsMatch(ProjectDirectory))
@@ -831,16 +856,29 @@ private void SyncSolution(IEnumerable<Assembly> assemblies)
 			SyncSolutionFileIfNotChanged(solutionFile, SolutionText(assemblies, previousSolution));
 		}
 
-		private string SolutionText(IEnumerable<Assembly> assemblies, Solution previousSolution = null)
+		internal virtual string SolutionText(IEnumerable<Assembly> assemblies, Solution previousSolution = null)
 		{
 			const string fileversion = "12.00";
 			const string vsversion = "15";
 
+			var projects = GetSolutionProjects(assemblies, previousSolution);
+			var properties = previousSolution != null ? previousSolution.Properties : null;
+
+			string propertiesText = GetPropertiesText(properties);
+			string projectEntriesText = GetProjectEntriesText(projects);
+
+			// do not generate configurations for SolutionFolders
+			var configurableProjects = projects.Where(p => !p.IsSolutionFolderProjectFactory());
+			string projectConfigurationsText = string.Join(k_WindowsNewline, configurableProjects.Select(p => GetProjectActiveConfigurations(p.ProjectGuid)).ToArray());
+
+			return string.Format(GetSolutionText(), fileversion, vsversion, projectEntriesText, projectConfigurationsText, propertiesText);
+		}
+
+		internal List<SolutionProjectEntry> GetSolutionProjects(IEnumerable<Assembly> assemblies, Solution previousSolution = null)
+		{
 			var relevantAssemblies = RelevantAssembliesForMode(assemblies);
 			var generatedProjects = ToProjectEntries(relevantAssemblies).ToList();
 
-			SolutionProperties[] properties = null;
-
 			// First, add all projects generated by Unity to the solution
 			var projects = new List<SolutionProjectEntry>();
 			projects.AddRange(generatedProjects);
@@ -853,34 +891,9 @@ private string SolutionText(IEnumerable<Assembly> assemblies, Solution previousS
 					.Where(p => generatedProjects.All(gp => gp.FileName != p.FileName));
 
 				projects.AddRange(externalProjects);
-				properties = previousSolution.Properties;
 			}
 
-			string propertiesText = GetPropertiesText(properties);
-			string projectEntriesText = GetProjectEntriesText(projects);
-
-			// do not generate configurations for SolutionFolders
-			var configurableProjects = projects.Where(p => !p.IsSolutionFolderProjectFactory());
-			string projectConfigurationsText = string.Join(k_WindowsNewline, configurableProjects.Select(p => GetProjectActiveConfigurations(p.ProjectGuid)).ToArray());
-
-			const string solutionText = 
-				"" + k_WindowsNewline
-				+ "Microsoft Visual Studio Solution File, Format Version {0}" + k_WindowsNewline
-				+ "# Visual Studio {1}" + k_WindowsNewline
-				+ "{2}" + k_WindowsNewline
-				+ "Global" + k_WindowsNewline
-				+ "\tGlobalSection(SolutionConfigurationPlatforms) = preSolution" + k_WindowsNewline
-				+ "\t\tDebug|Any CPU = Debug|Any CPU" + k_WindowsNewline
-				+ "\t\tRelease|Any CPU = Release|Any CPU" + k_WindowsNewline
-				+ "\tEndGlobalSection" + k_WindowsNewline
-				+ "\tGlobalSection(ProjectConfigurationPlatforms) = postSolution" + k_WindowsNewline
-				+ "{3}" + k_WindowsNewline
-				+ "\tEndGlobalSection" + k_WindowsNewline
-				+ "{4}" + k_WindowsNewline
-				+ "EndGlobal" + k_WindowsNewline
-				+ "";
-
-			return string.Format(solutionText, fileversion, vsversion, projectEntriesText, projectConfigurationsText, propertiesText);
+			return projects;
 		}
 
 		private static IEnumerable<Assembly> RelevantAssembliesForMode(IEnumerable<Assembly> assemblies)
@@ -942,6 +955,7 @@ private string GetProjectEntriesText(IEnumerable<SolutionProjectEntry> entries)
 		private IEnumerable<SolutionProjectEntry> ToProjectEntries(IEnumerable<Assembly> assemblies)
 		{
 			foreach (var assembly in assemblies)
+			{
 				yield return new SolutionProjectEntry()
 				{
 					ProjectFactoryGuid = SolutionGuid(assembly),
@@ -950,6 +964,7 @@ private IEnumerable<SolutionProjectEntry> ToProjectEntries(IEnumerable<Assembly>
 					ProjectGuid = ProjectGuid(assembly),
 					Metadata = k_WindowsNewline
 				};
+			}
 		}
 
 		/// <summary>
diff --git a/Packages/com.unity.ide.visualstudio/Editor/ProjectGeneration/SdkStyleProjectGeneration.cs b/Packages/com.unity.ide.visualstudio/Editor/ProjectGeneration/SdkStyleProjectGeneration.cs
@@ -4,7 +4,7 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
-using System;
+using System.Collections.Generic;
 using System.IO;
 using System.Text;
 using UnityEditor.Compilation;
@@ -122,5 +122,27 @@ internal static void GetCapabilityBlock(StringBuilder footerBuilder, string impo
 			}
 			footerBuilder.Append(@"  </ItemGroup>").Append(k_WindowsNewline);
 		}
+
+		internal override string SolutionFileImpl()
+		{
+			return base.SolutionFileImpl() + "x";
+		}
+
+		internal override string SolutionText(IEnumerable<Assembly> assemblies, Solution previousSolution = null)
+		{
+			var projects = GetSolutionProjects(assemblies, previousSolution);
+
+			var content = new StringBuilder();
+			content.Append("<Solution>").Append(k_WindowsNewline);
+
+			foreach (var project in projects)
+			{
+				content.Append("  ").Append("<Project Path=\"").Append(project.FileName).Append("\" />").Append(k_WindowsNewline);
+			}
+
+			content.Append("</Solution>").Append(k_WindowsNewline);
+
+			return content.ToString();
+		}
 	}
 }
diff --git a/Packages/com.unity.ide.visualstudio/Editor/SolutionParser.cs b/Packages/com.unity.ide.visualstudio/Editor/SolutionParser.cs
@@ -2,8 +2,11 @@
  *  Copyright (c) Microsoft Corporation. All rights reserved.
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
+using System;
 using System.Collections.Generic;
+using System.IO;
 using System.Text.RegularExpressions;
+using System.Xml.Linq;
 
 namespace Microsoft.Unity.VisualStudio.Editor
 {
@@ -16,10 +19,20 @@ internal static class SolutionParser
 
 		public static Solution ParseSolutionFile(string filename, IFileIO fileIO)
 		{
-			return ParseSolutionContent(fileIO.ReadAllText(filename));
+			var extension = Path.GetExtension(filename).ToLower();
+			if (extension == ".sln")
+			{
+				return ParseSolutionContent(fileIO.ReadAllText(filename));
+			}
+			if (extension == ".slnx")
+			{
+				return ParseSolutionXmlContent(fileIO.ReadAllText(filename));
+			}
+
+			throw new NotSupportedException();
 		}
 
-		public static Solution ParseSolutionContent(string content)
+		private static Solution ParseSolutionContent(string content)
 		{
 			return new Solution
 			{
@@ -76,5 +89,30 @@ private static SolutionProperties[] ParseSolutionProperties(string content)
 
 			return properties.ToArray();
 		}
+
+		private static Solution ParseSolutionXmlContent(string content)
+		{
+			return new Solution
+			{
+				Projects = ParseSolutionXmlProjects(content),
+				Properties = Array.Empty<SolutionProperties>(),
+			};
+		}
+
+		private static SolutionProjectEntry[] ParseSolutionXmlProjects(string content)
+		{
+			var document = XDocument.Parse(content);
+			var projects = new List<SolutionProjectEntry>();
+
+			foreach (var project in document.Descendants("Project"))
+			{
+				projects.Add(new SolutionProjectEntry
+				{
+					FileName = project.Attribute("Path")?.Value,
+				});
+			}
+
+			return projects.ToArray();
+		}
 	}
 }
diff --git a/Packages/com.unity.ide.visualstudio/Editor/VisualStudioForWindowsInstallation.cs b/Packages/com.unity.ide.visualstudio/Editor/VisualStudioForWindowsInstallation.cs
@@ -26,6 +26,8 @@ internal class VisualStudioForWindowsInstallation : VisualStudioInstallation
 		// C# language version support for Visual Studio
 		private static readonly VersionPair[] WindowsVersionTable =
 		{
+			new VersionPair(18, 0, /* => */ 14,0),
+
 			// VisualStudio 2022
 			new VersionPair(17,12, /* => */ 13,0),
 			new VersionPair(17, 8, /* => */ 12,0),
@@ -43,8 +45,7 @@ internal class VisualStudioForWindowsInstallation : VisualStudioInstallation
 			new VersionPair(15,0, /* => */ 7,0),
 		};
 
-		private static string _vsWherePath = null;
-		private static readonly IGenerator _generator = GeneratorFactory.GetInstance(GeneratorStyle.Legacy);
+		private static string _vsWherePath;
 
 		public override bool SupportsAnalyzers
 		{
@@ -124,7 +125,7 @@ public override IGenerator ProjectGenerator
 		{
 			get
 			{
-				return _generator;
+				return GeneratorFactory.GetInstance(Version.Major >= 18 ? GeneratorStyle.SDK : GeneratorStyle.Legacy);
 			}
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@ internal class VisualStudioForWindowsInstallation : VisualStudioInstallation`
`26`	`26`	`// C# language version support for Visual Studio`
`27`	`27`	`private static readonly VersionPair[] WindowsVersionTable =`
`28`	`28`	`{`
	`29`	`+ new VersionPair(18, 0, /* => */ 14,0),`
	`30`	`+`
`29`	`31`	`// VisualStudio 2022`
`30`	`32`	`new VersionPair(17,12, /* => */ 13,0),`
`31`	`33`	`new VersionPair(17, 8, /* => */ 12,0),`
`@@ -43,8 +45,7 @@ internal class VisualStudioForWindowsInstallation : VisualStudioInstallation`
`43`	`45`	`new VersionPair(15,0, /* => */ 7,0),`
`44`	`46`	`};`
`45`	`47`
`46`		`- private static string _vsWherePath = null;`
`47`		`- private static readonly IGenerator _generator = GeneratorFactory.GetInstance(GeneratorStyle.Legacy);`
	`48`	`+ private static string _vsWherePath;`
`48`	`49`
`49`	`50`	`public override bool SupportsAnalyzers`
`50`	`51`	`{`
`@@ -124,7 +125,7 @@ public override IGenerator ProjectGenerator`
`124`	`125`	`{`
`125`	`126`	`get`
`126`	`127`	`{`
`127`		`- return _generator;`
	`128`	`+ return GeneratorFactory.GetInstance(Version.Major >= 18 ? GeneratorStyle.SDK : GeneratorStyle.Legacy);`
`128`	`129`	`}`
`129`	`130`	`}`
`130`	`131`