fix(ai-studio): live execution progress behind buffering proxies + visualize copy feedback#62
Merged
Merged
Conversation
A fronting proxy that buffers the stream delivers every event in one burst at stream close, so live node progress renders only after the run finishes. Send X-Accel-Buffering: no from the route and re-emit it from the deploy nginx (nginx hides upstream X-Accel-* headers), so a TLS-terminating proxy in front of the stack sees it too.
The visualize copy buttons gave no feedback at all, so a successful copy looked broken. Copy image now flips to a checkmark for 1.5s and is disabled while AI adaptation is in flight (its target element does not exist yet, so clicking was a silent no-op). Copy source is removed from both the card and the modal.
librowski
requested review from
lukasz-jazwa,
piotrblaszczyk and
szymon-t-sc
as code owners
kbcki
approved these changes
Jul 3, 2026
szymon-t-sc
approved these changes
Jul 3, 2026
One clipboard entry with both image/png and text/plain: pasting yields the image where images are accepted and the raw result text in text-only fields. Copying only the image looked broken when pasted into a text field.
Palette drops get mode: 'auto' from defaultPropertiesData, but the hand-authored template nodes omitted it, so their Render as select showed an empty value.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two follow-up fixes to #48.
Live execution feedback (SSE buffering)
On the deployed demo, node highlighting and the execution log stayed empty during a run and everything appeared at once when it finished. Measured cause: a proxy hop in front of the stack buffers the SSE response and flushes it only at stream close - the frontend receives every event in a single burst after the run completes. Locally (no proxy) the same build streams progressively.
X-Accel-Buffering: noon/api/executions/:id/streamX-Accel-*headers by default, so without the re-emit a TLS-terminating proxy in front of thewebcontainer would never see itNeeds a
backend+webredeploy to take effect.Visualize copy buttons
Copying worked but gave zero feedback, so it looked broken.
copySourceutilCopyImageButtoncomponent with a spec covering feedback, the download fallback, and the missing-target case