Реализует issue #25: добавлена source_position, импорт сохраняет индекс операции в исходном массиве, история использует стабильные tie-breaker для одинаковых дат и безопасный backfill старых записей.
Версия backend 0.9.0.
Реализует issue #25: добавлена source_position, импорт сохраняет индекс операции в исходном массиве, история использует стабильные tie-breaker для одинаковых дат и безопасный backfill старых записей.
Версия backend 0.9.0.
Review: перед слиянием нужны тесты. PR меняет критичную для импорта/истории логику, но не добавляет проверок. Нужен минимальный integration/unit test: импорт нескольких операций с одинаковым operationAt без времени; source_position 0/1/2; порядок date DESC (source order), date ASC (обратный); детерминированный tie-breaker при одинаковых датах/позициях; отдельная проверка backfill старых строк. По коду сортировка соответствует критериям issue #25, замечаний по amount sort не вижу. API не позволяет текущему аккаунту оставить formal REQUEST_CHANGES на собственный PR, поэтому оставляю review comment здесь.
Review: перед слиянием нужны тесты. PR меняет критичную для импорта/истории логику, но не добавляет проверок. Нужен минимальный integration/unit test: импорт нескольких операций с одинаковым operationAt без времени; source_position 0/1/2; порядок date DESC (source order), date ASC (обратный); детерминированный tie-breaker при одинаковых датах/позициях; отдельная проверка backfill старых строк. По коду сортировка соответствует критериям issue #25, замечаний по amount sort не вижу. API не позволяет текущему аккаунту оставить formal REQUEST_CHANGES на собственный PR, поэтому оставляю review comment здесь.
Повторный review: добавленный test:transactions покрывает только генерацию SQL ORDER BY (date asc/desc и amount), это полезно, но не проверяет два ключевых пункта issue #25: что importStatement сохраняет source_position 0/1/2 и что migration backfill корректно заполняет старые строки. Формальный REQUEST_CHANGES текущим аккаунтом невозможен (автор PR). Рекомендую добавить хотя бы runnable check для import mapping/source position или явно зафиксировать интеграционное покрытие; после этого логика сортировки и версия/changelog выглядят корректно.
Повторный review: добавленный `test:transactions` покрывает только генерацию SQL ORDER BY (date asc/desc и amount), это полезно, но не проверяет два ключевых пункта issue #25: что importStatement сохраняет source_position 0/1/2 и что migration backfill корректно заполняет старые строки. Формальный REQUEST_CHANGES текущим аккаунтом невозможен (автор PR). Рекомендую добавить хотя бы runnable check для import mapping/source position или явно зафиксировать интеграционное покрытие; после этого логика сортировки и версия/changelog выглядят корректно.
Повторный review завершён: блокирующих замечаний не осталось. Import integration test теперь проверяет сохранение source_position [0,1] для операций с одинаковой датой; helper test покрывает date ASC/DESC и amount tie-breakers. Migration/backfill SQL детерминированный, версия 0.9.2 и changelog обновлены. Backend build проходит. Формальный review API недоступен автору собственного PR, поэтому фиксирую результат комментарием.
Повторный review завершён: блокирующих замечаний не осталось. Import integration test теперь проверяет сохранение source_position [0,1] для операций с одинаковой датой; helper test покрывает date ASC/DESC и amount tie-breakers. Migration/backfill SQL детерминированный, версия 0.9.2 и changelog обновлены. Backend build проходит. Формальный review API недоступен автору собственного PR, поэтому фиксирую результат комментарием.
admin
merged commit efc9854064 into main2026-08-20 20:58:27 +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 #25: добавлена source_position, импорт сохраняет индекс операции в исходном массиве, история использует стабильные tie-breaker для одинаковых дат и безопасный backfill старых записей.
Версия backend 0.9.0.
Review: перед слиянием нужны тесты. PR меняет критичную для импорта/истории логику, но не добавляет проверок. Нужен минимальный integration/unit test: импорт нескольких операций с одинаковым operationAt без времени; source_position 0/1/2; порядок date DESC (source order), date ASC (обратный); детерминированный tie-breaker при одинаковых датах/позициях; отдельная проверка backfill старых строк. По коду сортировка соответствует критериям issue #25, замечаний по amount sort не вижу. API не позволяет текущему аккаунту оставить formal REQUEST_CHANGES на собственный PR, поэтому оставляю review comment здесь.
Повторный review: добавленный
test:transactionsпокрывает только генерацию SQL ORDER BY (date asc/desc и amount), это полезно, но не проверяет два ключевых пункта issue #25: что importStatement сохраняет source_position 0/1/2 и что migration backfill корректно заполняет старые строки. Формальный REQUEST_CHANGES текущим аккаунтом невозможен (автор PR). Рекомендую добавить хотя бы runnable check для import mapping/source position или явно зафиксировать интеграционное покрытие; после этого логика сортировки и версия/changelog выглядят корректно.Повторный review завершён: блокирующих замечаний не осталось. Import integration test теперь проверяет сохранение source_position [0,1] для операций с одинаковой датой; helper test покрывает date ASC/DESC и amount tie-breakers. Migration/backfill SQL детерминированный, версия 0.9.2 и changelog обновлены. Backend build проходит. Формальный review API недоступен автору собственного PR, поэтому фиксирую результат комментарием.