Skip to content

cmd/sgai: keep live workspace detail tabs responsive under load#23

Merged
ucirello merged 1 commit into
mainfrom
push-nyzwtktptlys
Mar 31, 2026
Merged

cmd/sgai: keep live workspace detail tabs responsive under load#23
ucirello merged 1 commit into
mainfrom
push-nyzwtktptlys

Conversation

@ucirello

@ucirello ucirello commented Mar 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • split workspace detail refresh into per-surface API endpoints and revision tokens so SSE-driven state updates only refetch the active tab instead of reloading every detail payload
  • cache workspace detail subscriptions with useSyncExternalStore and keep event, message, and log tabs on bounded tail windows so live churn preserves focus and avoids runaway rendering
  • add Go, React, and Playwright high-load coverage around selective revisions, stable tail rendering, and editor usability while unrelated updates continue streaming, and absorb the updated browser-bug-testing workflow into cmd/sgai/skel/.sgai

@ucirello ucirello left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ucirello

Copy link
Copy Markdown
Owner Author

I checked the failing run at https://github.com/ucirello/sgai/actions/runs/23621220941. Both jobs are red because make test fails in the React suite; the actionable failure is the Dashboard test that times out waiting for [data-testid="redirect-target"] in cmd/sgai/webapp/src/pages/__tests__/Dashboard.test.tsx:922. I am keeping the PR in draft; no PR metadata changes looked necessary beyond recording the current state.

@ucirello ucirello force-pushed the push-nyzwtktptlys branch 2 times, most recently from 2176a44 to 5c36f2f Compare March 27, 2026 14:18
@ucirello ucirello marked this pull request as ready for review March 27, 2026 14:18
@ucirello ucirello force-pushed the push-nyzwtktptlys branch 10 times, most recently from 384bc49 to e019a08 Compare March 31, 2026 19:13
@ucirello ucirello force-pushed the push-nyzwtktptlys branch from e019a08 to 14001aa Compare March 31, 2026 21:43
@ucirello ucirello marked this pull request as draft March 31, 2026 21:44
@ucirello ucirello marked this pull request as ready for review March 31, 2026 23:48
@ucirello ucirello merged commit fc7e08a into main Mar 31, 2026
2 checks passed
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