Wallets basically have two types: custodial (you don't have private key) and non custodial (you own private key). The advice is always "It's not your key, it's not your bitcoin" so surely non custodial wallets are most recommended.
A next thing is open source or close source and the recommendation is going with open source wallets that are reproducible by anyone from the source code.