ui: revert mascot from pixel-art crab to simple 3-line bot#116
Merged
emal-avala merged 1 commit intomainfrom Apr 15, 2026
Merged
ui: revert mascot from pixel-art crab to simple 3-line bot#116emal-avala merged 1 commit intomainfrom
emal-avala merged 1 commit intomainfrom
Conversation
The welcome banner now renders a minimal 3-line robot head (▐▛██▜▌ / ▝▜██▛▘ / ▘▘) in the current theme accent color, matching the original design from before commit aad7893 replaced it with a 14-row pixel-art crab plus shimmer animation. The crab used hardcoded #A422E1 purple and ignored the user's theme, and the shimmer added 360ms of startup delay (3 x 120ms frames) plus ~220 lines of pixel/color plumbing. The simple bot follows the theme, is instant, and matches the minimal aesthetic of the rest of the REPL. Removed from tui.rs: render_crab_banner, render_crab_shimmer, render_half_block, rgb_fg, rgb_bg — all crab-exclusive helpers. Net -245 lines.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
emal-avala
added a commit
that referenced
this pull request
Apr 15, 2026
Two formatting issues slipped through on PR #116 and are breaking CI on every branch off main: - crates/cli/src/ui/repl.rs:412-416 — the last println! in the bot mascot block fits on one line; rustfmt collapses it. - crates/cli/src/ui/tui.rs — sed left a double newline at EOF after removing the crab block. rustfmt wants a single trailing newline. Both fixes are mechanical and match what cargo fmt would produce.
3 tasks
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.
Summary
Replaces the welcome-banner mascot (pixel-art crab + shimmer animation) with the simple 3-line robot head that existed before commit
aad7893.Why
#A422E1purple and ignored the user's theme. The bot now usest.accent, so it matches whatever theme is configured.❯, dividers, status line) is one-line and subtle. The 14-row crab was visually inconsistent.The mascot
Three lines of Unicode half-blocks, rendered with
t.accentbold on the top row andt.accentregular on the lower rows.Cleanup
Also deleted the following dead functions from
tui.rs(the crab was the only consumer):Test plan
cargo check/cargo clippy(will be verified by CI)