I got caught up in the name "cache". this is actually a memoization pattern, isn't it?  (and, probably not a big deal, and might have just made sense if I wasn't reviewing a diff)