Draft
Conversation
21 tasks
Co-authored-by: ebhills <53243273+ebhills@users.noreply.github.com>
Co-authored-by: ebhills <53243273+ebhills@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add new search.ai_mode wrangle with SerpAPI
Add SerpAPI Google AI Mode search wrangle
Aug 20, 2026
Collaborator
|
@copilot a few things that jump out at me immediately:
These are just my initial findings, with more tests and a working function, I will have another look. |
Co-authored-by: thomasstvr <65630492+thomasstvr@users.noreply.github.com>
Contributor
Author
Addressed in Recommended disposition: Comment only Next steps
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Linked issue
What changes
Adds
search.ai_mode, combining cited web search and synthesized content into one normalized SerpAPI request while preserving existing search contracts.n_results, preserves query order and blank rows, avoids ambiguous currency inference, and makes raw provider responses opt-in.wrangles.searchas the core module while retainingSerpApiWranglesClient; classic search and retrieval behavior is unchanged.How it was verified
action_required; zero jobs started).Compatibility and risk
No recipe migration is required. Existing
search.find_linksandsearch.retrieve_link_contentcontracts remain unchanged.SERPAPI_API_KEYis required for nonblank requests unlessapi_keyis supplied. Provider schema drift and request cost are the primary risks; normalization tests cover the documented response sections, andno_cacheremains explicit. Safest rollback is reverting the two feature commits.Ready-for-review checklist
mainand has no merge conflictsSee the pull request workflow.