Would it also work if we'd check if s.referenceBackend is empty, and if so, fill it with img.Name() (or whatever is needed)? Something like;

var refs []reference.Named
if s.referenceBackend != nil {
	refs = s.referenceBackend.References(img.ID().Digest())
} else {
	// FIXME refs should contain all references of the image
	refs = append(refs, img.Name())
}

Or would we need the image name passed for that?

(Mostly seeing if we can keep the change to where it's a problem, and to have a clear TODO that needs fixing)

Sure we could just drop this line, but my goal here is to prepare the required refactoring so we get the codebase ready while I investigate how to retrieve all tags from containerd snaphsotter

options.Details also does the size calculation etc; perhaps we can always set the references, but continue to make the rest of details optional ;

references := i.referenceStore.References(img.ID().Digest())
if options.Details {
	 // get size etc
	img.Details = &image.Details{
		References:  references,
		Size:        size,
		Metadata:    layerMetadata,
		Driver:      i.layerStore.DriverName(),
		LastUpdated: lastUpdated,
	}
} else {
	img.Details = &image.Details{
		References:  references,
	}
}

^^ discussed this with @ndeloof, and for this use-case (docker image inspect) we already have options.Details enabled so there's currently no need to set Details.references if it's not set.