Best practice automation

I have a lot of simple automations that says “If no movement from motion sensor X turn off light A”.
Sometimes light A was turned off manually but, of course, the automations still runs.

My question is: is better to add some conditions (like “runs only if light A is on”) or I don’t have to care about it ?

Everything is working correctly but I was wondering which is the best practice about it.

I’ve been going through my automations doing exactly that. It keeps the log clean and lowers the system load (very slightly).

One particular example was when an automation set an input select to a value it was already at. This would generate warnings in the log but everything worked ok.