zumito-framework 1.7.3 → 1.8.0

package/dist/services/SlashCommandRefresher.js

@@ -15,11 +15,19 @@ export class SlashCommandRefresher {
         const commands = Array.from(this.framework.commands.getAll().values())
             .filter((command) => (command.type == CommandType.slash ||
             command.type == CommandType.separated ||
-            command.type == CommandType.any)
-            && !command.parent)
+            command.type == CommandType.any) &&
+            !command.parent)
             .map(command => this.mapCommand(command));
-        const data = await rest.put(Routes.applicationCommands(this.framework.settings.discordClientOptions.clientId), { body: commands });
-        console.debug(`Successfully reloaded ${data.length} of ${commands.length} application (/) commands.`);
+        const uniqueCommands = [];
+        const names = new Set();
+        for (const cmd of commands) {
+            if (!names.has(cmd.name)) {
+                uniqueCommands.push(cmd);
+                names.add(cmd.name);
+            }
+        }
+        const data = await rest.put(Routes.applicationCommands(this.framework.settings.discordClientOptions.clientId), { body: uniqueCommands });
+        console.debug(`Successfully reloaded ${data.length} of ${uniqueCommands.length} application (/) commands.`);
     }
     mapCommand(command, commandBuilder) {
         const slashCommand = commandBuilder || new SlashCommandBuilder();
@@ -77,9 +85,9 @@ export class SlashCommandRefresher {
         Array.from(this.framework.commands.getAll().values())
             .filter((subcommand) => (subcommand.type == CommandType.slash ||
             subcommand.type == CommandType.separated ||
-            subcommand.type == CommandType.any)
-            && subcommand.parent
-            && subcommand.parent == command.name)
+            subcommand.type == CommandType.any) &&
+            subcommand.parent &&
+            subcommand.parent == command.name)
             .forEach(subcommand => {
             if (!commandBuilder) {
                 slashCommand.addSubcommand((subcommandSlashBuilder) => {

package/dist/services/handlers/ErrorHandler.d.ts

@@ -24,6 +24,6 @@ export declare class ErrorHandler {
     handleError(error: any, options: ErrorOptions): void;
     handleCommandError(error: Error, options: CommandErrorOptions): void;
     handleShapeShiftErrors(error: any): void;
-    printErrorStack(error: Error): void;
+    printErrorStack(error: any): void;
 }
 export {};

package/dist/services/handlers/ErrorHandler.js

@@ -11,28 +11,30 @@ export class ErrorHandler {
         if (options?.type == ErrorType.CommandInstance || options?.type == ErrorType.CommandLoad || options?.type == ErrorType.CommandRun) {
             this.handleCommandError(error, options);
         }
-        else if (error.constructor.name == 'CombinedError') {
+        else if (error?.constructor?.name == 'CombinedError') {
             this.handleShapeShiftErrors(error);
         }
-        else if (error.constructor?.name == "ExpectedValidationError") {
+        else if (error?.constructor?.name == "ExpectedValidationError") {
             console.error(`Validation error: Expected ${error.expected}, but received ${error.given}.`);
             console.line('');
         }
-        else if (error.constructor.name == "ValidationError") {
+        else if (error?.constructor?.name == "ValidationError") {
             console.error(`Validation error: ${error.validator} received invalid input: ${error.given}`);
             console.line('');
         }
         else if (options?.type == ErrorType.Api) {
             console.group(`[❌] Error in API endpoint ${options.endpoint} (${options.method})`);
             console.line(chalk.red('Error:'));
-            console.line(error.toString());
+            console.line(error?.toString?.() || 'Unknown error');
             console.line('');
             console.groupEnd();
         }
         else {
-            console.error(error.toString());
+            console.error(error?.toString?.() || 'Unknown error');
             console.line('');
         }
+        // Emit framework error event so external modules can listen and report
+        this.framework.eventEmitter.emit('error', error, options);
         this.printErrorStack(error);
         if (options.exit)
             process.exit(1);
@@ -49,19 +51,19 @@ export class ErrorHandler {
                 console.group(`[❌] Error running command ${options.command.name}`);
                 break;
         }
-        if (error.constructor.name == 'CombinedError') {
+        if (error?.constructor?.name == 'CombinedError') {
             console.groupEnd();
             this.handleShapeShiftErrors(error);
         }
         else {
             console.line(chalk.red('Error:'));
-            console.line(error.toString());
+            console.line(error?.toString?.() || 'Unknown error');
             console.line('');
             console.groupEnd();
         }
     }
     handleShapeShiftErrors(error) {
-        if (error.constructor.name == 'CombinedError') {
+        if (error?.constructor?.name == 'CombinedError') {
             error.errors.forEach(err => {
                 this.handleError(err, {
                     type: ErrorType.Other
@@ -70,7 +72,15 @@ export class ErrorHandler {
         }
     }
     printErrorStack(error) {
-        const stackParsedError = ErrorStackParser.parse(error);
+        if (!error || !(error.stack || error.stacktrace))
+            return;
+        let stackParsedError;
+        try {
+            stackParsedError = ErrorStackParser.parse(error);
+        }
+        catch {
+            return;
+        }
         let functionColor = 'blue';
         const lines = [];
         let codeFragment;

package/dist/services/managers/CommandManager.js

@@ -126,9 +126,10 @@ export class CommandManager {
     async refreshSlashCommands() {
         const rest = new REST({ version: '10' }).setToken(this.framework.settings.discordClientOptions.token);
         const commands = Array.from(this.commands.values())
-            .filter((command) => command.type == CommandType.slash ||
+            .filter((command) => (command.type == CommandType.slash ||
             command.type == CommandType.separated ||
-            command.type == CommandType.any)
+            command.type == CommandType.any) &&
+            !command.parent)
             .map((command) => {
             const slashCommand = new SlashCommandBuilder()
                 .setName(command.name)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zumito-framework",
-  "version": "1.7.3",
+  "version": "1.8.0",
   "description": "Discord.js bot framework",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",