fix(build): отрицательные сканы приёмки проверяют форму кода, а не прозу
Комментарий, объясняющий, почему чего-то больше нет, обязан называть это по
имени. Скан по голой подстроке такой комментарий от кода не отличает и падает
на документации к выполненной им же работе. Найдено три таких гейта, все на
пути ближайшей сборки:
скан иконок -> блочный комментарий в SvgIcon/sprite.ts
скан имён раннеров -> слово `systemd-run` в прозе firewall.ts
скан cancelFirewall... -> комментарий о разделении функции
Второй сломан моим же комментарием из 2259f7c. Третий сломан с момента своего
появления (330a63b) и не падал только потому, что сборка до него не доходила:
её останавливали более ранние гейты.
Исправления по форме, а не удалением комментариев:
- скан прежних имён раннеров требует, чтобы перед именем не стоял дефис. В
JavaScript идентификатор после дефиса не начинается, поэтому исключение
точное и ни один настоящий вызов не пропускает;
- скан cancelFirewallRollback ищет имя со скобкой, то есть объявление или
вызов, а не упоминание;
- литеральный скан по virtual:svg-icons-register удалён: его роль исполняет
более сильный и более ранний гейт — production `vite build`, где активный
импорт неразрешимого виртуального модуля роняет сборку bundle. Оставлены две
точные проверки: отсутствие плагина в package.json и вызов registerSvgIcons
в main.ts.
В шапке acceptance.sh зафиксировано правило для отрицательных сканов и
ограничение code_without_comments: строчные комментарии он отбрасывает,
блочные — нет, и блок-парсер сознательно не заводится (наивный стриппер
спотыкается о `/*` внутри строк и регулярных выражений, а это ложный PASS).
Все отрицательные сканы приёмки прогнаны по текущему дереву: срабатываний
больше нет; новые шаблоны проверены на синтетическом регрессе — ловят.
This commit is contained in:
@@ -8,10 +8,29 @@ set -euo pipefail
|
||||
# — и проверка, ищущая просто подстроку, начинает падать ровно на той
|
||||
# документации, которая описывает выполненную ею же работу.
|
||||
#
|
||||
# Урок в этом файле уже дважды оплачен: скан versions contract ловил сам себя на
|
||||
# строке '/hui', а dead-route скан падал на router_test.go, который ПЕРЕЧИСЛЯЕТ
|
||||
# удалённые маршруты, чтобы доказать их отсутствие. Помощник существует, чтобы
|
||||
# третьего раза не было.
|
||||
# Урок в этом файле оплачен уже пять раз: скан versions contract ловил сам себя
|
||||
# на строке '/hui'; dead-route скан падал на router_test.go, который ПЕРЕЧИСЛЯЕТ
|
||||
# удалённые маршруты, чтобы доказать их отсутствие; скан иконок падал на
|
||||
# блочном комментарии, объясняющем замену плагина; скан прежних имён раннеров —
|
||||
# на слове `systemd-run` в прозе; скан `cancelFirewallRollback` — на
|
||||
# комментарии, описывающем её разделение.
|
||||
#
|
||||
# Отсюда правило для КАЖДОЙ отрицательной проверки по дереву исходников:
|
||||
#
|
||||
# формулируй синтаксическую форму, а не подстроку.
|
||||
#
|
||||
# Вызов — это имя со скобкой или с обратной кавычкой; импорт — это `import` со
|
||||
# спецификатором; зависимость — это ключ в package.json. Прозаическое
|
||||
# упоминание удалённой вещи обязано быть разрешено: комментарий, объясняющий,
|
||||
# почему чего-то больше нет, называет это по имени — иначе он бесполезен.
|
||||
#
|
||||
# Ограничение помощника, о котором важно помнить: он отбрасывает СТРОЧНЫЕ
|
||||
# комментарии (`//`, `#`), но НЕ блочные `/* ... */`. Блок-парсер здесь
|
||||
# сознательно не заводится: наивный стриппер спотыкается о `/*` внутри строк и
|
||||
# регулярных выражений и может вычистить настоящий код, а это ложный PASS —
|
||||
# отказ хуже болезни. Для файлов с блочными комментариями формулируйте
|
||||
# синтаксическую форму либо опирайтесь на более сильный гейт (сборка bundle,
|
||||
# компилятор, тесты).
|
||||
#
|
||||
# Вид комментария выбирается по расширению файла.
|
||||
#
|
||||
@@ -260,10 +279,21 @@ run_fix20_acceptance_subset() {
|
||||
# `Patched versions: <0.0.0` — исправленной версии не существует.
|
||||
! grep -q '"vite-plugin-svg-icons"' apps/frontend/package.json \
|
||||
|| fail "acceptance: неподдерживаемый vite-plugin-svg-icons вернулся в зависимости"
|
||||
! grep -rq 'virtual:svg-icons-register' apps/frontend/src \
|
||||
|| fail "acceptance: виртуальный модуль удалённого плагина иконок вернулся"
|
||||
grep -q 'registerSvgIcons' apps/frontend/src/main.ts \
|
||||
|| fail "acceptance: спрайт локальных иконок не регистрируется при старте приложения"
|
||||
# Литерального скана по `virtual:svg-icons-register` здесь СОЗНАТЕЛЬНО нет.
|
||||
#
|
||||
# Он ловил не код, а прозу: единственное вхождение строки в дереве — в
|
||||
# блочном комментарии `components/SvgIcon/sprite.ts`, который объясняет, что
|
||||
# именно заменено. Комментарий, называющий удалённую вещь по имени, обязан
|
||||
# существовать — иначе он бесполезен, — и гейт, падающий на нём, запрещает
|
||||
# документировать собственную работу.
|
||||
#
|
||||
# Утверждение при этом не потеряно, а обеспечено более сильной проверкой:
|
||||
# production `vite build` выше по сборке уже прошёл. Активный
|
||||
# `import "virtual:svg-icons-register"` при отсутствующем плагине не
|
||||
# разрешается резолвером, и сборка bundle падает — то есть проверяется
|
||||
# исполняемый импорт, а не совпадение подстроки.
|
||||
|
||||
# Каждая иконка обязана давать symbol с viewBox.
|
||||
#
|
||||
@@ -410,7 +440,15 @@ run_clean_install_acceptance() {
|
||||
|| fail "acceptance: process.ts must expose an explicit read-only runner"
|
||||
grep -q 'export async function runMutating' orchestrator/src/lib/process.ts \
|
||||
|| fail "acceptance: process.ts must expose an explicit mutating runner"
|
||||
! grep -rEq '(^|[^A-Za-z0-9_])(run|runVisible|runHidden|runSecret|runRawVisible)`' orchestrator/src \
|
||||
# Дефис в списке исключённых предшествующих символов — не косметика.
|
||||
#
|
||||
# Скан ищет ВЫЗОВ прежнего раннера: имя, за которым сразу идёт обратная
|
||||
# кавычка tagged template. Без дефиса под это описание попадала любая проза
|
||||
# вида `systemd-run` — имя команды, взятое в кавычки в комментарии, — и гейт
|
||||
# падал на строке документации, не содержащей никакого вызова. В JavaScript
|
||||
# идентификатор не может начинаться после дефиса, поэтому исключение точное:
|
||||
# ни один настоящий вызов оно не пропускает.
|
||||
! grep -rEq '(^|[^-A-Za-z0-9_])(run|runVisible|runHidden|runSecret|runRawVisible)`' orchestrator/src \
|
||||
|| fail "acceptance: the pre-split runner names must not come back"
|
||||
# Порог считается от числа мутирующих раннеров, а не задан константой: новый
|
||||
# раннер обязан приносить с собой и проверку guard'а, а не проходить под
|
||||
@@ -1081,7 +1119,13 @@ run_single_owner_acceptance() {
|
||||
|| fail "acceptance: снятие таймера автоотката не отделено от удаления резервных копий"
|
||||
grep -q 'export async function cleanupFirewallRollback' orchestrator/src/steps/firewall.ts \
|
||||
|| fail "acceptance: удаление резервных копий firewall не выделено в отдельную операцию"
|
||||
! grep -rq 'cancelFirewallRollback' orchestrator/src \
|
||||
# Ищется ОБЪЯВЛЕНИЕ или ВЫЗОВ, то есть имя со скобкой, а не упоминание.
|
||||
#
|
||||
# Скан по голой подстроке был сломан с самого своего появления: комментарий у
|
||||
# `disarmFirewallRollback` объясняет разделение, называя прежнюю функцию по
|
||||
# имени, и гейт падал бы на нём. Не падал он только потому, что до этого шага
|
||||
# сборка не доходила — более ранние гейты останавливали её раньше.
|
||||
! grep -rEq 'cancelFirewallRollback[[:space:]]*\(' orchestrator/src \
|
||||
|| fail "acceptance: объединённая cancelFirewallRollback вернулась; она удаляла копии до фиксации успеха"
|
||||
"$BUN_BIN" -e '
|
||||
const fs = require("node:fs");
|
||||
|
||||
Reference in New Issue
Block a user