Skip to content

update to include reflow#795

Open
taigrr wants to merge 7 commits intocharmbracelet:mainfrom
taigrr:feat/softwrap
Open

update to include reflow#795
taigrr wants to merge 7 commits intocharmbracelet:mainfrom
taigrr:feat/softwrap

Conversation

@taigrr
Copy link
Copy Markdown
Contributor

@taigrr taigrr commented Mar 5, 2026

  • I have read CONTRIBUTING.md.
  • I have created a discussion that was approved by a maintainer (for new features).

Comment thread vt/screen.go Outdated
Comment thread vt/screen.go Outdated
Comment thread vt/screen.go Outdated
Comment thread vt/screen.go Outdated
Comment thread vt/scrollback.go Outdated
Comment thread vt/scrollback.go Outdated
Comment thread vt/scrollback.go Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 68.06283% with 61 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.54%. Comparing base (fe36e8c) to head (59eda50).

Files with missing lines Patch % Lines
vt/screen.go 68.63% 36 Missing and 17 partials ⚠️
vt/emulator.go 45.45% 5 Missing and 1 partial ⚠️
vt/scrollback.go 77.77% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #795      +/-   ##
==========================================
+ Coverage   36.42%   39.54%   +3.11%     
==========================================
  Files         209       21     -188     
  Lines       17591     2003   -15588     
==========================================
- Hits         6408      792    -5616     
+ Misses      10788     1102    -9686     
+ Partials      395      109     -286     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

taigrr added 4 commits March 5, 2026 19:51
Remove ScrollbackLine struct - Scrollback now stores []uv.Line with a
parallel []bool for soft-wrap state. Remove entries() and replaceLines()
methods; Screen.reflowScrollback() accesses the fields directly since
it owns the reflow logic.

Addresses review feedback from @aymanbagabas.
@meowgorithm
Copy link
Copy Markdown
Member

@aymanbagabas is this one good to go?

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.

3 participants