agentjido / req_llm / 59946ee0fb9c2267927f5c80a45ed9d0dce5b65d-PR-108

50%

main: 49%

Build Type

Pull #108

github

Committed by mikehostetler

Commit Message

Fix #65: Add HTTP status validation to StreamServer

- Check HTTP status code before processing data chunks
- Build proper API error from error JSON response body
- Halt stream with error when status >= 400
- Prevents error JSON from being passed to SSE parser

Previously, HTTP error responses (4xx/5xx) were being passed to the
SSE parser which couldn't parse them, resulting in empty streams with
no error information. Now errors are properly captured and propagated
through the metadata task.

Amp-Thread-ID: https://ampcode.com/threads/T-2bb95463-44a7-4d42-887c-2c7d24d5155a
Co-authored-by: Amp <amp@ampcode.com>

Pull Request Pull Request #108: Fix #65: Add HTTP status validation to StreamServer

Run Details

4 of 14 new or added lines in 2 files covered. (28.57%)

175 existing lines in 3 files now uncovered.

3275 of 6516 relevant lines covered (50.26%)

445.83 hits per line