🧑‍💻 Fixed some more code smells

src/jobs/shop/modules/roles/components/overDueForPayment.ts

@@ -51,11 +51,8 @@ export const execute = async (client: Client, role: IShopRole) => {
             );
           });
       })
-      .catch(async (err) => {
+      .catch(() => {
-        throw new Error(
+        throw new Error(`Failed removing role from user.`);
-          `Error removing role ${roleId} from user ${userId}.`,
-          err
-        );
       });
 
     throw new Error("User does not have enough credits.");
@@ -70,16 +67,16 @@ export const execute = async (client: Client, role: IShopRole) => {
       role.lastPayed = new Date();
       await role
         .save()
-        .then(async () => {
+        .then(() => {
           logger.silly(`Shop role ${roleId} has been updated.`);
         })
-        .catch(async (err) => {
+        .catch(() => {
-          throw new Error(`Error updating shop role ${roleId}.`, err);
+          throw new Error("Failed updating shop role.");
         });
 
       logger.debug(`Shop role ${roleId} has been paid.`);
     })
-    .catch(async (err) => {
+    .catch(() => {
-      throw new Error(`Error updating user ${userId}.`, err);
+      throw new Error("Failed updating user.");
     });
 };

src/jobs/timeouts/index.ts

@@ -24,7 +24,7 @@ export const execute = async () => {
             timeoutId,
             cooldown,
           })
-          .then(async () => {
+          .then(() => {
             logger.debug(
               `Timeout document ${timeoutId} has been deleted from user ${userId}.`
             );

src/plugins/commands/config/modules/audits/index.ts

@@ -78,9 +78,10 @@ export default {
           text: footerText,
         });
 
-      return interaction.editReply({
+      await interaction.editReply({
         embeds: [embedSuccess],
       });
+      return;
     });
   },
 };

src/plugins/commands/config/modules/cpgg/index.ts

@@ -80,9 +80,10 @@ export default {
             text: footerText,
           });
 
-        return interaction?.editReply({
+        await interaction?.editReply({
           embeds: [interactionEmbed],
         });
+        return;
       });
   },
 };

src/plugins/commands/config/modules/credits/index.ts

@@ -126,9 +126,10 @@ export default {
           text: footerText,
         });
 
-      return interaction?.editReply({
+      await interaction?.editReply({
         embeds: [interactionEmbed],
       });
+      return;
     });
   },
 };

src/plugins/commands/config/modules/embeds/index.ts

@@ -86,9 +86,10 @@ export default {
           },
         ]);
 
-      return interaction.editReply({
+      await interaction.editReply({
         embeds: [embed],
       });
+      return;
     });
   },
 };

src/plugins/commands/config/modules/points/index.ts

@@ -98,9 +98,10 @@ export default {
           text: footerText,
         });
 
-      return interaction?.editReply({
+      await interaction?.editReply({
         embeds: [interactionEmbed],
       });
+      return;
     });
   },
 };

src/plugins/commands/config/modules/shop/index.ts

@@ -79,9 +79,10 @@ export default {
           text: footerText,
         });
 
-      return interaction?.editReply({
+      await interaction?.editReply({
         embeds: [interactionEmbed],
       });
+      return;
     });
   },
 };

src/plugins/commands/config/modules/welcome/index.ts

@@ -171,9 +171,10 @@ export default {
           text: footerText,
         });
 
-      return interaction?.editReply({
+      await interaction?.editReply({
         embeds: [interactionEmbed],
       });
+      return;
     });
   },
 };

src/plugins/commands/reputation/modules/give/index.ts

@@ -49,7 +49,7 @@ export default {
     if (!userObj) throw new Error("User is undefined");
 
     // Pre-checks
-    await noSelfReputation(optionTarget, user);
+    noSelfReputation(optionTarget, user);
 
     // Check if user is on cooldown otherwise create one
     await cooldown.command(interaction, process.env.REPUTATION_TIMEOUT);