Skip to content

[Fix] Terminal views flood browser error telemetry - #1430

Draft
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/terminal-console-errors-3nnsqt0s6kzoo
Draft

[Fix] Terminal views flood browser error telemetry#1430
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/terminal-console-errors-3nnsqt0s6kzoo

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Opened on behalf of @daniel-lxs. Follow up by mentioning @roomote-roomote, in the web UI, or in Slack.

What changed

  • Defer xterm fitting outside ResizeObserver callbacks and coalesce repeated resize notifications into one animation frame.
  • Remove the duplicate initial terminal fit that amplified layout feedback.
  • Enable PostHog client exception capture so replay errors also reach Error Tracking.
  • Cover resize deferral/coalescing and the analytics exception-capture contract with focused client tests.

Why this change was made

A task replay produced an extreme browser error volume while terminal and log views were active. The terminal synchronously changed observed layout from its resize callback, matching a ResizeObserver feedback loop, while exception capture was not enabled for the PostHog client and left Error Tracking without the corresponding diagnostic events.

Impact

Terminal task sessions no longer repeatedly refit during the same resize notification cycle, reducing browser error and replay noise. Future uncaught client failures are also available in PostHog Error Tracking for diagnosis.

@roomote-community

roomote-community Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed bf15f53

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