reflex 0.7.13a1__py3-none-any.whl → 0.7.14__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
@@ -18,8 +19,9 @@ from reflex.components.component import (
     CustomComponent,
     StatefulComponent,
 )
-from reflex.config import environment, get_config
+from reflex.config import get_config
 from reflex.constants.compiler import PageNames
+from reflex.environment import environment
 from reflex.state import BaseState
 from reflex.style import SYSTEM_COLOR_MODE
 from reflex.utils import console, path_ops
@@ -200,15 +202,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 +219,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 +257,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 +684,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 +713,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 +727,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 +752,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 +886,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

@@ -8,7 +8,7 @@ from typing import Any
 from reflex.components.component import BaseComponent, Component, ComponentStyle
 from reflex.components.tags import Tag
 from reflex.components.tags.tagless import Tagless
-from reflex.config import PerformanceMode, environment
+from reflex.environment import PerformanceMode, environment
 from reflex.utils import console
 from reflex.utils.decorator import once
 from reflex.utils.imports import ParsedImportDict
@@ -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)
 
@@ -143,17 +141,29 @@ class Bare(Component):
                 custom_code |= component._get_all_custom_code()
         return custom_code
 
-    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 components that should be wrapped in the app.
 
+        Args:
+            ignore_ids: The ids to ignore when collecting components.
+
         Returns:
             The components that should be wrapped in the app.
         """
-        app_wrap_components = super()._get_all_app_wrap_components()
+        ignore_ids = ignore_ids or set()
+        app_wrap_components = super()._get_all_app_wrap_components(
+            ignore_ids=ignore_ids
+        )
         if isinstance(self.contents, Var):
             for component in _components_from_var(self.contents):
-                if isinstance(component, Component):
-                    app_wrap_components |= component._get_all_app_wrap_components()
+                component_id = id(component)
+                if isinstance(component, Component) and component_id not in ignore_ids:
+                    ignore_ids.add(component_id)
+                    app_wrap_components |= component._get_all_app_wrap_components(
+                        ignore_ids=ignore_ids
+                    )
         return app_wrap_components
 
     def _get_all_refs(self) -> set[str]:

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.
         """
-        ...