-
Notifications
You must be signed in to change notification settings - Fork 1.1k
feat(cursor): let the Codex Fast toggle reach Cursor's fast variant #3225
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,6 +18,7 @@ import { | |
| cursorModelInputModalities, | ||
| cursorModelReasoningEfforts, | ||
| } from "../adapters/cursor/discovery"; | ||
| import { cursorFastCapableBases } from "../adapters/cursor/catalog"; | ||
| import { COMMAND_CODE_MODEL_REASONING_EFFORTS } from "./command-code-efforts"; | ||
| import { isCanonicalOpenRouterTarget } from "./openrouter-routing"; | ||
|
|
||
|
|
@@ -1120,6 +1121,15 @@ export const PROVIDER_REGISTRY: readonly ProviderRegistryEntry[] = [ | |
| defaultModel: "auto", | ||
| modelContextWindows: cursorModelContextWindows(CURSOR_STATIC_MODELS), | ||
| modelDisplayNames: cursorModelDisplayNames(), | ||
| // Cursor's Fast product is a model VARIANT, not a service_tier field, so the wire kind | ||
| // is cursor-variant and the request builder consumes the decision. | ||
| fastWire: { kind: "cursor-variant", canonicalToWire: { priority: "fast" }, foreignCallerTiers: "drop" }, | ||
| // Deliberately NO provider-level supportsServiceTier: resolveFastPolicy short-circuits on | ||
| // `capability.provider === false` BEFORE consulting the per-model map, which would make | ||
| // these entries dead config. Absent leaves unlisted bases "unclassified", and a | ||
| // non-service-tier adapter cannot forward a caller tier, so they still publish no toggle. | ||
| modelSupportsServiceTier: Object.fromEntries(cursorFastCapableBases().map(id => [id, true])), | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When an existing session or explicit request uses a retained alias such as AGENTS.md reference: AGENTS.md:L339-L342 Useful? React with 👍 / 👎. |
||
| fastTierDescription: "Cursor Fast variant", | ||
| modelInputModalities: cursorModelInputModalities(CURSOR_STATIC_MODELS), | ||
| modelReasoningEfforts: cursorModelReasoningEfforts(CURSOR_STATIC_MODELS), | ||
| // Kimi K3 documents `max` as its API default, and its Cursor ladder has no `medium` | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This declaration makes Fast newly visible for five Cursor models and changes requests from their ordinary/thinking identity to Cursor's Fast model variant, but the commit contains no
docs-site/update explaining which models support the toggle or that Cursor does not send an OpenAIservice_tierfield. Update the English Cursor/Codex model documentation and keep translated pages consistent so operators can understand the new user-visible routing behavior.AGENTS.md reference: AGENTS.md:L343-L344
Useful? React with 👍 / 👎.