docs(skill): prefer React 19 ref-as-prop over forwardRef#746
docs(skill): prefer React 19 ref-as-prop over forwardRef#746paanSinghCoder merged 1 commit intomainfrom
Conversation
Apsara is on React 19, and no existing component uses `forwardRef`/`ElementRef`. Update the add-new-component skill so the component template, Object.assign snippet, and Base UI Content template use plain function components with `ComponentProps<'tag'>` / `Primitive.*.Props` typings; `ref` flows through the spread automatically or is destructured when it must target a non-default element. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 57 minutes and 48 seconds. ⌛ 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: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
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 |
Summary
forwardRef/ElementRef— theadd-new-componentskill was still recommending the legacy pattern for new workObject.assignsnippet, Base UIContenttemplate, and quality checklist in.claude/skills/add-new-component/SKILL.mdto match the actual convention (plain function components +ComponentProps<'tag'>/Primitive.*.Propstypings)refflows through...propsautomatically; destructure explicitly only when it must target a non-default nested element (e.g. the Popup inside Portal/Positioner)Test plan
SKILL.mdfor internal consistencyCallout,Toolbar,Box,FilterChip)🤖 Generated with Claude Code