Skip to content

fix(ci): restore Cypress binary setup for e2e runs#5254

Open
fpigeonjr wants to merge 4 commits intomainfrom
ci/cypress
Open

fix(ci): restore Cypress binary setup for e2e runs#5254
fpigeonjr wants to merge 4 commits intomainfrom
ci/cypress

Conversation

@fpigeonjr
Copy link
Contributor

Summary

  • cache the Cypress binary in GitHub Actions and install/verify it before e2e and a11y Cypress runs
  • prevent fresh CI runners from failing when the npm package exists but the Cypress binary is missing from the runner cache
  • switch the local TruffleHog pre-commit hook to the proven changed-file filesystem scan so commits work cleanly from git worktrees

Testing

  • ran bunx cypress install && bunx cypress verify in frontend/
  • ran pre-commit run trufflehog --files .pre-commit-config.yaml .github/workflows/ci.yml .github/workflows/e2e_test_reusable.yml

@fpigeonjr fpigeonjr self-assigned this Mar 11, 2026
@fpigeonjr fpigeonjr added the devex developer experiece label Mar 11, 2026
@fpigeonjr fpigeonjr requested a review from Copilot March 11, 2026 20:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates CI and local developer tooling to make Cypress-based E2E/a11y runs more reliable on fresh runners and to improve the TruffleHog pre-commit experience (notably for git worktrees).

Changes:

  • Cache the Cypress binary in GitHub Actions and run cypress install/verify before E2E and a11y Cypress runs.
  • Update the local TruffleHog pre-commit hook to scan changed files via trufflehog filesystem (instead of a git-based scan).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
.pre-commit-config.yaml Changes TruffleHog local hook entry to compute changed files and scan via filesystem.
.github/workflows/e2e_test_reusable.yml Adds Cypress binary caching + install/verify prior to running E2E specs.
.github/workflows/ci.yml Adds Cypress binary caching + install/verify prior to running the a11y regression Cypress specs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@fpigeonjr fpigeonjr marked this pull request as ready for review March 11, 2026 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devex developer experiece

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants