From 35d9d3aafba527bdcc3ca7347c6f1bed05a33804 Mon Sep 17 00:00:00 2001 From: Joey Yakimowich-Payne Date: Sun, 26 Apr 2026 22:06:01 -0600 Subject: [PATCH] =?UTF-8?q?fix(e2e):=20run-pw.sh=20=E2=80=94=20don't=20set?= =?UTF-8?q?=20CI=3Dtrue;=20reuse=20docker=20compose=20dev=20server?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The helper previously set CI=true, which flipped Playwright's `reuseExistingServer` to false. That caused Playwright to spawn its own Vite dev server on :5173, colliding with the docker compose stack (paratype-web-dev container). Removing CI=true makes Playwright connect to the running dev server in docker. Verified: 146/146 e2e tests pass against docker-compose.dev.yml. 0 fixmes. 0 skips. EXIT_CODE=0. --- .sisyphus/scripts/run-pw.sh | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.sisyphus/scripts/run-pw.sh b/.sisyphus/scripts/run-pw.sh index bdacfc5..95eefa1 100755 --- a/.sisyphus/scripts/run-pw.sh +++ b/.sisyphus/scripts/run-pw.sh @@ -12,9 +12,14 @@ OUTFILE="${1:?need output file path}" shift # Spawn detached; touch DONE marker when complete (success or failure) +# NOTE: do NOT set CI=true — that flips Playwright's reuseExistingServer to +# false and makes it try to spawn its own webServer on :5173, which collides +# with the docker compose dev stack (paratype-web-dev). Leaving CI unset +# means reuseExistingServer=true → Playwright connects to the running dev +# server in docker. nohup bash -c " cd /home/joey/Projects/rules - export CI=true GIT_TERMINAL_PROMPT=0 GIT_PAGER=cat PAGER=cat + export GIT_TERMINAL_PROMPT=0 GIT_PAGER=cat PAGER=cat bunx playwright test $* > '$OUTFILE' 2>&1 echo \"EXIT_CODE=\$?\" >> '$OUTFILE' touch '$OUTFILE.done'