improve kick message for players without reactor

[whichtwix]

Currently players get confused when they see reactor as the reason for their friends getting kicked from lobbies, at least due to not knowing what reactor is. The pr changes the message to indicate they don't have mods required by all players.

[js6pak]

Thanks for the PR, but it has a few issues.
Firstly officials will "ban" you for either bypassing the chat cooldown or the message length limit.
And because you manually create the modsList string instead just using string.Join you will always end up with a comma at the end.

[js6pak]

[Pietrodjaowjao]

and ALSO illegal string

[Pietrodjaowjao]

reactor should have a custom rpc for this instead

[js6pak]

and ALSO illegal string

New lines are fine for now at least, we can change it to a comma later if needed.

reactor should have a custom rpc for this instead

This whole kicking mechanic is only used when both the server doesn't have Reactor handshake support (so on officials) and the joining player doesn't have Reactor installed at all. In any other case they will be kicked either by the server¹ or the host with a custom GameData message².

Footnotes

1. https://github.com/NuclearPowered/Reactor.Impostor/blob/62b1e75710681df716c84606c807d8f93584180f/Reactor.Impostor/HandshakeEventListener.cs#L88 ↩

2. https://github.com/NuclearPowered/Reactor/blob/21c0e61f57ec40677ec769167191f53336367453/Reactor/Networking/Patches/ClientPatches.cs#L141 ↩