For a public routing node is very important the response time. Running over Tor is adding huge lag and unreliability. If you didn't know that until now, it means that you still have to learn a lot about running public routing nodes.

Inbound liquidity is not negative per se, but your node liquidity in total how is managed. In order to offer good routes, fees and enough liquidity, you need to study a lot the traffic with a peer, to see how much is moving during x period, how many txs, how much large are the HTLCs etc... and then open accordingly a good size of a channel based on those observations and apply the fees policies that could offer a good flow in and out through that channel.

Just because you opened a channel towards SN node it means nothing if you will not route too much through it. You've just badly allocate capital.