is it bad to fetch data inside of the definition of a custom procedure?

here I created a custom procedure that had the user and username attributes attached to it from clerk, is this an anti pattern or something that could lead to low preformance?