fix(next): ensure registry:ui components don't depend on themselves

[ieedan]

fixes #1489

Previously if a component in the registry had a reference to itself it could add itself to its own registryDependencies. If not resolved correctly by the CLI this would result in stack overflows because of a circular dependency.

This PR fixes the CLI's handling of these circular dependencies as well as prevents them from being built into the registry by simply removing the component from it's own dependency array if it exists.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 1702d88

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

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
shadcn-svelte	✅ Ready (View Log)	Visit Preview	1702d88

[AdrianGonz97]

This is more so an error in the component itself rather than in the CLI. No components should ever have a reference to itself. Pushing a fix momentarily

[ieedan]

This is more so an error in the component itself rather than in the CLI. No components should ever have a reference to itself. Pushing a fix momentarily

Wouldn't it still be a good thing to check since this is an easy mistake to make? Maybe instead of ignoring it would give an error while building?

[AdrianGonz97]

I guess we could add a an error for it in the build script so we can catch it early on

[ieedan]

I guess we could add a an error for it in the build script so we can catch it early on

I think instead of removing it from the map it just reports an error and tries and help you along.

Would you like to add it or should I give it a shot?

[ieedan]

haha guess we caught a few more!

[AdrianGonz97]

alright i think that's all of them

[huntabyte]

Studs!