[dmccoystephenson]

Summary

Stage B (unit-test expansion) cycle, scoped to drawButton.

• A characterization test, test_draw_button_paints_before_invoking_the_callback, is added. It pins the existing order inside drawButton: the box and label are painted first, and function() is invoked last.
• The ordering is consumer-observable. A callback that paints (for example, a menu button that switches to another screen) currently lands on top of the button. If the hit test were moved ahead of the drawing (e.g. to choose a hover color), the button would paint over the callback's output in the same frame, and every existing drawButton test would stay green.
• The test asserts both halves: the box pixel is already colorBox when the callback runs, and the callback's full-surface fill survives once drawButton returns.
• No production code is changed. The change is additive and test-only, so the public API seen by Roam/Apex/Ophidian/Patchwork/Tic-Tak-Toe is unaffected.

Test plan

• python3 -m pytest -q: 63 passed (62 existing + 1 new), headless via conftest.py's dummy SDL drivers
• Mutation check: with the hit test temporarily moved ahead of the drawing in graphik.py, the new test FAILS. With graphik.py restored, it PASSES.
• CI Test workflow green on 3.9–3.13

Deferred issues

All four open issues were deliberately skipped, because each needs a maintainer decision or touches a path excluded from autonomous change:

• release: publish.yml's wheel smoke test never imports Graphik, so a wheel missing graphik.py would still publish #46 and ci: test.yml's header comment claims it runs on every push, but pushes outside main are only covered via pull_request #39 can only be fixed by editing .github/workflows/, which is excluded from autonomous change.
• release: version tags are pushed without the 'v' prefix publish.yml requires, so no release has ever been published #34 needs a maintainer choice between fixing the tag practice and broadening the publish trigger.
• packaging: the empty preponderous/__init__.py ships a regular package, which will collide with any future preponderous.* distribution #40 depends on a maintainer call about the preponderous.* namespace convention used by sibling projects.

No tracking issue: the coverage gap was found during this cycle's test-expansion sweep.

This PR description was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).

🤖 Generated with Claude Code

---

drafted by Claude on behalf of Daniel Stephenson

[dmccoystephenson]

Self-review rubric (inline review performed in the same session; no independent reviewer was available because adding Copilot as a reviewer failed with Could not resolve user with login 'copilot'):

• Scope: PASS. One file changed (src/test/python/preponderous/graphik/test_graphik.py, +27/-0), and every line belongs to the new test.
• Tests-new: PASS (n/a). No new public method was added.
• Tests-fix (empirical): PASS. This is characterization, not a fix. It was still mutation-checked: with the hit test moved ahead of the drawing in graphik.py, the test failed at test_graphik.py:974. With graphik.py restored via git checkout, all 63 tests passed.
• Sibling structure: PASS. Written in the shape of its neighbours, test_draw_button_fires_callback_once_per_call_while_mouse_held and _click_at: it uses _make_graphik, _rgb, and monkeypatches pygame.mouse.get_pos/get_pressed.
• Sibling renames: PASS (n/a). Nothing was renamed.
• Docs: PASS. The README and the drawButton docstring say nothing about callback ordering, so no documentation is affected. It was left undocumented on purpose, since documenting it would widen the promised contract and that is a maintainer call.
• Issue resolution: PASS (n/a). The PR has no Closes. The body records the skip reasons for release: version tags are pushed without the 'v' prefix publish.yml requires, so no release has ever been published #34, ci: test.yml's header comment claims it runs on every push, but pushes outside main are only covered via pull_request #39, packaging: the empty preponderous/__init__.py ships a regular package, which will collide with any future preponderous.* distribution #40, and release: publish.yml's wheel smoke test never imports Graphik, so a wheel missing graphik.py would still publish #46.
• Manual validation: PASS. The Test workflow is green on 3.9, 3.10, 3.11, 3.12, and 3.13. Locally, python3 -m pytest -q reports 63 passed.
• camelCase: PASS (n/a). No method was added to Graphik.
• Backward-compat: PASS. graphik.py is untouched.
• Headless: PASS. The test runs under conftest.py's SDL_VIDEODRIVER=dummy.
• Version sync: PASS (n/a). No version string changed.
• No new deps: PASS. No new imports were added.

Finding (non-blocking):

• src/test/python/preponderous/graphik/test_graphik.py:972: the in-callback probe checks only a box pixel (11, 11), not the label, so it does not on its own prove the label was drawn before the callback. The final full-surface assertion does cover the label: any label drawn after the callback would leave non-green pixels behind. The behaviour the test protects (the callback's output stays on top) is therefore pinned in full. No change is planned.

The review is clean apart from the one non-blocking observation above.

This review comment was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).

---

drafted by Claude on behalf of Daniel Stephenson

[dmccoystephenson]

Correction to the self-review above: the in-callback probe is at src/test/python/preponderous/graphik/test_graphik.py:968, not :972. The finding itself is unchanged.

This comment was drafted during a Gardener session (https://github.com/Stephenson-Software/gardener).

---

drafted by Claude on behalf of Daniel Stephenson