Skip to content

Conversation

@jacekradko
Copy link
Member

@jacekradko jacekradko commented Dec 15, 2025

Description

Fixes: USER-3857

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

  • Chores
    • Removed the transitive runtime dependency regenerator-runtime and corresponding runtime imports from builds, reducing bundled polyfills.
    • No public APIs, exports, or signatures were changed; observable behavior should be identical except in environments that relied on the removed polyfill.

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

@changeset-bot
Copy link

changeset-bot bot commented Dec 15, 2025

🦋 Changeset detected

Latest commit: 177cfb8

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/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 Dec 15, 2025

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

Project Deployment Review Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Dec 15, 2025 9:25pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 15, 2025

Walkthrough

Removed the transitive dependency regenerator-runtime and deleted its side-effect imports from clerk-js entry files and the package manifest; exported APIs and application logic were not modified.

Changes

Cohort / File(s) Summary
Dependency configuration
packages/clerk-js/package.json
Removed the regenerator-runtime dependency (0.14.1) from package.json; other dependencies unchanged.
Entry point imports
packages/clerk-js/src/index.ts, packages/clerk-js/src/index.headless.ts, packages/clerk-js/src/index.headless.browser.ts, packages/clerk-js/src/index.legacy.browser.ts
Deleted the side-effect import regenerator-runtime/runtime from all listed entry files; no public API or control-flow changes.
Release metadata
.changeset/remove-regenerator-runtime.md
Added a changeset documenting the removal for a patch release.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Repetitive, limited-scope edits: dependency removal and identical import deletions across multiple entry files.
  • Review attention:
    • Confirm target build outputs and published bundles remain functional without the polyfill.
    • Verify CI/build matrices for older runtimes that might have depended on the runtime helper.

Poem

🐇 I hopped through lines and found a seam,
A tiny runtime left the team.
I nudged four files and closed the gate,
A lighter package — small and straight.
Crunching carrots, shipping clean 🍃

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: removal of regenerator-runtime dependency from clerk-js package, which is reflected across package.json and multiple source files.
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 feat/remove-regenerator-runtime

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 15, 2025

Open in StackBlitz

@clerk/agent-toolkit

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

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/dev-cli

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

@clerk/expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@7473

@clerk/upgrade

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

@clerk/vue

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

commit: 177cfb8

@jacekradko jacekradko merged commit 7c6f593 into main Dec 16, 2025
45 checks passed
@jacekradko jacekradko deleted the feat/remove-regenerator-runtime branch December 16, 2025 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants