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

reflex/compiler/compiler.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import sys
 from collections.abc import Iterable, Sequence
 from datetime import datetime
 from inspect import getmodule
@@ -200,15 +201,14 @@ def _validate_stylesheet(stylesheet_full_path: Path, assets_app_path: Path) -> N
     """
     suffix = stylesheet_full_path.suffix[1:] if stylesheet_full_path.suffix else ""
     if suffix not in constants.Reflex.STYLESHEETS_SUPPORTED:
-        raise ValueError(f"Stylesheet file {stylesheet_full_path} is not supported.")
+        msg = f"Stylesheet file {stylesheet_full_path} is not supported."
+        raise ValueError(msg)
     if not stylesheet_full_path.absolute().is_relative_to(assets_app_path.absolute()):
-        raise FileNotFoundError(
-            f"Cannot include stylesheets from outside the assets directory: {stylesheet_full_path}"
-        )
+        msg = f"Cannot include stylesheets from outside the assets directory: {stylesheet_full_path}"
+        raise FileNotFoundError(msg)
     if not stylesheet_full_path.name:
-        raise ValueError(
-            f"Stylesheet file name cannot be empty: {stylesheet_full_path}"
-        )
+        msg = f"Stylesheet file name cannot be empty: {stylesheet_full_path}"
+        raise ValueError(msg)
     if (
         len(
             stylesheet_full_path.absolute()
@@ -218,9 +218,8 @@ def _validate_stylesheet(stylesheet_full_path: Path, assets_app_path: Path) -> N
         == 1
         and stylesheet_full_path.stem == PageNames.STYLESHEET_ROOT
     ):
-        raise ValueError(
-            f"Stylesheet file name cannot be '{PageNames.STYLESHEET_ROOT}': {stylesheet_full_path}"
-        )
+        msg = f"Stylesheet file name cannot be '{PageNames.STYLESHEET_ROOT}': {stylesheet_full_path}"
+        raise ValueError(msg)
 
 
 RADIX_THEMES_STYLESHEET = "@radix-ui/themes/styles.css"
@@ -257,9 +256,8 @@ def _compile_root_stylesheet(stylesheets: list[str]) -> str:
             stylesheet_full_path = assets_app_path / stylesheet.strip("/")
 
             if not stylesheet_full_path.exists():
-                raise FileNotFoundError(
-                    f"The stylesheet file {stylesheet_full_path} does not exist."
-                )
+                msg = f"The stylesheet file {stylesheet_full_path} does not exist."
+                raise FileNotFoundError(msg)
 
             if stylesheet_full_path.is_dir():
                 all_files = (
@@ -685,12 +683,12 @@ def into_component(component: Component | ComponentCallable) -> Component:
     """
     if (converted := _into_component_once(component)) is not None:
         return converted
+    if not callable(component):
+        msg = f"Expected a Component or callable, got {component!r} of type {type(component)}"
+        raise TypeError(msg)
+
     try:
-        if (
-            callable(component)
-            and (converted := _into_component_once(component())) is not None
-        ):
-            return converted
+        component_called = component()
     except KeyError as e:
         if isinstance(e, ReflexError):
             raise
@@ -714,9 +712,10 @@ def into_component(component: Component | ComponentCallable) -> Component:
                     "Cannot pass a Var to a built-in function. Consider using .length() for accessing the length of an iterable Var."
                 ).with_traceback(e.__traceback__) from None
             if message.endswith(
-                "indices must be integers or slices, not NumberCastedVar"
-            ) or message.endswith(
-                "indices must be integers or slices, not BooleanCastedVar"
+                (
+                    "indices must be integers or slices, not NumberCastedVar",
+                    "indices must be integers or slices, not BooleanCastedVar",
+                )
             ):
                 raise TypeError(
                     "Cannot index into a primitive sequence with a Var. Consider calling rx.Var.create() on the sequence."
@@ -727,7 +726,11 @@ def into_component(component: Component | ComponentCallable) -> Component:
             ).with_traceback(e.__traceback__) from None
         raise
 
