Filip Hrisafov

Thanks for the detailed analysis @thunderhook. I had a look at your tests, I'm not sure that I can say that the tests that are failing on top of main...

I think that when you created the issue it was about the use of the `isNotEmpty` in the first if (the one with `uuid`). That's a known thing and it...

>I feel you. I would have loved to implement it this way. However, the current logic has been extracted (and adapted) from the original MapStruct source (`MapperCreationProcessor`). This is how...

Sorry I have not reviewed this. @jarlesat it would indeed be good to rebase it on top of main. Additionally, if you can move the test in `org.mapstruct.ap.test.collection.remover` it would...

This is somewhat similar to #2713. I am not sure whether defining a custom `@Mapper` is technically possible (see https://github.com/mapstruct/mapstruct/issues/2713#issuecomment-1015683434 for the reasons). What you can currently do is: ```java...

@monandszy how is your comment different from the requested issue? If you read https://github.com/mapstruct/mapstruct/issues/3285#issuecomment-1565301231 you'll see that `@SpringMapper` is most likely not technically possible.

@monandszy sorry, but I am not following. This issue is not for `@AnnotateWith`. This is for meta annotations for `@Mapper`. If `@AnnotateWith` isn't working as a meta annotation, please then...

This could be combined with something like https://github.com/mapstruct/mapstruct/issues/3626#issuecomment-2282840874.

@985177520 can you please provide an example process showing this problem?

Everything here works as designed. From what I can see the `TestMapper` looks like: ```java SomeModelWithUserId fromUserIdAndSomeOtherData(UserIdProvider userIdProvider, String someOtherData); ``` Let's dissect it. The target is `SomeModelWithUserId` which is...