Skip to content

fix(tui): bound terminal frame symbols - #104

Merged
m1ngsama merged 1 commit into
mainfrom
agent/issue-87-symbol-budget
Aug 11, 2026
Merged

m1ngsama merged 1 commit into
mainfrom
agent/issue-87-symbol-budget

Conversation

@m1ngsama

Copy link
Copy Markdown
Owner

Summary

  • include heap-backed Cell.symbol allocations in the conservative 64 MiB terminal-frame accounting budget
  • make the real body/chrome/help/tiny render path fallible and charge actual symbols before publication
  • use Terminal::try_draw so rejected frames do not draw, flush, or swap, while preserving backend I/O errors
  • document the corrected limit and add deterministic capacity, atomicity, retry, and maximum-geometry coverage

Closes #87.

Verification

  • current Rust strict Clippy and full tests: 374 unit + 28 CLI/PTY + 5 documentation
  • Rust 1.88 strict Clippy and full tests
  • overflow-checked release full tests
  • cargo fmt --check, git diff --check, markdownlint, mandoc -T lint
  • dirty-package equivalent of make distcheck, including install/installcheck/uninstall
  • independent review: no blocker

@m1ngsama
m1ngsama marked this pull request as ready for review August 11, 2026 07:54
@m1ngsama
m1ngsama merged commit 766dfef into main Aug 11, 2026
3 checks passed
@m1ngsama
m1ngsama deleted the agent/issue-87-symbol-budget branch August 11, 2026 07:57
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.

fix(tui): account for heap-backed cell symbols in the terminal budget

1 participant