Skip to content

fix(sidebar): avoid re-renders on streaming updates - #158

Merged
GoDiao merged 2 commits into
GoDiao:mainfrom
tedy69:fix/sidebar-running-session-subscription-157
Oct 22, 2026
Merged

GoDiao merged 2 commits into
GoDiao:mainfrom
tedy69:fix/sidebar-running-session-subscription-157

Conversation

@tedy69

@tedy69 tedy69 commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Streaming text no longer re-renders the sidebar. It now subscribes only to running session IDs, while indicators still follow chat and tab status.

Adds regression tests for streamed updates and starting and stopping sessions. Closes #157.

Desktop tests, typecheck, frontend build, and browser profiling pass. Server tests pass locally with inherited Anthropic overrides unset. CI has six MCP/H5 test failures that also occur on main.

Written with help from OpenAI Codex.

@GoDiao
GoDiao merged commit 30eb14e into GoDiao:main Oct 22, 2026
4 checks passed
GoDiao added a commit that referenced this pull request Oct 22, 2026
@GoDiao

GoDiao commented Oct 22, 2026

Copy link
Copy Markdown
Owner

Merged, thanks @tedy69! Streaming no longer re-renders the whole sidebar, and the two Profiler tests pin the behavior down nicely. The Server tests failure wasn't yours: main had a test leak that's fixed now, so I updated your branch and CI went green. I also tweaked the CHANGELOG entry to our contributor format. If you enjoy DreamCoder, a ⭐ on the repo would mean a lot to us. Hope to see more PRs from you!

@GoDiao GoDiao mentioned this pull request Oct 25, 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.

Re-render the sidebar only when the set of running sessions changes

2 participants