From 735b98c8ca7c27a870252309fad5bffdd3b4f38c Mon Sep 17 00:00:00 2001 From: Matysh Date: Sun, 23 Aug 2026 09:41:12 +0300 Subject: [PATCH] ci: name the model invariants in the review gate set Issue: #254 User-Visible: no --- .github/workflows/process.yml | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/.github/workflows/process.yml b/.github/workflows/process.yml index 93bfc1d4..5740f77a 100644 --- a/.github/workflows/process.yml +++ b/.github/workflows/process.yml @@ -359,6 +359,16 @@ jobs: выбирать тут нечего. Пропуск этого шага в #230 и #234 оставил `dev` с красным job `docs` до следующей задачи (#237). + Если diff трогает геометрию или ссылки на неё — рёбра комнат, + записи толщины, `layout`, `marker.space`, `open_spans` — обязательны + инварианты модели (#254): `npm test` уже гоняет их на всех моделях + проекта, а на конкретной конфигурации они проверяются командой + `npm run invariants -- --config <экспорт или ответ config/get>`. + Два вопроса, на которые они отвечают, и оба уже стоили продукту + дефектов: не исчезла ли запись толщины (#253) и разрешима ли каждая + ссылка (#244, #252). Если задача меняет геометрию, а инварианты в + отчёте не названы — это непрогнанный гейт, а не мелочь. + По необходимости, и «необходимость» определяется diff'ом и AC: - браузерные смоки `demo/smoke_*.mjs` — названные в AC плюс те, что печатает `node scripts/smoke-select.mjs --base --head `.