Skip to content

Move playwright install to commands_pre#193

Open
ksy36 wants to merge 1 commit intomainfrom
plawright_browser_fix
Open

Move playwright install to commands_pre#193
ksy36 wants to merge 1 commit intomainfrom
plawright_browser_fix

Conversation

@ksy36
Copy link
Copy Markdown
Collaborator

@ksy36 ksy36 commented May 4, 2026

From this error it looks like running uv run --extra=dev playwright install --with-deps chromium firefox, in CI installs browsers matching the playwright version pinned in uv.lock (playwright firefox v1509):

Firefox 146.0.1 (playwright firefox v1509) downloaded to /home/runner/.cache/ms-playwright/firefox-1509

But tox is installing a newer playwright version, so can't locate the browsers that were just installed:

        except Exception as error:
>           raise rewrite_error(error, f"{parsed_st['apiName']}: {error}") from None
E           playwright._impl._errors.Error: BrowserType.launch: Executable doesn't exist at /home/runner/.cache/ms-playwright/firefox-1511/firefox/firefox

Moving the browser install to commands_pre so it runs inside tox virtualenv seems to help.

@ksy36 ksy36 requested a review from jgraham May 4, 2026 18:11
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