Skip to content
This repository was archived by the owner on Jul 19, 2026. It is now read-only.

fix: update You.com Search endpoint to latest docs - #4

Merged
haasonsaas merged 2 commits into
evalops:mainfrom
mouse-value-add:feat/youcom-search-provider-v2
Apr 15, 2026
Merged

fix: update You.com Search endpoint to latest docs#4
haasonsaas merged 2 commits into
evalops:mainfrom
mouse-value-add:feat/youcom-search-provider-v2

Conversation

@mouse-value-add

Copy link
Copy Markdown
Contributor

Thanks again for reviewing this integration. I noticed the initial implementation used a legacy You.com Search endpoint shape, so this follow-up aligns with the latest docs from the official youdotcom-api skill.\n\n### What changed\n- Update default endpoint to \n- Use query param (instead of legacy )\n- Prefer env var (with kept as a backward-compatible alias)\n- Update README examples and function signature docs to match current API contract\n\n### Validation\n- \n\n### Compatibility\n- Existing users with continue to work.\n- Existing call sites remain unchanged unless they explicitly override endpoint/params.\n\nHappy to adjust naming or env-var strategy if you prefer stricter conventions.

@cursor

cursor Bot commented Apr 15, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Medium risk because it changes the external HTTP contract (endpoint and query params) for the You.com search tool, which could break live search if the API differs; tests cover key formatting and legacy key fallback but don’t validate the real service.

Overview
Updates register_you_com_search_tool to use the new You.com Search endpoint (https://ydc-index.io/v1/search), send count instead of num_web_results, and clamp requested results once for request/formatting consistency.

Switches the default API key env var to YDC_API_KEY while supporting YOUCOM_API_KEY as a legacy alias when api_key_env is unchanged, and updates the README plus tests (including a new legacy-alias test) accordingly.

Reviewed by Cursor Bugbot for commit 6e4c62a. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 042198f. Configure here.

Comment thread tests/test_you_search_tool.py
Comment thread agent_harness.py Outdated
@haasonsaas
haasonsaas force-pushed the feat/youcom-search-provider-v2 branch from 042198f to a2fb76e Compare April 15, 2026 01:57
@haasonsaas
haasonsaas force-pushed the feat/youcom-search-provider-v2 branch from a2fb76e to 6e4c62a Compare April 15, 2026 01:58
@haasonsaas
haasonsaas merged commit 7e4c507 into evalops:main Apr 15, 2026
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants