Исправить переводы из закрытых накопительных счетов ВТБ #40
Reference in New Issue
Block a user
Delete Branch "fix/vtb-savings-transfers"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Закрывает искажение P&L при импорте существующих PDF→JSON выписок накопительных счетов ВТБ: реальные формулировки пополнения/закрытия вклада распознаются как внутренние переводы.
Проверки: npm run build -w backend; npm run test:import:direction -w backend.
CSV-парсер не добавляется: имеющиеся выписки уже соответствуют JSON schema 1.0.
Независимое ревью: блокирующее замечание. Изменение корректно распознаёт все три реальные формулировки PDF→JSON ВТБ и тест/сборка проходят, но не выполняет критерии issue #27: CSV-импорт и тест с реальным CSV по-прежнему отсутствуют (это прямо зафиксировано в issue). PR нельзя считать завершающим #27, пока issue не пересмотрен отдельным решением либо не реализован его заявленный CSV-контур.
Повторное независимое ревью по обновлённому issue #27: без блокирующих замечаний. PDF→JSON schema 1.0 принимается штатным endpoint; реальные формулировки пополнения счёта/вклада и закрытия вклада классифицируются как transfer. Идемпотентность обеспечена существующим fingerprint-индексом, а analytics не исключает accounts со status=closed.
npm run test:import:direction -w backendиnpm run build -w backendпроходят. Предыдущее замечание о CSV больше не применимо к обновлённому scope issue.Финальное независимое ревью полного PR после changelog-коммита: без блокирующих замечаний. CHANGELOG корректно отражает user-visible исправление и версию Backend 0.10.3; версия в package.json согласована. Все актуальные критерии issue #27 сохранены: PDF→JSON штатно импортируется, реальные переводы распознаются как transfer, дедупликация и фильтры closed accounts опираются на существующую проверенную логику. Финально прошли
npm run test:import:direction -w backendиnpm run build -w backend.