fix: parse sub cmd (and groups) options properly (#835)

* fix: parse cub cmd (and groups) options properly

* refactor: don't unnecessarily convert options

* refactor: change around option check a little

Author: AstreaTSS

interactions/client/bot.py

@@ -668,7 +668,7 @@ def __check_sub_group(_sub_group: Option):
                     11, message="A sub command group cannot contain more than 25 sub commands!"
                 )
             for _sub_command in _sub_group.options:
-                __check_sub_command(Option(**_sub_command), _sub_group)
+                __check_sub_command(_sub_command, _sub_group)
 
         def __check_sub_command(_sub_command: Option, _sub_group: Option = MISSING):
             nonlocal _sub_cmds_present
@@ -704,7 +704,7 @@ def __check_sub_command(_sub_command: Option, _sub_group: Option = MISSING):
                     )
                 _sub_opt_names = []
                 for _opt in _sub_command.options:
-                    __check_options(Option(**_opt), _sub_opt_names, _sub_command)
+                    __check_options(_opt, _sub_opt_names, _sub_command)
                 del _sub_opt_names
 
         def __check_options(_option: Option, _names: list, _sub_command: Option = MISSING):

interactions/client/models/command.py

@@ -114,7 +114,9 @@ class Option(DictSerializerMixin):
     def __attrs_post_init__(self):
         # needed for nested classes
         self.options = (
-            [Option(**option) for option in self.options] if self.options is not None else None
+            [Option(**option) if isinstance(option, dict) else option for option in self.options]
+            if self.options is not None
+            else None
         )