fix(ui): prevent infinite spinner on factor-two without active 2FA session

[nikosdouvlis]

Summary

Users navigating directly to /sign-in#/factor-two without an active sign-in requiring 2FA would see an infinite loading spinner. This adds a status check to redirect them back to sign-in start, matching the existing pattern in SignInFactorOne.

• Added useEffect to SignInFactorTwo that redirects when signIn.status is null, needs_identifier, or needs_first_factor
• Dependency array only includes __internal_setActiveInProgress (not signIn.status) to avoid flash during valid sign-in completion

Why not signIn.status !== 'needs_second_factor'?

That condition catches complete status, causing a flash to the sign-in page before the final redirect. By explicitly checking for invalid statuses only, we avoid interfering with the normal sign-in completion flow.

Test plan

• Direct navigation to /sign-in#/factor-two without active sign-in → redirects to sign-in start
• Valid MFA sign-in flow → completes without flash to sign-in page

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes

  • Fixed an infinite loading spinner when navigating directly to the two-factor authentication sign-in page without an active 2FA session; users are now redirected to the correct sign-in step.

• Chores

  • Relaxed commit message validation from failing to warning for subject-case violations to reduce CI failures while keeping guidance.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
clerk-js-sandbox	Ready	Preview, Comment	Feb 6, 2026 9:35am

[changeset-bot]

🦋 Changeset detected

Latest commit: aecc1b5

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

This PR includes changesets to release 2 packages

Name	Type
@clerk/ui	Patch
@clerk/chrome-extension	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

[coderabbitai]

Caution

Review failed

The head commit changed during the review from 48f40bc to fa494da.

📝 Walkthrough

Walkthrough

Adds a patch changelog entry noting a fix for an infinite loading spinner on the factor-two sign-in route. Updates SignInFactorTwo to import runtime Clerk hooks and a router, and adds a useEffect that redirects to the sign-in start path when mounted if signIn.status is null, 'needs_identifier', or 'needs_first_factor', unless an active in-progress operation exists; it skips redirect when status is 'complete'. commitlint's subject-case rule was relaxed from error to warning.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: fixing an infinite spinner issue on the factor-two sign-in route when there's no active 2FA session. It's specific, concise, and directly reflects the primary fix in the changeset.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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

[pkg-pr-new]

Open in StackBlitz

@clerk/agent-toolkit

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

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/dev-cli

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

@clerk/expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: aecc1b5