Improve getNodeProperty by using conditional types

Looks like this:

export type TsNodePropertyNames<NodeType extends ts.Node> = {
    [KeyName in keyof NodeType]:
        KeyName extends "_tsSimpleAstBrand" | CompilerApiNodeBrandPropertyNamesType ? never : KeyName;
}[keyof NodeType];
export type TsNodeToWrappedType<NodeType extends ts.Node, KeyName extends TsNodePropertyNames<NodeType>, NonNullableNodeType = NonNullable<NodeType[KeyName]>> =
    NodeType[KeyName] extends ts.NodeArray<infer ArrayNodeType> ? Node<ArrayNodeType>[] :
    NodeType[KeyName] extends ts.Node ? Node<NodeType[KeyName]> :
    NonNullableNodeType extends ts.Node ? Node<NonNullableNodeType> | undefined :
    NodeType[KeyName];

But unfortunately only compiles in typescript@next due to bugs so I’ll have to wait. Maybe it will work in the next 2.8.x version. In the meantime, I’ve checked this into the getNodeProperty branch.

[dsherret]

@lazarljubenovic it might be nice in some scenarios, but it’s good for people to prompted about something possibly being possibly null and then from there decide whether they want to handle it being null or do something else like throwing an exception.

[lazarljubenovic]

I’m personally using orThrow variants so much I’d almost prefer them being the regular ones and the others orNull. 😅