agentjido / req_llm / 2efaac748d0d8b3d0ad85c2efe8179a44c2a7091

49%

push

github

Fix #65: Add HTTP status validation to StreamServer (#109)

* 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>

* Add test coverage for HTTP error status code handling

- Test 401 authentication errors with JSON error response
- Test 5xx server errors
- Test non-JSON error responses
- Verify error details are properly extracted and returned

* Formatting

---------

Co-authored-by: Amp <amp@ampcode.com>

10 of 12 new or added lines in 1 file covered. (83.33%)

3285 of 6516 relevant lines covered (50.41%)

447.04 hits per line