I would like to ask if using both OSPF and MPLS loopbacks on a router along with adding in both /routing ospf network and /routing ospf interface, ospf and mpls loopback interfaces as passive is good general practice or can there be any issues in doing so?
Typically you only really need one loopback per routing table for an MPLS/OSPF network. having said that, there isn’t anything to prevent you from using more if you have a use case and need them.
As far as passive goes, because you aren’t forming an OSPF adjacency using the loopback, then passive is accepatble.