refactor(skills): consolidate frontend skills into frontend-workflow + per-tree CLAUDE.md symlinks - #1811
Conversation
Adds animation-vocabulary, apple-design, color-system, frontend-workflow, make-interfaces-feel-better, and migrate-radix-to-base skill packages for use during the base-ui migration and general frontend/design work.
Bundle color-system, make-interfaces-feel-better, apple-design and vercel-react-best-practices under frontend-workflow as color/, polish/, motion/ and react-performance/. Their icon, colour and motion rules overlapped with no shared parent to say which wins; the router now does. Keep shadcn top-level — its allowed-tools grant and live `info --json` substitution only work in a discovered SKILL.md. Drop disable-model-invocation from frontend-workflow, which had kept the router and its precedence rules out of context entirely. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude Code reads CLAUDE.md, not AGENTS.md, and only the repo root had the symlink — so the per-tree AGENTS.md files never loaded, making "nearest AGENTS.md wins" false in practice. Nested memory files load lazily when a file in that directory is read, so each tree's rules now reach an agent working in it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
@AnishSarkar22 is attempting to deploy a commit to the Rohan Verma's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
AnishSarkar22
changed the title
AnishSarkar22
changed the title
1 participant
What
Why
Fixes #
How to test
High-level PR Summary
This PR consolidates frontend development guidance into a unified
frontend-workflowskill that bundles React performance patterns, color systems, UI polish rules, and motion design principles, while adding a separatemigrate-radix-to-baseskill for component library migration. Thefrontend-workflowskill serves as the single entry point for all UI work acrosssurfsense_webandsurfsense_local/frontend, orchestrating React/Next.js best practices, shadcn/ui integration, color tokens, typography, animations, and Apple-style motion design. A comprehensive Radix UI to Base UI migration skill provides detailed prop mappings, part renames, and workflow guidance for converting component libraries. Project-level AGENTS.md and CLAUDE.md files are updated to route frontend work through the new workflow, withshadcnnow explicitly positioned as a specialized tool loaded after the main workflow rather than a replacement for it.⏱️ Estimated Review Time: 1-3 hours
💡 Review Order Suggestion
.agents/skills/frontend-workflow/SKILL.md.agents/skills/frontend-workflow/color/SKILL.md.agents/skills/frontend-workflow/color/PALETTE.css.agents/skills/frontend-workflow/polish/SKILL.md.agents/skills/frontend-workflow/polish/typography.md.agents/skills/frontend-workflow/polish/surfaces.md.agents/skills/frontend-workflow/polish/animations.md.agents/skills/frontend-workflow/polish/icons.md.agents/skills/frontend-workflow/polish/performance.md.agents/skills/frontend-workflow/motion/apple-design.md.agents/skills/frontend-workflow/react-performance/SKILL.md.agents/skills/migrate-radix-to-base/SKILL.md.agents/skills/migrate-radix-to-base/universal-patterns.md.agents/skills/migrate-radix-to-base/overlays.md.agents/skills/migrate-radix-to-base/menus.md.agents/skills/migrate-radix-to-base/form-controls.md.agents/skills/migrate-radix-to-base/disclosure.md.agents/skills/migrate-radix-to-base/display-misc.md.agents/skills/migrate-radix-to-base/consumer-props.md.agents/skills/migrate-radix-to-base/class-mapping.md.agents/skills/migrate-radix-to-base/wrapper-shapes.md.agents/skills/shadcn/SKILL.mdAGENTS.mdsurfsense_backend/CLAUDE.mdsurfsense_local/AGENTS.mdsurfsense_local/CLAUDE.mdsurfsense_mcp/CLAUDE.mdsurfsense_web/AGENTS.mdsurfsense_web/CLAUDE.md.gitignore