refactor: implement command class structure for better organization and error handling; update existing commands to extend from the new base class

bot.js

@@ -53,18 +53,24 @@ client.on(Events.InteractionCreate, async interaction => {
   try {
     await command.execute(interaction);
   } catch (error) {
-    logger.error(error);
+    // Log detailed error context
-    if (interaction.replied || interaction.deferred) {
+    logger.error(
-      await interaction.followUp({
+      `Command "${interaction.commandName}" failed for user ${interaction.user.tag}:`,
-        content: 'There was an error while executing this command!',
+      error,
+    );
+
+    // Prepare error response
+    const isProduction = process.env.NODE_ENV === 'production';
+    const errorMessage = isProduction
+      ? `❌ Command failed. Please try again later.`
+      : `❌ Command failed: ${error.message}\n\n${error.stack}`;
+
+    const responseMethod = interaction.replied || interaction.deferred ? 'followUp' : 'reply';
+
+    await interaction[responseMethod]({
+      content: errorMessage,
       ephemeral: true,
     });
-    } else {
-      await interaction.reply({
-        content: 'There was an error while executing this command!',
-        ephemeral: true,
-      });
-    }
   }
 });
 

commands/ask.js

@@ -8,6 +8,7 @@
 
 import { SlashCommandBuilder } from 'discord.js';
 import axios from 'axios';
+import Command from '../utils/Command.js';
 
 const config = {
   webSearch: {
@@ -16,8 +17,9 @@ const config = {
   },
 };
 
-export default {
+export default class AskCommand extends Command {
-  data: new SlashCommandBuilder()
+  defineCommand() {
+    return new SlashCommandBuilder()
       .setName('ask')
       .setDescription('Ask a question to the AI')
       .addStringOption(option =>
@@ -28,9 +30,10 @@ export default {
           .setName('websearch')
           .setDescription('Enable web search for more up-to-date information')
           .setRequired(false),
-    ),
+      );
+  }
 
-  async execute(interaction) {
+  async run(interaction) {
     await interaction.deferReply();
 
     const prompt = interaction.options.getString('prompt');
@@ -42,7 +45,6 @@ export default {
         ? userWebSearchOption // Use user's choice if override is allowed and option was provided
         : config.webSearch.enabled; // Otherwise use default config
 
-    try {
     const response = await axios.post(
       'https://openrouter.ai/api/v1/chat/completions',
       {
@@ -130,54 +132,23 @@ export default {
       await chunks.reduce(
         (promise, chunk) =>
           promise.then(async () => {
-              try {
             await interaction.followUp(chunk);
-                return undefined;
-              } catch (error) {
-                console.error('Error sending message chunk:', {
-                  chunkLength: chunk.content.length,
-                  error: error.message,
-                });
-                // Attempt to send error notification
-                try {
-                  await interaction.followUp({
-                    content: 'Failed to send complete response. Please try again.',
-                    ephemeral: true,
-                  });
-                } catch (e) {
-                  // If even the error notification fails, log it
-                  console.error('Failed to send error notification:', e.message);
-                }
-                // Reject to stop processing remaining chunks
-                return Promise.reject(error);
-              }
           }),
         Promise.resolve(),
       );
     }
-    } catch (error) {
+  }
-      // Log detailed error information
-      console.error('Error in ask command:', {
-        message: error.message,
-        response: error.response?.data,
-        status: error.response?.status,
-        stack: error.stack,
-      });
 
-      // Provide more specific error messages to users
+  getErrorMessage(error) {
-      let errorMessage = 'Sorry, there was an error processing your request.';
     if (error.response?.status === 429) {
-        errorMessage = 'The AI service is currently busy. Please try again in a few moments.';
+      return 'The AI service is currently busy. Please try again in a few moments.';
-      } else if (error.code === 'ECONNABORTED' || error.code === 'ETIMEDOUT') {
+    }
-        errorMessage = 'The request timed out. Please try again.';
+    if (error.code === 'ECONNABORTED' || error.code === 'ETIMEDOUT') {
-      } else if (error.response?.status === 400) {
+      return 'The request timed out. Please try again.';
-        errorMessage = 'Invalid request. Please try rephrasing your question.';
+    }
+    if (error.response?.status === 400) {
+      return 'Invalid request. Please try rephrasing your question.';
+    }
+    return 'Sorry, there was an error processing your request.';
   }
-
-      await interaction.followUp({
-        content: errorMessage,
-        ephemeral: true,
-      });
 }
-  },
-};

commands/help.js

@@ -7,10 +7,14 @@
 // (at your option) any later version.
 
 import { SlashCommandBuilder, EmbedBuilder } from 'discord.js';
+import Command from '../utils/Command.js';
 
-export default {
+export default class HelpCommand extends Command {
-  data: new SlashCommandBuilder().setName('help').setDescription('Lists all available commands'),
+  defineCommand() {
-  async execute(interaction) {
+    return new SlashCommandBuilder().setName('help').setDescription('Lists all available commands');
+  }
+
+  async run(interaction) {
     const { commands } = interaction.client;
     const helpEmbed = new EmbedBuilder()
       .setColor('#5dc67b')
@@ -26,5 +30,5 @@ export default {
     });
 
     await interaction.reply({ embeds: [helpEmbed], ephemeral: true });
-  },
+  }
-};
+}

commands/kick.js

@@ -7,49 +7,36 @@
 // (at your option) any later version.
 
 import { SlashCommandBuilder, PermissionFlagsBits } from 'discord.js';
+import Command from '../utils/Command.js';
 
-// eslint-disable-next-line import/extensions
+export default class KickCommand extends Command {
-import logger from '../logger.js';
+  defineCommand() {
-
+    return new SlashCommandBuilder()
-export default {
-  data: new SlashCommandBuilder()
       .setName('kick')
       .setDescription('Kick a user from the server')
       .addUserOption(option =>
         option.setName('target').setDescription('The user to kick').setRequired(true),
       )
       .addStringOption(option => option.setName('reason').setDescription('Reason for kicking'))
-    .setDefaultMemberPermissions(PermissionFlagsBits.KickMembers),
+      .setDefaultMemberPermissions(PermissionFlagsBits.KickMembers);
-  async execute(interaction) {
+  }
+
+  async run(interaction) {
     const target = interaction.options.getMember('target');
     const reason = interaction.options.getString('reason') ?? 'No reason provided';
 
     if (!target) {
-      return interaction.reply({
+      throw new Error('That user is not in this server!');
-        content: 'That user is not in this server!',
-        ephemeral: true,
-      });
     }
 
     if (!target.kickable) {
-      return interaction.reply({
+      throw new Error('I cannot kick this user! They may have higher permissions than me.');
-        content: 'I cannot kick this user! They may have higher permissions than me.',
-        ephemeral: true,
-      });
     }
 
-    try {
     await target.kick(reason);
-      return await interaction.reply({
+    await interaction.reply({
       content: `Successfully kicked ${target.user.tag}\nReason: ${reason}`,
       ephemeral: true,
     });
-    } catch (error) {
-      logger.error(error);
-      return interaction.reply({
-        content: 'There was an error trying to kick this user!',
-        ephemeral: true,
-      });
   }
-  },
+}
-};

commands/ping.js

@@ -7,10 +7,14 @@
 // (at your option) any later version.
 
 import { SlashCommandBuilder } from 'discord.js';
+import Command from '../utils/Command.js';
 
-export default {
+export default class PingCommand extends Command {
-  data: new SlashCommandBuilder().setName('ping').setDescription('Replies with Pong!'),
+  defineCommand() {
-  async execute(interaction) {
+    return new SlashCommandBuilder().setName('ping').setDescription('Replies with Pong!');
+  }
+
+  async run(interaction) {
     await interaction.reply('Pong! 🏓');
-  },
+  }
-};
+}

commands/prune.js

@@ -7,9 +7,11 @@
 // (at your option) any later version.
 
 import { SlashCommandBuilder, PermissionFlagsBits } from 'discord.js';
+import Command from '../utils/Command.js';
 
-export default {
+export default class PruneCommand extends Command {
-  data: new SlashCommandBuilder()
+  defineCommand() {
+    return new SlashCommandBuilder()
       .setName('prune')
       .setDescription('Prune up to 99 messages.')
       .addIntegerOption(option =>
@@ -20,22 +22,15 @@ export default {
           .setMaxValue(99)
           .setRequired(true),
       )
-    .setDefaultMemberPermissions(PermissionFlagsBits.ManageMessages),
+      .setDefaultMemberPermissions(PermissionFlagsBits.ManageMessages);
-  async execute(interaction) {
+  }
-    const amount = interaction.options.getInteger('amount');
 
-    try {
+  async run(interaction) {
+    const amount = interaction.options.getInteger('amount');
     const deleted = await interaction.channel.bulkDelete(amount, true);
     await interaction.reply({
       content: `Successfully deleted ${deleted.size} message(s).`,
       ephemeral: true,
     });
-    } catch (error) {
-      console.error(error);
-      await interaction.reply({
-        content: 'There was an error trying to prune messages in this channel!',
-        ephemeral: true,
-      });
   }
-  },
+}
-};

logger.js

@@ -1,28 +1,79 @@
 import chalk from 'chalk';
 
+/**
+ * Enhanced logger utility with timestamps and consistent formatting
+ */
 class Logger {
+  /**
+   * Create a new Logger instance
+   * @param {string} moduleName - Name of the module for context
+   */
   constructor(moduleName) {
+    if (typeof moduleName !== 'string') {
+      throw new Error('Logger requires a string moduleName');
+    }
     this.moduleName = moduleName;
   }
 
+  /**
+   * Get formatted timestamp
+   * @private
+   */
+  #getTimestamp() {
+    return new Date().toISOString();
+  }
+
+  /**
+   * Format log message with consistent structure
+   * @private
+   * @param {string} emoji - Log level emoji
+   * @param {string} level - Log level name
+   * @param {string} message - Message to log
+   */
+  #formatMessage(emoji, level, message) {
+    const timestamp = this.#getTimestamp();
+    return `${chalk.gray(timestamp)} ${emoji} ${chalk.cyan(`[${this.moduleName}]`)} ${level}: ${message}`;
+  }
+
+  /**
+   * Log informational message
+   * @param {string} message - Message to log
+   */
   log(message) {
-    console.log(`${chalk.blue('📝')} ${chalk.blue(`[${this.moduleName}]`)} ${message}`);
+    console.log(this.#formatMessage(chalk.blue('📝'), 'LOG', message));
   }
 
+  /**
+   * Log error message
+   * @param {string|Error} message - Error message or Error object
+   */
   error(message) {
-    console.error(`${chalk.red('❌')} ${chalk.red(`[${this.moduleName}]`)} ${message}`);
+    const msg = message instanceof Error ? message.stack || message.message : message;
+    console.error(this.#formatMessage(chalk.red('❌'), 'ERROR', msg));
   }
 
+  /**
+   * Log warning message
+   * @param {string} message - Warning message
+   */
   warn(message) {
-    console.warn(`${chalk.yellow('⚠️')} ${chalk.yellow(`[${this.moduleName}]`)} ${message}`);
+    console.warn(this.#formatMessage(chalk.yellow('⚠️'), 'WARN', message));
   }
 
+  /**
+   * Log info message
+   * @param {string} message - Info message
+   */
   info(message) {
-    console.info(`${chalk.green('ℹ️')} ${chalk.green(`[${this.moduleName}]`)} ${message}`);
+    console.info(this.#formatMessage(chalk.green('ℹ️'), 'INFO', message));
   }
 
+  /**
+   * Log debug message
+   * @param {string} message - Debug message
+   */
   debug(message) {
-    console.debug(`${chalk.gray('🔧')} ${chalk.gray(`[${this.moduleName}]`)} ${message}`);
+    console.debug(this.#formatMessage(chalk.gray('🔧'), 'DEBUG', message));
   }
 }
 

utils/command.js

@@ -0,0 +1,69 @@
+// utils/Command.js - Base command class for Discord bot commands
+// Copyright (C) 2025  Luis Bauza
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+
+import { SlashCommandBuilder } from 'discord.js';
+import logger from '../logger.js';
+
+export default class Command {
+  constructor() {
+    if (this.constructor === Command) {
+      throw new Error('Abstract class Command cannot be instantiated');
+    }
+
+    this._data = this.defineCommand();
+    this._logger = new logger(this.constructor.name);
+  }
+
+  get data() {
+    return this._data;
+  }
+
+  // Abstract method - must be implemented by subclasses
+  defineCommand() {
+    throw new Error('Method defineCommand() must be implemented');
+  }
+
+  // Common execute method with standardized error handling
+  async execute(interaction) {
+    try {
+      this._logger.info(`Executing command: ${interaction.commandName}`);
+      await this.run(interaction);
+    } catch (error) {
+      this.handleError(interaction, error);
+    }
+  }
+
+  // Abstract method - must be implemented by subclasses
+  async run(interaction) {
+    throw new Error('Method run() must be implemented');
+  }
+
+  // Standardized error handling
+  handleError(interaction, error) {
+    this._logger.error(`Command ${interaction.commandName} failed:`, error);
+
+    const response = interaction.deferred ? 'followUp' : 'reply';
+    const errorMessage = this.getErrorMessage(error);
+
+    interaction[response]({
+      content: errorMessage,
+      ephemeral: true,
+    });
+  }
+
+  // Customizable error message handling
+  getErrorMessage(error) {
+    if (error.response?.status === 429) {
+      return 'The service is currently busy. Please try again later.';
+    }
+    if (error.code === 'ECONNABORTED' || error.code === 'ETIMEDOUT') {
+      return 'The request timed out. Please try again.';
+    }
+    return `Command failed: ${error.message}`;
+  }
+}

utils/commandLoader.js

@@ -1,6 +1,19 @@
 import { readdirSync } from 'node:fs';
 import { join } from 'node:path';
 
+/**
+ * Asynchronously loads all command modules from a directory
+ * @async
+ * @param {string} commandsPath - Path to the commands directory
+ * @param {Logger} logger - Logger instance for logging loading progress
+ * @returns {Promise<Array<Object>>} Array of command objects with:
+ * @property {SlashCommandBuilder} data - Command definition
+ * @property {Function} execute - Command execution function
+ * @throws {TypeError} If commandsPath is not a string
+ * @example
+ * // Load all commands from './commands'
+ * const commands = await loadCommands('./commands', logger);
+ */
 export async function loadCommands(commandsPath, logger) {
   const commands = [];
   const commandFiles = readdirSync(commandsPath).filter(file => file.endsWith('.js'));
@@ -10,21 +23,28 @@ export async function loadCommands(commandsPath, logger) {
       try {
         const filePath = join(commandsPath, file);
         const commandModule = await import(filePath);
-        const command = commandModule.default;
+        const Command = commandModule.default;
 
-        if (!command?.data || !command?.execute) {
+        const CommandBase = (await import('./Command.js')).default;
-          logger.warn(
+        if (Command.prototype instanceof CommandBase) {
-            `[WARNING] The command at ${filePath} is missing a required "data" or "execute" property.`
+          // New style - class extending Command
-          );
+          const commandInstance = new Command();
+          // Wrap run() in execute() for backward compatibility
+          commandInstance.execute = commandInstance.run.bind(commandInstance);
+          commands.push(commandInstance);
+          logger.log(`Loaded command: ${commandInstance.data.name}`);
+        } else if (Command?.data && Command?.execute) {
+          // Old style - plain object (maintain backward compatibility)
+          commands.push(Command);
+          logger.log(`Loaded command: ${Command.data.name}`);
+        } else {
+          logger.warn(`[WARNING] The command at ${filePath} is missing required properties.`);
           return;
         }
-
-        commands.push(command);
-        logger.log(`Loaded command: ${command.data.name}`);
       } catch (error) {
-        logger.error(`Error loading command from ${file}:`, error);
+        logger.error(`Error loading command from ${file}: ${error.stack}`);
       }
-    })
+    }),
   );
 
   return commands;