Fixed
Project:
AG-UI
Version:
1.0.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Apr 2026 at 22:38 UTC
Updated:
4 May 2026 at 21:41 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
b_sharpe commentedComment #4
b_sharpe commentedComment #5
b_sharpe commentedComment #6
rakhimandhania commentedComment #7
ahmad-khalil-imagexComment #8
ahmad-khalil-imagexTested this MR on Drupal 10 with DDEV (nginx + PHP-FPM 8.3) against an AI Assistant routed through
/agui/api/chat. Confirmed the patch behaves as designed:Server-side wiring
StreamedResponseheaders correctly includeContent-Type: text/event-stream,Cache-Control: no-cache,Connection: keep-alive, andX-Accel-Buffering: no.Transfer-Encoding: chunked(noContent-Length), TTFB ~57ms with content download spanning the full 11s of the run — true incremental streaming.Padding / delay knobs
initialPaddingSize=4096produces the leading:SSE-comment with 4096 spaces.chunkPaddingSize=1024produces a:comment between everydata:event, visible in the raw response body.chunkPadding=0(via query string or POST body) removes the inter-event padding lines.chunkDelay=500introduces a clearly visible per-chunk pause; the assistant message paints token-by-token with the configured delay.Abuse caps from the follow-up commit
chunkDelay=99999is correctly clamped to 1000ms server-side.chunkPadding=99999is correctly clamped to 8192 bytes server-side.Both transports
{"chunkDelay":..., "chunkPadding":...}) and the URL query string (?chunkDelay=...&chunkPadding=...).Caveat
RTBC from my perspective.
Comment #10
b_sharpe commented