I think its reasonable to enforce this nullability semantic on unary and binary nodes, but we should add something to their scaladoc.  Maybe also just make `SortOrder` not a UnaryNode.