Open
Conversation
jamby77
approved these changes
Apr 2, 2026
* Converting to color palette from the website * roborev fixes * fixed bugbot findings * fixed bugbot link finding * fixed dialog issue * fixed bugbot issue for canvas colors * color selection fixed * dark theme + tests (#86) * dark theme + tests * fixed bugbot finding * fixed bug bot finding for flashing theme
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| ? 'bg-red-100 text-red-700' | ||
| : 'bg-gray-100 text-gray-700'; | ||
| ? 'bg-muted text-foreground' | ||
| : 'bg-muted text-foreground'; |
There was a problem hiding this comment.
Redis and Unknown badges now visually identical
Medium Severity
The DbBadge component's ternary for colorClass now produces the same value 'bg-muted text-foreground' for both the redis and fallback (Unknown) branches. Previously Redis had a distinct red badge (bg-red-100 text-red-700) while Unknown had gray (bg-gray-100 text-gray-700). Users can no longer visually distinguish Redis databases from unknown database types in the migration summary view.
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
Fixed Shadcn setup/init. This is a PR with 2 branches following on it and merging to it
Checklist
- [ ] Unit / integration tests addedN/A- [ ] Docs added / updatedroborev review --branchor/roborev-review-branchin Claude Code (internal)- [ ] Competitive analysis done / discussed (internal)- [ ] Blog post about it discussed (internal)Note
Medium Risk
Introduces new theme persistence and pre-paint dark-mode handling, plus refactors key UI flows (connection dialogs/selects) onto new shadcn/radix primitives; regressions would mainly be in client-side UI state, styling, and accessibility.
Overview
Adds a shadcn-based UI foundation: new
components.json, new shadcn/radix primitives (button,dialog,select,switch,alert), updated dependencies (Radix, Inter font, animations), and a new tokenized theme inindex.css(including chart color variables).Implements dark mode support with a new
useThemehook (localStorage + system preference listener), a sidebarModeToggle, and anindex.htmlinline script to apply thedarkclass before first paint.Refactors
ConnectionSelectorto use shadcnSelect/Dialogcomponents and resets form + test state on dialog close; adds targeted tests for the cancel/reset behavior and theme toggle/hook behavior. Large-scale UI color/styling updates replace hardcoded grays/blues/reds with semantic Tailwind tokens (primary,muted,destructive) and CSS variables across dashboards, cluster views, migration, and webhooks.Written by Cursor Bugbot for commit 2b60599. This will update automatically on new commits. Configure here.