I think it is also a timing issue, like the previous rpc kick thing.
probably if you wrap this part into a nextTick it could already help. (not sure about scrap heli though, since the death could appear sooner). In this case you could add a damage prevention if entity.ownerid = player.userID, so that at least the owner never gets damage from their own scrapper...
yes, i can confirm
NextTick(() => {
if (vehicle.mountPoints != null && vehicle.mountPoints.Count > 0)
{
var driverSeat = vehicle.mountPoints.FirstOrDefault()?.mountable;
if (driverSeat != null)
{
driverSeat.MountPlayer(player);
player.SendNetworkUpdate();
}
}
});
helps with the mountpoint issue. But since we're running TruePVE with no scrap heli damage to players, I can't speak for the timing of scrapper-kills.
My suggestion here would be for orange to add an owner damage protection, if the timing can be off and scrapheli sawns kill players.