Lcg/chef 12 attr

See https://gist.github.com/danielsdeleo/da9a8d9a91cc8c2781b0
I've mostly been moving them to the bottom, but in this case it reads better to keep the private helper together with the code that uses it.  It probably is an indicator that the complexity of the object is getting too high and has too many different concerns, but i'm not going to refactor the node attributes today...