The auto-service plugin doesn't buy us much: it just generates a one-line descriptor file
that will rarely, if ever, change.
On the downside, it pulls a transitive dependency to Guava that is quite error-prone: it's easy to accidentally import non-shaded Guava types in the processor code, and that will results in runtime errors if the auto-service dependency is kept optional (as is intended).
On a side note, we also runs all processors in the build, which means the mapper processor runs on itself. This is not really a problem right now, but we have a few annotated classes in the test classpath (
) that currently get processed unnecessarily. It will also be a blocker if we ever need sample files that contain configuration errors.
remove the dependency to auto-service, maintain the descriptor file manually instead.
add proc:none to the mapper-processor build