GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||
GeezerVison is showing github.com — feat: add Claude Sonnet 5.5 and GPT-6.1 Sol to Agents and AI Gateway by ibetitsmike · Pull Request #30159 · coder/coder Conversation
|
| Model | Added to |
|---|---|
| Claude Sonnet 5.5 | anthropic (reasoningEffort: high), azure, bedrock (global.anthropic.claude-sonnet-5-5), openrouter, vercel |
| GPT-6.1 Sol | openai (reasoningEffort: medium), azure, openrouter, vercel |
Bedrock. bedrock-runtime serves Sonnet 5.5 only through its global inference profile, so the catalog curates the global. ID without a bare alias: fantasy's Bedrock client prefixes a bare anthropic.* ID with a regional profile (us., eu., apac.) that this model lacks. bedrock-mantle takes the bare ID and the gateway looks prices up by exact provider and model, so overrides.jq injects the bare ID as a copy of the global price entry. The two GPT Daybreak alias injections are gone: models.dev now lists both aliases, and their guards error out once it does.
Reasoning effort. Sonnet 5.5 rejects the thinking: {"type": "disabled"} that fantasy sends for effort none, and GPT-6.1 Sol rejects none outright, both with HTTP 400. applyReasoningEffort now maps none to low for Sonnet 5.5 on the Anthropic and Bedrock providers, next to the Opus 5.5 clamp, and maps none and minimal to low for GPT-6.1 Sol on OpenAI and Azure through a new chatopenai.IsGPT61Sol, next to the GPT-6 Astra clamp. GPT-6 Sol accepts none and stays unclamped; the test pins that. AI Gateway's Bedrock adaptive-thinking allowlist already matched Sonnet 5.5 through its anthropic.claude-sonnet-5 substring; a table case now covers it.
Price book drift. prices.json includes every model of its 8 providers, so the regeneration also brings in upstream changes since the last refresh: 28 new entries (such as Grok 4.7 on Bedrock and Mistral Large 2512 on OpenRouter), 7 entries models.dev delisted (mostly free variants on OpenRouter and Vercel), and 49 price changes (39 OpenRouter, 10 Vercel). Other uncurated upstream models, such as -pro and -fast variants and openai.* on Bedrock, stay out of the catalog, matching how earlier releases were curated.
The Reasoning effort section of coderd/x/chatd/ARCHITECTURE.md documents only the GPT-6 Astra clamp and should gain Sonnet 5.5 and GPT-6.1 Sol. That doc edit is left to a human author.
Validation: regenerating from a fresh models.dev fetch reproduces both generated files byte for byte. Go tests pass for scripts/aibridgepricesgen, coderd/aibridge/prices, coderd/x/chatd/chatprovider, coderd/x/chatd/chatopenai, and aibridge/intercept/messages. Removing either clamp with go test -overlay fails the new cases on exactly the Sonnet 5.5 and GPT-6.1 Sol inputs. Vitest passes for src/modules/aiModels (176 tests) and the Models and Coder Agents settings pages (24 tests). The pre-commit hook passed; local pre-push checks were skipped, and CI passed on 78abe17e69d.
Review record (1 completed pass): Codex, triggered by marking the PR ready, reviewed 78abe17e69d and raised one P1 asking for the ARCHITECTURE.md update above. Declined and resolved: the author's standing instruction keeps agents out of that file. No remote UAT was run.
Xum opened this PR on behalf of @ibetitsmike.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Curate Claude Sonnet 5.5 and GPT-6.1 Sol for every catalog provider that lists them, and regenerate prices.json and the known models catalog from models.dev. The regeneration also picks up other upstream additions, delistings, and price changes. Bedrock serves Sonnet 5.5 only through its global inference profile, so the catalog uses the global ID and overrides.jq injects the bare ID for bedrock-mantle pricing. The GPT Daybreak alias injections are dropped now that models.dev lists both aliases. Both models reject reasoning effort none, so chatd starts them at low, matching Claude Opus 5.5 and GPT-6 Astra. GPT-6.1 Sol also maps minimal to low.
There was an error while loading. Please reload this page.
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. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
Here are some automated review suggestions for this pull request.
Reviewed commit: 78abe17e69
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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".
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.