Repository navigation
Release v1.9.5: build-cover-dist.sh emits the local cover snapshot - #66
Merged
Merged
Conversation
Adds a LOCAL_COPY phase (default docs/cover.html) so the committed, browsable snapshot of the landing page is generated rather than hand-synced. The manual step produced two bugs in a row: a copy several versions behind, and a hero image that never resolved. Hand-maintaining it could not work. The version badge is injected at build time, so an edited duplicate is stale as soon as the version moves; and because the copy lands outside dist-cover/, its relative asset paths dangle. The phase runs after the secret scan, so a snapshot is never written carrying a leak the scan would have rejected. Asset references are re-pointed at COVER_SRC relative to wherever the copy lands; anchors, absolute URLs, and refs that do not resolve in the source tree are left untouched rather than silently rewritten. LOCAL_COPY="" skips it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why this had to move into the build
docs/cover.htmlwas hand-synced, and produced two bugs in a row:vv1.8.0while the repo was on 1.9.x).actop-demo.gifrelative todocs/, where no such file exists.Neither was carelessness; the manual step cannot be done correctly. The version badge is injected at build time from
pyproject.toml, so any hand-edited duplicate is stale the moment the version moves. And because the copy lands outsidedist-cover/, every relative asset path in it dangles by construction.What changed
New
LOCAL_COPYphase (defaultdocs/cover.html,""to skip):COVER_SRC, computed relative to wherever the copy lands —docs/cover.htmlgets../cover/actop-demo.gif.#features), absolute URLs, protocol-relative and already-relative refs, and anything that does not resolve to a real file in the source tree. It rewrites only what it can verify, rather than silently pointing something somewhere wrong.index.htmlonly — this is a browsable snapshot, not a second dist.Verified
Asset resolves; anchors and external links untouched; badge matches the tag.
bash -nclean.Skill docs updated with the phase, the env var, and an explicit "never edit it by hand" note.
🤖 Generated with Claude Code