Hmm, I just noticed that .removeprefix requires py3.9, maybe I should do this some other way.

On the other hand, errbotio errbot docker image uses 3.9 though, so it may be fine?

Are you to elaborate on the configuration your using for errbot + err-stackstorm as well as the stackstorm action-alias definition that is being stripped?

err-stackstorm isn't constrained to running in a docker image and used in production on server running python3.8.

We have modifications that allow for chatops commands to be made without the bot prefix, which is why the text that matches the bot prefix may or may not be the bot prefix itself and could be part of the rest of the command, such that:

bot prefix: mrbot

with prefix:

command "mrbot please do something for mrbot" -> removes first instance of "mrbot" and leaves "please do something for mrbot"

with our modifications to not require the prefix:

command: "please do something for mrbot" -> removes first instance of "mrbot" and leaves "please do something for"

But, unless you are using non-prefixed commands, this isn't something that would come up. And without a guarantee of py3.9, this change would be more complicated and more ugly. So, maybe picking up this change would not be worthwhile.

I'm reluctant to integrate code changes to accommodate a modified version of the plugin. That said, the fix looks reasonable for the actual use cases supported by the plugin. If you want to leave this open, I'll merge it after py3.8 goes EOL.

Will do. Thanks!

The day has finally come, py3.8 support is dropped. Thanks for the PR.