-
Notifications
You must be signed in to change notification settings - Fork 12
Add web search tool #32
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
Conversation
- Updated .env.example to include WEB_SEARCH_BASE_URL and WEB_SEARCH_API_KEY. - Introduced web_search.py to implement a tool for fetching Google Search grounded responses from the Gemini model.
…h query extraction - Introduced RETRYABLE_STATUS for managing retryable HTTP status codes. - Updated the _post method to _post_payload for clarity and added a docstring. - Enhanced error handling in the backoff decorator to include specific exceptions and retry logic based on RETRYABLE_STATUS.
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.
Pull Request Overview
This PR adds a web search tool that integrates Google Search grounded responses from the Gemini model via a proxy server. The implementation includes configurable model settings, retry logic, and citation formatting.
Key changes:
- New
GeminiGroundingWithGoogleSearchclass that handles API communication with retry logic and concurrency control - Citation formatting utility to add inline source links to Gemini responses
- Environment variable configuration for the proxy server base URL and API key
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/utils/tools/web_search.py | Implements the core web search tool with Gemini integration, including model configuration, API client, retry logic, and citation formatting |
| tests/tool_tests/test_web_search.py | Adds integration test for the web search functionality |
| .env.example | Adds environment variable placeholders for web search proxy configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ValueError for missing environment variables.
- Updated the add_citations function to return a tuple of synthesized text and a mapping of citation IDs to source labels. - Enhanced the GroundedResponse class to include a citations field. - Improved error handling and type checks throughout the citation processing logic. - Introduced a new _collect_citations function to streamline citation ID collection from candidates.
PR Type
Feature
Short Description
Tests Added
tests/tool_tests/test_gemini_grounding.py