-    raise TypeError(f"Expected a Component, got {type(component)}")
+    if (converted := _into_component_once(component_called)) is not None:
+        return converted
+
+    msg = f"Expected a Component, got {component_called!r} of type {type(component_called)}"
+    raise TypeError(msg)
 
 
 def compile_unevaluated_page(
@@ -748,52 +751,61 @@ def compile_unevaluated_page(
 
     Returns:
         The compiled component and whether state should be enabled.
+
+    Raises:
+        Exception: If an error occurs while evaluating the page.
     """
-    # Generate the component if it is a callable.
-    component = into_component(page.component)
+    try:
+        # Generate the component if it is a callable.
+        component = into_component(page.component)
 
-    component._add_style_recursive(style or {}, theme)
+        component._add_style_recursive(style or {}, theme)
 
-    enable_state = False
-    # Ensure state is enabled if this page uses state.
-    if state is None:
-        if page.on_load or component._has_stateful_event_triggers():
-            enable_state = True
-        else:
-            for var in component._get_vars(include_children=True):
-                var_data = var._get_all_var_data()
-                if not var_data:
-                    continue
-                if not var_data.state:
-                    continue
+        enable_state = False
+        # Ensure state is enabled if this page uses state.
+        if state is None:
+            if page.on_load or component._has_stateful_event_triggers():
                 enable_state = True
-                break
-
-    from reflex.app import OverlayFragment
-    from reflex.utils.format import make_default_page_title
-
-    component = OverlayFragment.create(component)
-
-    meta_args = {
-        "title": (
-            page.title
-            if page.title is not None
-            else make_default_page_title(get_config().app_name, route)
-        ),
-        "image": page.image,
-        "meta": page.meta,
-    }
-
-    if page.description is not None:
-        meta_args["description"] = page.description
-
-    # Add meta information to the component.
-    utils.add_meta(
-        component,
-        **meta_args,
-    )
+            else:
+                for var in component._get_vars(include_children=True):
+                    var_data = var._get_all_var_data()
+                    if not var_data:
+                        continue
+                    if not var_data.state:
+                        continue
+                    enable_state = True
+                    break
+
+        from reflex.app import OverlayFragment
+        from reflex.utils.format import make_default_page_title
+
+        component = OverlayFragment.create(component)
+
+        meta_args = {
+            "title": (
+                page.title
+                if page.title is not None
+                else make_default_page_title(get_config().app_name, route)
+            ),
+            "image": page.image,
+            "meta": page.meta,
+        }
+
+        if page.description is not None:
+            meta_args["description"] = page.description
+
+        # Add meta information to the component.
+        utils.add_meta(
+            component,
+            **meta_args,
+        )
 
-    return component, enable_state
+    except Exception as e:
+        if sys.version_info >= (3, 11):
+            e.add_note(f"Happened while evaluating page {route!r}")
+        raise
+    else:
+        return component, enable_state
 
 
 class ExecutorSafeFunctions:
@@ -873,5 +885,6 @@ class ExecutorSafeFunctions:
             ValueError: If the style is not set.
         """
         if style is None:
-            raise ValueError("STYLE should be set")
+            msg = "STYLE should be set"
+            raise ValueError(msg)
         return compile_theme(style)

reflex/compiler/utils.py

@@ -60,7 +60,8 @@ def compile_import_statement(fields: list[ImportVar]) -> tuple[str, list[str]]:
     # Check for default imports.
     defaults = {field for field in fields_set if field.is_default}
     if len(defaults) >= 2:
-        raise ValueError("Only one default import is allowed.")
+        msg = "Only one default import is allowed."
+        raise ValueError(msg)
 
     # Get the default import, and the specific imports.
     default = next(iter({field.name for field in defaults}), "")
@@ -91,9 +92,8 @@ def validate_imports(import_dict: ParsedImportDict):
                 ):
                     used_tags[import_name] = lib if lib[0] == "$" else already_imported
                     continue
-                raise ValueError(
-                    f"Can not compile, the tag {import_name} is used multiple time from {lib} and {used_tags[import_name]}"
-                )
+                msg = f"Can not compile, the tag {import_name} is used multiple time from {lib} and {used_tags[import_name]}"
+                raise ValueError(msg)
             if import_name is not None:
                 used_tags[import_name] = lib
 
@@ -130,9 +130,11 @@ def compile_imports(import_dict: ParsedImportDict) -> list[dict]:
         for path, (default, rest) in compiled.items():
             if not lib:
                 if default:
-                    raise ValueError("No default field allowed for empty library.")
+                    msg = "No default field allowed for empty library."
+                    raise ValueError(msg)
                 if rest is None or len(rest) == 0:
-                    raise ValueError("No fields to import.")
+                    msg = "No fields to import."
+                    raise ValueError(msg)
                 import_dicts.extend(get_import_dict(module) for module in sorted(rest))
                 continue
 

reflex/components/base/app_wrap.pyi

@@ -50,4 +50,3 @@ class AppWrap(Fragment):
         Returns:
             A new AppWrap component containing {children}.
         """
-        ...

reflex/components/base/bare.py

@@ -43,9 +43,8 @@ def validate_str(value: str):
                 f"Output includes {value!s} which will be displayed as a string. If you are calling `str` on a Var, consider using .to_string() instead."
             )
         elif perf_mode == PerformanceMode.RAISE:
-            raise ValueError(
-                f"Output includes {value!s} which will be displayed as a string. If you are calling `str` on a Var, consider using .to_string() instead."
-            )
+            msg = f"Output includes {value!s} which will be displayed as a string. If you are calling `str` on a Var, consider using .to_string() instead."
+            raise ValueError(msg)
 
 
 def _components_from_var(var: Var) -> Sequence[BaseComponent]:
@@ -72,10 +71,9 @@ class Bare(Component):
             if isinstance(contents, LiteralStringVar):
                 validate_str(contents._var_value)
             return cls._unsafe_create(children=[], contents=contents)
-        else:
-            if isinstance(contents, str):
-                validate_str(contents)
-            contents = Var.create(contents if contents is not None else "")
+        if isinstance(contents, str):
+            validate_str(contents)
+        contents = Var.create(contents if contents is not None else "")
 
         return cls._unsafe_create(children=[], contents=contents)
 

reflex/components/base/body.pyi

@@ -255,4 +255,3 @@ class Body(elements.Body):
         Returns:
             The component.
         """
-        ...

reflex/components/base/document.pyi

@@ -61,7 +61,6 @@ class NextDocumentLib(Component):
         Returns:
             The component.
         """
-        ...
 
 class Html(NextDocumentLib):
     @overload
@@ -114,7 +113,6 @@ class Html(NextDocumentLib):
         Returns:
             The component.
         """
-        ...
 
 class DocumentHead(NextDocumentLib):
     @overload
@@ -166,7 +164,6 @@ class DocumentHead(NextDocumentLib):
         Returns:
             The component.
         """
-        ...
 
 class Main(NextDocumentLib):
     @overload
@@ -218,7 +215,6 @@ class Main(NextDocumentLib):
         Returns:
             The component.
         """
-        ...
 
 class NextScript(NextDocumentLib):
     @overload
@@ -270,4 +266,3 @@ class NextScript(NextDocumentLib):
         Returns:
             The component.
         """
-        ...

reflex/components/base/error_boundary.pyi

@@ -70,6 +70,5 @@ class ErrorBoundary(Component):
         Returns:
             The ErrorBoundary component.
         """
-        ...
 
 error_boundary = ErrorBoundary.create

reflex/components/base/fragment.pyi

@@ -61,6 +61,5 @@ class Fragment(Component):
         Returns:
             The component.
         """
-        ...
 
 fragment = Fragment.create

reflex/components/base/head.pyi

@@ -61,7 +61,6 @@ class NextHeadLib(Component):
         Returns:
             The component.
         """
-        ...
 
 class Head(NextHeadLib, MemoizationLeaf):
     @overload
@@ -113,6 +112,5 @@ class Head(NextHeadLib, MemoizationLeaf):
         Returns:
             The memoization leaf
         """
-        ...
 
 head = Head.create

reflex/components/base/link.pyi

@@ -259,7 +259,6 @@ class RawLink(BaseHTML):
         Returns:
             The component.
         """
-        ...
 
 class ScriptTag(BaseHTML):
     @overload
@@ -519,4 +518,3 @@ class ScriptTag(BaseHTML):
         Returns:
             The component.
         """
-        ...

reflex/components/base/meta.py

@@ -20,7 +20,8 @@ class Title(elements.Title):
         """
         # Make sure the title is a single string.
         if len(self.children) != 1 or not isinstance(self.children[0], Bare):
-            raise ValueError("Title must be a single string.")
+            msg = "Title must be a single string."
+            raise ValueError(msg)
         return super().render()
 
 

reflex/components/base/meta.pyi

@@ -62,7 +62,6 @@ class Title(elements.Title):
         Returns:
             The component.
         """
-        ...
 
 class Meta(elements.Meta):
     @overload
@@ -318,7 +317,6 @@ class Meta(elements.Meta):
         Returns:
             The component.
         """
-        ...
 
 class Description(elements.Meta):
     @overload
@@ -572,7 +570,6 @@ class Description(elements.Meta):
         Returns:
             The component.
         """
-        ...
 
 class Image(elements.Meta):
     @overload
@@ -828,4 +825,3 @@ class Image(elements.Meta):
         Returns:
             The component.
         """
-        ...

reflex/components/base/script.py

@@ -68,7 +68,8 @@ class Script(Component):
             ValueError: when neither children nor `src` are specified.
         """
         if not children and not props.get("src"):
-            raise ValueError("Must provide inline script or `src` prop.")
+            msg = "Must provide inline script or `src` prop."
+            raise ValueError(msg)
         return super().create(*children, **props)
 
 

reflex/components/base/script.pyi

@@ -86,6 +86,5 @@ class Script(Component):
         Raises:
             ValueError: when neither children nor `src` are specified.
         """
-        ...
 
 script = Script.create

reflex/components/base/strict_mode.pyi

@@ -61,4 +61,3 @@ class StrictMode(Component):
         Returns:
             The component.
         """
-        ...

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.
@@ -2222,7 +2218,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 +2229,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 +2309,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 +2318,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 +2538,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]: ...