Fnm and node for POSIX

reflex/utils/exec.py

@@ -7,6 +7,8 @@
 import sys
 from pathlib import Path
 
+import typer
+
 from reflex import constants
 from reflex.config import get_config
 from reflex.utils import console, path_ops, prerequisites, processes
@@ -53,14 +55,31 @@ def run_frontend(
     Args:
         root: The root path of the project.
         port: The port to run the frontend on.
+
+    Raises:
+        Exit: If the package manager is invalid.
     """
     # Start watching asset folder.
     start_watching_assets_folder(root)
+    package_manager = prerequisites.get_package_manager()
+
+    if not package_manager:
+        console.error(
+            "Could not find NPM package manager. Make sure you have node installed."
+        )
+        raise typer.Exit(1)
+
+    if not prerequisites.check_node_version():
+        node_version = prerequisites.get_node_version()
+        console.error(
+            f"Reflex requires node version {constants.NODE_VERSION_MIN} or higher to run, but the detected version is {node_version}",
+        )
+        raise typer.Exit(1)
 
     # Run the frontend in development mode.
     console.rule("[bold green]App Running")
     os.environ["PORT"] = str(get_config().frontend_port if port is None else port)
-    run_process_and_launch_url([prerequisites.get_package_manager(), "run", "dev"])
+    run_process_and_launch_url([package_manager, "run", "dev"])
 
 
 def run_frontend_prod(
@@ -72,13 +91,28 @@ def run_frontend_prod(
     Args:
         root: The root path of the project (to keep same API as run_frontend).
         port: The port to run the frontend on.
+
+    Raises:
+        Exit: If the package manager is invalid.
     """
     # Set the port.
     os.environ["PORT"] = str(get_config().frontend_port if port is None else port)
+    package_manager = prerequisites.get_package_manager()
+    if not package_manager:
+        console.error(
+            "Could not find NPM package manager. Make sure you have node installed."
+        )
+        raise typer.Exit(1)
 
+    if not prerequisites.check_node_version():
+        node_version = prerequisites.get_node_version()
+        console.error(
+            f"Reflex requires node version {constants.NODE_VERSION_MIN} or higher to run, but the detected version is {node_version}",
+        )
+        raise typer.Exit(1)
     # Run the frontend in production mode.
     console.rule("[bold green]App Running")
-    run_process_and_launch_url([prerequisites.get_package_manager(), "run", "prod"])
+    run_process_and_launch_url([package_manager, "run", "prod"])
 
 
 def run_backend(
@@ -178,7 +212,7 @@ def output_system_info():
 
     dependencies = [
         f"[Reflex {constants.VERSION} with Python {platform.python_version()} (PATH: {sys.executable})]",
-        f"[Node {prerequisites.get_node_version()} (Expected: {constants.NODE_VERSION}) (PATH:{constants.NODE_PATH})]",
+        f"[Node {prerequisites.get_node_version()} (Expected: {constants.NODE_VERSION}) (PATH:{path_ops.get_node_path()})]",
     ]
 
     system = platform.system()
@@ -208,8 +242,8 @@ def output_system_info():
         console.debug(f"{dep}")
 
     console.debug(
-        f"Using package installer at: {prerequisites.get_install_package_manager()}"
+        f"Using package installer at: {prerequisites.get_install_package_manager()}"  # type: ignore
     )
-    console.debug(f"Using package executer at: {prerequisites.get_package_manager()}")
+    console.debug(f"Using package executer at: {prerequisites.get_package_manager()}")  # type: ignore
     if system != "Windows":
         console.debug(f"Unzip path: {path_ops.which('unzip')}")

reflex/utils/path_ops.py

@@ -4,8 +4,11 @@
 
 import os
 import shutil
+from pathlib import Path
 from typing import Optional
 
+from reflex import constants
+
 # Shorthand for join.
 join = os.linesep.join
 
@@ -107,3 +110,37 @@ def which(program: str) -> Optional[str]:
         The path to the executable.
     """
     return shutil.which(program)
+
+
+def get_node_bin_path() -> Optional[str]:
+    """Get the node binary dir path.
+
+    Returns:
+        The path to the node bin folder.
+    """
+    if not os.path.exists(constants.NODE_BIN_PATH):
+        str_path = which("node")
+        return str(Path(str_path).parent) if str_path else str_path
+    return constants.NODE_BIN_PATH
+
+
+def get_node_path():
+    """Get the node binary path.
+
+    Returns:
+        The path to the node binary file.
+    """
+    if not os.path.exists(constants.NODE_PATH):
+        return which("node")
+    return constants.NODE_PATH
+
+
+def get_npm_path():
+    """Get npm binary path.
+
+    Returns:
+        The path to the npm binary file.
+    """
+    if not os.path.exists(constants.NODE_PATH):
+        return which("npm")
+    return constants.NPM_PATH

reflex/utils/prerequisites.py

@@ -50,7 +50,7 @@ def get_node_version() -> Optional[version.Version]:
         The version of node.
     """
     try:
-        result = processes.new_process([constants.NODE_PATH, "-v"], run=True)
+        result = processes.new_process([path_ops.get_node_path(), "-v"], run=True)
         # The output will be in the form "vX.Y.Z", but version.parse() can handle it
         return version.parse(result.stdout)  # type: ignore
     except FileNotFoundError:
@@ -71,7 +71,7 @@ def get_bun_version() -> Optional[version.Version]:
         return None
 
 
-def get_install_package_manager() -> str:
+def get_install_package_manager() -> Optional[str]:
     """Get the package manager executable for installation.
       Currently on unix systems, bun is used for installation only.
 
@@ -80,20 +80,20 @@ def get_install_package_manager() -> str:
     """
     # On Windows, we use npm instead of bun.
     if constants.IS_WINDOWS:
-        return constants.NPM_PATH
+        return path_ops.get_npm_path()
 
     # On other platforms, we use bun.
     return get_config().bun_path
 
 
-def get_package_manager() -> str:
+def get_package_manager() -> Optional[str]:
     """Get the package manager executable for running app.
       Currently on unix systems, npm is used for running the app only.
 
     Returns:
         The path to the package manager.
     """
-    return constants.NPM_PATH
+    return path_ops.get_npm_path()
 
 
 def get_app() -> ModuleType:
@@ -275,7 +275,7 @@ def download_and_extract_fnm_zip():
     # Download the zip file
     url = constants.FNM_INSTALL_URL
     console.debug(f"Downloading {url}")
-    fnm_zip_file = os.sep.join([constants.FNM_DIR, f"{constants.FNM_FILENAME}.zip"])
+    fnm_zip_file = os.path.join(constants.FNM_DIR, f"{constants.FNM_FILENAME}.zip")
     # Function to download and extract the FNM zip release.
     try:
         # Download the FNM zip release.

reflex/utils/processes.py

@@ -13,8 +13,7 @@
 import psutil
 import typer
 
-from reflex import constants
-from reflex.utils import console, prerequisites
+from reflex.utils import console, path_ops, prerequisites
 
 
 def kill(pid):
@@ -126,10 +125,16 @@ def new_process(args, run: bool = False, show_logs: bool = False, **kwargs):
     Returns:
         Execute a child program in a new process.
     """
+    node_bin_path = path_ops.get_node_bin_path()
+    if not node_bin_path:
+        console.warn(
+            "The path to the Node binary could not be found. Please ensure that Node is properly "
+            "installed and added to your system's PATH environment variable."
+        )
     # Add the node bin path to the PATH environment variable.
     env = {
         **os.environ,
-        "PATH": os.pathsep.join([constants.NODE_BIN_PATH, os.environ["PATH"]]),
+        "PATH": os.pathsep.join([node_bin_path, os.environ["PATH"]]),  # type: ignore
         **kwargs.pop("env", {}),
     }
     kwargs = {