Skip to content

fix(cli): suggest matching ffmpeg installer for video recording - #44088

Merged
Yury Semikhatsky (yury-s) merged 1 commit into
mainfrom
fix-540-43324709454-1
Nov 12, 2026
Merged

Yury Semikhatsky (yury-s) merged 1 commit into
mainfrom
fix-540-43324709454-1

Conversation

@microsoft-playwright-automation

Copy link
Copy Markdown
Contributor

Hi, I'm the Playwright bot and I fixed the missing-ffmpeg hint for video recording.

video-start starts a screencast after browser launch, so it bypasses the package-specific missing-executable error handling. Share that handling with screencast startup: CLI mode now suggests playwright-cli install-browser ffmpeg, MCP mode uses its own installer, and both retain the expected executable path.

The regression extends the existing CLI video-start-stop test with an empty ffmpeg cache.

Refs microsoft/playwright-cli#540

Triaged on Linux by the Playwright bot - agent run

@yury-s
Yury Semikhatsky (yury-s) marked this pull request as ready for review November 12, 2026 23:30

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's rather scope your av1 test to page.screenast and land it.

@yury-s
Yury Semikhatsky (yury-s) merged commit c59cf20 into main Nov 12, 2026
16 of 18 checks passed
@yury-s
Yury Semikhatsky (yury-s) deleted the fix-540-43324709454-1 branch November 12, 2026 23:49
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

4 failed
❌ [chrome] › mcp/crash.spec.ts:31 › crash recovery › reports the navigation error, then resets and logs the crash @mcp-ubuntu-latest-chrome
❌ [chrome] › mcp/crash.spec.ts:58 › crash recovery › lists only one tab @mcp-ubuntu-latest-chrome
❌ [chrome] › mcp/crash.spec.ts:79 › crash recovery › marks non-current crashed tab in the tab list @mcp-ubuntu-latest-chrome
❌ [chromium] › mcp/sse.spec.ts:196 › sse transport browser lifecycle (persistent) @mcp-ubuntu-latest-chromium

5231 passed, 631 skipped


Merge workflow run.

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