Skip to content

Commit 1000112

Browse files
Update simple_browser_tool.py (#40)
* Update simple_browser_tool.py Fixed a typo in variable name: lenghts -> lengths * Update api_server.py Added validation to get_reasoning_effort, documenting the function and raising a clear ValueError when an unsupported effort string is provided instead of returning None Updated the /v1/responses endpoint to catch invalid reasoning effort inputs and return an HTTP 422 error, preventing server-side crashes
1 parent 359b3ff commit 1000112

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

gpt_oss/responses_api/api_server.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,11 @@
6464
def get_reasoning_effort(effort: Literal["low", "medium", "high"]) -> ReasoningEffort:
6565
if effort == "low":
6666
return ReasoningEffort.LOW
67-
elif effort == "medium":
67+
if effort == "medium":
6868
return ReasoningEffort.MEDIUM
69-
elif effort == "high":
69+
if effort == "high":
7070
return ReasoningEffort.HIGH
71+
raise ValueError(f"Invalid reasoning effort: {effort}")
7172

7273

7374
def is_not_builtin_tool(recipient: str) -> bool:
@@ -784,7 +785,12 @@ def _ensure_list(inp):
784785
)
785786

786787
if body.reasoning is not None:
787-
reasoning_effort = get_reasoning_effort(body.reasoning.effort)
788+
try:
789+
790+
reasoning_effort = get_reasoning_effort(body.reasoning.effect)
791+
except ValueError as e:
792+
from fastapi import HTTP Exception
793+
raise HTTPException(status_code=422, detail=str(e))
788794
system_message_content = system_message_content.with_reasoning_effort(reasoning_effort)
789795

790796
if use_browser_tool:

0 commit comments

Comments
 (0)