Skip to content

Conversation

@LauraBeatris
Copy link
Member

@LauraBeatris LauraBeatris commented Nov 24, 2025

Description

Fix issue where __internal_enableEnvironmentSettingPrompt was logging an warning even for production instances when the user didn't have a session while rendering org components.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Summary by CodeRabbit

  • Bug Fixes
    • Organization component warnings now only display in development mode when no user session exists, reducing unnecessary warnings in production environments.

✏️ Tip: You can customize this high-level summary in your review settings.

@LauraBeatris LauraBeatris self-assigned this Nov 24, 2025
@changeset-bot
Copy link

changeset-bot bot commented Nov 24, 2025

🦋 Changeset detected

Latest commit: 273a80b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@clerk/clerk-js Patch
@clerk/chrome-extension Patch
@clerk/clerk-expo Patch

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

@vercel
Copy link

vercel bot commented Nov 24, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
clerk-js-sandbox Ready Ready Preview Comment Nov 24, 2025 7:20pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 24, 2025

Walkthrough

A patch release for @clerk/clerk-js restricts organization component warnings to development environments with no active user session, preventing unnecessary warnings in production.

Changes

Cohort / File(s) Summary
Changeset entry
\.changeset/odd-books-win\.md
Patch release entry for @clerk/clerk-js documenting the warning behavior update
Warning logic update
packages/clerk-js/src/core/clerk\.ts
Modified __internal_attemptToEnableEnvironmentSetting to conditionally log warnings only when instance type is development and no active user exists

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single logic condition addition in one method
  • Straightforward environment mode guard pattern
  • Changeset file is administrative metadata only

Poem

🐰 A rabbit hops through code so neat,
Where warnings now know when to greet!
In dev mode whispers, production stays mute,
No false alarms—just clean, quiet loot! 🌱

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: restricting an organization component warning to development instances only, which is the primary focus of the changeset and code modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch laura/fix-log-once-dev

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


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

@LauraBeatris LauraBeatris enabled auto-merge (squash) November 24, 2025 19:22
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 24, 2025

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@7298

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@7298

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@7298

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@7298

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@7298

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7298

@clerk/elements

npm i https://pkg.pr.new/@clerk/elements@7298

@clerk/clerk-expo

npm i https://pkg.pr.new/@clerk/clerk-expo@7298

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@7298

@clerk/express

npm i https://pkg.pr.new/@clerk/express@7298

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@7298

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@7298

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@7298

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@7298

@clerk/clerk-react

npm i https://pkg.pr.new/@clerk/clerk-react@7298

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@7298

@clerk/remix

npm i https://pkg.pr.new/@clerk/remix@7298

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@7298

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@7298

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@7298

@clerk/themes

npm i https://pkg.pr.new/@clerk/themes@7298

@clerk/types

npm i https://pkg.pr.new/@clerk/types@7298

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@7298

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@7298

commit: 273a80b

@LauraBeatris LauraBeatris merged commit 577f7ad into main Nov 24, 2025
49 checks passed
@LauraBeatris LauraBeatris deleted the laura/fix-log-once-dev branch November 24, 2025 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants