this is a bit weird, as IDEA use @noinspection tag, and you do not use it.
@noinspectionreason is checkstyle specific javadoc tag to keep context why suppression happened.

this means that IDEA inspection might not complain on this code any more, can you try to remove it ?

reduced to @noinspection ChainOfInstanceofChecks like other usage.

yes need it to make CI happy.

just c&p and we good to go.

good teamwork, thanks.

[ERROR] [checkstyle] [ERROR] /home/circleci/project/src/main/java/com/puppycrawl/tools/checkstyle/site/SiteUtil.java:1103:7: @noinspection Javadoc tags should be accompanied by a @noinspectionreason tag, explaining why we suppressed inspection. [MatchXpath]

Both should be used.
But try without all first, looks like idea inspection are complaining in this any more

Short: No. Its working and we are off topic losing focus on random experiments checking CI or idea both outside of tickets scope.

Merge or i will close, have invested enough on this already.

Long:
Both are already in use as required. We have failed multiple times having every variant, which is very annoying and makes me want to reject this.
I don't see the need to investigate why it works now or why it doesn't in other cases. These details aren't relevant.

We haven't eliminated all variants yet, and asking for more iterations is wasting time:

• none
• one
• just the reason
• just the marker

There's no value in repeating these redundancies expecting different results from the current state.

Let's consider experiments later. This feature has been open too long and we're losing focus. CI and IDEA are not related to Java migration - that's the scope. The feature is working, assuming this PR has gone through enough pipelines.

Since it works, there's no need for further trial and error on unrelated issues. If needed, we can open a separate issue, but this one is complete.

Please move forward with finishing this instead of exploring new checks.