Right now, EmittedEvents struct is defined to have optional transaction_index and event_index fields. That is not in line with spec, but was necessary to do because EmittedEvents fetched from forking origin are missing those fields. As soon as they become available, this should be fixed.