Problem
On the Chat page, after clicking Approve (or Reject) on a HITL action card, the handlers (frontend/src/pages/chat.tsx) await POST /agents/sessions/{id}/approve but discard the ApprovalResponse and only clear the card. The operator sees nothing — no confirmation on success, and crucially no error when the approved action's execution fails (e.g. create_alias → "Run not found", which the backend returns as status="rejected" + result.error).
Fix
Capture the ApprovalResponse and append a one-line report to the transcript for every outcome:
executed → success- approved but
rejected with result.error → execution failed (show the cause)
rejected (operator declined) → no action takenexpired → approval window lapsed
A pure formatApprovalReport(actionLabel, res) helper (frontend/src/lib/approval-report.ts) is unit-tested across all outcomes.
Acceptance
- Clicking Approve/Reject always surfaces a visible result in chat.
- A failed execution shows the RFC-7807-derived error cause, not silence.
Found while dogfooding the Ollama HITL flow alongside #344.
Problem
On the Chat page, after clicking Approve (or Reject) on a HITL action card, the handlers (
frontend/src/pages/chat.tsx)awaitPOST /agents/sessions/{id}/approvebut discard theApprovalResponseand only clear the card. The operator sees nothing — no confirmation on success, and crucially no error when the approved action's execution fails (e.g.create_alias→ "Run not found", which the backend returns asstatus="rejected"+result.error).Fix
Capture the
ApprovalResponseand append a one-line report to the transcript for every outcome:executed→ successrejectedwithresult.error→ execution failed (show the cause)rejected(operator declined) → no action takenexpired→ approval window lapsedA pure
formatApprovalReport(actionLabel, res)helper (frontend/src/lib/approval-report.ts) is unit-tested across all outcomes.Acceptance
Found while dogfooding the Ollama HITL flow alongside #344.