-
Notifications
You must be signed in to change notification settings - Fork 246
feat(cua): add hyperbrowser provider for cua agent #1052
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
base: main
Are you sure you want to change the base?
Conversation
Hi @NikhilShahi - thanks for the contribution! It's great to offer freedom of choice for underlying components like this. However my concern with this specific change would be that it forces the user to install dependencies for all providers rather than installing only the packages that they need to support their desired configuration. This sort of extensibility may be something that @bracesproul has already considered, so I'm afraid that I'll need to defer to him on this. In the meantime, if you can turn this PR into one that allows the user to bring any arbitrary provider without further modification to the code, that would be much easier to accept. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some feedback, very good start. thanks for putting this PR up!
8c24e0e
to
94e6668
Compare
Hey @bracesproul , thanks for the feedback. I think I have implemented the changes requested, let me know if there is anything else I should change. |
This PR adds Hyperbrowser as an optional provider for using with LangGraph and CUA. It uses Hyperbrowser browser session and playwright to allow CUA to interact with the browser page. Scrapybara is kept as the default provider.