I updated the whitespace; let me know what you'd like to do instead for the changeType signature. The only reason I changed it was because I wanted to represent that both bodyReps *and* attachments changed in the same event and I wasn't sure how to do that cleanly without extra cruft. Maybe we can just check for the presence of updated attachments on the event instead?