fix(sse): propagate malformed Anthropic JSON - #329
Conversation
|
Warning Review limit reachedNext included review available in 11 minutes. View limit detailsLimit details: You’ve used the included review currently available. This review ran on the open-source allowance, not this organization's plan, because the pull request author doesn't have an assigned seat. Waiting won't change this — ask an organization admin to assign them a seat, or add seats in Billing if every seat is already assigned, then retry. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
Comment |
There was a problem hiding this comment.
All reported issues were addressed across 2 files
Shadow auto-approve: would not auto-approve because issues were found.
Re-trigger cubic
|
❌ The last analysis has failed. |
There was a problem hiding this comment.
0 issues found across 2 files (changes from recent commits).
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Shadow auto-approve: would auto-approve. This PR fixes a bug where malformed Anthropic SSE JSON was silently ignored and instead propagates a provider-specific parse error, with parser and end-to-end regression tests. The change preserves non-delta/message_start behavior, adds null handling, and is tightly scoped to parser error handling.
Re-trigger cubic
Summary
message_startandcontent_block_deltaSSE framesFixes #307
Verification
maingithub.com; repository PR checks should provide the executable verification