Skip to content

Conversation

@jrgarciadev
Copy link
Member

@jrgarciadev jrgarciadev commented Oct 20, 2025

…ts but adjusting the main compound component

Closes #

📝 Description

⛳️ Current behavior (updates)

🚀 New behavior

💣 Is this a breaking change (Yes/No):

📝 Additional Information


Note

Introduce RSC‑compatible compound component API: main parts now use .Root (with continued named exports), and migrate docs, stories, and demos accordingly; add smarter build directives and update changelog to v3.0.0‑alpha.35.

  • React components (API):
    • Introduce .Root for main compound components with slot-style namespace exports (e.g., Accordion.Root, Alert.Root, etc.).
    • Keep named exports functional; add corresponding RootProps types and slot exports; update indexes across components (Accordion, Alert, Avatar, Card, Disclosure, Fieldset, Kbd, Link, Popover, RadioGroup, Slider, Switch, Tabs, Tooltip).
  • Docs, stories, demos:
    • Migrate all usage to .Root pattern and new slot names; update API tables and examples for all affected components.
    • Adjust internal app links/usages (e.g., LinkRoot wrappers in docs app).
  • Changelog:
    • Add v3.0.0-alpha.35: RSC support for compound components, React 19 improvements, breaking change requiring .Root for main component, and a minor Calendar fix.
  • Build:
    • Enhance build script to insert appropriate directives: add "use client" for files compiled from .tsx and "use strict" for .ts sources.
  • Misc:
    • Bump deps in lockfile; minor storybook updates to reflect new API.

Written by Cursor Bugbot for commit b34e672. This will update automatically on new commits. Configure here.

…ts but adjusting the main compound component
@changeset-bot
Copy link

changeset-bot bot commented Oct 20, 2025

⚠️ No Changeset found

Latest commit: b34e672

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 20, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/dot-notation

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jrgarciadev jrgarciadev merged commit 796b1cd into alpha-35 Oct 20, 2025
1 check passed
@jrgarciadev jrgarciadev deleted the refactor/dot-notation branch October 20, 2025 22:45
<PopoverContent>
<PopoverDialog>
<PopoverHeading>Popover Title</PopoverHeading>
<PopoverHeading>PopoverRoot Title</PopoverHeading>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Internal Suffix Leaks into User-Facing Content

User-visible text, accessibility labels, and comments were inadvertently updated to include the internal Root suffix (e.g., PopoverRoot Title instead of Popover Title). This affects PopoverHeading content, Switch descriptive text, Tabs aria-label, and Avatar comments, impacting user experience, accessibility, and code readability.

Additional Locations (5)

Fix in Cursor Fix in Web


import {chipVariants} from "@heroui/react";
import Link from "next/link";
import LinkRoot from "next/link";
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Incorrect Link Component Import Causes Runtime Error

The Link component from both next/link and fumadocs-core/link is incorrectly imported as LinkRoot. This causes a runtime error because the respective modules export a component named Link, not LinkRoot.

Additional Locations (1)

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants