Hello Java community, We’re excited to introduce advanced class cycle detection for Java projects with two new rules: S7091 – Circular dependencies between classes across packages S7027 – Circular dependencies between classes in the same package These rules will highlight the most problematic cl…

Nice feature. Yet for JPA/Hibernate bi-directional entities it’s practically unavoidable to have a cycle between classes (e.g. see https://www.baeldung.com/jpa-hibernate-associations#1-one-to-many-bidirectional-association )

Thanks for the feedback @lrozenblyum and @SlavikZ . We’ll look into these. Feel free to share more of what works for you or doesn’t here!

Thanks for the feature, it is a good thing to consider. However, I also found other false positive detection (in addition to JPA entities) in the custom validation annotations creation: https://www.baeldung.com/spring-mvc-custom-validator It marks the interface of the annotation and the usage in th…

Thanks Orlando for sharing, and welcome to the sonar Community! We are tracking all of these to find a good solution to reduce the noise. Keep them coming!

Considering: public sealed class BaseRestController permits ChargingSessionRestController, SiteRestController, VehicleRestController, VehicleTypeRestController {} and public non-sealed class VehicleRestController extends BaseRestController {} Sonar concludes that: “This class is part of 4 cycle…

The work I was doing got flagged by this new rule … it was somewhat of an effort to refactor to avoid cycles: in particular I ended up splitting some classes up, in the ugliest case into a superclass/subclass pair. The end result I think is cleaner and more maintainable. In the ugly case, the class…

Thanks for sharing Jeremy, it is an interesting example. Welcome to the Sonar Community!

Unfortunately we disabled these 2 rules due to the high number of false positives, mainly due to our usage of JSR 380 annotations and JPA/Hibernate (which make cycles unavoidable). IMHO, S7027 is unrealistic. Regarding the S7091 rule, I would suggest that the rule implementation ignores cycles app…

Hi Cédric, Welcome to the Sonar Community and thank you for sharing. Indeed, your feedback about JPA/Hibernate resonates with what we’ve been hearing from other users. I created a ticket to try to solve this in our next hardening. It would be super helpful if you could share a minimal reproducer …

Advanced Class Cycle Detection for Java

Sonar Updates

SlavikZ (Yaroslav Zakharuk) October 7, 2024, 9:43am 3

I like the feature, but we also have false positive detections when using Jackson’s @JsonSubTypes.Type annotation (for an example, see Jackson Polymorphic Type Handling Annotations). Circular dependencies between base and child classes exist there “by design”.

Topic		Replies	Views
Sonar Cloud rule - S7027 preventing from using JPA/Hibernate Associations Writing rules java , sonarqube , coverage , sonarqube-cloud	4	1165	August 28, 2025
S7027 prevents from using sealed classes Report False-positive / False-negative... java , sonarqube	7	1835	June 23, 2025
S7027 False positive on JPA/Hibernate relationships Report False-positive / False-negative... java	1	348	August 28, 2025
javaarchitecture:S7027 and jackson deserialization with inheritance Report False-positive / False-negative...	0	142	November 21, 2025
False Positive w/ javaarchitecture:S7027 when using sealed interfaces SonarQube Server / Community Build java , java21	4	587	March 11, 2025

Advanced Class Cycle Detection for Java

Related topics