Add new ansi theme - #35
Conversation
Reviewer's GuideAdds a ninth built-in Flow diagram for ANSI theme color selectionflowchart LR
Theme[ansi theme] --> UsesANSI[usesANSI]
UsesANSI -->|true| ANSI[ansiStop terminal palette index]
UsesANSI -->|false| RGB[Existing RGB gradient]
ANSI --> Metrics[Download and upload colors]
ANSI --> Borders[Download and upload borders]
ANSI --> Logo[Logo and animated accents]
RGB --> Metrics
RGB --> Borders
RGB --> Logo
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe theme system adds a built-in ChangesANSI theme support
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This change adds an ANSI-based visual theme while preserving existing RGB themes and does not alter application services, data, permissions, or deployment behavior. No actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 10.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 10 functions across 1 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
wow, looks beautiful, thanks, ill merge it asap!!! |
|
merged, thanks for the contrib! |
Summary
Adds a simple new theme that utilizes the existing terminal ANSI colors instead of hard code values.
Type of change
Checklist
make checkpasses locally with zero warnings (make fmt && make vet && make lint && make test)Testing done
Built and ran on MacOS 26.6.2
Screenshots / recordings
Summary by Sourcery
Add an ANSI theme that makes the application follow the terminal’s configured color palette.
New Features:
ansibuilt-in theme that uses the terminal’s 16-color palette for text, gradients, borders, and logo rendering.Enhancements:
Documentation:
Summary by CodeRabbit