Hide/remove some JSON utility classes (#1039)

dotnet/samples/AgentWebChat/AgentWebChat.Web/AgentDiscoveryClient.cs

@@ -1,7 +1,6 @@
 // Copyright (c) Microsoft. All rights reserved.
 
 using System.Text.Json;
-using Microsoft.Agents.AI.Hosting;
 
 namespace AgentWebChat.Web;
 
@@ -13,7 +12,7 @@ public class AgentDiscoveryClient(HttpClient httpClient, ILogger<AgentDiscoveryC
         response.EnsureSuccessStatusCode();
 
         var json = await response.Content.ReadAsStringAsync(cancellationToken);
-        var agents = JsonSerializer.Deserialize<List<AgentDiscoveryCard>>(json, AgentHostingJsonUtilities.DefaultOptions) ?? [];
+        var agents = JsonSerializer.Deserialize<List<AgentDiscoveryCard>>(json) ?? [];
 
         logger.LogInformation("Retrieved {AgentCount} agents from the API", agents.Count);
         return agents;

dotnet/src/Microsoft.Agents.AI.Hosting/AgentHostingJsonUtilities.cs

@@ -1,56 +0,0 @@
-// Copyright (c) Microsoft. All rights reserved.
-
-using System.Diagnostics.CodeAnalysis;
-using System.Text.Json;
-using System.Text.Json.Serialization;
-
-namespace Microsoft.Agents.AI.Hosting;
-
-/// <summary>Provides a collection of utility methods for working with JSON data in the context of agent hosting.</summary>
-public static partial class AgentHostingJsonUtilities
-{
-    /// <summary>
-    /// Gets the <see cref="JsonSerializerOptions"/> singleton used as the default in JSON serialization operations.
-    /// </summary>
-    /// <remarks>
-    /// <para>
-    /// For Native AOT or applications disabling <see cref="JsonSerializer.IsReflectionEnabledByDefault"/>, this instance
-    /// includes source generated contracts for all common exchange types contained in this library.
-    /// </para>
-    /// <para>
-    /// It additionally turns on the following settings:
-    /// <list type="number">
-    /// <item>Enables <see cref="JsonSerializerDefaults.Web"/> defaults.</item>
-    /// <item>Enables <see cref="JsonIgnoreCondition.WhenWritingNull"/> as the default ignore condition for properties.</item>
-    /// <item>Enables <see cref="JsonNumberHandling.AllowReadingFromString"/> as the default number handling for number types.</item>
-    /// </list>
-    /// </para>
-    /// </remarks>
-    public static JsonSerializerOptions DefaultOptions { get; } = CreateDefaultOptions();
-
-    /// <summary>
-    /// Creates default options to use for agent hosting-related serialization.
-    /// </summary>
-    /// <returns>The configured options.</returns>
-    [UnconditionalSuppressMessage("ReflectionAnalysis", "IL3050:RequiresDynamicCode", Justification = "Converter is guarded by IsReflectionEnabledByDefault check.")]
-    [UnconditionalSuppressMessage("Trimming", "IL2026:Members annotated with 'RequiresUnreferencedCodeAttribute' require dynamic access", Justification = "Converter is guarded by IsReflectionEnabledByDefault check.")]
-    private static JsonSerializerOptions CreateDefaultOptions()
-    {
-        // Copy the configuration from the source generated context.
-        JsonSerializerOptions options = new(JsonContext.Default.Options);
-
-        // Chain with all supported types from Microsoft.Agents.AI.Abstractions.
-        options.TypeInfoResolverChain.Add(AgentAbstractionsJsonUtilities.DefaultOptions.TypeInfoResolver!);
-
-        options.MakeReadOnly();
-        return options;
-    }
-
-    // Keep in sync with CreateDefaultOptions above.
-    [JsonSourceGenerationOptions(JsonSerializerDefaults.Web,
-        DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull,
-        NumberHandling = JsonNumberHandling.AllowReadingFromString)]
-    [JsonSerializable(typeof(AgentThread))]
-    [ExcludeFromCodeCoverage]
-    internal sealed partial class JsonContext : JsonSerializerContext;
-}

dotnet/src/Microsoft.Agents.AI/AgentJsonUtilities.cs

@@ -7,7 +7,7 @@ using System.Text.Json.Serialization;
 namespace Microsoft.Agents.AI;
 
 /// <summary>Provides a collection of utility methods for working with JSON data in the context of agents.</summary>
-public static partial class AgentJsonUtilities
+internal static partial class AgentJsonUtilities
 {
     /// <summary>
     /// Gets the <see cref="JsonSerializerOptions"/> singleton used as the default in JSON serialization operations.