be less whiny about bun version and suggest alternatives (#5106)

* be less whiny about bun version and suggest alternatives

* fix tests

* only validate on version mismatch

* dang it darglint

Author: adhami3310

reflex/utils/prerequisites.py

@@ -192,13 +192,16 @@ def get_node_version() -> version.Version | None:
         return None
 
 
-def get_bun_version() -> version.Version | None:
+def get_bun_version(bun_path: Path | None = None) -> version.Version | None:
     """Get the version of bun.
 
+    Args:
+        bun_path: The path to the bun executable.
+
     Returns:
         The version of bun.
     """
-    bun_path = path_ops.get_bun_path()
+    bun_path = bun_path or path_ops.get_bun_path()
     if bun_path is None:
         return None
     try:
@@ -1153,13 +1156,21 @@ def install_bun():
             "Creating project directories in OneDrive is not recommended for bun usage on windows. This will fallback to npm."
         )
 
+    bun_path = path_ops.get_bun_path()
+
     # Skip if bun is already installed.
-    if (current_version := get_bun_version()) and current_version >= version.parse(
-        constants.Bun.MIN_VERSION
+    if (
+        bun_path
+        and (current_version := get_bun_version(bun_path=bun_path))
+        and current_version >= version.parse(constants.Bun.MIN_VERSION)
     ):
         console.debug("Skipping bun installation as it is already installed.")
         return
 
+    if bun_path and path_ops.use_system_bun():
+        validate_bun(bun_path=bun_path)
+        return
+
     #  if unzip is installed
     if constants.IS_WINDOWS:
         processes.new_process(
@@ -1394,13 +1405,16 @@ def is_latest_template() -> bool:
     return app_version == constants.Reflex.VERSION
 
 
-def validate_bun():
+def validate_bun(bun_path: Path | None = None):
     """Validate bun if a custom bun path is specified to ensure the bun version meets requirements.
 
+    Args:
+        bun_path: The path to the bun executable. If None, the default bun path is used.
+
     Raises:
         Exit: If custom specified bun does not exist or does not meet requirements.
     """
-    bun_path = path_ops.get_bun_path()
+    bun_path = bun_path or path_ops.get_bun_path()
 
     if bun_path is None:
         return
@@ -1414,14 +1428,12 @@ def validate_bun():
             )
             raise typer.Exit(1)
         elif bun_version < version.parse(constants.Bun.MIN_VERSION):
-            console.error(
+            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 "
-                f"that satisfies the minimum version requirement."
+                f"that satisfies the minimum version requirement. You can upgrade bun by running [bold]bun upgrade[/bold]."
             )
 
-            raise typer.Exit(1)
-
 
 def validate_frontend_dependencies(init: bool = True):
     """Validate frontend dependencies to ensure they meet requirements.
@@ -1438,15 +1450,12 @@ def validate_frontend_dependencies(init: bool = True):
         except FileNotFoundError as e:
             raise typer.Exit(1) from e
 
-    if prefer_npm_over_bun():
-        if not check_node_version():
-            node_version = get_node_version()
-            console.error(
-                f"Reflex requires node version {constants.Node.MIN_VERSION} or higher to run, but the detected version is {node_version}",
-            )
-            raise typer.Exit(1)
-    else:
-        validate_bun()
+    if prefer_npm_over_bun() and not check_node_version():
+        node_version = get_node_version()
+        console.error(
+            f"Reflex requires node version {constants.Node.MIN_VERSION} or higher to run, but the detected version is {node_version}",
+        )
+        raise typer.Exit(1)
 
 
 def ensure_reflex_installation_id() -> int | None:

tests/units/utils/test_utils.py

@@ -226,8 +226,8 @@ def test_validate_bun_path_incompatible_version(mocker):
         return_value=version.parse("0.6.5"),
     )
 
-    with pytest.raises(typer.Exit):
-        prerequisites.validate_bun()
+    # This will just warn the user, not raise an error
+    prerequisites.validate_bun()
 
 
 def test_remove_existing_bun_installation(mocker):