!addcommand/!removecommand shouldn't work on built-ins
Closed this issue · 0 comments
r4v5 commented
Let's say some hapless fool calls !addcommand Help this streamer do great things!
because they forgot the command they wanted to define. This text then shadows !help for the lifetime of the bot's process, and also affects aliased commands like !halp
. !removecommand help
manages to remove the errant command but doesn't actually restore !help
's default behavior, so it actually removes the command (and any aliases like !halp
). (This also extends to !removecommand removecommand
, so, uh, be careful.)
We probably should detect when !addcommand or !removecommand would collide with a built-in and produce a helpful "I'm afraid I can't let you do that" type message.