Re-add logging for commands

src/app/commands/addroles.command.ts

@@ -65,7 +65,8 @@ const command = {
 
         interaction.followUp(`Successfully added: ${role.name}`);
       })
-      .catch(() => {
+      .catch((e) => {
+        container.loggerService.error(`Error adding role: ${e}`);
         interaction.followUp(`Failed to add role: ${role.name}`);
       });
   },

src/app/commands/delroles.command.ts

@@ -15,7 +15,7 @@ const command = {
     },
   ],
 
-  async execute({ interaction }) {
+  async execute({ interaction, container }) {
     const member = interaction.member as GuildMember;
     if (!member) {
       await interaction.reply('Could not resolve you to a member');
@@ -39,7 +39,8 @@ const command = {
       .then(() => {
         interaction.followUp(`Successfully removed: ${role.name}`);
       })
-      .catch(() => {
+      .catch((e) => {
+        container.loggerService.error(`Error removing role: ${e}`);
         interaction.followUp(`Failed to remove role: ${role.name}`);
       });
   },

src/app/listener.ts

@@ -9,8 +9,8 @@ import {
 } from 'discord.js';
 import Constants from '../common/constants';
 import { Handler } from '../common/handler';
-import { commands, ICommand } from '../common/slash';
-import { IContainer, IHandler, IMessage, Mode } from '../common/types';
+import { ICommand, commands } from '../common/slash';
+import { IContainer, IHandler, IMessage, IPluginEvent, Mode } from '../common/types';
 
 export class Listener {
   constructor(public container: IContainer) {
@@ -93,12 +93,32 @@ export class Listener {
 
       const command = commands.get(interaction.commandName) as ICommand | undefined;
 
-      // For future proofing against any new command types we verify the command type
-      // being run is intentional.
-      command?.execute({
-        interaction: interaction,
-        container: this.container,
-      });
+      const pEvent: IPluginEvent = {
+        status: 'starting',
+        pluginName: interaction.commandName,
+        args: interaction.options.data,
+        user: interaction.user.id,
+      };
+
+      try {
+        this.container.loggerService.info(JSON.stringify(pEvent));
+
+        // For future proofing against any new command types we verify the command type
+        // being run is intentional.
+        await command?.execute({
+          interaction: interaction,
+          container: this.container,
+        });
+
+        pEvent.status = 'fulfillCommand';
+        this.container.loggerService.info(JSON.stringify(pEvent));
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      } catch (e: any) {
+        pEvent.status = 'error';
+        pEvent.error = e.message as string;
+        pEvent.stack = e.stack as string;
+        this.container.loggerService.error(JSON.stringify(pEvent));
+      }
     });
 
     this.container.clientService.on('messageCreate', async (message: IMessage) => {

src/common/types.ts

@@ -175,13 +175,13 @@ export enum RoleType {
 }
 
 export interface IEvent {
-  status: string;
+  status: 'starting' | 'fulfillCommand' | 'error' | 'fulfillJob';
   error?: string;
   stack?: string;
 }
 
 export interface IPluginEvent extends IEvent {
-  args: string[];
+  args: readonly discord.CommandInteractionOption<discord.CacheType>[];
   user: string;
   pluginName: string;
 }