Bug: limit slot name when generating, and give appropriate error (Currently ' ' generates)

[jacobmix]

What happened?

Currently you can generate with player yamls containing: name: ' '
Probably not a good thing. Maybe throw an error if Slot/Player Name doesn't pass a check or something.
Discord convo.

What were the expected results?

name: ' ' would throw an error saying yaml player name is not allowed/needs to change.

Software

Local generation

[jacobmix]

It seems there's not many restrictions when making slot names.
Archipelago just deletes trailing/leading spaces, and removes any character after 16.
new_name.strip()[:16].strip()
Would probably be good to change rules such you need: at least one non-whitepace character, and not allow:[, ], {, }, ".

Note: if your name uses ' enclose the name with ". Example: name: "Player's Name".

Also maybe give a warning if player name change? Currently if you use more than 16 characters Archipelago just cuts it off.
Granted it's only an issue for the one not following the 16 limit as stated in the yaml. But will error if similar names break 16.