fix(clerk-js,types): Add hasBeenFinalized to prevent null updates

[dstaley]

Description

This PR fixes an issue where SignIn(null) was propagated to our signals when the signIn was completed. This is due to client piggybacking. Now, the SignIn(null) will only be propagated if hasBeenFinalized is true, which indicates that signIn.finalize was called on the instance (and that it can be safely overwritten since there's nothing left to do to it).

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:

[changeset-bot]

🦋 Changeset detected

Latest commit: a58434a

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

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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]

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

Project	Deployment	Preview	Comments	Updated (UTC)
clerk-js-sandbox	Ready	Preview	Comment	Oct 31, 2025 8:09pm

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch ds.fix/signals-has-been-finalized

---

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

[blacksmith-sh]

Found 19 test failures on Blacksmith runners:

Test	View Logs
[chrome] › integration/tests/ components.test.ts:102:9 › component smoke tests @generic › long-running--react.vite.wi thEmailCodes › UserAvatar supports fallback	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:110:9 › pricing table @billing › long-running--withBilling.astro. node › when signed in flow › subscribes to a plan	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:110:9 › pricing table @billing › long-running--withBilling.next.a ppRouter › when signed in flow › subscribes to a plan	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:110:9 › pricing table @billing › long-running--withBilling.vue.vi te › when signed in flow › subscribes to a plan	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:250:7 › pricing table @billing › long-running--withBilling.astro. node › starts free trial subscription for new user	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:250:7 › pricing table @billing › long-running--withBilling.next.a ppRouter › starts free trial subscription for new user	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:385:7 › pricing table @billing › long-running--withBilling.next.a ppRouter › subscribing to other paid plans while on free trial is immediate cancellatio n	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:456:9 › pricing table @billing › long-running--withBilling.vue.vi te › in UserProfile › renders pricing table, subscribes to a plan, revalidates payment method on complete and then downgrades to free	View Logs
[chrome] › integration/tests/ pricing-table.test.ts:588:9 › pricing table @billing › long-running--withBilling.next.a ppRouter › in UserProfile › adds payment method via checkout and resets stripe setup in tent after failed payment	View Logs
[chrome] › integration/tests/ session-tasks-multi-session.test.ts:33:9 › session tasks multi-session flow @nextjs › l ong-running--next.appRouter.withSessionTasks › when switching sessions, navigate to tas k	View Logs
[chrome] › integration/tests/ session-tasks-sign-up.test.ts:79:9 › session tasks after sign-up flow @nextjs › long-ru nning--next.appRouter.withSessionTasks › with sso, navigate to task on after sign-up	View Logs
[chrome] › integration/tests/ sign-in-flow.test.ts:101:7 › sign in flow @generic @nextjs › long-running--next.appRout er.withEmailCodes › can reset password	View Logs
[chrome] › integration/tests/ sign-in-flow.test.ts:101:7 › sign in flow @generic @nextjs › long-running--next.appRout er.withEmailCodes › can reset password	View Logs
[chrome] › integration/tests/ sign-in-flow.test.ts:101:7 › sign in flow @generic @nextjs › long-running--next.appRout er.withEmailCodes › can reset password	View Logs
[chrome] › integration/tests/ sign-in-flow.test.ts:101:7 › sign in flow @generic @nextjs › long-running--next.appRout er.withEmailCodes › can reset password	View Logs
[chrome] › integration/tests/ sign-in-flow.test.ts:101:7 › sign in flow @generic @nextjs › long-running--react.vite.w ithEmailCodes › can reset password	View Logs
[chrome] › integration/tests/ sign-in-or-up-flow.test.ts:115:9 › sign-in-or-up flow @nextjs › long-running--next.appR outer.withSignInOrUpFlow › sign-in › can reset password	View Logs
[chrome] › integration/tests/ sign-in-or-up-flow.test.ts:115:9 › sign-in-or-up flow @nextjs › long-running--next.appR outer.withSignInOrUpFlow › sign-in › can reset password	View Logs
[chrome] › integration/tests/ sign-in-or-up-flow.test.ts:115:9 › sign-in-or-up flow @nextjs › long-running--next.appR outer.withSignInOrUpFlow › sign-in › can reset password	View Logs