I want to control two LEDs (red and green). The circuit is powered by 5V DC. The green LED is active 3.2V/20mA when the ESP32 pin is LOW (0V). When the pin is HIGH (3.3V), the green LED is inactive and activates the red LED 2V/20mA. I am using only one pin on the ESP32. ONLY ONE LED is active at a time!
Everything works, but the MOSFET is in linear mode, not saturation mode. Is this a problem? The green LED is active 23/7 at around 13mA, the red LED is only active for 1 hour out of 24 at around 13mA. Is this a problem with continuous operation? Is this circuit good for all components? Does it not overheat or consume more power than necessary? Can I change some of the components or redesign the circuit for better performance? Is it necessary to put some kind of pull-up or pull-down resistor to prevent the red LED from being activated incorrectly? My circuit is on image below:

