Skip to content

fix: decouple LiteLLM web search from Codex search tool - #23

Merged
AviBackToBlack merged 6 commits into
mainfrom
fix/v0.3-web-search-tool-semantics
Sep 14, 2026
Merged

AviBackToBlack merged 6 commits into
mainfrom
fix/v0.3-web-search-tool-semantics

Conversation

@AviBackToBlack

@AviBackToBlack AviBackToBlack commented Sep 14, 2026 •

Copy link
Copy Markdown
Owner

Summary

Follow up the merged foreign web-search compatibility contract from #22 with the intentionally small production fix it identified.

This PR:

  • stops mapping LiteLLM supports_web_search=false to Codex supports_search_tool=false for exact templates
  • stops treating supports_web_search overrides as provenance/ownership for exact supports_search_tool
  • keeps foreign supports_search_tool=false as a conservative tool-search/deferred-discovery decision, not a hosted-web-search suppression guarantee
  • keeps LiteLLM/override supports_web_search visible as capability evidence in explain
  • adds regressions for raw exact evidence, exact overrides, and foreign override provenance
  • aligns the older model-group aggregation design text with the source-backed web-search contract

No foreign hosted-web-search enablement is added. Provider/runtime hosted-search behavior remains deferred under docs/foreign-web-search.md.

Refs #5


Devin Review

Copilot AI lite review requested due to automatic review settings September 14, 2026 12:01

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@devin-ai-integration devin-ai-integration 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@AviBackToBlack
AviBackToBlack merged commit 249a79d into main Sep 14, 2026
9 checks passed
@AviBackToBlack
AviBackToBlack deleted the fix/v0.3-web-search-tool-semantics branch September 14, 2026 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants