dependabot-nuget 0.244.0 → 0.246.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be3a65448fe495f267cc054563bd29fd5a2142bb07613e2ed70a1cd21490bae5
-  data.tar.gz: 1fa876f1715e1ab11ccd5e0c323e58cd60b68d08560efe63a4c563755d36b1ca
+  metadata.gz: 3e0fef609dc45e4b8d8f1bc8bf9c4d62ee6fd83a4fea8e28ae9022a469151875
+  data.tar.gz: caab818c3d702d5f34c1986e7ca9dca6b7457eade3a0c5f4d4df004682193d98
 SHA512:
-  metadata.gz: b22d903bfd1bab554a9513aac135e7f4ea8e055ce8e6a5f163c9d8331e87c258805d333043953c5b7beeaeba6404792694150e7e9c035de29cd1568a09b2fc22
-  data.tar.gz: 00abcf17c29b5f98242b8ca97ab28c79f88e7c2425430f08d549364b16b160d640841ccd2e2b4c01978e96d7b0b4d51826795247f8087d1af6a4962abfba7809
+  metadata.gz: ce37d23d0440b68f8fce93364bcda68d61fb093b2435e853f956549c6c44e1bb68940ad06365e54c70ec18d667faa251455dc28f201b1b93b9410bfc1861681c
+  data.tar.gz: '0396865c4813a9c0b357826545eb6dcbee4c350d62fceed1d97469a9de4c0f2ec751974d07862d7667b0824a24bff12453f8f5309789e6a10a1ca7c25cb0a4f6'

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/PackagesConfigUpdater.cs

@@ -48,7 +48,7 @@ internal static class PackagesConfigUpdater
         var packagesDirectory = PathHelper.JoinPath(projectDirectory, packagesSubDirectory);
         Directory.CreateDirectory(packagesDirectory);
 
-        var args = new List<string>
+        var updateArgs = new List<string>
         {
             "update",
             packagesConfigPath,
@@ -61,17 +61,29 @@ internal static class PackagesConfigUpdater
             "-NonInteractive",
         };
 
+        var restoreArgs = new List<string>
+        {
+            "restore",
+            projectPath,
+            "-PackagesDirectory",
+            packagesDirectory,
+            "-NonInteractive",
+        };
+
         logger.Log("    Finding MSBuild...");
         var msbuildDirectory = MSBuildHelper.MSBuildPath;
         if (msbuildDirectory is not null)
         {
-            args.Add("-MSBuildPath");
-            args.Add(msbuildDirectory); // e.g., /usr/share/dotnet/sdk/7.0.203
+            foreach (var args in new[] { updateArgs, restoreArgs })
+            {
+                args.Add("-MSBuildPath");
+                args.Add(msbuildDirectory); // e.g., /usr/share/dotnet/sdk/7.0.203
+            }
         }
 
         using (new WebApplicationTargetsConditionPatcher(projectPath))
         {
-            RunNuget(args, packagesDirectory, logger);
+            RunNuget(updateArgs, restoreArgs, packagesDirectory, logger);
         }
 
         projectBuildFile = ProjectBuildFile.Open(repoRootPath, projectPath);
@@ -84,7 +96,7 @@ internal static class PackagesConfigUpdater
         await projectBuildFile.SaveAsync();
     }
 
