mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-29 03:09:36 +00:00
Matysh
65339f63ad
#387 закрыл классификацию — какие job запускать. Здесь остаток того же дефекта: гейты, которые судят сам диапазон коммитов. Провенанс, процессный гейт и «новый код не добавляет any» брали диапазон от головы предыдущего пуша, а concurrency отменяет прогон предыдущего пуша штатно. Тогда его коммиты не судит никто: свой прогон отменён, а следующий пуш сравнивает уже с ними. Окно не закрывается никогда. Уязвим был прямой пуш в dev — основной режим конвейера. На ветках дефекта нет: no-new-any там всегда считает от merge-base, а resolveValidationRange подменяет осиротевший before на origin/dev (#315). База стала последним предком с успешно завершённым Validate. Фолбэк, когда такого нет, сознательно оставлен прежним — before, но с пометкой в summary «диапазон недоказуем». Расширять диапазон здесь нельзя: гейт, который сам красит прогон, лишил бы следующий пуш зелёного предка и запер dev в красноте навсегда. Фолбэк обязан не зависеть от собственного успеха гейта. Дыра сужается с «всегда, когда прогон предыдущего пуша отменён» до «когда во всём окне обхода нет ни одного успешного прогона». Находки no-new-any теперь называют коммит, добавивший строку: диапазон стал шире, и без имени источника сообщение обвиняло бы того, кто пушнул следующим, — ровно то, что чинили в #386 для golden. Issue: #388 User-Visible: no