The Heart board joins the family look - #169
Merged
Merged
Conversation
The health board carried its own copy of the GitHub-grey stylesheet: a plain h1, dark-only, GitHub-blue links, and verdict pills in colours picked here rather than from the logo above the README. It renders through the Brain's `board/_theme.py` instead: the hero (the heart with its ECG and check badge, the wordmark, the hairline-and-dot rule, the tagline), the accent as link and heading colour, and the verdict as the family's banner. The theme is imported, never copied — heart-health.yml and heart-tests.yml check PyAutoBrain out beside this repo, and a local run resolves the sibling checkout the way the other PyAuto tools find each other. Only the html surface calls the theme. The md, json and badge surfaces are untouched, so the Health Agent and mobile keep reading the same contract with no PyAutoBrain in reach — a look is not evidence, and nothing here changes what the Heart observes or how it scores. The per-row state dot, the evidence lists and the stale banner stay here, written against the theme's variables so this board follows the family accent rather than setting a second palette. Verdict tone maps once (red/yellow/stale/green → the theme's bad/warn/warn/ok); `_VERDICT_STATE` stays the internal truth. Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_01FhsBsESQUAoJQ2fvLiDD5j
The tenant firewall caught it (PyAutoHeart#169 CI): the theme resolver's last candidate was a literal `~/Code/PyAutoLabs/PyAutoBrain`, and the org's own directory name is an instance fact — organ code may not carry one, whatever the local convention happens to be. The fallback reads `$PYAUTO_ROOT` instead (defaulting to `~/Code`), the same declared surface the dev-flow doors use. `$PYAUTO_BRAIN` and the two checkout paths are unchanged, so CI and a sibling workspace both resolve exactly as before. Co-Authored-By: Claude Opus 5 <[email protected]> Claude-Session: https://claude.ai/code/session_01FhsBsESQUAoJQ2fvLiDD5j
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
2 participants
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.
One of five: the boards that had not yet adopted the shared theme (PyAutoBrain#262, which drew each organ's logo glyph as the hero mark). Siblings: PyAutoHands, PyAutoMemory, PyAutoScientist.
The health board carried its own copy of the GitHub-grey stylesheet — a plain
<h1>, dark-only, GitHub-blue links, verdict pills in colours picked here rather than from the logo above the README.What changed
board/_theme.py: the hero (the heart with its ECG and check badge, the wordmark, the hairline-and-dot rule, the tagline), the accent as link and heading colour, light and dark.heart-health.ymlandheart-tests.ymlcheck PyAutoBrain out beside this repo; a local run resolves$PYAUTO_BRAIN, either checkout path, or$PYAUTO_ROOT(default~/Code). Vendoring a fifth copy of the stylesheet is the drift_theme.pyexists to end._VERDICT_STATEstays the internal truth.data-cmd), replacing the per-page script.Verification
pytest tests/ -q— 587 passed; both CI legs (3.12, 3.13) green.#58a6ffis gone.CI caught one thing (second commit): the first resolver spelled its last fallback
~/Code/PyAutoLabs/PyAutoBrain, and the tenant firewall rightly rejects an org name in organ code. It reads$PYAUTO_ROOTinstead — same resolution, no instance fact.