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

reflex/components/component.py

@@ -157,7 +157,8 @@ class ComponentField(Generic[FIELD_TYPE]):
             return self.default
         if self.default_factory is not None:
             return self.default_factory()
-        raise ValueError("No default value or factory provided.")
+        msg = "No default value or factory provided."
+        raise ValueError(msg)
 
     def __repr__(self) -> str:
         """Represent the field in a readable format.
@@ -194,7 +195,8 @@ def field(
         ValueError: If both default and default_factory are specified.
     """
     if default is not MISSING and default_factory is not None:
-        raise ValueError("cannot specify both default and default_factory")
+        msg = "cannot specify both default and default_factory"
+        raise ValueError(msg)
     return ComponentField(  # pyright: ignore [reportReturnType]
         default=default,
         default_factory=default_factory,
@@ -286,10 +288,11 @@ class BaseComponentMeta(ABCMeta):
 
         namespace["_own_fields"] = own_fields
         namespace["_inherited_fields"] = inherited_fields
-        namespace["_fields"] = inherited_fields | own_fields
+        all_fields = inherited_fields | own_fields
+        namespace["_fields"] = all_fields
         namespace["_js_fields"] = {
             key: value
-            for key, value in own_fields.items()
+            for key, value in all_fields.items()
             if value.is_javascript is True
         }
         return super().__new__(cls, name, bases, namespace)
@@ -769,11 +772,12 @@ class Component(BaseComponent, ABC):
                 and key not in component_specific_triggers
                 and key not in props
             ):
