fix: handling of Optional[...] in command option type resolution (Pycord-Development#2852)

Signed-off-by: Lumouille <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Lala Sabathil <[email protected]>
(cherry picked from commit 2cf71df)

Author: 3 people

CHANGELOG.md

@@ -157,6 +157,8 @@ These changes are available on the `master` branch, but have not yet been releas
   ([#2843](https://github.com/Pycord-Development/pycord/pull/2843))
 - Fixed `TypeError` when using `@option` with certain annotations and along with
   `channel_types`. ([#2835](https://github.com/Pycord-Development/pycord/pull/2835))
+- Fixed `TypeError` when using `Optional[...]` or `... | None` in command option type.
+  ([#2852](https://github.com/Pycord-Development/pycord/pull/2852))
 - Fixed type-hinting for `PermissionOverwrite.update`.
   ([#2878](https://github.com/Pycord-Development/pycord/pull/2878))
 - Fixed `AttributeError` when accessing `AuditLogEntry.changes` more than once.

discord/commands/options.py

@@ -26,8 +26,9 @@
 
 import inspect
 import logging
+import types
 from enum import Enum
-from typing import TYPE_CHECKING, Literal, Optional, Type, Union
+from typing import TYPE_CHECKING, Literal, Optional, Type, Union, get_args
 
 from ..abc import GuildChannel, Mentionable
 from ..channel import (
@@ -193,6 +194,7 @@ def __init__(self, input_type: InputType = str, /, description: str | None = Non
         if self.name is not None:
             self.name = str(self.name)
         self._parameter_name = self.name  # default
+        input_type = self._strip_none_type(input_type)
         self._raw_type: InputType | tuple = input_type
 
         enum_choices = []
@@ -329,6 +331,26 @@ def __init__(self, input_type: InputType = str, /, description: str | None = Non
         if input_type is None:
             raise TypeError("input_type cannot be NoneType.")
 
+    @staticmethod
+    def _strip_none_type(input_type):
+        if input_type is type(None):
+            raise TypeError("Option type cannot be only NoneType")
+
+        if isinstance(input_type, (types.UnionType, tuple)):
+            args = (
+                get_args(input_type)
+                if isinstance(input_type, types.UnionType)
+                else input_type
+            )
+            filtered = tuple(t for t in args if t is not type(None))
+            if not filtered:
+                raise TypeError("Option type cannot be only NoneType")
+            if len(filtered) == 1:
+                return filtered[0]
+            return filtered
+
+        return input_type
+
     def to_dict(self) -> dict:
         as_dict = {
             "name": self.name,