GeyserMC
Introduce sendCommand API method
Looks good to me. Only question I'd have - do we want to allow running internal Geyser commands (e.g. geyser achievements / geyser ping / extension commands) Om platforms such as standalone, before sending this off to the Java server?
If not, we should mark the method that it'll always send a command directly towards the Java server.
I think adding another method with a boolean to decide whether or not it will pass through Geyser's commands would make sense, then this default method would just do that and ensure it passes through Geyser's commands, so we provide some options
default void sendCommand(String command) {
this.sendCommand(command, true);
}
void sendCommand(String command, boolean includeGeyserCommands);
If that makes sense
Not really a fan of adding that - it'd server no use on all platforms except standalone/viaproxy. I'd rather make that default behavior; similar how we handle command requests from the Bedrock side currently
Makes sense, just thought if we wanted both way but yeah, lemme just update it to allow Geyser and extension commands to run too