disagreement 0.1.0rc2__tar.gz → 0.1.0rc3__tar.gz

disagreement-0.1.0rc3/MANIFEST.in

@@ -0,0 +1,4 @@
+graft docs
+graft examples
+include LICENSE
+include disagreement/py.typed

{disagreement-0.1.0rc2/disagreement.egg-info → disagreement-0.1.0rc3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: disagreement
-Version: 0.1.0rc2
+Version: 0.1.0rc3
 Summary: A Python library for the Discord API.
 Author-email: Slipstream <me@slipstreamm.dev>
 License: BSD 3-Clause

{disagreement-0.1.0rc2 → disagreement-0.1.0rc3}/disagreement/__init__.py

@@ -14,7 +14,7 @@ __title__ = "disagreement"
 __author__ = "Slipstream"
 __license__ = "BSD 3-Clause License"
 __copyright__ = "Copyright 2025 Slipstream"
-__version__ = "0.1.0rc2"
+__version__ = "0.1.0rc3"
 
 from .client import Client, AutoShardedClient
 from .models import Message, User, Reaction
@@ -35,7 +35,11 @@ from .enums import GatewayIntent, GatewayOpcode  # Export enums
 from .error_handler import setup_global_error_handler
 from .hybrid_context import HybridContext
 from .ext import tasks
+from .logging_config import setup_logging
 
-# Set up logging if desired
-# import logging
-# logging.getLogger(__name__).addHandler(logging.NullHandler())
+import logging
+
+
+# Configure a default logger if none has been configured yet
+if not logging.getLogger().hasHandlers():
+    setup_logging(logging.INFO)

{disagreement-0.1.0rc2 → disagreement-0.1.0rc3}/disagreement/client.py

@@ -123,14 +123,10 @@ class Client:
 
         self._closed: bool = False
         self._ready_event: asyncio.Event = asyncio.Event()
-        self.application_id: Optional[Snowflake] = None  # For Application Commands
         self.user: Optional["User"] = (
             None  # The bot's own user object, populated on READY
         )
 
-        # Initialize AppCommandHandler
-        self.app_command_handler: AppCommandHandler = AppCommandHandler(client=self)
-
         # Internal Caches
         self._guilds: Dict[Snowflake, "Guild"] = {}
         self._channels: Dict[Snowflake, "Channel"] = (

{disagreement-0.1.0rc2 → disagreement-0.1.0rc3}/disagreement/ext/app_commands/handler.py

@@ -1,6 +1,7 @@
 # disagreement/ext/app_commands/handler.py
 
 import inspect
+import logging
 from typing import (
     TYPE_CHECKING,
     Dict,
@@ -64,6 +65,9 @@ if not TYPE_CHECKING:
     Message = Any
 
 
+logger = logging.getLogger(__name__)
+
+
 class AppCommandHandler:
     """
     Manages application command registration, parsing, and dispatching.
@@ -544,7 +548,7 @@ class AppCommandHandler:
                 await command.invoke(ctx, *parsed_args, **parsed_kwargs)
 
         except Exception as e:
-            print(f"Error invoking app command '{command.name}': {e}")
+            logger.error("Error invoking app command '%s': %s", command.name, e)
             await self.dispatch_app_command_error(ctx, e)
             # else:
             #     # Default error reply if no handler on client
@@ -594,34 +598,43 @@ class AppCommandHandler:
                 payload = cmd_or_group.to_dict()
                 commands_to_sync.append(payload)
             except AttributeError:
-                print(
-                    f"Warning: Command or group '{cmd_or_group.name}' does not have a to_dict() method. Skipping."
+                logger.warning(
+                    "Command or group '%s' does not have a to_dict() method. Skipping.",
+                    cmd_or_group.name,
                 )
             except Exception as e:
-                print(
-                    f"Error converting command/group '{cmd_or_group.name}' to dict: {e}. Skipping."
+                logger.error(
+                    "Error converting command/group '%s' to dict: %s. Skipping.",
+                    cmd_or_group.name,
+                    e,
                 )
 
         if not commands_to_sync:
-            print(
-                f"No commands to sync for {'guild ' + str(guild_id) if guild_id else 'global'} scope."
+            logger.info(
+                "No commands to sync for %s scope.",
+                f"guild {guild_id}" if guild_id else "global",
             )
             return
 
         try:
             if guild_id:
-                print(
-                    f"Syncing {len(commands_to_sync)} commands for guild {guild_id}..."
+                logger.info(
+                    "Syncing %s commands for guild %s...",
+                    len(commands_to_sync),
+                    guild_id,
                 )
                 await self.client._http.bulk_overwrite_guild_application_commands(
                     application_id, guild_id, commands_to_sync
                 )
             else:
-                print(f"Syncing {len(commands_to_sync)} global commands...")
+                logger.info(
+                    "Syncing %s global commands...",
+                    len(commands_to_sync),
+                )
                 await self.client._http.bulk_overwrite_global_application_commands(
                     application_id, commands_to_sync
                 )
-            print("Command sync successful.")
+            logger.info("Command sync successful.")
         except Exception as e:
-            print(f"Error syncing application commands: {e}")
+            logger.error("Error syncing application commands: %s", e)
             # Consider re-raising or specific error handling

{disagreement-0.1.0rc2 → disagreement-0.1.0rc3}/disagreement/ext/app_commands/hybrid.py

@@ -58,4 +58,4 @@ class HybridCommand(SlashCommand, PrefixCommand):  # Inherit from both
     # The correct one will be called depending on how the command is dispatched.
     # The AppCommandHandler will use AppCommand.invoke (via SlashCommand).
     # The prefix CommandHandler will use PrefixCommand.invoke.
-    # This seems acceptable.
+    # This seems acceptable.

{disagreement-0.1.0rc2 → disagreement-0.1.0rc3}/disagreement/ext/commands/cog.py

@@ -1,6 +1,7 @@
 # disagreement/ext/commands/cog.py
 
 import inspect
+import logging
 from typing import TYPE_CHECKING, List, Tuple, Callable, Awaitable, Any, Dict, Union
 
 if TYPE_CHECKING:
@@ -16,6 +17,8 @@ else:  # pragma: no cover - runtime imports for isinstance checks
     # EventDispatcher might be needed if cogs register listeners directly
     # from disagreement.event_dispatcher import EventDispatcher
 
+logger = logging.getLogger(__name__)
+
 
 class Cog:
     """
@@ -59,8 +62,10 @@ class Cog:
                 cmd.cog = self  # Assign the cog instance to the command
                 if cmd.name in self._commands:
                     # This should ideally be caught earlier or handled by CommandHandler
-                    print(
-                        f"Warning: Duplicate command name '{cmd.name}' in cog '{self.cog_name}'. Overwriting."
+                    logger.warning(
+                        "Duplicate command name '%s' in cog '%s'. Overwriting.",
+                        cmd.name,
+                        self.cog_name,
                     )
                 self._commands[cmd.name.lower()] = cmd
                 # Also register aliases
@@ -79,8 +84,10 @@ class Cog:
                     # For AppCommandGroup, its commands will have cog set individually if they are AppCommands
                     self._app_commands_and_groups.append(app_cmd_obj)
                 else:
-                    print(
-                        f"Warning: Member '{member_name}' in cog '{self.cog_name}' has '__app_command_object__' but it's not an AppCommand or AppCommandGroup."
+                    logger.warning(
+                        "Member '%s' in cog '%s' has '__app_command_object__' but it's not an AppCommand or AppCommandGroup.",
+                        member_name,
+                        self.cog_name,
                     )
 
             elif isinstance(member, (AppCommand, AppCommandGroup)):
@@ -92,8 +99,10 @@ class Cog:
                 # This is a method decorated with @commands.Cog.listener or @commands.listener
                 if not inspect.iscoroutinefunction(member):
                     # Decorator should have caught this, but double check
-                    print(
-                        f"Warning: Listener '{member_name}' in cog '{self.cog_name}' is not a coroutine. Skipping."
+                    logger.warning(
+                        "Listener '%s' in cog '%s' is not a coroutine. Skipping.",
+                        member_name,
+                        self.cog_name,
                     )
                     continue
 

{disagreement-0.1.0rc2 → disagreement-0.1.0rc3}/disagreement/ext/commands/core.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import asyncio
+import logging
 import inspect
 from typing import (
     TYPE_CHECKING,
@@ -31,6 +32,8 @@ from .errors import (
 from .converters import run_converters, DEFAULT_CONVERTERS, Converter
 from disagreement.typing import Typing
 
+logger = logging.getLogger(__name__)
+
 if TYPE_CHECKING:
     from .cog import Cog
     from disagreement.client import Client
@@ -224,8 +227,10 @@ class CommandHandler:
         self.commands[command.name.lower()] = command
         for alias in command.aliases:
             if alias in self.commands:
-                print(
-                    f"Warning: Alias '{alias}' for command '{command.name}' conflicts with an existing command or alias."
+                logger.warning(
+                    "Alias '%s' for command '%s' conflicts with an existing command or alias.",
+                    alias,
+                    command.name,
                 )
             self.commands[alias.lower()] = command
 
@@ -241,6 +246,7 @@ class CommandHandler:
 
     def add_cog(self, cog_to_add: "Cog") -> None:
         from .cog import Cog
+
         if not isinstance(cog_to_add, Cog):
             raise TypeError("Argument must be a subclass of Cog.")
 
@@ -258,8 +264,9 @@ class CommandHandler:
             for event_name, callback in cog_to_add.get_listeners():
                 self.client._event_dispatcher.register(event_name.upper(), callback)
         else:
-            print(
-                f"Warning: Client does not have '_event_dispatcher'. Listeners for cog '{cog_to_add.cog_name}' not registered."
+            logger.warning(
+                "Client does not have '_event_dispatcher'. Listeners for cog '%s' not registered.",
+                cog_to_add.cog_name,
             )
 
         if hasattr(cog_to_add, "cog_load") and inspect.iscoroutinefunction(
@@ -267,7 +274,7 @@ class CommandHandler:
         ):
             asyncio.create_task(cog_to_add.cog_load())
 
-        print(f"Cog '{cog_to_add.cog_name}' added.")
+        logger.info("Cog '%s' added.", cog_to_add.cog_name)
 
     def remove_cog(self, cog_name: str) -> Optional["Cog"]:
         cog_to_remove = self.cogs.pop(cog_name, None)
@@ -277,8 +284,11 @@ class CommandHandler:
 
             if hasattr(self.client, "_event_dispatcher"):
                 for event_name, callback in cog_to_remove.get_listeners():
-                    print(
-                        f"Note: Listener '{callback.__name__}' for event '{event_name}' from cog '{cog_name}' needs manual unregistration logic in EventDispatcher."
+                    logger.debug(
+                        "Listener '%s' for event '%s' from cog '%s' needs manual unregistration logic in EventDispatcher.",
+                        callback.__name__,
+                        event_name,
+                        cog_name,
                     )
 
             if hasattr(cog_to_remove, "cog_unload") and inspect.iscoroutinefunction(
@@ -287,7 +297,7 @@ class CommandHandler:
                 asyncio.create_task(cog_to_remove.cog_unload())
 
             cog_to_remove._eject()
-            print(f"Cog '{cog_name}' removed.")
+            logger.info("Cog '%s' removed.", cog_name)
         return cog_to_remove
 
     async def get_prefix(self, message: "Message") -> Union[str, List[str], None]:
@@ -493,11 +503,11 @@ class CommandHandler:
             ctx.kwargs = parsed_kwargs
             await command.invoke(ctx, *parsed_args, **parsed_kwargs)
         except CommandError as e:
-            print(f"Command error for '{command.name}': {e}")
+            logger.error("Command error for '%s': %s", command.name, e)
             if hasattr(self.client, "on_command_error"):
                 await self.client.on_command_error(ctx, e)
         except Exception as e:
-            print(f"Unexpected error invoking command '{command.name}': {e}")
+            logger.error("Unexpected error invoking command '%s': %s", command.name, e)
             exc = CommandInvokeError(e)
             if hasattr(self.client, "on_command_error"):
                 await self.client.on_command_error(ctx, exc)