Fixed exception occur if commands are added without bot

Author: eunwoo1104

discord_slash/client.py

@@ -4,6 +4,7 @@
 from discord.ext import commands
 from . import http
 from . import model
+from .utils import manage_commands
 
 
 class SlashCommand:
@@ -12,25 +13,29 @@ class SlashCommand:
 
     :param client: discord.py Bot class. Although it accepts :class:`discord.Client` at init, using it is not allowed since :class:`discord.Client` doesn't support :func:`add_listener`.
     :type client: Union[discord.Client, discord.ext.commands.Bot]
+    :param auto_register: Whether to register commands automatically. Default `False`.
+    :type auto_register: bool
 
     :ivar _discord: Discord client of this client.
     :ivar commands: Dictionary of the registered commands via :func:`.slash` decorator.
     :ivar req: :class:`.http.SlashCommandRequest` of this client.
     :ivar logger: Logger of this client.
+    :ivar auto_register: Whether to register commands automatically.
     """
     def __init__(self,
-                 client: typing.Union[discord.Client,
-                                      commands.Bot]
+                 client: typing.Union[discord.Client, commands.Bot],
+                 auto_register: bool = False
                  ):
         if isinstance(client, discord.Client) and not isinstance(client, commands.Bot):
             raise Exception("Currently only commands.Bot is supported.")
         self._discord = client
         self.commands = {}
         self.req = http.SlashCommandRequest()
         self.logger = logging.getLogger("discord_slash")
+        self.auto_register = auto_register
         self._discord.add_listener(self.on_socket_response)
 
-    def slash(self, name=None, auto_convert: dict = None):
+    def slash(self, name: str = None, description: str = None, auto_convert: dict = None, guild_id: int = None):
         """
         Decorator that registers coroutine as a slash command.\n
         1 arg is required for ctx(:class:`.model.SlashContext`), and if your slash command has some args, then those args are also required.\n
@@ -63,13 +68,22 @@ async def _pick(ctx, choice1, choice2): # Command with 1 or more args.
              "option_user": 6,            # Also can use number for type
              "option_channel": "CHANNEL"} # and all upper case.
 
-        :param name: Name of the slash command.
-        :param auto_convert: Dictionary of how to convert option values.
-        :type auto_convert: dict
+        :param name: Name of the slash command. Default name of the coroutine.
+        :param description: Description of the slash command. Default `None`.
+        :param auto_convert: Dictionary of how to convert option values. Default `None`.
+        :param guild_id: Guild ID of where the command will be used. Default `None`, which will be global command.
         """
         def wrapper(cmd):
             self.commands[cmd.__name__ if not name else name] = [cmd, auto_convert]
             self.logger.debug(f"Added command `{cmd.__name__ if not name else name}`")
+            """
+            if self.auto_register:
+                manage_commands.add_slash_command(self._discord.user.id,
+                                                  self._discord.http.token,
+                                                  guild_id,
+                                                  cmd.__name__ if not name else name,
+                                                  description)
+            """
             return cmd
         return wrapper
 
@@ -85,6 +99,9 @@ def process_options(self, guild: discord.Guild, options: list, auto_convert: dic
         :type auto_convert: dict
         :return: list
         """
+        if not guild:
+            self.logger.info("This command invoke is missing guild. Skipping option process.")
+            return [x["value"] for x in options]
         converters = [guild.get_member, guild.get_role, guild.get_role]
         types = {
             "user": 0,
@@ -126,4 +143,5 @@ async def on_socket_response(self, msg):
             selected_cmd = self.commands[to_use["data"]["name"]]
             ctx = model.SlashContext(self.req, to_use, self._discord)
             args = self.process_options(ctx.guild, to_use["data"]["options"], selected_cmd[1]) if "options" in to_use["data"] else []
+            self.logger.debug(f"Command {to_use['data']['name']} invoked.")
             await selected_cmd[0](ctx, *args)

discord_slash/model.py

@@ -27,8 +27,8 @@ def __init__(self,
         self.command_id = _json["data"]["id"]
         self._http = _http
         self.guild: discord.Guild = _discord.get_guild(int(_json["guild_id"]))
-        self.author: discord.Member = self.guild.get_member(int(_json["member"]["user"]["id"]))
-        self.channel = self.guild.get_channel(int(_json["channel_id"]))
+        self.author: discord.Member = self.guild.get_member(int(_json["member"]["user"]["id"])) if self.guild else None
+        self.channel = self.guild.get_channel(int(_json["channel_id"])) if self.guild else None
 
     async def send(self,
                    send_type: int = 4,