feat: add dark-mode fix category to suggest_fix#183
Conversation
Adds fixDarkMode handler covering theme-scope-standard-property and theme-scope-shadow-piercing issues. Generates concrete CSS custom property replacements with tokenPrefix support. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (6)
✨ 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 |
Maps darkMode category to suggest_fix dark-mode type in both styling_preflight and css-file-validator. Dark mode issues now get concrete fix suggestions with CSS custom property replacements. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds component-specific dark mode warning for components with CSS custom properties — warns against using standard properties on the host in theme scopes and shows the correct custom property approach. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Summary\n\n- Adds
dark-modefix type tosuggest_fixcovering both issue rules fromcheck_dark_mode_patterns\n-theme-scope-standard-property: generates CSS custom property replacement with tokenPrefix support\n-theme-scope-shadow-piercing: suggests ::part() or custom properties with host-only selector\n- 4 new tests covering all dark mode fix paths\n\n## Test plan\n- [x] All 39 suggest-fix tests pass\n- [x] Standard property fix uses component tag name for token naming\n- [x] tokenPrefix override works correctly\n- [x] Shadow piercing fix shows both ::part() and custom property options\n\n🤖 Generated with Claude Code