fix(openai): default reasoning effort for mini models#1824
Conversation
🦋 Changeset detectedLatest commit: 7a8a791 The changes in this PR will be included in the next version bump. This PR includes changesets to release 35 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
| export function supportsReasoningEffort(model: ChatModels | string): boolean { | ||
| return [ | ||
| 'gpt-5.4', | ||
| 'gpt-5.3-chat-latest', | ||
| 'gpt-5.4-mini', | ||
| 'gpt-5.2', | ||
| 'gpt-5.2-chat-latest', | ||
| 'gpt-5.1', | ||
| 'gpt-5.1-chat-latest', | ||
| 'gpt-5', | ||
| 'gpt-5-mini', | ||
| 'gpt-5-nano', | ||
| ].includes(model); |
There was a problem hiding this comment.
🚩 gpt-5.4-nano missing from supportsReasoningEffort despite being in inference model types
The inference LLM types at agents/src/inference/llm.ts:23 include 'openai/gpt-5.4-nano', but supportsReasoningEffort at plugins/openai/src/models.ts:213-222 does not list gpt-5.4-nano. This means no default reasoning effort is applied for that model. This may be intentional (the model might not support reasoning), and since gpt-5.4-nano is also absent from the plugin's ChatModels type, it's not a first-class plugin model. Worth confirming this omission is deliberate.
Was this helpful? React with 👍 or 👎 to provide feedback.
Ports livekit/agents#6148 to agents-js.
Summary
gpt-5.4-minito OpenAI chat model support.reasoning_efforttononeforgpt-5.4-miniwhen unset.Testing
pnpm --filter @livekit/agents buildpnpm --filter @livekit/agents-plugins-test buildpnpm --filter @livekit/agents-plugin-silero buildpnpm --filter @livekit/agents-plugin-openai buildpnpm --filter @livekit/agents-plugin-openai lint(warnings only)pnpm format:checkNotes
pnpm --filter @livekit/agents-plugin-openai api:checkcurrently fails before validating this change because API Extractor does not support the existingexport * as ___syntax inplugins/openai/dist/index.d.ts.Ported from livekit/agents#6148
Original PR description
Solves #6147