ci(#3274): гейт против возврата фронта в общую команду и снятия ретрая (часть 2b/3) #3447

Merged

bot-backend merged 1 commit from fix/3274-part2b-gate into main

2026-09-11 20:56:41 +00:00

bot-backend commented

2026-09-11 20:29:00 +00:00

Collaborator

Часть 2b из 3 по #3274. Мержить ПОСЛЕ частей 1 (#3442, в main) и 2a (#3446) — гейт проверяет обе половины разом и на main без них покраснеет.

Только scripts/check-frontend-swap-window.py + шаг в .forgejo/workflows/ci.yml: по фильтрам ни deploy.yml (backend/** frontend/** caddy/** Caddyfile docker-compose.prod.yml data/sql/** auth/** ops/*), ни deploy-tradein.yml (tradein-mvp/** .forgejo/workflows/deploy-tradein.yml) эти пути не матчат — мерж не запускает ни одного деплоя.

Что стережёт. Обе половины правки живут в разных файлах и обе легко отменяются случайно:

frontend возвращается в общий SERVICES воркфлоу (окно 30–90 с вернётся молча);
новый публичный путь копируют с соседнего блока Caddy без import tradein_frontend_retry (остаток 0,5 с вернётся на этом пути).

Гейт фальсифицирован мутациями (deep-ревью, не только --selftest): возврат frontend в SERVICES — ловит; SERVICES="$SERVICES frontend" — ловит; фронт дописан прямо в команду пачки — ловит; import закомментирован — ловит; снипет без lb_try_duration — ловит. Комментарий с той же формулировкой гейт НЕ роняет (проверено отдельно — иначе объяснение дефекта в коде ломало бы проверку, это уже случалось в репозитории).

Известная дыра гейта, оставлена осознанно: check-frontend-swap-window.py:52 требует открывающей скобки у блока, поэтому однострочный reverse_proxy tradein-frontend:3000 без { пройдёт мимо. Сегодня все 9 блоков со скобкой, так что это про будущее; закрывается матчем без { с явной ошибкой. Записано здесь, чтобы не выглядело проверенным.

Часть 2b из 3 по #3274. **Мержить ПОСЛЕ частей 1 (#3442, в main) и 2a (#3446)** — гейт проверяет обе половины разом и на main без них покраснеет. Только `scripts/check-frontend-swap-window.py` + шаг в `.forgejo/workflows/ci.yml`: по фильтрам ни `deploy.yml` (`backend/** frontend/** caddy/** Caddyfile docker-compose.prod.yml data/sql/** auth/** ops/*`), ни `deploy-tradein.yml` (`tradein-mvp/** .forgejo/workflows/deploy-tradein.yml`) эти пути не матчат — мерж не запускает ни одного деплоя. **Что стережёт.** Обе половины правки живут в разных файлах и обе легко отменяются случайно: - `frontend` возвращается в общий `SERVICES` воркфлоу (окно 30–90 с вернётся молча); - новый публичный путь копируют с соседнего блока Caddy без `import tradein_frontend_retry` (остаток 0,5 с вернётся на этом пути). **Гейт фальсифицирован мутациями** (deep-ревью, не только `--selftest`): возврат `frontend` в `SERVICES` — ловит; `SERVICES="$SERVICES frontend"` — ловит; фронт дописан прямо в команду пачки — ловит; `import` закомментирован — ловит; снипет без `lb_try_duration` — ловит. Комментарий с той же формулировкой гейт НЕ роняет (проверено отдельно — иначе объяснение дефекта в коде ломало бы проверку, это уже случалось в репозитории). **Известная дыра гейта, оставлена осознанно:** `check-frontend-swap-window.py:52` требует открывающей скобки у блока, поэтому однострочный `reverse_proxy tradein-frontend:3000` без `{` пройдёт мимо. Сегодня все 9 блоков со скобкой, так что это про будущее; закрывается матчем без `{` с явной ошибкой. Записано здесь, чтобы не выглядело проверенным.

Rows
Columns