Exclude feature-specific exceptions from resilience policy retries

- MessageBusBase: Exclude MessageBusException from retries

- CacheLockProvider: Exclude CacheException from retries

- ResiliencePolicy: Add BrokenCircuitException to default unhandled exceptions

- Update resilience documentation with new defaults

These exceptions represent deliberate application-level failures that should not be retried by the resilience policy. Retrying them would be wasteful and could mask the underlying issue.

Author: niemyjski

docs/guide/resilience.md

@@ -151,12 +151,14 @@ var policy = new ResiliencePolicyBuilder()
 
 ### Unhandled Exceptions
 
+By default, `OperationCanceledException` and `BrokenCircuitException` are never retried. You can add additional exception types:
+
 ```csharp
 var policy = new ResiliencePolicyBuilder()
     .WithMaxAttempts(5)
     // These exceptions will be thrown immediately without retry
-    .WithUnhandledException<OperationCanceledException>()
     .WithUnhandledException<ArgumentException>()
+    .WithUnhandledException<InvalidOperationException>()
     .Build();
 ```
 
@@ -581,6 +583,12 @@ public class MyService
 .WithUnhandledException<ArgumentException>()
 ```
 
+> **Note:** By default, `OperationCanceledException` and `BrokenCircuitException` are never retried. Additionally, Foundatio's built-in components automatically exclude their feature-specific exceptions from retries:
+> - `MessageBusBase` excludes `MessageBusException`
+> - `CacheLockProvider` excludes `CacheException`
+>
+> This ensures that deliberate application-level failures (like a blocked RabbitMQ connection or a cache operation error) are not wastefully retried.
+
 ### 5. Log Retry Attempts
 
 ```csharp

src/Foundatio/Lock/CacheLockProvider.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Concurrent;
 using System.Diagnostics;
 using System.Diagnostics.Metrics;
@@ -41,7 +41,9 @@ public CacheLockProvider(ICacheClient cacheClient, IMessageBus messageBus, TimeP
         _messageBus = messageBus;
 
         _resiliencePolicyProvider = resiliencePolicyProvider ?? cacheClient.GetResiliencePolicyProvider();
-        _resiliencePolicy = _resiliencePolicyProvider.GetPolicy<CacheLockProvider, ILockProvider>(_logger, _timeProvider);
+        _resiliencePolicy = _resiliencePolicyProvider.GetPolicy<CacheLockProvider, ILockProvider>(
+            builder => builder.WithUnhandledException<CacheException>(),
+            _logger, _timeProvider);
 
         _lockWaitTimeHistogram = FoundatioDiagnostics.Meter.CreateHistogram<double>("foundatio.lock.wait.time", description: "Time waiting for locks", unit: "ms");
         _lockTimeoutCounter = FoundatioDiagnostics.Meter.CreateCounter<int>("foundatio.lock.failed", description: "Number of failed attempts to acquire a lock");

src/Foundatio/Messaging/MessageBusBase.cs

@@ -35,7 +35,9 @@ public MessageBusBase(TOptions options)
         _timeProvider = options.TimeProvider ?? TimeProvider.System;
 
         _resiliencePolicyProvider = options.ResiliencePolicyProvider;
-        _resiliencePolicy = _resiliencePolicyProvider.GetPolicy<MessageBusBase<TOptions>, IMessageBus>(_logger, _timeProvider);
+        _resiliencePolicy = _resiliencePolicyProvider.GetPolicy<MessageBusBase<TOptions>, IMessageBus>(
+            builder => builder.WithUnhandledException<MessageBusException>(),
+            _logger, _timeProvider);
 
         _serializer = options.Serializer ?? DefaultSerializer.Instance;
         MessageBusId = _options.Topic + Guid.NewGuid().ToString("N").Substring(10);

src/Foundatio/Resilience/ResiliencePolicy.cs

@@ -39,9 +39,9 @@ public ILogger Logger
     public int MaxAttempts { get; set; } = 3;
 
     /// <summary>
-    /// A collection of exception types that will not be handled by the policy. These exceptions will be thrown immediately without retrying. Default includes OperationCanceledException.
+    /// A collection of exception types that will not be handled by the policy. These exceptions will be thrown immediately without retrying. Default includes OperationCanceledException and BrokenCircuitException.
     /// </summary>
-    public HashSet<Type> UnhandledExceptions { get; set; } = [typeof(OperationCanceledException)];
+    public HashSet<Type> UnhandledExceptions { get; set; } = [typeof(OperationCanceledException), typeof(BrokenCircuitException)];
 
     /// <summary>
     /// A function that determines whether to retry based on the attempt number and exception.