refactor: simplify frontend state and mutation boundaries #44

2026-09-28T20:02:50+03:00

codex-bot commented

2026-09-28 20:02:50 +03:00

CardPage передавал во view целые query-объекты и сеттеры состояния, а банковские формы создавали семь mutations независимо от используемой операции. Теперь view получает данные и именованные действия, а потребители создают только необходимые mutations с общей политикой invalidation.

Изменения:

Сужены интерфейс CardPage VM и публичные exports банковских mutations.
Удалён useTbankSyncStatus, использовавшийся только тестом; проверка поздних ответов выполняется через реальный polling useScreenSync.
Удалены промежуточные presentation VM поиска акций и облигаций.
Сопоставление категорий и иконок проверяется напрямую; доступность иконки проверяется в пакете icons.
Повторяющиеся сценарии отображения позиций объединены в табличные тесты.
Добавлены проверки открытия и отмены четырёх диалогов CardPage с возвратом фокуса.

Поведение денежных операций, конфликтов версий, invalidation, cached snapshot и защиты от поздних ответов сохранено. Новых зависимостей нет; суммарно удалено 133 строки.

Проверено:

npm ci и полный npm run verify: 415 тестов в 58 файлах, включая browser-тесты; typecheck, lint, OpenAPI, Knip, dependency-cruiser, сборки приложения и Storybook прошли.
Coverage: строки 92.14%, ветви 84.08%.
git diff --check, gitleaks и commit hooks прошли; hooks не изменили проверенный diff.

Сборка Storybook сообщает предупреждение о чанках больше 500 kB.

CardPage передавал во view целые query-объекты и сеттеры состояния, а банковские формы создавали семь mutations независимо от используемой операции. Теперь view получает данные и именованные действия, а потребители создают только необходимые mutations с общей политикой invalidation. Изменения: - Сужены интерфейс CardPage VM и публичные exports банковских mutations. - Удалён useTbankSyncStatus, использовавшийся только тестом; проверка поздних ответов выполняется через реальный polling useScreenSync. - Удалены промежуточные presentation VM поиска акций и облигаций. - Сопоставление категорий и иконок проверяется напрямую; доступность иконки проверяется в пакете icons. - Повторяющиеся сценарии отображения позиций объединены в табличные тесты. - Добавлены проверки открытия и отмены четырёх диалогов CardPage с возвратом фокуса. Поведение денежных операций, конфликтов версий, invalidation, cached snapshot и защиты от поздних ответов сохранено. Новых зависимостей нет; суммарно удалено 133 строки. Проверено: - npm ci и полный npm run verify: 415 тестов в 58 файлах, включая browser-тесты; typecheck, lint, OpenAPI, Knip, dependency-cruiser, сборки приложения и Storybook прошли. - Coverage: строки 92.14%, ветви 84.08%. - git diff --check, gitleaks и commit hooks прошли; hooks не изменили проверенный diff. Сборка Storybook сообщает предупреждение о чанках больше 500 kB.

codex-bot added 1 commit 2026-09-28 20:02:50 +03:00

refactor: simplify frontend state and mutation boundaries

Frontend verify / commits (pull_request) Successful in 31s

Details

Frontend verify / security (pull_request) Successful in 11s

Details

Frontend verify / verify (pull_request) Successful in 9m1s

Details

Frontend verify / commits (push) Skipped

Details

Frontend verify / security (push) Successful in 10s

Details

Frontend verify / verify (push) Successful in 9m59s

Details

a7537da344

ksv741 scheduled this pull request to auto merge when all checks succeed 2026-09-28 20:11:38 +03:00

ksv741 merged commit a7537da344 into develop

2026-09-28 20:12:39 +03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: WealthLedger/frontend#44