Changing "Read more" to "Read less" is something a junior developer does because they think it makes perfect sense. It's just a mapping between a boolean value and view.
For the end user this doesn't make any sense whatsoever. I just decided to "read more", why would I want to read less now? What does it mean anyway? Even "unread" would make more sense.