Repository navigation
fix(explorer): ship the built UI in the release wheel - #371
polakamtejas wants to merge 2 commits into
Conversation
The published wheel has no explorer/static/, so `agent-env up` from a PyPI install serves only the API. The release now builds the UI in a job of its own and the framework job puts it in the wheel, failing if index.html is missing. explorer/static is a git-ignored hatch build artifact, like published_images.json. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…uilt from `uv build` builds the wheel from the source archive, which dropped the git-ignored explorer/static. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
🤖 Automated Claude routine — Env Pod PR Context Review. This is a bot, not Edgar Arakelyan; not a human review. Context from outside this diff — the release path, not the change itself. The new
Net: nothing to change here — just that the built UI first reaches a wheel on the earliest release tag whose commit has a green Generated by Claude Code |
Problem
agent-env upfrom a PyPI install serves only the API: the wheel has noexplorer/static/(0 files in 0.9.1374), thoughpyproject.tomlsays the built UI ships there. Nothing in the release builds it.Change
publish-pypi.yml: a newexplorer-uijob runsnpm ci && npm run build:staticand uploadsout/. Theframeworkjob downloads it intosrc/agent_env/explorer/static/beforeuv build, and fails if the wheel has noindex.html. The UI builds in its own job, so the job that publishes to PyPI still runs no npm.pyproject.tomland.gitignore:explorer/staticis a git-ignored hatch build artifact, likepublished_images.json.[explorer] static_dir.Testing
uv build --no-sources --out-dir dist, with a placeholderexplorer/static/, ships it in both the source archive and the wheel, leavesexplorer/ui/out, andgit statusstays clean.🤖 Generated with Claude Code


Confidence Score: 5/5
The PR appears safe to merge; the latest change addresses the missing UI in the source archive.Summary
The release workflow builds the explorer UI separately and includes it in the Python package.
agent-env up.Diagram
%%{init: {'theme': 'neutral'}}%% flowchart LR A[Build explorer UI] --> B[Upload built files] B --> C[Download into explorer/static] C --> D[Build source archive] D --> E[Build wheel] E --> F[Check index.html] F --> G[Publish to PyPI]Reviews (2) · Last reviewed commit: "fix(explorer): keep the built UI in the ..." · Reviewed by Greptile