site: enable GitHub Pages deploy and serve the live curated corpus - #71
Open
andrewsu wants to merge 1 commit into
Open
site: enable GitHub Pages deploy and serve the live curated corpus#71andrewsu wants to merge 1 commit into
andrewsu wants to merge 1 commit into
Conversation
Pages is now configured with build_type=workflow, so wire up the deploy and switch the site off the bundled sample data. deploy-site.yml: lift the two dormancy safeties — add the `push` trigger on main for web-revamp-mockup/** (plus the workflow itself), and widen the deploy guard so a merge publishes automatically while a manual dispatch stays a build-only dry run unless deploy=true. Stale DORMANT/maintainer-checklist header replaced with what the workflow now does. Added a sanity check for a non-empty data/index.json, since the site runs in live mode and would render an empty browse page without it. dmdb-data.js: flip SOURCE to "live". record.html reads ?id= so browse results deep-link to a specific record; browse.html gains a "Curation run" facet and shows per-record sourced/flagged edge counts. scripts/build_site_data.py + web-revamp-mockup/data/: the generator and its output (index.json + 135 records across the 4 agentic curation runs, 59 distinct path ids). Data is committed rather than rebuilt in CI so the deploy stays dependency-free and the published site is exactly the reviewed diff; re-run the script after a curation and commit the result. Verified: workflow YAML parses; a dry-run dispatch on main packaged the artifact successfully; the generator reproduces the committed data byte for byte; all 135 index rows resolve to record files with no orphans; every page and JSON endpoint returns 200 over local HTTP; no root-absolute asset paths, so the site works under the /DrugMechDB-agentic/ subpath. 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
1 participant
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.
Pages is now enabled on the repo with
build_type=workflow(site URL: https://sulab.github.io/DrugMechDB-agentic/). This wires up the deploy and switches the site off the bundled sample data.What changed
.github/workflows/deploy-site.yml— dormant → live. The workflow already existed with two deliberate safeties; both are lifted:pushtrigger onmainforweb-revamp-mockup/**and the workflow file itselfgithub.event_name == 'push' || (workflow_dispatch && inputs.deploy == 'true'), so a merge publishes automatically while a manual dispatch stays a build-only dry run unless someone passesdeploy=trueThe stale DORMANT / maintainer-checklist header is replaced with what the workflow actually does now. Added a sanity check that
data/index.jsonis non-empty — the site runs in live mode, so without it the browse page would deploy green and render nothing.Site → live data.
dmdb-data.jsflipsSOURCEto"live".record.htmlreads?id=so browse results deep-link to a specific record.browse.htmlgains a Curation run facet and shows per-record sourced/flagged edge counts.scripts/build_site_data.py+web-revamp-mockup/data/— the generator and its output:index.json+ 135 records across the 4 agentic curation runs (59 distinct path ids).Why commit the generated data
The deploy stays dependency-free (no Python, no build step) and the published site is exactly the reviewed diff — a data refresh goes through normal review like any other change. Re-run
python scripts/build_site_data.pyafter a curation and commit the result. Total payload is ~1.4 MB.Verification
mainsucceeded (13s) — artifact packaging works against the real Pages configdiff -rqclean)/DrugMechDB-agentic/subpathNote for the reviewer
The site changes here (
browse.html,dmdb-data.js,record.html,build_site_data.py,data/) were sitting uncommitted in the working tree; they're carried in as-is, unmodified. Deploying without them would have published the sample-data mockup instead. The mock banners now readLIVE · …but still say the design itself is a mockup.On merge this publishes automatically to https://sulab.github.io/DrugMechDB-agentic/.
🤖 Generated with Claude Code