From 806a581ff1f45a3715afd40ddc05b1f59c8aef88 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rados=C5=82aw=20Stachowiak?= Date: Sun, 21 Dec 2025 23:32:28 +0100 Subject: [PATCH] Fix reasoning_effort option not being passed to API The reasoning_effort option was defined in ReasoningOptions but never extracted in _build_kwargs, so it was silently ignored when using -o reasoning_effort with reasoning models like gpt-5.1. Also added 'minimal' to ReasoningEffortEnum to match the main llm library's enum values. --- llm_openai.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/llm_openai.py b/llm_openai.py index 4013c31..53bb652 100644 --- a/llm_openai.py +++ b/llm_openai.py @@ -77,6 +77,7 @@ class ImageDetailEnum(str, Enum): class ReasoningEffortEnum(str, Enum): + minimal = "minimal" low = "low" medium = "medium" high = "high" @@ -281,6 +282,7 @@ def _build_kwargs(self, prompt, conversation): "top_p", "store", "truncation", + "reasoning_effort", ): value = getattr(prompt.options, option, None) if value is not None: