Skip to content

fix(multimodal): handle outputless SSE frames and request IDs - #287

Closed
lzsweb wants to merge 3 commits into
mainfrom
dev/issues-12
Closed

lzsweb wants to merge 3 commits into
mainfrom
dev/issues-12

Conversation

@lzsweb

@lzsweb lzsweb commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

Ignore successful usage/search/metadata SSE frames without model output in both multimodal streaming APIs, while retaining output-bearing frames and explicit business errors.

Propagate request IDs from response headers, response bodies, and WebSocket task IDs into HTTP/SSE, WebSocket, audio, and result diagnostics, with an explicit unknown fallback. Add regression coverage for request-ID precedence and outputless events.

Bump the SDK version to 2.23.5 and align the affected Java files with the CI formatter. This branch includes the fixes from #285 plus the formatting correction; #285 remains unchanged.

Test plan

  • Run the full Java formatting check with JDK 8 and google-java-format 1.7: all 557 Java files pass.
  • Run the full Maven test suite with JDK 8: 308 passed, 3 skipped, no failures or errors.
  • Verify the formatting correction changes whitespace only and run git diff --check.
  • Confirm GitHub Actions formatting, test, and build checks pass on this PR.

zhansheng.lzs added 3 commits November 3, 2026 12:38
Propagate request IDs through HTTP/SSE, WebSocket, and audio logs so users can correlate failures with service requests. Add regression coverage for header, body, task ID, and unknown fallbacks.
@lzsweb lzsweb closed this Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant