feat(ui): add experimental flag to gate the OIDC self-serve flow#9198
Conversation
🦋 Changeset detectedLatest commit: d148e4f The changes in this PR will be included in the next version bump. This PR includes changesets to release 23 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds the experimental Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.changeset/empty-ducks-tease.md:
- Around line 1-2: Add a Changesets package bump declaration for packages/shared
in the frontmatter of empty-ducks-tease.md, using the appropriate release level
for the published TypeScript API change; retain valid Changesets formatting so
the change is included in release versioning.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 3a6a669c-9d81-41df-8ddb-15f96e26ace0
📒 Files selected for processing (3)
.changeset/empty-ducks-tease.mdpackages/shared/src/types/clerk.tspackages/ui/src/components/ConfigureSSO/ConfigureSSOContext.tsx
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
clerk/clerk_go(manual)clerk/dashboard(manual)clerk/accounts(manual)clerk/backoffice(manual)clerk/clerk(manual)clerk/clerk-docs(manual)clerk/cloudflare-workers(manual)
There was a problem hiding this comment.
Caution
Inline review comments failed to post. This is likely due to GitHub's internal server error or limits when posting large numbers of comments. If you are seeing this consistently it is likely a permissions issue. Please check "Moderation" -> "Code review limits" under your organization settings.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.changeset/empty-ducks-tease.md:
- Around line 1-2: Add a Changesets package bump declaration for packages/shared
in the frontmatter of empty-ducks-tease.md, using the appropriate release level
for the published TypeScript API change; retain valid Changesets formatting so
the change is included in release versioning.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: 3a6a669c-9d81-41df-8ddb-15f96e26ace0
📒 Files selected for processing (3)
.changeset/empty-ducks-tease.mdpackages/shared/src/types/clerk.tspackages/ui/src/components/ConfigureSSO/ConfigureSSOContext.tsx
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
clerk/clerk_go(manual)clerk/dashboard(manual)clerk/accounts(manual)clerk/backoffice(manual)clerk/clerk(manual)clerk/clerk-docs(manual)clerk/cloudflare-workers(manual)
🛑 Comments failed to post (1)
.changeset/empty-ducks-tease.md (1)
1-2: 📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
Add a package version entry to this changeset.
This PR changes a published TypeScript API, but this file contains no package or bump declaration, so the change will not be recorded for release. Add the appropriate
packages/sharedchangeset entry.As per coding guidelines,
.changeset/**must use Changesets for version management and changelogs. Based on learnings, empty changesets are appropriate for documentation-only or non-published changes, not this public type change.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.changeset/empty-ducks-tease.md around lines 1 - 2, Add a Changesets package bump declaration for packages/shared in the frontmatter of empty-ducks-tease.md, using the appropriate release level for the published TypeScript API change; retain valid Changesets formatting so the change is included in release versioning.Sources: Coding guidelines, Learnings
| @@ -0,0 +1,2 @@ | |||
| --- | |||
There was a problem hiding this comment.
you'll want to add a changeset or else it won't release the change to a new version
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
API Changes Report
Summary
@clerk/sharedCurrent version: 4.25.5 Subpath
|
Summary
Adds an experimental
oidcSelfServeflag to the ClerkProviderexperimentaloptions, laying the groundwork to gate a forthcoming OIDC self-serve configuration flow in<ConfigureSSO />.<ClerkProvider experimental={{ oidcSelfServe: true }} />(default off).isOIDCFlowEnabledviauseConfigureSSO().experimentalnamespace.