Skip to content

Commit e0ce0d8

Browse files
daniel-rudaevclaude
andcommitted
fix(mcp): change reasoning.effort from 'minimal' to 'low' in factory
The actual hardcoded 'minimal' value was in factories.py line 136, not in graphiti-core's DEFAULT_REASONING constant. The factory explicitly passes reasoning='minimal' when creating OpenAIClient for reasoning models, overriding any default. 'minimal' is not supported by gpt-5.4-mini and newer models. Supported values: 'none', 'low', 'medium', 'high', 'xhigh'. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent dc9ee26 commit e0ce0d8

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

mcp_server/src/services/factories.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ def create(config: LLMConfig) -> LLMClient:
133133

134134
# Only pass reasoning/verbosity parameters for reasoning models (gpt-5 family)
135135
if is_reasoning_model:
136-
return OpenAIClient(config=llm_config, reasoning='minimal', verbosity='low')
136+
return OpenAIClient(config=llm_config, reasoning='low', verbosity='low')
137137
else:
138138
# For non-reasoning models, explicitly pass None to disable these parameters
139139
return OpenAIClient(config=llm_config, reasoning=None, verbosity=None)

0 commit comments

Comments
 (0)