cleanup of /cmdlist and practice commands

[BlaiZephyr]

see https://github.com/ddnet/ddnet/pull/8356#issuecomment-2106043277

i want to bring this up again since i dont expect any response on a small and soon to be closed PR.

we currently have /un{weapon} and /removeweapon {weapon-id}.

from my previous PR comment:

can't we just remove the un{weapon} thingies and do it like this: first use of /shotgun gives the weapon second use of /shotgun removes the weapon Commands Like deep shotgun etc. should just toggle between on and off, makes binding those easier aswell.

if we dont want to change their entire logic we should stick to un{state} or remove{state} commands and not have two options that do the same thing with different syntax.