From 31b29b6d424ccaf3f71ceb17cfd110be11ee9653 Mon Sep 17 00:00:00 2001 From: Anna Larch Date: Mon, 4 May 2026 19:16:05 +0200 Subject: [PATCH] fix(cypress): exclude screenshot spec from regular PR CI runs The screenshots.cy.ts spec is intended for the manual screenshots.yml workflow only. Including it in the default e2e spec pattern caused it to run on every PR, where the reduced app setup (viewer only) triggered a JS crash before any test could start. excludeSpecPattern is ignored when a spec is passed explicitly, so the screenshots.yml workflow is unaffected. AI-Assisted-By: Claude Sonnet 4.6 Signed-off-by: Anna Larch --- cypress.config.ts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/cypress.config.ts b/cypress.config.ts index 59307470c..19aac1f4f 100644 --- a/cypress.config.ts +++ b/cypress.config.ts @@ -38,6 +38,10 @@ export default defineConfig({ trashAssetsBeforeRuns: true, e2e: { + // Exclude the documentation screenshot spec from regular CI runs. + // It is run explicitly by the screenshots.yml workflow_dispatch workflow. + excludeSpecPattern: '**/screenshots.cy.ts', + // We've imported your old cypress plugins here. // You may want to clean this up later by importing these. async setupNodeEvents(on, config) {