Skip to content

perf: bound mobile terminal WebView queues#4100

Merged
nwparker merged 1 commit into
mainfrom
nwparker/perf-mobile-terminal-webview-queues
May 31, 2026
Merged

perf: bound mobile terminal WebView queues#4100
nwparker merged 1 commit into
mainfrom
nwparker/perf-mobile-terminal-webview-queues

Conversation

@nwparker

Copy link
Copy Markdown
Contributor

Summary

  • bound pre-ready mobile terminal WebView write backlog by bytes and message count while preserving queued control messages
  • drain embedded xterm writes with an indexed queue instead of Array.shift()
  • dispose old xterm observer subscriptions on terminal replacement and clear successful ready/measure fallback timers

Validation

  • pnpm test (mobile)
  • pnpm exec tsc --noEmit (mobile)
  • pnpm lint (mobile)
  • git diff --check

Risk: low. The patch keeps the existing write order for retained chunks and intentionally leaves the after-drain readiness barrier behavior unchanged.

@nwparker nwparker added the risk:medium Medium risk change label May 31, 2026
@nwparker
nwparker merged commit 81f4559 into main May 31, 2026
3 checks passed
@nwparker
nwparker deleted the nwparker/perf-mobile-terminal-webview-queues branch May 31, 2026 11:23
@nwparker nwparker added risk:low Low risk change risk:medium Medium risk change and removed risk:medium Medium risk change risk:low Low risk change labels May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:medium Medium risk change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant