Merge branch 'main' into local-storage

Author: williamneto

.gitignore

@@ -11,6 +11,7 @@
 .jupyter
 
 # --- Python ---
+.hatch
 .venv
 venv
 MANIFEST

.pre-commit-config.yaml

@@ -3,15 +3,9 @@ repos:
     hooks:
       - id: lint-py-fix
         name: Fix Python Lint
-        entry: hatch run lint-py --fix
-        language: system
-        pass_filenames: false
-  - repo: local
-    hooks:
-      - id: lint-py-check
-        name: Check Python Lint
         entry: hatch run lint-py
         language: system
+        args: [--fix]
         pass_filenames: false
   - repo: local
     hooks:
@@ -20,6 +14,13 @@ repos:
         entry: hatch run lint-js --fix
         language: system
         pass_filenames: false
+  - repo: local
+    hooks:
+      - id: lint-py-check
+        name: Check Python Lint
+        entry: hatch run lint-py
+        language: system
+        pass_filenames: false
   - repo: local
     hooks:
       - id: lint-js-check

docs/pyproject.toml

@@ -1,5 +1,5 @@
 [tool.poetry]
-name = "docs"
+name = "docs_app"
 version = "0.0.0"
 description = "docs"
 authors = ["rmorshea <[email protected]>"]

docs/source/_exts/reactpy_example.py

@@ -2,7 +2,7 @@
 
 import re
 from pathlib import Path
