1. We were using gethash on chronometrist-events rather than
chronometrist-key-history 🤦♀
2. If an event has more than one key, determining the value of
old-keys in the outer loop meant that the first key would be
replaced by the second.
We also rename local variables so they tell us what objects they
represent, and reindent the code so it is more condensed.