diff options
Diffstat (limited to 'CLAUDE.md')
| -rw-r--r-- | CLAUDE.md | 14 |
1 files changed, 8 insertions, 6 deletions
@@ -136,8 +136,9 @@ out of a possibly-partial MP3, and the JSON path covers the shows that mark chap - **Ad-chapter skipping trusts `toc: false` and nothing else.** No title matching: a show discussing its own sponsorship deal has a chapter called "Sponsors" that is the actual - programme. It is also off by default per show, being the one part of the feature that can - remove audio nobody has heard. + programme. It shipped off by default out of caution and Peter asked for it on after living + with it (migration 5 → 6 moves existing shows); it is still per-show and still only ever acts + on sections a publisher explicitly marked as not content. - **Auto-downloads are not retried.** A rule's download that fails is left FAILED rather than requeued: retrying forever against a 404 would burn the battery, and the failed row is visible @@ -204,10 +205,11 @@ out of a possibly-partial MP3, and the JSON path covers the shows that mark chap pulled RetentionSweeper out of its Worker, both of which found real bugs. That value only survives if the list stays honest. - `jacocoFullReport` adds instrumented coverage on top, for when you want the fuller picture - with a device attached. The commit gate deliberately does not read it: device data only exists - after a device run, so folding it in would make the gate report different numbers for - identical code. + Feeding instrumented coverage into the report as well was tried and abandoned: + `enableAndroidTestCoverage` instruments every class in the debug build, which measured at + +25MB on the APK (118MB → 143MB) plus runtime probes, on every sideload, for a report that + would be run rarely. It would also have made the gate report different numbers for identical + code depending on whether a device run happened recently. - **Detekt config has been relaxed five times**, all documented inline in `config/detekt/detekt.yml`. Twice the better answer was restructuring instead: `Screens.kt` was split per screen and |