Skip to content

Conversation

@itzlambda
Copy link
Owner

Changes Made

  • Removed retry logic:

    • Deleted RetryPolicy enum and all retry-related configuration
    • Removed exponential backoff, fixed delay, and API-guided retry strategies
    • Cleaned up retry-related methods from ProviderConfig trait
  • Removed circuit breaker functionality:

    • Eliminated circuit breaker middleware from the stack
    • Removed circuit breaker configuration options
  • Simplified middleware stack to focus on:

    • Rate limiting
    • Timeouts
    • Connection pooling
    • Logging and metrics
  • Updated dependencies:

    • Removed retry feature from Tower in Cargo.toml
  • Documentation updates:

    • Updated CLAUDE.md to reflect simplified middleware architecture
    • Removed references to retry/circuit breaker from all example documentation
    • Deleted retry_policy_example.rs
    • Simplified middleware_usage.rs example

- Remove RetryPolicy configuration and related retry logic
- Remove circuit breaker functionality from middleware stack
- Simplify middleware to focus on core features: rate limiting, timeouts, connection pooling, and metrics
- Update Cargo.toml to remove retry Tower feature
- Update CLAUDE.md to reflect simplified middleware architecture
- Remove retry_policy_example.rs and simplify middleware_usage.rs example
- Update all example documentation to remove references to retry/circuit breaker features

BREAKING CHANGE: Retry and circuit breaker middleware has been removed. Applications relying on these features will need to implement them at the application level or use external libraries.
@itzlambda itzlambda marked this pull request as ready for review October 5, 2025 12:41
@itzlambda itzlambda merged commit 33265df into main Oct 5, 2025
5 checks passed
@itzlambda itzlambda deleted the refactor/simplify-middleware branch October 5, 2025 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant