`NodeValue` struct

[stoically]

Currently Node::value is just Option<Expr>. Introducing NodeValue as a wrapper around Expr would allow for a few enhancements, namely

• Implement TryInto instead of the currently used value_as_* methods on Node
• Get the span for just the value from NodeValue instead of from Node