dara-core 1.17.6__py3-none-any.whl → 1.18.0__py3-none-any.whl

dara/core/__init__.py

@@ -14,6 +14,8 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 """
+# ruff: noqa: F403, F405
+
 from importlib.metadata import version
 
 from pydantic import BaseModel

dara/core/actions.py

@@ -16,8 +16,7 @@ limitations under the License.
 """
 
 # Re-export actions so users can import from dara.core.actions instead of dara_core.interactivity
-# pylint: disable=unused-import
-from dara.core.interactivity import (
+from dara.core.interactivity import (  # noqa: F401
     DownloadContent,
     DownloadContentImpl,
     DownloadVariable,

dara/core/auth/basic.py

@@ -91,10 +91,10 @@ class BaseBasicAuthConfig(BaseAuthConfig):
                 )
             )
             return decoded
-        except jwt.ExpiredSignatureError:
-            raise AuthError(EXPIRED_TOKEN_ERROR, 401)
-        except jwt.DecodeError:
-            raise AuthError(INVALID_TOKEN_ERROR, 401)
+        except jwt.ExpiredSignatureError as e:
+            raise AuthError(EXPIRED_TOKEN_ERROR, 401) from e
+        except jwt.DecodeError as e:
+            raise AuthError(INVALID_TOKEN_ERROR, 401) from e
 
 
 class BasicAuthConfig(BaseBasicAuthConfig):
@@ -121,7 +121,7 @@ class DefaultAuthConfig(BaseAuthConfig):
         logout=BasicAuthLogout,
     )
 
-    def get_token(self, _: SessionRequestBody) -> TokenResponse:
+    def get_token(self, body: SessionRequestBody) -> TokenResponse:
         """
         Get a session token.
 
@@ -142,7 +142,7 @@ class DefaultAuthConfig(BaseAuthConfig):
             decoded = jwt.decode(token, get_settings().jwt_secret, algorithms=[JWT_ALGO])
             SESSION_ID.set(decoded.get('session_id'))
             return TokenData.parse_obj(decoded)
-        except jwt.ExpiredSignatureError:
-            raise AuthError(EXPIRED_TOKEN_ERROR, 401)
-        except jwt.DecodeError:
-            raise AuthError(INVALID_TOKEN_ERROR, 401)
+        except jwt.ExpiredSignatureError as e:
+            raise AuthError(EXPIRED_TOKEN_ERROR, 401) from e
+        except jwt.DecodeError as e:
+            raise AuthError(INVALID_TOKEN_ERROR, 401) from e

dara/core/auth/routes.py

@@ -75,13 +75,13 @@ async def verify_session(
             return SESSION_ID.get()
         except jwt.ExpiredSignatureError as e:
             dev_logger.error('Expired Token Signature', error=e)
-            raise HTTPException(status_code=401, detail=EXPIRED_TOKEN_ERROR)
+            raise HTTPException(status_code=401, detail=EXPIRED_TOKEN_ERROR) from e
         except jwt.PyJWTError as e:
             dev_logger.error('Invalid Token', error=e)
-            raise HTTPException(status_code=401, detail=INVALID_TOKEN_ERROR)
+            raise HTTPException(status_code=401, detail=INVALID_TOKEN_ERROR) from e
         except AuthError as err:
             dev_logger.error('Auth Error', error=err)
-            raise HTTPException(status_code=err.code, detail=err.detail)
+            raise HTTPException(status_code=err.code, detail=err.detail) from err
     raise HTTPException(status_code=400, detail=BAD_REQUEST_ERROR('No auth credentials passed'))
 
 
@@ -162,11 +162,11 @@ async def handle_refresh_token(
         # Explicitly handle expired signature error
         if isinstance(e, jwt.ExpiredSignatureError):
             dev_logger.error('Expired Token Signature', error=e)
-            raise HTTPException(status_code=401, detail=EXPIRED_TOKEN_ERROR, headers=headers)
+            raise HTTPException(status_code=401, detail=EXPIRED_TOKEN_ERROR, headers=headers) from e
 
         # Otherwise show a generic invalid token error
         dev_logger.error('Invalid Token', error=cast(Exception, e))
-        raise HTTPException(status_code=401, detail=INVALID_TOKEN_ERROR, headers=headers)
+        raise HTTPException(status_code=401, detail=INVALID_TOKEN_ERROR, headers=headers) from e
 
 
 # Request to retrieve a session token from the backend. The app does this on startup.

dara/core/auth/utils.py

@@ -44,10 +44,10 @@ def decode_token(token: str, **kwargs) -> TokenData:
     """
     try:
         return TokenData.parse_obj(jwt.decode(token, get_settings().jwt_secret, algorithms=[JWT_ALGO], **kwargs))
