-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Support Claude 3.7 Sonnet "Thinking" in OpenRouter #1195
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
Conversation
|
src/api/providers/openrouter.ts
Outdated
| } as OpenRouterChatCompletionParams) | ||
| } | ||
|
|
||
| console.log("OpenRouter completionParams:", completionParams) |
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.
✂️
src/api/providers/openrouter.ts
Outdated
|
|
||
| for await (const chunk of stream as unknown as AsyncIterable<OpenAI.Chat.Completions.ChatCompletionChunk>) { | ||
| // openrouter returns an error object instead of the openai sdk throwing an error | ||
| console.log("OpenRouter chunk:", chunk) |
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.
✂️
Description
Type of change
How Has This Been Tested?
Checklist:
Additional context
Related Issues
Reviewers
Important
Adds support for Claude 3.7 Sonnet "Thinking" model in OpenRouter, including model parameter handling and UI updates.
openrouter.ts.thinkingattribute.completionParamsto includethinkinglogic.thinkingattribute toModelInfoinapi.ts.anthropicModelsandopenRouterDefaultModelInfoto includethinking.ApiOptions.tsxto handlethinkingmodels with a slider for token budget.slider.tsxfor UI consistency.This description was created by
for 28e18a6. It will automatically update as commits are pushed.