Any reason why you’re keeping Finn’s botched Kamikaze run?
I don’t see any real issue with it. Finn’s arc is about learning to care about the Resistance, it’s just a practical way to demonstrate that.
I think it’s ways going to feel a bit weird because Finn’s arc is already functionally complete before then - when he leaves for “home” with Rose in the shuttle. The entirety of Crait is structurally in the denouement of the story, and ends up functioning like a second third act, which isn’t ideal, but there’s nothing to do about it.