All CLA requirements met.

Cool!
What do you think about the following:
if if already matches, we could also suggest the then schema, otherwise the else and the if

I think that should already be what is happening, although I haven't validated it. I was more worried that I was breaking some invariants of SchemaCollector in my code, as there is not very much documentation

@Levertion works correct for me. Thanks for enhancement.

@Levertion The test is still failing...

Sorry, I've been busy with other stuff. My latest commit was to clear my working tree before I opened #30.
I should now have the time to look into finishing these tests, so I'll do that now.

OK - I've written enough tests to convince myself that the behaviour is correct, although I'm unsure about the formatting.

Should we format automatically somehow - clearly the style is to use ', but my personal habit is to use " so that is what I have used. I would expect there to be automatic formatting, but the builtin vscode formatting changes the style of the rest of the file, prettier is not being used and the tslint rules used does not control formatting.

My personal preference is prettier, but if the current formatting needs manually editing, let me know.

just use the built-in TypeScript formatter with default settings

I have tried that, using both format document and format selection after commenting out my entire settings.json and disabling all extensions for the workspace. This however led to only the minimal diff seen in 9f3ac83, which doesn't seem optimal. Is that enough?

Yes, that's fine. The formatter preserves new lines, that's why the changes are so small.

Well in that case I think this PR is finished, unless there are any other changes you want me to make

It's great, thanks a lot @Levertion !