feat(claude): add claude-fable-5-1 and bump Claude Code fingerprint to 2.1.251 - #5405
feat(claude): add claude-fable-5-1 and bump Claude Code fingerprint to 2.1.251#5405gkinter wants to merge 1 commit into
Conversation
…o 2.1.251 Anthropic's /v1/models now returns `claude-fable-5-1` for subscription (OAuth) accounts, but the proxy rejects it with "unknown provider for model claude-fable-5-1" because it is missing from the embedded registry. Adding the model alone is not sufficient: Anthropic gates it on the advertised client version and returns "Claude Code 2.1.220 does not support this model; version 2.1.251 or newer is required. Run 'claude update' ..." so `defaultClaudeFingerprintUserAgent` is bumped from 2.1.220 to 2.1.251. Verified against a live subscription account: with both changes, `claude-fable-5-1` appears in /v1/models and chat completions succeed. Existing models (claude-fable-5, claude-opus-5, claude-sonnet-5) are unaffected. The registry entry mirrors claude-fable-5 (1M context, 128k max completion, thinking levels low..max, text+image in / text out).
|
This pull request targeted The base branch has been automatically changed to |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 43849b1120
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| const ( | ||
| defaultClaudeFingerprintUserAgent = "claude-cli/2.1.220 (external, cli)" | ||
| defaultClaudeFingerprintUserAgent = "claude-cli/2.1.251 (external, cli)" |
There was a problem hiding this comment.
Update tests for the 2.1.251 fingerprint
Changing this baseline alters both the emitted User-Agent and the billing header version returned by DefaultClaudeVersion, but the committed executor tests still require claude-cli/2.1.220 and cc_version=2.1.220 (notably TestClaudeExecutor_NonClaudeRequestUsesClaudeCode220CLIFingerprint and TestClaudeExecutor_ConfirmedClaudeCodeRequestPreservesInteractiveIdentity). Consequently, the required go test ./... suite will fail once dependencies are available; update the affected fixtures and assertions alongside the baseline.
Useful? React with 👍 / 👎.
Anthropic's
/v1/modelsnow returnsclaude-fable-5-1for subscription (OAuth) accounts, but the proxy rejects it withunknown provider for model claude-fable-5-1because it is missing from the embedded registry ininternal/registry/models/models.json.Adding the model alone is not sufficient. Anthropic gates it on the advertised client version and returns:
so this also bumps
defaultClaudeFingerprintUserAgentininternal/runtime/executor/helps/claude_device_profile.gofrom2.1.220to2.1.251.Verification
Built from this branch and run against a live Claude subscription account:
POST /v1/chat/completionswithclaude-fable-5-1->400 unknown provider for model400 Claude Code 2.1.220 does not support this model200, model echoes back asclaude-fable-5-1/v1/modelsgoes from 38 to 39 entries. Regression-checkedclaude-fable-5,claude-opus-5andclaude-sonnet-5— all still200.Notes
The registry entry mirrors
claude-fable-5: 1M context, 128k max completion,thinking levels
low..max,text+imagein /textout.🤖 Generated with Claude Code