Skip to content

Fix flag description for Browser Session Management#245

Merged
yury-s merged 1 commit intomicrosoft:mainfrom
ZAAI-com:main
Feb 9, 2026
Merged

Fix flag description for Browser Session Management#245
yury-s merged 1 commit intomicrosoft:mainfrom
ZAAI-com:main

Conversation

@manuelgruber
Copy link
Contributor

This pull request updates the documentation for managing named browser sessions in Playwright CLI. The main change is clarifying the correct flag to use for isolating browser contexts.

Documentation update:

  • Updated the session management documentation in session-management.md to indicate that the -s flag (instead of -b) should be used to isolate browser contexts.

Copilot AI review requested due to automatic review settings February 8, 2026 12:06
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects the Playwright CLI session management documentation to reference the proper flag for isolating named browser sessions (browser contexts).

Changes:

  • Updated the session isolation guidance from using -b to using -s.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@manuelgruber
Copy link
Contributor Author

manuelgruber commented Feb 8, 2026

Closes #243

@Skn0tt
Copy link
Member

Skn0tt commented Feb 9, 2026

Hi! We renamed this a couple days ago, see microsoft/playwright#39156. So I think -b is right, maybe you're using a slightly old version of the CLI?

@Skn0tt Skn0tt closed this Feb 9, 2026
@yury-s yury-s reopened this Feb 9, 2026
@yury-s
Copy link
Member

yury-s commented Feb 9, 2026

You are right about the issue, the file used to create the kill via playwright-cli install lives upstream thought, I'm updating it there microsoft/playwright#39200.

@yury-s yury-s merged commit c146746 into microsoft:main Feb 9, 2026
9 of 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.

3 participants