Reworking Command Framework

[woojiahao]

Current command framework might bug if the command is overloaded with a sentence at the end:

Simulation

1. Overload strike command to not receive the Integer() input
2. This should default the strike to 1 with the reason
3. Call the strike command
4. With the current strike system, it should throw an exception that the first word in the sentence is not a valid input, this is not meant to happen

Fix

• Parsing of arguments should be done for each set of potential argument to command. The latest that matches will be returned
• Commands should be sorted by number of argument types

[woojiahao]

Not a possible fix given the current approach towards parsing arguments, will be closing this for now, might re-open if there is a better approach towards creating and parsing commands.