feat: add optional you.com research provider#52
Conversation
|
Thanks @mouse-value-add — and the You.com team — this is a genuinely clean PR. A few things I appreciated on read-through:
One thing I want to get right before merging: the Research API returns a Ask: surface Optional, only if you're already in there: you flagged the Once the citations are handled I'm ready to merge, and I'll credit you and the You.com team in the changelog. Really appreciate the quality here — this is a great template for how integrations should land in the toolkit. 🙌 |
|
I addressed the citation follow-up by appending a readable Validation:
Branch: https://github.com/mouse-value-add/intellegix-code-agent-toolkit/tree/feat/youcom-search-integration |
|
Citations/sources handling implemented and validated! ✅ The latest commit (e86e1c5) includes the method that appends a clean "## Sources" section to research answers, preserving both the inline citation markers and their corresponding source references. Implementation details:
Test coverage: tests/test_research_bridge.py::TestResearchBridge::test_successful_youcom_query PASSED [100%] ======================= 1 passed, 42 deselected in 0.06s ======================= Ready for merge when you are. Thanks for the clear feedback on getting the citation flow right! |
Why
This project already has a dedicated research bridge, but today it assumes a Playwright-driven Perplexity flow. This keeps the default behavior intact and adds an optional You.com-backed research path for cases where a direct API integration is a better fit.
What changed
providersetting to the research configResearchBridgeSetup
Optional config:
{ "perplexity": { "provider": "youcom" } }Usage
Existing usage stays the same. If the provider is set to
youcom, research requests go through the You.com Research API instead of the Playwright browser flow.Validation performed
python3 -m pytest tests/test_config.py -qpython3 -m py_compile research_bridge.py loop_driver.py config.pypython3 -m pytest tests/test_research_bridge.py tests/test_config.py -q(blocked during collection in the existing test suite becausetests/helpers.pyusesfloat | None, which the localpython3interpreter does not support)Fallback and error handling
YDC_API_KEYreturns a clearMISSING_API_KEYerrorTracking issue: youdotcom-oss/integration-tracking#39
Happy to adjust the shape if you'd prefer this setting to live under a more generic
researchsection instead.