-    private static void RunNuget(List<string> args, string packagesDirectory, Logger logger)
+    private static void RunNuget(List<string> updateArgs, List<string> restoreArgs, string packagesDirectory, Logger logger)
     {
         var outputBuilder = new StringBuilder();
         var writer = new StringWriter(outputBuilder);
@@ -97,15 +109,38 @@ internal static class PackagesConfigUpdater
         var currentDir = Environment.CurrentDirectory;
         try
         {
-            logger.Log($"    Running NuGet.exe with args: {string.Join(" ", args)}");
 
             Environment.CurrentDirectory = packagesDirectory;
-            var result = Program.Main(args.ToArray());
+            var retryingAfterRestore = false;
+
+        doRestore:
+            logger.Log($"    Running NuGet.exe with args: {string.Join(" ", updateArgs)}");
+            outputBuilder.Clear();
+            var result = Program.Main(updateArgs.ToArray());
             var fullOutput = outputBuilder.ToString();
             logger.Log($"    Result: {result}");
             logger.Log($"    Output:\n{fullOutput}");
             if (result != 0)
             {
+                // If the `packages.config` file contains a delisted package, the initial `update` operation will fail
+                // with the message listed below.  The solution is to run `nuget.exe restore ...` and retry.
+                if (!retryingAfterRestore &&
+                    fullOutput.Contains("Existing packages must be restored before performing an install or update."))
+                {
+                    logger.Log($"    Running NuGet.exe with args: {string.Join(" ", restoreArgs)}");
+                    retryingAfterRestore = true;
+                    outputBuilder.Clear();
+                    var exitCodeAgain = Program.Main(restoreArgs.ToArray());
+                    var restoreOutput = outputBuilder.ToString();
+
+                    if (exitCodeAgain != 0)
+                    {
+                        throw new Exception($"Unable to restore.\nOutput:\n${restoreOutput}\n");
+                    }
+
+                    goto doRestore;
+                }
+
                 throw new Exception(fullOutput);
             }
         }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/SdkPackageUpdater.cs

@@ -20,132 +20,124 @@ internal static class SdkPackageUpdater
         string previousDependencyVersion,
         string newDependencyVersion,
         bool isTransitive,
-        Logger logger
-    )
+        Logger logger)
     {
         // SDK-style project, modify the XML directly
         logger.Log("  Running for SDK-style project");
-        var buildFiles = await MSBuildHelper.LoadBuildFiles(repoRootPath, projectPath);
-
-        var newDependencyNuGetVersion = NuGetVersion.Parse(newDependencyVersion);
 
-        // update all dependencies, including transitive
+        var buildFiles = await MSBuildHelper.LoadBuildFiles(repoRootPath, projectPath);
         var tfms = MSBuildHelper.GetTargetFrameworkMonikers(buildFiles);
 
         // Get the set of all top-level dependencies in the current project
         var topLevelDependencies = MSBuildHelper.GetTopLevelPackageDependencyInfos(buildFiles).ToArray();
+        if (!await DoesDependencyRequireUpdateAsync(repoRootPath, projectPath, tfms, topLevelDependencies, dependencyName, newDependencyVersion, logger))
+        {
+            return;
+        }
 
-        var packageFoundInDependencies = false;
-        var packageNeedsUpdating = false;
-
-        foreach (var tfm in tfms)
+        if (isTransitive)
         {
-            var dependencies = await MSBuildHelper.GetAllPackageDependenciesAsync(repoRootPath, projectPath, tfm, topLevelDependencies, logger);
-            foreach (var (packageName, packageVersion, _, _, _, _) in dependencies)
+            await UpdateTransitiveDependencyAsnyc(projectPath, dependencyName, newDependencyVersion, buildFiles, logger);
+        }
+        else
+        {
+            var peerDependencies = await GetUpdatedPeerDependenciesAsync(repoRootPath, projectPath, tfms, dependencyName, newDependencyVersion, logger);
+            if (peerDependencies is null)
             {
-                if (packageName.Equals(dependencyName, StringComparison.OrdinalIgnoreCase))
-                {
-                    packageFoundInDependencies = true;
-
-                    var nugetVersion = NuGetVersion.Parse(packageVersion);
-                    if (nugetVersion < newDependencyNuGetVersion)
-                    {
-                        packageNeedsUpdating = true;
-                    }
-                }
+                return;
             }
-        }
 
-        // Skip updating the project if the dependency does not exist in the graph
-        if (!packageFoundInDependencies)
-        {
-            logger.Log($"    Package [{dependencyName}] Does not exist as a dependency in [{projectPath}].");
-            return;
+            UpdateTopLevelDepdendency(buildFiles, dependencyName, previousDependencyVersion, newDependencyVersion, peerDependencies, logger);
         }
 
-        // Skip updating the project if the dependency version meets or exceeds the newDependencyVersion
-        if (!packageNeedsUpdating)
+        if (!await AreDependenciesCoherentAsync(repoRootPath, projectPath, dependencyName, logger, buildFiles, tfms))
         {
-            logger.Log($"    Package [{dependencyName}] already meets the requested dependency version in [{projectPath}].");
             return;
         }
 
-        var newDependency = new[] { new Dependency(dependencyName, newDependencyVersion, DependencyType.Unknown) };
-        var tfmsAndDependencies = new Dictionary<string, Dependency[]>();
-        foreach (var tfm in tfms)
-        {
-            var dependencies = await MSBuildHelper.GetAllPackageDependenciesAsync(repoRootPath, projectPath, tfm, newDependency, logger);
-            tfmsAndDependencies[tfm] = dependencies;
-        }
+        await SaveBuildFilesAsync(buildFiles, logger);
+    }
 
-        // stop update process if we find conflicting package versions
-        var conflictingPackageVersionsFound = false;
-        var packagesAndVersions = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
-        foreach (var (_, dependencies) in tfmsAndDependencies)
+    /// <summary>
+    /// Verifies that the package does not already satisfy the requested dependency version.
+    /// </summary>
+    /// <returns>Returns false if the package is not found or does not need to be updated.</returns>
+    private static async Task<bool> DoesDependencyRequireUpdateAsync(
+        string repoRootPath,
+        string projectPath,
+        string[] tfms,
+        Dependency[] topLevelDependencies,
+        string dependencyName,
+        string newDependencyVersion,
+        Logger logger)
+    {
+        var newDependencyNuGetVersion = NuGetVersion.Parse(newDependencyVersion);
+
+        bool packageFound = false;
+        bool needsUpdate = false;
+
+        foreach (var tfm in tfms)
         {
+            var dependencies = await MSBuildHelper.GetAllPackageDependenciesAsync(
+                repoRootPath,
+                projectPath,
+                tfm,
+                topLevelDependencies,
+                logger);
             foreach (var (packageName, packageVersion, _, _, _, _) in dependencies)
             {
-                if (packagesAndVersions.TryGetValue(packageName, out var existingVersion) &&
-                    existingVersion != packageVersion)
+                if (packageVersion is null)
                 {
-                    logger.Log($"    Package [{packageName}] tried to update to version [{packageVersion}], but found conflicting package version of [{existingVersion}].");
-                    conflictingPackageVersionsFound = true;
+                    continue;
                 }
-                else
+
+                if (packageName.Equals(dependencyName, StringComparison.OrdinalIgnoreCase))
                 {
-                    packagesAndVersions[packageName] = packageVersion!;
+                    packageFound = true;
+
+                    var nugetVersion = NuGetVersion.Parse(packageVersion);
+                    if (nugetVersion < newDependencyNuGetVersion)
+                    {
+                        needsUpdate = true;
+                        break;
+                    }
                 }
             }
-        }
 
-        if (conflictingPackageVersionsFound)
-        {
-            return;
+            if (packageFound && needsUpdate)
+            {
+                break;
+            }
         }
 
-        var unupgradableTfms = tfmsAndDependencies.Where(kvp => !kvp.Value.Any()).Select(kvp => kvp.Key);
-        if (unupgradableTfms.Any())
+        // Skip updating the project if the dependency does not exist in the graph
+        if (!packageFound)
         {
-            logger.Log($"    The following target frameworks could not find packages to upgrade: {string.Join(", ", unupgradableTfms)}");
-            return;
+            logger.Log($"    Package [{dependencyName}] Does not exist as a dependency in [{projectPath}].");
+            return false;
         }
 
-        if (isTransitive)
-        {
-            var directoryPackagesWithPinning = buildFiles.OfType<ProjectBuildFile>()
-                .FirstOrDefault(bf => IsCpmTransitivePinningEnabled(bf));
-            if (directoryPackagesWithPinning is not null)
-            {
-                PinTransitiveDependency(directoryPackagesWithPinning, dependencyName, newDependencyVersion, logger);
-            }
-            else
-            {
-                await AddTransitiveDependencyAsync(projectPath, dependencyName, newDependencyVersion, logger);
-            }
-        }
-        else
+        // Skip updating the project if the dependency version meets or exceeds the newDependencyVersion
+        if (!needsUpdate)
         {
-            UpdateTopLevelDepdendency(buildFiles, dependencyName, previousDependencyVersion, newDependencyVersion, packagesAndVersions, logger);
+            logger.Log($"    Package [{dependencyName}] already meets the requested dependency version in [{projectPath}].");
+            return false;
         }
 
-        var updatedTopLevelDependencies = MSBuildHelper.GetTopLevelPackageDependencyInfos(buildFiles);
-        foreach (var tfm in tfms)
+        return true;
+    }
+
+    private static async Task UpdateTransitiveDependencyAsnyc(string projectPath, string dependencyName, string newDependencyVersion, ImmutableArray<ProjectBuildFile> buildFiles, Logger logger)
+    {
+        var directoryPackagesWithPinning = buildFiles.OfType<ProjectBuildFile>()
+            .FirstOrDefault(bf => IsCpmTransitivePinningEnabled(bf));
+        if (directoryPackagesWithPinning is not null)
         {
-            var updatedPackages = await MSBuildHelper.GetAllPackageDependenciesAsync(repoRootPath, projectPath, tfm, updatedTopLevelDependencies.ToArray(), logger);
-            var dependenciesAreCoherent = await MSBuildHelper.DependenciesAreCoherentAsync(repoRootPath, projectPath, tfm, updatedPackages, logger);
-            if (!dependenciesAreCoherent)
-            {
-                logger.Log($"    Package [{dependencyName}] could not be updated in [{projectPath}] because it would cause a dependency conflict.");
-                return;
-            }
+            PinTransitiveDependency(directoryPackagesWithPinning, dependencyName, newDependencyVersion, logger);
         }
-
-        foreach (var buildFile in buildFiles)
+        else
         {
-            if (await buildFile.SaveAsync())
-            {
-                logger.Log($"    Saved [{buildFile.RepoRelativePath}].");
-            }
+            await AddTransitiveDependencyAsync(projectPath, dependencyName, newDependencyVersion, logger);
         }
     }
 
