fix(clerk-js): include ticket strategy in future helpers (#8641)

Co-authored-by: Dylan Staley <88163+dstaley@users.noreply.github.com>

Author: jeremy-clerk

.changeset/future-ticket-strategy.md

@@ -0,0 +1,5 @@
+---
+"@clerk/clerk-js": patch
+---
+
+Send the ticket strategy when using Core 3 sign-in and sign-up ticket helpers.

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

@@ -1414,7 +1414,7 @@ class SignInFuture implements SignInFutureResource {
 
   async ticket(params?: SignInFutureTicketParams): Promise<{ error: ClerkError | null }> {
     const ticket = params?.ticket ?? getClerkQueryParam('__clerk_ticket');
-    return this.create({ ticket: ticket ?? undefined });
+    return this.create({ strategy: 'ticket', ticket: ticket ?? undefined });
   }
 
   async finalize(params?: SignInFutureFinalizeParams): Promise<{ error: ClerkError | null }> {

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

@@ -1135,7 +1135,7 @@ class SignUpFuture implements SignUpFutureResource {
 
   async ticket(params?: SignUpFutureTicketParams): Promise<{ error: ClerkError | null }> {
     const ticket = params?.ticket ?? getClerkQueryParam('__clerk_ticket');
-    return this.create({ ...params, ticket: ticket ?? undefined });
+    return this.create({ ...params, strategy: 'ticket', ticket: ticket ?? undefined });
   }
 
   async finalize(params?: SignUpFutureFinalizeParams): Promise<{ error: ClerkError | null }> {

packages/clerk-js/src/core/resources/__tests__/SignIn.test.ts

@@ -2270,7 +2270,43 @@ describe('SignIn', () => {
           expect.objectContaining({
             method: 'POST',
             path: '/client/sign_ins',
-            body: { ticket: 'ticket_from_query' },
+            body: { strategy: 'ticket', ticket: 'ticket_from_query' },
+          }),
+        );
+      });
+
+      it('uses provided ticket parameter', async () => {
+        const mockSearchParams = new URLSearchParams('?__clerk_ticket=ticket_from_query');
+        vi.stubGlobal('window', {
+          location: {
+            search: '?__clerk_ticket=ticket_from_query',
+            href: 'https://example.com?__clerk_ticket=ticket_from_query',
+          },
+        });
+        vi.stubGlobal('URLSearchParams', vi.fn().mockReturnValue(mockSearchParams));
+
+        SignIn.clerk = {
+          __internal_environment: {
+            displayConfig: {
+              captchaOauthBypass: [],
+            },
+          },
+        } as any;
+
+        const mockFetch = vi.fn().mockResolvedValue({
+          client: null,
+          response: { id: 'signin_123' },
+        });
+        BaseResource._fetch = mockFetch;
+
+        const signIn = new SignIn();
+        await signIn.__internal_future.ticket({ ticket: 'provided_ticket' });
+
+        expect(mockFetch).toHaveBeenCalledWith(
+          expect.objectContaining({
+            method: 'POST',
+            path: '/client/sign_ins',
+            body: { strategy: 'ticket', ticket: 'provided_ticket' },
           }),
         );
       });

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

@@ -790,6 +790,7 @@ describe('SignUp', () => {
             method: 'POST',
             path: '/client/sign_ups',
             body: expect.objectContaining({
+              strategy: 'ticket',
               ticket: 'provided_ticket',
             }),
           }),
@@ -1296,6 +1297,7 @@ describe('SignUp', () => {
             method: 'POST',
             path: '/client/sign_ups',
             body: expect.objectContaining({
+              strategy: 'ticket',
               ticket: 'ticket_from_query',
             }),
           }),
@@ -1319,13 +1321,37 @@ describe('SignUp', () => {
         BaseResource._fetch = mockFetch;
 
         const signUp = new SignUp();
-        await signUp.__internal_future.ticket({ ticket: 'provided_ticket' });
+        await signUp.__internal_future.ticket({ ticket: 'provided_ticket', firstName: 'Test' });
+
+        expect(mockFetch).toHaveBeenCalledWith(
+          expect.objectContaining({
+            method: 'POST',
+            path: '/client/sign_ups',
+            body: expect.objectContaining({
+              strategy: 'ticket',
+              ticket: 'provided_ticket',
+              firstName: 'Test',
+            }),
+          }),
+        );
+      });
+
+      it('forces the ticket strategy', async () => {
+        const mockFetch = vi.fn().mockResolvedValue({
+          client: null,
+          response: { id: 'signup_123' },
+        });
+        BaseResource._fetch = mockFetch;
+
+        const signUp = new SignUp();
+        await signUp.__internal_future.ticket({ strategy: 'oauth_google', ticket: 'provided_ticket' } as any);
 
         expect(mockFetch).toHaveBeenCalledWith(
           expect.objectContaining({
             method: 'POST',
             path: '/client/sign_ups',
             body: expect.objectContaining({
+              strategy: 'ticket',
               ticket: 'provided_ticket',
             }),
           }),