Доска ИИ/Prodamus Pay 1.0/PROJ-127

Ошибка округления при частичном возврате

Вернуть разработчикуПринять и продолжить
PROJ-127БагНужен человекОткрыть в Jira · Ветка ai/PROJ-127 · Трейс в Langfuse · Воркфлоу в Restate
ВоспроизвестиГотово
Тестировщикtester-mainПадающий тест создан
ИсправлениеГотово
Разработчикdeveloper-main3 круга
ГейтыГотово
CIbuild, lint, testsВсе проверки пройдены
РевьюСтоп
Критикreviewer-strong2 blocking, лимит кругов
АудитЖдёт
Аудиторauditor-main
MRЖдёт
Финализацияrebase, гейты

Ревью, круг 3 из 3

Критик остановился на лимите кругов

Исправление закрывает округление в основном сценарии, но при нескольких частичных возвратах подряд сумма снова расходится на копейку. Разработчик дважды правил формулу, не меняя порядок округления.

Blocking
refunds/calc.ts:88Округление на каждой позиции копит ошибкуПри трёх частичных возвратах подряд итог расходится на 0,01. Округлять нужно итоговую сумму.
Blocking
refunds/calc.test.tsНет теста на серию частичных возвратовТекущий тест проверяет только один возврат и не ловит накопление.
Non-blocking
refunds/calc.ts:41Магическое число 100 для копеекВынести в константу рядом с остальными денежными утилитами.