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

reflex/utils/prerequisites.py

@@ -129,7 +129,6 @@ def check_latest_package_version(package_name: str):
             )
     except Exception:
         console.debug(f"Failed to check for the latest version of {package_name}.")
-        pass
 
 
 def get_or_set_last_reflex_version_check_datetime():
@@ -255,9 +254,8 @@ def get_nodejs_compatible_package_managers(
     package_managers = list(filter(None, package_managers))
 
     if not package_managers and raise_on_none:
-        raise FileNotFoundError(
-            "Bun or npm not found. You might need to rerun `reflex init` or install either."
-        )
+        msg = "Bun or npm not found. You might need to rerun `reflex init` or install either."
+        raise FileNotFoundError(msg)
 
     return package_managers
 
@@ -310,9 +308,8 @@ def get_js_package_executor(raise_on_none: bool = False) -> Sequence[Sequence[st
     package_managers = list(filter(None, package_managers))
 
     if not package_managers and raise_on_none:
-        raise FileNotFoundError(
-            "Bun or npm not found. You might need to rerun `reflex init` or install either."
-        )
+        msg = "Bun or npm not found. You might need to rerun `reflex init` or install either."
+        raise FileNotFoundError(msg)
 
     return package_managers
 
@@ -345,10 +342,11 @@ def _check_app_name(config: Config):
         RuntimeError: If the app name is not set in the config.
     """
     if not config.app_name:
-        raise RuntimeError(
+        msg = (
             "Cannot get the app module because `app_name` is not set in rxconfig! "
             "If this error occurs in a reflex test case, ensure that `get_app` is mocked."
         )
+        raise RuntimeError(msg)
 
 
 def get_app(reload: bool = False) -> ModuleType:
@@ -395,11 +393,14 @@ def get_app(reload: bool = False) -> ModuleType:
         return app
 
 
-def get_and_validate_app(reload: bool = False) -> AppInfo:
+def get_and_validate_app(
+    reload: bool = False, check_if_schema_up_to_date: bool = False
+) -> AppInfo:
     """Get the app instance based on the default config and validate it.
 
     Args:
         reload: Re-import the app module from disk
+        check_if_schema_up_to_date: If True, check if the schema is up to date.
 
     Returns:
         The app instance and the app module.
@@ -412,23 +413,34 @@ def get_and_validate_app(reload: bool = False) -> AppInfo:
     app_module = get_app(reload=reload)
     app = getattr(app_module, constants.CompileVars.APP)
     if not isinstance(app, App):
-        raise RuntimeError(
-            "The app instance in the specified app_module_import in rxconfig must be an instance of rx.App."
-        )
+        msg = "The app instance in the specified app_module_import in rxconfig must be an instance of rx.App."
+        raise RuntimeError(msg)
+
+    if check_if_schema_up_to_date:
+        check_schema_up_to_date()
+
     return AppInfo(app=app, module=app_module)
 
 
-def validate_app(reload: bool = False) -> None:
+def validate_app(
+    reload: bool = False, check_if_schema_up_to_date: bool = False
+) -> None:
     """Validate the app instance based on the default config.
 
     Args:
         reload: Re-import the app module from disk
+        check_if_schema_up_to_date: If True, check if the schema is up to date.
     """
-    get_and_validate_app(reload=reload)
+    get_and_validate_app(
+        reload=reload, check_if_schema_up_to_date=check_if_schema_up_to_date
+    )
 
 
 def get_compiled_app(
-    reload: bool = False, export: bool = False, dry_run: bool = False
+    reload: bool = False,
+    export: bool = False,
+    dry_run: bool = False,
+    check_if_schema_up_to_date: bool = False,
 ) -> ModuleType:
     """Get the app module based on the default config after first compiling it.
 
@@ -436,11 +448,14 @@ def get_compiled_app(
         reload: Re-import the app module from disk
         export: Compile the app for export
         dry_run: If True, do not write the compiled app to disk.
+        check_if_schema_up_to_date: If True, check if the schema is up to date.
 
     Returns:
         The compiled app based on the default config.
     """
-    app, app_module = get_and_validate_app(reload=reload)
+    app, app_module = get_and_validate_app(
+        reload=reload, check_if_schema_up_to_date=check_if_schema_up_to_date
+    )
     # For py3.9 compatibility when redis is used, we MUST add any decorator pages
     # before compiling the app in a thread to avoid event loop error (REF-2172).
     app._apply_decorated_pages()
@@ -449,7 +464,10 @@ def get_compiled_app(
 
 
 def compile_app(
-    reload: bool = False, export: bool = False, dry_run: bool = False
+    reload: bool = False,
+    export: bool = False,
+    dry_run: bool = False,
+    check_if_schema_up_to_date: bool = False,
 ) -> None:
     """Compile the app module based on the default config.
 
@@ -457,8 +475,14 @@ def compile_app(
         reload: Re-import the app module from disk
         export: Compile the app for export
         dry_run: If True, do not write the compiled app to disk.
+        check_if_schema_up_to_date: If True, check if the schema is up to date.
     """
-    get_compiled_app(reload=reload, export=export, dry_run=dry_run)
+    get_compiled_app(
+        reload=reload,
+        export=export,
+        dry_run=dry_run,
+        check_if_schema_up_to_date=check_if_schema_up_to_date,
+    )
 
 
 def _can_colorize() -> bool:
@@ -503,20 +527,23 @@ def _can_colorize() -> bool:
         return file.isatty()
 
 
-def compile_or_validate_app(compile: bool = False) -> bool:
+def compile_or_validate_app(
+    compile: bool = False, check_if_schema_up_to_date: bool = False
+) -> bool:
     """Compile or validate the app module based on the default config.
 
     Args:
         compile: Whether to compile the app.
+        check_if_schema_up_to_date: If True, check if the schema is up to date.
 
     Returns:
         If the app is compiled successfully.
     """
     try:
         if compile:
-            compile_app()
+            compile_app(check_if_schema_up_to_date=check_if_schema_up_to_date)
         else:
-            validate_app()
+            validate_app(check_if_schema_up_to_date=check_if_schema_up_to_date)
     except Exception as e:
         if isinstance(e, click.exceptions.Exit):
             return False
@@ -575,9 +602,8 @@ def parse_redis_url() -> str | None:
     if not config.redis_url:
         return None
     if not config.redis_url.startswith(("redis://", "rediss://", "unix://")):
-        raise ValueError(
-            "REDIS_URL must start with 'redis://', 'rediss://', or 'unix://'."
-        )
+        msg = "REDIS_URL must start with 'redis://', 'rediss://', or 'unix://'."
+        raise ValueError(msg)
     return config.redis_url
 
 
@@ -1157,15 +1183,16 @@ def download_and_run(url: str, *args, show_status: bool = False, **env):
         raise click.exceptions.Exit(1) from None
 
     # Save the script to a temporary file.
-    script = Path(tempfile.NamedTemporaryFile().name)
+    with tempfile.NamedTemporaryFile() as tempfile_file:
+        script = Path(tempfile_file.name)
 
-    script.write_text(response.text)
+        script.write_text(response.text)
 
-    # Run the script.
-    env = {**os.environ, **env}
-    process = processes.new_process(["bash", str(script), *args], env=env)
-    show = processes.show_status if show_status else processes.show_logs
-    show(f"Installing {url}", process)
+        # Run the script.
+        env = {**os.environ, **env}
+        process = processes.new_process(["bash", str(script), *args], env=env)
+        show = processes.show_status if show_status else processes.show_logs
+        show(f"Installing {url}", process)
 
 
 def install_bun():
@@ -1213,7 +1240,8 @@ def install_bun():
         )
     else:
         if path_ops.which("unzip") is None:
-            raise SystemPackageMissingError("unzip")
+            msg = "unzip"
+            raise SystemPackageMissingError(msg)
 
         # Run the bun install script.
         download_and_run(
@@ -1262,13 +1290,15 @@ def cached_procedure(
         ValueError: If both cache_file and cache_file_fn are provided.
     """
     if cache_file and cache_file_fn is not None:
-        raise ValueError("cache_file and cache_file_fn cannot both be provided.")
+        msg = "cache_file and cache_file_fn cannot both be provided."
+        raise ValueError(msg)
 
     def _inner_decorator(func: Callable):
         def _inner(*args, **kwargs):
             _cache_file = cache_file_fn() if cache_file_fn is not None else cache_file
             if not _cache_file:
-                raise ValueError("Unknown cache file, cannot cache result.")
+                msg = "Unknown cache file, cannot cache result."
+                raise ValueError(msg)
             payload = _read_cached_procedure_file(_cache_file)
             new_payload = payload_fn(*args, **kwargs)
             if payload != new_payload:
@@ -1446,7 +1476,7 @@ def validate_bun(bun_path: Path | None = None):
                 "Failed to obtain bun version. Make sure the specified bun path in your config is correct."
             )
             raise click.exceptions.Exit(1)
-        elif bun_version < version.parse(constants.Bun.MIN_VERSION):
+        if bun_version < version.parse(constants.Bun.MIN_VERSION):
             console.warn(
                 f"Reflex requires bun version {constants.Bun.MIN_VERSION} or higher to run, but the detected version is "
                 f"{bun_version}. If you have specified a custom bun path in your config, make sure to provide one "
@@ -1657,8 +1687,7 @@ def fetch_app_templates(version: str) -> dict[str, Template]:
     if asset is None:
         console.warn(f"Templates metadata not found for version {version}")
         return {}
-    else:
-        templates_url = asset["browser_download_url"]
+    templates_url = asset["browser_download_url"]
 
     templates_data = net.get(templates_url, follow_redirects=True).json()["templates"]
 
@@ -1864,7 +1893,7 @@ def initialize_app(app_name: str, template: str | None = None) -> str | None:
     # Check if the app is already initialized.
     if constants.Config.FILE.exists():
         telemetry.send("reinit")
-        return
+        return None
 
     templates: dict[str, Template] = {}
 

reflex/utils/processes.py

@@ -137,11 +137,10 @@ def handle_port(service_name: str, port: int, auto_increment: bool) -> int:
         return port
     if auto_increment:
         return change_port(port, service_name)
-    else:
-        console.error(
-            f"{service_name.capitalize()} port: {port} is already in use by PID: {process.pid}."
-        )
-        raise click.exceptions.Exit()
+    console.error(
+        f"{service_name.capitalize()} port: {port} is already in use by PID: {process.pid}."
+    )
+    raise click.exceptions.Exit
 
 
 @overload
@@ -316,7 +315,6 @@ def stream_logs(
                 # But if the process is still running that is weird.
                 raise
             # If the process exited, break out of the loop for post processing.
-            pass
 
     # Check if the process failed (not printing the logs for SIGINT).
 

reflex/utils/pyi_generator.py

@@ -30,6 +30,8 @@ logger = logging.getLogger("pyi_generator")
 
 PWD = Path.cwd()
 
+PYI_HASHES = "pyi_hashes.json"
+
 EXCLUDED_FILES = [
     "app.py",
     "component.py",
@@ -157,9 +159,8 @@ def _get_type_hint(
             res_args.sort()
             if len(res_args) == 1:
                 return f"{res_args[0]} | None"
-            else:
-                res = f"{' | '.join(res_args)}"
-                return f"{res} | None"
+            res = f"{' | '.join(res_args)}"
+            return f"{res} | None"
 
         res_args = [
             _get_type_hint(arg, type_hint_globals, rx_types.is_optional(arg))
@@ -183,10 +184,11 @@ def _get_type_hint(
             value.__module__ not in ["builtins", "__builtins__"]
             and value.__name__ not in type_hint_globals
         ):
-            raise TypeError(
+            msg = (
                 f"{value.__module__ + '.' + value.__name__} is not a default import, "
                 "add it to DEFAULT_IMPORTS in pyi_generator.py"
             )
+            raise TypeError(msg)
 
         res = f"{value.__name__}[{', '.join(inner_container_type_args)}]"
 
@@ -445,7 +447,7 @@ def type_to_ast(typ: Any, cls: type) -> ast.expr:
 
                 return ast.Name(id=typ.__module__ + "." + typ.__name__)
             return ast.Name(id=typ.__name__)
-        elif hasattr(typ, "_name"):
+        if hasattr(typ, "_name"):
             return ast.Name(id=typ._name)
         return ast.Name(id=str(typ))
 
@@ -510,7 +512,8 @@ def _generate_component_create_functiondef(
         TypeError: If clz is not a subclass of Component.
     """
     if not issubclass(clz, Component):
-        raise TypeError(f"clz must be a subclass of Component, not {clz!r}")
+        msg = f"clz must be a subclass of Component, not {clz!r}"
+        raise TypeError(msg)
 
     # add the imports needed by get_type_hint later
     type_hint_globals.update(
@@ -654,7 +657,7 @@ def _generate_component_create_functiondef(
         defaults=[],
     )
 
-    definition = ast.FunctionDef(  # pyright: ignore [reportCallIssue]
+    return ast.FunctionDef(  # pyright: ignore [reportCallIssue]
         name="create",
         args=create_args,
         body=[
@@ -676,7 +679,6 @@ def _generate_component_create_functiondef(
         lineno=lineno,
         returns=ast.Constant(value=clz.__name__),
     )
-    return definition
 
 
 def _generate_staticmethod_call_functiondef(
@@ -710,7 +712,7 @@ def _generate_staticmethod_call_functiondef(
             else []
         ),
     )
-    definition = ast.FunctionDef(  # pyright: ignore [reportCallIssue]
+    return ast.FunctionDef(  # pyright: ignore [reportCallIssue]
         name="__call__",
         args=call_args,
         body=[
@@ -729,7 +731,6 @@ def _generate_staticmethod_call_functiondef(
             )
         ),
     )
-    return definition
 
 
 def _generate_namespace_call_functiondef(
@@ -841,6 +842,7 @@ class StubGenerator(ast.NodeTransformer):
             and issubclass((clz := self.classes[self.current_class]), Component)
         ):
             return clz
+        return None
 
     def visit_Module(self, node: ast.Module) -> ast.Module:
         """Visit a Module node and remove docstring from body.
@@ -1021,7 +1023,7 @@ class StubGenerator(ast.NodeTransformer):
             if isinstance(target, ast.Tuple):
                 for name in target.elts:
                     if isinstance(name, ast.Name) and name.id.startswith("_"):
-                        return
+                        return None
 
         return node
 
@@ -1107,7 +1109,7 @@ class PyiGenerator:
         pyright_ignore_imports = getattr(mod, "_PYRIGHT_IGNORE_IMPORTS", [])
 
         if not sub_mods and not sub_mod_attrs:
-            return
+            return None
         sub_mods_imports = []
         sub_mod_attrs_imports = []
 
@@ -1162,7 +1164,7 @@ class PyiGenerator:
         }
         is_init_file = _relative_to_pwd(module_path).name == "__init__.py"
         if not class_names and not is_init_file:
-            return
+            return None
 
         if is_init_file:
             new_tree = InitStubGenerator(module, class_names).visit(
@@ -1170,7 +1172,7 @@ class PyiGenerator:
             )
             init_imports = self._get_init_lazy_imports(module, new_tree)
             if not init_imports:
-                return
+                return None
             content_hash = self._write_pyi_file(module_path, init_imports)
         else:
             new_tree = StubGenerator(module, class_names).visit(
@@ -1264,45 +1266,48 @@ class PyiGenerator:
                     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(

reflex/utils/redir.py

@@ -5,9 +5,9 @@ import webbrowser
 
 import httpx
 
+from reflex import constants
 from reflex.utils import net
 
-from .. import constants
 from . import console
 
 

reflex/utils/serializers.py

@@ -76,7 +76,8 @@ def serializer(
 
         # Make sure the function takes a single argument.
         if len(args) != 1:
-            raise ValueError("Serializer must take a single argument.")
+            msg = "Serializer must take a single argument."
+            raise ValueError(msg)
 
         # Get the type of the argument.
         type_ = type_hints[args[0]]
@@ -166,8 +167,7 @@ def serialize(
     # Return the serialized value and the type.
     if get_type:
         return serialized, get_serializer_type(type(value))
-    else:
-        return serialized
+    return serialized
 
 
 @functools.lru_cache
@@ -427,7 +427,7 @@ with contextlib.suppress(ImportError):
         """
         return [
             [str(d) if isinstance(d, (list, tuple)) else d for d in data]
-            for data in list(df.values.tolist())
+            for data in list(df.to_numpy().tolist())
         ]
 
     @serializer

reflex/utils/telemetry.py

@@ -19,7 +19,12 @@ from reflex.config import environment
 from reflex.utils import console
 from reflex.utils.decorator import once_unless_none
 from reflex.utils.exceptions import ReflexError
-from reflex.utils.prerequisites import ensure_reflex_installation_id, get_project_hash
+from reflex.utils.prerequisites import (
+    ensure_reflex_installation_id,
+    get_bun_version,
+    get_node_version,
+    get_project_hash,
+)
 
 UTC = timezone.utc
 POSTHOG_API_URL: str = "https://app.posthog.com/capture/"
@@ -62,6 +67,22 @@ def get_reflex_version() -> str:
     return constants.Reflex.VERSION
 
 
+def _get_app_class_name() -> str | None:
+    """Get the app class name if available.
+
+    Returns:
+        The app class name (e.g. "App", "AppEnterprise") or None if not available.
+    """
+    try:
+        from reflex.utils.prerequisites import get_and_validate_app
+
+        app_info = get_and_validate_app()
+    except Exception:
+        return None
+    else:
+        return app_info.app.__class__.__name__
+
+
 def get_cpu_count() -> int:
     """Get the number of CPUs.
 
@@ -106,6 +127,8 @@ class _Properties(TypedDict):
     user_os_detail: str
     reflex_version: str
     python_version: str
+    node_version: str | None
+    bun_version: str | None
     cpu_count: int
     memory: int
     cpu_info: dict
@@ -153,6 +176,12 @@ def _get_event_defaults() -> _DefaultEvent | None:
             "user_os_detail": get_detailed_platform_str(),
             "reflex_version": get_reflex_version(),
             "python_version": get_python_version(),
+            "node_version": (
+                str(node_version) if (node_version := get_node_version()) else None
+            ),
+            "bun_version": (
+                str(bun_version) if (bun_version := get_bun_version()) else None
+            ),
             "cpu_count": get_cpu_count(),
             "memory": get_memory(),
             "cpu_info": dataclasses.asdict(cpuinfo) if cpuinfo else {},
@@ -184,12 +213,16 @@ def _prepare_event(event: str, **kwargs) -> _Event | None:
     if not event_data:
         return None
 
-    additional_keys = ["template", "context", "detail", "user_uuid"]
+    additional_keys = ["template", "context", "detail", "user_uuid", "app_class"]
 
     properties = event_data["properties"]
 
+    # Auto-detect app class if not provided
+    if "app_class" not in kwargs:
+        kwargs["app_class"] = _get_app_class_name()
+
     for key in additional_keys:
-        if key in properties or key not in kwargs:
+        if key in properties or key not in kwargs or kwargs[key] is None:
             continue
 
         properties[key] = kwargs[key]
@@ -232,6 +265,9 @@ def _send(event: str, telemetry_enabled: bool | None, **kwargs) -> bool:
     return False
 
 
+background_tasks = set()
+
+
 def send(event: str, telemetry_enabled: bool | None = None, **kwargs):
     """Send anonymous telemetry for Reflex.
 
@@ -246,7 +282,9 @@ def send(event: str, telemetry_enabled: bool | None = None, **kwargs):
 
     try:
         # Within an event loop context, send the event asynchronously.
-        asyncio.create_task(async_send(event, telemetry_enabled, **kwargs))
+        task = asyncio.create_task(async_send(event, telemetry_enabled, **kwargs))
+        background_tasks.add(task)
+        task.add_done_callback(background_tasks.discard)
     except RuntimeError:
         # If there is no event loop, send the event synchronously.
         warnings.filterwarnings("ignore", category=RuntimeWarning)