reflex 0.7.13a1__py3-none-any.whl → 0.7.14__py3-none-any.whl

reflex/event.py

@@ -228,9 +228,8 @@ class EventHandler(EventActionsMixin):
             ),
             Unset,
         ):
-            raise EventHandlerTypeError(
-                f"Event handler {self.fn.__name__} received repeated argument {repeated_arg}."
-            )
+            msg = f"Event handler {self.fn.__name__} received repeated argument {repeated_arg}."
+            raise EventHandlerTypeError(msg)
 
         if not isinstance(
             extra_arg := next(
@@ -238,9 +237,10 @@ class EventHandler(EventActionsMixin):
             ),
             Unset,
         ):
-            raise EventHandlerTypeError(
+            msg = (
                 f"Event handler {self.fn.__name__} received extra argument {extra_arg}."
             )
+            raise EventHandlerTypeError(msg)
 
         fn_args = fn_args[: len(args)] + list(kwargs)
 
@@ -257,9 +257,8 @@ class EventHandler(EventActionsMixin):
             try:
                 values.append(LiteralVar.create(arg))
             except TypeError as e:
-                raise EventHandlerTypeError(
-                    f"Arguments to event handlers must be Vars or JSON-serializable. Got {arg} of type {type(arg)}."
-                ) from e
+                msg = f"Arguments to event handlers must be Vars or JSON-serializable. Got {arg} of type {type(arg)}."
+                raise EventHandlerTypeError(msg) from e
         payload = tuple(zip(fn_args, values, strict=False))
 
         # Return the event spec.
@@ -353,9 +352,8 @@ class EventSpec(EventActionsMixin):
             for arg in args:
                 values.append(LiteralVar.create(value=arg))  # noqa: PERF401, RUF100
         except TypeError as e:
-            raise EventHandlerTypeError(
-                f"Arguments to event handlers must be Vars or JSON-serializable. Got {arg} of type {type(arg)}."
-            ) from e
+            msg = f"Arguments to event handlers must be Vars or JSON-serializable. Got {arg} of type {type(arg)}."
+            raise EventHandlerTypeError(msg) from e
         new_payload = tuple(zip(fn_args, values, strict=False))
         return self.with_args(self.args + new_payload)
 
@@ -408,7 +406,8 @@ class CallableEventSpec(EventSpec):
         from reflex.utils.exceptions import EventHandlerTypeError
 
         if self.fn is None:
-            raise EventHandlerTypeError("CallableEventSpec has no associated function.")
+            msg = "CallableEventSpec has no associated function."
+            raise EventHandlerTypeError(msg)
         return self.fn(*args, **kwargs)
 
 
@@ -453,7 +452,7 @@ class EventChain(EventActionsMixin):
         if isinstance(value, Var):
             if isinstance(value, EventChainVar):
                 return value
-            elif isinstance(value, EventVar):
+            if isinstance(value, EventVar):
                 value = [value]
             elif safe_issubclass(value._var_type, (EventChain, EventSpec)):
                 return cls.create(
@@ -463,9 +462,8 @@ class EventChain(EventActionsMixin):
                     **event_chain_kwargs,
                 )
             else:
-                raise ValueError(
-                    f"Invalid event chain: {value!s} of type {value._var_type}"
-                )
+                msg = f"Invalid event chain: {value!s} of type {value._var_type}"
+                raise ValueError(msg)
         elif isinstance(value, EventChain):
             # Trust that the caller knows what they're doing passing an EventChain directly
             return value
@@ -485,15 +483,17 @@ class EventChain(EventActionsMixin):
                     # Call the lambda to get the event chain.
                     result = call_event_fn(v, args_spec, key=key)
                     if isinstance(result, Var):
-                        raise ValueError(
+                        msg = (
                             f"Invalid event chain: {v}. Cannot use a Var-returning "
                             "lambda inside an EventChain list."
                         )
+                        raise ValueError(msg)
                     events.extend(result)
                 elif isinstance(v, EventVar):
                     events.append(v)
                 else:
-                    raise ValueError(f"Invalid event: {v}")
+                    msg = f"Invalid event: {v}"
+                    raise ValueError(msg)
 
         # If the input is a callable, create an event chain.
         elif isinstance(value, Callable):
@@ -507,7 +507,8 @@ class EventChain(EventActionsMixin):
 
         # Otherwise, raise an error.
         else:
-            raise ValueError(f"Invalid event chain: {value}")
+            msg = f"Invalid event chain: {value}"
+            raise ValueError(msg)
 
         # Add args to the event specs if necessary.
         events = [
@@ -783,9 +784,11 @@ class FileUpload:
                     on_upload_progress, self.on_upload_progress_args_spec
                 )
             else:
-                raise ValueError(f"{on_upload_progress} is not a valid event handler.")
+                msg = f"{on_upload_progress} is not a valid event handler."
+                raise ValueError(msg)
             if isinstance(events, Var):
-                raise ValueError(f"{on_upload_progress} cannot return a var {events}.")
+                msg = f"{on_upload_progress} cannot return a var {events}."
+                raise ValueError(msg)
             on_upload_progress_chain = EventChain(
                 events=[*events],
                 args_spec=self.on_upload_progress_args_spec,
@@ -1081,7 +1084,8 @@ def download(
 
     if isinstance(url, str):
         if not url.startswith("/"):
-            raise ValueError("The URL argument should start with a /")
+            msg = "The URL argument should start with a /"
+            raise ValueError(msg)
 
         # if filename is not provided, infer it from url
         if filename is None:
@@ -1092,7 +1096,8 @@ def download(
 
     if data is not None:
         if url is not None:
-            raise ValueError("Cannot provide both URL and data to download.")
+            msg = "Cannot provide both URL and data to download."
+            raise ValueError(msg)
 
         if isinstance(data, str):
             # Caller provided a plain text string to download.
@@ -1115,9 +1120,8 @@ def download(
             b64_data = b64encode(data).decode("utf-8")
             url = "data:application/octet-stream;base64," + b64_data
         else:
-            raise ValueError(
-                f"Invalid data type {type(data)} for download. Use `str` or `bytes`."
-            )
+            msg = f"Invalid data type {type(data)} for download. Use `str` or `bytes`."
+            raise ValueError(msg)
 
     return server_side(
         "_download",
@@ -1323,23 +1327,21 @@ def _check_event_args_subclass_of_callback(
             except TypeError as te:
                 callback_name_context = f" of {callback_name}" if callback_name else ""
                 key_context = f" for {key}" if key else ""
-                raise TypeError(
-                    f"Could not compare types {args_types_without_vars[i]} and {callback_param_name_to_type[arg]} for argument {arg}{callback_name_context}{key_context}."
-                ) from te
+                msg = f"Could not compare types {args_types_without_vars[i]} and {callback_param_name_to_type[arg]} for argument {arg}{callback_name_context}{key_context}."
+                raise TypeError(msg) from te
 
             if compare_result:
                 type_match_found[arg] = True
                 continue
-            else:
-                type_match_found[arg] = False
-                as_annotated_in = (
-                    f" as annotated in {callback_name}" if callback_name else ""
-                )
-                delayed_exceptions.append(
-                    EventHandlerArgTypeMismatchError(
-                        f"Event handler {key} expects {args_types_without_vars[i]} for argument {arg} but got {callback_param_name_to_type[arg]}{as_annotated_in} instead."
-                    )
+            type_match_found[arg] = False
+            as_annotated_in = (
+                f" as annotated in {callback_name}" if callback_name else ""
+            )
+            delayed_exceptions.append(
+                EventHandlerArgTypeMismatchError(
+                    f"Event handler {key} expects {args_types_without_vars[i]} for argument {arg} but got {callback_param_name_to_type[arg]}{as_annotated_in} instead."
                 )
+            )
 
         if all(type_match_found.values()):
             delayed_exceptions.clear()
@@ -1495,8 +1497,7 @@ def resolve_annotation(annotations: dict[str, Any], arg_name: str, spec: ArgsSpe
     if annotation is None:
         if not isinstance(spec, types.LambdaType):
             raise MissingAnnotationError(var_name=arg_name)
-        else:
-            return dict[str, dict]
+        return dict[str, dict]
     return annotation
 
 
@@ -1570,12 +1571,13 @@ def check_fn_match_arg_spec(
     number_of_event_args = len(parsed_event_args)
 
     if number_of_user_args - number_of_user_default_args > number_of_event_args:
-        raise EventFnArgMismatchError(
+        msg = (
             f"Event {key} only provides {number_of_event_args} arguments, but "
             f"{func_name or user_func} requires at least {number_of_user_args - number_of_user_default_args} "
             "arguments to be passed to the event handler.\n"
             "See https://reflex.dev/docs/events/event-arguments/"
         )
+        raise EventFnArgMismatchError(msg)
 
 
 def call_event_fn(
@@ -1630,9 +1632,8 @@ def call_event_fn(
 
         # Make sure the event spec is valid.
         if not isinstance(e, EventSpec):
-            raise EventHandlerValueError(
-                f"Lambda {fn} returned an invalid event spec: {e}."
-            )
+            msg = f"Lambda {fn} returned an invalid event spec: {e}."
+            raise EventHandlerValueError(msg)
 
         # Add the event spec to the chain.
         events.append(e)
@@ -1696,7 +1697,8 @@ def fix_events(
         if isinstance(e, EventHandler):
             e = e()
         if not isinstance(e, EventSpec):
-            raise ValueError(f"Unexpected event type, {type(e)}.")
+            msg = f"Unexpected event type, {type(e)}."
+            raise ValueError(msg)
         name = format.format_event_handler(e.handler)
         payload = {k._js_expr: v._decode() for k, v in e.args}
 
@@ -1749,9 +1751,8 @@ class EventVar(ObjectVar, python_types=(EventSpec, EventHandler)):
         Raises:
             TypeError: EventVar cannot be converted to a boolean.
         """
-        raise TypeError(
-            f"Cannot convert {self._js_expr} of type {type(self).__name__} to bool."
-        )
+        msg = f"Cannot convert {self._js_expr} of type {type(self).__name__} to bool."
+        raise TypeError(msg)
 
 
 @dataclasses.dataclass(
@@ -1798,9 +1799,8 @@ class LiteralEventVar(VarOperationCall, LiteralVar, EventVar):
             try:
                 value = call_event_handler(value, no_args)
             except EventFnArgMismatchError:
-                raise EventFnArgMismatchError(
-                    f"Event handler {value.fn.__qualname__} used inside of a rx.cond() must not take any arguments."
-                ) from None
+                msg = f"Event handler {value.fn.__qualname__} used inside of a rx.cond() must not take any arguments."
+                raise EventFnArgMismatchError(msg) from None
 
         return cls(
             _js_expr="",
@@ -1835,9 +1835,8 @@ class EventChainVar(BuilderFunctionVar, python_types=EventChain):
         Raises:
             TypeError: EventChainVar cannot be converted to a boolean.
         """
-        raise TypeError(
-            f"Cannot convert {self._js_expr} of type {type(self).__name__} to bool."
-        )
+        msg = f"Cannot convert {self._js_expr} of type {type(self).__name__} to bool."
+        raise TypeError(msg)
 
 
 @dataclasses.dataclass(
@@ -1906,9 +1905,8 @@ class LiteralEventChainVar(ArgsFunctionOperationBuilder, LiteralVar, EventChainV
             invocation = value.invocation
 
         if invocation is not None and not isinstance(invocation, FunctionVar):
-            raise ValueError(
-                f"EventChain invocation must be a FunctionVar, got {invocation!s} of type {invocation._var_type!s}."
-            )
+            msg = f"EventChain invocation must be a FunctionVar, got {invocation!s} of type {invocation._var_type!s}."
+            raise ValueError(msg)
 
         return cls(
             _js_expr="",
@@ -2142,12 +2140,12 @@ class EventNamespace:
                 if not inspect.iscoroutinefunction(
                     func
                 ) and not inspect.isasyncgenfunction(func):
-                    raise TypeError(
-                        "Background task must be async function or generator."
-                    )
+                    msg = "Background task must be async function or generator."
+                    raise TypeError(msg)
                 setattr(func, BACKGROUND_TASK_MARKER, True)
             if getattr(func, "__name__", "").startswith("_"):
-                raise ValueError("Event handlers cannot be private.")
+                msg = "Event handlers cannot be private."
+                raise ValueError(msg)
 
             qualname: str | None = getattr(func, "__qualname__", None)
 

reflex/experimental/__init__.py

@@ -6,9 +6,9 @@ from reflex.components.datadisplay.shiki_code_block import code_block as code_bl
 from reflex.components.props import PropsBase
 from reflex.components.radix.themes.components.progress import progress as progress
 from reflex.components.sonner.toast import toast as toast
+from reflex.utils.console import warn
+from reflex.utils.misc import run_in_thread
 
-from ..utils.console import warn
-from ..utils.misc import run_in_thread
 from . import hooks as hooks
 from .client_state import ClientStateVar as ClientStateVar
 from .layout import layout as layout

reflex/experimental/client_state.py

@@ -111,7 +111,8 @@ class ClientStateVar(Var):
             var_name = get_unique_variable_name()
         id_name = "id_" + get_unique_variable_name()
         if not isinstance(var_name, str):
-            raise ValueError("var_name must be a string.")
+            msg = "var_name must be a string."
+            raise ValueError(msg)
         if default is NoValue:
             default_var = Var(_js_expr="")
         elif not isinstance(default, Var):
@@ -158,7 +159,9 @@ class ClientStateVar(Var):
             hooks[f"{_client_state_ref(var_name)} ??= {var_name!s}"] = None
             hooks[f"{_client_state_ref_dict(var_name)} ??= {{}}"] = None
             hooks[f"{_client_state_ref_dict(setter_name)} ??= {{}}"] = None
-            hooks[f"{_client_state_ref_dict(var_name)}[{id_name}] = {var_name}"] = None
+            hooks[
+                f"{_client_state_ref_dict(var_name)}[{id_name}] = {_client_state_ref(var_name)}"
+            ] = None
             hooks[
                 f"{_client_state_ref_dict(setter_name)}[{id_name}] = {setter_name}"
             ] = None
@@ -269,7 +272,8 @@ class ClientStateVar(Var):
             ValueError: If the ClientStateVar is not global.
         """
         if not self._global_ref:
-            raise ValueError("ClientStateVar must be global to retrieve the value.")
+            msg = "ClientStateVar must be global to retrieve the value."
+            raise ValueError(msg)
         return run_script(_client_state_ref(self._getter_name), callback=callback)
 
     def push(self, value: Any) -> EventSpec:
@@ -287,6 +291,7 @@ class ClientStateVar(Var):
             ValueError: If the ClientStateVar is not global.
         """
         if not self._global_ref:
-            raise ValueError("ClientStateVar must be global to push the value.")
+            msg = "ClientStateVar must be global to push the value."
+            raise ValueError(msg)
         value = Var.create(value)
         return run_script(f"{_client_state_ref(self._setter_name)}({value})")

reflex/experimental/layout.pyi

@@ -237,7 +237,6 @@ class Sidebar(Box, MemoizationLeaf):
         Returns:
             The sidebar component.
         """
-        ...
 
     def add_style(self) -> dict[str, Any] | None: ...
     def add_hooks(self) -> list[Var]: ...
@@ -309,7 +308,6 @@ class DrawerSidebar(DrawerRoot):
         Returns:
             The drawer sidebar component.
         """
-        ...
 
 sidebar_trigger_style = {
     "position": "fixed",
@@ -362,7 +360,6 @@ class SidebarTrigger(Fragment):
         Returns:
             The sidebar trigger component.
         """
-        ...
 
 class Layout(Box):
     @overload
@@ -587,7 +584,6 @@ class Layout(Box):
         Returns:
             The layout component.
         """
-        ...
 
 class LayoutNamespace(ComponentNamespace):
     drawer_sidebar = staticmethod(DrawerSidebar.create)
@@ -814,6 +810,5 @@ class LayoutNamespace(ComponentNamespace):
         Returns:
             The layout component.
         """
-        ...
 
 layout = LayoutNamespace()

reflex/istate/manager.py

@@ -17,7 +17,8 @@ from redis.asyncio.client import PubSub
 from typing_extensions import override
 
 from reflex import constants
-from reflex.config import environment, get_config
+from reflex.config import get_config
+from reflex.environment import environment
 from reflex.state import BaseState, _split_substate_key, _substate_key
 from reflex.utils import console, path_ops, prerequisites
 from reflex.utils.exceptions import (
@@ -66,9 +67,8 @@ class StateManager(ABC):
                     lock_expiration=config.redis_lock_expiration,
                     lock_warning_threshold=config.redis_lock_warning_threshold,
                 )
-        raise InvalidStateManagerModeError(
-            f"Expected one of: DISK, MEMORY, REDIS, got {config.state_manager_mode}"
-        )
+        msg = f"Expected one of: DISK, MEMORY, REDIS, got {config.state_manager_mode}"
+        raise InvalidStateManagerModeError(msg)
 
     @abstractmethod
     async def get_state(self, token: str) -> BaseState:
@@ -80,7 +80,6 @@ class StateManager(ABC):
         Returns:
             The state for the token.
         """
-        pass
 
     @abstractmethod
     async def set_state(self, token: str, state: BaseState):
@@ -90,7 +89,6 @@ class StateManager(ABC):
             token: The token to set the state for.
             state: The state to set.
         """
-        pass
 
     @abstractmethod
     @contextlib.asynccontextmanager
@@ -145,7 +143,6 @@ class StateManagerMemory(StateManager):
             token: The token to set the state for.
             state: The state to set.
         """
-        pass
 
     @override
     @contextlib.asynccontextmanager
@@ -269,6 +266,7 @@ class StateManagerDisk(StateManager):
                     return BaseState._deserialize(fp=file)
             except Exception:
                 pass
+        return None
 
     async def populate_substates(
         self, client_token: str, state: BaseState, root_state: BaseState
@@ -449,9 +447,8 @@ class StateManagerRedis(StateManager):
             InvalidLockWarningThresholdError: If the lock warning threshold is invalid.
         """
         if self.lock_warning_threshold >= (lock_expiration := self.lock_expiration):
-            raise InvalidLockWarningThresholdError(
-                f"The lock warning threshold({self.lock_warning_threshold}) must be less than the lock expiration time({lock_expiration})."
-            )
+            msg = f"The lock warning threshold({self.lock_warning_threshold}) must be less than the lock expiration time({lock_expiration})."
+            raise InvalidLockWarningThresholdError(msg)
 
     def _get_required_state_classes(
         self,
@@ -557,9 +554,8 @@ class StateManagerRedis(StateManager):
             # Get the State class associated with the given path.
             state_cls = self.state.get_class_substate(state_path)
         else:
-            raise RuntimeError(
-                f"StateManagerRedis requires token to be specified in the form of {{token}}_{{state_full_name}}, but got {token}"
-            )
+            msg = f"StateManagerRedis requires token to be specified in the form of {{token}}_{{state_full_name}}, but got {token}"
+            raise RuntimeError(msg)
 
         # Determine which states we already have.
         flat_state_tree: dict[str, BaseState] = (
@@ -601,11 +597,12 @@ class StateManagerRedis(StateManager):
                 )
                 parent_state = flat_state_tree.get(parent_state_name)
                 if parent_state is None:
-                    raise RuntimeError(
+                    msg = (
                         f"Parent state for {state.get_full_name()} was not found "
                         "in the state tree, but should have already been fetched. "
-                        "This is a bug",
+                        "This is a bug"
                     )
+                    raise RuntimeError(msg)
                 parent_state.substates[state_name] = state
                 state.parent_state = parent_state
 
@@ -638,12 +635,13 @@ class StateManagerRedis(StateManager):
             lock_id is not None
             and await self.redis.get(self._lock_key(token)) != lock_id
         ):
-            raise LockExpiredError(
+            msg = (
                 f"Lock expired for token {token} while processing. Consider increasing "
                 f"`app.state_manager.lock_expiration` (currently {self.lock_expiration}) "
                 "or use `@rx.event(background=True)` decorator for long-running tasks."
             )
-        elif lock_id is not None:
+            raise LockExpiredError(msg)
+        if lock_id is not None:
             time_taken = self.lock_expiration / 1000 - (
                 await self.redis.ttl(self._lock_key(token))
             )
@@ -657,9 +655,8 @@ class StateManagerRedis(StateManager):
         client_token, substate_name = _split_substate_key(token)
         # If the substate name on the token doesn't match the instance name, it cannot have a parent.
         if state.parent_state is not None and state.get_full_name() != substate_name:
-            raise RuntimeError(
-                f"Cannot `set_state` with mismatching token {token} and substate {state.get_full_name()}."
-            )
+            msg = f"Cannot `set_state` with mismatching token {token} and substate {state.get_full_name()}."
+            raise RuntimeError(msg)
 
         # Recursively set_state on all known substates.
         tasks = [

reflex/istate/proxy.py

@@ -122,9 +122,8 @@ class StateProxy(wrapt.ObjectProxy):
             self._self_actx_lock.locked()
             and current_task == self._self_actx_lock_holder
         ):
-            raise ImmutableStateError(
-                "The state is already mutable. Do not nest `async with self` blocks."
-            )
+            msg = "The state is already mutable. Do not nest `async with self` blocks."
+            raise ImmutableStateError(msg)
 
         from reflex.state import _substate_key
 
@@ -173,7 +172,8 @@ class StateProxy(wrapt.ObjectProxy):
         Raises:
             TypeError: always, because only async contextmanager protocol is supported.
         """
-        raise TypeError("Background task must use `async with self` to modify state.")
+        msg = "Background task must use `async with self` to modify state."
+        raise TypeError(msg)
 
     def __exit__(self, *exc_info: Any) -> None:
         """Exit the regular context manager protocol.
@@ -181,7 +181,6 @@ class StateProxy(wrapt.ObjectProxy):
         Args:
             exc_info: The exception info tuple.
         """
-        pass
 
     def __getattr__(self, name: str) -> Any:
         """Get the attribute from the underlying state instance.
@@ -196,10 +195,11 @@ class StateProxy(wrapt.ObjectProxy):
             ImmutableStateError: If the state is not in mutable mode.
         """
         if name in ["substates", "parent_state"] and not self._is_mutable():
-            raise ImmutableStateError(
+            msg = (
                 "Background task StateProxy is immutable outside of a context "
                 "manager. Use `async with self` to modify state."
             )
+            raise ImmutableStateError(msg)
 
         value = super().__getattr__(name)
         if not name.startswith("_self_") and isinstance(value, MutableProxy):
@@ -243,10 +243,11 @@ class StateProxy(wrapt.ObjectProxy):
             super().__setattr__(name, value)
             return
 
-        raise ImmutableStateError(
+        msg = (
             "Background task StateProxy is immutable outside of a context "
             "manager. Use `async with self` to modify state."
         )
+        raise ImmutableStateError(msg)
 
     def get_substate(self, path: Sequence[str]) -> BaseState:
         """Only allow substate access with lock held.
@@ -261,10 +262,11 @@ class StateProxy(wrapt.ObjectProxy):
             ImmutableStateError: If the state is not in mutable mode.
         """
         if not self._is_mutable():
-            raise ImmutableStateError(
+            msg = (
                 "Background task StateProxy is immutable outside of a context "
                 "manager. Use `async with self` to modify state."
             )
+            raise ImmutableStateError(msg)
         return self.__wrapped__.get_substate(path)
 
     async def get_state(self, state_cls: type[BaseState]) -> BaseState:
@@ -280,10 +282,11 @@ class StateProxy(wrapt.ObjectProxy):
             ImmutableStateError: If the state is not in mutable mode.
         """
         if not self._is_mutable():
-            raise ImmutableStateError(
+            msg = (
                 "Background task StateProxy is immutable outside of a context "
                 "manager. Use `async with self` to modify state."
             )
+            raise ImmutableStateError(msg)
         return type(self)(
             await self.__wrapped__.get_state(state_cls), parent_state_proxy=self
         )
@@ -323,7 +326,8 @@ class ReadOnlyStateProxy(StateProxy):
             # Special case attributes of the proxy itself, not applied to the wrapped object.
             super().__setattr__(name, value)
             return
-        raise NotImplementedError("This is a read-only state proxy.")
+        msg = "This is a read-only state proxy."
+        raise NotImplementedError(msg)
 
     def mark_dirty(self):
         """Mark the state as dirty.
@@ -331,7 +335,8 @@ class ReadOnlyStateProxy(StateProxy):
         Raises:
             NotImplementedError: Always raised when trying to mark the proxied state as dirty.
         """
-        raise NotImplementedError("This is a read-only state proxy.")
+        msg = "This is a read-only state proxy."
+        raise NotImplementedError(msg)
 
 
 class MutableProxy(wrapt.ObjectProxy):
@@ -460,6 +465,7 @@ class MutableProxy(wrapt.ObjectProxy):
         self._self_state._mark_dirty()
         if wrapped is not None:
             return wrapped(*args, **(kwargs or {}))
+        return None
 
     @classmethod
     def _is_mutable_type(cls, value: Any) -> bool:
@@ -748,10 +754,11 @@ class ImmutableMutableProxy(MutableProxy):
             ImmutableStateError: if the StateProxy is not mutable.
         """
         if not self._self_state._is_mutable():
-            raise ImmutableStateError(
+            msg = (
                 "Background task StateProxy is immutable outside of a context "
                 "manager. Use `async with self` to modify state."
             )
+            raise ImmutableStateError(msg)
         return super()._mark_dirty(
             wrapped=wrapped, instance=instance, args=args, kwargs=kwargs
         )

reflex/model.py

@@ -21,7 +21,8 @@ import sqlalchemy.orm
 from alembic.runtime.migration import MigrationContext
 
 from reflex.base import Base
-from reflex.config import environment, get_config
+from reflex.config import get_config
+from reflex.environment import environment
 from reflex.utils import console
 from reflex.utils.compat import sqlmodel, sqlmodel_field_has_primary_key
 
@@ -83,7 +84,8 @@ def get_engine(url: str | None = None) -> sqlalchemy.engine.Engine:
     conf = get_config()
     url = url or conf.db_url
     if url is None:
-        raise ValueError("No database url configured")
+        msg = "No database url configured"
+        raise ValueError(msg)
 
     global _ENGINE
     if url in _ENGINE:
@@ -125,7 +127,8 @@ def get_async_engine(url: str | None) -> sqlalchemy.ext.asyncio.AsyncEngine:
                     f"db_url `{_safe_db_url_for_logging(conf.db_url)}`."
                 )
     if url is None:
-        raise ValueError("No async database url configured")
+        msg = "No async database url configured"
+        raise ValueError(msg)
 
     global _ASYNC_ENGINE
     if url in _ASYNC_ENGINE:
@@ -271,7 +274,7 @@ class Model(Base, sqlmodel.SQLModel):  # pyright: ignore [reportGeneralTypeIssue
         """
         if hasattr(value, "dict"):
             return value.dict()
-        elif isinstance(value, list):
+        if isinstance(value, list):
             return [cls._dict_recursive(item) for item in value]
         return value
 
@@ -481,7 +484,7 @@ class Model(Base, sqlmodel.SQLModel):  # pyright: ignore [reportGeneralTypeIssue
             None - indicating the process was skipped.
         """
         if not environment.ALEMBIC_CONFIG.get().exists():
-            return
+            return None
 
         with cls.get_db_engine().connect() as connection:
             cls._alembic_upgrade(connection=connection)