Merge pull request #57 from InfinityGhost/log-command-exceptions

Log exceptions from commands and respond appropriately

Author: InfinityGhost

TabletBot.Discord/LogExtensions.cs

@@ -1,4 +1,5 @@
-using System.Threading.Tasks;
+using System;
+using System.Threading.Tasks;
 using Discord;
 using TabletBot.Common;
 using LogMessage = Discord.LogMessage;

TabletBot.Discord/Watchers/Commands/CommandMessageWatcher.cs

@@ -38,8 +38,26 @@ public async Task Receive(IMessage message)
         {
             if (message.Content.StartsWith(_settings.CommandPrefix))
             {
-                var context = new CommandContext(_discordClient, message as IUserMessage);
-                await _commandService.ExecuteAsync(context, 1, _serviceProvider).ConfigureAwait(false);
+                try
+                {
+                    var context = new CommandContext(_discordClient, message as IUserMessage);
+                    await _commandService.ExecuteAsync(context, 1, _serviceProvider).ConfigureAwait(false);
+                }
+                catch (Exception e)
+                {
+                    Log.Exception(e);
+                    try
+                    {
+                        string exMessage = e.GetType().FullName + ": " + e.Message;
+                        await (message as IUserMessage).ReplyAsync(exMessage);
+                    }
+                    catch (Exception respondEx)
+                    {
+                        Log.Exception(respondEx);
+                    }
+
+                    throw;
+                }
             }
         }
 

TabletBot.Discord/Watchers/Commands/SlashCommandInteractionWatcher.cs

@@ -31,7 +31,32 @@ IEnumerable<SlashCommandModule> commands
 
         public async Task HandleInteraction(SocketInteraction interaction)
         {
-            await Task.WhenAll(_commands.Select(m => m.HandleInteraction(interaction)));
+            try
+            {
+                await Task.WhenAll(_commands.Select(m => m.HandleInteraction(interaction)));
+            }
+            catch (Exception e)
+            {
+                Log.Exception(e);
+                try
+                {
+                    string message = e.GetType().FullName + ": " + e.Message;
+                    if (interaction.HasResponded)
+                    {
+                        await interaction.FollowupAsync(message, ephemeral: true);
+                    }
+                    else
+                    {
+                        await interaction.RespondAsync(message, ephemeral: true);
+                    }
+                }
+                catch (Exception responseEx)
+                {
+                    Log.Exception(responseEx);
+                }
+
+                throw;
+            }
         }
 
         public async Task InitializeAsync()