fix(clerk-js,shared): Do not expect phoneNumber for sendPhoneCode

Author: dstaley

.changeset/every-terms-care.md

@@ -0,0 +1,6 @@
+---
+'@clerk/clerk-js': patch
+'@clerk/shared': patch
+---
+
+Fix issue where `signUp.verifications.sendPhoneCode()` expected to be provided a `phoneNumber`.

packages/clerk-js/src/core/resources/SignUp.ts

@@ -905,6 +905,26 @@ class SignUpFuture implements SignUpFutureResource {
     });
   }
 
+  async sendPhoneCode(params: SignUpFuturePhoneCodeSendParams): Promise<{ error: ClerkError | null }> {
+    const { channel = 'sms' } = params;
+    return runAsyncResourceTask(this.#resource, async () => {
+      await this.#resource.__internal_basePost({
+        body: { strategy: 'phone_code', channel },
+        action: 'prepare_verification',
+      });
+    });
+  }
+
+  async verifyPhoneCode(params: SignUpFuturePhoneCodeVerifyParams): Promise<{ error: ClerkError | null }> {
+    const { code } = params;
+    return runAsyncResourceTask(this.#resource, async () => {
+      await this.#resource.__internal_basePost({
+        body: { strategy: 'phone_code', code },
+        action: 'attempt_verification',
+      });
+    });
+  }
+
   async sendEmailLink(params: SignUpFutureEmailLinkSendParams): Promise<{ error: ClerkError | null }> {
     const { verificationUrl } = params;
     return runAsyncResourceTask(this.#resource, async () => {
@@ -945,34 +965,6 @@ class SignUpFuture implements SignUpFutureResource {
     });
   }
 
-  async sendPhoneCode(params: SignUpFuturePhoneCodeSendParams): Promise<{ error: ClerkError | null }> {
-    const { phoneNumber, channel = 'sms' } = params;
-    return runAsyncResourceTask(this.#resource, async () => {
-      if (!this.#resource.id) {
-        const { captchaToken, captchaWidgetType, captchaError } = await this.getCaptchaToken();
-        await this.#resource.__internal_basePost({
-          path: this.#resource.pathRoot,
-          body: { phoneNumber, captchaToken, captchaWidgetType, captchaError },
-        });
-      }
-
-      await this.#resource.__internal_basePost({
-        body: { strategy: 'phone_code', channel },
-        action: 'prepare_verification',
-      });
-    });
-  }
-
-  async verifyPhoneCode(params: SignUpFuturePhoneCodeVerifyParams): Promise<{ error: ClerkError | null }> {
-    const { code } = params;
-    return runAsyncResourceTask(this.#resource, async () => {
-      await this.#resource.__internal_basePost({
-        body: { strategy: 'phone_code', code },
-        action: 'attempt_verification',
-      });
-    });
-  }
-
   async sso(params: SignUpFutureSSOParams): Promise<{ error: ClerkError | null }> {
     const {
       strategy,

packages/shared/src/types/signUpFuture.ts

@@ -178,12 +178,6 @@ export type SignUpFuturePasswordParams = SignUpFutureAdditionalParams & {
   );
 
 export interface SignUpFuturePhoneCodeSendParams {
-  /**
-   * The user's phone number in [E.164 format](https://en.wikipedia.org/wiki/E.164). Only supported if
-   * [phone number](https://clerk.com/docs/guides/configure/auth-strategies/sign-up-sign-in-options#phone) is enabled.
-   * Keep in mind that the phone number requires an extra verification process.
-   */
-  phoneNumber?: string;
   /**
    * The mechanism to use to send the code to the provided phone number. Defaults to `'sms'`.
    */