Add builder classes, make them the only parameter to macro methods

[jakemac53]

This is still a WIP, but I wanted to get it out there for an initial review.

• Static analysis now is clean
• JsonCodable works, haven't tried the other ones
• I still need to get all the tests passing (I am sure many fail, haven't actually checked), will address this tomorrow
• All builder classes still have the same query API, which needs to be evaluated and hopefully restricted per phase in some way.
• The QueryClass macro was actually breaking the rules. It is no longer allowed to, so I changed a bit how it works.

[davidmorgan]

Looks good, thanks.

[jakemac53]

@davidmorgan I made some substantive changes to fix some of the tests that you should look at - lazy merged maps are now considered equal if their left/right maps are equal, and _keyof and getParents also were modified for lazy merged maps, which was necessary for qualifiedNameOf to function correctly in the case where a query is made by the macro.

[jakemac53]

To fix the remaining tests I had to change the expectations, the qualifiedNameOf function now technically works for cycles and re-used nodes (although the result could be surprising especially in the re-used nodes case).

I am not sure if this is a bad thing or a good thing... but these nodes shouldn't be re-used anyways.

[davidmorgan]

Thanks! Still LGTM.

Re: lists and cycles, both seem fine for now.