As there is a modern consensus that constructor injection should be preferred to field injection, I suggest making the injection strategy default to CONSTRUCTOR. I'm aware that this might be considered a breaking change and a candidate for 2.0, but since MapStruct is based around interface definitions it might conceivably make sense in 1.4.