fix(cli): accept install-browser --no-shell

[Yury Semikhatsky (yury-s)]

Summary

• playwright-cli install-browser --no-shell was rejected with Unknown option: --shell: minimist parses it as { shell: false }, but the command declares no-shell. Flag validation now accepts negated flags declared as no-*.
• Drop the unused noShell option from installBrowsers.

Fixes #42804

[github-actions]

Test results for "tests 1"

8 flaky

⚠️ [chromium-library] › library/browsercontext-page-event.spec.ts:160 › should work with Shift-clicking `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/browsercontext-page-event.spec.ts:173 › should work with Ctrl-clicking `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:690 › screencast › should capture full viewport `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:762 › screencast › should work with video+trace `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@chromium-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:257 › third party 'Partitioned;' cookies `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsercontext-cookies-third-party.spec.ts:470 › top level 'Partitioned;' cookie and same origin iframe `@firefox-ubuntu-22.04-node20`
⚠️ [webkit-page] › page/page-leaks.spec.ts:136 › expect should not leak `@webkit-ubuntu-22.04-node20`

52148 passed, 1253 skipped

---

Merge workflow run.

[github-actions]

Test results for "MCP"

1 failed
❌ [firefox] › mcp/cli-session.spec.ts:54 › idle timeout shuts the session down @mcp-windows-latest-firefox

8704 passed, 1474 skipped

---

Merge workflow run.

[github-actions]

🟢 CI is clear — the one failure is a known Windows/Firefox flake

Hi, I'm the Playwright bot and I triaged the CI failures here.

[firefox] › mcp/cli-session.spec.ts:54 › idle timeout shuts the session down @mcp-windows-latest-firefox is the only red test, and it fails on plenty of SHAs this PR has nothing to do with. Nothing in the diff reaches it.

Details

Pre-existing flake / infra

• [firefox] › mcp/cli-session.spec.ts:54 › idle timeout shuts the session down — flaky on mcp-windows-latest-firefox specifically: 58 of 184 runs failed (32%) in the aggregated CI results, while the same test is 0/187-or-so failures on all twelve other MCP bots (ubuntu/macos/windows × chromium, chrome, msedge, firefox, webkit). Same error signature every time — the session is still listed as open after the 3s idle timeout, Timeout 5000ms exceeded while waiting on the predicate. Identical failure on PR fix(highlight): resolve highlights relative to the frame of the locator #42868 (run 35790687400) ~50 minutes before this PR's run, and on feat(coverage): generate html coverage report without istanbul libs #42784, chore(coverage): istanbul instrumentation for the trace viewer and html reporter #42785, docs(cli): mention webmcp-list next to the status and snapshot in the skill #42783, fix(firefox): preserve response header field boundaries #42832 and pushes to main.

  Last 30 runs of this test on that bot (oldest first, 🟩 passed / 🟥 failed):

  🟩🟥🟥🟩🟩🟥🟩🟩🟩🟩🟩🟩🟥🟥🟩🟩🟩🟥🟩🟩🟥🟩🟩🟩🟩🟩🟩🟥🟥🟩

  The diff also doesn't reach it: validateFlags() only gains two extra continue branches, so it is strictly more permissive than before, and --idle-timeout=3000 parses to a non-false value that the new args[key] === false branch never touches. The other three files are the install-browser path and a new test.

Flaky (passed on retry)

• The 8 warnings in the "tests 1" report (browsercontext-page-event, video, beforeunload, browsercontext-cookies-third-party, page-leaks) all went green on retry and are unrelated to CLI flag parsing.

_{Triaged by the Playwright bot - agent run}