fix: fix deepsource errors

Author: dev-737

Dockerfile

@@ -18,11 +18,7 @@
 import type BaseCommand from '#src/core/BaseCommand.js';
 import Context from '#src/core/CommandContext/Context.js';
 import Logger from '#src/utils/Logger.js';
-import {
-  extractUserId,
-  extractChannelId,
-  extractRoleId,
-} from '#src/utils/Utils.js';
+import { extractUserId, extractChannelId, extractRoleId } from '#src/utils/Utils.js';
 import {
   type APIApplicationCommandBasicOption,
   type APIModalInteractionResponseCallbackData,
@@ -71,10 +67,7 @@ export default class PrefixContext extends Context<{
     // Split arguments with quote handling
     const commandOptions = new Map(
       // eslint-disable-next-line @typescript-eslint/no-unused-vars
-      Object.entries(command.options).map(([_i, option]) => [
-        option.name,
-        option,
-      ]),
+      Object.entries(command.options).map(([_i, option]) => [option.name, option]),
     );
 
     // Store parsed arguments with resolved values
@@ -88,15 +81,13 @@ export default class PrefixContext extends Context<{
   private parseArguments(
     rawArgs: string[],
     definedOpts: Map<string, APIApplicationCommandBasicOption>,
-  ):
-		| Collection<
-		  string,
-		  {
-		    value: string | number | boolean | null;
-		    type: ApplicationCommandOptionType;
-		  }
-		>
-		| undefined {
+  ): Collection<
+    string,
+    {
+      value: string | number | boolean | null;
+      type: ApplicationCommandOptionType;
+    }
+  > | null {
     const args = new Collection<
       string,
       {
@@ -114,21 +105,21 @@ export default class PrefixContext extends Context<{
         if (!option) {
           this.argumentValidationPassed = false;
           Logger.error(`Unknown option: ${name}`);
-          return;
+          return null;
         }
 
         if (!value && option.required) {
           this.argumentValidationPassed = false;
           Logger.error(`Missing required option: ${option.name}`);
-          return;
+          return null;
         }
 
         if (value) {
           const parsed = this.processArg(value, option.type);
           if (parsed === null) {
             this.argumentValidationPassed = false;
             Logger.error(`Invalid value for ${name}`);
-            return;
+            return null;
           }
           args.set(option.name, { value: parsed, type: option.type });
           definedOpts.delete(option.name);
@@ -146,21 +137,21 @@ export default class PrefixContext extends Context<{
       if (!option) {
         this.argumentValidationPassed = false;
         Logger.error(`Unknown option: ${value}`);
-        return;
+        return null;
       }
 
       if (!value && option.required) {
         this.argumentValidationPassed = false;
         Logger.error(`Missing required option: ${option.name}`);
-        return;
+        return null;
       }
 
       if (value) {
         const parsed = this.processArg(value, option.type);
         if (parsed === null) {
           this.argumentValidationPassed = false;
           Logger.error(`Invalid value for ${value}`);
-          return;
+          return null;
         }
         args.set(option.name, { value: parsed, type: option.type });
       }
@@ -205,9 +196,7 @@ export default class PrefixContext extends Context<{
 
   public async reply(data: string | MessageReplyOptions) {
     this.lastReply = await this.interaction.reply(
-      typeof data === 'string'
-        ? { content: data }
-        : { ...data, content: data.content ?? '' },
+      typeof data === 'string' ? { content: data } : { ...data, content: data.content ?? '' },
     );
     return this.lastReply;
   }
@@ -227,18 +216,16 @@ export default class PrefixContext extends Context<{
   public async editReply(data: string | MessageEditOptions) {
     return (
       (await this.lastReply?.edit(
-        typeof data === 'string'
-          ? { content: data }
-          : { ...data, content: data.content ?? '' },
+        typeof data === 'string' ? { content: data } : { ...data, content: data.content ?? '' },
       )) ?? null
     );
   }
 
   public async showModal(
     modal:
-			| JSONEncodable<APIModalInteractionResponseCallbackData>
-			| ModalComponentData
-			| APIModalInteractionResponseCallbackData,
+      | JSONEncodable<APIModalInteractionResponseCallbackData>
+      | ModalComponentData
+      | APIModalInteractionResponseCallbackData,
   ) {
     const reply = await this.reply({
       content: 'Click button to enter data.',
@@ -254,8 +241,7 @@ export default class PrefixContext extends Context<{
 
     const collector = reply?.createMessageComponentCollector({
       componentType: ComponentType.Button,
-      filter: (i) =>
-        i.customId === 'openForm' && i.user.id === this.interaction.author.id,
+      filter: (i) => i.customId === 'openForm' && i.user.id === this.interaction.author.id,
       idle: 60000,
     });
 

src/core/CommandContext/PrefixContext.ts

@@ -62,7 +62,10 @@ export default class MessageCreate extends BaseEventListener<'messageCreate'> {
 
   private async handlePrefixCommand(message: Message): Promise<void> {
     const userData = await fetchUserData(message.author.id);
-    if (!userData?.acceptedRules) return await showRulesScreening(message, userData);
+    if (!userData?.acceptedRules) {
+      await showRulesScreening(message, userData);
+      return;
+    }
 
     const { command, prefixArgs } = resolveCommand(message);
     if (!command) return;

src/events/messageCreate.ts

@@ -43,7 +43,7 @@ export default class AntiSpamManager {
     });
   }
 
-  public async handleMessage(message: Message): Promise<UserMessageInfo | undefined> {
+  public async handleMessage(message: Message): Promise<UserMessageInfo | null> {
     const userId = message.author.id;
     const currentTime = Date.now();
     const userInfo = await this.getUserInfo(userId);
@@ -60,6 +60,7 @@ export default class AntiSpamManager {
     }
 
     this.setUserInfo(userId, { ...userInfo, lastMessage: currentTime });
+    return null;
   }
 
   private async getUserInfo(userId: string): Promise<UserMessageInfo> {

src/managers/AntiSpamManager.ts

@@ -135,7 +135,10 @@ export default class HubLogManager {
   }
 
   async removeRoleId(type: RoleIdLogConfigs) {
-    if (!this.config[type]) return await this.resetLog(type);
+    if (!this.config[type]) {
+      await this.resetLog(type);
+      return;
+    }
 
     await this.updateLogConfig({
       [type]: { set: { channelId: this.config[type].channelId } },

src/managers/HubLogManager.ts

@@ -51,7 +51,10 @@ export class MessageProcessor {
     const { hub, hubConnections, connection } = hubData;
 
     const userData = await fetchUserData(message.author.id);
-    if (!userData?.acceptedRules) return await showRulesScreening(message, userData);
+    if (!userData?.acceptedRules) {
+      await showRulesScreening(message, userData);
+      return;
+    }
 
     const attachmentURL = await this.broadcastService.resolveAttachmentURL(message);
 

src/services/MessageProcessor.ts

@@ -89,7 +89,7 @@ function parseArgs(input: string): string[] {
   // Process matches to handle key-value pairs with quoted values
   return matches.map((match) => {
     // Check if the match is a key-value pair with a quoted value
-    if (/=.+/.test(match)) {
+    if (/[=].+/.test(match)) {
       const [key, value] = match.split('=');
       // Remove surrounding quotes from the value if present
       const cleanedValue = value.replace(quoteRegex, '');