fix(e2e): add a globalTimeout under the 45m CI cap

[rubenvdlinde]

Part of the fleet-wide Playwright instrument sweep for ConductionNL/.github#188. Neither change can alter a verdict — both change whether you can see why a verdict happened.

trace

This repo's trace is already retain-on-failure, so only the timeout half applies here.

Reproduced with two minimal Playwright projects differing only in the trace value, same deliberately failing test, retries: 0 in both:

config	trace zips written
retries: 0 + on-first-retry	0
retries: 0 + retain-on-failure	1 (7.4 KB)

globalTimeout: 38 * 60_000

The shared quality.yml Playwright job is timeout-minutes: 45. A job cancelled by that cap yields no verdict and no artifacts — the trace upload is if: failure(), the report upload is if: always(), and neither runs on a cancelled job, while gh pr checks still renders it as "fail". Runs cancelled at ~45m16s have been observed in this fleet.

Margin, measured rather than assumed: overhead before Run Playwright tests starts is 2.0-2.4 min (openconnector run 31257480415 = 2m20s; opencatalogi, doriath, openregister in the same band); uploads after it take seconds. 38m + ~2.5m setup + uploads sits ~7 min under the cap. Verified that a fired globalTimeout exits with a tally (3 did not run / 1 passed) plus an HTML report and a trace zip on disk — exactly what a cancelled job does not give you.

Matches the value already landed in nldesign.

[github-actions]

Quality Report — ConductionNL/procest @ 797ab86

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	✅
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		✅
stylelint		✅
build		✅
check-manifest		✅
check-vue3-compile		✅
test-l10n		✅
composer			✅	✅ 100/100
npm			✅	✅ 552/552
PHPUnit					✅
Newman					⏭️
Playwright					✅
Hydra gates					✅

---

Quality workflow — 2026-08-08 13:41 UTC

Download the full PDF report from the workflow artifacts.

[rubenvdlinde]

Second commit added: this repo ships a nested Playwright config, and that is the one CI actually loads.

quality.yml (~L2218) resolves CONFIG="${playwright-test-path}/playwright.config.ts" and only falls back to the app-root playwright.config.ts when that file does not exist. Measured across the fleet: 16 of 23 repos with a Playwright config have such a nested file, so the app-root config the original issue tabulated is not the one the gate reads in any of them.

Both files are now fixed — the app-root one (what a developer gets from npx playwright test at the repo root) and the nested one (what CI runs).

[github-actions]

Quality Report — ConductionNL/procest @ b49b24e

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	✅
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		✅
stylelint		✅
build		✅
check-manifest		✅
check-vue3-compile		✅
test-l10n		✅
composer			✅	✅ 100/100
npm			✅	✅ 552/552
PHPUnit					✅
Newman					⏭️
Playwright					❌
Hydra gates					❌

---

Quality workflow — 2026-08-08 14:25 UTC

Download the full PDF report from the workflow artifacts.