Conversation
…act-19-rsc-support
…istribute-esm-unbundled
…istribute-esm-unbundled
…ar-file-architecture
…ependencies-first-pass
…-api-as-exports-generator-source-of-truth
…into chore/remove-leaked-styling-library-types-from-public-api
…into chore/remove-leaked-radix-ui-types
…ependencies-first-pass
…-api-as-exports-generator-source-of-truth
…into chore/remove-leaked-styling-library-types-from-public-api
…into chore/remove-leaked-radix-ui-types
…ar-file-architecture
…ependencies-first-pass
…-api-as-exports-generator-source-of-truth
…ar-file-architecture
…ependencies-first-pass
…-api-as-exports-generator-source-of-truth
…into chore/remove-leaked-styling-library-types-from-public-api
…into chore/remove-leaked-radix-ui-types
…ar-file-architecture
…ependencies-first-pass
…-api-as-exports-generator-source-of-truth
…into chore/remove-leaked-styling-library-types-from-public-api
…pi' into chore/remove-leaked-radix-ui-types
|
Claude finished @punkbit's task in 2m 8s —— View job Code ReviewNice work decoupling the public API from Radix UI internals! The approach is solid. There's one build-breaking bug to fix before merge. 🚨 Must Fix
💡 Suggestions (non-blocking)
The |
📚 Storybook Preview Deployed✅ Preview URL: https://click-nyjygnzsj-clickhouse.vercel.app Built from commit: |
Why?
Introduces click-ui's own
DialogPropsandDialogTriggerPropstypes, replacing direct Radix UI type re-exports (leaks). This decouples the public API from internal implementation details.🤖 On #845 merge, change base branch to main
How?
Preview?
N/A