Skip to content

Forward buttons in /orchestrate/test Conversion Block #362

@nuwangeek

Description

@nuwangeek

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:

  1. After Task-02 is complete, add buttons=response.buttons to this copy block.
  2. This is a one-line change but must not be forgotten — without it, the test endpoint silently drops button data.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

Status

Todo

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions