Skip to content

fix(tui): replace stream badge glyph U+26A1 with plain » - #116

Merged
jkyberneees merged 1 commit into
mainfrom
fix/stream-badge-glyph
Sep 12, 2026
Merged

fix(tui): replace stream badge glyph U+26A1 with plain »#116
jkyberneees merged 1 commit into
mainfrom
fix/stream-badge-glyph

Conversation

@jkyberneees

Copy link
Copy Markdown
Contributor

What

The ⚡ stream badge (U+26A1) never painted on screen in some terminals while its bytes were verifiably emitted (clipboard contained it; a plain printf ⚡ rendered fine). U+26A1 is East-Asian-ambiguous and emoji-presentation-capable — the exact glyph class that mis-widths or mis-paints inside styled runs.

Swapped to » (U+00BB, single-cell everywhere) in:

  • header model badge (view.go)
  • cockpit stream row (cockpit.go)
  • /server link row (commands.go — caught by expert review, initial swap missed it)
  • tests (spec_v2_test.go, cockpit_test.go) and README

The latency-stat ⚡ (statTime) and the odek-banner-matching ⚡ in internal/server are deliberately untouched: different surface / external protocol.

Verification

  • make fmt / make vet / make lint — 0 issues
  • make test — full race suite green
  • Expert review (2 adversarial reviewers): both flagged the two leftovers now fixed; header/compactHeader clip-safety verified

The ⚡ stream badge (U+26A1) is East-Asian-ambiguous and
emoji-presentation-capable, so several terminals mis-width or mis-paint
it inside styled runs: the badge bytes are emitted (visible when copied)
but never painted on screen. Swap to » (U+00BB), single-cell everywhere,
in the header badge, cockpit stream row, and /server link row; update
the affected tests and README.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 12, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
bodek 22e1fbe Commit Preview URL

Branch Preview URL
Sep 12 2026, 03:24 PM

@jkyberneees
jkyberneees merged commit c88e606 into main Sep 12, 2026
9 checks passed
@jkyberneees
jkyberneees deleted the fix/stream-badge-glyph branch September 12, 2026 15:26
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.

1 participant