Skip to content

feat: add deferred failures and batch image review UI#374

Open
FRSgit wants to merge 3 commits into
mainfrom
feat/deferred-failures-and-batch-review-ui
Open

feat: add deferred failures and batch image review UI#374
FRSgit wants to merge 3 commits into
mainfrom
feat/deferred-failures-and-batch-review-ui

Conversation

@FRSgit

@FRSgit FRSgit commented Jun 30, 2026

Copy link
Copy Markdown
Member

All cy.matchImage() calls no longer fail immediately when the diff threshold is exceeded. Instead, failures are accumulated and reported as a single error at the end of the spec (opt-in via pluginVisualRegressionDeferFailures env var).

A persistent floating action button (plugin logo) is injected into the Cypress runner and shows a badge with the count of pending diffs. Clicking it opens:

  • an informational modal when there are no diffs
  • a fullscreen carousel for reviewing all changed images when diffs exist

The carousel supports Replace image / Skip this change / keyboard navigation and shows a success animation on the FAB after approvals.

All cy.matchImage() calls no longer fail immediately when the diff
threshold is exceeded. Instead, failures are accumulated and reported
as a single error at the end of the spec (opt-in via
pluginVisualRegressionDeferFailures env var).

A persistent floating action button (plugin logo) is injected into the
Cypress runner and shows a badge with the count of pending diffs.
Clicking it opens:
- an informational modal when there are no diffs
- a fullscreen carousel for reviewing all changed images when diffs exist

The carousel supports Replace image / Skip this change / keyboard
navigation and shows a success animation on the FAB after approvals.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
FRSgit and others added 2 commits June 30, 2026 23:11
…erage

- Run prettier on commands.ts and support.ts (fix lint CI failure)
- Add c8 ignore on browser-only template helpers in support.ts
- Add unit tests for recordPendingDiffTask, getPendingDiffsTask, and
  clearPendingDiffsTask to bring function coverage back above 90%

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ew-ui

# Conflicts:
#	packages/cypress-plugin-visual-regression-diff/src/commands.ts
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