ravenlab / commander

A platform agnostic Minecraft command framework

ravenlab

commander's Issues

Seperate command annotations

Have annotations per property I.E: Command, Permission, PermissionMessage etc

Make children inherit their parent permission message

Children should inherit their parent's permission message if explicitly defined.

Make world class resolver optional

Write javadocs

Add automatic tab complete support

Default tab complete for legacy server and brigadier support for 1.13+ servers.

Look into alternatives to stubbing Bukkit interfaces

This would be good for future compatability. This may be able to be accomplished by using bytebuddy or something similar. Mockito also has spies which may work https://github.com/mockito/mockito/wiki/Using-Spies-%28and-Fakes%29

Add velocity support

Add sponge support

Started in a773cc2

Refactor registerAlias method

Refactor registerAlias() to the Commander class. Each platform commander should provide the registered aliases. This data then can be used in the Commander class to then handle registering there.

Add enum transformer

Blocked by #17

Add command usage field

Add command usage field if a command does not execute it should return the usage back to the user if they have permission to do so.

Add automatic unregistering support

Make an interface for commanders so that a listener can be added for when a plugin is disabled so that commands can be automatically unregistered.

More transformers

• boolean

Add uuid transformer

Mock bukkit server

Currently can't mock the bukkit server due to needing to access fields via reflection. Mockito should be investigated further and if mockito doesn't have something for this then we might need to make something for it.

Refactor transformer classes pass class to transform method

Allow commands to be configured via code

Commands should be able to be configured via code. This will be useful so that if a user wants to make command usage, permissions etc a plugin could implement it.

Refactor classes into carwash project

The wrapper classes should be refactored into the carwash project. This was needed originally for DynamicGui but it was never done. This should be done for future projects to ensure a central code base.

Optimize children finding

This section may be performance heavy, this should probably be optimized in some way. Caching (possibly with a graph) may be useful. Reference code https://github.com/ravenlab/commander/blob/master/src/main/java/com/github/ravenlab/commander/command/parser/CommandParser.java#L9-L21

Rewrite test harness

The current test harness has a lot of overlap when implementing new platforms. These tests should probably be abstracted in some way, maybe junit runners? Some code won't be able to be abstracted such as testing internal components.

Support Java 15

Gradle needs to be updated

Make children inherit their parent's attributes

If not defined

• Permission
• Permission message

Support command wrapping and unwrapping

CommandMap map;

Command original = map.get("test");
if(original != null) {
	map.put("test", new Command() {
		public void command(sender, command, label, args) {
			if(args.length == 0) {
				//execute registered command
			} else {
				original.command(sender, command, label, args);
			}
		}
	});
} else {
	//Just register regular command
}

If a command is wrapped on unregister it should unwrap.

Automatic brigadier support