Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(community): Stagehand tools integration #7177

Merged
merged 36 commits into from
Nov 25, 2024
Merged

feat(community): Stagehand tools integration #7177

merged 36 commits into from
Nov 25, 2024

Conversation

filip-michalsky
Copy link
Contributor

@filip-michalsky filip-michalsky commented Nov 10, 2024 •
edited
Loading

This PR introduces an integration of Langchain with Stagehand, the best in class AI web automation framework from Browserbase HQ.

Users can now use Stagehand web navigation tooling to power their agents with simple atomic web interactions!

@vercel Vercel
Copy link

vercel bot commented Nov 10, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Nov 25, 2024 4:21pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Nov 25, 2024 4:21pm

@filip-michalsky filip-michalsky changed the title Stagehand tools [DRAFT] Stagehand tools Nov 10, 2024
@filip-michalsky filip-michalsky changed the title [DRAFT] Stagehand tools [DRAFT] Stagehand tools integration Nov 10, 2024
@filip-michalsky filip-michalsky changed the title [DRAFT] Stagehand tools integration Stagehand tools integration Nov 14, 2024
@filip-michalsky filip-michalsky marked this pull request as ready for review November 14, 2024 05:07
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. auto:documentation Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Nov 14, 2024
@filip-michalsky
Copy link
Contributor Author

filip-michalsky commented Nov 21, 2024 •
edited
Loading

Thanks! See comments, please also remember to run yarn format and yarn lint.

All changes addressed. I can't get this thing to compile anymore after syncing with main but before I sync, the build works fine locally.

filip-michalsky
filip-michalsky commented Nov 24, 2024
View reviewed changes
libs/langchain-community/package.json Outdated
@@ -190,6 +193,7 @@
"node-llama-cpp": "3.1.1",
"notion-to-md": "^3.1.0",
"officeparser": "^4.0.4",
"openai": "^4.73.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we need this for stagehand extract tool

filip-michalsky
filip-michalsky commented Nov 24, 2024
View reviewed changes
libs/langchain-community/package.json
@@ -319,6 +324,7 @@
"neo4j-driver": "*",
"notion-to-md": "^3.1.0",
"officeparser": "^4.0.4",
"openai": "*",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we need this for stagehand extract tool

jacoblee93
jacoblee93 approved these changes Nov 25, 2024
View reviewed changes
Copy link
Collaborator

@jacoblee93 jacoblee93 left a comment

Choose a reason for hiding this comment

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

Thanks for your patience! Looks great, thank you!

@dosubot dosubot bot added the lgtm PRs that are ready to be merged as-is label Nov 25, 2024
@jacoblee93 jacoblee93 changed the title Stagehand tools integration feat(community): Stagehand tools integration Nov 25, 2024
jacoblee93
jacoblee93 reviewed Nov 25, 2024
View reviewed changes
yarn.lock Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a really big diff and something is funky with running lint from root - investigating

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jacoblee93 thank you, pls let me know if/how I can help

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I needed the langgraph dep for the example:

https://langchainjs-docs-6b8e73its-langchain.vercel.app/docs/integrations/tools/stagehand/#usage-with-langgraph-agents

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think resolved! Commented out the test for now - @langchain/langgraph also depends on community as a dev dep so it can cause some weird issues when developing locally

We can comment the test back in when needed

@jacoblee93 jacoblee93 merged commit de7bcda into langchain-ai:main Nov 25, 2024
37 checks passed
FilipZmijewski pushed a commit to FilipZmijewski/langchainjs that referenced this pull request Nov 27, 2024
@filip-michalsky @jacoblee93 @FilipZmijewski
feat(community): Stagehand tools integration (langchain-ai#7177)
b40f981 
Co-authored-by: jacoblee93 <jacoblee93@gmail.com>
syntaxsec pushed a commit to aks-456/langchainjs that referenced this pull request Dec 13, 2024
@filip-michalsky @jacoblee93 @syntaxsec
feat(community): Stagehand tools integration (langchain-ai#7177)
a32a1a8 
Co-authored-by: jacoblee93 <jacoblee93@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jacoblee93 jacoblee93 jacoblee93 approved these changes

Assignees
No one assigned
Labels
auto:documentation Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder auto:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features close PRs that need one or two touch-ups to be ready lgtm PRs that are ready to be merged as-is size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@filip-michalsky @jacoblee93