-                raise ValueError(
+                msg = (
                     f"The {(comp_name := type(self).__name__)} does not take in an `{key}` event trigger. If {comp_name}"
                     f" is a third party component make sure to add `{key}` to the component's event triggers. "
                     f"visit https://reflex.dev/docs/wrapping-react/guide/#event-triggers for more info."
                 )
+                raise ValueError(msg)
             if key in component_specific_triggers:
                 # Event triggers are bound to event chains.
                 is_var = False
@@ -844,7 +848,8 @@ class Component(BaseComponent, ABC):
         style = kwargs.get("style", {})
         if isinstance(style, Sequence):
             if any(not isinstance(s, Mapping) for s in style):
-                raise TypeError("Style must be a dictionary or a list of dictionaries.")
+                msg = "Style must be a dictionary or a list of dictionaries."
+                raise TypeError(msg)
             # Merge styles, the later ones overriding keys in the earlier ones.
             style = {
                 k: v
@@ -879,14 +884,12 @@ class Component(BaseComponent, ABC):
                     if not isinstance(c, StringVar) and not issubclass(
                         c._var_type, str
                     ):
-                        raise TypeError(
-                            f"Invalid class_name passed for prop {type(self).__name__}.class_name, expected type str, got value {c._js_expr} of type {c._var_type}."
-                        )
+                        msg = f"Invalid class_name passed for prop {type(self).__name__}.class_name, expected type str, got value {c._js_expr} of type {c._var_type}."
+                        raise TypeError(msg)
                     has_var = True
                 else:
-                    raise TypeError(
-                        f"Invalid class_name passed for prop {type(self).__name__}.class_name, expected type str, got value {c} of type {type(c)}."
-                    )
+                    msg = f"Invalid class_name passed for prop {type(self).__name__}.class_name, expected type str, got value {c} of type {type(c)}."
+                    raise TypeError(msg)
             if has_var:
                 kwargs["class_name"] = LiteralArrayVar.create(
                     class_name, _var_type=list[str]
@@ -898,9 +901,8 @@ class Component(BaseComponent, ABC):
             and not isinstance(class_name, StringVar)
             and not issubclass(class_name._var_type, str)
         ):
-            raise TypeError(
-                f"Invalid class_name passed for prop {type(self).__name__}.class_name, expected type str, got value {class_name._js_expr} of type {class_name._var_type}."
-            )
+            msg = f"Invalid class_name passed for prop {type(self).__name__}.class_name, expected type str, got value {class_name._js_expr} of type {class_name._var_type}."
+            raise TypeError(msg)
         # Construct the component.
         for key, value in kwargs.items():
             setattr(self, key, value)
@@ -1015,7 +1017,7 @@ class Component(BaseComponent, ABC):
         Returns:
             The unique fields.
         """
-        return set(cls.get_fields()) - set(Component.get_fields())
+        return set(cls.get_js_fields())
 
     @classmethod
     @functools.cache
@@ -1227,9 +1229,8 @@ class Component(BaseComponent, ABC):
         """
         # 1. Default style from `_add_style`/`add_style`.
         if type(self)._add_style != Component._add_style:
-            raise UserWarning(
-                "Do not override _add_style directly. Use add_style instead."
-            )
+            msg = "Do not override _add_style directly. Use add_style instead."
+            raise UserWarning(msg)
         new_style = self._add_style()
         style_vars = [new_style._var_data]
 
@@ -1347,9 +1348,8 @@ class Component(BaseComponent, ABC):
                 validate_child(child.default)
 
             if self._invalid_children and child_name in self._invalid_children:
-                raise ValueError(
-                    f"The component `{comp_name}` cannot have `{child_name}` as a child component"
-                )
+                msg = f"The component `{comp_name}` cannot have `{child_name}` as a child component"
+                raise ValueError(msg)
 
             if self._valid_children and child_name not in [
                 *self._valid_children,
@@ -1358,9 +1358,8 @@ class Component(BaseComponent, ABC):
                 valid_child_list = ", ".join(
                     [f"`{v_child}`" for v_child in self._valid_children]
                 )
-                raise ValueError(
-                    f"The component `{comp_name}` only allows the components: {valid_child_list} as children. Got `{child_name}` instead."
-                )
+                msg = f"The component `{comp_name}` only allows the components: {valid_child_list} as children. Got `{child_name}` instead."
+                raise ValueError(msg)
 
             if child._valid_parents and all(
                 clz_name not in [*child._valid_parents, *allowed_components]
@@ -1369,9 +1368,8 @@ class Component(BaseComponent, ABC):
                 valid_parent_list = ", ".join(
                     [f"`{v_parent}`" for v_parent in child._valid_parents]
                 )
-                raise ValueError(
-                    f"The component `{child_name}` can only be a child of the components: {valid_parent_list}. Got `{comp_name}` instead."
-                )
+                msg = f"The component `{child_name}` can only be a child of the components: {valid_parent_list}. Got `{comp_name}` instead."
+                raise ValueError(msg)
 
         for child in children:
             validate_child(child)
@@ -1502,13 +1500,9 @@ class Component(BaseComponent, ABC):
         """
         if self.event_triggers and self._event_trigger_values_use_state():
             return True
-        else:
-            for child in self.children:
-                if (
-                    isinstance(child, Component)
-                    and child._has_stateful_event_triggers()
-                ):
-                    return True
+        for child in self.children:
+            if isinstance(child, Component) and child._has_stateful_event_triggers():
+                return True
         return False
 
     @classmethod
@@ -1767,6 +1761,7 @@ class Component(BaseComponent, ABC):
                         {on_unmount or ""}
                     }}
                 }}, []);"""
+        return None
 
     def _get_ref_hook(self) -> Var | None:
         """Generate the ref hook for the component.
@@ -1780,6 +1775,7 @@ class Component(BaseComponent, ABC):
                 f"const {ref} = useRef(null); {Var(_js_expr=ref)._as_ref()!s} = {ref};",
                 _var_data=VarData(position=Hooks.HookPosition.INTERNAL),
             )
+        return None
 
     def _get_vars_hooks(self) -> dict[str, VarData | None]:
         """Get the hooks required by vars referenced in this component.
@@ -1980,24 +1976,38 @@ class Component(BaseComponent, ABC):
         """
         return {}
 
-    def _get_all_app_wrap_components(self) -> dict[tuple[int, str], Component]:
+    def _get_all_app_wrap_components(
+        self, *, ignore_ids: set[int] | None = None
+    ) -> dict[tuple[int, str], Component]:
         """Get the app wrap components for the component and its children.
 
+        Args:
+            ignore_ids: A set of component IDs to ignore. Used to avoid duplicates.
+
         Returns:
             The app wrap components.
         """
+        ignore_ids = ignore_ids or set()
         # Store the components in a set to avoid duplicates.
         components = self._get_app_wrap_components()
 
         for component in tuple(components.values()):
-            components.update(component._get_all_app_wrap_components())
+            component_id = id(component)
+            if component_id in ignore_ids:
+                continue
+            ignore_ids.add(component_id)
+            components.update(
+                component._get_all_app_wrap_components(ignore_ids=ignore_ids)
+            )
 
         # Add the app wrap components for the children.
         for child in self.children:
+            child_id = id(child)
             # Skip BaseComponent and StatefulComponent children.
-            if not isinstance(child, Component):
+            if not isinstance(child, Component) or child_id in ignore_ids:
                 continue
-            components.update(child._get_all_app_wrap_components())
+            ignore_ids.add(child_id)
+            components.update(child._get_all_app_wrap_components(ignore_ids=ignore_ids))
 
         # Return the components.
         return components
@@ -2198,7 +2208,35 @@ class CustomComponent(Component):
         Returns:
             The code to render the component.
         """
-        return self.component_fn(*self.get_prop_vars())
+        component = self.component_fn(*self.get_prop_vars())
+
+        try:
+            from reflex.utils.prerequisites import get_and_validate_app
+
+            style = get_and_validate_app().app.style
+        except Exception:
+            style = {}
+
+        component._add_style_recursive(style)
+        return component
+
+    def _get_all_app_wrap_components(
+        self, *, ignore_ids: set[int] | None = None
+    ) -> dict[tuple[int, str], Component]:
+        """Get the app wrap components for the custom component.
+
+        Args:
+            ignore_ids: A set of IDs to ignore to avoid infinite recursion.
+
+        Returns:
+            The app wrap components.
+        """
+        ignore_ids = ignore_ids or set()
+        component = self.get_component()
+        if id(component) in ignore_ids:
+            return {}
+        ignore_ids.add(id(component))
+        return self.get_component()._get_all_app_wrap_components(ignore_ids=ignore_ids)
 
 
 CUSTOM_COMPONENTS: dict[str, CustomComponent] = {}
@@ -2222,7 +2260,7 @@ def _register_custom_component(
                 _var_type=unwrap_var_annotation(annotation),
             ).guess_type()
             if not types.safe_issubclass(annotation, EventHandler)
-            else EventSpec(handler=EventHandler(fn=lambda: []))
+            else EventSpec(handler=EventHandler(fn=no_args_event_spec))
         )
         for prop, annotation in typing.get_type_hints(component_fn).items()
         if prop != "return"
@@ -2233,7 +2271,8 @@ def _register_custom_component(
         **dummy_props,
     )
     if dummy_component.tag is None:
-        raise TypeError(f"Could not determine the tag name for {component_fn!r}")
+        msg = f"Could not determine the tag name for {component_fn!r}"
+        raise TypeError(msg)
     CUSTOM_COMPONENTS[dummy_component.tag] = dummy_component
 
 
@@ -2312,7 +2351,8 @@ class NoSSRComponent(Component):
         # extract the correct import name from library name
         base_import_name = self._get_import_name()
         if base_import_name is None:
-            raise ValueError("Undefined library for NoSSRComponent")
+            msg = "Undefined library for NoSSRComponent"
+            raise ValueError(msg)
         import_name = format.format_library_name(base_import_name)
 
         library_import = f"const {self.alias if self.alias else self.tag} = dynamic(() => import('{import_name}')"
@@ -2320,7 +2360,7 @@ class NoSSRComponent(Component):
             # https://nextjs.org/docs/pages/building-your-application/optimizing/lazy-loading#with-named-exports
             f".then((mod) => mod.{self.tag})" if not self.is_default else ""
         )
-        return "".join((library_import, mod_import, opts_fragment))
+        return library_import + mod_import + opts_fragment
 
 
 class StatefulComponent(BaseComponent):
@@ -2540,7 +2580,7 @@ class StatefulComponent(BaseComponent):
         var_name = var_name.strip()
 
         # Break up array and object destructuring if used.
-        if var_name.startswith("[") or var_name.startswith("{"):
+        if var_name.startswith(("[", "{")):
             return [
                 v.strip().replace("...", "") for v in var_name.strip("[]{}").split(",")
             ]

reflex/components/core/auto_scroll.pyi

@@ -256,7 +256,6 @@ class AutoScroll(Div):
         Returns:
             An AutoScroll component.
         """
-        ...
 
     def add_imports(self) -> ImportDict | list[ImportDict]: ...
     def add_hooks(self) -> list[str | Var]: ...

reflex/components/core/banner.py

@@ -16,9 +16,9 @@ from reflex.components.radix.themes.components.dialog import (
 from reflex.components.radix.themes.layout.flex import Flex
 from reflex.components.radix.themes.typography.text import Text
 from reflex.components.sonner.toast import ToastProps, toast_ref
-from reflex.config import environment
 from reflex.constants import Dirs, Hooks, Imports
 from reflex.constants.compiler import CompileVars
+from reflex.environment import environment
 from reflex.utils.imports import ImportVar
 from reflex.vars import VarData
 from reflex.vars.base import LiteralVar, Var

reflex/components/core/banner.pyi

@@ -93,7 +93,6 @@ class ConnectionToaster(Fragment):
         Returns:
             The connection toaster component.
         """
-        ...
 
 class ConnectionBanner(Component):
     @overload
@@ -137,7 +136,6 @@ class ConnectionBanner(Component):
         Returns:
             The connection banner component.
         """
-        ...
 
 class ConnectionModal(Component):
     @overload
@@ -181,7 +179,6 @@ class ConnectionModal(Component):
         Returns:
             The connection banner component.
         """
-        ...
 
 class WifiOffPulse(Icon):
     @overload
@@ -235,7 +232,6 @@ class WifiOffPulse(Icon):
         Returns:
             The icon component with default props applied.
         """
-        ...
 
     def add_imports(self) -> dict[str, str | ImportVar | list[str | ImportVar]]: ...
 
@@ -482,7 +478,6 @@ class ConnectionPulser(Div):
         Returns:
             The connection pulser component.
         """
-        ...
 
 class BackendDisabled(Div):
     @overload
@@ -727,7 +722,6 @@ class BackendDisabled(Div):
         Returns:
             The backend disabled component.
         """
-        ...
 
 connection_banner = ConnectionBanner.create
 connection_modal = ConnectionModal.create

reflex/components/core/breakpoints.py

@@ -75,19 +75,17 @@ class Breakpoints(dict[K, V]):
 
         if custom is not None:
             if any(threshold is not None for threshold in thresholds):
-                raise ValueError("Named props cannot be used with custom thresholds")
+                msg = "Named props cannot be used with custom thresholds"
+                raise ValueError(msg)
 
             return Breakpoints(custom)
-        else:
-            return Breakpoints(
-                {
-                    k: v
-                    for k, v in zip(
-                        ["initial", *breakpoint_names], thresholds, strict=True
-                    )
-                    if v is not None
-                }
-            )
+        return Breakpoints(
+            {
+                k: v
+                for k, v in zip(["initial", *breakpoint_names], thresholds, strict=True)
+                if v is not None
+            }
+        )
 
 
 breakpoints = Breakpoints.create

reflex/components/core/client_side_routing.pyi

@@ -65,7 +65,6 @@ class ClientSideRouting(Component):
         Returns:
             The component.
         """
-        ...
 
 def wait_for_client_redirect(component: Component) -> Component: ...
 
@@ -120,4 +119,3 @@ class Default404Page(Component):
         Returns:
             The component.
         """
-        ...

reflex/components/core/clipboard.pyi

@@ -70,7 +70,6 @@ class Clipboard(Fragment):
         Returns:
             The Clipboard Component.
         """
-        ...
 
     def add_imports(self) -> dict[str, ImportVar]: ...
     def add_hooks(self) -> list[str | Var[str]]: ...

reflex/components/core/colors.py

@@ -32,19 +32,22 @@ def color(
     """
     if isinstance(color, str):
         if color not in COLORS and REFLEX_VAR_OPENING_TAG not in color:
-            raise ValueError(f"Color must be one of {COLORS}, received {color}")
+            msg = f"Color must be one of {COLORS}, received {color}"
+            raise ValueError(msg)
     elif not isinstance(color, Var):
-        raise ValueError("Color must be a string or a Var")
+        msg = "Color must be a string or a Var"
+        raise ValueError(msg)
 
     if isinstance(shade, int):
         if shade < MIN_SHADE_VALUE or shade > MAX_SHADE_VALUE:
-            raise ValueError(
-                f"Shade must be between {MIN_SHADE_VALUE} and {MAX_SHADE_VALUE}"
-            )
+            msg = f"Shade must be between {MIN_SHADE_VALUE} and {MAX_SHADE_VALUE}"
+            raise ValueError(msg)
     elif not isinstance(shade, Var):
-        raise ValueError("Shade must be an integer or a Var")
+        msg = "Shade must be an integer or a Var"
+        raise ValueError(msg)
 
     if not isinstance(alpha, (bool, Var)):
-        raise ValueError("Alpha must be a boolean or a Var")
+        msg = "Alpha must be a boolean or a Var"
+        raise ValueError(msg)
 
     return Color(color, shade, alpha)

reflex/components/core/cond.py

@@ -132,7 +132,8 @@ def cond(condition: Any, c1: Any, c2: Any = types.Unset(), /) -> Component | Var
     # Convert the condition to a Var.
     cond_var = LiteralVar.create(condition)
     if cond_var is None:
-        raise ValueError("The condition must be set.")
+        msg = "The condition must be set."
+        raise ValueError(msg)
 
     # If the first component is a component, create a Cond component.
     if isinstance(c1, BaseComponent):
@@ -145,7 +146,8 @@ def cond(condition: Any, c1: Any, c2: Any = types.Unset(), /) -> Component | Var
     if isinstance(c2, BaseComponent):
         return Cond.create(cond_var.bool(), Fragment.create(c1), c2)
     if isinstance(c2, types.Unset):
-        raise ValueError("For conditional vars, the second argument must be set.")
+        msg = "For conditional vars, the second argument must be set."
+        raise ValueError(msg)
 
     # convert the truth and false cond parts into vars so the _var_data can be obtained.
     c1_var = Var.create(c1)

reflex/components/core/debounce.py

@@ -70,14 +70,16 @@ class DebounceInput(Component):
             ValueError: if the child element does not have an on_change handler.
         """
         if len(children) != 1:
-            raise RuntimeError(
+            msg = (
                 "Provide a single child for DebounceInput, such as rx.input() or "
-                "rx.text_area()",
+                "rx.text_area()"
             )
+            raise RuntimeError(msg)
 
         child = children[0]
         if "on_change" not in child.event_triggers:
-            raise ValueError("DebounceInput child requires an on_change handler")
+            msg = "DebounceInput child requires an on_change handler"
+            raise ValueError(msg)
 
         # Carry known props and event_triggers from the child.
         props_from_child = {

reflex/components/core/debounce.pyi

@@ -74,6 +74,5 @@ class DebounceInput(Component):
             RuntimeError: unless exactly one child element is provided.
             ValueError: if the child element does not have an on_change handler.
         """
-        ...
 
 debounce_input = DebounceInput.create

reflex/components/core/foreach.py

@@ -69,19 +69,19 @@ class Foreach(Component):
         )
 
         if iterable._var_type == Any:
-            raise ForeachVarError(
+            msg = (
                 f"Could not foreach over var `{iterable!s}` of type Any. "
                 "(If you are trying to foreach over a state var, add a type annotation to the var). "
                 "See https://reflex.dev/docs/library/dynamic-rendering/foreach/"
             )
+            raise ForeachVarError(msg)
 
         if (
             hasattr(render_fn, "__qualname__")
             and render_fn.__qualname__ == ComponentState.create.__qualname__
         ):
-            raise TypeError(
-                "Using a ComponentState as `render_fn` inside `rx.foreach` is not supported yet."
-            )
+            msg = "Using a ComponentState as `render_fn` inside `rx.foreach` is not supported yet."
+            raise TypeError(msg)
 
         if isinstance(iterable, ObjectVar):
             iterable = iterable.entries()
@@ -90,10 +90,11 @@ class Foreach(Component):
             iterable = iterable.split()
 
         if not isinstance(iterable, ArrayVar):
-            raise ForeachVarError(
+            msg = (
                 f"Could not foreach over var `{iterable!s}` of type {iterable._var_type}. "
                 "See https://reflex.dev/docs/library/dynamic-rendering/foreach/"
             )
+            raise ForeachVarError(msg)
 
         if types.is_optional(iterable._var_type):
             iterable = cond(iterable, iterable, [])
@@ -122,11 +123,12 @@ class Foreach(Component):
 
         # Validate the render function signature.
         if len(params) == 0 or len(params) > 2:
-            raise ForeachRenderError(
+            msg = (
                 "Expected 1 or 2 parameters in foreach render function, got "
                 f"{[p.name for p in params]}. See "
                 "https://reflex.dev/docs/library/dynamic-rendering/foreach/"
             )
+            raise ForeachRenderError(msg)
 
         if len(params) >= 1:
             # Determine the arg var name based on the params accepted by render_fn.

reflex/components/core/html.py

@@ -30,9 +30,9 @@ class Html(Div):
         """
         # If children are not provided, throw an error.
         if len(children) != 1:
-            raise ValueError("Must provide children to the html component.")
-        else:
-            props["dangerouslySetInnerHTML"] = {"__html": children[0]}
+            msg = "Must provide children to the html component."
+            raise ValueError(msg)
+        props["dangerouslySetInnerHTML"] = {"__html": children[0]}
 
         # Apply the default classname
         given_class_name = props.pop("class_name", [])

reflex/components/core/html.pyi

@@ -260,6 +260,5 @@ class Html(Div):
         Raises:
             ValueError: If children are not provided or more than one child is provided.
         """
-        ...
 
 html = Html.create