chore: rename __repr_info__ -> __repr_attributes__, enable RUF005 (DisnakeDev#1320)

Signed-off-by: vi <8530778+shiftinv@users.noreply.github.com>
Co-authored-by: Eneg <42005170+Enegg@users.noreply.github.com>

Author: shiftinv

disnake/app_commands.py

@@ -506,7 +506,7 @@ class ApplicationCommand(ABC):  # noqa: B024  # this will get refactored eventua
         .. versionadded:: 2.10
     """
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = (
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
         "type",
         "name",
         "default_member_permissions",
@@ -614,7 +614,7 @@ def dm_permission(self, value: bool) -> None:
         self._dm_permission = value
 
     def __repr__(self) -> str:
-        attrs = " ".join(f"{key}={getattr(self, key)!r}" for key in self.__repr_info__)
+        attrs = " ".join(f"{key}={getattr(self, key)!r}" for key in self.__repr_attributes__)
         return f"<{type(self).__name__} {attrs}>"
 
     def __str__(self) -> str:
@@ -726,7 +726,7 @@ def localize(self, store: LocalizationProtocol) -> None:
 
 
 class _APIApplicationCommandMixin:
-    __repr_info__ = ("id",)
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = ("id",)
 
     def _update_common(self, data: ApplicationCommandPayload) -> None:
         if not isinstance(self, ApplicationCommand):
@@ -776,7 +776,9 @@ class UserCommand(ApplicationCommand):
         .. versionadded:: 2.10
     """
 
-    __repr_info__ = tuple(n for n in ApplicationCommand.__repr_info__ if n != "type")
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = tuple(
+        n for n in ApplicationCommand.__repr_attributes__ if n != "type"
+    )
 
     def __init__(
         self,
@@ -840,7 +842,10 @@ class APIUserCommand(UserCommand, _APIApplicationCommandMixin):
         Autoincrementing version identifier updated during substantial record changes.
     """
 
-    __repr_info__ = UserCommand.__repr_info__ + _APIApplicationCommandMixin.__repr_info__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
+        *UserCommand.__repr_attributes__,
+        *_APIApplicationCommandMixin.__repr_attributes__,
+    )
 
     @classmethod
     def from_dict(cls, data: ApplicationCommandPayload) -> Self:
@@ -899,7 +904,9 @@ class MessageCommand(ApplicationCommand):
         .. versionadded:: 2.10
     """
 
-    __repr_info__ = tuple(n for n in ApplicationCommand.__repr_info__ if n != "type")
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = tuple(
+        n for n in ApplicationCommand.__repr_attributes__ if n != "type"
+    )
 
     def __init__(
         self,
@@ -963,7 +970,10 @@ class APIMessageCommand(MessageCommand, _APIApplicationCommandMixin):
         Autoincrementing version identifier updated during substantial record changes.
     """
 
-    __repr_info__ = MessageCommand.__repr_info__ + _APIApplicationCommandMixin.__repr_info__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
+        *MessageCommand.__repr_attributes__,
+        *_APIApplicationCommandMixin.__repr_attributes__,
+    )
 
     @classmethod
     def from_dict(cls, data: ApplicationCommandPayload) -> Self:
@@ -1032,7 +1042,8 @@ class SlashCommand(ApplicationCommand):
         The list of options the slash command has.
     """
 
-    __repr_info__ = tuple(n for n in ApplicationCommand.__repr_info__ if n != "type") + (
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
+        *tuple(n for n in ApplicationCommand.__repr_attributes__ if n != "type"),
         "description",
         "options",
     )
@@ -1179,7 +1190,10 @@ class APISlashCommand(SlashCommand, _APIApplicationCommandMixin):
         Autoincrementing version identifier updated during substantial record changes.
     """
 
-    __repr_info__ = SlashCommand.__repr_info__ + _APIApplicationCommandMixin.__repr_info__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
+        *SlashCommand.__repr_attributes__,
+        *_APIApplicationCommandMixin.__repr_attributes__,
+    )
 
     @classmethod
     def from_dict(cls, data: ApplicationCommandPayload) -> Self:

disnake/components.py

@@ -185,7 +185,7 @@ class Component:
 
     __slots__: Tuple[str, ...] = ("type", "id")
 
-    __repr_info__: ClassVar[Tuple[str, ...]]
+    __repr_attributes__: ClassVar[Tuple[str, ...]]
 
     # subclasses are expected to overwrite this if they're only usable with `MessageFlags.is_components_v2`
     is_v2: ClassVar[bool] = False
@@ -194,7 +194,7 @@ class Component:
     id: int
 
     def __repr__(self) -> str:
-        attrs = " ".join(f"{key}={getattr(self, key)!r}" for key in self.__repr_info__)
+        attrs = " ".join(f"{key}={getattr(self, key)!r}" for key in self.__repr_attributes__)
         return f"<{self.__class__.__name__} {attrs}>"
 
     @classmethod
@@ -236,7 +236,7 @@ class ActionRow(Component, Generic[ActionRowChildComponentT]):
 
     __slots__: Tuple[str, ...] = ("children",)
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     def __init__(self, data: ActionRowPayload) -> None:
         self.type: Literal[ComponentType.action_row] = ComponentType.action_row
@@ -303,7 +303,7 @@ class Button(Component):
         "sku_id",
     )
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     def __init__(self, data: ButtonComponentPayload) -> None:
         self.type: Literal[ComponentType.button] = ComponentType.button
@@ -404,7 +404,9 @@ class BaseSelectMenu(Component):
     )
 
     # FIXME: this isn't pretty; we should decouple __repr__ from slots
-    __repr_info__: ClassVar[Tuple[str, ...]] = tuple(s for s in __slots__ if s != "default_values")
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = tuple(
+        s for s in __slots__ if s != "default_values"
+    )
 
     # n.b: ideally this would be `BaseSelectMenuPayload`,
     # but pyright made TypedDict keys invariant and doesn't
@@ -480,7 +482,10 @@ class StringSelectMenu(BaseSelectMenu):
 
     __slots__: Tuple[str, ...] = ("options",)
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = BaseSelectMenu.__repr_info__ + __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
+        *BaseSelectMenu.__repr_attributes__,
+        *__slots__,
+    )
     type: Literal[ComponentType.string_select]
 
     def __init__(self, data: StringSelectMenuPayload) -> None:
@@ -677,7 +682,10 @@ class ChannelSelectMenu(BaseSelectMenu):
 
     __slots__: Tuple[str, ...] = ("channel_types",)
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = BaseSelectMenu.__repr_info__ + __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
+        *BaseSelectMenu.__repr_attributes__,
+        *__slots__,
+    )
     type: Literal[ComponentType.channel_select]
 
     def __init__(self, data: ChannelSelectMenuPayload) -> None:
@@ -884,7 +892,7 @@ class TextInput(Component):
         "min_length",
     )
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     def __init__(self, data: TextInputPayload) -> None:
         self.type: Literal[ComponentType.text_input] = ComponentType.text_input
@@ -953,7 +961,7 @@ class Section(Component):
 
     __slots__: Tuple[str, ...] = ("children", "accessory")
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     is_v2 = True
 
@@ -1000,7 +1008,7 @@ class TextDisplay(Component):
 
     __slots__: Tuple[str, ...] = ("content",)
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     is_v2 = True
 
@@ -1112,7 +1120,7 @@ class Thumbnail(Component):
         "spoiler",
     )
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     is_v2 = True
 
@@ -1163,7 +1171,7 @@ class MediaGallery(Component):
 
     __slots__: Tuple[str, ...] = ("items",)
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     is_v2 = True
 
@@ -1271,7 +1279,7 @@ class FileComponent(Component):
 
     __slots__: Tuple[str, ...] = ("file", "spoiler", "name", "size")
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     is_v2 = True
 
@@ -1322,7 +1330,7 @@ class Separator(Component):
 
     __slots__: Tuple[str, ...] = ("divider", "spacing")
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     is_v2 = True
 
@@ -1375,7 +1383,7 @@ class Container(Component):
         "spoiler",
     )
 
-    __repr_info__: ClassVar[Tuple[str, ...]] = __slots__
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = __slots__
 
     is_v2 = True
 

disnake/ext/commands/ctx_menus_core.py

@@ -136,7 +136,7 @@ async def __call__(
         **kwargs: Any,
     ) -> None:
         # the target may just not be passed in
-        args = (target or interaction.target,) + args
+        args = (target or interaction.target, *args)
         if self.cog is not None:
             await safe_call(self.callback, self.cog, interaction, *args, **kwargs)
         else:
@@ -248,7 +248,7 @@ async def __call__(
         **kwargs: Any,
     ) -> None:
         # the target may just not be passed in
-        args = (target or interaction.target,) + args
+        args = (target or interaction.target, *args)
         if self.cog is not None:
             await safe_call(self.callback, self.cog, interaction, *args, **kwargs)
         else:

disnake/ext/commands/params.py

@@ -292,7 +292,7 @@ def __class_getitem__(cls, params: Tuple[Any, ...]) -> Self:
             )
 
             # infer type from min/max values
-            params = (cls._infer_type(params),) + params
+            params = (cls._infer_type(params), *params)
 
         if len(params) != 3:
             raise TypeError(

disnake/interactions/application_command.py

@@ -282,7 +282,7 @@ def _get_chain_and_kwargs(
         for option in self.options:
             if option.value is None:
                 # Extend the chain and collect kwargs in the nesting
-                return option._get_chain_and_kwargs(chain + (option.name,))
+                return option._get_chain_and_kwargs((*chain, option.name))
             return chain, {o.name: o.value for o in self.options}
         return chain, {}
 
@@ -367,7 +367,7 @@ def _get_chain_and_kwargs(
         for option in self.options:
             if option.value is None:
                 # Extend the chain and collect kwargs in the nesting
-                return option._get_chain_and_kwargs(chain + (option.name,))
+                return option._get_chain_and_kwargs((*chain, option.name))
             return chain, {o.name: o.value for o in self.options}
         return chain, {}
 

disnake/state.py

@@ -2035,7 +2035,7 @@ def parse_guild_soundboard_sound_create(self, data: gateway.GuildSoundboardSound
         self._handle_soundboard_update(
             guild,
             # append new sound
-            guild.soundboard_sounds + (sound,),
+            (*guild.soundboard_sounds, sound),
         )
 
     def parse_guild_soundboard_sound_update(self, data: gateway.GuildSoundboardSoundUpdate) -> None:

disnake/ui/select/channel.py

@@ -90,7 +90,8 @@ class ChannelSelect(BaseSelect[ChannelSelectMenu, "AnyChannel", V_co]):
         A list of channels that have been selected by the user.
     """
 
-    __repr_attributes__: ClassVar[Tuple[str, ...]] = BaseSelect.__repr_attributes__ + (
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (
+        *BaseSelect.__repr_attributes__,
         "channel_types",
     )
 

disnake/ui/select/string.py

@@ -105,7 +105,7 @@ class StringSelect(BaseSelect[StringSelectMenu, str, V_co]):
         A list of values that have been selected by the user.
     """
 
-    __repr_attributes__: ClassVar[Tuple[str, ...]] = BaseSelect.__repr_attributes__ + ("options",)
+    __repr_attributes__: ClassVar[Tuple[str, ...]] = (*BaseSelect.__repr_attributes__, "options")
 
     # In practice this should never be used by anything, might as well have it anyway though.
     _default_value_type_map: ClassVar[

disnake/utils.py

@@ -1135,7 +1135,7 @@ def flatten_literal_params(parameters: Iterable[Any]) -> Tuple[Any, ...]:
 
 def normalise_optional_params(parameters: Iterable[Any]) -> Tuple[Any, ...]:
     none_cls = type(None)
-    return tuple(p for p in parameters if p is not none_cls) + (none_cls,)
+    return (*tuple(p for p in parameters if p is not none_cls), none_cls)
 
 
 def _resolve_typealiastype(

noxfile.py

@@ -168,7 +168,7 @@ def codemod(session: nox.Session) -> None:
     session.run_always("pdm", "install", "-dG", "codemod", external=True)
 
     base_command = ["python", "-m", "libcst.tool"]
-    base_command_codemod = base_command + ["codemod"]
+    base_command_codemod = [*base_command, "codemod"]
     if not session.interactive:
         base_command_codemod += ["--hide-progress"]