Repository navigation
docs: site parity after #1531 — TDD line and one verified basis for the figures - #1537
Conversation
#1531 put "TDD protects your code. This protects your tests." on the README's first screen, directly under the demo. The site carried the same headline, the same install line and the same demo, but not this. It is the first objection an engineer raises after watching the clip, so it belongs next to the clip on both surfaces. Placed after the figure, inside the header, mirroring the README's order. The new paragraph has no stylesheet rule of its own, so it renders at body size and colour rather than the muted `.lede` treatment — which is wanted here, since it is the punchline rather than the subtitle. tests/ci: 81 passed. The limitations blockquote is untouched, so it stays pinned to the README's beta note. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019cEymaSpmZ1arjngxx1VNM
The README said 5,000+ tests and an 80% floor; the site said 7,500+ tests and 85%+. The site's 7,500+ was right on 2026-10-14 (7,891 at d26a5c8) and is no longer true. The README's figures were true and undersold. Measured on main @ 507c6e0, 2026-10-28, Backend Unit Tests job: 6,035 passed, 86.13% coverage against fail_under = 80. Both surfaces now say 6,000+ tests and 85%+ coverage with an 80% floor, rounded down per the site's convention, and both carry a comment with the measurement and a pointer to the other file. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019cEymaSpmZ1arjngxx1VNM
|
Review: LGTM. This is a docs-only change with no code or security impact.
Optional: nothing checks that the README and site figures agree, so a small 🤖 Generated with Claude Code |
|
Re-review: no changes since my earlier LGTM, and I still have nothing blocking. This is a docs-only diff. The README and site figures now match (6,000+ tests, 85%+ coverage, 80% floor), and each file has a comment saying where the numbers came from. The earlier optional suggestion stands: a small 🤖 Generated with Claude Code |
tests/test_lifecycle_gates_948.py holds README.md to fail_under: any "NN% test coverage" it states must be a number CI will actually enforce. The previous commit wrote "85%+ test coverage" — true today (86.13%), enforced by nothing — and the guard failed it, which is what it exists to do. The README shipped an 88% badge and an 85%+ ask before #948 for the same reason: a measured figure nothing checks drifts until it is false. README now states the 80% floor. The site states the same, so neither surface carries a coverage figure nothing enforces. The measured value stays in the HTML comment with its commit, as a dated fact rather than a claim. Full CI selection locally (tests/ --ignore=tests/e2e -m "not lifecycle"): 6041 passed, 0 failed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019cEymaSpmZ1arjngxx1VNM
|
Claude finished @frankbria's task in 1m 24s —— View job Third-party review — ✅ no defects found
Docs-only change. Figures are consistent on both surfaces and rounded down from a plausible measurement (5,290 static test definitions + parametrization ≈ the cited 6,035 collected; I could not re-query CI from this environment). The |
|
Re-review: the diff has changed since my earlier LGTMs, and I have nothing blocking. It is still docs-only.
I didn't run the tests. 🤖 Generated with Claude Code |
|
✅ Third Party Review: no defects found. Docs-only diff (README.md, site/index.html). Verified against the checked-out tree:
Model: k3[1m] |
Brings the landing page and the README into line after #1531.
1. The TDD contrast on the site
#1531 put this on the README's first screen, under the demo. The site had the same headline, install line and demo, but not this:
Placed after the figure, inside
<header>, mirroring the README..tddhas no stylesheet rule, so it renders at body size and colour rather than the muted.lede— deliberate, since it is the punchline.2. One verified basis for the figures, on both surfaces
The README and the site stated different figures for the same thing, and one of the site's had gone stale.
The site's 7,500+ was correct when written (7,891 at
d26a5c8, 14 Oct) and is no longer true. The README's figures were true but undersold.Source: the Backend Unit Tests job on
main@507c6e0c, 2026-10-28 —6035 passed, 38 skipped, 2 deselectedandRequired test coverage of 80.0% reached. Total coverage: 86.13%.Rounded down, following the site's existing convention, and both places now carry an HTML comment with the measurement and a pointer to the other file, so the next person can re-check rather than trust.
Verification
uv run pytest tests/ci/ -q→ 81 passed<blockquote id="limitations">untouched; still pinned to the README's beta note🤖 Generated with Claude Code
https://claude.ai/code/session_019cEymaSpmZ1arjngxx1VNM