Resolving certain reverse links is broken

[apdavison]

e.g.

repo = omcore.FileRepository.from_id("c569b826-9d2e-4ba0-9363-561906c67cd6", kg_client)
files = repo.files.resolve(kg_client)
assert len(files) == 37

This fails, and gives len(files) => 10000

The problem is that in the FileRepository class the reverse property files has reverse="file_repositories", which does not match the forward property name in File, which is file_repository. This results in the query being built incorrectly.

[apdavison]

This was partially fixed in 28da285 but there are still some cases that fail.

[apdavison]

Related upstream work: openMINDS PR #97 adds reverse-link names to the schema vocabulary. Those names conflict with fairgraph's in many cases: fairgraph reuses a single reverse name across several forward properties (usedIn covers technique, protocol, language, software, setup and more), which is the root cause of the mismatches here.

Adopting the openMINDS names, once settled upstream, would remove the hand-maintained name mapping in the builder and the ambiguity along with it. Worth resolving that before fixing the mismatches individually, so the work isn't done twice.

[apdavison]

The 17 remaining mismatches, as of 2026-08-22 — reverse properties whose declared reverse= name matches no forward property on the named target class. "Likely intended" is the closest forward property name on that class.

Reverse property	reverse=	Target class	Likely intended
AgeCategory.is_age_category_of	age_categories	SubjectState	age_category
BiologicalSex.is_biological_sex_of	biological_sex	SubjectGroup	biological_sexes
BiologicalSex.is_biological_sex_of	biological_sex	TissueSampleCollection	biological_sexes
CellType.sample	origin	TissueSampleCollection	origins
ContentType.is_format_of	format	ModelVersion	formats
File.is_location_of	data_location	ParcellationTerminologyVersion	data_locations
File.specifies	specification	EphysStimulus	specifications
License.is_applied_to	license	ModelVersion	licenses
License.is_applied_to	license	SoftwareVersion	licenses
License.is_applied_to	license	ValidationTestVersion	licenses
License.is_applied_to	license	WorkflowRecipeVersion	licenses
Organ.sample	origin	TissueSampleCollection	origins
OrganismSubstance.sample	origin	TissueSampleCollection	origins
PreparationType.used_for	preparation_design	DatasetVersion	preparation_designs
PropertyValueList.specifies	specification	EphysStimulus	specifications
RRID.identifies	digital_identifier	Organization	digital_identifiers
TissueSampleType.is_type_of	type	TissueSampleCollection	types

Found by checking every reverse property's declared reverse= name against the target class's forward properties — 2601 (reverse-property, target-class) pairs in all, of which these 17 matched nothing. A reverse property declaring several alternative names counted as satisfied if any one matched.

Every case is a pluralisation mismatch. In 16 the reverse name is singular where the target's forward property is plural (license vs licenses); in one, AgeCategory.is_age_category_of, it is the other way round.

Useful as a validation set: after adopting the openMINDS names, re-running the check should leave zero unmatched.