Yes, this is kind of the problem for me. Having Palpatine kill her parents was clearly meant to provide her personal motivation for wanting to kill Palpatine, but they had to make her actually want to avenge them by making her parents sacrifice themselves for her. This is a clear contradiction to how they are presented in TLJ, which, in my opinion, is the more interesting approach. And I just hate how the story goes, “Your parents sold you for drinking money” to “No, your parents actually loved and died for you!”
I wholeheartedly agree. If you’re going through the trouble of keeping Rey a nobody, you should also maintain the core point behind this revelation in TLJ.
The thing she has to overcome isn’t just that her parents weren’t anybody important, but that they were filthy drunks that didn’t care about her. The whole point was Rey needs to move past her fixation over them, which was a big struggle for her not only in TLJ but also in TFA.
When Kylo says they were “filthy junk traders who sold you off for drinking money”, he’s just saying aloud what Rey already knows, so suddenly making her parents noble and heroic contradicts Rey’s own beliefs and nullifies her inner struggle. TRoS already has themes of family being something that goes beyond blood relation, with Rey even taking on the Skywalker name by the end, so there really is no point in clearing her parents name.