Применять правило ко всем похожим операциям #32

agent · 2026-08-20T06:18:29Z

agent commented

2026-08-20 06:18:29 +00:00

Реализует issue #22: создание правила атомарно применяет его ко всем подходящим некатегоризированным или неподтвержденным операциям и возвращает поле applied с количеством изменений. Подтвержденные ручные категории не перезаписываются.

Версии: backend 0.7.5, shared 0.3.1.

Реализует issue #22: создание правила атомарно применяет его ко всем подходящим некатегоризированным или неподтвержденным операциям и возвращает поле applied с количеством изменений. Подтвержденные ручные категории не перезаписываются. Версии: backend 0.7.5, shared 0.3.1.

agent added 1 commit 2026-08-20 06:18:29 +00:00

feat: apply category rules on creation 6515e73ead

agent commented

2026-08-20 06:19:32 +00:00

Review PR #32: есть одно блокирующее замечание — отсутствуют тесты для новой критичной логики createRule: атомарность (ошибка применения не должна оставлять правило), количество applied, применение только к некатегоризированным/неподтверждённым операциям и сохранение подтверждённых ручных категорий. В текущем наборе тестов нет покрытия categoryRules, поэтому регрессия в SQL/транзакции останется незамеченной. Добавьте минимальный интеграционный тест (или изолированный тест с проверяемым SQL) и повторите review. Остальная реализация по чтению diff выглядит совместимой: условие UPDATE не перезаписывает подтверждённые категории, COMMIT/ROLLBACK обёрнуты в одной транзакции.

Review PR #32: есть одно блокирующее замечание — отсутствуют тесты для новой критичной логики `createRule`: атомарность (ошибка применения не должна оставлять правило), количество `applied`, применение только к некатегоризированным/неподтверждённым операциям и сохранение подтверждённых ручных категорий. В текущем наборе тестов нет покрытия categoryRules, поэтому регрессия в SQL/транзакции останется незамеченной. Добавьте минимальный интеграционный тест (или изолированный тест с проверяемым SQL) и повторите review. Остальная реализация по чтению diff выглядит совместимой: условие UPDATE не перезаписывает подтверждённые категории, COMMIT/ROLLBACK обёрнуты в одной транзакции.

admin merged commit 0003d92583 into main

2026-08-20 06:25:50 +00:00

admin referenced this issue from a commit

2026-08-20 06:25:51 +00:00

Merge pull request 'Применять правило ко всем похожим операциям' (#32) from feature/apply-category-rule 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#32