Учитывать подтверждённый вывод с брокерского счёта #52

agent · 2026-08-28T20:41:44Z

agent commented

2026-08-28 20:41:44 +00:00

Если брокерская операция подтверждена встречным зачислением на текущем/накопительном счёте, она считается выводом даже без слова «вывод» в описании (например, «Сальдо расчётов по сделкам»). Добавлен тест.

agent added 1 commit 2026-08-28 20:41:45 +00:00

fix: count matched broker cash withdrawals 19a924647e

agent commented

2026-08-28 20:43:06 +00:00

Независимое ревью завершено. Проверил diff PR #52, классификацию подтвержденного встречным переводом вывода, исключение неподтвержденных pending-операций, версию и changelog. npm run test:portfolio:performance -w backend и npm run build -w backend проходят. Блокирующих замечаний нет.

agent added 1 commit 2026-08-28 20:53:00 +00:00

fix: reconcile split broker withdrawals 22e9656e92

agent commented

2026-08-28 20:53:36 +00:00

Повторное ревью актуального diff: найден блокирующий момент. matchExternalCashTransfers по-прежнему ограничивает брокерские кандидаты regex /вывод|сальдо расчетов|перевод|перечисление/. Поэтому подтвержденный встречным переводом withdrawal с иным текстом не попадет в matchedBrokerTransfers, хотя требование — считать его выводом независимо от текста; pending без пары при этом должен оставаться other. Добавленный тест проверяет только Сальдо расчетов и не покрывает произвольное описание. Нужен тест на произвольный текст и удаление текстового фильтра (или эквивалентная проверка только факта встречного перевода). npm run test:portfolio:performance -w backend проходит.

Повторное ревью актуального diff: найден блокирующий момент. `matchExternalCashTransfers` по-прежнему ограничивает брокерские кандидаты regex `/вывод|сальдо расчетов|перевод|перечисление/`. Поэтому подтвержденный встречным переводом withdrawal с иным текстом не попадет в `matchedBrokerTransfers`, хотя требование — считать его выводом независимо от текста; pending без пары при этом должен оставаться `other`. Добавленный тест проверяет только `Сальдо расчетов` и не покрывает произвольное описание. Нужен тест на произвольный текст и удаление текстового фильтра (или эквивалентная проверка только факта встречного перевода). `npm run test:portfolio:performance -w backend` проходит.

agent added 1 commit 2026-08-28 20:54:17 +00:00

fix: match broker transfers regardless of description a6250f1592

agent commented

2026-08-28 20:55:04 +00:00

Финальное независимое ревью актуального PR завершено. Исправление проверено: групповой matcher больше не фильтрует брокерскую операцию по описанию; встречный перевод подтверждает вывод даже при произвольном тексте, а pending без пары остается other. Тест дополнен произвольным описанием и проверкой суммы двух списаний. npm run test:portfolio:performance -w backend и npm run build -w backend проходят. Блокирующих замечаний нет.

Финальное независимое ревью актуального PR завершено. Исправление проверено: групповой matcher больше не фильтрует брокерскую операцию по описанию; встречный перевод подтверждает вывод даже при произвольном тексте, а pending без пары остается `other`. Тест дополнен произвольным описанием и проверкой суммы двух списаний. `npm run test:portfolio:performance -w backend` и `npm run build -w backend` проходят. Блокирующих замечаний нет.

agent added 1 commit 2026-08-28 22:03:03 +00:00

fix: count confirmed broker withdrawals by account reference a6a46e0d84

agent commented

2026-08-28 22:04:04 +00:00

Финальное независимое ревью актуального PR после a6a46e0 завершено. Проверил: учитываются только положительные внешние операции с формулировкой Вывод денежных средств с брокерского счета N, счёт связывается по номеру; частичный/разбитый вывод не теряется, pending без такой внешней операции не добавляется; логика contributions сохранена. Тесты и сборка проходят: npm run test:portfolio:performance -w backend, npm run build -w backend. Версия и changelog обновлены. Блокирующих замечаний нет.

Финальное независимое ревью актуального PR после a6a46e0 завершено. Проверил: учитываются только положительные внешние операции с формулировкой `Вывод денежных средств с брокерского счета N`, счёт связывается по номеру; частичный/разбитый вывод не теряется, pending без такой внешней операции не добавляется; логика contributions сохранена. Тесты и сборка проходят: `npm run test:portfolio:performance -w backend`, `npm run build -w backend`. Версия и changelog обновлены. Блокирующих замечаний нет.

admin merged commit 353face40e into main

2026-08-28 22:09:17 +00:00

admin referenced this issue from a commit

2026-08-28 22:09:20 +00:00

Merge pull request 'Учитывать подтверждённый вывод с брокерского счёта' (#52) from fix/confirmed-broker-withdrawals into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: admin/family_budget#52