You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Restyles the entire Proddy frontend to a clean fintech-dashboard aesthetic (à la the Fundora/Finexy reference dashboards): soft-gray app canvas, white rounded-2xl cards with 1px borders + soft shadows, pill controls, stat tiles with delta chips, clean roomy tables, and quiet CSS-first micro-animations — keeping the existing primary purple (#4A0D68) as the single interactive accent.
Delivered via a foundation phase + 19 file-disjoint restyle packages across 3 waves, each gated with bun run type + biome check + next build.
Foundation (STYLE-GUIDE.md is the contract)
Tokens: soft-gray --background, white --card, new --success/--warning/--sidebar*, purple focus --ring, 12px radius, softer shadows; coherent .dark remap (light-first).
Font: Inter via next/font (Poppins was referenced but never loaded).
Primitives: all 29 ui/* → pills, rounded-2xl cards (+interactive hover-lift), soft-focus bg-card inputs, soft-tint badges, roomy tables; switch/checkbox/slider off magenta.
--secondary (magenta) is demoted to an AI-accent/marketing garnish. Third-party editors (Quill/BlockNote/Excalidraw/Stream) are themed via documented props/CSS-vars only.
Design notes / judgment calls worth a look
Integration cards (Manage): third-party brand colors neutralized to bg-muted + purple Connect buttons — easily reverted if you want Slack/Linear/etc. recognizability back.
Tag chips (notes): per-tag rainbow collapsed to neutral pills (loses color-coding).
Marketing category legends (features/use-cases): kept their 4-color coding (intentional legend, not converted to single-accent).
Hero product-demo image is a static marketing asset (still shows old chrome).
Verification
bun run type, bun run check, bun run build all green at every wave.
Visual smoke on public routes (home/auth/pricing) confirms the new look renders cleanly, no console render errors.
Token contrast values chosen for WCAG AA (muted-foreground 42% → 5.17:1 on the gray canvas).
This PR contains 232 files, which is 82 over the limit of 150.
To get a review, narrow the scope:
• coderabbit review --type committed # exclude uncommitted changes
• coderabbit review --dir # limit to a subdirectory
• coderabbit review --base # compare against a closer base
Upgrade to a paid plan to raise the limit.
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: b306c52b-7875-4b47-9f86-54a5cefb050c
📥 Commits
Reviewing files that changed from the base of the PR and between 9751d6b and 370f9be.
We reviewed changes in 9751d6b...6244bc0 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements a significant UI overhaul to a 'fintech-dashboard' aesthetic, but several implementation details undermine the stability and consistency of the application. Most importantly, two logic bugs require immediate attention: the line chart component will fail for single-point datasets due to a division-by-zero error, and the new meeting modal uses an asynchronous pattern that will trigger browser popup blockers.
From a quality and architectural standpoint, Codacy results are not up to standards. There are multiple instances where the new 'never raw hex' rule in the STYLE-GUIDE.md is ignored in favor of hardcoded values, particularly in the 'Meet' feature. Additionally, the intent to remove fragile CSS overrides was only partially achieved, as new '!important' HSL hacks were introduced in the global styles. These issues, along with the maintainability concerns raised regarding the Sonner component's theme detection and missing forwardRef support in shared components, should be resolved before approval.
About this PR
New !important overrides using raw HSL values rather than tokens were added to globals.css and editor components, continuing the pattern of fragile CSS hacks that the PR description intended to eliminate.
The PR introduces multiple hardcoded hex values (e.g., in the 'Meet' feature), which directly contradicts the 'never raw hex' rule established in the new STYLE-GUIDE.md. These should be migrated to tokens.
Test suggestions
Missing: Verify getDeltaVisual logic correctly maps positive values to 'success' and negative values to 'destructive' tones with corresponding icons.
Missing: Verify AnimatedNumber component respects the useReducedMotion hook by rendering the final value statically.
Missing: Verify NewMeetingModal correctly differentiates between 'channel' and 'person' targets to generate valid meeting URLs with appropriate query params.
Missing: Verify password strength color mapping (SCORE_BAR_COLORS) aligns with the new success/warning/destructive design tokens.
Missing: Verify PageShell component correctly enforces the max-w-[1400px] constraint except when the maxWidth prop is set to 'full'.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Missing: Verify getDeltaVisual logic correctly maps positive values to 'success' and negative values to 'destructive' tones with corresponding icons.
2. Missing: Verify AnimatedNumber component respects the useReducedMotion hook by rendering the final value statically.
3. Missing: Verify NewMeetingModal correctly differentiates between 'channel' and 'person' targets to generate valid meeting URLs with appropriate query params.
4. Missing: Verify password strength color mapping (SCORE_BAR_COLORS) aligns with the new success/warning/destructive design tokens.
5. Missing: Verify PageShell component correctly enforces the max-w-[1400px] constraint except when the maxWidth prop is set to 'full'.
Low confidence findings
The modification to Sonner using a manual MutationObserver on the document root bypasses the standard next-themes implementation. This creates a risk of state desync or race conditions during theme transitions.
The reason will be displayed to describe this comment to others. Learn more.
🔴 HIGH RISK
This calculation will result in NaN or Infinity if data.length is 1, which will cause the SVG path to fail and render an empty chart. Since single-point datasets are valid for short-term reports, add a guard to handle the case where the denominator is zero.
The reason will be displayed to describe this comment to others. Learn more.
🔴 HIGH RISK
Browser popup blockers will likely block this window because window.open is called after asynchronous network requests (await createMessage). To ensure the meeting opens reliably across all browsers, you should open the window immediately with a placeholder (about:blank) and set its location.href once the async tasks complete, or use a current-tab redirect.
The reason will be displayed to describe this comment to others. Learn more.
🟡 MEDIUM RISK
Returning a void expression from an arrow function shorthand is discouraged in effect cleanups to prevent accidental return value confusion. This might be a simple fix:
The reason will be displayed to describe this comment to others. Learn more.
🟡 MEDIUM RISK
Hardcoded hex values used here violate the project's new Style Guide. These should be converted to CSS tokens or mapped in the 'dark' theme block in globals.css.
The reason will be displayed to describe this comment to others. Learn more.
🟡 MEDIUM RISK
Suggestion: Using the bg-secondary (magenta) token for a generic progress bar indicator violates the project's new style guide (STYLE-GUIDE.md), which specifies that magenta is reserved for AI-accent and marketing garnish only. This should use bg-primary or success-based tokens.
The reason will be displayed to describe this comment to others. Learn more.
⚪ LOW RISK
Suggestion: StatCard is not a forwardRef component, which prevents it from being used directly as a trigger for tooltips or popovers (e.g., using TooltipTrigger asChild). Refactoring it to support refs would improve reusability across the dashboard.
…er nav
- Move UserButton from top-right toolbar to the sidebar footer (bottom-left)
as a full account row (avatar + name + email); self-handles the
?openUserSettings deep link so the toolbar wiring is dropped
- Fix workspace-switcher clipping: remove mt-5 that overflowed the fixed-height
header and clipped the top
- Section headers read as quiet group labels (tighter padding, muted icons),
distinct from nav pills
- Default-expand Planning/Messages/Settings so their links are visible without
an extra click; add top breathing room to the scroll area
We reviewed changes in 9751d6b...370f9be on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.
Some issues found as part of this review are outside of the diff in this pull request and aren't shown in the inline review comments due to GitHub's API limitations. You can see those issues on the DeepSource dashboard.
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
The reason will be displayed to describe this comment to others. Learn more.
Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.
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
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.
![codacy-production[bot]](https://avatars.githubusercontent.com/in/56611?s=60&v=4){kind=small}
What
Restyles the entire Proddy frontend to a clean fintech-dashboard aesthetic (à la the Fundora/Finexy reference dashboards): soft-gray app canvas, white rounded-2xl cards with 1px borders + soft shadows, pill controls, stat tiles with delta chips, clean roomy tables, and quiet CSS-first micro-animations — keeping the existing primary purple (
#4A0D68) as the single interactive accent.Delivered via a foundation phase + 19 file-disjoint restyle packages across 3 waves, each gated with
bun run type+biome check+next build.Foundation (
STYLE-GUIDE.mdis the contract)--background, white--card, new--success/--warning/--sidebar*, purple focus--ring, 12px radius, softer shadows; coherent.darkremap (light-first).next/font(Poppins was referenced but never loaded).ui/*→ pills,rounded-2xlcards (+interactivehover-lift), soft-focusbg-cardinputs, soft-tint badges, roomy tables; switch/checkbox/slider off magenta.StatCard/getDeltaVisual,StatusDot,PageShell,EmptyState,AnimatedNumber.!importantdark-mode hacks (bg-secondary,edit-mode-button,chat-send-button,widget-title-icon), the never-usedrechartsdep, and glassmorphism utilities.Surface coverage (all 19 packages)
Dashboard · Reports (charts + sections) · Manage · Usage + Billing · Tasks + Calendar · Issues/Outbox/Threads/Meeting-notes · Chat core + message features · Board (kanban + gantt + drawer) · Projects (sprints/roadmap) · Canvas/Notes/Live (wrapper chrome only — editor internals untouched) · Assistant · Members + modals + pickers + audio · Auth + profile · Marketing home + subpages (pricing
gray-*cleanup) · Meet (kept dark, harmonized to a tokenized palette + pill controls).--secondary(magenta) is demoted to an AI-accent/marketing garnish. Third-party editors (Quill/BlockNote/Excalidraw/Stream) are themed via documented props/CSS-vars only.Design notes / judgment calls worth a look
bg-muted+ purple Connect buttons — easily reverted if you want Slack/Linear/etc. recognizability back.Verification
bun run type,bun run check,bun run buildall green at every wave.Frontend-only;
convex/**untouched.