Skip to content

feat(tui): swap spinners to cli-spinners sand and aesthetic - #129

Merged
quickbeard merged 3 commits into
mainfrom
new-spinners
Jul 29, 2026
Merged

quickbeard merged 3 commits into
mainfrom
new-spinners

Conversation

@quickbeard

Copy link
Copy Markdown
Owner

What

Replaces both TUI spinners with definitions from cli-spinners (MIT):

  • Thinking/tool spinner (component/spinner.tsx): SPINNER_FRAMES swapped from the 10-frame braille "dots" to the 35-frame "sand" falling-braille animation. Interval already matched at 80 ms. The shared Spinner component also drives tool rows, startup loading, and session-list dialogs, so the whole TUI switches consistently.
  • Busy footer spinner (component/prompt/index.tsx): the Knight Rider block scanner is replaced with the 7-char "aesthetic" ▰▱ progress ramp at 80 ms (was 40). spinnerDef keeps the agent-color resolution — ▰ renders in the agent accent color, ▱ in a dimmed variant via the already-exported deriveInactiveColor. ui/spinner.ts itself is untouched.

The animations-off fallbacks (⋯ / [⋯]) and the busy-state gating are unchanged. Frames are vendored verbatim with an MIT source comment.

Bookkeeping

  • Two new DIVERGENCES.md rows (one per upstream file) with on-conflict guidance.
  • CHANGELOG.md Unreleased entry under TUI.

Verification

  • bun typecheck passes in packages/tui.
  • Live tmux run: footer ramp animates through all fill levels (▰▱▱▱▱▱▱ → ▰▰▰▰▰▰▰) in the agent color next to esc interrupt; thinking rows cycle sand frames (⠁, ⠂).

🤖 Generated with Claude Code

quickbeard and others added 3 commits July 29, 2026 15:44
The shared Spinner component (thinking/tool rows, startup loading,
session-list dialogs) now animates cli-spinners' 35-frame "sand"
braille set instead of "dots". The busy footer spinner replaces the
Knight Rider block scanner with the "aesthetic" ▰▱ ramp at 80ms,
keeping the agent accent color for filled blocks and a dimmed variant
(deriveInactiveColor) for unfilled ones. ui/spinner.ts is untouched.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@quickbeard
quickbeard merged commit a600634 into main Jul 29, 2026
0 of 5 checks passed
@quickbeard
quickbeard deleted the new-spinners branch July 29, 2026 08:52
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