Yeah, ideally it should send a "snapshot" of the tx with that metadata. In theory if the pubsub gets backed up for whatever reason and a new tx/block that also affects one of the transactions that was emitted before, the event would have wrong data.
Since this refactor doesn't affect this behavior I think it's OK to merge and address it in a different PR in the future.
Originally posted by @jansegre in #1561 (comment)