Реализует issue #22: создание правила атомарно применяет его ко всем подходящим некатегоризированным или неподтвержденным операциям и возвращает поле applied с количеством изменений. Подтвержденные ручные категории не перезаписываются.
Версии: backend 0.7.5, shared 0.3.1.
Реализует issue #22: создание правила атомарно применяет его ко всем подходящим некатегоризированным или неподтвержденным операциям и возвращает поле applied с количеством изменений. Подтвержденные ручные категории не перезаписываются.
Версии: backend 0.7.5, shared 0.3.1.
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 main2026-08-20 06:25:50 +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.
Реализует issue #22: создание правила атомарно применяет его ко всем подходящим некатегоризированным или неподтвержденным операциям и возвращает поле applied с количеством изменений. Подтвержденные ручные категории не перезаписываются.
Версии: backend 0.7.5, shared 0.3.1.
Review PR #32: есть одно блокирующее замечание — отсутствуют тесты для новой критичной логики
createRule: атомарность (ошибка применения не должна оставлять правило), количествоapplied, применение только к некатегоризированным/неподтверждённым операциям и сохранение подтверждённых ручных категорий. В текущем наборе тестов нет покрытия categoryRules, поэтому регрессия в SQL/транзакции останется незамеченной. Добавьте минимальный интеграционный тест (или изолированный тест с проверяемым SQL) и повторите review. Остальная реализация по чтению diff выглядит совместимой: условие UPDATE не перезаписывает подтверждённые категории, COMMIT/ROLLBACK обёрнуты в одной транзакции.