Skip to content

Report missing dependencies with an actionable message - #130

Merged
smnandre merged 3 commits into
playwright-php:mainfrom
kbond:actionable-setup-errors
Nov 7, 2026
Merged

smnandre merged 3 commits into
playwright-php:mainfrom
kbond:actionable-setup-errors

Conversation

@kbond

@kbond kbond commented Aug 5, 2026 •

Copy link
Copy Markdown
Contributor

Throw MissingDependencyException with an install command when Playwright or its browser dependencies are missing. Both launch() and launchServer() use the same diagnostics.

What’s missing Command shown
Playwright Node package vendor/bin/playwright-install --browsers
Browser binary vendor/bin/playwright-install --browsers
Host libraries vendor/bin/playwright-install --with-deps

When Node exits, include redacted stderr from that process in the exception.

@codecov

codecov Bot commented Aug 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@kbond
kbond force-pushed the actionable-setup-errors branch from fa75b3e to 1131972 Compare November 7, 2026 12:55
@smnandre
smnandre force-pushed the actionable-setup-errors branch from 1131972 to 6b96acf Compare November 7, 2026 16:34
@kbond
kbond force-pushed the actionable-setup-errors branch from 6b96acf to 09b2c4a Compare November 7, 2026 17:19
@smnandre
smnandre force-pushed the actionable-setup-errors branch from fa0d704 to 8a86ede Compare November 7, 2026 19:19
@smnandre
smnandre merged commit bc953b4 into playwright-php:main Nov 7, 2026
10 checks passed
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.

2 participants