Adding Foundatio builders for configuring in IServiceCollection. Various other improvements.

[ejsmith]

// configure Foundatio services
builder.Services.AddFoundatio()
    .Storage.UseFolder()
    .Caching.UseInMemory()
    .Locking.UseCache()
    .Messaging.UseInMemory()
    .Queueing.UseInMemory<SampleWork>()
    .AddResilience(b => b.WithPolicy<Sample1Job>(p => p.WithMaxAttempts(5).WithLinearDelay().WithJitter()));


// override services
builder.Services.AddFoundatio()
  .Caching.UseRedis()
  .Messaging.UseRedis();

This pull request introduces resilience policies into the Sample1Job, enhances service configuration in the hosting sample, refines startup action methods, and adds utility methods for replacing singleton services. The changes improve job reliability, simplify service management, and enhance extensibility.

Resilience and Job Enhancements:

• samples/Foundatio.HostingSample/Jobs/Sample1Job.cs: Integrated resilience policies using IResiliencePolicy to handle retries and simulate failures in Sample1Job. This ensures the job can recover from transient errors.

Service Configuration Improvements:

• samples/Foundatio.HostingSample/Program.cs: Added Foundatio services configuration to streamline caching, messaging, locking, and resilience policies. Redis is used for distributed caching and messaging, replacing in-memory implementations. [1] [2]

Startup Action Refinements:

• src/Foundatio.Extensions.Hosting/Startup/StartupExtensions.cs: Updated AddStartupAction methods to return IServiceCollection, improving method chaining and consistency. Added null-coalescing assignments for health check tags and predicates to simplify code. [1] [2] [3] [4]

Utility Methods for Service Replacement:

• src/Foundatio/Extensions/ServicesExtensions.cs: Introduced ReplaceSingleton methods to replace existing service descriptors with new singleton services, enhancing service configurability.

Minor Fixes:

• src/Foundatio.TestHarness/Jobs/JobQueueTestsBase.cs and src/Foundatio.TestHarness/Jobs/ThrottledJob.cs: Removed redundant parameters in ThrottlingLockProvider instantiation, simplifying code. [1] [2]

[Copilot]

Pull Request Overview

This PR enhances Foundatio’s configuration and extensibility by introducing fluent builder APIs, integrating resilience policies, refining startup actions, and adding utilities for replacing singleton services.

• Adds FoundatioBuilder extensions for configuring caching, storage, messaging, locking, resilience, and serialization via IServiceCollection.
• Integrates IResiliencePolicyProvider into Sample1Job and refactors ResiliencePolicyProvider defaults and builder methods.
• Refactors SystemTextJsonSerializer to support custom JsonSerializerOptions and updates related tests.
• Updates startup action extension methods to return IServiceCollection for improved chaining.

Reviewed Changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.

File	Description
tests/Foundatio.Tests/Serializer/SystemTextJsonSerializerTests.cs	Added option-based serializer tests and a benchmark stub.
src/Foundatio/Serializer/SystemTextJsonSerializer.cs	Refactored default serialization/deserialization option logic.
src/Foundatio/Resilience/ResiliencePolicyProvider.cs	Changed default policy initialization and optional builders.
samples/Foundatio.HostingSample/Program.cs	Updated sample to use FoundatioBuilder and resilience APIs.

Comments suppressed due to low confidence (3)

src/Foundatio/Resilience/ResiliencePolicyProvider.cs:24

• The default policy no longer sets a non-zero MaxAttempts (previously 5), changing retry behavior. Consider restoring MaxAttempts = 5 or making it configurable to avoid unexpected single-attempt defaults.

        _defaultPolicy = new ResiliencePolicy(_loggerFactory.CreateLogger<ResiliencePolicy>(), _timeProvider);

samples/Foundatio.HostingSample/Program.cs:218

• [nitpick] This second AddFoundatio() call inside ConfigureServices duplicates the builder registration from the top of Program.cs. Consider reusing the existing FoundatioBuilder instance instead of registering twice.

    builder.Services.AddFoundatio()

src/Foundatio/Serializer/SystemTextJsonSerializer.cs:18

• [nitpick] Falling back to serializeOptions for deserialization may omit converters (e.g., type-inference logic). Consider always merging user-supplied options with _defaultDeserializeOptions to preserve required converters.

        _deserializeOptions = deserializeOptions ?? serializeOptions ?? _defaultDeserializeOptions;

[niemyjski]

One question that I had is if you call AddQueueing(s => s.AddResilency(...options)) And then you could also override it like this. AddQueueing().UseAzureStorageQueue().UseSQSQueue(f => f.AddResilency(...options)) (example would work better on repos or queues. qon

[ejsmith]

I don't know what you are asking. Yes, we currently have a few different ways to configure things. This PR is trying to reduce those with the builders.