-from typing import Any
+from typing import Any, ClassVar
 
 from docs_app.examples import (
     SOURCE_DIR,
@@ -21,7 +21,7 @@ class WidgetExample(SphinxDirective):
     required_arguments = 1
     _next_id = 0
 
-    option_spec = {
+    option_spec: ClassVar[dict[str, Any]] = {
         "result-is-default-tab": directives.flag,
         "activate-button": directives.flag,
     }

docs/source/_exts/reactpy_view.py

@@ -1,7 +1,5 @@
 import os
-import sys
-
-print(sys.path)
+from typing import Any, ClassVar
 
 from docs_app.examples import get_normalized_example_name
 from docutils.nodes import raw
@@ -20,7 +18,7 @@ class IteractiveWidget(SphinxDirective):
     required_arguments = 1
     _next_id = 0
 
-    option_spec = {
+    option_spec: ClassVar[dict[str, Any]] = {
         "activate-button": directives.flag,
         "margin": float,
     }

docs/source/about/contributor-guide.rst

@@ -118,6 +118,26 @@ Then, you should be able to activate your development environment with:
 
     hatch shell
 
+From within the shell, to install the projects in this repository, you should then run:
+
+.. code-block:: bash
+
+    invoke env
+
+Project Structure
+-----------------
+
+This repository is set up to be able to manage many applications and libraries written
+in a variety of languages. All projects can be found under the ``src`` directory:
+
+- ``src/py/{project}`` - Python packages
+- ``src/js/app`` - ReactPy's built-in JS client
+- ``src/js/packages/{project}`` - JS packages
+
+At the root of the repository is a ``pyproject.toml`` file that contains scripts and
+their respective dependencies for managing all other projects. Most of these global
+scripts can be run via ``hatch run ...`` however, for more complex scripting tasks, we
+rely on Invoke_. Scripts implements with Invoke can be found in ``tasks.py``.
 
 Running The Tests
 -----------------
@@ -308,6 +328,8 @@ you should refer to their respective documentation in the links below:
 .. Links
 .. =====
 
+.. _Hatch: https://hatch.pypa.io/
+.. _Invoke: https://www.pyinvoke.org/
 .. _Google Chrome: https://www.google.com/chrome/
 .. _Docker: https://docs.docker.com/get-docker/
 .. _Git: https://git-scm.com/book/en/v2/Getting-Started-Installing-Git

pyproject.toml

@@ -18,6 +18,9 @@ dependencies = [
   "flake8",
   "flake8-pyproject",
   "reactpy-flake8 >=0.7",
+  # types
+  "mypy",
+  "types-toml",
   # publish
   "semver >=2, <3",
   "twine",
@@ -42,7 +45,15 @@ test-docs = "invoke test-docs"
 target-version = ["py39"]
 line-length = 88
 
-# --- Flake8 ----------------------------------------------------------------------------
+# --- MyPy -----------------------------------------------------------------------------
+
+[tool.mypy]
+ignore_missing_imports = true
+warn_unused_configs = true
+warn_redundant_casts = true
+warn_unused_ignores = true
+
+# --- Flake8 ---------------------------------------------------------------------------
 
 [tool.flake8]
 select = ["RPY"]  # only need to check with reactpy-flake8

src/py/reactpy/reactpy/core/events.py

@@ -21,7 +21,7 @@ def event(
 
 @overload
 def event(
-    function: Literal[None] = None,
+    function: Literal[None] = ...,
     *,
     stop_propagation: bool = ...,
     prevent_default: bool = ...,

src/py/reactpy/reactpy/core/layout.py

@@ -37,16 +37,16 @@
 class Layout:
     """Responsible for "rendering" components. That is, turning them into VDOM."""
 
-    __slots__ = [
+    __slots__: tuple[str, ...] = (
         "root",
         "_event_handlers",
         "_rendering_queue",
         "_root_life_cycle_state_id",
         "_model_states_by_life_cycle_state_id",
-    ]
+    )
 
     if not hasattr(abc.ABC, "__weakref__"):  # nocov
-        __slots__.append("__weakref__")
+        __slots__ += ("__weakref__",)
 
     def __init__(self, root: ComponentType) -> None:
         super().__init__()

tasks.py

@@ -15,6 +15,7 @@
 from invoke import task
 from invoke.context import Context
 from invoke.exceptions import Exit
+from invoke.runners import Result
 
 # --- Typing Preamble ------------------------------------------------------------------
 
@@ -77,14 +78,21 @@ def env(context: Context):
 @task
 def env_py(context: Context):
     """Install Python development environment"""
-    for py_proj in PY_PROJECTS:
-        py_proj_toml = toml.load(py_proj / "pyproject.toml")
-        hatch_default_env = py_proj_toml["tool"]["hatch"]["envs"].get("default", {})
-        hatch_default_features = hatch_default_env.get("features", [])
-        hatch_default_deps = hatch_default_env.get("dependencies", [])
+    for py_proj in [
+        DOCS_DIR,
+        # Docs installs non-editable versions of packages - ensure
+        # we overwrite that by installing projects afterwards.
+        *PY_PROJECTS,
+    ]:
+        py_proj_toml_tools = toml.load(py_proj / "pyproject.toml")["tool"]
+        if "hatch" in py_proj_toml_tools:
+            install_func = install_hatch_project
+        elif "poetry" in py_proj_toml_tools:
+            install_func = install_poetry_project
+        else:
+            raise Exit(f"Unknown project type: {py_proj}")
         with context.cd(py_proj):
-            context.run(f"pip install '.[{','.join(hatch_default_features)}]'")
-        context.run(f"pip install {' '.join(map(repr, hatch_default_deps))}")
+            install_func(context, py_proj)
 
 
 @task
@@ -103,6 +111,7 @@ def lint_py(context: Context, fix: bool = False):
     """Run linters and type checkers"""
     if fix:
         context.run("ruff --fix .")
+        context.run("black .")
     else:
         context.run("ruff .")
         context.run("black --check --diff .")
@@ -278,7 +287,9 @@ def get_packages(context: Context) -> dict[str, PackageInfo]:
 
 def make_py_pkg_info(context: Context, pkg_dir: Path) -> PackageInfo:
     with context.cd(pkg_dir):
-        proj_metadata = json.loads(context.run("hatch project metadata").stdout)
+        proj_metadata = json.loads(
+            ensure_result(context, "hatch project metadata").stdout
+        )
     return PackageInfo(
         name=proj_metadata["name"],
         path=pkg_dir,
@@ -321,7 +332,9 @@ def get_current_tags(context: Context) -> set[str]:
         line
         for line in map(
             str.strip,
-            context.run("git tag --points-at HEAD", hide=True).stdout.splitlines(),
+            ensure_result(
+                context, "git tag --points-at HEAD", hide=True
+            ).stdout.splitlines(),
         )
         if line
     }
@@ -410,10 +423,36 @@ def publish(dry_run: bool):
 
             context.run(
                 "twine upload dist/*",
-                env_dict={
+                env={
                     "TWINE_USERNAME": twine_username,
                     "TWINE_PASSWORD": twine_password,
                 },
             )
 
     return publish
+
+
+def install_hatch_project(context: Context, path: Path) -> None:
+    py_proj_toml = toml.load(path / "pyproject.toml")
+    hatch_default_env = py_proj_toml["tool"]["hatch"]["envs"].get("default", {})
+    hatch_default_features = hatch_default_env.get("features", [])
+    hatch_default_deps = hatch_default_env.get("dependencies", [])
+    context.run(f"pip install -e '.[{','.join(hatch_default_features)}]'")
+    context.run(f"pip install {' '.join(map(repr, hatch_default_deps))}")
+
+
+def install_poetry_project(context: Context, path: Path) -> None:
+    # install dependencies from poetry into the current environment - not in Poetry's venv
+    poetry_lock = toml.load(path / "poetry.lock")
+    packages_to_install = [
+        f"{package['name']}=={package['version']}" for package in poetry_lock["package"]
+    ]
+    context.run("pip install -e .")
+    context.run(f"pip install {' '.join(packages_to_install)}")
+
+
+def ensure_result(context: Context, *args: Any, **kwargs: Any) -> Result:
+    result = context.run(*args, **kwargs)
+    if result is None:
+        raise Exit("Command failed")
+    return result