refactor: update return types in 2FA processing functions for better type safety

Author: ggazzo

apps/meteor/client/lib/2fa/process2faReturn.ts

@@ -92,15 +92,15 @@ export async function process2faReturn({
 	}
 }
 
-export async function process2faAsyncReturn({
+export async function process2faAsyncReturn<TResult>({
 	error,
 	onCode,
 	emailOrUsername,
 }: {
 	error: unknown;
-	onCode: (code: string, method: string) => unknown | Promise<unknown>;
+	onCode: (code: string, method: string) => TResult | Promise<TResult>;
 	emailOrUsername: string | null | undefined;
-}): Promise<unknown> {
+}): Promise<TResult> {
 	// if the promise is rejected, we need to check if it's a 2fa error
 	// if it's not a 2fa error, we reject the promise
 	if (!(isTotpRequiredError(error) || isTotpInvalidError(error)) || !hasRequiredTwoFactorMethod(error)) {

apps/meteor/client/meteorOverrides/totpOnCall.ts

@@ -45,8 +45,8 @@ const withSyncTOTP = (call: (name: string, ...args: any[]) => any) => {
 	};
 };
 
-const withAsyncTOTP = (callAsync: (name: string, ...args: any[]) => Promise<any>) => {
-	return async function callAsyncWithTOTP(methodName: string, ...args: unknown[]): Promise<unknown> {
+const withAsyncTOTP = <TResult>(callAsync: (name: string, ...args: any[]) => Promise<TResult>) => {
+	return async function callAsyncWithTOTP(methodName: string, ...args: unknown[]): Promise<TResult> {
 		try {
 			return await callAsync(methodName, ...args);
 		} catch (error: unknown) {