-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
base: main
Are you sure you want to change the base?
Conversation
@anuragts hey what else do i need to add or check? thanks a lot |
hi can you please check now? @anuragts |
Hi @CoderOMaster |
@dirkbrnd hey i have added it, let me it know if its good enough? |
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.
LGTM
|
||
self.api_key = api_key or getenv("SERPER_API_KEY") | ||
if not self.api_key: | ||
log_debug("No Serper API key provided") |
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.
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", |
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.
change this to relevant parameter name: could be 'search_location'
Anything we can do to push this through? Serper MCP server unneeded complication. |
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
Checklist
./scripts/format.sh
and./scripts/validate.sh
)Additional Notes
Add any important context (deployment instructions, screenshots, security considerations, etc.)