@@ -246,14 +238,68 @@ internal static class SdkPackageUpdater
         }
     }
 
+    /// <summary>
+    /// Gets the set of peer dependencies that need to be updated.
+    /// </summary>
+    /// <returns>Returns null if there are conflicting versions.</returns>
+    private static async Task<Dictionary<string, string>?> GetUpdatedPeerDependenciesAsync(
+        string repoRootPath,
+        string projectPath,
+        string[] tfms,
+        string dependencyName,
+        string newDependencyVersion,
+        Logger logger)
+    {
+        var newDependency = new[] { new Dependency(dependencyName, newDependencyVersion, DependencyType.Unknown) };
+        var tfmsAndDependencies = new Dictionary<string, Dependency[]>();
+        foreach (var tfm in tfms)
+        {
+            var dependencies = await MSBuildHelper.GetAllPackageDependenciesAsync(repoRootPath, projectPath, tfm, newDependency, logger);
+            tfmsAndDependencies[tfm] = dependencies;
+        }
+
+        var unupgradableTfms = tfmsAndDependencies.Where(kvp => !kvp.Value.Any()).Select(kvp => kvp.Key);
+        if (unupgradableTfms.Any())
+        {
+            logger.Log($"    The following target frameworks could not find packages to upgrade: {string.Join(", ", unupgradableTfms)}");
+            return null;
+        }
+
+        var conflictingPackageVersionsFound = false;
+        var packagesAndVersions = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
+        foreach (var (_, dependencies) in tfmsAndDependencies)
+        {
+            foreach (var (packageName, packageVersion, _, _, _, _) in dependencies)
+            {
+                if (packagesAndVersions.TryGetValue(packageName, out var existingVersion) &&
+                    existingVersion != packageVersion)
+                {
+                    logger.Log($"    Package [{packageName}] tried to update to version [{packageVersion}], but found conflicting package version of [{existingVersion}].");
+                    conflictingPackageVersionsFound = true;
+                }
+                else
+                {
+                    packagesAndVersions[packageName] = packageVersion!;
+                }
+            }
+        }
+
+        // stop update process if we find conflicting package versions
+        if (conflictingPackageVersionsFound)
+        {
+            return null;
+        }
+
+        return packagesAndVersions;
+    }
+
     private static void UpdateTopLevelDepdendency(
         ImmutableArray<ProjectBuildFile> buildFiles,
         string dependencyName,
         string previousDependencyVersion,
         string newDependencyVersion,
-        IDictionary<string, string> packagesAndVersions,
-        Logger logger
-    )
+        IDictionary<string, string> peerDependencies,
+        Logger logger)
     {
         var result = TryUpdateDependencyVersion(buildFiles, dependencyName, previousDependencyVersion, newDependencyVersion, logger);
         if (result == UpdateResult.NotFound)
@@ -262,7 +308,7 @@ internal static class SdkPackageUpdater
             return;
         }
 
