When a model config includes `reasoning_effort` as an extra YAML field
(ModelConfig uses `extra="allow"`), and the thinking-disabled code path
also injects `reasoning_effort="minimal"` into kwargs, the previous
`model_class(**kwargs, **model_settings_from_config)` call raises:
TypeError: got multiple values for keyword argument 'reasoning_effort'
Fix by merging the two dicts before instantiation, giving runtime kwargs
precedence over config values: `{**model_settings_from_config, **kwargs}`.
Fixes #1977
Co-authored-by: octo-patch <octo-patch@github.com>
|
||
|---|---|---|
| .. | ||
| __init__.py | ||
| claude_provider.py | ||
| credential_loader.py | ||
| factory.py | ||
| openai_codex_provider.py | ||
| patched_deepseek.py | ||
| patched_minimax.py | ||
| patched_openai.py | ||
| vllm_provider.py | ||