During review of I2fb05b9117 (or perhaps I1433ea7e32, not sure), Jonas pointed out that it’s a bit strange to see words like “cache” pop up in CheckingResultsBuilder, which shouldn’t know anything about caching. After some discussion, we agreed on some changes we wanted to make – we wanted to do the necessary refactorings in a pair session, but didn’t find the time. In short, what’s currently called CachingMetadata should be split up, and all associated classes renamed to remove the “cache” from the name; and there could also be a ConstraintReport class, which would be a list of CheckResults plus their merged metadata, and perhaps DelegatingConstraintChecker would be changed to return that and renamed to something like ConstraintReportBuilder (though I don’t think we reached an agreement on this). See the subtasks for details.