Command support for mob types

[Colusite]

Feature request

Short description

We should have a way to do add commands to more places, like after the monster value

Implementation details

The ability to add commands to more places would allow further customization and remove the need for MobArena Extension and support for countless more plugins.
e.g
Boss1:
monster: /mm mobs spawn SlimeBoss
name: Steven
health: 69
abilities: fire-aura, /smite @p, /effect give @e[type=minecraft:slime] regeneration 10 1

This should function the same way as the command(s) support for rewards/upgrade waves
e.g
Upgrade1:
wave: 10
all: /op

[garbagemule]

The intent behind this suggestion is great; hooking into other plugins via commands (or other mechanisms that don't require a direct integration with said plugins) is a very effective path to flexibility and customization... if it works.

From a technical perspective, however, there aren't really a lot of places where we can realistically implement something like this unless the command itself has a well-defined, accessible "target". For example, commands as rewards are fairly simple, because the commands usually have a "player target" in the form of a player name, which MobArena can fill in for you if you use the <player> variable in the command expression.

However, when commands don't have a well-defined, accessible target, there is no way for MobArena to know what happened, because running a command doesn't return anything to the caller. The best way to demonstrate the issue is with an absurd example: what happens if you swap the /mm mobs spawn SlimeBoss out with /say Hello? It's important to understand that these two commands are the same from MobArena's perspective. There is literally no difference between them, because they are just "fire and forget". Running a command doesn't result in a monster reference that MobArena can track.

Boss abilities are currently static constructs that must be registered in MobArena's ability manager. They are parsed by keywords, and they would need to be reworked completely to support variables and such. But even if we could introduce commands into the mix, we would need a means of also providing a targeting mechanism. What would <player> mean in a boss ability? If it's pull-distant, for example, it's not one player but potentially several, or none.

There are probably ways around some of these issues, but I don't see a way to fit any of this into the current code base. At the very least, it has to be lumped in with the Mobs Rework effort.