fix: The issue of multiple client unable to load the commands from the same extension (#1134)

* fix the multiple client mode

* Update bot.py

* Remove some IDE format accident

* fix: sys.module deleting modules twice

* Update interactions/client/bot.py

Co-authored-by: Damego <[email protected]>

Co-authored-by: DeltaX <[email protected]>
Co-authored-by: Damego <[email protected]>

Author: 3 people

interactions/client/bot.py

@@ -535,7 +535,7 @@ def __resolve_commands(self) -> None:
             This is an internal method. Do not call it unless you know what you are doing!
         """
         for cmd in self._commands:
-            if cmd.resolved:
+            if cmd.coro.__qualname__ in [_cmd.__qualname__ for _cmd in self.__command_coroutines]:
                 continue
 
             cmd.listener = self._websocket._dispatch
@@ -581,7 +581,6 @@ def __resolve_commands(self) -> None:
                     self._scopes.add(cmd.scope if isinstance(cmd.scope, int) else cmd.scope.id)
 
             self.event(coro, name=f"command_{cmd.name}")
-            cmd.resolved = True
 
     async def __sync(self) -> None:  # sourcery no-metrics
         """
@@ -1413,6 +1412,7 @@ def load(
         else:
             log.debug(f"Loaded extension {name}.")
             self._extensions[_name] = module
+            del sys.modules[name]
             return extension
 
     def remove(
@@ -1450,7 +1450,6 @@ def remove(
                         self._loop.create_task(
                             _extension.teardown(remove_commands=remove_commands)
                         )  # made for Extension, usable by others
-            del sys.modules[_name]
 
         else:
             with contextlib.suppress(AttributeError):

interactions/client/models/command.py

@@ -411,7 +411,6 @@ class Command(DictSerializerMixin):
     :ivar Dict[str, int] num_options: The dictionary of the number of options per subcommand.
     :ivar Dict[str, Union[Callable[..., Awaitable], str]] autocompletions: The dictionary of autocompletions for the command.
     :ivar Optional[str] recent_group: The name of the group most recently utilized.
-    :ivar bool resolved: Whether the command is synced. Defaults to ``False``.
     :ivar Optional[Extension] extension: The extension that the command belongs to, if any.
     :ivar Client client: The client that the command belongs to.
     :ivar Optional[Listener] listener: The listener, used for dispatching command errors.
@@ -436,7 +435,6 @@ class Command(DictSerializerMixin):
     )
     recent_group: Optional[str] = field(default=None, init=False)
     error_callback: Optional[Callable[..., Awaitable]] = field(default=None, init=False)
-    resolved: bool = field(default=False, init=False)
     extension: Optional["Extension"] = field(default=None, init=False)
     client: "Client" = field(default=None, init=False)
     listener: Optional["Listener"] = field(default=None, init=False)