As noted in other issue threads (mapstruct/mapstruct-examples#119), the reference implementation of the CyclicGraphContext does not play well with the Lombok Builder annotation.

The main issue being that the context will attempt to store the Builder class instead of the target DTO type. Part of why this happens is because the storeMappedInstance method is annotated with @BeforeMapping. I found that annotating with @AfterMapping with a @MappingTarget parameter, the context would correctly store the actual target type after calling .build().

My question is - is there a reason why @BeforeMapping is suggested in the referenced implementation? Is there any reason to avoid using @AfterMapping in this situation?