From e2e56ac5af290b60ea01037e58e8bc00545c021c Mon Sep 17 00:00:00 2001 From: Lukasz Kasprzak Date: Thu, 20 Aug 2026 11:32:24 +0200 Subject: feat(render): Roman week numbers and date spans, as data The ordo booklet's week header repeated the month name on every week even though the heading above already established it -- pure noise. Replace it with a Roman week number plus the span of dates the week covers, e.g. "Hebdomada I (Ian 1-2)", following the project's own rule that a presentation choice is data, not code. lang/{la,en}.ini gain a [month_abbr] section (three-letter month abbreviations); Lang.month_abbr follows Lang.month's exact shape, including the out-of-range and miss-returns-the-key contracts. The coverage test now fails loudly if an abbreviation goes missing, the same as [month] already does. Every week object in the view gains num_roman (Roman numeral, num stays as the arabic original -- Roman is a presentation choice, not an engine change), first_dom/last_dom (the day-of-month of the week's first and last IN-MONTH days, padding excluded), month_abbr (resolved through Lang.month_abbr), and single_day (true when the week holds exactly one in-month day). single_day is a flag, not a preformatted span string: the engine is logic-less and cannot itself decide between "Ian 1" and "Ian 1-2", so a template makes that call from the flag instead -- the same "shape the data, not the template" discipline in_month and last already follow. Weeks are built per month with padding only at the two ends, so a week's in-month days never cross a month boundary -- verified, not assumed: every week always has at least one real day since no month is shorter than a single week. Covered by three new View tests, including a real single-day-week witness (January 2027's own trailing week is a lone Sunday, the 31st). --- test/test_lang_coverage.ml | 3 +++ 1 file changed, 3 insertions(+) (limited to 'test/test_lang_coverage.ml') diff --git a/test/test_lang_coverage.ml b/test/test_lang_coverage.ml index 9ed633c..b4de5be 100644 --- a/test/test_lang_coverage.ml +++ b/test/test_lang_coverage.ml @@ -78,6 +78,9 @@ let test_vocabularies_are_complete () = done; for n = 1 to 12 do if L.month t n = string_of_int n then Alcotest.failf "no Latin month for %d" n + done; + for n = 1 to 12 do + if L.month_abbr t n = string_of_int n then Alcotest.failf "no Latin month abbreviation for %d" n done (* Defect 1: the booklet used to print the bare rite id ("ef") because no -- cgit v1.3