From 2747bb862759321b3df93edf39278bab39cb3876 Mon Sep 17 00:00:00 2001 From: Jason Ross Date: Sun, 23 Aug 2026 00:22:53 -0500 Subject: [PATCH] Measure the coverage figure instead of carrying it forward CLAUDE.md has said "~31% of lines" since the lint/format work landed. Measured on main today it is 29.8% (629/2113 lines, 408/1424 branches). The number went DOWN, which is worth stating rather than quietly correcting. The JVM suite went from 11 test files to 43 over the same period, so the intuition -- and the review finding that prompted this, which called the direction certain -- was that coverage must have risen. It did not: main source grew from 4,114 to 5,715 lines as the fixes added Reattachment, JobSnapshots, JobTags, InputQuery, StagingNames, StagingSweep, NativeLoadFailure and an Application class. The denominator outran the numerator. That is not an argument against the tests. It is an argument against quoting a coverage percentage from memory, which is exactly how the stale figure survived. The line now carries the measurement, its date, and the instruction to re-measure. R30 / #39 Co-Authored-By: Claude Opus 5 (1M context) --- CLAUDE.md | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 80c0667..2e88d77 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -96,8 +96,12 @@ install for code that can never run — and on API 37 the full APK does not fit - The `model` package is excluded from `ReturnCount` and `CyclomaticComplexMethod` only. It is the decision layer, where one branch is one documented user-visible outcome and the metric counts answers rather than complexity. Every other rule still applies there. -- **Coverage is reported, not gated** — currently ~31% of lines. A floor needs a baseline that has - settled first. +- **Coverage is reported, not gated** — **29.8% of lines (629/2113), 28.7% of branches**, measured + on `main` 2026-08-23 with `./gradlew :app:jacocoTestReport`. A floor needs a baseline that has + settled first, and this one has not: the figure **fell** from the ~31% recorded earlier even + though the JVM suite went from 11 test files to 43. Main source grew 4,114 -> 5,715 lines over + the same period, so the denominator outran the numerator. Re-measure before quoting it; do not + assume more tests means a higher percentage here. - `kotlin.code.style=official`. Gradle stays Kotlin DSL. ## Dependency versions -- 2.47.3