Skip to content

render: bound streaming code highlights - #1331

Open
simonklee wants to merge 1 commit into
mainfrom
issue-1321/work
Open

render: bound streaming code highlights#1331
simonklee wants to merge 1 commit into
mainfrom
issue-1321/work

Conversation

@simonklee

Copy link
Copy Markdown
Member

Serialize Tree-sitter highlighting per CodeRenderable and coalesce pending updates to the latest snapshot, preventing obsolete streamed content from creating unbounded worker work. Include queued reruns in highlight settlement so rendered and scrollback surfaces converge to current content.

Correlate one-shot worker errors with their message IDs so failed requests reject instead of remaining unresolved.

Fix #1321

Serialize Tree-sitter highlighting per CodeRenderable and coalesce
pending updates to the latest snapshot, preventing obsolete streamed
content from creating unbounded worker work. Include queued reruns in
highlight settlement so rendered and scrollback surfaces converge to
current content.

Correlate one-shot worker errors with their message IDs so failed
requests reject instead of remaining unresolved.

Fix #1321
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.

Possible Memory leak: CodeRenderable floods the tree-sitter worker during streaming, causing unbounded memory growth

1 participant