Paper

Paper copied to clipboard

Fixes #6441 (kinda)

So it was a simpler fix that I originally thought. blocked because it requires part of #5527 (specifically moving back datapack parsing)

Just a note that has to be resolved, this is going to break a lot of datapacks as they currently are. Since the /minecraft: command prefix is not a thing in vanilla, they all don't have it. And some plugins (like essentials) override some of those commands. Which means they are going to break.

I think the solution is, that pursuant to a boolean config option, when parsing command functions, when trying to find LiteralCommandNodes, we need to do an additional check if one with the minecraft: prefix exists and prioritize that.

Do we still want an early fix for this? My brig api will resolve this issue by properly registering commands into the dispatcher.

Closed due to being handled by brig api.