Don't use recursive type references, due to ugly syntax and incompata…

…bility with union operator
ThePalaceProject · May 14, 2024 · 1a02e63 · 1a02e63
1 parent 5d20a43
commit 1a02e63
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/palace/manager/search/document.py b/src/palace/manager/search/document.py
@@ -1,11 +1,9 @@
 from __future__ import annotations
 
 from abc import ABC, abstractmethod
-from collections.abc import Mapping
+from typing import Any
 
-SearchMappingSerialization = Mapping[
-    str, "str | bool | Sequence[str] | SearchMappingSerialization"
-]
+SearchMappingSerialization = dict[str, Any]
 
 
 class SearchMappingFieldType(ABC):
@@ -193,7 +191,7 @@ def __init__(self) -> None:
 
     def serialize(self) -> SearchMappingSerialization:
         output = self._basic.serialize()
-        output["fields"]["keyword"] = {  # type: ignore[index]
+        output["fields"]["keyword"] = {
             "type": "keyword",
             "index": True,
             "store": False,

diff --git a/src/palace/manager/search/service.py b/src/palace/manager/search/service.py
@@ -71,7 +71,7 @@ def from_bulk_error(cls, error: dict[str, Any]) -> SearchServiceFailedDocument:
             )
 
 
-SearchDocument = dict[str, "str | int | bool | None | Sequence[str] | SearchDocument"]
+SearchDocument = dict[str, Any]
 
 
 class SearchService(ABC):
-Original file line number
+Diff line change
@@ Expand Up @@
                 )
-    SearchDocument = dict[str, "str | int | bool | None | Sequence[str] | SearchDocument"]
+    SearchDocument = dict[str, Any]
     class SearchService(ABC):
@@ Expand Down @@