fix(mosaic): hold phone entry until verification is ready by austincalvelage · Pull Request #10561 · clerk/javascript

austincalvelage · 2026-10-30T22:57:33Z

Description

Fixes CORE-4106. Keep the phone entry screen pending while the first verification code is sent, then advance to verification once the request settles. Failed sends show the existing error and Resend action without creating another phone number.

Disable Cancel while submitting or resending a verification code, matching the controller's existing behavior. Cancel becomes available again when the request settles.

Phone entry while the first code is being sent:

Pending code submission:

Pending code resend:

Checklist

pnpm test runs as expected. Verified 33 phone unit tests and 35 phone feature tests in Mosaic.
pnpm build runs as expected. Verified the Mosaic package build.
(If applicable) JSDoc comments have been added or updated for any package exports
(If applicable) Documentation has been updated

Type of change

vercel · 2026-10-30T22:57:38Z

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

Project	Deployment	Actions	Updated
clerk-js-sandbox	Ready	Preview	Oct 30, 2026 11:08pm UTC
swingset	Ready	Preview	Oct 30, 2026 11:08pm UTC

changeset-bot · 2026-10-30T22:57:38Z

🦋 Changeset detected

Latest commit: 276bdbe

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

This PR includes changesets to release 2 packages

Name	Type
@clerk/mosaic	Patch
@clerk/swingset	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 · 2026-10-30T22:57:55Z

No actionable comments were generated in the recent review.

Recent review info

Run configuration

Source mode: Auto
Repository: clerk/javascript
Branch: austin/core-4106-disable-phone-verification-cancel-while-cancellation-is
File: .coderabbit.yaml
Commit: 276bdbe
Layers: Repository YAML (base), Organization UI (inherited)
Review profile: ASSERTIVE
Plan: Team
Run ID: 277ec51c-84f9-4714-820c-d887cc87ebcd

Commits

Reviewing files that changed from the base of the PR and between b02d5e7 and 276bdbe.

Files selected for processing: 3

.changeset/quiet-phones-wait.md
packages/mosaic/src/features/user-profile/user-profile-phone-section/__tests__/user-profile-phone-section.feature.test.tsx
packages/mosaic/src/features/user-profile/user-profile-phone-section/user-profile-add-phone.controller.ts

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)
clerk/clerk-android (auto-detected)
clerk/clerk-ios (auto-detected)
clerk/cli (auto-detected)

Included review availability: This review used your included allowance. 1 included review remains after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

Walkthrough

The phone add controller enters a dedicated state while it sends the first verification code. It keeps the phone form active and reports pending state during that request. The default Cancel action is disabled while verification or resend is pending. Tests cover initial-send completion and failure, verification errors, resend requests, and Cancel availability.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

clerk/javascript#10406: Changes the phone START transition and first-code preparation flow in the same controller.
clerk/javascript#10446: Adds a first-code preparation state that invokes the same code-sending logic.
clerk/javascript#10520: Changes phone-controller behavior tied to preparation state and pending status.

Suggested reviewers: alexcarpenter

Merge Risk: Minimal · up to 276bd

No merge-blocking risk is established from the available evidence.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Title check		The title clearly describes the main change: keeping phone entry pending until verification is ready.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description check		The pull request description directly describes the phone verification changes, including pending behavior, disabled Cancel actions, error recovery, tests, and build verification.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 2 functions across 3 files. (1 skipped: 1 unsupported.)

Resolution

Write docstrings for the functions missing them to satisfy the coverage threshold.

Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Linked repositories: Some configured repositories could not be included in this review. Cross-repository analysis may be incomplete. Please check linked-repository access and settings.

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

pkg-pr-new · 2026-10-30T23:00:46Z

Open in StackBlitz

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/electron

npm i https://pkg.pr.new/@clerk/electron@10561

@clerk/electron-passkeys

npm i https://pkg.pr.new/@clerk/electron-passkeys@10561

@clerk/eslint-plugin

npm i https://pkg.pr.new/@clerk/eslint-plugin@10561

@clerk/expo

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

@clerk/expo-biometrics

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

@clerk/expo-google-signin

npm i https://pkg.pr.new/@clerk/expo-google-signin@10561

@clerk/expo-native-components

npm i https://pkg.pr.new/@clerk/expo-native-components@10561

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@10561

@clerk/localizations

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

@clerk/mosaic

npm i https://pkg.pr.new/@clerk/mosaic@10561

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/react

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

@clerk/react-router

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

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/ui

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

@clerk/upgrade

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

@clerk/vue

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

commit: 276bdbe

fix(mosaic): disable phone verification cancel while pending

b02d5e7

github-actions Bot added the mosaic label Oct 30, 2026

vercel Bot deployed to Preview – clerk-js-sandbox October 30, 2026 22:58

View deployment

vercel Bot deployed to Preview – swingset October 30, 2026 23:00

View deployment

fix(mosaic): hold phone entry while sending the first code

276bdbe

austincalvelage changed the title ~~fix(mosaic): disable phone verification Cancel while pending~~ fix(mosaic): hold phone entry until verification is ready Oct 30, 2026

vercel Bot deployed to Preview – clerk-js-sandbox October 30, 2026 23:06

View deployment

vercel Bot deployed to Preview – swingset October 30, 2026 23:08

View deployment

austincalvelage requested a review from alexcarpenter October 30, 2026 23:13

Navigation Menu

Repository navigation

fix(mosaic): hold phone entry until verification is ready - #10561

fix(mosaic): hold phone entry until verification is ready#10561
austincalvelage wants to merge 2 commits into
mainfrom
austin/core-4106-disable-phone-verification-cancel-while-cancellation-is

austincalvelage commented Oct 30, 2026 •

edited

Loading

Uh oh!

vercel Bot commented Oct 30, 2026 •

edited

Loading

Uh oh!

changeset-bot Bot commented Oct 30, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Oct 30, 2026 •

edited

Loading

Uh oh!

pkg-pr-new Bot commented Oct 30, 2026 •

edited

Loading

Uh oh!

This branch was successfully deployed

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

austincalvelage commented Oct 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Type of change

Uh oh!

vercel Bot commented Oct 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

changeset-bot Bot commented Oct 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

coderabbitai Bot commented Oct 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pkg-pr-new Bot commented Oct 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This branch was successfully deployed

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

austincalvelage commented Oct 30, 2026 •

edited

Loading

vercel Bot commented Oct 30, 2026 •

edited

Loading

changeset-bot Bot commented Oct 30, 2026 •

edited

Loading

coderabbitai Bot commented Oct 30, 2026 •

edited

Loading

pkg-pr-new Bot commented Oct 30, 2026 •

edited

Loading