Merge branch 'main' into enable-wide-assortion-of-ruff-rules

Author: adhami3310

.github/actions/setup_build_env/action.yml

@@ -34,3 +34,7 @@ runs:
         prune-cache: false
         activate-environment: true
         cache-dependency-glob: "uv.lock"
+    - name: Install Dependencies
+      if: inputs.run-uv-sync == 'true'
+      run: uv sync
+      shell: bash

pyproject.toml

@@ -228,7 +228,7 @@ fail_fast = true
 
 [[tool.pre-commit.repos]]
 repo = "https://github.com/astral-sh/ruff-pre-commit"
-rev = "v0.11.11"
+rev = "v0.11.12"
 hooks = [
   { id = "ruff-format", args = [
     "reflex",

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
@@ -682,12 +683,13 @@ def into_component(component: Component | ComponentCallable) -> Component:
     """
     if (converted := _into_component_once(component)) is not None:
         return converted
+    if not callable(component):
+        raise TypeError(
+            f"Expected a Component or callable, got {component!r} of type {type(component)}"
+        )
+
     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
@@ -725,8 +727,12 @@ def into_component(component: Component | ComponentCallable) -> Component:
             ).with_traceback(e.__traceback__) from None
         raise
 
-    msg = f"Expected a Component, got {type(component)}"
-    raise TypeError(msg)
+    if (converted := _into_component_once(component_called)) is not None:
+        return converted
+
+    raise TypeError(
+        f"Expected a Component, got {component_called!r} of type {type(component_called)}"
+    )
 
 
 def compile_unevaluated_page(
@@ -747,52 +753,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:

reflex/components/radix/primitives/progress.py

@@ -15,7 +15,7 @@
 class ProgressComponent(RadixPrimitiveComponentWithClassName):
     """A Progress component."""
 
-    library = "@radix-ui/react-progress@1.1.6"
+    library = "@radix-ui/react-progress@1.1.7"
 
 
 class ProgressRoot(ProgressComponent):

reflex/components/radix/primitives/slider.py

@@ -17,7 +17,7 @@
 class SliderComponent(RadixPrimitiveComponentWithClassName):
     """Base class for all @radix-ui/react-slider components."""
 
-    library = "@radix-ui/react-slider@1.3.4"
+    library = "@radix-ui/react-slider@1.3.5"
 
 
 def on_value_event_spec(

reflex/constants/installer.py

@@ -14,7 +14,7 @@ class Bun(SimpleNamespace):
     """Bun constants."""
 
     # The Bun version.
-    VERSION = "1.2.14"
+    VERSION = "1.2.15"
 
     # Min Bun Version
     MIN_VERSION = "1.2.8"
@@ -137,7 +137,7 @@ def DEPENDENCIES(cls) -> dict[str, str]:
 
     DEV_DEPENDENCIES = {
         "autoprefixer": "10.4.21",
-        "postcss": "8.5.3",
+        "postcss": "8.5.4",
         "postcss-import": "16.1.0",
     }
     OVERRIDES = {

reflex/utils/pyi_generator.py

@@ -30,6 +30,8 @@
 
 PWD = Path.cwd()
 
+PYI_HASHES = "pyi_hashes.json"
+
 EXCLUDED_FILES = [
     "app.py",
     "component.py",
@@ -1264,45 +1266,48 @@ def scan_all(
                     file_parent = file_path.parent
                     while len(file_parent.parts) > len(top_dir.parts):
                         file_parent = file_parent.parent
+                    while len(top_dir.parts) > len(file_parent.parts):
+                        top_dir = top_dir.parent
                     while not file_parent.samefile(top_dir):
                         file_parent = file_parent.parent
                         top_dir = top_dir.parent
 
-                pyi_hashes_file = top_dir / "pyi_hashes.json"
-                if not pyi_hashes_file.exists():
-                    while top_dir.parent and not (top_dir / "pyi_hashes.json").exists():
-                        top_dir = top_dir.parent
-                    another_pyi_hashes_file = top_dir / "pyi_hashes.json"
-                    if another_pyi_hashes_file.exists():
-                        pyi_hashes_file = another_pyi_hashes_file
-
-                pyi_hashes_file.write_text(
-                    json.dumps(
-                        dict(
-                            zip(
-                                [
-                                    f.relative_to(pyi_hashes_file.parent).as_posix()
-                                    for f in file_paths
-                                ],
-                                hashes,
-                                strict=True,
-                            )
-                        ),
-                        indent=2,
-                        sort_keys=True,
+                while (
+                    not top_dir.samefile(top_dir.parent)
+                    and not (top_dir / PYI_HASHES).exists()
+                ):
+                    top_dir = top_dir.parent
+
+                pyi_hashes_file = top_dir / PYI_HASHES
+
+                if pyi_hashes_file.exists():
+                    pyi_hashes_file.write_text(
+                        json.dumps(
+                            dict(
+                                zip(
+                                    [
+                                        f.relative_to(pyi_hashes_file.parent).as_posix()
+                                        for f in file_paths
+                                    ],
+                                    hashes,
+                                    strict=True,
+                                )
+                            ),
+                            indent=2,
+                            sort_keys=True,
+                        )
+                        + "\n",
                     )
-                    + "\n",
-                )
             elif file_paths:
                 file_paths = list(map(Path, file_paths))
                 pyi_hashes_parent = file_paths[0].parent
                 while (
-                    pyi_hashes_parent.parent
-                    and not (pyi_hashes_parent / "pyi_hashes.json").exists()
+                    not pyi_hashes_parent.samefile(pyi_hashes_parent.parent)
+                    and not (pyi_hashes_parent / PYI_HASHES).exists()
                 ):
                     pyi_hashes_parent = pyi_hashes_parent.parent
 
-                pyi_hashes_file = pyi_hashes_parent / "pyi_hashes.json"
+                pyi_hashes_file = pyi_hashes_parent / PYI_HASHES
                 if pyi_hashes_file.exists():
                     pyi_hashes = json.loads(pyi_hashes_file.read_text())
                     for file_path, hashed_content in zip(