Pearl th-24cdf3: apply smooth-glow-up (Presence) to th code - #285
Merged
Conversation
- Greeting printed twice on cold start (splash line + an identical System: ChatMessage from app.rs). Removed the duplicate. - Status bar was flat pipe-soup with the health dot buried mid-row. Presence now leads (health glyph opens the line), live state next, static keybindings dimmed + right-aligned; middots replace pipes. - Health state was color-only — three identical dots differing by hue alone, invisible under NO_COLOR/mono/colorblind. Shape now carries it: filled=awake, half=degraded, hollow=unknown. health_glyph() and status_gap() extracted so both properties are unit-tested (shapes must differ; padding saturates instead of underflowing on a tiny terminal). Verified by rendering under tmux at 110/120/80 cols and NO_COLOR=1. 252 tests green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TwUg6nyMqXHzFDsibzTKmp
🦋 Changeset detectedLatest commit: 4a79786 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
brentrager
enabled auto-merge (squash)
July 26, 2026 21:44
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
1 participant
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.
First application of the
smooth-glow-upskill (#282) to a real surface — and it found three things, each traced to a rule in the language rather than to taste.Before / after
What changed
The greeting printed twice on every cold start — the splash renders "Type a message to get started. /help for commands." and
app.rsadded an identicalSystem:message immediately beneath it. Objective defect; duplicate removed.Status-bar hierarchy ("alignment beats ornament"). Identity, metrics, and keybindings were one uniform gray separated by
|, with the health dot buried mid-row. Now presence leads — the health glyph opens the line, since it's Big Smooth's "I'm awake" signal — live state reads next, and the static keybindings are dimmed and right-aligned so alignment separates instead of a wall of pipes.Health state was color-only ("encode state in FORM, not just color"). Three identical
●differing only by hue communicated nothing underNO_COLOR, on a mono terminal, or to the ~8% of users who can't separate green from amber. The shape now carries it:●awake,◐degraded,○unknown.Verification — rendered, not reasoned
Per the skill, captured under tmux at 110, 120 and 80 columns and with
NO_COLOR=1; right-alignment holds at every width with no wrap.health_glyph()andstatus_gap()were extracted so both properties are unit-tested: the three glyphs must differ in shape, and the padding saturates to 0 on a terminal narrower than its content instead of underflowing (the case that would otherwise panic). 252 tests green.Deliberately not changed
The "AI Agent Orchestration Platform" tagline. Presence would argue for something warmer, but that's brand positioning — a product call, not design polish. Flagging rather than changing it unilaterally.
🤖 Generated with Claude Code