Remove the Reply logic from Dispatcher
bobthemighty opened this issue · 1 comments
bobthemighty commented
The dispatcher is doing way too much work.
The model where a Conversation returns a Reply is nice for testability but the current design sucks.
We should consider either:
- A polymorphic Reply type that encapsulates the logic that's in handle_reply OR
- Moving the reply logic into the Conversation.
Either of these will require us to rewrite all the conversation tests.
At the end, the Dispatcher should be much simpler.
bobthemighty commented