I think you're right. Actually I'm wondering whether `BeanMapping` is required at all.

While working on the model refactoring (with the goal that each model element itself knows how to create the corresponding representation in the resulting source code), it turned out that all the information such as source and return type etc. is needed on the level of `MappingMethod`, leaving `BeanMapping` just with references to mapping and reverse mapping method, and as you say this link is not really required.