fix minor bug that guilds couldn't be checked

Author: The-LukeZ

index.js

@@ -1,4 +1,4 @@
-const { Client, REST, Collection, Routes } = require("discord.js");
+const { Client, REST } = require("discord.js");
 const { readdirSync } = require("node:fs");
 const path = require("node:path");
 
@@ -165,10 +165,11 @@ class cDClient extends Client {
             if (privateCommands.length) {
                 // TODO: Add support for sharding
 
-                let clientGuilds = [];
+                let clientGuilds = this.guilds.cache;
 
-                if (!this.isReady() || this.ws.status != 5) {
+                if (!this.isReady() || !this.guilds.cache.size) {
                     // Wait for the guilds to cache before continuing
+                    console.log(this.guilds.cache.size);
                     while (!this.guilds.cache.size) {
                         await new Promise((resolve) =>
                             setTimeout(resolve, 1000)
@@ -228,50 +229,42 @@ class cDClient extends Client {
      *
      * @param {string} command The commands's name or ID
      * @param {string | null} guildId The guild's ID to delete the command in (not needed for a global command)
-     * @param {Collection | null} commands The command-map like explained [in the guide](https://discordjs.guide/creating-your-bot/command-handling). This is important when using only the name of the command (the command name needs to be associated with the command id)
-     * @returns {Promise<boolean>} `true` if the operation was successfull. Otherwise an error will be raised.
+     * @returns {Promise<void>}
      */
-    async deleteCommand(command, guildId = null, commands = new Collection()) {
-        if (!this.isReady() || !this.ws.status == 0) {
-            throw new Error("The client must be logged in!");
+    async deleteCommand(command, guildId = null) {
+        if (!this.isReady()) {
+            console.error("The client must be logged in!");
+            return;
         } else if (guildId && !/^\d+$/i.test(guildId)) {
-            throw new Error(
-                "The guildId is invalid! Must be a numerous string."
-            );
+            console.error("The guildId is invalid! Must be a numerous string.");
+            return;
         }
 
         try {
             if (/^\d+$/i.test(command)) {
                 await this.application.commands.delete(command, guildId);
             } else {
-                let theCommand = commands.find((cmd) => cmd.name === command);
-                if (!theCommand) {
-                    const commands = await this.rest.get(
-                        Routes.applicationGuildCommands(
-                            this.application.id,
-                            guildId
-                        )
-                    );
-
-                    theCommand = commands.find((cmd) => cmd.name === command);
-                }
+                const theCommand =
+                    this.application.commands.cache.get(command) ??
+                    (await this.application.commands.fetch({
+                        guildId: guildId,
+                        cache: true,
+                    }));
 
                 if (!theCommand) {
-                    throw new Error(
+                    console.error(
                         `❌ Command '${command}' not found in guild '${guildId}'`
                     );
+                    return;
                 }
 
                 await this.application.commands.delete(theCommand.id, guildId);
             }
         } catch (err) {
             console.error(
-                `❌ Error while deleting command '${command}' in guild '${guildId}':`,
+                `❌ Error while deleting a command in guild '${guildId}'`,
                 err
             );
-            throw new Error(
-                `Error while deleting command '${command}' in guild '${guildId}'`
-            );
         }
         return;
     }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "djs-command-deployer",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "",
   "main": "./index.js",
   "scripts": { },