Fix skills+plugins section collapse and loc/cost label visibility - #121
Open
tmck-code wants to merge 2 commits into
Open
Fix skills+plugins section collapse and loc/cost label visibility#121tmck-code wants to merge 2 commits into
tmck-code wants to merge 2 commits into
Conversation
The skills+plugins trailing column vanished entirely when empty instead of rendering as a blank bordered section, dragging the loc r/w and cost sections' borders/labels along with it. Separately, the loc r/w and cost labels silently disappeared with show_icons=False because label detection sniffed for an icon glyph instead of using an explicit flag. Also centers the loc r/w label within its section using the abbreviated label form actually rendered. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
tmck-code
marked this pull request as ready for review
August 19, 2026 14:33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
Fixes three related statusline rendering bugs surfacing across width/config combinations users actually hit:
skills + pluginstrailing column vanished entirely (instead of rendering as an empty bordered section) when there were no skills/plugins loaded, which dragged the borders and labels for the neighboringloc r/wandcost sess/daysections along with it.loc r/wandcost sess/daylabels silently disappeared from the border row whenshow_icons=False, because label detection sniffed for an icon glyph that's gated onshow_iconsinstead of using an explicit flag.loc r/wlabel wasn't centered within its section — it centered against the long label form's width instead of the abbreviated form actually rendered.Also adds a new optional "tokens over time" full-width row (off by default,
YAS_SHOW_TOKENS_OVER_TIME).Bumps version to 0.7.2 (patch — bug fixes only, no new user-facing features).
Changes
Renderer / layout
claude/yas/renderer.py—tokens_costno longer gates the skills+plugins trailing column on non-empty content; it now always renders (blank-padded) when width allows. Also returns an explicithas_linesflag instead of forcing callers to re-derive presence from rendered glyphs.claude/yas/layout.py— consumeshas_linesdirectly for the loc r/w label anchor; centers the loc r/w label using the abbreviated label form's width (matching what_fit_labelactually renders) instead of the long form's width.claude/yas/render/text.py,claude/yas/config.py,claude/yas/constants.py— support for the new tokens-over-time row and related label constants.Tests
test_tokens_cost.py,test_layout_seam.py,test_labels_layout.py,test_tool_counts_row.py,test_config.pyfor the new column/border layout.test/test_tokens_over_time.pyfor the new row.test/fixtures/claude_dark_wide.ansisnapshot to match the corrected renderer output.Checklist
Screenshots / recording
Benchmark
mainPRSystem info