Andreas/LibreChat
1
0
Fork 0
mirror of https://github.com/danny-avila/LibreChat.git synced 2026-01-03 17:18:51 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

🧠 feat: Enable xhigh reasoning for OpenAI gpt-5.2 (#10924)

Browse source
This commit is contained in:
Alex Ferrari 2025-12-14 08:42:42 +01:00
parent b288d81f5a
commit 2fd2a62886
6 changed files with 76 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

26
packages/api/src/endpoints/openai/llm.spec.ts
View file

@ -210,6 +210,32 @@ describe('getOpenAILLMConfig', () => {
});
});
it('should allow xhigh reasoning_effort for gpt-5.2 models', () => {
const result = getOpenAILLMConfig({
apiKey: 'test-api-key',
streaming: true,
modelOptions: {
model: 'gpt-5.2-pro',
reasoning_effort: ReasoningEffort.xhigh,
},
});
expect(result.llmConfig).toHaveProperty('reasoning_effort', ReasoningEffort.xhigh);
});
it('should drop xhigh reasoning_effort for non-gpt-5.2 models', () => {
const result = getOpenAILLMConfig({
apiKey: 'test-api-key',
streaming: true,
modelOptions: {
model: 'gpt-5-pro',
reasoning_effort: ReasoningEffort.xhigh,
},
});
expect(result.llmConfig).not.toHaveProperty('reasoning_effort');
});
it('should NOT exclude parameters for gpt-5-chat (it supports sampling params)', () => {
const result = getOpenAILLMConfig({
apiKey: 'test-api-key',

12
packages/api/src/endpoints/openai/llm.ts
View file

@ -220,20 +220,24 @@ export function getOpenAILLMConfig({
llmConfig.include_reasoning = true;
}
const allowXHigh = modelOptions.model ? /^gpt-5\\.2/.test(String(modelOptions.model)) : false;
const sanitizedReasoningEffort =
reasoning_effort === 'xhigh' && !allowXHigh ? undefined : reasoning_effort;
if (
hasReasoningParams({ reasoning_effort, reasoning_summary }) &&
hasReasoningParams({ reasoning_effort: sanitizedReasoningEffort, reasoning_summary }) &&
(llmConfig.useResponsesApi === true ||
(endpoint !== EModelEndpoint.openAI && endpoint !== EModelEndpoint.azureOpenAI))
) {
llmConfig.reasoning = removeNullishValues(
{
effort: reasoning_effort,
effort: sanitizedReasoningEffort,
summary: reasoning_summary,
},
true,
) as OpenAI.Reasoning;
} else if (hasReasoningParams({ reasoning_effort })) {
llmConfig.reasoning_effort = reasoning_effort;
} else if (hasReasoningParams({ reasoning_effort: sanitizedReasoningEffort })) {
llmConfig.reasoning_effort = sanitizedReasoningEffort;
}
if (llmConfig.max_tokens != null) {