add configure action for json options state viewer

Author: lucasteles

src/Backdash/Network/Client/PeerClient.cs

@@ -13,12 +13,12 @@ namespace Backdash.Network.Client;
 public interface IMessageHandler<T> where T : struct
 {
     /// <summary>
-    ///     Handles sent message.
+    ///     Handles messages sent.
     /// </summary>
     void AfterSendMessage(int bytesSent);
 
     /// <summary>
-    ///     Prepare message to be sent.
+    ///     Prepare messages to be sent.
     /// </summary>
     void BeforeSendMessage(ref T message);
 }
@@ -133,6 +133,7 @@ async Task StartSending(CancellationToken cancellationToken)
 
                     entry.Callback?.BeforeSendMessage(ref entry.Body);
 
+                    // LATER: move the .Span outside the loop (requires >=.NET10)
                     var bodySize = serializer.Serialize(in entry.Body, buffer.Span);
                     var sentSize = await socket.SendToAsync(buffer[..bodySize], entry.Recipient, cancellationToken)
                         .ConfigureAwait(false);

src/Backdash/Options/SyncTestOptions.cs

@@ -82,6 +82,14 @@ public SyncTestOptions<TInput> UseJsonStateParser(JsonSerializerOptions? options
         return this;
     }
 
+    /// <inheritdoc cref="UseJsonStateParser(JsonSerializerOptions?)" />
+    public SyncTestOptions<TInput> UseJsonStateParser(Action<JsonSerializerOptions> configure)
+    {
+        var options = JsonStateStringParser.CreateDefaultJsonOptions();
+        configure.Invoke(options);
+        return UseJsonStateParser(options);
+    }
+
     /// <inheritdoc cref="StateStringParser" />
     public SyncTestOptions<TInput> UseStateStringParser<T>() where T : IStateStringParser, new()
     {

src/Backdash/Session/NetcodeSessionBuilder.cs

@@ -488,7 +488,7 @@ public NetcodeSessionBuilder<TInput> WithSocketFactory(Func<int, NetcodeOptions,
         WithSocketFactory(new DelegateSocketFactory(factory));
 
     /// <summary>
-    ///     Add plugin type
+    ///     Register a plugin by type
     /// </summary>
     /// <seealso cref="ServicesConfig{TInput}" />
     [MemberNotNull(nameof(sessionServices))]
@@ -508,19 +508,17 @@ public NetcodeSessionBuilder<TInput> UsePlugin(INetcodePlugin plugin)
 
 
     /// <summary>
-    ///     Add new custom job
+    ///     Add a new custom job
     /// </summary>
+    /// <seealso cref="ServicesConfig{TInput}" />
     [MemberNotNull(nameof(sessionServices))]
     public NetcodeSessionBuilder<TInput> AddJob(INetcodeJob job)
     {
         ArgumentNullException.ThrowIfNull(job);
         return ConfigureServices(services => services.Jobs.Add(job));
     }
 
-    /// <summary>
-    ///     Add new custom job
-    /// </summary>
-    /// <seealso cref="ServicesConfig{TInput}" />
+    /// <inheritdoc cref="AddJob(INetcodeJob)" />
     [MemberNotNull(nameof(sessionServices))]
     public NetcodeSessionBuilder<TInput> AddJob<TPlugin>() where TPlugin : INetcodeJob, new() =>
         AddJob(new TPlugin());
@@ -700,9 +698,9 @@ public InputTypeSelected<T> Integer<T>(bool isUnsigned) where T : unmanaged, IBi
             new(e => IntegerBinarySerializer.Create<T>(isUnsigned, e));
 
         /// <summary>
-        ///     Choose a raw unmanaged value type as input type.
+        ///     Choose a raw unmanaged value type as an input type.
         ///     Must not be a reference type or a value type that contains references.
-        ///     This *DO NOT* use custom <see cref="Endianness" /> for <typeparamref name="T" /> integer fields.
+        ///     This *DOES NOT* use custom <see cref="Endianness" /> for <typeparamref name="T" /> integer fields.
         /// </summary>
         /// <seealso cref="RuntimeHelpers.IsReferenceOrContainsReferences{T}" />
         public InputTypeSelected<T> Struct<T>() where T : unmanaged =>

src/Backdash/Synchronizing/State/IStateStringParser.cs

@@ -49,20 +49,21 @@ public sealed class JsonStateStringParser(
 ) : IStateStringParser
 {
     internal Logger? Logger = null;
-
-    readonly JsonSerializerOptions jsonOptions = options ?? new()
-    {
-        WriteIndented = true,
-        IncludeFields = true,
-        AllowTrailingCommas = true,
-        ReferenceHandler = ReferenceHandler.IgnoreCycles,
-        Encoder = JavaScriptEncoder.UnsafeRelaxedJsonEscaping,
-        Converters = { new JsonStringEnumConverter() },
-    };
-
+    readonly JsonSerializerOptions jsonOptions = options ?? CreateDefaultJsonOptions();
     readonly IStateStringParser fallback = stateStringFallback ?? new DefaultStateStringParser();
     readonly HexStateStringParser nullFallback = new();
 
+    internal static JsonSerializerOptions CreateDefaultJsonOptions() =>
+        new()
+        {
+            WriteIndented = true,
+            IncludeFields = true,
+            AllowTrailingCommas = true,
+            ReferenceHandler = ReferenceHandler.IgnoreCycles,
+            Encoder = JavaScriptEncoder.UnsafeRelaxedJsonEscaping,
+            Converters = { new JsonStringEnumConverter() },
+        };
+
     /// <inheritdoc />
     public string GetStateString(in Frame frame, ref readonly BinaryBufferReader reader, object? currentState)
     {

tests/Backdash.Tests/Usings.cs

@@ -6,6 +6,6 @@
 global using FluentAssertions;
 global using FsCheck;
 global using Xunit;
-global using Max = Backdash.Core.NetcodeConstants.Max;
 global using GameInput = Backdash.Synchronizing.Input.GameInput<Backdash.Tests.TestUtils.TestInput>;
+global using Max = Backdash.Core.NetcodeConstants.Max;
 global using Random = System.Random;