fully functional build sequence for JS

Author: Archmonger

pyproject.toml

@@ -58,9 +58,8 @@ installer = "uv"
 
 [[tool.hatch.build.hooks.build-scripts.scripts]]
 commands = [
-  'bun install --cwd "src/js/packages/@reactpy/app" --production --frozen-lockfile',
-  # TODO: Remove 'assets' from this path after turning ReactPy into ASGI middleware.
-  'bun build "./src/js/packages/@reactpy/app/src/index.ts" --outdir "./src/reactpy/static/assets/" --minify --sourcemap=linked',
+  "hatch run javascript:build",
+  'hatch run "src/build_scripts/copy_dir.py" "src/js/packages/@reactpy/app/dist" "src/reactpy/static/assets"',
 ]
 artifacts = []
 
@@ -166,13 +165,23 @@ type_check = "mypy --strict src/reactpy"
 detached = true
 
 [tool.hatch.envs.javascript.scripts]
-check = [
-  'bun install --cwd "src/js" --frozen-lockfile',
-  'bun run --cwd "src/js" check',
+check = ['bun install --cwd "src/js"', 'bun run --cwd "src/js" check']
+fix = ['bun install --cwd "src/js"', 'bun run --cwd "src/js" format']
+build = [
+  'hatch run "src/build_scripts/clean_js_dir.py"',
+  'hatch run javascript:build_packages',
+  'hatch run javascript:build_app',
 ]
-fix = [
-  'bun install --cwd "src/js" --frozen-lockfile',
-  'bun run --cwd "src/js" format',
+build_packages = [
+  'bun install --cwd "src/js"',
+  'bun install --cwd "src/js/packages/event-to-object"',
+  'bun install --cwd "src/js/packages/@reactpy/client"',
+  'bun run --cwd "src/js/packages/event-to-object" build',
+  'bun run --cwd "src/js/packages/@reactpy/client" build',
+]
+build_app = [
+  'bun install --cwd "src/js/packages/@reactpy/app"',
+  'bun run --cwd "src/js/packages/@reactpy/app" build',
 ]
 
 #########################

src/build_scripts/clean_js_dir.py

@@ -0,0 +1,41 @@
+# /// script
+# requires-python = ">=3.11"
+# dependencies = []
+# ///
+
+# Deletes `dist`, `node_modules`, and `tsconfig.tsbuildinfo` from all JS packages in the JS source directory.
+
+import contextlib
+import glob
+import os
+import pathlib
+import shutil
+
+# Get the path to the JS source directory
+js_src_dir = pathlib.Path(__file__).parent.parent / "js"
+
+# Get the paths to all `dist` folders in the JS source directory
+dist_dirs = glob.glob(str(js_src_dir / "**/dist"), recursive=True)
+
+# Get the paths to all `node_modules` folders in the JS source directory
+node_modules_dirs = glob.glob(str(js_src_dir / "**/node_modules"), recursive=True)
+
+# Get the paths to all `tsconfig.tsbuildinfo` files in the JS source directory
+tsconfig_tsbuildinfo_files = glob.glob(
+    str(js_src_dir / "**/tsconfig.tsbuildinfo"), recursive=True
+)
+
+# Delete all `dist` folders
+for dist_dir in dist_dirs:
+    with contextlib.suppress(FileNotFoundError):
+        shutil.rmtree(dist_dir)
+
+# Delete all `node_modules` folders
+for node_modules_dir in node_modules_dirs:
+    with contextlib.suppress(FileNotFoundError):
+        shutil.rmtree(node_modules_dir)
+
+# Delete all `tsconfig.tsbuildinfo` files
+for tsconfig_tsbuildinfo_file in tsconfig_tsbuildinfo_files:
+    with contextlib.suppress(FileNotFoundError):
+        os.remove(tsconfig_tsbuildinfo_file)

src/build_scripts/copy_dir.py

@@ -0,0 +1,40 @@
+# /// script
+# requires-python = ">=3.11"
+# dependencies = []
+# ///
+
+# ruff: noqa: INP001
+import logging
+import shutil
+import sys
+from pathlib import Path
+
+
+def copy_files(source: Path, destination: Path) -> None:
+    if destination.exists():
+        shutil.rmtree(destination)
+    destination.mkdir()
+
+    for file in source.iterdir():
+        if file.is_file():
+            shutil.copy(file, destination / file.name)
+        else:
+            copy_files(file, destination / file.name)
+
+
+if __name__ == "__main__":
+    if len(sys.argv) != 3:
+        logging.error(
+            "Script used incorrectly!\nUsage: python copy_dir.py <source_dir> <destination>"
+        )
+        sys.exit(1)
+
+    root_dir = Path(__file__).parent.parent.parent
+    src = Path(root_dir / sys.argv[1])
+    dest = Path(root_dir / sys.argv[2])
+
+    if not src.exists():
+        logging.error("Source directory %s does not exist", src)
+        sys.exit(1)
+
+    copy_files(src, dest)

src/js/packages/@reactpy/app/bun.lockb

@@ -3,12 +3,15 @@
   "description": "ReactPy's client-side entry point. This is strictly for internal use and is not designed to be distributed.",
   "license": "MIT",
   "dependencies": {
-    "@reactpy/client": "^0.3.2",
-    "event-to-object": "^0.1.2",
+    "@reactpy/client": "file:../client",
+    "event-to-object": "file:../../event-to-object",
     "preact": "^10.7.0"
   },
   "devDependencies": {
     "@types/react": "^17.0",
     "@types/react-dom": "^17.0"
+  },
+  "scripts": {
+    "build": "bun build \"src/index.ts\" --outdir \"dist\" --minify --sourcemap=linked"
   }
 }

src/js/packages/@reactpy/app/package.json

@@ -17,7 +17,6 @@
   "type": "module",
   "main": "dist/index.js",
   "dependencies": {
-    "event-to-object": "^0.1.2",
     "json-pointer": "^0.6.2",
     "preact": "^10.25.4"
   },
@@ -26,6 +25,9 @@
     "@types/react": "^17.0",
     "@types/react-dom": "^17.0"
   },
+  "peerDependencies": {
+    "event-to-object": "<1.0.0"
+  },
   "scripts": {
     "build": "tsc -b",
     "checkTypes": "tsc --noEmit"