Skip to content

Conversation

@flemzord
Copy link
Member

Supply database connection options to the Fx container to resolve a Circuit Breaker module dependency.

The Circuit Breaker storage module required *bunconnect.ConnectionOptions as an Fx dependency to create its database connection, but these options were not being supplied to the Fx container.


Open in Cursor Open in Web

…dency injection

The Circuit Breaker module in go-libs requires *bunconnect.ConnectionOptions as a dependency. This was not being provided to the fx container, causing dependency injection failures. Added fx.Supply(connectionOptions) to make these options available to the Circuit Breaker module.
@cursor
Copy link

cursor bot commented Dec 29, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 29, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • main

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cursor/circuit-breaker-db-connection-5a8e

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

@flemzord flemzord changed the title Circuit breaker db connection fix: provide database connection options to fx container for circuit breaker Dec 29, 2025
@flemzord flemzord marked this pull request as ready for review December 29, 2025 14:59
@flemzord flemzord requested a review from a team as a code owner December 29, 2025 14:59
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.

4 participants