OnError Hook not getting called

[XamHans]

This is my action:

export const createSaleUseCase = actionClient
  .schema(createSaleWithItemsInputSchema)
  .action(async ({ parsedInput }) => {
    try {
      const result = await createSaleWithItems(parsedInput);
      return { success: result };
    } catch (error) {
      console.error('Error creating sale:', error);
      return { failure: 'Failed to create sale: ' + (error instanceof Error ? error.message : String(error)) };
    }
  });

This is how I call it:

 const createSaleAction = useAction(createSaleUseCase, {
    // @ts-ignore
    onSuccess: ({ data }) => {
      console.log('on success data from create sale', data);
      
    },
    // @ts-ignore
    onError: (error) => {
      toast({
        title: 'Error',
        duration: 1000,
        description: `Sale could not be created ${error}`,
      });
    },
  });

So when an error happens (in my case on db operation site) the onSuccess code is still getting invoked with the failure object:
{
"failure": "Failed to create sale: Member not found or birthday not set"
}

What's wrong ? I am using v7 with nextjs 14

EDIT:
Sorry my bad, of course it needs to be

export const createSaleUseCase = actionClient
  .schema(createSaleWithItemsInputSchema)
  .action(async ({ parsedInput }) => {
    try {
      const result = await createSaleWithItems(parsedInput);
      return { success: result };
    } catch (error) {
      console.error('Error creating sale:', error);
      actionClient.throwServerError({ failure: 'Failed to create sale: ' + (error instanceof Error ? error.message : String(error)) })
    }
  });

[TheEdoRan]

You're catching the error inside the action, that's why onSuccess gets called.