Overall, looks good! I think caching the value makes sense, though, and would like to see that in.