dependabot-nuget 0.310.0 → 0.312.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60d194f7c1aa9c0a61df000bad1d3030110a66a73cb86615c97fe0a3d053cc56
-  data.tar.gz: d87118a0fb76c9b571b27ec5a66a3ae65d808e17daa9d68cb734962ee3aa5fd7
+  metadata.gz: acb9d775c29330f4cae905aa23b7ad9c36f0ed161d8cc1c2fbfc7391fb52646b
+  data.tar.gz: 97b60783a35914969fe10efc5235a91d29a470faf40e4520cf331bf6bafe21f1
 SHA512:
-  metadata.gz: 6d9fcc54e90b7c3ea93eedab675ce512a989176b02c676d442f9b9edcbbb5653dd9bacc777d77c52f1061ede4048523dfc9693a50c01ce089c5205dcf1cdfddb
-  data.tar.gz: 12abc6e3b3c04933d6d341ea9532d9dbae1cc121310c1f4252e0f7d9f2776861c1b3cb696d813a1c37717363c219c4196743665001ecf7ade03d8bf40aaf58bc
+  metadata.gz: d1d117e611b42497834c3276f08e35e3a70a3b2d0f960ac3d666dde3c12eb32697f911e0af3409e76500977f4c1db835c4979a85879e86bbd00e0ec0df4cc34c
+  data.tar.gz: b3f8f76bc8be51fc23493c1660cb41e5f6b9c10721941bd98c72c49652c807bb320def722cdc3c08d822f2cd2ed0dbe993b75cd6d43d0f1593de6890a753af32

data/helpers/lib/NuGetUpdater/Directory.Packages.props

@@ -38,7 +38,7 @@
     <PackageVersion Include="System.Text.Json" Version="9.0.3" />
     <PackageVersion Include="System.Text.RegularExpressions" Version="4.3.1" />
     <PackageVersion Include="System.Threading.Tasks.Dataflow" Version="9.0.3" />
-    <PackageVersion Include="xunit" Version="2.9.3" />
+    <PackageVersion Include="xunit.v3" Version="2.0.1" />
     <PackageVersion Include="xunit.runner.visualstudio" Version="3.0.2" />
   </ItemGroup>
 </Project>

data/helpers/lib/NuGetUpdater/DotNetPackageCorrelation.Test/DotNetPackageCorrelation.Test.csproj

@@ -7,7 +7,7 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.NET.Test.Sdk" />
-    <PackageReference Include="xunit" />
+    <PackageReference Include="xunit.v3" />
     <PackageReference Include="xunit.runner.visualstudio" />
   </ItemGroup>
 

data/helpers/lib/NuGetUpdater/NuGetUpdater.Cli.Test/EntryPointTests.Run.cs

@@ -58,7 +58,7 @@ public partial class EntryPointTests
             );
         }
 
-        private static async Task RunAsync(TestFile[] files, Job job, string[] expectedUrls, MockNuGetPackage[]? packages = null)
+        private static async Task RunAsync(TestFile[] files, Job job, string[] expectedUrls, MockNuGetPackage[]? packages = null, string? repoContentsPath = null)
         {
             using var tempDirectory = new TemporaryDirectory();
 
@@ -90,7 +90,7 @@ public partial class EntryPointTests
                 "--job-path",
                 jobPath,
                 "--repo-contents-path",
-                tempDirectory.DirectoryPath,
+                repoContentsPath ?? tempDirectory.DirectoryPath,
                 "--api-url",
                 http.BaseUrl,
                 "--job-id",

data/helpers/lib/NuGetUpdater/NuGetUpdater.Cli.Test/NuGetUpdater.Cli.Test.csproj

@@ -13,7 +13,7 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.NET.Test.Sdk" />
-    <PackageReference Include="xunit" />
+    <PackageReference Include="xunit.v3" />
     <PackageReference Include="xunit.runner.visualstudio" />
   </ItemGroup>
 

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Analyze/VersionFinder.cs

@@ -1,5 +1,7 @@
 using System.Collections.Immutable;
 
+using Newtonsoft.Json;
+
 using NuGet.Common;
 using NuGet.Configuration;
 using NuGet.Frameworks;
@@ -98,6 +100,11 @@ internal static class VersionFinder
                 // if anything goes wrong here, the package source obviously doesn't contain the requested package
                 continue;
             }
+            catch (JsonReaderException ex)
+            {
+                // unable to parse server response
+                throw new BadResponseException(ex.Message, source.Source);
+            }
 
             var feedVersions = (await feed.GetVersions(
                 packageId,

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/BadResponseException.cs

@@ -0,0 +1,12 @@
+namespace NuGetUpdater.Core;
+
+internal class BadResponseException : Exception
+{
+    public string Uri { get; }
+
+    public BadResponseException(string message, string uri)
+        : base(message)
+    {
+        Uri = uri;
+    }
+}

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Discover/SdkProjectDiscovery.cs

@@ -1,9 +1,11 @@
 using System.Collections.Immutable;
+using System.Text.Json;
 using System.Xml.Linq;
 using System.Xml.XPath;
 
 using Microsoft.Build.Logging.StructuredLogger;
 
+using NuGet.Frameworks;
 using NuGet.Versioning;
 
 using NuGetUpdater.Core.Utilities;
@@ -39,6 +41,8 @@ internal static class SdkProjectDiscovery
     // these packages are resolved during restore, but aren't really updatable and shouldn't be reported as dependencies
     private static readonly HashSet<string> NonReportedPackgeNames = new HashSet<string>(StringComparer.OrdinalIgnoreCase)
     {
+        "Microsoft.NETCore.Platforms",
+        "Microsoft.NETCore.Targets",
         "NETStandard.Library"
     };
 
@@ -84,6 +88,9 @@ internal static class SdkProjectDiscovery
         Dictionary<string, Dictionary<string, Dictionary<string, string>>> packagesReplacedBySdkPerProject = new(PathComparer.Instance);
         //    projectPath                tfm        packageName  packageVersion
 
+        Dictionary<string, Dictionary<string, HashSet<string>>> packageDependencies = new(PathComparer.Instance);
+        //    projectPath                tfm  packageNames
+
         Dictionary<string, Dictionary<string, string>> resolvedProperties = new(PathComparer.Instance);
         //    projectPath       propertyName  propertyValue
 
@@ -228,6 +235,28 @@ internal static class SdkProjectDiscovery
                                         }
                                     }
                                 }
+
+                                // track all referenced projects in case they have no assemblies and can't be otherwise reported
+                                if (addItem.Name.Equals("PackageDependencies", StringComparison.OrdinalIgnoreCase))
+                                {
+                                    var projectEvaluation = GetNearestProjectEvaluation(node);
+                                    if (projectEvaluation is not null)
+                                    {
+                                        var specificPackageDeps = packageDependencies.GetOrAdd(projectEvaluation.ProjectFile, () => new(StringComparer.OrdinalIgnoreCase));
+                                        var tfm = GetPropertyValueFromProjectEvaluation(projectEvaluation, "TargetFramework");
+                                        if (tfm is not null)
+                                        {
+                                            var packagesByTfm = specificPackageDeps.GetOrAdd(tfm, () => new(StringComparer.OrdinalIgnoreCase));
+                                            foreach (var package in addItem.Children.OfType<Item>())
+                                            {
+                                                if (!NonReportedPackgeNames.Contains(package.Name))
+                                                {
+                                                    packagesByTfm.Add(package.Name);
+                                                }
+                                            }
+                                        }
+                                    }
+                                }
                             }
                             break;
                         case Target target when target.Name == "_HandlePackageFileConflicts":
@@ -330,7 +359,7 @@ internal static class SdkProjectDiscovery
         }
 
         // and done
