discord-bot/CompatBot
2019-10-24 11:52:22 -03:00
..
Commands further fixes to answers 2019-10-24 11:52:22 -03:00
Database restore balance 2019-10-02 23:09:11 +05:00
EventHandlers command to restore update notification message when it was nuked accidentally 2019-10-14 16:06:28 +05:00
Properties better azure pipelines integration and building docker image 2019-09-19 23:32:05 +05:00
ThumbScrapper fix psn scraper by hardcoding the known psn stores 2019-06-03 17:46:55 +05:00
Utils add check for ppu threads 2019-10-14 02:08:41 +05:00
CompatBot.csproj try to fix crash on linux 2019-09-25 13:46:58 +05:00
Config.cs try to fix log path issues 2019-09-23 20:10:43 +05:00
GlobalSuppressions.cs fix compiler warnings 2019-03-01 15:56:54 +05:00
Program.cs better azure pipelines integration and building docker image 2019-09-19 23:32:05 +05:00
readme.md More typo fixes 2019-04-10 03:41:41 +05:30
Watchdog.cs this is not useful anymore 2019-04-28 23:10:11 +05:00

RPCS3 Compatibility Bot

Configuration

Currently all configurable tunables are stored in the Config static class that is initialized once on startup. This is mostly done like this because it's very easy to implement and is enough for current needs.

Some settings are grouped in additional static classes for easier use (e.g. Reactions and Colors).

Everything is initialized with default values that correspond to the main bot instance (except for sensitive tokens), and require overriding through configuration for any test instance.

Currently, configuration is possible through the $ dotnet user-secrets command. Configuration through the environment variables was disabled as it had some unintended consequences between bot restarts (preserved values; require complete manual shutdown and restart to update configuration).

Be careful with input validation during configuration, as unhandled exceptions in static constructor will lead to TypeInitializationException and program termination that might be tricky to debug.

In addition to the configuration variables, Config contains the Log instance and the global CancellationTokenSource.

We're using NLog for logging, configured to mimic the default Log4Net layout (mostly because I already have the syntax highlighter for Sublime Text 3 for it). It is also configured to ignore TaskCancelledExceptions that occur when CancellationTokens are being cancelled to reduce spam in logs.

Do log exceptions as an argument to log methods, instead of calling .ToString() on them

Global CancellationTokenSource (Config.Cts) is used to signal the program shutdown.

Do check it and abort whenever possible to reduce the restart time and prevent infinite code execution.

Program entry point

On startup we check for other instances. We wait a bit for their shutdown, or shutdown ourselves otherwise. This is done through spinning a separate thread and using global mutex. Unlike semaphores, mutex must be released in the same thread it was acquired, which is impossible to do in asynchronous code, thus the dedicated thread.

Note that Config initialization will happen on first call to the class.

Next we open the databases and run migrations to upgrade their structure if needed. Currently we have two of them:

  • BotDb is used to store all the settings and custom data for the bot.
  • ThumbsDb is used to store PSN metadata and game thumbnail links.

When databases are ready, we immediately restore bot runtime statistics.

Next we start all the background tasks that will run periodically while the bot is up and running. This includes thumbnail scraping, AppVeyor build history scraper, AMD Driver version updater, etc.

Next we configure the discord client. This includes registering all the commands and event handlers. We use the built-in help formatter and hook up the built-in discord client logging to our own logger.

Of particular note is the GuildAvailable event, where we check and make sure the bot is running in the configured guild. There was one time when bot wasn't configured properly and someone quietly added it to their own private server, which caused crash on startup. That was fun.

We also fun backlog checks where it makes sense for moderation, in case something slipped past while the bot was unavailable.

On restart we try to gracefully wait for a bit to let any outstanding task to complete, but not too long. This is why it is important to check Config.Cts cancellation status whenever possible.