From 1dddf13747208d23d66bd12c50610a348c6b541d Mon Sep 17 00:00:00 2001 From: Paillat Date: Sat, 14 Dec 2024 16:20:13 +0100 Subject: [PATCH] :label: Boring typing stuff --- discord/commands/options.py | 51 ++++++++++++++++++++++--------------- 1 file changed, 31 insertions(+), 20 deletions(-) diff --git a/discord/commands/options.py b/discord/commands/options.py index ea0bf3e209..8b4649a4a5 100644 --- a/discord/commands/options.py +++ b/discord/commands/options.py @@ -26,8 +26,9 @@ import inspect import logging +from collections.abc import Awaitable, Callable, Iterable from enum import Enum -from typing import TYPE_CHECKING, Iterable, Literal, Optional, Type, Union +from typing import TYPE_CHECKING, Any, Literal, Optional, Type, TypeVar, Union from ..abc import GuildChannel, Mentionable from ..channel import ( @@ -46,6 +47,7 @@ from ..utils import MISSING, basic_autocomplete if TYPE_CHECKING: + from ..cog import Cog from ..ext.commands import Converter from ..member import Member from ..message import Attachment @@ -71,6 +73,25 @@ Type[DiscordEnum], ] + AutocompleteReturnType = Union[ + Iterable["OptionChoice"], Iterable[str], Iterable[int], Iterable[float] + ] + T = TypeVar("T", bound=AutocompleteReturnType) + MaybeAwaitable = Union[T, Awaitable[T]] + AutocompleteFunction = Union[ + Callable[[AutocompleteContext], MaybeAwaitable[AutocompleteReturnType]], + Callable[[Cog, AutocompleteContext], MaybeAwaitable[AutocompleteReturnType]], + Callable[ + [AutocompleteContext, Any], # pyright: ignore [reportExplicitAny] + MaybeAwaitable[AutocompleteReturnType], + ], + Callable[ + [Cog, AutocompleteContext, Any], # pyright: ignore [reportExplicitAny] + MaybeAwaitable[AutocompleteReturnType], + ], + ] + + __all__ = ( "ThreadOption", "Option", @@ -111,11 +132,6 @@ def __init__(self, thread_type: Literal["public", "private", "news"]): self._type = type_map[thread_type] -AutocompleteReturnType = Union[ - Iterable["OptionChoice"], Iterable[str], Iterable[int], Iterable[float] -] - - class Option: """Represents a selectable option for a slash command. @@ -268,7 +284,7 @@ def __init__( ) self.default = kwargs.pop("default", None) - self._autocomplete = None + self._autocomplete: AutocompleteFunction | None = None self.autocomplete = kwargs.pop("autocomplete", None) if len(enum_choices) > 25: self.choices: list[OptionChoice] = [] @@ -388,22 +404,17 @@ def __repr__(self): return f"" @property - def autocomplete(self): + def autocomplete(self) -> AutocompleteFunction | None: """ The autocomplete handler for the option. Accepts a callable (sync or async) - that takes a single required argument of :class:`AutocompleteContext`. + that takes a single required argument of :class:`AutocompleteContext` or two arguments + of :class:`discord.Cog` (being the command's cog) and :class:`AutocompleteContext`. The callable must return an iterable of :class:`str` or :class:`OptionChoice`. Alternatively, :func:`discord.utils.basic_autocomplete` may be used in place of the callable. Returns ------- - Union[ - Callable[[Self, AutocompleteContext, Any], AutocompleteReturnType], - Callable[[AutocompleteContext, Any], AutocompleteReturnType], - Callable[[Self, AutocompleteContext, Any], Awaitable[AutocompleteReturnType]], - Callable[[AutocompleteContext, Any], Awaitable[AutocompleteReturnType]], - None - ] + Optional[AutocompleteFunction] .. versionchanged:: 2.7 @@ -413,17 +424,17 @@ def autocomplete(self): return self._autocomplete @autocomplete.setter - def autocomplete(self, value) -> None: + def autocomplete(self, value: AutocompleteFunction | None) -> None: self._autocomplete = value # this is done here so it does not have to be computed every time the autocomplete is invoked if self._autocomplete is not None: - self._autocomplete._is_instance_method = ( + self._autocomplete._is_instance_method = ( # pyright: ignore [reportFunctionMemberAccess] sum( 1 for param in inspect.signature( - self.autocomplete + self._autocomplete ).parameters.values() - if param.default == param.empty + if param.default == param.empty # pyright: ignore[reportAny] and param.kind not in (param.VAR_POSITIONAL, param.VAR_KEYWORD) ) == 2