Running ReactPy in a backhaul thread (#163)

- The entirety of the ReactPy rendering stack is encapsulated within a single backhaul thread.
- Backhaul thread can be disabled via settings.py:REACTPY_BACKHAUL_THREAD
- Implementation is "thread-safe" since all ReactPy code runs on the exact same thread.

Author: Archmonger

.github/workflows/test-docs.yml

@@ -33,5 +33,4 @@ jobs:
                   pip install -r requirements/check-style.txt
                   mypy --show-error-codes docs/python/
                   black docs/python/ --check
-                  isort docs/python/ --check-only
-                  flake8 docs/python/
+                  ruff check docs/python/

.github/workflows/test-src.yml

@@ -1,35 +1,35 @@
 name: Test
 
 on:
-  push:
-    branches:
-      - main
-  pull_request:
-    branches:
-      - main
-  schedule:
-    - cron: "0 0 * * *"
+    push:
+        branches:
+            - main
+    pull_request:
+        branches:
+            - main
+    schedule:
+        - cron: "0 0 * * *"
 
 jobs:
-  source:
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        python-version: ["3.8", "3.9", "3.10"]
-    steps:
-      - uses: actions/checkout@v3
-      - uses: nanasess/setup-chromedriver@master
-      - uses: actions/setup-node@v3
-        with:
-          node-version: "14"
-      - name: Use Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v4
-        with:
-          python-version: ${{ matrix.python-version }}
-      - name: Install Python Dependencies
-        run: pip install -r requirements/test-run.txt
-      - name: Run Tests
-        run: |
-          npm install -g npm@latest
-          npm --version
-          nox -s test
+    source:
+        runs-on: ubuntu-latest
+        strategy:
+            matrix:
+                python-version: ["3.9", "3.10", "3.11"]
+        steps:
+            - uses: actions/checkout@v3
+            - uses: nanasess/setup-chromedriver@master
+            - uses: actions/setup-node@v3
+              with:
+                  node-version: "14"
+            - name: Use Python ${{ matrix.python-version }}
+              uses: actions/setup-python@v4
+              with:
+                  python-version: ${{ matrix.python-version }}
+            - name: Install Python Dependencies
+              run: pip install -r requirements/test-run.txt
+            - name: Run Tests
+              run: |
+                  npm install -g npm@latest
+                  npm --version
+                  nox -s test

CHANGELOG.md

@@ -34,7 +34,14 @@ Using the following categories, list your changes in this order:
 
 ## [Unreleased]
 
--   Nothing yet!
+### Added
+
+-   `REACTPY_BACKHAUL_THREAD` setting to enable/disable threading behavior.
+
+### Changed
+
+-   By default, ReactPy will now use a backhaul thread to increase performance.
+-   Minimum Python version required is now `3.9`
 
 ## [3.2.1] - 2023-06-29
 

docs/python/settings.py

@@ -20,3 +20,7 @@
 #   2. You are using Django's `AUTHENTICATION_BACKENDS` setting and...
 #   3. Your Django user model does not define a `backend` attribute
 REACTPY_AUTH_BACKEND = None
+
+# Whether to enable rendering ReactPy via a dedicated backhaul thread
+# This allows the webserver to process traffic while during ReactPy rendering
+REACTPY_BACKHAUL_THREAD = True

docs/src/contribute/code.md

@@ -12,7 +12,7 @@
 
 If you plan to make code changes to this repository, you will need to install the following dependencies first:
 
--   [Python 3.8+](https://www.python.org/downloads/)
+-   [Python 3.9+](https://www.python.org/downloads/)
 -   [Git](https://git-scm.com/downloads)
 -   [NPM](https://docs.npmjs.com/try-the-latest-stable-version-of-npm) for installing and managing Javascript
 

docs/src/contribute/docs.md

@@ -8,7 +8,7 @@
 
 If you plan to make changes to this documentation, you will need to install the following dependencies first:
 
--   [Python 3.8+](https://www.python.org/downloads/)
+-   [Python 3.9+](https://www.python.org/downloads/)
 -   [Git](https://git-scm.com/downloads)
 
 Once done, you should clone this repository:

docs/src/contribute/running-tests.md

@@ -10,7 +10,7 @@ This repository uses [Nox](https://nox.thea.codes/en/stable/) to run tests. For
 
 If you plan to run tests, you will need to install the following dependencies first:
 
--   [Python 3.8+](https://www.python.org/downloads/)
+-   [Python 3.9+](https://www.python.org/downloads/)
 -   [Git](https://git-scm.com/downloads)
 
 Once done, you should clone this repository:

docs/src/dictionary.txt

@@ -35,3 +35,4 @@ backends
 backend
 frontend
 frontends
+backhaul

noxfile.py

@@ -6,7 +6,6 @@
 import nox
 from nox.sessions import Session
 
-
 HERE = Path(__file__).parent
 POSARGS_PATTERN = re.compile(r"^(\w+)\[(.+)\]$")
 
@@ -21,14 +20,6 @@ def manage(session: Session) -> None:
     session.run("python", "manage.py", *session.posargs)
 
 
-@nox.session(reuse_venv=True)
-def format(session: Session) -> None:
-    """Run automatic code formatters"""
-    install_requirements_file(session, "check-style")
-    session.run("black", ".")
-    session.run("isort", ".")
-
-
 @nox.session
 def test(session: Session) -> None:
     """Run the complete test suite"""
@@ -70,15 +61,14 @@ def test_types(session: Session) -> None:
 def test_style(session: Session) -> None:
     """Check that style guidelines are being followed"""
     install_requirements_file(session, "check-style")
-    session.run("flake8", "src/reactpy_django", "tests")
     session.run(
         "black",
         ".",
         "--check",
         "--extend-exclude",
         "/migrations/",
     )
-    session.run("isort", ".", "--check-only")
+    session.run("ruff", "check", ".")
 
 
 def install_requirements_file(session: Session, name: str) -> None:

pyproject.toml

@@ -2,16 +2,6 @@
 requires = ["setuptools>=42", "wheel"]
 build-backend = "setuptools.build_meta"
 
-[tool.isort]
-multi_line_output = 3
-force_grid_wrap = 0
-use_parentheses = "True"
-ensure_newline_before_comments = "True"
-include_trailing_comma = "True"
-line_length = 88
-lines_after_imports = 2
-extend_skip_glob = ["*/migrations/*", '.nox/*', '.venv/*', 'build/*']
-
 [tool.mypy]
 exclude = [
     'migrations/.*',
@@ -22,3 +12,11 @@ warn_redundant_casts = true
 warn_unused_ignores = true
 check_untyped_defs = true
 incremental = false
+
+[tool.ruff.isort]
+known-first-party = ["src", "tests"]
+
+[tool.ruff]
+ignore = ["E501"]
+extend-exclude = ["*/migrations/*", ".venv/*", ".eggs/*", ".nox/*", "build/*"]
+line-length = 120