web: commit built frontend assets so cold-start from a clone works

blankjul · blankjul · commit 4787ec5835f6 · 2026-06-30T18:47:04.000-07:00
The built SPA (src/pyclawd/web/static/) was gitignored and shipped only as a
build-time hatch artifact — present on a release machine after npm run build,
but absent from any fresh git clone. Installing '.[web]' from a checkout thus
produced a wheel with no UI and 'pyclawd web serve' served a blank dashboard.

Commit the built assets (rebuilt fresh), drop the now-redundant artifact globs,
and warn in 'web serve' if the static dir is ever missing again. No Node needed
downstream; rebuild + commit web/static after changing the frontend source.
diff --git a/.gitignore b/.gitignore
@@ -223,9 +223,11 @@ __marimo__/
 # Claude Code agent worktrees (transient — never commit)
 .claude/worktrees/
 
-# pyclawd web — built frontend assets (produced by `npm run build` in
-# src/pyclawd/web_frontend; shipped into the wheel via hatch artifacts, not git).
-src/pyclawd/web/static/
+# pyclawd web — the built frontend assets in src/pyclawd/web/static/ are
+# COMMITTED (not ignored) so a fresh `git clone` + `pip install '.[web]'` ships a
+# working dashboard with no Node toolchain required. Regenerate them with
+# `npm run build` in src/pyclawd/web_frontend after changing the frontend source.
+# Only the Node install dir is ignored.
 src/pyclawd/web_frontend/node_modules/
 
 # Repo-root convenience symlink → src/pyclawd/skills (real tree is the package
diff --git a/pyproject.toml b/pyproject.toml
@@ -72,24 +72,24 @@ path = "src/pyclawd/__init__.py"
 # the default VCS file-walk: in this repo the walk silently dropped the
 # skills/pyclawd and skills/pyclawd-doctor subtrees, so we exclude skills from the
 # walk and force-include the whole tree deterministically. The BUILT web/static
-# frontend (gitignored) rides along as an artifact. (The repo-root `skills` symlink is
-# untracked — see .gitignore — so it cannot shadow the real package tree.)
+# frontend is now COMMITTED (see .gitignore), so the VCS walk picks it up on any
+# clone — no Node needed to build a complete wheel from a fresh checkout. (The
+# repo-root `skills` symlink is untracked — see .gitignore — so it cannot shadow the
+# real package tree.)
 [tool.hatch.build.targets.sdist]
 exclude = ["src/pyclawd/skills"]
-artifacts = ["src/pyclawd/web/static/**"]
 
 [tool.hatch.build.targets.sdist.force-include]
 "src/pyclawd/skills" = "src/pyclawd/skills"
 
 [tool.hatch.build.targets.wheel]
 packages = ["src/pyclawd"]
 include = ["src/pyclawd/py.typed"]  # PEP 561 — ship the typed marker (we're a typed lib)
-# Ship the BUILT web frontend (gitignored, so listed as an artifact to override VCS
-# exclusion) and keep its SOURCE (the Vite/React project) OUT of the wheel — only
-# `pyclawd[web]` users need the static bundle, nobody needs the TS sources or Node.
+# The BUILT web frontend (src/pyclawd/web/static/) is committed, so the package walk
+# ships it automatically. Keep its SOURCE (the Vite/React project) OUT of the wheel —
+# only `pyclawd[web]` users need the static bundle, nobody needs the TS sources or Node.
 # Exclude skills from the package walk; it is force-included below (same reason as the
 # sdist — the VCS walk drops subtrees of it).
-artifacts = ["src/pyclawd/web/static/**"]
 exclude = ["src/pyclawd/web_frontend", "src/pyclawd/skills"]
 
 # force-include ships, by literal filesystem copy (no VCS heuristics):
diff --git a/src/pyclawd/commands/web.py b/src/pyclawd/commands/web.py
@@ -74,7 +74,17 @@ def serve(
     _require_web_stack()
     import uvicorn
 
-    from pyclawd.web.app import create_app
+    from pyclawd.web.app import _STATIC_DIR, create_app
+
+    if not _STATIC_DIR.is_dir():
+        typer.secho(
+            "warning: the built frontend is missing — only the JSON API will be served, "
+            "the dashboard UI will not load.\n"
+            "In a source checkout, build it with:  "
+            "cd src/pyclawd/web_frontend && npm install && npm run build",
+            fg="yellow",
+            err=True,
+        )
 
     reg = _registry()
     default = None
diff --git a/src/pyclawd/web/static/assets/index-C9ezW9Ye.css b/src/pyclawd/web/static/assets/index-C9ezW9Ye.css
diff --git a/src/pyclawd/web/static/assets/index-CQrkiI_I.js b/src/pyclawd/web/static/assets/index-CQrkiI_I.js
diff --git a/src/pyclawd/web/static/index.html b/src/pyclawd/web/static/index.html
@@ -0,0 +1,13 @@
+<!doctype html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+    <title>pyclawd web</title>
+    <script type="module" crossorigin src="/assets/index-CQrkiI_I.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-C9ezW9Ye.css">
+  </head>
+  <body>
+    <div id="root"></div>
+  </body>
+</html>
