Skip to content

feat(tests): preflight трьох build-артефактів перед прогоном - #507

Merged
vitaliytv merged 1 commit into
chore/limit-forks-drop-hookfrom
feat/preflight-artifacts
Aug 27, 2026
Merged

feat(tests): preflight трьох build-артефактів перед прогоном#507
vitaliytv merged 1 commit into
chore/limit-forks-drop-hookfrom
feat/preflight-artifacts

Conversation

@vitaliytv

Copy link
Copy Markdown
Member

Стоїть на #506 — обидві зміни правлять ті самі рядки vitest-конфігів. Мержити після нього.

Щоб JS-суїт давав достовірний результат, потрібні три артефакти локальної збірки, і про це ніде не було сказано. Свіжий checkout не має жодного.

артефакт без нього
rules-napi (release) loader бере старий або відсутній бінар
rules-cli (release) ~88 тестів падають із порожнім STACK_TRACE_ERROR
wasm-плагіни + builtin-pins.json оманливе DetectorError('немає main.mjs')

Ціна вже виміряна на цій сесії

#502 переніс підказку про builtin-pins.json у саме повідомлення помилки — але це лікування симптому. Через цю прогалину я двічі помилково оголосив реальні фейли «преекзистуючими», а агент згаяв ~1.5 години на пошук неіснуючого дефекту.

Рішення

  • globalSetup у обох конфігах: один раз до старту тестів, у тому самому процесі, що виконує bun run test і CI.
  • Усі відсутні одразу — «відсутні N/3» з наслідком по кожному. Інакше людина проходить три ітерації по десять хвилин; сьогодні це рівно те, що сталося з агентом, який знайшов брак rules-cli лише після того, як полагодив попереднє.
  • N_RULES_TEST_PREFLIGHT_SKIP=1 — escape-hatch для свідомо суто-JS прогону, щоб перевірка не стала перепоною там, де зайва.
  • Наявні per-test гейти лишаються. Вони спрацьовують глибоко в суїті й по одному; preflight — раніше й повним списком.

Перевірка

Зроблено особисто: з прибраним builtin-pins.json прогін падає з точним повідомленням саме про цей артефакт, а не про наслідок. Юніт-тести модуля — 10/10.

Error: test-preflight: відсутні 1/3 локальні build-артефакти …
  1. [wasm-плагіни] … відсутній — без них wasm-parity-тести падають прямим
     hard-fail, а лінт-детектори дають оманливе DetectorError('немає main.mjs')
Зберіть усе перелічене вище ОДРАЗУ, потім перезапустіть тести.

🤖 Generated with Claude Code

Щоб JS-суїт давав достовірний результат, потрібні три артефакти локальної
збірки, і про це ніде не було сказано: rules-napi (release), rules-cli
(release) і wasm-плагіни разом із gitignored builtin-pins.json. Свіжий
checkout не має жодного.

Ціна вже виміряна: відсутність wasm-плагінів давала оманливе
DetectorError('немає main.mjs') — #502 переніс підказку в саме повідомлення,
але це лікування симптому. Через це я двічі помилково оголосив реальні фейли
преекзистуючими, а агент згаяв близько півтори години на пошук неіснуючого
дефекту. Відсутність rules-cli дає порожній STACK_TRACE_ERROR, що не каже
нічого взагалі.

- `globalSetup` у обох vitest-конфігах: спрацьовує один раз до старту тестів,
  у тому самому процесі, що виконує `bun run test` і CI.
- Повідомляє про ВСІ відсутні одразу («відсутні N/3») з наслідком по кожному —
  інакше людина проходить три ітерації по десять хвилин, що сьогодні й сталося
  з агентом.
- `N_RULES_TEST_PREFLIGHT_SKIP=1` — escape-hatch для свідомо суто-JS прогону,
  щоб перевірка не стала перепоною там, де вона зайва.
- Наявні per-test гейти лишаються: вони спрацьовують глибоко в суїті й по
  одному, preflight — раніше й повним списком.

Перевірено дією: з прибраним builtin-pins.json прогін падає з точним
повідомленням про саме цей артефакт. Юніт-тести модуля — 10/10.

Гілка стоїть на #506 (ліміт форків) — обидві зміни правлять ті самі рядки
конфігів, тож розведення в незалежні гілки дало б конфлікт на рівному місці.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@vitaliytv
vitaliytv merged commit 17d0a2c into chore/limit-forks-drop-hook Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant