Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bot "thanks" regex is too permissive #371

Open
taboege opened this issue Dec 4, 2019 · 2 comments
Open

Bot "thanks" regex is too permissive #371

taboege opened this issue Dec 4, 2019 · 2 comments
Labels
notable Issues related to Notable bot

Comments

@taboege
Copy link
Member

taboege commented Dec 4, 2019

People can thank the whateverable bots, but the regex detecting this is so lax that, for instance, notable swallowed a weekly note today with thanks for someone else:

<AlexDaniel> weekly: Thanks to unclechu++ for a reminder, there's now [...]
<notable6> AlexDaniel, You're welcome!
@vrurg
Copy link

vrurg commented Dec 4, 2019

@AlexDaniel perhaps it'd make sense either disable the feature (through I like it a lot), or reduce it to support bare Thanks, Thank you, etc. with not continuation.

@AlexDaniel AlexDaniel added the notable Issues related to Notable bot label Dec 4, 2019
@AlexDaniel
Copy link
Member

Ha-ha! I didn't even notice it 😂

The assumption was that most bots don't take free text, and it's unlikely that your code or command will start with “Thanks”. But there's a notable exception… 🤣

I think a simple addition of <!before ‘to’> can resolve this issue. Whateverable bots are very lax and try to do what you mean in many cases, and that's by design. Of course there are edge cases when a message can trick the bots to do something else, but they're more of conversationalists than precise tooling. Anyway, I keep track of these situations here: https://github.com/perl6/whateverable/wiki/Humor

So yeah, I'd prefer to improve the heuristic instead of removing the easter egg/feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notable Issues related to Notable bot
Projects
None yet
Development

No branches or pull requests

3 participants