Если брокерская операция подтверждена встречным зачислением на текущем/накопительном счёте, она считается выводом даже без слова «вывод» в описании (например, «Сальдо расчётов по сделкам»). Добавлен тест.
Если брокерская операция подтверждена встречным зачислением на текущем/накопительном счёте, она считается выводом даже без слова «вывод» в описании (например, «Сальдо расчётов по сделкам»). Добавлен тест.
Повторное ревью актуального 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` проходит.
Финальное независимое ревью актуального 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` проходят. Блокирующих замечаний нет.
Финальное независимое ревью актуального 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 main2026-08-28 22:09:17 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Если брокерская операция подтверждена встречным зачислением на текущем/накопительном счёте, она считается выводом даже без слова «вывод» в описании (например, «Сальдо расчётов по сделкам»). Добавлен тест.
Независимое ревью завершено. Проверил diff PR #52, классификацию подтвержденного встречным переводом вывода, исключение неподтвержденных pending-операций, версию и changelog. npm run test:portfolio:performance -w backend и npm run build -w backend проходят. Блокирующих замечаний нет.
Повторное ревью актуального diff: найден блокирующий момент.
matchExternalCashTransfersпо-прежнему ограничивает брокерские кандидаты regex/вывод|сальдо расчетов|перевод|перечисление/. Поэтому подтвержденный встречным переводом withdrawal с иным текстом не попадет вmatchedBrokerTransfers, хотя требование — считать его выводом независимо от текста; pending без пары при этом должен оставатьсяother. Добавленный тест проверяет толькоСальдо расчетови не покрывает произвольное описание. Нужен тест на произвольный текст и удаление текстового фильтра (или эквивалентная проверка только факта встречного перевода).npm run test:portfolio:performance -w backendпроходит.Финальное независимое ревью актуального PR завершено. Исправление проверено: групповой matcher больше не фильтрует брокерскую операцию по описанию; встречный перевод подтверждает вывод даже при произвольном тексте, а pending без пары остается
other. Тест дополнен произвольным описанием и проверкой суммы двух списаний.npm run test:portfolio:performance -w backendиnpm run build -w backendпроходят. Блокирующих замечаний нет.Финальное независимое ревью актуального PR после
a6a46e0завершено. Проверил: учитываются только положительные внешние операции с формулировкойВывод денежных средств с брокерского счета N, счёт связывается по номеру; частичный/разбитый вывод не теряется, pending без такой внешней операции не добавляется; логика contributions сохранена. Тесты и сборка проходят:npm run test:portfolio:performance -w backend,npm run build -w backend. Версия и changelog обновлены. Блокирующих замечаний нет.