lmarkus/hubot-conversation

Don't use robot.hear /.*/

Closed this issue · 0 comments

Awesome work!

"match everything" listeners (robot.hear /.*/) are an anti-pattern and should be replaced with middleware. In this case, I think receive middleware is the right solution, but if not, let's talk about what the ideal interface would look like, and I'll get it built into the core.

Filing this now so I don't forget; I'll do some more noodling soon and give more detailed suggestions.