Skip to content

[fix]: Context & Cookie Insertion Fix #62

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
May 8, 2025
Merged

Conversation

Kylejeong2
Copy link
Collaborator

What

Fixing cookies and context insertion to be within config flags, added some examples to readme.

// Use non-null assertion after check
projectId: config.browserbaseProjectId!,
proxies: config.proxies,
browserSettings: {
Copy link
Contributor

Choose a reason for hiding this comment

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

how do we feel ab this? pretty sure there is a browser_resize tool implemented from playwright mcp

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

i just found that it works better at that size

Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't this be the default config then?

@Kylejeong2 Kylejeong2 merged commit f97b56a into ap/bbplaywright May 8, 2025
@Kylejeong2 Kylejeong2 deleted the kj/context branch May 16, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
close