CI #44: retain real-stack Playwright failure diagnostics #45

Closed

lost-rob0t wants to merge 5 commits from rage/44-playwright-failure-artifacts into main

lost-rob0t commented

2026-08-25 08:18:43 +00:00

(Migrated from github.com)

Fixes the diagnostic gap in #44 without weakening or skipping the real-stack E2E gate.

Current main CI #718 and Auto-Dig lifecycle PR #43 CI #740 both fail at the same Real-stack desktop, mobile, mutation, and reload paths step, but the workflow retains no Playwright report/trace artifact. That prevents root-cause diagnosis from authoritative CI evidence.

This PR:

keeps the GitHub reporter;
adds Playwright HTML reporting in CI;
uploads frontend/playwright-report/ and retry frontend/test-results/ only when CI has failed;
retains diagnostics for 7 days;
does not make the E2E step optional, allowed-to-fail, skipped, or retried outside Playwright's existing policy.

The expected first run may remain RED because this branch is based directly on the known-red main; the point of that run is to retain the exact failing spec/assertion/trace so #44 can be fixed TDD-first at its owning UI/E2E boundary.

Fixes the diagnostic gap in #44 without weakening or skipping the real-stack E2E gate. Current `main` CI #718 and Auto-Dig lifecycle PR #43 CI #740 both fail at the same `Real-stack desktop, mobile, mutation, and reload paths` step, but the workflow retains no Playwright report/trace artifact. That prevents root-cause diagnosis from authoritative CI evidence. This PR: - keeps the GitHub reporter; - adds Playwright HTML reporting in CI; - uploads `frontend/playwright-report/` and retry `frontend/test-results/` only when CI has failed; - retains diagnostics for 7 days; - does not make the E2E step optional, allowed-to-fail, skipped, or retried outside Playwright's existing policy. The expected first run may remain RED because this branch is based directly on the known-red `main`; the point of that run is to retain the exact failing spec/assertion/trace so #44 can be fixed TDD-first at its owning UI/E2E boundary.

lost-rob0t commented

2026-08-25 09:11:05 +00:00

(Migrated from github.com)

Automation continuation on #44 diagnostics:

CI #744 on ee85f2936b289db556468da36be7458dd90bcdef never reached Playwright; npm run check failed because the diagnostic edit made frontend/playwright.config.ts non-Prettier-compliant.
A first formatting-only correction (d96477803a0e07a4504ea581be2d9de38a8626b1) still failed the same formatter, so the diagnostic approach was simplified rather than expanding config churn.
Current head aaf15631ebe5d256f6a3bd182b46931a08eb9f4f restores frontend/playwright.config.ts byte-for-byte to main and enables CI-only github,html reporters at the workflow command (npm run test:e2e -- --reporter=github,html) with PLAYWRIGHT_HTML_OPEN=never.
The failure-artifact upload remains blocking-neutral: it runs only after failure and does not skip/quarantine/allow-failure the E2E gate.

Await exact-head CI on aaf15631…; do not merge #45 or mutate #43 until this branch reaches the baseline Playwright failure and retains the report/trace evidence.

Automation continuation on #44 diagnostics: - CI #744 on `ee85f2936b289db556468da36be7458dd90bcdef` never reached Playwright; `npm run check` failed because the diagnostic edit made `frontend/playwright.config.ts` non-Prettier-compliant. - A first formatting-only correction (`d96477803a0e07a4504ea581be2d9de38a8626b1`) still failed the same formatter, so the diagnostic approach was simplified rather than expanding config churn. - Current head `aaf15631ebe5d256f6a3bd182b46931a08eb9f4f` restores `frontend/playwright.config.ts` byte-for-byte to `main` and enables CI-only `github,html` reporters at the workflow command (`npm run test:e2e -- --reporter=github,html`) with `PLAYWRIGHT_HTML_OPEN=never`. - The failure-artifact upload remains blocking-neutral: it runs only after failure and does not skip/quarantine/allow-failure the E2E gate. Await exact-head CI on `aaf15631…`; do not merge #45 or mutate #43 until this branch reaches the baseline Playwright failure and retains the report/trace evidence.

nsaspy changed title from ~~WIP: CI #44: retain real-stack Playwright failure diagnostics~~ to CI #44: retain real-stack Playwright failure diagnostics

2026-08-27 19:50:34 +00:00

nsaspy referenced this pull request

2026-08-27 19:50:34 +00:00

RAGE #44: repair real-stack E2E baseline after unified graph shell #46