Skip to content

RoutableMessageBus#20

Closed
f3l1x wants to merge 2 commits intomasterfrom
claude/add-failing-test-messenger-mCQqc
Closed

RoutableMessageBus#20
f3l1x wants to merge 2 commits intomasterfrom
claude/add-failing-test-messenger-mCQqc

Conversation

@f3l1x
Copy link
Copy Markdown
Member

@f3l1x f3l1x commented Dec 26, 2025

No description provided.

When dispatching a message to eventBus (allowNoHandlers: true) via sync
transport, SyncTransport incorrectly uses messageBus middleware instead of
respecting the BusNameStamp. This causes NoHandlerForMessageException even
though eventBus allows no handlers.
Make RoutableMessageBus the default autowired MessageBusInterface instead
of the first registered bus. This ensures that when SyncTransport dispatches
messages, it respects the BusNameStamp and routes to the correct bus with
its specific middleware configuration.

Changes:
- Individual buses are no longer autowired for MessageBusInterface
- RoutableMessageBus is now the autowired MessageBusInterface
- Added fallback bus (first registered bus) for messages without BusNameStamp
@f3l1x
Copy link
Copy Markdown
Member Author

f3l1x commented Mar 9, 2026

Resolved in master

@f3l1x f3l1x closed this Mar 9, 2026
@f3l1x f3l1x deleted the claude/add-failing-test-messenger-mCQqc branch March 9, 2026 20:55
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.

2 participants