Skip to content

fix(launch): gate ENABLE_TOOL_SEARCH on tool_surface_reduction#148

Merged
KokaKiwi merged 2 commits into
mainfrom
fix/EOSS-68-tool-search
Jul 21, 2026
Merged

fix(launch): gate ENABLE_TOOL_SEARCH on tool_surface_reduction#148
KokaKiwi merged 2 commits into
mainfrom
fix/EOSS-68-tool-search

Conversation

@KokaKiwi

Copy link
Copy Markdown
Member

Checklist

  • I have read the Contributor Guide
  • I have read and agree to the Code of Conduct
  • I have added a description of my changes and why I'd like them included in the section below

Description of Changes

Companion fix to edgee-ai/gateway#449 for the ToolSearch / tool_surface_reduction conflict: when launching Claude Code (edgee launch claude), the client's own MCP Tool Search should stay off whenever the resolved key has server-side tool_surface_reduction enabled, so the two mechanisms never race in the first place.

  • Behavior: ENABLE_TOOL_SEARCH=false is now set by default only when the resolved API key has tool_surface_reduction enabled and the user hasn't already set the var themselves — with surface reduction off, the CLI leaves it untouched entirely
  • No extra API calls: login::ensure_valid_provider_key now returns ProviderKeyStatus { created, compression } instead of a bare bool, surfacing the Compression it already fetches via get_key_by_id while validating the cached key — claude.rs reads tool_surface_reduction off that instead of issuing a second, identical request
  • Other callers: codex.rs, crush.rs, opencode.rs, codebuddy.rs, and relay/mod.rs (2 call sites) only needed .created off the new struct — no behavior change there

Related Issues

Related to EOSS-68
Related to #128
Related to edgee-ai/gateway#449

@KokaKiwi
KokaKiwi requested a review from a team as a code owner July 20, 2026 15:30
@KokaKiwi
KokaKiwi merged commit 34799f8 into main Jul 21, 2026
4 checks passed
@KokaKiwi
KokaKiwi deleted the fix/EOSS-68-tool-search branch July 21, 2026 07:18
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