The find-attach-target should be in a method attach. That's the only point where the resolver is guaranteed to be in the right state to find the correct attach target. Granted, it's not going to be in the wrong compunit, but it's still a matter of correctness.