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 <noreply@anthropic.com>
This commit is contained in:
parent
3598a4de9e
commit
9276e8c64d
1 changed files with 0 additions and 9 deletions
9
TODO.md
9
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
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue