isSuccess ? is this a behaviour change?  we should limit this PR to just docs (and maybe rename of CacheNode, since it's purely rename) rather than behaviour changes