Skip to content

ci(e2e): start the dev server and wait for it before Cypress runs - #125

Merged
Gkrumbach07 merged 1 commit into
mainfrom
fix/ci-e2e-server-wait
Oct 22, 2026
Merged

Gkrumbach07 merged 1 commit into
mainfrom
fix/ci-e2e-server-wait

Conversation

@Gkrumbach07

Copy link
Copy Markdown
Contributor

What

The e2e job now starts the dev server itself, waits for it with a curl loop (up to 60 s, printing Vite's log if it never answers), and then runs Cypress with the action's install and wait turned off.

Why

Since 2026-10-22 every e2e run fails before a test runs, on main too (a83a4cc, run 39783306252). The Cypress action's wait-on stops at the first refused connection ("Socket closed before the connection was established") instead of retrying. Nothing in the repository changed, and the runner image and action SHA match the last passing run.

A red CI on main makes the Release workflow skip, so merging #118 (OpenShell 0.1.5) cut no release. Merging this should turn main green and let the 0.1.5 release run.

This is fedd07a from #123, cherry-picked unchanged. #123's CI passed with it.

🤖 Generated with Claude Code

From 2026-10-22 every e2e run failed before a test ran, on main as well
(a83a4cc): the Cypress action's wait-on gave up on its first refused
connection ("Socket closed before the connection was established") instead
of retrying, before Vite had printed a line. Nothing in the repository had
changed; the runner image and the action's SHA were the same as in the last
passing run.

The job now builds, starts the dev server in the background and waits for
it with curl, printing Vite's log when it does not come up, and then runs
Cypress against it with the action's install and wait left out.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Gkrumbach07
Gkrumbach07 merged commit 49511c8 into main Oct 22, 2026
14 checks passed
@Gkrumbach07
Gkrumbach07 deleted the fix/ci-e2e-server-wait branch November 6, 2026 17:02
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