ci: синхронизировать process.yml с dev
Полные бенчмарки производительности / Бенчмарки рендера и геометрии (blend) (push) Failing after 26m23s
Полные бенчмарки производительности / Бенчмарки рендера и геометрии (large-house) (push) Failing after 38m27s
Проверка (CI) / Документация: гайды, ченджлоги, скриншот-индекс (push) Successful in 32s
Проверка (CI) / Процесс: process.yml идентичен в main и dev (push) Successful in 1m2s
Проверка (CI) / Провенанс коммитов: трейлеры и эталоны (push) Successful in 1m14s
Проверка (CI) / Процессный гейт: диапазон, трейлеры, статусы issue (push) Failing after 1m23s
Проверка (CI) / Классификация изменённых файлов (push) Successful in 1m12s
Проверка (CI) / HACS: валидация репозитория (push) Skipped
Проверка (CI) / Hassfest: манифест интеграции (push) Skipped
Проверка (CI) / Фронтенд: типы, юниты, мутанты, синхрон бандла (push) Skipped
Проверка (CI) / Переиспользование: это дерево уже проверено (push) Successful in 56s
Проверка (CI) / Смоки в браузере (шард 1 из 3) (push) Skipped
Проверка (CI) / Смоки в браузере (шард 2 из 3) (push) Skipped
Проверка (CI) / Смоки в браузере (шард 3 из 3) (push) Skipped
Проверка (CI) / Смоки: все шарды зелёные (push) Skipped
Проверка (CI) / Golden-кадры против принятых эталонов (push) Skipped
Проверка (CI) / Перф-смок: бюджет времени кадра (push) Skipped
Проверка (CI) / Бэкенд: pytest в Home Assistant (push) Skipped
Полные бенчмарки производительности / Бенчмарки рендера и геометрии (overlay) (push) Failing after 44m51s
Полные бенчмарки производительности / Бенчмарки рендера и геометрии (isometric) (push) Failing after 45m32s
Полные бенчмарки производительности / Бенчмарки рендера и геометрии (plan-snap) (push) Failing after 29m56s

Конвейер исполняет версию из ветки по умолчанию, поэтому файл обязан совпадать
в main и dev побайтово. Содержательная правка сделана в dev (#365), здесь копия.

Issue: #365
User-Visible: no
This commit is contained in:
Claude
2026-08-29 10:38:57 +03:00
parent d2f4f94159
commit 88dc61dd40
+39 -12
View File
@@ -642,14 +642,31 @@ jobs:
marker=CODE-REVIEW
if [ "$STAGE" = "spec" ]; then marker=SPEC-REVIEW; fi
doc="docs/reviews/${marker}-${NUM}-r${CYCLE}.md"
# Рабочая копия отбрасывается ДО того, как документ попадёт в дерево:
# ревьюер правит код, проверяя «умеет ли тест падать», и его правки
# публиковаться не должны.
git checkout -- . 2>/dev/null || true
# docs/reviews исключён из уборки: ревьюер мог написать документ по
# старому пути, и клин не должен его съесть до `git add` — ровно так
# оба пути остаются работоспособными.
git clean -fd -e docs/reviews -e node_modules >/dev/null 2>&1 || true
# Документ спасается ПЕРВЫМ делом. Ревьюер мог написать его по старому
# пути прямо в рабочую копию, а дальше эта копия будет отброшена
# целиком — и вместе с ней пропал бы артефакт (#220).
if [ ! -f "$SOURCE" ] && [ -f "$doc" ]; then
cp "$doc" "$SOURCE"
echo "документ найден в рабочей копии и сохранён в $SOURCE"
fi
# Reset, а не checkout+clean, и вот почему (#365).
#
# 28.08 коммит bb2919f уехал в dev с тридцатью файлами вместо одного
# markdown: откатил отревьюженную реализацию #359, вернул старые чанки
# и держал dev откаченным три часа. Механизм воспроизведён:
# `git checkout -- .` восстанавливает рабочее дерево ИЗ ИНДЕКСА, а
# `git clean -fd` убирает неотслеживаемое — ни то, ни другое индекс не
# трогает. Ревьюер работает с Bash и в ходе проверки «умеет ли тест
# падать» вполне может сделать `git add`; всё, что осталось у него в
# индексе, прежняя уборка сохраняла, и следующий же `git commit`
# забирал это вместе с документом. Сообщение при этом невинное, и от
# рутины инцидент отличается только диффом.
#
# `reset --hard` снимает и индекс, и дерево разом. Терять нечего:
# документ приезжает извне репозитория, из RUNNER_TEMP.
git fetch -q origin "$target"
git reset -q --hard "origin/$target"
git clean -fdq -e node_modules >/dev/null 2>&1 || true
# Документ приезжает извне репозитория (#220). Три раунда подряд он
# терялся, пока лежал некоммитнутым файлом в том же дереве, которое
# ревьюер мутирует и затем восстанавливает: `git checkout -- .` плюс
@@ -661,11 +678,12 @@ jobs:
cp "$SOURCE" "$doc"
echo "документ взят из $SOURCE ($(wc -c < "$doc") байт)"
else
# Совместимость: ревьюер мог написать по старому пути, если промпт
# ещё не обновился в этой ветке.
echo "::warning::$SOURCE не найден — ищу документ в рабочей копии"
echo "::warning::$SOURCE не найден — документа для публикации нет"
fi
git add docs/reviews 2>/dev/null || true
# Индексируется ровно один путь, а не каталог: `git add docs/reviews`
# забрал бы всё, что там окажется, а после reset там не должно быть
# ничего постороннего — но полагаться на «не должно» здесь нельзя.
git add -- "$doc" 2>/dev/null || true
if git diff --cached --quiet; then
# Пустая рабочая копия — ещё не провал: ревьюер иногда коммитит
# документ сам, своим app-токеном мимо этого шага (CODE-REVIEW-150-r1,
@@ -683,6 +701,8 @@ jobs:
echo "::error::вердикт есть, а документа нет: ни $SOURCE, ни $doc в рабочей копии, ни $doc в $target — ревью без артефакта (#171, #220)"
exit 1
fi
# Первый рубеж: что вообще проиндексировано.
git diff --cached --name-only | node scripts/review-doc-guard.mjs
git -c user.name="claude[bot]" \
-c user.email="209825114+claude[bot]@users.noreply.github.com" \
commit -q -F - <<EOF
@@ -691,6 +711,10 @@ jobs:
Issue: #$NUM
User-Visible: no
EOF
# Второй рубеж, и он главный: что пуш ДОБАВИТ в целевую ветку. Первый
# судит намерение шага, этот — результат, а расходились они именно
# тогда, когда база оказывалась не той.
git diff --name-only "origin/$target...HEAD" | node scripts/review-doc-guard.mjs
# Публикация в dev идёт из детачнутого состояния поверх ветки задачи
# либо dev, поэтому push нужен с явным перебазированием при гонке:
# dev мог уйти вперёд, пока шло ревью — оно длится до 45 минут.
@@ -705,6 +729,9 @@ jobs:
echo "::error::документ ревью не удалось опубликовать в $target: конфликт (#171)"
exit 1
fi
# После ребейза набор путей другой — проверяется заново. Форс здесь
# запрещён и не появляется: ветка двигается только вперёд.
git diff --name-only "origin/$target...HEAD" | node scripts/review-doc-guard.mjs
git push -q "https://x-access-token:$TOKEN@github.com/${{ github.repository }}" \
"HEAD:$target"
fi