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.

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

fix: Made the apiKey parameter optional in the TavilyAnswerTool and TavilySearchResultsTool classes #646

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

Nana-Kwame-bot
Copy link
Contributor

@Nana-Kwame-bot Nana-Kwame-bot commented Feb 2, 2025

  • Description: This pull request introduces new documentation and updates for integrating Tavily tools, specifically the TavilyAnswerTool and TavilySearchResultsTool, into the project. It also includes some code refactoring to make the API key optional for these tools.

Documentation Updates:

  • Added new documentation pages for TavilyAnswerTool and TavilySearchResultsTool in docs/modules/agents/tools and docs_v2/docs/05-integrations/tools directories. These pages provide detailed instructions on installation, configuration, usage, and error handling.
  • Updated docs/_sidebar.md to include links to the new Tavily documentation pages.
  • Updated docs_v2/sidebars.js to replace the old Tavily search page with the new Tavily answer and search results pages.
  • Updated packages/langchain/README.md to include links to the new Tavily tools documentation.
  • Updated packages/langchain_community/README.md to add DirectoryLoader and update the description for Tavily tools.

Code Refactoring:

  • Made the apiKey parameter optional in the TavilyAnswerTool and TavilySearchResultsTool classes.
  • Updated tests in packages/langchain_community/test/tools/tavily_test.dart and packages/tavily_dart/test/tavily_test.dart to reflect the optional apiKey parameter.
  • Removed the api_key requirement from the tavily_openapi.yaml file.

@davidmigloz davidmigloz self-requested a review February 3, 2025 10:13
@davidmigloz davidmigloz added c:tools Tools. p:langchain_community langchain_community package. t:bug Something isn't working labels Feb 3, 2025
@davidmigloz davidmigloz added this to the Next release milestone Feb 3, 2025
Copy link
Owner

@davidmigloz davidmigloz left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for your work!

@davidmigloz davidmigloz merged commit 5085ea4 into davidmigloz:main Feb 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:tools Tools. p:langchain_community langchain_community package. t:bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants