Wouldn't it make sense to declare this as `lazy val` (or just val) so mapping is not done on every method invocation?