Since the original branch was very out-of-date, here's a new branch with the same initial changes but with some refactoring on top of that:
Cleaned up the Socket object. The main difference is that it uses an EventEmitter to signal events (open, close, and message), rather than overriding existing functions.
The gameplay/chat specialisations have been removed in favour of event callbacks. This means we also no longer have that circular dependency problem!
The Socket instances for the chat and gameplay are now managed by the game and DiscordManager modules.
Cleaned up DiscordManager, mainly regarding private variables and the db parameter.
The user auth check is now part of the chat socket logic, rather than the gameplay one.
Added init/stop function for game to make the relevant code a bit cleaner, though game still needs work as a whole.
Since the original branch was very out-of-date, here's a new branch with the same initial changes but with some refactoring on top of that:
Socketobject. The main difference is that it uses anEventEmitterto signal events (open,close, andmessage), rather than overriding existing functions.gameplay/chatspecialisations have been removed in favour of event callbacks. This means we also no longer have that circular dependency problem!Socketinstances for the chat and gameplay are now managed by thegameandDiscordManagermodules.DiscordManager, mainly regarding private variables and thedbparameter.gameto make the relevant code a bit cleaner, thoughgamestill needs work as a whole.Referencing issues Closes #200