detekt-core shouldn't depend on detekt-psi-utils

[3flex]

Expected Behavior

:detekt-core shouldn't depend on :detekt-psi-utils.

Observed Behavior

It does.

Context

I believe detekt-psi-utils should only be used to support rule writers. detekt-core shouldn't have an implementation dependency on a module like this, it should be runtimeOnly.

Calls to absolutePath() are simple to replace. The problem classes are AnnotationExcluder and FunctionMatcher which were moved to detekt-psi-utils in #7105 (cc @BraisGabin). I'm not proposing to revert that PR, as these classes don't belong in detekt-tooling either.

[BraisGabin]

My proposal would be to duplicate the code. On core and cli.

Does this dependency block anything? Or it is just to clean up?

[3flex]

Clean up. It would be great to get to the point that psi-utils API is really focused and valuable. I'm going to keep chipping away at it.

But going hand in hand with that is removing the dependency where it should not be needed.