-    except jwt.ExpiredSignatureError:
-        raise AuthError(code=401, detail=EXPIRED_TOKEN_ERROR)
-    except jwt.DecodeError:
-        raise AuthError(code=401, detail=INVALID_TOKEN_ERROR)
+    except jwt.ExpiredSignatureError as e:
+        raise AuthError(code=401, detail=EXPIRED_TOKEN_ERROR) from e
+    except jwt.DecodeError as e:
+        raise AuthError(code=401, detail=INVALID_TOKEN_ERROR) from e
 
 
 def sign_jwt(

dara/core/base_definitions.py

@@ -21,17 +21,16 @@ from __future__ import annotations
 # between other parts of the framework
 import abc
 import uuid
+from collections.abc import Awaitable, Mapping
 from enum import Enum
 from typing import (
     TYPE_CHECKING,
     Annotated,
     Any,
-    Awaitable,
     Callable,
     ClassVar,
     Dict,
     List,
-    Mapping,
     Optional,
     Tuple,
     Union,
@@ -67,7 +66,7 @@ def annotation_has_base_model(typ: Any) -> bool:
         type_args = get_args(typ)
         if len(type_args) > 0:
             return any(annotation_has_base_model(arg) for arg in type_args)
-    except:   # pylint: disable=bare-except
+    except:  # noqa: E722
         # canot get arguments, should be a simple type
         pass
 
@@ -79,9 +78,7 @@ def annotation_has_base_model(typ: Any) -> bool:
 # It works by adding SerializeAsAny to all fields of the model.
 # See https://github.com/pydantic/pydantic/issues/6381
 class SerializeAsAnyMeta(ModelMetaclass):
-    def __new__(
-        self, name: str, bases: Tuple[type], namespaces: Dict[str, Any], **kwargs
-    ):   # pylint: disable=bad-mcs-classmethod-argument
+    def __new__(cls, name: str, bases: Tuple[type], namespaces: Dict[str, Any], **kwargs):
         annotations: dict = namespaces.get('__annotations__', {}).copy()
 
         for base in bases:
@@ -97,11 +94,11 @@ class SerializeAsAnyMeta(ModelMetaclass):
                 if isinstance(annotation, str) or annotation is ClassVar:
                     continue
                 if annotation_has_base_model(annotation):
-                    annotations[field] = SerializeAsAny[annotation]   # type: ignore
+                    annotations[field] = SerializeAsAny[annotation]  # type: ignore
 
         namespaces['__annotations__'] = annotations
 
-        return super().__new__(self, name, bases, namespaces, **kwargs)
+        return super().__new__(cls, name, bases, namespaces, **kwargs)
 
 
 class DaraBaseModel(BaseModel, metaclass=SerializeAsAnyMeta):
@@ -135,16 +132,17 @@ class DaraBaseModel(BaseModel, metaclass=SerializeAsAnyMeta):
                 and annotation_has_base_model(field_info.annotation)
             ):
                 # Skip if it has metadata that is already annotated with SerializeAsAny
-                if any(isinstance(x, SerializeAsAny) for x in field_info.metadata):   # type: ignore
+                if any(isinstance(x, SerializeAsAny) for x in field_info.metadata):  # type: ignore
                     continue
                 # Skip if the type is already annotated with SerializeAsAny
                 if get_origin(field_info.annotation) is Annotated and any(
-                    isinstance(arg, SerializeAsAny) for arg in field_info.annotation.__metadata__  # type: ignore
+                    isinstance(arg, SerializeAsAny)  # pyright: ignore[reportArgumentType]
+                    for arg in field_info.annotation.__metadata__  # type: ignore
                 ):
                     continue
 
-                field_info.annotation = SerializeAsAny[field_info.annotation]   # type: ignore
-                field_info.metadata = list(field_info.annotation.__metadata__)   # type: ignore
+                field_info.annotation = SerializeAsAny[field_info.annotation]  # type: ignore
+                field_info.metadata = list(field_info.annotation.__metadata__)  # type: ignore
 
         # Rebuild again with force to ensure we rebuild the schema with new annotations
         return super().model_rebuild(
@@ -255,10 +253,8 @@ class Cache:
             if isinstance(arg, Cache.Type):
                 return LruCachePolicy(cache_type=arg)
 
-            if isinstance(arg, str):
-                # Check that the string is one of allowed cache members
-                if typ := Cache.Type.get_member(arg):
-                    return LruCachePolicy(cache_type=typ)
+            if isinstance(arg, str) and (typ := Cache.Type.get_member(arg)):
+                return LruCachePolicy(cache_type=typ)
 
             raise ValueError(
                 f'Invalid cache argument: {arg}. Please provide a Cache.Policy object or one of Cache.Type members'
@@ -349,12 +345,10 @@ class BaseTask(abc.ABC):
         super().__init__()
 
     @abc.abstractmethod
-    async def run(self, send_stream: Optional[MemoryObjectSendStream[TaskMessage]] = None) -> Any:
-        ...
+    async def run(self, send_stream: Optional[MemoryObjectSendStream[TaskMessage]] = None) -> Any: ...
 
     @abc.abstractmethod
-    async def cancel(self):
-        ...
+    async def cancel(self): ...
 
 
 class PendingTask(BaseTask):
@@ -504,12 +498,11 @@ class AnnotatedAction(BaseModel):
     dynamic_kwargs: Mapping[str, Any]
     """Dynamic kwargs of the action; uid -> variable instance"""
 
-    loading: 'Variable'   # type: ignore
+    loading: Variable  # type: ignore # noqa: F821
     """Loading Variable instance"""
 
     def __init__(self, **data):
         # Resolve the circular dependency to add a loading Variable to the model upon creation
-        # pylint: disable-next=import-error, import-outside-toplevel
         from dara.core.interactivity.plain_variable import Variable
 
         self.model_rebuild()

dara/core/cli.py

@@ -21,10 +21,10 @@ import pathlib
 import subprocess
 from typing import List, Optional
 
-import click
 import uvicorn
-from click.exceptions import UsageError
 
+import click
+from click.exceptions import UsageError
 from dara.core.internal.port_utils import find_available_port
 from dara.core.internal.settings import generate_env_file
 from dara.core.internal.utils import find_module_path
@@ -52,7 +52,7 @@ def cli():
 @click.option('--port', help='The port to run on', type=int)
 @click.option('--metrics-port', help='The port for the metrics server to run on', type=int)
 @click.option('--disable-metrics', is_flag=True, help='Whether to disable the metrics server')
-@click.option('--host', default='0.0.0.0', help='The host to run on')   # nosec B104 # default for local dev
+@click.option('--host', default='0.0.0.0', help='The host to run on')  # nosec B104 # default for local dev
 @click.option('--rebuild', is_flag=True, help='Whether to force a rebuild of the app')
 @click.option('--require-sso', is_flag=True, help='Whether to enforce that an SSO auth config is used')
 @click.option('--docker', is_flag=True, help='Whether to run in Docker mode - assumes assets are prebuilt')
@@ -214,10 +214,11 @@ def dev():
         stderr=subprocess.STDOUT,
         shell=True,
     ) as vite_process:
-        for line in vite_process.stdout:
-            decoded_line = line.decode('utf-8').strip()
-            if decoded_line != '':
-                print(decoded_line)   # pylint: disable=bad-builtin
+        if vite_process.stdout is not None:
+            for line in vite_process.stdout:
+                decoded_line = line.decode('utf-8').strip()
+                if decoded_line != '':
+                    print(decoded_line)
 
 
 @cli.command()

dara/core/configuration.py

@@ -14,6 +14,7 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 """
+
 import os
 import pathlib
 from inspect import isclass, isfunction
@@ -386,7 +387,7 @@ class ConfigurationBuilder:
         icon: Optional[str] = None,
         route: Optional[str] = None,
         include_in_menu: Optional[bool] = True,
-        reset_vars_on_load: Optional[List[AnyVariable]] = [],
+        reset_vars_on_load: Optional[List[AnyVariable]] = None,
         on_load: Optional[Action] = None,
     ):
         """
@@ -402,6 +403,8 @@ class ConfigurationBuilder:
         :param on_load: optional action to execute upon visiting the page
         """
         # Backwards compatibility - deprecated
+        if reset_vars_on_load is None:
+            reset_vars_on_load = []
         if reset_vars_on_load is not None and len(reset_vars_on_load) > 0:
             if on_load is not None:
                 raise ValueError('reset_vars_on_load and on_load cannot be used together')
@@ -485,7 +488,7 @@ class ConfigurationBuilder:
             if len(options) > 0:
                 dev_logger.warning(f'Options provided for a function middleware {middleware}, but they will be ignored')
         elif isclass(middleware):
-            constructed_middleware = Middleware(middleware, **options)   # type: ignore
+            constructed_middleware = Middleware(middleware, **options)  # type: ignore
         else:
             raise ValueError(f'Invalid middleware type: {type(middleware)}')
 

dara/core/css.py

@@ -18,8 +18,7 @@ limitations under the License.
 from typing import Literal, Optional
 
 # Re-export CSSProperties for easier importing
-# pylint: disable=unused-import
-from dara.core.visual.css import CSSProperties
+from dara.core.visual.css import CSSProperties  # noqa: F401
 
 IconStyle = Literal['solid', 'regular', 'brands']
 IconSize = Literal['1x', '2x', '3x', '4x', '5x', '6x', '7x', '8x', '9x', '10x', '2xs', 'xs', 'sm', 'lg', 'xl', '2xl']

dara/core/data_utils.py

@@ -116,7 +116,7 @@ class FileStore(BaseModel):
         if not os.path.exists(file_path):
             return None
 
-        return io.open(file_path, 'rb')
+        return open(file_path, 'rb')
 
     def write_file(self, cache_type: CacheType, name: str) -> io.BufferedWriter:
         """
@@ -130,7 +130,7 @@ class FileStore(BaseModel):
         """
         scope_path = self.get_scoped_path(cache_type)
         os.makedirs(scope_path, exist_ok=True)
-        return io.open(os.path.join(scope_path, name), 'wb')
+        return open(os.path.join(scope_path, name), 'wb')
 
     def delete_file(self, cache_type: CacheType, name: str) -> None:
         """

dara/core/defaults.py

@@ -17,7 +17,7 @@ limitations under the License.
 
 from __future__ import annotations
 
-from typing import TYPE_CHECKING, Dict
+from typing import TYPE_CHECKING, Dict, cast
 
 from dara.core.base_definitions import ActionDef
 from dara.core.interactivity.actions import (
@@ -77,8 +77,8 @@ CORE_COMPONENTS: Dict[str, ComponentTypeAnnotation] = {
     RouterContent.__name__: RouterContentDef,
     SideBarFrame.__name__: SideBarFrameDef,
     TopBarFrame.__name__: TopBarFrameDef,
-    Fallback.Default.py_component: DefaultFallbackDef,
-    Fallback.Row.py_component: RowFallbackDef,
+    cast(str, Fallback.Default.py_component): DefaultFallbackDef,
+    cast(str, Fallback.Row.py_component): RowFallbackDef,
     For.__name__: ForDef,
 }
 
@@ -93,9 +93,9 @@ CORE_ACTIONS: Dict[str, ActionDef] = {
     Notify.__name__: NotifyDef,
 }
 
+
 # Define a default layout template
 def default_template(config: Configuration) -> Template:
-
     template = TemplateBuilder(name='default')
 
     router = template.add_router_from_pages(list(config.pages.values()))
@@ -107,7 +107,6 @@ def default_template(config: Configuration) -> Template:
 
 # Define a blank template
 def blank_template(config: Configuration) -> Template:
-
     template = TemplateBuilder(name='default')
 
     router = template.add_router_from_pages(list(config.pages.values()))
@@ -119,7 +118,6 @@ def blank_template(config: Configuration) -> Template:
 
 # Define a top layout template
 def top_template(config: Configuration) -> Template:
-
     template = TemplateBuilder(name='default')
 
     router = template.add_router_from_pages(list(config.pages.values()))
@@ -131,7 +129,6 @@ def top_template(config: Configuration) -> Template:
 
 # Define a top layout template with menu
 def top_menu_template(config: Configuration) -> Template:
-
     template = TemplateBuilder(name='default')
 
     router = template.add_router_from_pages(list(config.pages.values()))

dara/core/definitions.py

@@ -19,15 +19,14 @@ from __future__ import annotations
 
 import json
 import uuid
+from collections.abc import Awaitable, Mapping
 from enum import Enum
 from typing import (
     Any,
-    Awaitable,
     Callable,
     ClassVar,
     List,
     Literal,
-    Mapping,
     Optional,
     Protocol,
     Type,
@@ -264,8 +263,7 @@ class CallableClassComponent(Protocol):
     Callable class component protocol. Describes any class with a __call__ instance method returning a component instance.
     """
 
-    def __call__(self) -> ComponentInstance:
-        ...
+    def __call__(self) -> ComponentInstance: ...
 
 
 DiscoverTarget = Union[Callable[..., ComponentInstance], ComponentInstance, Type[CallableClassComponent]]
@@ -279,7 +277,7 @@ def discover(outer_obj: DiscoverT) -> DiscoverT:
     Will make sure to statically register all encountered dependencies of marked functional component or component class.
     Should not be necessary in most cases, mainly useful when creating component libraries.
     """
-    outer_obj.__wrapped_by__ = discover   # type: ignore
+    outer_obj.__wrapped_by__ = discover  # type: ignore
     return outer_obj
 
 
@@ -419,9 +417,8 @@ class BaseFallback(StyledComponentInstance):
     @field_validator('suspend_render')
     @classmethod
     def validate_suspend_render(cls, value):
-        if isinstance(value, int):
-            if value < 0:
-                raise ValueError('suspend_render must be a positive integer')
+        if isinstance(value, int) and value < 0:
+            raise ValueError('suspend_render must be a positive integer')
 
         return value
 
@@ -505,7 +502,7 @@ class Page(BaseModel):
     icon: Optional[str] = None
     content: ComponentInstanceType
     name: str
-    sub_pages: Optional[List['Page']] = []
+    sub_pages: Optional[List[Page]] = []
     url_safe_name: str
     include_in_menu: Optional[bool] = None
     on_load: Optional[Action] = None

dara/core/http.py

@@ -18,7 +18,7 @@ limitations under the License.
 import inspect
 from collections import OrderedDict
 from functools import wraps
-from typing import Callable, Dict, List, Type
+from typing import Callable, Dict, List, Type, Union
 
 from fastapi import Depends
 from fastapi.params import Depends as DependsType
@@ -54,7 +54,10 @@ def _get_config_instances(annotations: Dict[str, type]) -> Dict[str, EndpointCon
 def _method_decorator(method: HttpMethod):
     """Create a decorator for a given HTTP method"""
 
-    def _decorator(url: str, dependencies: List[DependsType] = [], authenticated: bool = True):
+    def _decorator(url: str, dependencies: Union[List[DependsType], None] = None, authenticated: bool = True):
+        if dependencies is None:
+            dependencies = []
+
         def _inner(func: Callable):
             # Make sure we're using a copy of the dependencies list
             final_dependencies = dependencies[:]
@@ -104,7 +107,8 @@ def _method_decorator(method: HttpMethod):
 
             new_handler.__annotations__ = new_annotations
             new_handler.__signature__ = inspect.Signature(  # type: ignore
-                parameters=list(params.values()), return_annotation=sig.return_annotation  # type: ignore
+                parameters=list(params.values()),
+                return_annotation=sig.return_annotation,  # type: ignore
             )
 
             return ApiRoute(

dara/core/interactivity/actions.py

@@ -17,16 +17,17 @@ limitations under the License.
 
 from __future__ import annotations
 
+import contextlib
 import inspect
 import math
 import uuid
+from collections.abc import Awaitable
 from contextvars import ContextVar
 from enum import Enum
 from functools import partial, update_wrapper
 from typing import (
     TYPE_CHECKING,
     Any,
-    Awaitable,
     Callable,
     ClassVar,
     Dict,
@@ -50,9 +51,9 @@ from dara.core.base_definitions import (
     ActionImpl,
     ActionResolverDef,
     AnnotatedAction,
+    TaskProgressUpdate,
 )
 from dara.core.base_definitions import DaraBaseModel as BaseModel
-from dara.core.base_definitions import TaskProgressUpdate
 from dara.core.interactivity.data_variable import DataVariable
 from dara.core.internal.download import generate_download_code
 from dara.core.internal.registry_lookup import RegistryLookup
@@ -97,7 +98,7 @@ class ComponentActionContext(ActionContext):
     ActionContext for actions that only require component value
     """
 
-    inputs: ComponentActionInputs   # type: ignore
+    inputs: ComponentActionInputs  # type: ignore
 
 
 class UpdateVariableImpl(ActionImpl):
@@ -162,7 +163,7 @@ class UpdateVariableInputs(ActionInputs):
 
 
 class UpdateVariableContext(ActionContext):
-    inputs: UpdateVariableInputs   # type: ignore
+    inputs: UpdateVariableInputs  # type: ignore
 
 
 @deprecated('Use @action or `UpdateVariableImpl` for simple cases')
@@ -254,8 +255,8 @@ class UpdateVariable(AnnotatedAction):
         :param extras: any extra variables to resolve and pass to the resolution function context
         """
 
-        async def _update(ctx: action.Ctx, **kwargs):   # type: ignore
-            ctx = cast(ActionCtx, ctx)   # type: ignore
+        async def _update(ctx: action.Ctx, **kwargs):  # type: ignore
+            ctx = cast(ActionCtx, ctx)  # type: ignore
             old = kwargs.pop('old')
             extras = [kwargs[f'kwarg_{idx}'] for idx in range(len(kwargs))]
             old_ctx = UpdateVariableContext(inputs=UpdateVariableInputs(old=old, new=ctx.input), extras=extras)
@@ -271,7 +272,7 @@ class UpdateVariable(AnnotatedAction):
                 for idx in range(len(extras or []))
             ],
         ]
-        _update.__signature__ = inspect.Signature(params)   # type: ignore
+        _update.__signature__ = inspect.Signature(params)  # type: ignore
 
         # Pass in variable and extras as kwargs
         kwargs = {f'kwarg_{idx}': value for idx, value in enumerate(extras or [])}
@@ -428,11 +429,11 @@ def NavigateTo(
         return NavigateToImpl(url=url, new_tab=new_tab)
 
     # Otherwise create a new @action with the provided resolver
-    async def _navigate(ctx: action.Ctx, **kwargs):   # type: ignore
-        ctx = cast(ActionCtx, ctx)   # type: ignore
+    async def _navigate(ctx: action.Ctx, **kwargs):  # type: ignore
+        ctx = cast(ActionCtx, ctx)  # type: ignore
         extras = [kwargs[f'kwarg_{idx}'] for idx in range(len(kwargs))]
         old_ctx = ComponentActionContext(inputs=ComponentActionInputs(value=ctx.input), extras=extras)
-        result = await run_user_handler(url, args=(old_ctx,))   # type: ignore
+        result = await run_user_handler(url, args=(old_ctx,))  # type: ignore
         # Navigate to resulting url
         await ctx.navigate(result, new_tab)
 
@@ -444,14 +445,14 @@ def NavigateTo(
             for idx in range(len(extras or []))
         ],
     ]
-    _navigate.__signature__ = inspect.Signature(params)   # type: ignore
+    _navigate.__signature__ = inspect.Signature(params)  # type: ignore
 
     # Pass in variable and extras as kwargs
     kwargs = {f'kwarg_{idx}': value for idx, value in enumerate(extras or [])}
     return action(_navigate)(**kwargs)
 
 
-NavigateTo.Ctx = ComponentActionContext   # type: ignore
+NavigateTo.Ctx = ComponentActionContext  # type: ignore
 
 
 def Logout():
@@ -663,8 +664,8 @@ def DownloadContent(
     ```
     """
 
-    async def _download(ctx: action.Ctx, **kwargs):   # type: ignore
-        ctx = cast(ActionCtx, ctx)   # type: ignore
+    async def _download(ctx: action.Ctx, **kwargs):  # type: ignore
+        ctx = cast(ActionCtx, ctx)  # type: ignore
         extras = [kwargs[f'kwarg_{idx}'] for idx in range(len(kwargs))]
         old_ctx = ComponentActionContext(inputs=ComponentActionInputs(value=ctx.input), extras=extras)
         result = await run_user_handler(resolver, args=(old_ctx,))
@@ -678,7 +679,7 @@ def DownloadContent(
             for idx in range(len(extras or []))
         ],
     ]
-    _download.__signature__ = inspect.Signature(params)   # type: ignore
+    _download.__signature__ = inspect.Signature(params)  # type: ignore
 
     # Pass in extras as kwargs
     kwargs = {f'kwarg_{idx}': value for idx, value in enumerate(extras or [])}
@@ -686,7 +687,7 @@ def DownloadContent(
     return action(_download)(**kwargs)
 
 
-DownloadContent.Ctx = ComponentActionContext   # type: ignore
+DownloadContent.Ctx = ComponentActionContext  # type: ignore
 """@deprecated retained for backwards compatibility, to be removed in 2.0"""
 
 DownloadVariableDef = ActionDef(name='DownloadVariable', js_module='@darajs/core', py_module='dara.core')
@@ -769,8 +770,8 @@ def SideEffect(
     ```
     """
 
-    async def _effect(ctx: action.Ctx, **kwargs):   # type: ignore
-        ctx = cast(ActionCtx, ctx)   # type: ignore
+    async def _effect(ctx: action.Ctx, **kwargs):  # type: ignore
+        ctx = cast(ActionCtx, ctx)  # type: ignore
         extras = [kwargs[f'kwarg_{idx}'] for idx in range(len(kwargs))]
         old_ctx = ComponentActionContext(inputs=ComponentActionInputs(value=ctx.input), extras=extras)
         # Simply run the user handler
@@ -784,7 +785,7 @@ def SideEffect(
             for idx in range(len(extras or []))
         ],
     ]
-    _effect.__signature__ = inspect.Signature(params)   # type: ignore
+    _effect.__signature__ = inspect.Signature(params)  # type: ignore
 
     # Pass in extras as kwargs
     kwargs = {f'kwarg_{idx}': value for idx, value in enumerate(extras or [])}
@@ -792,7 +793,7 @@ def SideEffect(
     return action(_effect)(**kwargs)
 
 
-SideEffect.Ctx = ComponentActionContext   # type: ignore
+SideEffect.Ctx = ComponentActionContext  # type: ignore
 """@deprecated retained for backwards compatibility, to be removed in 2.0"""
 
 VariableT = TypeVar('VariableT')
@@ -825,12 +826,10 @@ class ActionCtx:
         self._on_action = _on_action
 
     @overload
-    async def update(self, variable: DataVariable, value: Optional[DataFrame]):
-        ...
+    async def update(self, variable: DataVariable, value: Optional[DataFrame]): ...
 
     @overload
-    async def update(self, variable: Union[Variable[VariableT], UrlVariable[VariableT]], value: VariableT):
-        ...
+    async def update(self, variable: Union[Variable[VariableT], UrlVariable[VariableT]], value: VariableT): ...
 
     async def update(self, variable: Union[Variable, UrlVariable, DataVariable], value: Any):
         """
@@ -1389,11 +1388,10 @@ class action:
         return bound_f
 
     @overload
-    def __call__(self, ctx: ActionCtx, *args: Any, **kwargs: Any) -> Any:
-        ...
+    def __call__(self, ctx: ActionCtx, *args: Any, **kwargs: Any) -> Any: ...
 
     @overload
-    def __call__(self, *args: Any, **kwargs: Any) -> AnnotatedAction:   # type: ignore
+    def __call__(self, *args: Any, **kwargs: Any) -> AnnotatedAction:  # type: ignore
         ...
 
     def __call__(self, *args, **kwargs) -> Union[AnnotatedAction, Any]:
@@ -1444,10 +1442,10 @@ class action:
         for key, value in all_kwargs.items():
             if key in self.func.__annotations__:
                 valid_value = True
-                try:
+                # The type is either not set or something tricky to verify, e.g. union
+                with contextlib.suppress(Exception):
                     valid_value = isinstance(value, (self.func.__annotations__[key], AnyVariable))
-                except Exception:
-                    pass  # The type is either not set or something tricky to verify, e.g. union
+
                 if not valid_value:
                     raise TypeError(
                         f'Argument: {key} was passed as a {type(value)}, but it should be '