refactor(db.py): increase batch_size from 10 to 25 to improve performance

style(error_handler.py): change error message format for better readability
feat(poll.py): trim leading and trailing whitespaces from poll options for cleaner data
refactor(rolecount.py): rename '_rhel' to '_redhat' for better clarity
chore(tools.py): remove unused encoding and decoding methods to clean up code

Author: kzndotsh

tux/cogs/admin/db.py

@@ -9,8 +9,6 @@
 
 from tux.database.controllers import DatabaseController
 
-# TODO: Move to a constants file or set a global check/error handler for all commands to avoid repetition.
-
 GUILD_ONLY_MESSAGE = "This command can only be used in a guild."
 
 
@@ -41,7 +39,7 @@ async def seed_members(self, interaction: discord.Interaction) -> None:
 
         members: Sequence[discord.Member] = interaction.guild.members
 
-        batch_size = 10
+        batch_size = 25
 
         for i in range(0, len(members), batch_size):
             batch = members[i : i + batch_size]
@@ -85,7 +83,7 @@ async def seed_roles(self, interaction: discord.Interaction) -> None:
 
         roles: Sequence[discord.Role] = interaction.guild.roles
 
-        batch_size = 10
+        batch_size = 25
 
         for i in range(0, len(roles), batch_size):
             batch = roles[i : i + batch_size]
@@ -129,7 +127,7 @@ async def seed_user_roles(self, interaction: discord.Interaction) -> None:
             return
 
         members: Sequence[discord.Member] = interaction.guild.members
-        batch_size = 10
+        batch_size = 25
 
         for i in range(0, len(members), batch_size):
             batch = members[i : i + batch_size]

tux/cogs/error_handler.py

@@ -28,8 +28,8 @@
     commands.CheckFailure: "You are not allowed to use this command.",
     commands.CommandNotFound: "This command was not found.",
     commands.CommandOnCooldown: "This command is on cooldown. Try again in {error.retry_after:.2f} seconds.",
-    commands.BadArgument: "Invalid argument passed. Correct usage:\n```{ctx.command.usage}```",
-    commands.MissingRequiredArgument: "Missing required argument. Correct usage:\n```{ctx.command.usage}```",
+    commands.BadArgument: "Invalid argument passed. Correct usage: `{ctx.command.usage}`",
+    commands.MissingRequiredArgument: "Missing required argument. Correct usage: `{ctx.command.usage}`",
     commands.MissingRequiredAttachment: "Missing required attachment.",
     commands.NotOwner: "You are not the owner of this bot.",
     commands.BotMissingPermissions: "The bot is missing the required permissions to use this command.",

tux/cogs/utility/poll.py

@@ -31,6 +31,9 @@ async def poll(self, interaction: discord.Interaction, title: str, options: str)
         # Split the options by comma
         options_list = options.split(",")
 
+        # Remove any leading or trailing whitespaces from the options
+        options_list = [option.strip() for option in options_list]
+
         # Check if the options count is between 2-9
         if len(options_list) < 2 or len(options_list) > 9:
             embed = EmbedCreator.create_error_embed(

tux/cogs/utility/remindme.py

@@ -30,9 +30,6 @@ def get_closest_reminder(reminders: list[Reminders]) -> Reminders | None:
     return min(reminders, key=lambda x: x.expires_at) if reminders else None
 
 
-# TODO: Refactor and clean up this code
-
-
 class RemindMe(commands.Cog):
     def __init__(self, bot: commands.Bot) -> None:
         self.bot = bot

tux/cogs/utility/rolecount.py

@@ -60,7 +60,7 @@
     [1192177499413684226, "_asahi"],
     [1207599112585740309, "_fedoraatomic"],
     [1232383833152819282, "_solus"],
-    [1210000519272079411, "_rhel"],
+    [1210000519272079411, "_redhat"],
     [1232199326722293790, "_mxlinux"],
     [1232387598107017227, "_netbsd"],
     [1232385920335089734, "_qubesos"],
@@ -136,7 +136,7 @@
     [1189236400571301958, "_chromium"],
 ]
 
-# # TODO: Figure out how to make this work without hard coding the roles and emojis.
+# # TODO: Figure out how to make rolecount work without hard coded ids
 
 
 class RoleCount(commands.Cog):

tux/cogs/utility/tldr.py

@@ -35,7 +35,7 @@ async def get_autocomplete(
             A list of up to 25 command names as autocomplete choices.
         """
 
-        # TODO: Why is interaction not used?
+        # TODO: Resolve why interaction is not being used.
 
         commands = self.get_tldrs()
 

tux/cogs/utility/tools.py

@@ -22,15 +22,9 @@ def __init__(self, bot: commands.Bot) -> None:
         self.bot = bot
         self.encodings = {
             "base64": self.encode_base64,
-            # "md5": self.encode_md5,
-            # "sha256": self.encode_sha256,
-            # "sha512": self.encode_sha512,
         }
         self.decodings = {
             "base64": self.decode_base64,
-            # "md5": self.decode_md5,
-            # "sha256": self.decode_sha256,
-            # "sha512": self.decode_sha512,
         }
 
     def encode_base64(self, input_string: str):
@@ -39,15 +33,6 @@ def encode_base64(self, input_string: str):
     def decode_base64(self, input_string: str):
         return b64decode(input_string.encode()).decode()
 
-    # def encode_md5(self, input_string: str):
-    #     return hashlib.md5(input_string.encode()).hexdigest()
-
-    # def encode_sha256(self, input_string: str):
-    #     return hashlib.sha256(input_string.encode()).hexdigest()
-
-    # def encode_sha512(self, input_string: str):
-    #     return hashlib.sha512(input_string.encode()).hexdigest()
-
     group = app_commands.Group(name="tools", description="Various tool commands.")
 
     @group.command(name="colors", description="Converts a color to different formats.")
@@ -173,9 +158,6 @@ async def send_message(
     @app_commands.choices(
         encoding=[
             app_commands.Choice[str](name="base64", value="base64"),
-            # app_commands.Choice[str](name="md5", value="md5"),
-            # app_commands.Choice[str](name="sha256", value="sha256"),
-            # app_commands.Choice[str](name="sha512", value="sha512"),
         ]
     )
     async def encode(