fix: more lint errors

Author: evolvewithevan

gitcord/bot.py

@@ -9,7 +9,7 @@
 
 from .config import config
 from .events import setup_events
-from .utils.logger import logger
+from .utils.logger import main_logger as logger
 
 
 class GitCordBot(commands.Bot):
@@ -57,16 +57,17 @@ async def _load_cogs(self) -> None:
         except Exception as e:
             logger.error("Failed to load cogs: %s", e)
 
-    async def on_command_error(self, ctx: commands.Context, error: commands.CommandError) -> None:
+    async def on_command_error(self, context, error):
         """Global command error handler."""
         logger.error("Global command error: %s", error)
-
         if isinstance(error, commands.CommandNotFound):
-            await ctx.send("Command not found. Try `!hello` or `!ping`!")
+            await context.send("Command not found. Try `!hello` or `!ping`!")
         elif isinstance(error, commands.MissingPermissions):
-            await ctx.send("You don't have permission to use this command!")
+            await context.send("You don't have permission to use this command!")
+        elif isinstance(error, commands.CommandError):
+            await context.send(f"A command error occurred: {error}")
         else:
-            await ctx.send(f"An error occurred: {error}")
+            raise error
 
 
 async def main() -> None:

gitcord/cogs/general.py

@@ -11,7 +11,7 @@
 import requests
 from bs4 import BeautifulSoup
 
-from ..utils.logger import logger
+from ..utils.logger import main_logger as logger
 from ..utils.helpers import format_latency, create_embed
 
 
@@ -90,8 +90,10 @@ async def fetchurl_prefix(self, ctx: commands.Context, url: str) -> None:
                 color=discord.Color.red()
             )
             await ctx.send(embed=embed)
-
-        except Exception as e:
+        except discord.DiscordException as e:
+            logger.error("Discord error in fetchurl command: %s", e)
+            await ctx.send("A Discord error occurred.")
+        except Exception as e:  # pylint: disable=broad-except
             logger.error("Error in fetchurl command: %s", e)
             embed = create_embed(
                 title="❌ Unexpected Error",
@@ -167,8 +169,10 @@ async def fetchurl(self, interaction: discord.Interaction, url: str) -> None:
                 color=discord.Color.red()
             )
             await interaction.followup.send(embed=embed)
-
-        except Exception as e:
+        except discord.DiscordException as e:
+            logger.error("Discord error in fetchurl command: %s", e)
+            await interaction.followup.send("A Discord error occurred.")
+        except Exception as e:  # pylint: disable=broad-except
             logger.error("Error in fetchurl command: %s", e)
             embed = create_embed(
                 title="❌ Unexpected Error",
@@ -222,4 +226,4 @@ async def on_command_error(self, ctx: commands.Context, error: commands.CommandE
 
 async def setup(bot: commands.Bot) -> None:
     """Set up the General cog."""
-    await bot.add_cog(General(bot))
+    await bot.add_cog(General(bot))

gitcord/config.py

@@ -44,4 +44,4 @@ def activity_name(self, value: str) -> None:
 
 
 # Global configuration instance
-config = Config()
+config = Config()

gitcord/events.py

@@ -6,11 +6,11 @@
 import discord
 from discord.ext import commands
 
-from .utils.logger import logger
+from .utils.logger import main_logger as logger
 from .config import config
 
 
-class EventHandler:
+class EventHandler:  # pylint: disable=too-few-public-methods
     """Handles Discord bot events."""
 
     def __init__(self, bot: commands.Bot):
@@ -43,7 +43,10 @@ async def _send_restart_messages(self) -> None:
                     try:
                         await channel.send("Bot has restarted successfully!")
                         logger.info("Sent restart message to %s in %s", channel.name, guild.name)
-                    except Exception as e:
+                    except discord.DiscordException as e:
+                        logger.error("Failed to send message to %s in %s: %s",
+                                   channel.name, guild.name, e)
+                    except Exception as e:  # pylint: disable=broad-except
                         logger.error("Failed to send message to %s in %s: %s",
                                    channel.name, guild.name, e)
                     break  # Only send to the first available channel
@@ -63,7 +66,9 @@ async def _sync_commands(self) -> None:
             synced_global = await self.bot.tree.sync()
             logger.info("Synced %d command(s) globally", len(synced_global))
 
-        except Exception as e:
+        except discord.DiscordException as e:
+            logger.error("Failed to sync commands: %s", e)
+        except Exception as e:  # pylint: disable=broad-except
             logger.error("Failed to sync commands: %s", e)
 
 

gitcord/utils/helpers.py

@@ -21,6 +21,7 @@ def format_latency(latency: float) -> str:
     return f"{round(latency * 1000)}ms"
 
 
+# pylint: disable=too-many-arguments, too-many-positional-arguments
 def create_embed(
     title: str,
     description: str,

gitcord/utils/logger.py

@@ -24,28 +24,31 @@ def setup_logger(name: str = "gitcord", level: int = logging.INFO) -> logging.Lo
     if not logger.handlers:
         handler = logging.StreamHandler()
         formatter = logging.Formatter(
-            '%(asctime)s - %(name)s - %(levelname)s - %(message)s'
+            '%(asctime)s - %(name)s - %(levelname)s - '
+            '%(message)s'
+        )
+        handler.setFormatter(
+            formatter
         )
-        handler.setFormatter(formatter)
         logger.addHandler(handler)
 
     return logger
 
 
-def log_error(logger: logging.Logger, error: Exception, context: Optional[str] = None) -> None:
+def log_error(log_instance: logging.Logger, error: Exception, context: Optional[str] = None) -> None:
+
     """
     Log an error with context.
 
     Args:
-        logger: Logger instance
+        log_instance: Logger instance
         error: Exception to log
         context: Optional context string
     """
     message = f"Error: {error}"
     if context:
         message = f"{context} - {message}"
-    logger.error(message, exc_info=True)
-
+    log_instance.error(message, exc_info=True)  # pylint: disable=line-too-long
 
 # Default logger instance
-logger = setup_logger()
+main_logger = setup_logger()

setup_env.py

@@ -41,8 +41,10 @@ def create_env_file():
         print("✅ .env file created successfully!")
         print("You can now run the bot with: python main.py")
 
-    except Exception as e:
+    except OSError as e:
         print(f"Error creating .env file: {e}")
+    except Exception as e:  # pylint: disable=broad-except
+        print(f"Unexpected error: {e}")
 
 
 def main():