8000 Added serper.dev as new tool with agno toolkit by CoderOMaster · Pull Request #2673 · agno-agi/agno · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Added serper.dev as new tool with agno toolkit #2673

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

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

CoderOMaster
Copy link
@CoderOMaster CoderOMaster commented Apr 4, 2025

Summary

This PR incorpates use of serper.dev api tool which works flawless and one of the best while performing google search. I have tested this locally and it worked like charm

Type of change

  • New feature

Checklist

  • Code complies with style guidelines
  • Ran format/validation scripts (./scripts/format.sh and ./scripts/validate.sh)
  • Self-review completed
  • Documentation updated (comments, docstrings)
  • Examples and guides: Relevant cookbook examples have been included or updated (if applicable)
  • Tested in clean environment

Additional Notes

Add any important context (deployment instructions, screenshots, security considerations, etc.)

@CoderOMaster CoderOMaster requested a review from a team as a code owner April 4, 2025 08:48
@CoderOMaster
Copy link
Author

@anuragts hey what else do i need to add or check? thanks a lot

@CoderOMaster
Copy link
Author

hi can you please check now? @anuragts

@dirkbrnd
Copy link
Contributor

Hi @CoderOMaster
Thanks for the contribution! Would you mind adding tests in /libs/agno/tests/unit/tools (see how the others work more or less). And also update the PR description so we know what the purpose is of the tool. Thanks!

@CoderOMaster
Copy link
Author

@dirkbrnd hey i have added it, let me it know if its good enough?

@CoderOMaster CoderOMaster changed the title Add files via upload Added serper.dev as new tool with agno toolkit Apr 18, 2025
Copy link
Contributor
@anuragts anuragts left a comment

Choose a reason for hiding this comment

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

LGTM


self.api_key = api_key or getenv("SERPER_API_KEY")
if not self.api_key:
log_debug("No Serper API key provided")
Copy link
Contributor
8000

Choose a reason for hiding this comment

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

We should raise a value error so user is made aware that API is not set.
raise ValueError("No Serper API key provided")

def __init__(
self,
api_key: str = "",
gl: str = "us",
Copy link
Contributor

Choose a reason for hiding this comment

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

change this to relevant parameter name: could be 'search_location'

@dmarkey
Copy link
dmarkey commented May 15, 2025

Anything we can do to push this through? Serper MCP server unneeded complication.

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.

5 participants
0