feat: Enable AOT-compatibility

.github/workflows/ci.yml

@@ -132,13 +132,9 @@ jobs:
           exit 1
 
       - name: Test (${{ matrix.docker.name }})
-        run: >-
-          dotnet test
-          --configuration Release
-          --framework ${{ matrix.dotnet.tfm }}
-          --no-restore
-          --no-build
-          --logger console
+        run: |
+          ./test/Docker.DotNet.Tests/bin/Release/${{ matrix.dotnet.tfm }}/linux-x64/publish/Docker.DotNet.Tests
+          ./test/Docker.DotNet.TestsV2/bin/Release/${{ matrix.dotnet.tfm }}/linux-x64/publish/Docker.DotNet.TestsV2
         env:
           DOCKER_HOST: ${{ matrix.docker.docker_host }}
           DOCKER_TLS_VERIFY: ${{ matrix.docker.tls_verify }}

Directory.Packages.props

@@ -10,9 +10,10 @@
     <PackageVersion Include="System.Net.Http.Json" Version="8.0.1" />
     <PackageVersion Include="System.Text.Json" Version="8.0.5" />
     <PackageVersion Include="BouncyCastle.Cryptography" Version="2.5.1" />
+    <PackageVersion Include="Microsoft.CodeAnalysis.CSharp" Version="5.3.0" />
+    <PackageVersion Include="Microsoft.CodeAnalysis.Analyzers" Version="5.3.0" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
-    <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
-    <PackageVersion Include="xunit.runner.visualstudio" Version="2.8.2" />
-    <PackageVersion Include="xunit" Version="2.9.2" />
+    <PackageVersion Include="xunit.v3.mtp-v2" Version="4.0.0-pre.81" />
+    <PackageVersion Include="xunit.v3.aot.mtp-v2" Version="4.0.0-pre.81" />
   </ItemGroup>
 </Project>

Docker.DotNet.slnx

@@ -23,4 +23,7 @@
     <Project Path="test/Docker.DotNet.Tests/Docker.DotNet.Tests.csproj" />
     <Project Path="test/Docker.DotNet.TestsV2/Docker.DotNet.TestsV2.csproj" />
   </Folder>
+  <Folder Name="/tools/">
+    <Project Path="tools/Docker.DotNet.SourceAnalyzers/Docker.DotNet.SourceAnalyzers.csproj" />
+  </Folder>
 </Solution>

global.json

@@ -0,0 +1,5 @@
+{
+  "test": {
+    "runner": "Microsoft.Testing.Platform"
+  }
+}

src/Directory.Build.props

@@ -3,6 +3,7 @@
     <Import Project="$([MSBuild]::GetPathOfFileAbove('Directory.Build.props', '$(MSBuildThisFileDirectory)../'))" />
 
     <PropertyGroup>
+        <IsAotCompatible>true</IsAotCompatible>
         <IsPackable>true</IsPackable>
         <TargetFrameworks>net8.0;net9.0;net10.0;netstandard2.0;netstandard2.1</TargetFrameworks>
         <Copyright>Copyright (c) .NET Foundation and Contributors; Andre Hofmeister</Copyright>

src/Docker.DotNet.NPipe/DockerHandlerFactory.cs

