I was wrong. It’s not fine since the properties that point to anything transient will be set to nil
after a RESTORE.
So since you have properties that refer to the cache, don’t make the cache transient. Unless you can manually restore those properties in port-restore or post-restart code.
But you should probably do a simple test to see if the amount of undo steps changes with a transient cache or not. If not (or if the difference is small) then I’d say it’s not worth the extra effort.