feat: add excluded codes option from hystrix error tracking

[dandynaufaldi]

Summary by CodeRabbit

• New Features

  • Added option to exclude specific gRPC status codes from triggering Hystrix fallbacks, allowing finer-grained control over error handling.

• Documentation

  • Updated README links to reflect current source references.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds Hystrix exclusion by gRPC status code: new client option to register excluded codes and updated interceptor logic that detects gRPC status errors and skips Hystrix fallback when the error's code matches an excluded code.

Changes

Cohort / File(s)	Summary
Documentation README.md	Updated line-numbered links to match current source file positions; purely presentational.
Hystrix status exclusion options.go, interceptors.go	Added excludedCodes []codes.Code to clientOptions and new public constructor WithHystrixExcludedCodes(grpcCodes ...codes.Code). Interceptor now parses gRPC status errors (google.golang.org/grpc/status) and, if the status code is in excludedCodes, returns nil to bypass Hystrix fallback; otherwise returns the original error.

Sequence Diagram(s)

sequenceDiagram
    participant Client
    participant Hystrix
    participant Interceptor
    participant Invoker
    participant gRPC_Status

    Client->>Hystrix: invoke command
    Hystrix->>Interceptor: call invoker wrapper
    Interceptor->>Invoker: invoke RPC
    Invoker-->>Interceptor: returns error
    Interceptor->>gRPC_Status: parse error -> status.Code
    alt code in excludedCodes
        Interceptor-->>Hystrix: return nil (skip Hystrix fallback)
    else not excluded
        Interceptor-->>Hystrix: return original error (Hystrix handles)
    end
    Hystrix-->>Client: final response/error

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat: add excluded error option from hystrix error tracking #11: Implements a related Hystrix exclusion mechanism (filters by error values); strongly related in purpose and area of the code.

Poem

🐰✨ I nibble on code beneath the moonlight,
Status codes chosen, some take flight,
Hystrix pausing for errors we name,
Quiet circuits, gentler game. 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main feature being added: support for excluding gRPC status codes from Hystrix error handling. It matches the core functionality changes across all modified files.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Note

Docstrings generation - SUCCESS
Generated docstrings for this pull request at #14

[Copilot]

Pull request overview

This PR adds functionality to exclude specific gRPC status codes from triggering Hystrix circuit breaker fallback, providing more granular control over error handling behavior.

Changes:

• Added a new WithHystrixExcludedCodes option function to allow specifying gRPC status codes that should be excluded from Hystrix error tracking
• Extended the Hystrix client interceptor to check for excluded gRPC status codes in addition to the existing excluded errors functionality
• Updated auto-generated README documentation to reflect line number changes in the source code

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
options.go	Added excludedCodes field to clientOptions struct and new WithHystrixExcludedCodes function to configure excluded gRPC status codes
interceptors.go	Enhanced HystrixClientInterceptor to check if error status code matches any excluded codes and skip Hystrix circuit breaker tracking for those codes
README.md	Updated auto-generated documentation links to reflect new line numbers after code additions

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.