I like that, but it feels a little vague as to what Rey and Palpatine’s relationship actually is, and why she’s a Palpatine. Perhaps we should develop 3 separate versions of this scene:

True Rey Nobody - No parents and no connection to existing characters. Would focus on Rey’s dark visions and her burden of carrying the legacy of the Jedi. Would probably have to change music/trim scene so it seems like less of a reveal.
Rey Palpatine - Rey is a clone/force creation of Palpatine. This version keeps her good parents in order to salvage as much footage from the original film as possible.
Rey Nobody Palpatine - Rey is a creation of Palpatine, but her parents are not mentioned or relevant to the story. Keeps a big reveal while still maintaining continuity from TLJ.

I’m not sure if I prefer 1 or 3 more. I think 3 works best to preserve continuity and still keep the core of TROS intact, but there is something so charming about Rey truly being a nobody who was chosen by the force when the moment called for it, it’s very Arthurian and a great idea for a character in a trilogy mainly about legacy.