fix: remove redundant instructions override in run_inference

The override would re-add `instructions` after the adapter had
intentionally converted it to a developer message for empty contexts.
Added a regression test.
This commit is contained in:
Paul Kompfner
2026-03-19 13:34:41 -04:00
parent b1a8588209
commit 348df9d4ce
2 changed files with 33 additions and 4 deletions

View File

@@ -385,10 +385,6 @@ class OpenAIResponsesLLMService(LLMService):
# Override for non-streaming
params["stream"] = False
# Override instructions if caller provided one explicitly
if system_instruction is not None:
params["instructions"] = system_instruction
if max_tokens is not None:
params["max_output_tokens"] = max_tokens