Remove weird parsing bit that kills Regex routes.

[zachfeldman]

If I leave this in, none of my regex "match" routes seem to work. Any ideas?

[dblock]

It's not that weird. It supports matching the bot name and aliases. Why don't you explain the bug that you're seeing, and, better, write a failing test for it? Happy to help fix.

[zachfeldman]

Sure, the bug I'm seeing is when I try to use a custom matcher rather than the command syntax (match, etc).

It seems that using the existing "parser" function, the regex matching fails because the bot name is prepended to my command before it's checked against the regex.

I don't really have time to write a failing test but just thought I'd bring this to your attention! My fork is working fine for me. Good luck!

[dblock]

Can you provide an example of the custom matcher?

[zachfeldman]

Sure, here's the whole bot:
https://github.com/zachfeldman/done-bot

[dblock]

@zachfeldman Would you please try HEAD without this change? Turns out I didn't even have match specs, so now that's corrected and the parser is a little smarter. The goal for the non-regex version is to support addressing the bot, it's hacky at best, but with my fix it works in all scenarios looks like.

[zachfeldman]

HEAD works!!

[dblock]

I released 0.4.5. Thanks for bringing this up.

[zachfeldman]

Sweet! Thanks for making such a sweet client to make this easier :)

[dblock]

👍