Skip to content

[sberbank-by] Поправлена дедупликация#1037

Open
Kabaye wants to merge 2 commits into
zenmoney:masterfrom
Kabaye:fix/sberbank-by-posted-dedup
Open

[sberbank-by] Поправлена дедупликация#1037
Kabaye wants to merge 2 commits into
zenmoney:masterfrom
Kabaye:fix/sberbank-by-posted-dedup

Conversation

@Kabaye
Copy link
Copy Markdown
Contributor

@Kabaye Kabaye commented May 13, 2026

Что исправлено

Исправлена логика дедупликации операций в плагине sberbank-by.

Раньше одна и та же операция могла повторно появляться как непрочитанная или дублироваться между синхронизациями, если банк сначала отдавал её как hold, а позже как проведённую posted-операцию.

Теперь:

  • при дедупликации приоритет отдаётся проведённой posted-операции вместо hold
  • ключ дедупликации для API-операций стал более устойчивым и предсказуемым
  • добавлен дополнительный guard для безопасной обработки транзакций при вторичной дедупликации
  • сохранена существующая логика приоритета записей с переводами, чтобы не сломать текущую группировку переводов

Почему это важно

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

Особенно это касается кейсов вроде:

  • ЕРИП
  • пополнения/движения по вкладам
  • операций, которые сначала видны только как карточные hold

Что проверено

  • целевые тесты sberbank-by: 10/10 suites, 45/45 tests
  • production-сборка плагина sberbank-by
  • локальный запуск debug UI

Kabaye and others added 2 commits May 13, 2026 22:50
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Kabaye Kabaye changed the title [sberbank-by] prefer posted transactions over holds [sberbank-by] Поправлена дедупликация May 14, 2026
@Kabaye Kabaye marked this pull request as ready for review May 14, 2026 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant