From 61c401b810cf7e5294c504cb591abff09c17b33e Mon Sep 17 00:00:00 2001 From: Matysh Date: Sat, 22 Aug 2026 18:33:45 +0300 Subject: [PATCH] ci: derive the browser-smoke shortlist from the diff Issue: #241 User-Visible: no --- .github/workflows/process.yml | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/.github/workflows/process.yml b/.github/workflows/process.yml index 5469af2a..93bfc1d4 100644 --- a/.github/workflows/process.yml +++ b/.github/workflows/process.yml @@ -360,13 +360,19 @@ jobs: с красным job `docs` до следующей задачи (#237). По необходимости, и «необходимость» определяется diff'ом и AC: - - браузерные смоки `demo/smoke_*.mjs` — названные в AC плюс - относящиеся к тронутым поверхностям. Их 163; прогон всех уместен - только когда задача действительно задевает всё. Выбирать по теме - недостаточно: регресс #234 поймал `smoke_wall_junctions`, который - по названию про стыки стен, а не про толщину отрезка. Надёжнее - grep — искать в `demo/smoke_*.mjs` имена изменённых функций и - полей, и прогонять найденное; + - браузерные смоки `demo/smoke_*.mjs` — названные в AC плюс те, + что печатает `node scripts/smoke-select.mjs --base --head `. + Их 167; прогон всех уместен только когда задача действительно + задевает всё. Выбирать по теме недостаточно: регресс #234 поймал + `smoke_wall_junctions`, который по названию про стыки стен, а не + про толщину отрезка. Инструмент печатает три вида ответа, и они + разные: «прямое совпадение» — смок называет изменённый символ, + «зарегистрированная связь» — смок проверяет следствие контракта, + не называя его, «НЕОПРЕДЕЛЁННОСТЬ» — связь не доказана, и это не + разрешение ничего не прогонять. Вывод инструмента прикладывается + к комментарию ревью вместе с решением по каждой строке: прогнал + либо не прогнал и почему. Слабые связи (одно распространённое + имя) — повод посмотреть, а не обязанность прогонять; - `npm run golden:verify` — если diff может изменить видимый результат: рендер, геометрия, стили, слои; - `python -m pytest tests_backend -q` — если тронут