removes CastlingDetails and refactors Move._checkCastlingMove to not need it