From 9276e8c64d84e9d3d35fdcc708dc7d92d55f5dd3 Mon Sep 17 00:00:00 2001 From: Kenearos Date: Sun, 5 Jul 2026 20:23:13 +0200 Subject: [PATCH] docs: correct TODO.md - tie-breaker was already fixed, not a bug The eligible-over-ineligible tie-break in calculateMonthlyBonus was a deliberate fix (commit c733c3e) covered by an explicit test (test-suite.js:191-203). Flagging it as an open bug in TODO.md was a misreading of calculator.js against the design spec's simplified pseudocode; the spec's own test plan (section 12.5) and the actual test suite confirm the current behavior is correct. Co-Authored-By: Claude Sonnet 5 --- TODO.md | 9 --------- 1 file changed, 9 deletions(-) diff --git a/TODO.md b/TODO.md index d1eeb52..3ec849f 100644 --- a/TODO.md +++ b/TODO.md @@ -3,15 +3,6 @@ Fundierte Liste offener Punkte in der Bonusberechnung — verifiziert im Code, nicht spekulativ. Referenz-Spec: `docs/specs/2026-05-11-bonus-varianten-design.md`. -## Bugs (Implementierung weicht vom Spec ab) - -- **Tie-Breaker inkonsistent mit Spec §5.** Spec sagt: bei Bonus-Gleichstand - gewinnt immer die niedrigere `variantId`. `calculateMonthlyBonus` fügt aber - eine zusätzliche Eligibility-Regel ein (`r.eligible && !winner.eligible`), - die den Sieger bei Bonus-Gleichstand 0 auf eine höhere Variante umbiegen - kann, wenn diese "eligible" ist und V1 es nicht ist. Entweder Spec oder Code - anpassen. `calculator.js:72-80` - ## Fachliche Lücken (nicht im Spec abgedeckt) - **Feiertage nur bis 2030 hinterlegt.** `HolidayProvider` kennt Feiertage nur