Skip to content

feat(leantui): remove context usage progress bar - #4157

Merged
rumpl merged 1 commit into
docker:mainfrom
rumpl:feat/leantui-remove-context-bar
Sep 4, 2026
Merged

feat(leantui): remove context usage progress bar#4157
rumpl merged 1 commit into
docker:mainfrom
rumpl:feat/leantui-remove-context-bar

Conversation

@rumpl

@rumpl rumpl commented Sep 3, 2026

Copy link
Copy Markdown
Member

What

Removes the context-usage progress bar (██████░░░░) from the lean TUI footer.

The second footer line goes from:

██████░░░░ 60% · 120.0k/200.0k · $0.05        openai/gpt-5 · high

to:

60% · 120.0k/200.0k · $0.05                   openai/gpt-5 · high

Details

  • Dropped ui.RenderBar and the ContextBarWidth constant.
  • The auto-compaction threshold still drives color escalation — it now colors the context text (normal → muted, warning, critical) instead of the gauge, so StatusModel.CompactionThreshold stays meaningful.
  • Compaction state is unchanged: compacting… · 9.0k/10.0k.

Tests

The scattered Contains/NotContains assertions on RenderContext are replaced by one table test asserting the exact ANSI-stripped output for every state (no usage, known limit, over-limit clamp, tokens without a limit, compacting with and without a limit, cost), plus a small test pinning the color-escalation bands.

Validation

go build ./pkg/..., go vet ./pkg/leantui/..., go test ./pkg/leantui/... ./pkg/tui/... — all green.

@rumpl
rumpl requested a review from a team as a code owner September 3, 2026 22:16
The footer gauge duplicated information already spelled out next to it, so
drop it and keep the percentage, token counts and cost. The compaction
threshold now colors the context text instead of the bar.
@rumpl
rumpl force-pushed the feat/leantui-remove-context-bar branch from 34a8ab0 to 669c955 Compare September 3, 2026 22:19
@rumpl
rumpl enabled auto-merge September 3, 2026 22:24
@aheritier aheritier added area/testing Test infrastructure, CI/CD, test runners, evaluation area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only. labels Sep 3, 2026
@rumpl
rumpl merged commit 296ef8c into docker:main Sep 4, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/testing Test infrastructure, CI/CD, test runners, evaluation area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants