Require Claude Code 2.1.248 and remove gateway auth workaround - #413
Merged
Conversation
andy-xu-db
enabled auto-merge (squash)
August 28, 2026 21:32
lilly-luo
reviewed
Aug 28, 2026
Collaborator
|
the model discovery proxy is now completely separate from the relayed auth proxy now right? model discovery is responsible for patching the model name where as relayed auth swaps headers. in that case, i think it makes sense to fully decouple them. |
Collaborator
Author
makes sense, i updated it so model discovery and relayed auth use separate proxies now |
…-248 # Conflicts: # src/ucode/anthropic_model_discovery_proxy.py
lilly-luo
approved these changes
Sep 1, 2026
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.
Summary
apiKeyHelperas the only credentialANTHROPIC_AUTH_TOKENand stop creating a refresh cache for native gateway model discoveryTesting
uv run pytest tests/test_agent_claude.py tests/test_anthropic_model_discovery_proxy.py tests/test_gateway_proxy.py -q(167 passed)uv run pytest tests/test_lint.py -q(3 passed)uv run ruff check .Live end-to-end verification
Verified on
eng-ml-inference-stagingwith Claude Code 2.1.251:ucode claude --enable-model-discovery --skip-preflightlaunched through the PR model-alias proxyANTHROPIC_BASE_URLand had noANTHROPIC_AUTH_TOKEN/modeldisplayed gateway models (system.ai.claude-sonnet-4-6[1m],system.ai.claude-haiku-4-5, plus 13 more), proving/v1/modelsauthenticated viaapiKeyHelperUCODE_PR_413_OKwith exit code 0Local full-suite note: the locally installed Claude Code 2.1.251 returned a canned response in
TestClaudeUserAgent::test_user_agent_arrives_at_gatewaywithout contacting its localhost capture server; the same test passed in GitHub CI.