Add warn_return_any=True and use typing.cast() if needed

Author: vdusek

mypy.ini

@@ -11,6 +11,7 @@ disallow_untyped_decorators = True
 disallow_untyped_defs = True
 no_implicit_optional = True
 warn_redundant_casts = True
+warn_return_any = True
 warn_unreachable = True
 warn_unused_ignores = True
 

src/apify/_memory_storage/resource_clients/base_resource_collection_client.py

@@ -2,7 +2,7 @@
 
 from abc import ABC, abstractmethod
 from operator import itemgetter
-from typing import TYPE_CHECKING, Generic, TypeVar
+from typing import TYPE_CHECKING, Generic, TypeVar, cast
 
 from apify_shared.models import ListPage
 from apify_shared.utils import ignore_docs
@@ -91,7 +91,8 @@ async def get_or_create(
                 id=_id,
             )
             if found:
-                return found._to_resource_info()
+                resource_info = found._to_resource_info()
+                return cast(dict, resource_info)
 
         new_resource = resource_client_class(
             id=_id,
@@ -110,4 +111,4 @@ async def get_or_create(
             write_metadata=self._memory_storage_client._write_metadata,
         )
 
-        return resource_info
+        return cast(dict, resource_info)

src/apify/_utils.py

@@ -109,7 +109,8 @@ def __get__(self: dualproperty, obj: DualPropertyOwner | None, owner: type[DualP
         Returns:
             The result of the getter.
         """
-        return self.getter(obj or owner)
+        val = self.getter(obj or owner)
+        return cast(DualPropertyType, val)
 
 
 @overload
@@ -337,7 +338,7 @@ def __getitem__(self: LRUCache, key: str) -> T:
         val = self._cache[key]
         # No 'key in cache' condition since the previous line would raise KeyError
         self._cache.move_to_end(key)
-        return val
+        return cast(T, val)
 
     # Sadly TS impl returns bool indicating whether the key was already present or not
     def __setitem__(self: LRUCache, key: str, value: T) -> None: