was surprised to find that the index property of Repo created a new IndexFile object on each reference. This would have lead to code with terrible performance if I had not read the source and discovered this. At a minimum this needs a warning in docs.

I would suggest that you either make index stable until the user asked for it to be updated.
Or drop the property in favor of a function that returns a new IndexFile().