Skip to content

Improved error messages for better clarity and user guidance #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
drasti-1650 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Improved error messages for better clarity and user guidance #46

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions bot.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ async def setup_hook(self) -> None:
await init_db()
logger.info("Initialized CodeBuddy database")
except Exception as e:
logger.error(f"Failed to initialize CodeBuddy database: {e}")
logger.error(f"Database initialization failed: Unable to start CodeBuddy database. Please check configuration. Error details: {e}")

# Load core cogs
core_cogs = [
Expand All @@ -83,7 +83,7 @@ async def setup_hook(self) -> None:
await self.load_extension(ext)
logger.info(f'Loaded {ext}')
except Exception as e:
logger.error(f'Failed to load {ext}: {e}')
logger.error(f"Extension loading failed: Could not load module '{ext}'. Please verify the file and dependencies. Error details: {e}")

# Load feature cogs (new/renamed)
feature_cogs = [
Expand Down Expand Up @@ -146,7 +146,7 @@ async def setup_hook(self) -> None:
logger.info(f"📝 Synced commands: {', '.join(command_names)}")

except Exception as e:
logger.error(f"❌ Failed to sync slash commands: {e}")
logger.error(f"Slash command sync failed: Unable to synchronize commands. Please check bot permissions and configuration. Error details: {e}")

# Also log commands from the tree
tree_commands = self.tree.get_commands()
Expand Down Expand Up @@ -208,8 +208,8 @@ async def _safe_ctx_send(message: str) -> None:
elif isinstance(error, commands.BadArgument):
await _safe_ctx_send("Invalid argument provided.")
else:
logger.error(f"Command error: {error}")
await _safe_ctx_send("An error occurred while processing your command.")
logger.error(f"Command execution failed: {error}. Please check your input and try again.")
await _safe_ctx_send("An error occurred while processing your command. Please try again or check your input.")

async def on_app_command_error(self, interaction: discord.Interaction, error: app_commands.AppCommandError):
"""Handle slash command errors."""
Expand All @@ -224,7 +224,7 @@ async def on_app_command_error(self, interaction: discord.Interaction, error: ap
)
else:
# For other app command errors, log and try to respond once
logger.error(f"Slash command error: {error}")
logger.error(f"Slash command execution failed: {error}. Please check command usage and try again.")
try:
if not interaction.response.is_done():
await interaction.response.send_message(
Expand All @@ -250,7 +250,7 @@ async def main():
except KeyboardInterrupt:
logger.info("Bot shutdown requested.")
except Exception as e:
logger.error(f"Bot encountered an error: {e}")
logger.error(f"Runtime error: The bot encountered an unexpected issue. Error details: {e}")
finally:
await bot.close()

Expand Down