agentjido / req_llm

50%

LAST BUILD ON BRANCH main

branch: SELECT

CHANGE BRANCH

x

Sync Branches

• No branch selected
• add-vllm-provider
• bedrock-clean
• bug/api-return-types
• bug/codec-tool-calls
• bug/debug-stream-return
• bug/incorrect-model-spec-docs
• bug/openai-tool-calls
• bug/streaming-race-condition
• cerebras
• chore/2025-10-14-update-fixtures
• chore/object-fixtures
• chore/object-fixtures-resurrected
• chore/refine-fixtures
• chore/refresh-coverage-tests
• chore/update-models-2025-09-21
• copilot/fix-32
• devtools
• egomes/fix-claude-multi-turn
• feat/context-json-serialization
• feat/google-upload-file
• feat/in-type-support
• feat/structured-output-openai-google
• feature/cerebras-provider
• feature/normalize-bedrock-inference-profiles
• feature/pre-release-fixes
• feature/refactor-llm-api-fixtures
• feature/refined-key-management
• feature/unique-model-provider-options
• feature/zai-fixtures
• fix-anthropic-streaming
• fix-duplicate-clause
• fix-google
• fix-groq-stream-error
• fix-mix-task-docs
• fix-openai-max-tokens-param
• fix/cost-calculation-in-usage
• fix/google-file-support
• fix/google-structured-output
• fix/issue-65-http-status-validation
• fix/issue-96-validation-error-fields
• fix/proxy-options
• fix/registry-get-provider-nil-module
• fix/tool_calls
• google-vision
• improve-metadata-provider-errors
• main
• patch-1
• put-max-tokens-model-options
• refactor/context-tools
• refactor/req-streaming
• refactor/xai-structured-objects
• zai

Build # 2efaac748d0d8b3d0ad85c2efe8179a44c2a7091

Build Type

push

github

Committed by web-flow

Commit Message

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>

Run Details

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