@@ -45,10 +45,10 @@ public ResolvedTransport CreateHandler(NPipeTransportOptions transportOptions, C
 
             var dockerStream = new DockerPipeStream(clientStream);
 
-#if NETSTANDARD
-            var namedPipeConnectTimeout = (int)transportOptions.ConnectTimeout.TotalMilliseconds;
-#else
+#if NET
             var namedPipeConnectTimeout = transportOptions.ConnectTimeout;
+#else
+            var namedPipeConnectTimeout = (int)transportOptions.ConnectTimeout.TotalMilliseconds;
 #endif
 
             await clientStream.ConnectAsync(namedPipeConnectTimeout, cancellationToken)

src/Docker.DotNet.NativeHttp/DockerHandlerFactory.cs

@@ -29,7 +29,7 @@ public ResolvedTransport CreateHandler(NativeHttpTransportOptions transportOptio
         var scheme = clientOptions.AuthProvider.TlsEnabled ? Uri.UriSchemeHttps : Uri.UriSchemeHttp;
         var uri = new UriBuilder(clientOptions.Endpoint) { Scheme = scheme }.Uri;
 
-#if NET6_0_OR_GREATER
+#if NET
         var handler = new SocketsHttpHandler
         {
             MaxConnectionsPerServer = MaxConnectionsPerServer,

src/Docker.DotNet.NativeHttp/NativeHttpTransportOptions.cs

@@ -5,7 +5,7 @@ namespace Docker.DotNet.NativeHttp;
 /// </summary>
 public sealed record NativeHttpTransportOptions
 {
-#if NET6_0_OR_GREATER
+#if NET
     /// <summary>
     /// Gets a callback that configures the created <see cref="SocketsHttpHandler"/> instance.
     /// </summary>

src/Docker.DotNet.X509/CertificateCredentials.cs

@@ -15,7 +15,7 @@ public CertificateCredentials(X509Certificate2? certificate)
 
     public HttpMessageHandler ConfigureHandler(HttpMessageHandler handler)
     {
-#if NET6_0_OR_GREATER
+#if NET
         if (handler is SocketsHttpHandler socketsHandler)
         {
             if (_certificate != null)

src/Docker.DotNet.X509/DockerTlsCertificates.cs

@@ -64,7 +64,7 @@ public static X509Certificate2 LoadCertificateFromPemFiles(string certPemPath, s
         }
 
         return certificate;
-#elif NET6_0_OR_GREATER
+#elif NET
         var certificate = X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
 
         if (OperatingSystem.IsWindows())
@@ -75,10 +75,8 @@ public static X509Certificate2 LoadCertificateFromPemFiles(string certPemPath, s
         }
 
         return certificate;
-#elif NETSTANDARD
-        return Polyfills.X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
 #else
-        return X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
+        return Polyfills.X509Certificate2.CreateFromPemFile(certPemPath, keyPemPath);
 #endif
     }
 
@@ -141,7 +139,7 @@ public static RemoteCertificateValidationCallback CreateCertificateAuthorityVali
             using var chain = new X509Chain();
             chain.ChainPolicy.RevocationMode = X509RevocationMode.NoCheck;
 
-#if NET5_0_OR_GREATER
+#if NET
             chain.ChainPolicy.TrustMode = X509ChainTrustMode.CustomRootTrust;
             chain.ChainPolicy.CustomTrustStore.Add(certificateAuthorityCertificate);
             return chain.Build(serverCertificate2);

src/Docker.DotNet/Docker.DotNet.csproj

@@ -21,6 +21,7 @@
     <ProjectReference Include="..\Docker.DotNet.NativeHttp\Docker.DotNet.NativeHttp.csproj" />
     <ProjectReference Include="..\Docker.DotNet.NPipe\Docker.DotNet.NPipe.csproj" />
     <ProjectReference Include="..\Docker.DotNet.Unix\Docker.DotNet.Unix.csproj" />
+    <ProjectReference Include="..\..\tools\Docker.DotNet.SourceAnalyzers\Docker.DotNet.SourceAnalyzers.csproj" OutputItemType="Analyzer" ReferenceOutputAssembly="false" />
   </ItemGroup>
   <ItemGroup>
     <Using Include="System" />
@@ -29,6 +30,7 @@
     <Using Include="System.Collections.Concurrent" />
     <Using Include="System.Collections.Generic" />
     <Using Include="System.Diagnostics" />
+    <Using Include="System.Diagnostics.CodeAnalysis" />
     <Using Include="System.Globalization" />
     <Using Include="System.IO" />
     <Using Include="System.IO.Pipelines" />
@@ -49,6 +51,7 @@
     <Using Include="System.Text" />
     <Using Include="System.Text.Json" />
     <Using Include="System.Text.Json.Serialization" />
+    <Using Include="System.Text.Json.Serialization.Metadata" />
     <Using Include="System.Threading" />
     <Using Include="System.Threading.Tasks" />
     <Using Include="Docker.DotNet" />

src/Docker.DotNet/DockerClient.cs

@@ -1,7 +1,5 @@
 namespace Docker.DotNet;
 
-using System;
-
 public sealed class DockerClient : IDockerClient
 {
     internal readonly IEnumerable<ApiResponseErrorHandlingDelegate> NoErrorHandlers = Enumerable.Empty<ApiResponseErrorHandlingDelegate>();
@@ -370,7 +368,7 @@ private async Task<HttpResponseMessage> PrivateMakeRequestAsync(
 
         if (Timeout.InfiniteTimeSpan == timeout)
         {
-#if NET6_0_OR_GREATER
+#if NET
             return await _client.SendAsync(request, completionOption, cancellationToken)
                 .ConfigureAwait(false);
 #else

src/Docker.DotNet/Endpoints/ConfigsOperations.cs

@@ -11,7 +11,7 @@ internal ConfigOperations(DockerClient client)
 
     public async Task<IList<SwarmConfig>> ListConfigsAsync(CancellationToken cancellationToken = default)
     {
-        return await _client.MakeRequestAsync<IList<SwarmConfig>>(_client.NoErrorHandlers, HttpMethod.Get, "configs", cancellationToken)
+        return await _client.MakeRequestAsync<SwarmConfig[]>(_client.NoErrorHandlers, HttpMethod.Get, "configs", cancellationToken)
             .ConfigureAwait(false);
     }
 

src/Docker.DotNet/Endpoints/ISwarmOperations.cs

@@ -121,7 +121,7 @@ public interface ISwarmOperations
     /// 500 - Server error.
     /// 503 - Node is not part of a swarm.
     /// </remarks>
-    Task<IEnumerable<SwarmService>> ListServicesAsync(ServiceListParameters? parameters = null, CancellationToken cancellationToken = default);
+    Task<IList<SwarmService>> ListServicesAsync(ServiceListParameters? parameters = null, CancellationToken cancellationToken = default);
 
     /// <summary>
     /// Update a service.
@@ -198,7 +198,7 @@ public interface ISwarmOperations
     /// 503 - Node is not part of a swarm.
     /// </remarks>
     /// <returns></returns>
-    Task<IEnumerable<NodeListResponse>> ListNodesAsync(CancellationToken cancellationToken = default);
+    Task<IList<NodeListResponse>> ListNodesAsync(CancellationToken cancellationToken = default);
 
     /// <summary>
     /// Inspect a node.

src/Docker.DotNet/Endpoints/SecretsOperations.cs

@@ -11,7 +11,7 @@ internal SecretsOperations(DockerClient client)
 
     public async Task<IList<Secret>> ListAsync(CancellationToken cancellationToken = default)
     {
-        return await _client.MakeRequestAsync<IList<Secret>>(_client.NoErrorHandlers, HttpMethod.Get, "secrets", cancellationToken)
+        return await _client.MakeRequestAsync<Secret[]>(_client.NoErrorHandlers, HttpMethod.Get, "secrets", cancellationToken)
             .ConfigureAwait(false);
     }
 

src/Docker.DotNet/Endpoints/SwarmOperations.cs

@@ -82,7 +82,7 @@ await _client.MakeRequestAsync([NotInSwarmResponseHandler], HttpMethod.Post, "sw
             .ConfigureAwait(false);
     }
 
-    public async Task<IEnumerable<SwarmService>> ListServicesAsync(ServiceListParameters? parameters = null, CancellationToken cancellationToken = default)
+    public async Task<IList<SwarmService>> ListServicesAsync(ServiceListParameters? parameters = null, CancellationToken cancellationToken = default)
     {
         var queryParameters = parameters == null ? null : new QueryString<ServiceListParameters>(parameters);
 
@@ -193,7 +193,7 @@ private static Dictionary<string, string> RegistryAuthHeaders(AuthConfig? authCo
         };
     }
 
-    public async Task<IEnumerable<NodeListResponse>> ListNodesAsync(CancellationToken cancellationToken = default)
+    public async Task<IList<NodeListResponse>> ListNodesAsync(CancellationToken cancellationToken = default)
     {
         return await _client.MakeRequestAsync<NodeListResponse[]>([NotInSwarmResponseHandler], HttpMethod.Get, "nodes", cancellationToken)
             .ConfigureAwait(false);

src/Docker.DotNet/Endpoints/TasksOperations.cs

@@ -18,7 +18,7 @@ public async Task<IList<TaskResponse>> ListAsync(TasksListParameters? parameters
     {
         var queryParameters = parameters == null ? null : new QueryString<TasksListParameters>(parameters);
 
-        return await _client.MakeRequestAsync<IList<TaskResponse>>(_client.NoErrorHandlers, HttpMethod.Get, "tasks", queryParameters, cancellationToken)
+        return await _client.MakeRequestAsync<TaskResponse[]>(_client.NoErrorHandlers, HttpMethod.Get, "tasks", queryParameters, cancellationToken)
             .ConfigureAwait(false);
     }
 

src/Docker.DotNet/JsonSerializer.cs

@@ -15,11 +15,15 @@ static JsonSerializer()
 
     private JsonSerializer()
     {
+        _options.TypeInfoResolver = JsonTypeInfoResolver.Combine(
+            DockerModelsJsonSerializerContext.Default,
+            DockerExtendedJsonSerializerContext.Default);
         _options.DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull;
         _options.Converters.Add(new JsonEnumMemberConverter<RestartPolicyKind>());
         _options.Converters.Add(new JsonEnumMemberConverter<TaskState>());
         _options.Converters.Add(new JsonDateTimeConverter());
         _options.Converters.Add(new JsonNullableDateTimeConverter());
+        _options.MakeReadOnly();
     }
 
     public static JsonSerializer Instance { get; }
@@ -38,26 +42,27 @@ public HttpContent GetHttpContent<T>(T value)
 
     public string Serialize<T>(T value)
     {
-        return System.Text.Json.JsonSerializer.Serialize(value, _options);
+        return System.Text.Json.JsonSerializer.Serialize(value, JsonTypeInfoCache<T>.Value);
     }
 
     public byte[] SerializeToUtf8Bytes<T>(T value)
     {
-        return System.Text.Json.JsonSerializer.SerializeToUtf8Bytes(value, _options);
+        return System.Text.Json.JsonSerializer.SerializeToUtf8Bytes(value, JsonTypeInfoCache<T>.Value);
     }
 
     public T Deserialize<T>(byte[] json)
     {
-        return System.Text.Json.JsonSerializer.Deserialize<T>(json, _options)!;
+        return System.Text.Json.JsonSerializer.Deserialize(json, JsonTypeInfoCache<T>.Value)!;
     }
 
     public Task<T> DeserializeAsync<T>(HttpContent content, CancellationToken cancellationToken)
     {
-        return content.ReadFromJsonAsync<T>(_options, cancellationToken)!;
+        return content.ReadFromJsonAsync(JsonTypeInfoCache<T>.Value, cancellationToken)!;
     }
 
     public async IAsyncEnumerable<T> DeserializeAsync<T>(Stream stream, [EnumeratorCancellation] CancellationToken cancellationToken)
     {
+        var jsonTypeInfo = JsonTypeInfoCache<T>.Value;
         var reader = PipeReader.Create(stream);
 
         while (true)
@@ -69,7 +74,7 @@ public async IAsyncEnumerable<T> DeserializeAsync<T>(Stream stream, [EnumeratorC
 
             while (!buffer.IsEmpty && TryParseJson(ref buffer, out var jsonDocument))
             {
-                yield return jsonDocument!.Deserialize<T>(_options)!;
+                yield return jsonDocument!.Deserialize(jsonTypeInfo)!;
             }
 
             if (result.IsCompleted)
@@ -95,4 +100,59 @@ private static bool TryParseJson(ref ReadOnlySequence<byte> buffer, out JsonDocu
 
         return false;
     }
-}
+
+    private static class JsonTypeInfoCache<T>
+    {
+        public static readonly JsonTypeInfo<T> Value = (JsonTypeInfo<T>)Instance._options.GetTypeInfo(typeof(T));
+    }
+}
+
+// Additional source-generated metadata for collections and dictionaries used by operations.
+
+[JsonSerializable(typeof(string))]
+[JsonSerializable(typeof(IList<string>))]
+
+// Filters
+[JsonSerializable(typeof(IDictionary<string, IDictionary<string, bool>>))]
+[JsonSerializable(typeof(IDictionary<string, string>))]
+
+// ConfigOperations.ListConfigsAsync
+[JsonSerializable(typeof(SwarmConfig[]))]
+
+// ContainerOperations.ListContainersAsync
+[JsonSerializable(typeof(ContainerListResponse[]))]
+// ContainerOperations.InspectChangesAsync
+[JsonSerializable(typeof(ContainerFileSystemChangeResponse[]))]
+
+// ImageOperations.ListImagesAsync
+[JsonSerializable(typeof(ImagesListResponse[]))]
+// ImageOperations.GetImageHistoryAsync
+[JsonSerializable(typeof(ImageHistoryResponse[]))]
+// ImageOperations.DeleteImageAsync
+[JsonSerializable(typeof(Dictionary<string, string>[]))]
+// ImageOperations.SearchImagesAsync
+[JsonSerializable(typeof(ImageSearchResponse[]))]
+// ImageOperations.RegistryConfigHeaders
+[JsonSerializable(typeof(Dictionary<string, AuthConfig>))]
+
+// NetworkOperations.ListNetworksAsync
+[JsonSerializable(typeof(NetworkResponse[]))]
+
+// PluginOperations.ListPluginsAsync
+[JsonSerializable(typeof(Plugin[]))]
+// PluginOperations.GetPrivilegesAsync
+[JsonSerializable(typeof(PluginPrivilege[]))]
+// PluginOperations.InstallPluginAsync
+[JsonSerializable(typeof(IList<PluginPrivilege>))]
+
+// SecretOperations.ListAsync
+[JsonSerializable(typeof(Secret[]))]
+
+// SwarmOperations.ListServicesAsync
+[JsonSerializable(typeof(SwarmService[]))]
+// SwarmOperations.ListNodesAsync
+[JsonSerializable(typeof(NodeListResponse[]))]
+
+// TaskOperations.ListAsync
+[JsonSerializable(typeof(TaskResponse[]))]
+internal sealed partial class DockerExtendedJsonSerializerContext : JsonSerializerContext { }

src/Docker.DotNet/Models/CommitContainerChangesParameters.Generated.cs

@@ -53,10 +53,10 @@ public CommitContainerChangesParameters(ContainerConfig Config)
         [QueryStringParameter("author", false)]
         public string? Author { get; set; }
 
-        [QueryStringParameter("changes", false, typeof(EnumerableQueryStringConverter))]
+        [QueryStringParameter<EnumerableQueryStringConverter>("changes", false)]
         public IList<string>? Changes { get; set; }
 
-        [QueryStringParameter("pause", false, typeof(BoolQueryStringConverter))]
+        [QueryStringParameter<BoolQueryStringConverter>("pause", false)]
         public bool? Pause { get; set; }
 
         [JsonPropertyName("Hostname")]

src/Docker.DotNet/Models/ContainerAttachParameters.Generated.cs

@@ -3,22 +3,22 @@ namespace Docker.DotNet.Models
 {
     public class ContainerAttachParameters // (main.ContainerAttachParameters)
     {
-        [QueryStringParameter("stream", false, typeof(BoolQueryStringConverter))]
+        [QueryStringParameter<BoolQueryStringConverter>("stream", false)]
         public bool? Stream { get; set; }
 
-        [QueryStringParameter("stdin", false, typeof(BoolQueryStringConverter))]
+        [QueryStringParameter<BoolQueryStringConverter>("stdin", false)]
         public bool? Stdin { get; set; }
 
-        [QueryStringParameter("stdout", false, typeof(BoolQueryStringConverter))]
+        [QueryStringParameter<BoolQueryStringConverter>("stdout", false)]
         public bool? Stdout { get; set; }
 
-        [QueryStringParameter("stderr", false, typeof(BoolQueryStringConverter))]
+        [QueryStringParameter<BoolQueryStringConverter>("stderr", false)]
         public bool? Stderr { get; set; }
 
         [QueryStringParameter("detachKeys", false)]
         public string? DetachKeys { get; set; }
 
-        [QueryStringParameter("logs", false, typeof(BoolQueryStringConverter))]
+        [QueryStringParameter<BoolQueryStringConverter>("logs", false)]
         public bool? Logs { get; set; }
     }
 }