ext.bridge improvements

discord/bot.py

@@ -644,8 +644,6 @@ async def sync_commands(
                     guild_commands, guild_id=guild_id, method=method, force=force, delete_existing=delete_existing
                 )
 
-        global_permissions: List = []
-
         for i in registered_commands:
             cmd = get(
                 self.pending_application_commands,

discord/commands/core.py

@@ -617,7 +617,7 @@ def __new__(cls, *args, **kwargs) -> SlashCommand:
         self.__original_kwargs__ = kwargs.copy()
         return self
 
-    def __init__(self, func: Callable, *args, **kwargs) -> None:
+    def __init__(self, func: Callable, **kwargs) -> None:
         super().__init__(func, **kwargs)
         if not asyncio.iscoroutinefunction(func):
             raise TypeError("Callback must be a coroutine.")
@@ -1010,10 +1010,10 @@ def __init__(
         parent: Optional[SlashCommandGroup] = None,
         **kwargs,
     ) -> None:
-        validate_chat_input_name(name)
-        validate_chat_input_description(description)
-        self.name = str(name)
-        self.description = description
+        self.name = name
+        self.description = description or "No description provided"
+        validate_chat_input_name(self.name)
+        validate_chat_input_description(self.description)
         self.input_type = SlashCommandOptionType.sub_command_group
         self.subcommands: List[Union[SlashCommand, SlashCommandGroup]] = self.__initial_commands__
         self.guild_ids = guild_ids

discord/ext/bridge/bot.py

@@ -30,7 +30,7 @@
 from ..commands import AutoShardedBot as ExtAutoShardedBot
 from ..commands import Bot as ExtBot
 from .context import BridgeApplicationContext, BridgeExtContext
-from .core import BridgeCommand, bridge_command
+from .core import BridgeCommand, BridgeCommandGroup, bridge_command, bridge_group
 
 __all__ = ("Bot", "AutoShardedBot")
 
@@ -75,6 +75,22 @@ def decorator(func) -> BridgeCommand:
 
         return decorator
 
+    def bridge_group(self, **kwargs):
+        """A decorator that is used to wrap a function as a bridge command group.
+
+        Parameters
+        ----------
+        kwargs: Optional[Dict[:class:`str`, Any]]
+            Keyword arguments that are directly passed to the respective command constructors. (:class:`.SlashCommandGroup` and :class:`.ext.commands.Group`)
+        """
+
+        def decorator(func) -> BridgeCommandGroup:
+            result = bridge_group(**kwargs)(func)
+            self.add_bridge_command(result)
+            return result
+
+        return decorator
+
 
 class Bot(BotBase, ExtBot):
     """Represents a discord bot, with support for cross-compatibility between command types.

discord/ext/bridge/context.py

@@ -23,7 +23,7 @@
 DEALINGS IN THE SOFTWARE.
 """
 from abc import ABC, abstractmethod
-from typing import TYPE_CHECKING, Any, Optional, Union
+from typing import Any, Optional, Union
 
 from discord.commands import ApplicationContext
 from discord.interactions import Interaction, InteractionMessage
@@ -50,10 +50,10 @@ class BridgeContext(ABC):
 
         @bot.bridge_command()
         async def example(ctx: BridgeContext):
-            if isinstance(ctx, BridgeExtContext):
-                command_type = "Traditional (prefix-based) command"
-            elif isinstance(ctx, BridgeApplicationContext):
+            if ctx.is_app:
                 command_type = "Application command"
+            else:
+                command_type = "Traditional (prefix-based) command"
             await ctx.send(f"This command was invoked with a(n) {command_type}.")
 
     .. versionadded:: 2.0
@@ -140,6 +140,10 @@ async def _defer(self, *args, **kwargs) -> None:
     async def _edit(self, *args, **kwargs) -> InteractionMessage:
         return await self._get_super("edit")(*args, **kwargs)
 
+    @property
+    def is_app(self) -> bool:
+        return True
+
 
 class BridgeExtContext(BridgeContext, Context):
     """
@@ -184,3 +188,7 @@ async def delete(
         """
         if self._original_response_message:
             await self._original_response_message.delete(delay=delay, reason=reason)
+
+    @property
+    def is_app(self) -> bool:
+        return False