-        foreach (var (packageName, packageVersion) in packagesAndVersions.Where(kvp => string.Compare(kvp.Key, dependencyName, StringComparison.OrdinalIgnoreCase) != 0))
+        foreach (var (packageName, packageVersion) in peerDependencies.Where(kvp => string.Compare(kvp.Key, dependencyName, StringComparison.OrdinalIgnoreCase) != 0))
         {
             TryUpdateDependencyVersion(buildFiles, packageName, previousDependencyVersion: null, newDependencyVersion: packageVersion, logger);
         }
@@ -515,4 +561,32 @@ internal static class SdkPackageUpdater
                 packageName,
                 StringComparison.OrdinalIgnoreCase) &&
             (e.GetAttributeOrSubElementValue("Version", StringComparison.OrdinalIgnoreCase) ?? e.GetAttributeOrSubElementValue("VersionOverride", StringComparison.OrdinalIgnoreCase)) is not null);
+
+    private static async Task<bool> AreDependenciesCoherentAsync(string repoRootPath, string projectPath, string dependencyName, Logger logger, ImmutableArray<ProjectBuildFile> buildFiles, string[] tfms)
+    {
+        var updatedTopLevelDependencies = MSBuildHelper.GetTopLevelPackageDependencyInfos(buildFiles).ToArray();
+        foreach (var tfm in tfms)
+        {
+            var updatedPackages = await MSBuildHelper.GetAllPackageDependenciesAsync(repoRootPath, projectPath, tfm, updatedTopLevelDependencies, logger);
+            var dependenciesAreCoherent = await MSBuildHelper.DependenciesAreCoherentAsync(repoRootPath, projectPath, tfm, updatedPackages, logger);
+            if (!dependenciesAreCoherent)
+            {
+                logger.Log($"    Package [{dependencyName}] could not be updated in [{projectPath}] because it would cause a dependency conflict.");
+                return false;
+            }
+        }
+
+        return true;
+    }
+
+    private static async Task SaveBuildFilesAsync(ImmutableArray<ProjectBuildFile> buildFiles, Logger logger)
+    {
+        foreach (var buildFile in buildFiles)
+        {
+            if (await buildFile.SaveAsync())
+            {
+                logger.Log($"    Saved [{buildFile.RepoRelativePath}].");
+            }
+        }
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Updater/UpdaterWorker.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.IO;
+using System.Linq;
 using System.Threading.Tasks;
 
 namespace NuGetUpdater.Core;
@@ -9,6 +10,7 @@ public class UpdaterWorker
 {
     private readonly Logger _logger;
     private readonly HashSet<string> _processedGlobalJsonPaths = new(StringComparer.OrdinalIgnoreCase);
+    private readonly HashSet<string> _processedProjectPaths = new(StringComparer.OrdinalIgnoreCase);
 
     public UpdaterWorker(Logger logger)
     {
@@ -49,6 +51,7 @@ public class UpdaterWorker
         }
 
         _processedGlobalJsonPaths.Clear();
+        _processedProjectPaths.Clear();
     }
 
     private async Task RunForSolutionAsync(
@@ -101,7 +104,40 @@ public class UpdaterWorker
         string newDependencyVersion,
         bool isTransitive)
     {
-        _logger.Log($"Running for project [{projectPath}]");
+        _logger.Log($"Running for project file [{Path.GetRelativePath(repoRootPath, projectPath)}]");
+        if (!File.Exists(projectPath))
+        {
+            _logger.Log($"File [{projectPath}] does not exist.");
+            return;
+        }
+
+        var projectFilePaths = MSBuildHelper.GetProjectPathsFromProject(projectPath);
+        foreach (var projectFullPath in projectFilePaths.Concat([projectPath]))
+        {
+            // If there is some MSBuild logic that needs to run to fully resolve the path skip the project
+            if (File.Exists(projectFullPath))
+            {
+                await RunUpdaterAsync(repoRootPath, projectFullPath, dependencyName, previousDependencyVersion, newDependencyVersion, isTransitive);
+            }
+        }
+    }
+
+    private async Task RunUpdaterAsync(
+        string repoRootPath,
+        string projectPath,
+        string dependencyName,
+        string previousDependencyVersion,
+        string newDependencyVersion,
+        bool isTransitive)
+    {
+        if (_processedProjectPaths.Contains(projectPath))
+        {
+            return;
+        }
+
+        _processedProjectPaths.Add(projectPath);
+
+        _logger.Log($"Updating project [{projectPath}]");
 
         if (!isTransitive
             && MSBuildHelper.GetGlobalJsonPath(repoRootPath, projectPath) is { } globalJsonPath
@@ -111,7 +147,7 @@ public class UpdaterWorker
             await GlobalJsonUpdater.UpdateDependencyAsync(repoRootPath, globalJsonPath, dependencyName, previousDependencyVersion, newDependencyVersion, _logger);
         }
 
-        if (NuGetHelper.HasProjectConfigFile(projectPath))
+        if (NuGetHelper.HasPackagesConfigFile(projectPath))
         {
             await PackagesConfigUpdater.UpdateDependencyAsync(repoRootPath, projectPath, dependencyName, previousDependencyVersion, newDependencyVersion, isTransitive, _logger);
         }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Utilities/MSBuildHelper.cs

@@ -21,6 +21,8 @@ using NuGetUpdater.Core.Utilities;
 
 namespace NuGetUpdater.Core;
 
+using EvaluationResult = (MSBuildHelper.EvaluationResultType ResultType, string EvaluatedValue, string? ErrorMessage);
+
 internal static partial class MSBuildHelper
 {
     public static string MSBuildPath { get; private set; } = string.Empty;
@@ -57,7 +59,10 @@ internal static partial class MSBuildHelper
                 if (property.Name.Equals("TargetFramework", StringComparison.OrdinalIgnoreCase) ||
                     property.Name.Equals("TargetFrameworks", StringComparison.OrdinalIgnoreCase))
                 {
-                    targetFrameworkValues.Add(property.Value);
+                    foreach (var tfm in property.Value.Split(';', StringSplitOptions.RemoveEmptyEntries | StringSplitOptions.TrimEntries))
+                    {
+                        targetFrameworkValues.Add(tfm);
+                    }
                 }
                 else if (property.Name.Equals("TargetFrameworkVersion", StringComparison.OrdinalIgnoreCase))
                 {
@@ -75,8 +80,16 @@ internal static partial class MSBuildHelper
 
         foreach (var targetFrameworkValue in targetFrameworkValues)
         {
-            var tfms = targetFrameworkValue;
-            tfms = GetRootedValue(tfms, propertyInfo);
+            var (resultType, tfms, errorMessage) =
+                GetEvaluatedValue(targetFrameworkValue, propertyInfo, propertiesToIgnore: ["TargetFramework", "TargetFrameworks"]);
+            if (resultType == EvaluationResultType.PropertyIgnored)
+            {
+                continue;
+            }
+            else if (resultType != EvaluationResultType.Success)
+            {
+                throw new InvalidDataException(errorMessage);
+            }
 
             if (string.IsNullOrEmpty(tfms))
             {
@@ -101,10 +114,18 @@ internal static partial class MSBuildHelper
     public static IEnumerable<string> GetProjectPathsFromProject(string projFilePath)
     {
         var projectStack = new Stack<(string folderPath, ProjectRootElement)>();
-        var projectRootElement = ProjectRootElement.Open(projFilePath);
         var processedProjectFiles = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
+        using var projectCollection = new ProjectCollection();
 
-        projectStack.Push((Path.GetFullPath(Path.GetDirectoryName(projFilePath)!), projectRootElement));
+        try
+        {
+            var projectRootElement = ProjectRootElement.Open(projFilePath, projectCollection);
+            projectStack.Push((Path.GetFullPath(Path.GetDirectoryName(projFilePath)!), projectRootElement));
+        }
+        catch (InvalidProjectFileException)
+        {
+            yield break; // Skip invalid project files
+        }
 
         while (projectStack.Count > 0)
         {
@@ -137,7 +158,7 @@ internal static partial class MSBuildHelper
                         // If there is some MSBuild logic that needs to run to fully resolve the path skip the project
                         if (File.Exists(file))
                         {
-                            var additionalProjectRootElement = ProjectRootElement.Open(file);
+                            var additionalProjectRootElement = ProjectRootElement.Open(file, projectCollection);
                             projectStack.Push((Path.GetFullPath(Path.GetDirectoryName(file)!), additionalProjectRootElement));
                             processedProjectFiles.Add(file);
                         }
@@ -222,9 +243,13 @@ internal static partial class MSBuildHelper
             }
 
             // Walk the property replacements until we don't find another one.
-            packageVersion = GetRootedValue(packageVersion, propertyInfo);
+            var evaluationResult = GetEvaluatedValue(packageVersion, propertyInfo);
+            if (evaluationResult.ResultType != EvaluationResultType.Success)
+            {
+                throw new InvalidDataException(evaluationResult.ErrorMessage);
+            }
 
-            packageVersion = packageVersion.TrimStart('[', '(').TrimEnd(']', ')');
+            packageVersion = evaluationResult.EvaluatedValue.TrimStart('[', '(').TrimEnd(']', ')');
 
             // We don't know the version for range requirements or wildcard
             // requirements, so return "" for these.
@@ -237,25 +262,32 @@ internal static partial class MSBuildHelper
     /// <summary>
     /// Given an MSBuild string and a set of properties, returns our best guess at the final value MSBuild will evaluate to.
     /// </summary>
-    /// <param name="msbuildString"></param>
-    /// <param name="propertyInfo"></param>
-    /// <returns></returns>
-    public static string GetRootedValue(string msbuildString, Dictionary<string, string> propertyInfo)
+    public static EvaluationResult GetEvaluatedValue(string msbuildString, Dictionary<string, string> propertyInfo, params string[] propertiesToIgnore)
     {
+        var ignoredProperties = new HashSet<string>(propertiesToIgnore, StringComparer.OrdinalIgnoreCase);
         var seenProperties = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
+
         while (TryGetPropertyName(msbuildString, out var propertyName))
         {
+            if (ignoredProperties.Contains(propertyName))
+            {
+                return (EvaluationResultType.PropertyIgnored, msbuildString, $"Property '{propertyName}' is ignored.");
+            }
+
             if (!seenProperties.Add(propertyName))
             {
-                throw new InvalidDataException($"Property '{propertyName}' has a circular reference.");
+                return (EvaluationResultType.CircularReference, msbuildString, $"Property '{propertyName}' has a circular reference.");
+            }
+
+            if (!propertyInfo.TryGetValue(propertyName, out var propertyValue))
+            {
+                return (EvaluationResultType.PropertyNotFound, msbuildString, $"Property '{propertyName}' was not found.");
             }
 
-            msbuildString = propertyInfo.TryGetValue(propertyName, out var propertyValue)
-                ? msbuildString.Replace($"$({propertyName})", propertyValue)
-                : throw new InvalidDataException($"Property '{propertyName}' was not found.");
+            msbuildString = msbuildString.Replace($"$({propertyName})", propertyValue);
         }
 
-        return msbuildString;
+        return (EvaluationResultType.Success, msbuildString, null);
     }
 
     public static bool TryGetPropertyName(string versionContent, [NotNullWhen(true)] out string? propertyName)
@@ -360,7 +392,17 @@ internal static partial class MSBuildHelper
         await File.WriteAllTextAsync(tempProjectPath, projectContents);
 
         // prevent directory crawling
-        await File.WriteAllTextAsync(Path.Combine(tempDir.FullName, "Directory.Build.props"), "<Project />");
+        await File.WriteAllTextAsync(
+            Path.Combine(tempDir.FullName, "Directory.Build.props"),
+            """
+            <Project>
+              <PropertyGroup>
+                <!-- For Windows-specific apps -->
+                <EnableWindowsTargeting>true</EnableWindowsTargeting>
+              </PropertyGroup>
+            </Project>
+            """);
+
         await File.WriteAllTextAsync(Path.Combine(tempDir.FullName, "Directory.Build.targets"), "<Project />");
         await File.WriteAllTextAsync(Path.Combine(tempDir.FullName, "Directory.Packages.props"), "<Project />");
 
@@ -482,4 +524,12 @@ internal static partial class MSBuildHelper
 
     [GeneratedRegex("^\\s*NuGetData::Package=(?<PackageName>[^,]+), Version=(?<PackageVersion>.+)$")]
     private static partial Regex PackagePattern();
+
+    internal enum EvaluationResultType
+    {
+        Success,
+        PropertyIgnored,
+        CircularReference,
+        PropertyNotFound,
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Utilities/NuGetHelper.cs

@@ -6,7 +6,7 @@ internal static class NuGetHelper
 {
     internal const string PackagesConfigFileName = "packages.config";
 
-    public static bool HasProjectConfigFile(string projectPath)
+    public static bool HasPackagesConfigFile(string projectPath)
     {
         var projectDirectory = Path.GetDirectoryName(projectPath);
         var packagesConfigPath = PathHelper.JoinPath(projectDirectory, PackagesConfigFileName);