-        var projectDiscoveryResults = BuildResults(
+        var projectDiscoveryResults = await BuildResults(
             repoRootPath,
             workspacePath,
             packagesPerProject,
@@ -338,6 +367,7 @@ internal static class SdkProjectDiscovery
             packagesReplacedBySdkPerProject,
             topLevelPackagesPerProject,
             resolvedProperties,
+            packageDependencies,
             referencedProjects,
             importedFiles,
             additionalFiles
@@ -345,7 +375,7 @@ internal static class SdkProjectDiscovery
         return projectDiscoveryResults;
     }
 
-    private static ImmutableArray<ProjectDiscoveryResult> BuildResults(
+    private static async Task<ImmutableArray<ProjectDiscoveryResult>> BuildResults(
         string repoRootPath,
         string workspacePath,
         Dictionary<string, Dictionary<string, Dictionary<string, string>>> packagesPerProject,
@@ -353,13 +383,14 @@ internal static class SdkProjectDiscovery
         Dictionary<string, Dictionary<string, Dictionary<string, string>>> packagesReplacedBySdkPerProject,
         Dictionary<string, Dictionary<string, HashSet<string>>> topLevelPackagesPerProject,
         Dictionary<string, Dictionary<string, string>> resolvedProperties,
+        Dictionary<string, Dictionary<string, HashSet<string>>> packageDependencies,
         Dictionary<string, HashSet<string>> referencedProjects,
         Dictionary<string, HashSet<string>> importedFiles,
         Dictionary<string, HashSet<string>> additionalFiles
     )
     {
         var projectDiscoveryResults = new List<ProjectDiscoveryResult>();
-        foreach (var projectPath in packagesPerProject.Keys.OrderBy(p => p)) //packagesPerProject.Keys.OrderBy(p => p).Select(projectPath =>
+        foreach (var projectPath in packagesPerProject.Keys.OrderBy(p => p))
         {
             // gather some project-level information
             var packagesByTfm = packagesPerProject[projectPath];
@@ -400,18 +431,99 @@ internal static class SdkProjectDiscovery
                 .SelectMany(kvp => kvp.Value)
                 .ToHashSet(StringComparer.OrdinalIgnoreCase);
 
+            var propertiesForProject = resolvedProperties.GetOrAdd(projectPath, () => new(StringComparer.OrdinalIgnoreCase));
+            var assetsJson = new Lazy<JsonElement?>(() =>
+            {
+                if (propertiesForProject.TryGetValue("ProjectAssetsFile", out var assetsFilePath))
+                {
+                    var assetsContent = File.ReadAllText(assetsFilePath);
+                    var assets = JsonDocument.Parse(assetsContent).RootElement;
+                    return assets;
+                }
+
+                return null;
+            });
+
             // create dependencies
             var tfms = packagesByTfm.Keys.OrderBy(tfm => tfm).ToImmutableArray();
-            var dependencies = tfms.SelectMany(tfm =>
+            var groupedDependencies = new Dictionary<string, Dependency>(StringComparer.OrdinalIgnoreCase);
+            foreach (var tfm in tfms)
             {
-                return packagesByTfm[tfm].Keys.OrderBy(p => p).Select(packageName =>
+                var parsedTfm = NuGetFramework.Parse(tfm);
+                var packages = packagesByTfm[tfm];
+
+                // augment with any packages that might not have reported assemblies
+                var assetsPackageVersions = new Lazy<Dictionary<string, string>>(() =>
+                {
+                    var result = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase);
+                    if (assetsJson.Value is { } assets &&
+                        assets.TryGetProperty("targets", out var tfmObjects))
+                    {
+                        foreach (var tfmObject in tfmObjects.EnumerateObject())
+                        {
+                            // TFM might have a RID suffix after a slash that we can't parse
+                            var tfmParts = tfmObject.Name.Split('/');
+                            var reportedTargetFramework = NuGetFramework.Parse(tfmParts[0]);
+                            if (reportedTargetFramework == parsedTfm)
+                            {
+                                foreach (var packageObject in tfmObject.Value.EnumerateObject())
+                                {
+                                    var parts = packageObject.Name.Split('/');
+                                    if (parts.Length == 2)
+                                    {
+                                        var packageName = parts[0];
+                                        var packageVersion = parts[1];
+                                        result[packageName] = packageVersion;
+                                    }
+                                }
+                            }
+                        }
+                    }
+
+                    return result;
+                });
+                var packageDepsForProject = packageDependencies.GetOrAdd(projectPath, () => new(StringComparer.OrdinalIgnoreCase));
+                var packageDepsForTfm = packageDepsForProject.GetOrAdd(tfm, () => new(StringComparer.OrdinalIgnoreCase));
+                foreach (var packageDepName in packageDepsForTfm)
+                {
+                    if (packages.ContainsKey(packageDepName))
+                    {
+                        // we already know about this
+                        continue;
+                    }
+
+                    // otherwise find the corresponding version through project.assets.json
+                    if (assetsPackageVersions.Value.TryGetValue(packageDepName, out var packageDepVersion))
+                    {
+                        packages[packageDepName] = packageDepVersion;
+                    }
+                }
+
+                foreach (var package in packages)
                 {
-                    var packageVersion = packagesByTfm[tfm][packageName]!;
+                    var packageName = package.Key;
+                    var packageVersion = package.Value;
                     var isTopLevel = topLevelPackageNames.Contains(packageName);
                     var dependencyType = isTopLevel ? DependencyType.PackageReference : DependencyType.Unknown;
-                    return new Dependency(packageName, packageVersion, dependencyType, TargetFrameworks: [tfm], IsDirect: isTopLevel, IsTransitive: !isTopLevel);
-                });
-            }).ToImmutableArray();
+                    var combinedTfms = new HashSet<string>([tfm], StringComparer.OrdinalIgnoreCase);
+                    if (groupedDependencies.TryGetValue(packageName, out var existingDependency) &&
+                        existingDependency.Version == packageVersion &&
+                        existingDependency.Type == dependencyType &&
+                        existingDependency.TargetFrameworks is not null)
+                    {
+                        // same dependency, combine tfms
+                        combinedTfms.AddRange(existingDependency.TargetFrameworks);
+                    }
+
+                    var normalizedTfms = combinedTfms.OrderBy(t => t).ToImmutableArray();
+                    groupedDependencies[package.Key] = new Dependency(packageName, packageVersion, dependencyType, TargetFrameworks: normalizedTfms, IsDirect: isTopLevel, IsTransitive: !isTopLevel);
+                }
+            }
+
+            var dependencies = groupedDependencies.Values
+                .OrderBy(d => d.Name)
+                .ThenBy(d => d.Version)
+                .ToImmutableArray();
 
             // others
             var properties = resolvedProperties[projectPath]

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/ClosePullRequest.cs

@@ -1,4 +1,5 @@
 using System.Collections.Immutable;
+using System.Text;
 using System.Text.Json.Serialization;
 
 namespace NuGetUpdater.Core.Run.ApiModel;
@@ -9,4 +10,16 @@ public sealed record ClosePullRequest : MessageBase
     public required ImmutableArray<string> DependencyNames { get; init; }
 
     public string Reason { get; init; } = "up_to_date";
+
+    public override string GetReport()
+    {
+        var report = new StringBuilder();
+        report.AppendLine($"{nameof(ClosePullRequest)}: {Reason}");
+        foreach (var dependencyName in DependencyNames)
+        {
+            report.AppendLine($"- {dependencyName}");
+        }
+
+        return report.ToString().Trim();
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/CreatePullRequest.cs

@@ -1,5 +1,8 @@
+using System.Text;
 using System.Text.Json.Serialization;
 
+using NuGet.Versioning;
+
 namespace NuGetUpdater.Core.Run.ApiModel;
 
 public sealed record CreatePullRequest : MessageBase
@@ -15,4 +18,21 @@ public sealed record CreatePullRequest : MessageBase
     public required string PrTitle { get; init; }
     [JsonPropertyName("pr-body")]
     public required string PrBody { get; init; }
+
+    public override string GetReport()
+    {
+        var dependencyNames = Dependencies
+            .OrderBy(d => d.Name, StringComparer.OrdinalIgnoreCase)
+            .ThenBy(d => NuGetVersion.Parse(d.Version!))
+            .Select(d => $"{d.Name}/{d.Version}")
+            .ToArray();
+        var report = new StringBuilder();
+        report.AppendLine(nameof(CreatePullRequest));
+        foreach (var d in dependencyNames)
+        {
+            report.AppendLine($"- {d}");
+        }
+
+        return report.ToString().Trim();
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/Job.cs

@@ -2,7 +2,6 @@ using System.Collections.Immutable;
 using System.Text.Json;
 using System.Text.Json.Serialization;
 
-using NuGet.Credentials;
 using NuGet.Versioning;
 
 namespace NuGetUpdater.Core.Run.ApiModel;
@@ -32,7 +31,7 @@ public sealed record Job
     public bool RejectExternalCode { get; init; } = false;
     public bool RepoPrivate { get; init; } = false;
     public CommitOptions? CommitMessageOptions { get; init; } = null;
-    public ImmutableArray<Dictionary<string, string>>? CredentialsMetadata { get; init; } = null;
+    public ImmutableArray<Dictionary<string, object>>? CredentialsMetadata { get; init; } = null;
     public int MaxUpdaterRunTime { get; init; } = 0;
 
     public IEnumerable<string> GetAllDirectories()

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/JobErrorBase.cs

@@ -1,4 +1,5 @@
 using System.Net;
+using System.Text;
 using System.Text.Json.Serialization;
 
 using Microsoft.Build.Exceptions;
@@ -20,11 +21,36 @@ public abstract record JobErrorBase : MessageBase
     [JsonPropertyName("error-details")]
     public Dictionary<string, object> Details { get; init; } = new();
 
+    public override string GetReport()
+    {
+        var report = new StringBuilder();
+        report.AppendLine($"Error type: {Type}");
+        foreach (var (key, value) in Details)
+        {
+            if (this is UnknownError && key == "error-backtrace")
+            {
+                // there's nothing meaningful in this field
+                continue;
+            }
+
+            var valueString = value.ToString();
+            if (value is IEnumerable<string> strings)
+            {
+                valueString = string.Join(", ", strings);
+            }
+
+            report.AppendLine($"- {key}: {valueString}");
+        }
+
+        return report.ToString().Trim();
+    }
+
     public static JobErrorBase ErrorFromException(Exception ex, string jobId, string currentDirectory)
     {
         return ex switch
         {
             BadRequirementException badRequirement => new BadRequirement(badRequirement.Message),
+            BadResponseException badResponse => new PrivateSourceBadResponse([badResponse.Uri]),
             DependencyNotFoundException dependencyNotFound => new DependencyNotFound(string.Join(", ", dependencyNotFound.Dependencies)),
             HttpRequestException httpRequest => httpRequest.StatusCode switch
             {

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/MessageBase.cs

@@ -2,4 +2,5 @@ namespace NuGetUpdater.Core.Run.ApiModel;
 
 public abstract record MessageBase
 {
+    public abstract string GetReport();
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/UnknownError.cs

@@ -1,13 +1,23 @@
+using System.Text.Json.Serialization;
+
 namespace NuGetUpdater.Core.Run.ApiModel;
 
 public record UnknownError : JobErrorBase
 {
+    [JsonIgnore]
+    public Exception Exception { get; init; }
+
     public UnknownError(Exception ex, string jobId)
         : base("unknown_error")
     {
+        Exception = ex;
+
+        // The following object is parsed by the server and the `error-backtrace` property is expected to be a Ruby
+        // stacktrace.  Since we're not in Ruby we can set an empty string there and append the .NET stacktrace to
+        // the message.
         Details["error-class"] = ex.GetType().Name;
-        Details["error-message"] = ex.Message;
-        Details["error-backtrace"] = ex.StackTrace ?? "<unknown>";
+        Details["error-message"] = ex.ToString();
+        Details["error-backtrace"] = "";
         Details["package-manager"] = "nuget";
         Details["job-id"] = jobId;
     }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/UpdatePullRequest.cs

@@ -1,4 +1,5 @@
 using System.Collections.Immutable;
+using System.Text;
 using System.Text.Json.Serialization;
 
 namespace NuGetUpdater.Core.Run.ApiModel;
@@ -25,4 +26,19 @@ public sealed record UpdatePullRequest : MessageBase
 
     [JsonPropertyName("dependency-group")]
     public required string? DependencyGroup { get; init; }
+
+    public override string GetReport()
+    {
+        var dependencyNames = DependencyNames
+            .Order(StringComparer.OrdinalIgnoreCase)
+            .ToArray();
+        var report = new StringBuilder();
+        report.AppendLine(nameof(UpdatePullRequest));
+        foreach (var d in dependencyNames)
+        {
+            report.AppendLine($"- {d}");
+        }
+
+        return report.ToString().Trim();
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/HttpApiHandler.cs

@@ -2,8 +2,6 @@ using System.Text;
 using System.Text.Json;
 using System.Text.Json.Serialization;
 
-using NuGetUpdater.Core.Run.ApiModel;
-
 namespace NuGetUpdater.Core.Run;
 
 public class HttpApiHandler : IApiHandler
@@ -25,39 +23,27 @@ public class HttpApiHandler : IApiHandler
         _jobId = jobId;
     }
 
-    public async Task RecordUpdateJobError(JobErrorBase error)
-    {
-        await PostAsJson("record_update_job_error", error);
-    }
-
-    public async Task UpdateDependencyList(UpdatedDependencyList updatedDependencyList)
+    public async Task SendAsync(string endpoint, object body, string method)
     {
-        await PostAsJson("update_dependency_list", updatedDependencyList);
-    }
-
-    public async Task IncrementMetric(IncrementMetric incrementMetric)
-    {
-        await PostAsJson("increment_metric", incrementMetric);
-    }
-
-    public async Task CreatePullRequest(CreatePullRequest createPullRequest)
-    {
-        await PostAsJson("create_pull_request", createPullRequest);
-    }
-
-    public async Task ClosePullRequest(ClosePullRequest closePullRequest)
-    {
-        await PostAsJson("close_pull_request", closePullRequest);
-    }
-
-    public async Task UpdatePullRequest(UpdatePullRequest updatePullRequest)
-    {
-        await PostAsJson("update_pull_request", updatePullRequest);
-    }
+        var uri = $"{_apiUrl}/update_jobs/{_jobId}/{endpoint}";
+        var payload = Serialize(body);
+        var content = new StringContent(payload, Encoding.UTF8, "application/json");
+        var httpMethod = new HttpMethod(method);
+        var message = new HttpRequestMessage(httpMethod, uri)
+        {
+            Content = content
+        };
+        var response = await HttpClient.SendAsync(message);
+        if (!response.IsSuccessStatusCode)
+        {
+            var responseContent = await response.Content.ReadAsStringAsync();
+            if (!string.IsNullOrEmpty(responseContent))
+            {
+                responseContent = string.Concat(": ", responseContent);
+            }
 
-    public async Task MarkAsProcessed(MarkAsProcessed markAsProcessed)
-    {
-        await PatchAsJson("mark_as_processed", markAsProcessed);
+            throw new HttpRequestException(message: $"{(int)response.StatusCode} ({response.StatusCode}){responseContent}", inner: null, statusCode: response.StatusCode);
+        }
     }
 
     internal static string Serialize(object body)
@@ -69,21 +55,4 @@ public class HttpApiHandler : IApiHandler
         var payload = JsonSerializer.Serialize(wrappedBody, SerializerOptions);
         return payload;
     }
-
-    private Task PostAsJson(string endpoint, object body) => SendAsJson(endpoint, body, "POST");
-    private Task PatchAsJson(string endpoint, object body) => SendAsJson(endpoint, body, "PATCH");
-
-    private async Task SendAsJson(string endpoint, object body, string method)
-    {
-        var uri = $"{_apiUrl}/update_jobs/{_jobId}/{endpoint}";
-        var payload = Serialize(body);
-        var content = new StringContent(payload, Encoding.UTF8, "application/json");
-        var httpMethod = new HttpMethod(method);
-        var message = new HttpRequestMessage(httpMethod, uri)
-        {
-            Content = content
-        };
-        var response = await HttpClient.SendAsync(message);
-        var _ = response.EnsureSuccessStatusCode();
-    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/IApiHandler.cs

@@ -4,11 +4,37 @@ namespace NuGetUpdater.Core.Run;
 
 public interface IApiHandler
 {
-    Task RecordUpdateJobError(JobErrorBase error);
-    Task UpdateDependencyList(UpdatedDependencyList updatedDependencyList);
-    Task IncrementMetric(IncrementMetric incrementMetric);
-    Task CreatePullRequest(CreatePullRequest createPullRequest);
-    Task ClosePullRequest(ClosePullRequest closePullRequest);
-    Task UpdatePullRequest(UpdatePullRequest updatePullRequest);
-    Task MarkAsProcessed(MarkAsProcessed markAsProcessed);
+    Task SendAsync(string endpoint, object body, string method);
+}
+
+public static class IApiHandlerExtensions
+{
+    public static async Task RecordUpdateJobError(this IApiHandler handler, JobErrorBase error)
+    {
+        await handler.PostAsJson("record_update_job_error", error);
+        if (error is UnknownError unknown)
+        {
+            await handler.PostAsJson("record_update_job_unknown_error", error);
+            var increment = new IncrementMetric()
+            {
+                Metric = "updater.update_job_unknown_error",
+                Tags =
+                {
+                    ["package_manager"] = "nuget",
+                    ["class_name"] = unknown.Exception.GetType().Name
+                },
+            };
+            await handler.IncrementMetric(increment);
+        }
+    }
+
+    public static Task UpdateDependencyList(this IApiHandler handler, UpdatedDependencyList updatedDependencyList) => handler.PostAsJson("update_dependency_list", updatedDependencyList);
+    public static Task IncrementMetric(this IApiHandler handler, IncrementMetric incrementMetric) => handler.PostAsJson("increment_metric", incrementMetric);
+    public static Task CreatePullRequest(this IApiHandler handler, CreatePullRequest createPullRequest) => handler.PostAsJson("create_pull_request", createPullRequest);
+    public static Task ClosePullRequest(this IApiHandler handler, ClosePullRequest closePullRequest) => handler.PostAsJson("close_pull_request", closePullRequest);
+    public static Task UpdatePullRequest(this IApiHandler handler, UpdatePullRequest updatePullRequest) => handler.PostAsJson("update_pull_request", updatePullRequest);
+    public static Task MarkAsProcessed(this IApiHandler handler, MarkAsProcessed markAsProcessed) => handler.PatchAsJson("mark_as_processed", markAsProcessed);
+
+    private static Task PostAsJson(this IApiHandler handler, string endpoint, object body) => handler.SendAsync(endpoint, body, "POST");
+    private static Task PatchAsJson(this IApiHandler handler, string endpoint, object body) => handler.SendAsync(endpoint, body, "PATCH");
 }