refactor(error_handling): enhance Sentry integration for improved exception tracking

- Added `capture_exception_safe` calls in the Starboard and StatusRoles modules to log exceptions with additional context, improving observability during error occurrences.
- Updated error handling in the Wiki module to capture API errors with specific endpoint context.
- Streamlined exception logging across various operations, ensuring that unexpected errors are reported to Sentry while maintaining appropriate logging levels for expected behaviors.

Author: kzndotsh

src/tux/modules/features/starboard.py

@@ -15,6 +15,7 @@
 from tux.core.bot import Tux
 from tux.core.checks import requires_command_permission
 from tux.core.converters import get_channel_safe
+from tux.services.sentry import capture_exception_safe
 from tux.ui.embeds import EmbedCreator, EmbedType
 
 
@@ -191,6 +192,13 @@ async def setup_starboard(
 
         except Exception as e:
             logger.error(f"Error configuring starboard: {e}")
+            capture_exception_safe(
+                e,
+                extra_context={
+                    "operation": "configure_starboard",
+                    "guild_id": str(ctx.guild.id) if ctx.guild else None,
+                },
+            )
             await ctx.send(f"An error occurred while configuring the starboard: {e}")
 
     @starboard.command(
@@ -236,6 +244,13 @@ async def remove_starboard(self, ctx: commands.Context[Tux]) -> None:
 
         except Exception as e:
             logger.error(f"Error removing starboard configuration: {e}")
+            capture_exception_safe(
+                e,
+                extra_context={
+                    "operation": "remove_starboard_config",
+                    "guild_id": str(ctx.guild.id) if ctx.guild else None,
+                },
+            )
             await ctx.send(
                 f"An error occurred while removing the starboard configuration: {e}",
             )
@@ -279,6 +294,15 @@ async def get_existing_starboard_message(
 
         except Exception as e:
             logger.error(f"Error while fetching starboard message: {e}")
+            capture_exception_safe(
+                e,
+                extra_context={
+                    "operation": "get_existing_starboard_message",
+                    "original_message_id": str(original_message.id)
+                    if original_message
+                    else None,
+                },
+            )
 
         return None
 
@@ -357,6 +381,15 @@ async def create_or_update_starboard_message(
 
         except Exception as e:
             logger.error(f"Error while creating or updating starboard message: {e}")
+            capture_exception_safe(
+                e,
+                extra_context={
+                    "operation": "create_or_update_starboard_message",
+                    "channel_id": str(starboard_channel.id)
+                    if starboard_channel
+                    else None,
+                },
+            )
 
     async def handle_starboard_reaction(
         self,
@@ -415,6 +448,14 @@ async def handle_starboard_reaction(
 
         except Exception as e:
             logger.error(f"Unexpected error in handle_starboard_reaction: {e}")
+            capture_exception_safe(
+                e,
+                extra_context={
+                    "operation": "handle_starboard_reaction",
+                    "guild_id": str(payload.guild_id) if payload.guild_id else None,
+                    "message_id": str(payload.message_id),
+                },
+            )
 
     async def handle_reaction_clear(
         self,
@@ -462,6 +503,14 @@ async def handle_reaction_clear(
 
         except Exception as e:
             logger.error(f"Error in handle_reaction_clear: {e}")
+            capture_exception_safe(
+                e,
+                extra_context={
+                    "operation": "handle_reaction_clear",
+                    "guild_id": str(payload.guild_id) if payload.guild_id else None,
+                    "message_id": str(payload.message_id),
+                },
+            )
 
 
 async def setup(bot: Tux) -> None:

src/tux/modules/features/status_roles.py

@@ -13,6 +13,7 @@
 
 from tux.core.base_cog import BaseCog
 from tux.core.bot import Tux
+from tux.services.sentry import capture_exception_safe
 from tux.shared.config import CONFIG
 
 
@@ -154,15 +155,28 @@ async def check_and_update_roles(self, member: discord.Member):
                     await member.remove_roles(role)
 
             except re.error:
-                logger.exception(
+                # Configuration error - don't send to Sentry
+                logger.warning(
                     f"Invalid regex pattern '{pattern}' in STATUS_ROLES config",
                 )
             except discord.Forbidden:
-                logger.exception(
+                # User error (permission denied) - don't send to Sentry
+                logger.warning(
                     f"Bot lacks permission to modify roles for {member.display_name} in {member.guild.name}",
                 )
-            except Exception:
-                logger.exception(f"Error updating roles for {member.display_name}")
+            except Exception as e:
+                # Unexpected error - send to Sentry
+                logger.error(f"Error updating roles for {member.display_name}: {e}")
+
+                capture_exception_safe(
+                    e,
+                    extra_context={
+                        "operation": "update_status_roles",
+                        "member_id": str(member.id),
+                        "guild_id": str(member.guild.id),
+                        "pattern": pattern,
+                    },
+                )
 
 
 async def setup(bot: Tux) -> None:

src/tux/modules/moderation/cases.py

@@ -513,7 +513,9 @@ async def _resolve_user(self, user_id: int) -> discord.User | MockUser:
             logger.warning(f"Could not find user with ID {user_id}")
             return MockUser(user_id)
         except Exception as e:
-            logger.exception(f"Error resolving user with ID {user_id}: {e}")
+            # Graceful fallback - don't use exception() level for expected fallback behavior
+            logger.warning(f"Error resolving user with ID {user_id}: {e}")
+            # Don't send to Sentry - this is expected fallback behavior
             return MockUser(user_id)
 
     async def _resolve_moderator(self, moderator_id: int) -> discord.User | MockUser:

src/tux/modules/tools/tldr.py

@@ -17,6 +17,7 @@
 from tux.core.base_cog import BaseCog
 from tux.core.bot import Tux
 from tux.core.flags import TldrFlags
+from tux.services.sentry import capture_exception_safe
 from tux.services.wrappers.tldr import SUPPORTED_PLATFORMS, TldrClient
 from tux.shared.functions import generate_usage
 from tux.ui.embeds import EmbedCreator
@@ -92,6 +93,17 @@ async def _initialize_cache_async(self):
                             f"TLDR Cog: Exception during cache update for '{lang_code}': {e}",
                             exc_info=True,
                         )
+                        # Capture exception with context for Sentry
+                        capture_exception_safe(
+                            e,
+                            extra_context={
+                                "tldr_cache_update": {
+                                    "language": lang_code,
+                                    "operation": "cache_update",
+                                    "cache_needs_update": True,
+                                },
+                            },
+                        )
                 else:
                     logger.debug(
                         f"TLDR Cog: Cache for '{lang_code}' is recent, skipping update.",

src/tux/modules/utility/wiki.py

@@ -12,6 +12,7 @@
 from tux.core.base_cog import BaseCog
 from tux.core.bot import Tux
 from tux.services.http_client import http_client
+from tux.services.sentry import capture_api_error
 from tux.ui.embeds import EmbedCreator
 
 
@@ -116,6 +117,8 @@ async def query_wiki(self, base_url: str, search_term: str) -> tuple[str, str]:
                     return title, url
         except Exception as e:
             logger.error(f"Wiki API request failed: {e}")
+
+            capture_api_error(e, endpoint="wiki_api")
             return "error", "error"
 
         return "error", "error"