-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Description
Priority: P4 — Correctness fix in the API layer
File: src/llm_orchestration_service_api.py
Handler: test_orchestrate_llm_request()
Type: Bug fix (future)
The explicit field copy from OrchestrationResponse to TestOrchestrationResponse must include buttons once that field exists (added in Task-02):
test_response = TestOrchestrationResponse(
llmServiceActive=response.llmServiceActive,
questionOutOfLLMScope=response.questionOutOfLLMScope,
inputGuardFailed=response.inputGuardFailed,
content=response.content,
buttons=response.buttons, # <-- add this
chunks=None,
)
Work:
- After Task-02 is complete, add buttons=response.buttons to this copy block.
- This is a one-line change but must not be forgotten — without it, the test endpoint silently drops button data.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
Todo